[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 1/5] checkpatch: add a check for utf-8 in co
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC PATCH 1/5] checkpatch: add a check for utf-8 in commit logs |
Date: |
Wed, 1 Feb 2017 16:27:15 +0000 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Mon, Jan 30, 2017 at 04:57:53PM +0100, Thomas Huth wrote:
> On 30.01.2017 15:12, Stefan Hajnoczi wrote:
> > On Thu, Jan 26, 2017 at 02:11:01PM +0100, Thomas Huth wrote:
> >> This patch is a port of the following commit from the Linux kernel:
> >>
> >> commit 15662b3e8644905032c2e26808401a487d4e90c1
> >> Author: Joe Perches <address@hidden>
> >> Date: Mon Oct 31 17:13:12 2011 -0700
> >>
> >> checkpatch: add a --strict check for utf-8 in commit logs
> >>
> >> Some find using utf-8 in commit logs inappropriate.
> >>
> >> Some patch commit logs contain unintended utf-8 characters when doing
> >> things like copy/pasting compilation output.
> >>
> >> Look for the start of any commit log by skipping initial lines that
> >> look
> >> like email headers and "From: " lines.
> >>
> >> Stop looking for utf-8 at the first signature line.
> >>
> >> Signed-off-by: Joe Perches <address@hidden>
> >> Suggested-by: Andrew Morton <address@hidden>
> >> Cc: Andy Whitcroft <address@hidden>
> >> Signed-off-by: Andrew Morton <address@hidden>
> >> Signed-off-by: Linus Torvalds <address@hidden>
> >>
> >> Signed-off-by: Thomas Huth <address@hidden>
> >> ---
> >> scripts/checkpatch.pl | 30 ++++++++++++++++++++++++++----
> >> 1 file changed, 26 insertions(+), 4 deletions(-)
> >
> > This patch prevents including names with non-ASCII characters in the
> > commit description. Some people care about the proper spelling of their
> > names.
> >
> > Allowing UTF-8 in Signed-off-by and other headers isn't enough.
>
> Right, and I guess the folks from the kernel checkpatch noticed this,
> too. That's likely why the next patch restricts this check to only
> happen if the patch is from a mail with non-UTF-8 content type, but
> still contains UTF-8 characters, i.e. there is really something fishy
> with the character set of the patch description. Maybe I should squash
> the two patches together, so that it is more obvious what is going on here?
In that case I'm okay with this patch :). There's no need to squash it.
Sorry, I didn't make the connection with the next patch. I thought that
was simply for checking that the byte stream is valid according to the
claimed charset.
Stefan
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [RFC PATCH 1/5] checkpatch: add a check for utf-8 in commit logs,
Stefan Hajnoczi <=