On Thu, Jan 16, 2025 at 6:22 PM Bruce Richardson <bruce.richard...@intel.com> wrote: > diff --git a/drivers/meson.build b/drivers/meson.build > index 495e21b54a..89545e618e 100644 > --- a/drivers/meson.build > +++ b/drivers/meson.build > @@ -47,7 +47,7 @@ enable_drivers = run_command(list_dir_globs, > enable_drivers, check: true).stdout > require_drivers = true > if enable_drivers.length() == 0 > require_drivers = false > - enable_drivers = run_command(list_dir_globs, '*/*', check: > true).stdout().split() > + enable_drivers = run_command(list_dir_globs, '*/*,*/*/*', check: > true).stdout().split() > endif > > # these drivers must always be enabled, otherwise the build breaks > @@ -143,6 +143,10 @@ foreach subpath:subdirs > testpmd_sources = [] > require_iova_in_mbuf = true > > + if name.contains('/') > + name = name.split('/')[1] > + endif > + > if not enable_drivers.contains(drv_path) > build = false > reason = 'not in enabled drivers build config'
Nice trick. This makes this move transparent when it comes to library names, dependency object name, log prefix etc... We maintain consistency in git history, so check-git-log.sh may need some update to handle this special case (iow requiring, for example, net/i40e and not net/intel/i40e prefix for title of commits touching this driver). -- David Marchand