Re: [PATCH] kconfig: fix localmodconfig

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2018-06-09 6:53 GMT+09:00 Sam Ravnborg <sam@xxxxxxxxxxxx>:
> From 47857284fd19ef953e3beb7338154924a74b62d2 Mon Sep 17 00:00:00 2001
> From: Sam Ravnborg <sam@xxxxxxxxxxxx>
> Date: Fri, 8 Jun 2018 23:48:31 +0200
> Subject: [PATCH 1/1] kconfig: fix localmodconfig
>
> When kconfig syntax moved to use $(FOO) for environment variables
> localmodconfig was not updated.
> Fix so it now works with the new syntax $(FOO)
>
> Fixes: 104daea149c45cc84842ce77a9bd6436d19f3dd8 ("kconfig: reference environment variables directly and remove 'option env='")
> Reported-by: Kevin Locke <kevin@xxxxxxxxxxxxxxx>
> Reported-by: Andrei Vagin <avagin@xxxxxxxxxxxxx>
> Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx>
> ---


Applied to linux-kbuild.  Thanks!


>  scripts/kconfig/streamline_config.pl | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/kconfig/streamline_config.pl b/scripts/kconfig/streamline_config.pl
> index a2e83ab17de3..4686531e2f8c 100755
> --- a/scripts/kconfig/streamline_config.pl
> +++ b/scripts/kconfig/streamline_config.pl
> @@ -165,10 +165,10 @@ sub read_kconfig {
>      my $last_source = "";
>
>      # Check for any environment variables used
> -    while ($source =~ /\$(\w+)/ && $last_source ne $source) {
> +    while ($source =~ /\$\((\w+)\)/ && $last_source ne $source) {
>         my $env = $1;
>         $last_source = $source;
> -       $source =~ s/\$$env/$ENV{$env}/;
> +       $source =~ s/\$\($env\)/$ENV{$env}/;
>      }
>
>      open(my $kinfile, '<', $source) || die "Can't open $kconfig";
> --
> 2.12.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Best Regards
Masahiro Yamada
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux