[Buildroot] [PATCH RFC] pkg-kconfig: support alternative .config location

Eric Le Bihan eric.le.bihan.dev at free.fr
Sun Sep 11 18:07:26 UTC 2016


Hi!

Le Sun, 11 Sep 2016 19:34:03 +0200,
"Yann E. MORIN" <yann.morin.1998 at free.fr> a écrit :

> On 2016-09-11 19:07 +0200, Eric Le Bihan spake thusly:
> > Kconfig clones, such as openconf used by xvisor [1], do not look
> > for .config at the root of the build directory, but in a
> > subdirectory (e.g. build/openconf).
> > 
> > This patch introduces a new Makefile variable named
> > $(2)_KCONFIG_BUILD_FILE, which defaults to $$($(2)_DIR)/.config and
> > can be overridden in the package Makefile.
> > 
> > This allows the use of the kconfig-package infrastructure with
> > packages relying on such clones.
> > 
> > [1] https://github.com/xvisor/xvisor/tree/master/tools/openconf
> > 
> > Signed-off-by: Eric Le Bihan <eric.le.bihan.dev at free.fr>
> > ---
> >  package/pkg-kconfig.mk | 14 ++++++++------
> >  1 file changed, 8 insertions(+), 6 deletions(-)
> > 
> > diff --git a/package/pkg-kconfig.mk b/package/pkg-kconfig.mk
> > index b0f5178..82561de 100644
> > --- a/package/pkg-kconfig.mk
> > +++ b/package/pkg-kconfig.mk
> > @@ -36,6 +36,7 @@ $(2)_KCONFIG_EDITORS ?= menuconfig
> >  $(2)_KCONFIG_OPTS ?=
> >  $(2)_KCONFIG_FIXUP_CMDS ?=
> >  $(2)_KCONFIG_FRAGMENT_FILES ?=
> > +$(2)_KCONFIG_BUILD_FILE ?= $$($(2)_DIR)/.config  
> 
> I think it would be better to name the variable
> $(2)_KCONFIG_DOTCONFIG_FILE or maybe just $(2)_KCONFIG_DOTCONFIG.

OK.

> Otherwise, the rest of the patch is a mere search-and-replace, so
> looks pretty OK.
> 
> However, I don't see the point in having that in Buildroot without an
> actuall package that uses that. Do you plan in sending such a package
> (like xvisor)?

Yes, I will send a patch to add a xvisor package shortly.

Thanks for the review.

-- 
ELB



More information about the buildroot mailing list