Re: driver_remote option defaults to auto

On Sun, Jun 20, 2021 at 05:10:40PM +0800, Jiatong Shen wrote: Hi, I see how this may be confusing. The current code is correct and there is no need to change anything as this one way how it can be done with meson. If some option is 'feature' type and the default value is set to 'auto' this is what meson returns: get_option('opt').disabled() -> false get_option('opt').auto() -> true get_option('opt').enabled() -> false In the case of 'driver_remote' option we have the following check: if not get_option('driver_remote').disabled() which means that it will disable 'driver_remote' only if user calls meson with `-Ddriver_remote=disabled`. If you look closer to the following lines you will see that we also check for xdr_dep. If xdr_dep is missing and 'driver_remote' is set to 'auto' it will not be enabled as XDR is required, but if 'driver_remote' is set to 'enabled' but user calling meson with `-Ddriver_remote=enabled` meson will fail with proper error message. Before the changes done by the first commit you mentioned the 'driver_remote' was always enabled and if XDR was missing it always failed and user had to explicitly use `-Ddriver_remote=disabled` on systems where XDR is not available or if the user was not interested in having 'driver_remote' available. Pavel