linux-kernel - Re: [PATCH 11/11] script/checkpatch.pl: warn about deprecated use of EXTRA

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 15 Aug 2011 01:10:38 -0400
From:	Arnaud Lacombe <lacombar@...il.com>
To:	linux-kernel@...r.kernel.org
Cc:	Arnaud Lacombe <lacombar@...il.com>,
	Sam Ravnborg <sam@...nborg.org>,
	Andy Whitcroft <apw@...onical.com>
Subject: Re: [PATCH 11/11] script/checkpatch.pl: warn about deprecated use of EXTRA_{A,C,CPP,LD}FLAGS

Hi,

[of course, I forgot to sign them :(]

On Mon, Aug 15, 2011 at 1:07 AM, Arnaud Lacombe <lacombar@...il.com> wrote:
> Usage of these flags has been deprecated for nearly 4 years by:
>
>    commit f77bf01425b11947eeb3b5b54685212c302741b8
>    Author: Sam Ravnborg <sam@...tun.(none)>
>    Date:   Mon Oct 15 22:25:06 2007 +0200
>
>        kbuild: introduce ccflags-y, asflags-y and ldflags-y
>
> Moreover, these flags (at least EXTRA_CFLAGS) have been documented for command
> line use. By default, gmake(1) do not override command line setting, so this is
> likely to result in build failure or unexpected behavior.
>
> Warn about their introduction in Makefile or Kbuild files.
>
> Cc: Sam Ravnborg <sam@...nborg.org>
> Cc: Andy Whitcroft <apw@...onical.com>
Signed-off-by: Arnaud Lacombe <lacombar@...il.com>

 - Arnaud

> ---
>  scripts/checkpatch.pl |   14 ++++++++++++++
>  1 files changed, 14 insertions(+), 0 deletions(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 9d761c9..955183a 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1661,6 +1661,20 @@ sub process {
>                        #print "is_end<$is_end> length<$length>\n";
>                }
>
> +               if (($realfile =~ /Makefile.*/ || $realfile =~ /Kbuild.*/) &&
> +                   ($line =~ /\+(EXTRA_[A-Z]+FLAGS).*/)) {
> +                       my $flag = $1;
> +                       my $replacement = {
> +                               'EXTRA_AFLAGS' =>   'asflags-y',
> +                               'EXTRA_CFLAGS' =>   'ccflags-y',
> +                               'EXTRA_CPPFLAGS' => 'cppflags-y',
> +                               'EXTRA_LDFLAGS' =>  'ldflags-y',
> +                       };
> +
> +                       WARN("DEPRECATED_VARIABLE",
> +                            "Use of $flag is deprecated, please use \`$replacement->{$flag} instead.\n" . $herecurr) if ($replacement->{$flag});
> +               }
> +
>  # check we are in a valid source file if not then ignore this hunk
>                next if ($realfile !~ /\.(h|c|s|S|pl|sh)$/);
>
> --
> 1.7.6.153.g78432
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/