From: Masahiro Yamada <masahiroy@kernel.org>
To: Ariel Marcovitch <arielmarcovitch@gmail.com>
Cc: Michal Marek <michal.lkml@markovi.net>,
Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Valentin Rothberg <valentinrothberg@gmail.com>
Subject: Re: [PATCH 0/3] checkkconfigsymbols.py: Fix various bugs
Date: Tue, 24 Aug 2021 22:23:41 +0900 [thread overview]
Message-ID: <CAK7LNAQ6G11a-4_UZUtorBmmU6RzKeEkNjEzAWTpVv+LZR_=7A@mail.gmail.com> (raw)
In-Reply-To: <20210822192205.43210-1-arielmarcovitch@gmail.com>
On Mon, Aug 23, 2021 at 4:22 AM Ariel Marcovitch
<arielmarcovitch@gmail.com> wrote:
>
> Hi!
>
> This series fixes some issues in the checkkconfigsymbols.py script.
>
> The first patch fixes a bug in the --ignore option that makes the
> script check only the files matching the pattern instead of ignoring
> them.
>
> The second patch fixes a parsing error in the Kconfig files parser
> that makes it ignore 'if' statements after 'help' sections.
>
> The third patch prevents the user from using 'HEAD' refs in the
> --commit option, because it doesn't really work.
Honestly, I didn't even know this script.
I added Valentin Rothberg, the main contributor
to this script.
> Thanks!
>
> -Ariel
>
> Ariel Marcovitch (3):
> checkkconfigsymbols.py: Fix the '--ignore' option
> checkkconfigsymbols.py: Fix Kconfig parsing to find 'if' lines
> checkkconfigsymbols.py: Forbid passing 'HEAD' to --commit
>
> scripts/checkkconfigsymbols.py | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
>
> base-commit: 36a21d51725af2ce0700c6ebcb6b9594aac658a6
> --
> 2.25.1
>
--
Best Regards
Masahiro Yamada
prev parent reply other threads:[~2021-08-24 13:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-22 19:22 [PATCH 0/3] checkkconfigsymbols.py: Fix various bugs Ariel Marcovitch
2021-08-22 19:22 ` [PATCH 1/3] checkkconfigsymbols.py: Fix the '--ignore' option Ariel Marcovitch
2021-08-24 13:25 ` Masahiro Yamada
2021-08-22 19:22 ` [PATCH 2/3] checkkconfigsymbols.py: Fix Kconfig parsing to find 'if' lines Ariel Marcovitch
2021-08-24 13:30 ` Masahiro Yamada
2021-08-29 13:17 ` Ariel Marcovitch
2021-08-29 23:41 ` Masahiro Yamada
2021-09-01 15:17 ` Ariel Marcovitch
2021-08-22 19:22 ` [PATCH 3/3] checkkconfigsymbols.py: Forbid passing 'HEAD' to --commit Ariel Marcovitch
2021-08-24 13:31 ` Masahiro Yamada
2021-08-29 13:23 ` Ariel Marcovitch
2021-08-29 23:22 ` Masahiro Yamada
2021-08-24 13:23 ` Masahiro Yamada [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAK7LNAQ6G11a-4_UZUtorBmmU6RzKeEkNjEzAWTpVv+LZR_=7A@mail.gmail.com' \
--to=masahiroy@kernel.org \
--cc=arielmarcovitch@gmail.com \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=michal.lkml@markovi.net \
--cc=valentinrothberg@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).