> From: Morten Brørup [mailto:m...@smartsharesystems.com] > Sent: Tuesday, 1 August 2023 12.49 > > > 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):
Sorry... This was David's suggestion, which I prefer. I thought I was replying to Bruce's reply. > > Acked-by: Morten Brørup <m...@smartsharesystems.com> My ACK still valid.
