Re: [Buildroot] [PATCH 6/7] docs/manual: add migration notes for changes in the kconfig syntax

[Petr Vorel <petr.vorel@gmail.com>]

Hi Titouan,

> Previous patches introduced some changes in the KConfig syntax,
> namely on variable expansion and choice entries. Reflect that in
> the migration guide.

> Assisted-by: Claude:claude-opus-5-5
> Signed-off-by: Titouan Christophe <titouan.christophe@mind.be>
> ---
>  docs/manual/migrating.adoc | 13 +++++++++++++
>  1 file changed, 13 insertions(+)

> diff --git a/docs/manual/migrating.adoc b/docs/manual/migrating.adoc
> index 86b040a19c..b2e0cb24b4 100644
> --- a/docs/manual/migrating.adoc
> +++ b/docs/manual/migrating.adoc
> @@ -270,3 +270,16 @@ changed, therefore the suffix of such tarballs has been changed from

>  Due to the update of Dovecot to version 2.4.x the runtime configuration
>  files need to be updated.
> +
> +Following the update of the bundled kconfig infrastructure, the syntax
> +of +Config.in+ files changed in a few ways, which affects **all
> +br2-external trees**:
> +
> +- variable names in +source+ statements and prompts must now be
> +  enclosed in parentheses. For example,
> +  +source "$BR2_EXTERNAL_FOO_PATH/bar"+ must now be rewritten as
> +  +source "$(BR2_EXTERNAL_FOO_PATH)/bar"+;
> +- +option env="VAR"+ is no longer supported. Environment variables
> +  are referenced directly as +$(VAR)+ instead;
> +- the prompt of a +choice+ must now be given with +prompt "..."+
> +  instead of +bool "..."+.

I wonder if \$(ARCH) should be here also mentioned.

Otherwise LGTM.
Reviewed-by: Petr Vorel <petr.vorel@gmail.com>

Kind regards,
Petr
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot