[Buildroot] [PATCH 1/2] packages infra: add function to get a Kconfig option

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998 at free.fr> writes:

 > From: "Yann E. MORIN" <yann.morin.1998 at free.fr>
 > We so far have no mean to get the value from a Kconfig option from the
 > .config file of a package (eg. linux, busybox...).

 > Add a new function that returns the unmangled value of an option.
 > It expect two arguments:
 >   - the Kconfig option name (complete, with leading CONFIG if necessary)
 >   - the .config file to get it from

 > Note that, if the Kconfig option is a string, the returned value will
 > contain the leading and trailing double-quotes.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
 > ---
 >  package/pkg-utils.mk | 6 ++++++
 >  1 file changed, 6 insertions(+)

 > diff --git a/package/pkg-utils.mk b/package/pkg-utils.mk
 > index 851575c..2f70acc 100644
 > --- a/package/pkg-utils.mk
 > +++ b/package/pkg-utils.mk
 > @@ -52,6 +52,12 @@ define KCONFIG_DISABLE_OPT
 >  	echo "# $(1) is not set" >> $(2)
 >  endef
 
 > +# Note: we do not indent this, since we want to avoid any leading
 > +# space or tabs when calling this function
 > +define KCONFIG_GET_OPT
 > +$(shell sed -e "/\\<$(1)\\>=\\(.*\\)$$/!d; s//\\1/" $(2))

Should this perhaps use $(SED)?

Sorry, I'm probably missing something, but I don't right away see why we
don't just use:

$(SED) -n 's/^$(1)=//p' $(2)

-- 
Bye, Peter Korsgaard