> From: David Marchand [mailto:david.march...@redhat.com] > Sent: Tuesday, 1 August 2023 12.21 > > On Tue, Aug 1, 2023 at 11:25 AM Bruce Richardson > <bruce.richard...@intel.com> wrote: > > > > On Tue, Aug 01, 2023 at 10:52:53AM +0200, David Marchand wrote: > > > Drivers implementing a class of devices (for example, drivers/event) > > > depend on the associated abstraction library (lib/eventdev). > > > This dependency is expressed in the top level meson.build for this class > > > (drivers/event/meson.build). > > > > > > As we are making more libraries optional, custom constructs referencing > > > the class dependencies in some drivers meson.build (event/dlb2) may break. > > > > > > It would be possible to add more checks in those drivers meson.build but > > > it is more straightforward to not even consider a driver meson.build when > > > the class dependencies are not met. > > > > > > Signed-off-by: David Marchand <david.march...@redhat.com> > > > --- > > This gives the following output (testing on top of your series > extending optional libs and a fix on the new reasm perf test). > https://github.com/david- > marchand/dpdk/actions/runs/5725429526/job/15513923381#step:19:346 > > ... > ================= > Content Skipped > ================= > ... > drivers: > ... > mempool/dpaa2: not in enabled drivers build config > mempool/octeontx: not in enabled drivers build config > mempool/stack: not in enabled drivers build config > dma/*: missing internal dependency, "dmadev" > net/af_packet: not in enabled drivers build config > net/af_xdp: not in enabled drivers build config > net/ark: not in enabled drivers build config > ... >
For the concept suggested by David (and preferably with Bruce's output): Acked-by: Morten Brørup <m...@smartsharesystems.com>
