Re: [PATCH 3/3] build: support disabling drivers with meson

[Thomas Monjalon <thomas@monjalon.net>]

06/01/2022 09:23, Morten Brørup:
> > From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> > Sent: Wednesday, 25 September 2019 16.56
> > 
> > Add support for a new build option to turn off certain drivers. Any
> > other
> > drivers which depend on the one being disabled will also be disabled
> > with a
> > suitable debug message.
> > 
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> 
> Hi Bruce,
> 
> I was trying something along the lines of:
> 
> meson configure -Ddisable_drivers=net/* -Denable_drivers=net/i40e
> 
> But it seems that enable_drivers does not override disable_drivers. Ninja says:

It is the opposite logic. "disable" overrides "enable":

        if not enable_drivers.contains(drv_path)
            build = false
            reason = 'not in enabled drivers build config'
        elif disable_drivers.contains(drv_path)
            if always_enable.contains(drv_path)
                message('Driver @0@ cannot be disabled, not disabling.'.format(drv_path))
            else
                build = false
                reason = 'explicitly disabled via build config'
            endif
        endif

In this case, you don't need to disable everything because of this:

if enable_drivers.length() == 0
    enable_drivers = run_command(list_dir_globs, '*/*', check: true).stdout().split()
endif