Re: [PATCH] checkkconfigsymbols.py: filter reports for tools/
From: Valentin Rothberg
Date: Tue Mar 03 2015 - 02:55:53 EST
Hi Paul,
thanks for your answer.
On Tue, Mar 3, 2015 at 1:57 AM, Paul Bolle <pebolle@xxxxxxxxxx> wrote:
> On Wed, 2015-02-25 at 15:15 +0100, Valentin Rothberg wrote:
>> @@ -46,8 +46,9 @@ def main():
>> stdout = stdout[:-1]
>>
>> for gitfile in stdout.rsplit("\n"):
>> - if ".git" in gitfile or "ChangeLog" in gitfile or \
>> - ".log" in gitfile or os.path.isdir(gitfile):
>> + if ".git" in gitfile or "ChangeLog" in gitfile or \
>> + ".log" in gitfile or os.path.isdir(gitfile) or \
>> + gitfile.startswith("tools/"):
>
> Perhaps just
> gitfile == "tools/perf/config/Makefile"
>
> (but I'm unsure if that's valid python)?
>
>> continue
>> if REGEX_FILE_KCONFIG.match(gitfile):
>> kconfig_files.append(gitfile)
>
> This patch was triggered by perf changes that hit next-20150225, wasn't
> it? If so, we might want to find out why the perf people need to use
Yes, it was in next-20150225. However, more recent changes have the
same problem. I fear it get's worse for us : )
> their
> "$(call detected,CONFIG_EXAMPLE)"
>
> hack. Especially because that hack is also used on existing Kconfig
> symbols (I spotted X86, X86_64, AUDIT, and NUMA). And the usage of both
> valid Kconfig macros and faux Kconfig macros in that hack looks odd to
> me.
AFAIU it's independent from Kconfig / Kbuild. The usage of Kconfig
symbols seems completely random to me.
Ignoring tools entirely also seems a little too much, since some tools
are still Kconfig sensitive. Hence, I vote to ignore only perf:
+ gitfile.startswith("tools/perf"):
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/