On Tue, Sep 28, 2021 at 03:34:00PM +0100, Anthony PERARD wrote: > On Tue, Sep 28, 2021 at 03:46:01PM +0200, Jan Beulich wrote: > > On 28.09.2021 10:39, Anthony PERARD wrote: > > > This will help prevent the CI loop from having build failures when > > > `checkpolicy` isn't available when doing "randconfig" jobs. > > > > > > To prevent "randconfig" from selecting XSM_FLASK_POLICY when > > > `checkpolicy` isn't available, we will actually override the config > > > output with the use of KCONFIG_ALLCONFIG. > > > > > > Doing this way still allow a user/developer to set XSM_FLASK_POLICY > > > even when "checkpolicy" isn't available. It also prevent the build > > > system from reset the config when "checkpolicy" isn't available > > > anymore. And XSM_FLASK_POLICY is still selected automatically when > > > `checkpolicy` is available. > > > But this also work well for "randconfig", as it will not select > > > XSM_FLASK_POLICY when "checkpolicy" is missing. > > > > > > This patch allows to easily add more override which depends on the > > > environment. > > > > > > Also, move the check out of Config.mk and into xen/ build system. > > > Nothing in tools/ is using that information as it's done by > > > ./configure. > > > > > > We named the new file ".allconfig.tmp" as ".*.tmp" are already ignored > > > via .gitignore. > > > > > > Remove '= y' in Kconfig as it isn't needed, only a value "y" is true, > > > anything else is considered false. > > > > Seeing you say this explicitly makes me wonder - is this actually true? > > I've check that this was true by empirical testing before sending the > patch. But the documentation isn't clear to me about the meaning of > 'default y if "m"'. So would you rather keep '= y' just to stay on the > safe side?
I've sent v5 with this change to the Kconfig file removed. -- Anthony PERARD
