[EXT] Re: [PATCH 1/2] app/test: add support for optional dependencies

[David Marchand]

On Thu, Sep 28, 2023 at 3:06 PM Akhil Goyal <gakhil at marvell.com> wrote:
> > > Some tests make optionally use a component but don't require it for
> >
> > s/make //
> >
> > > building. If we include the dependency in the per-file lists, then tests
> > > may be unnecessarily omitted, as the dependency is not required.
> > > On the other hand, removing the optional dependency from the list can
> > > cause build failures, as the test case may include the optional code,
> > > but then fail to build as the build system won't have added the necessary
> > > paths for header inclusion, or the necessary libraries for linking.
> > >
> > > Resolve this by explicitly providing a list of optional dependencies.
> > > Any items in this list will be added to the dependency list if
> > > available, but otherwise won't be involved in enable/disable of specific
> > > tests.
> > >
> > > Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
> > Reviewed-by: David Marchand <david.marchand at redhat.com>
> Tested-by: Akhil Goyal <gakhil at marvell.com>

Series applied, thanks for the quick fix Bruce.


-- 
David Marchand