On Thu, Sep 28, 2023 at 3:06 PM Akhil Goyal <[email protected]> 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 <[email protected]> > > Reviewed-by: David Marchand <[email protected]> > Tested-by: Akhil Goyal <[email protected]>
Series applied, thanks for the quick fix Bruce. -- David Marchand
