From: Simon Horman <horms@verge.net.au>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: Liu Song <liusong@linux.alibaba.com>,
bhelgaas@google.com, pablo@netfilter.org, davem@davemloft.net,
nathan@kernel.org, apw@canonical.com, joe@perches.com,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Kconfig: eliminate abnormal characters displayed under menuconfig
Date: Mon, 15 Aug 2022 18:18:57 +0100 [thread overview]
Message-ID: <YvqAATtWg1xLzyRs@vergenet.net> (raw)
In-Reply-To: <20220809025147.GA1264335@bhelgaas>
On Mon, Aug 08, 2022 at 09:51:47PM -0500, Bjorn Helgaas wrote:
> On Tue, Aug 02, 2022 at 06:12:33PM +0800, Liu Song wrote:
> > From: Liu Song <liusong@linux.alibaba.com>
> >
> > Use "find ./linux/* | grep Kconfig | xargs file | grep UTF", can find
> > files with utf-8 encoded characters, these files will display garbled
> > characters in menuconfig, except for characters with special meanings
> > that cannot be modified, modify the characters with obvious errors to
> > eliminate the wrong display under meunconfig.
> >
> > Signed-off-by: Liu Song <liusong@linux.alibaba.com>
> > Suggested-by: Bjorn Helgaas <helgaas@kernel.org>
>
> I did suggest fixing everything at once. But I think there are
> actually two problems here. The first is that some are just wrong,
> e.g., drivers/pci/Kconfig using CYRILLIC SMALL LETTER DZE instead of
> "s", or net/netfilter/ipvs/Kconfig using NO-BREAK SPACE for no good
> reason.
>
> arch/Kconfig looks like it's using RIGHT SINGLE QUOTATION MARK instead
> of the basic APOSTROPHE. That one is debatable; I think it's fair to
> argue that RIGHT SINGLE QUOTATION MARK would be preferred if rendered
> correctly.
>
> The second problem is that I think menuconfig doesn't handle UTF-8
> correctly. For example, in a gnome terminal with the default
> LC_ALL=en_US.UTF-8, these look fine:
>
> $ grep -A10 MTD_SPI_NOR_USE_4K_SECTORS drivers/mtd/spi-nor/Kconfig
> ...
> 64 KiB block instead of 16 × 4 KiB sectors.
>
> $ grep -A1 MTD_NAND_CAFE drivers/mtd/nand/raw/Kconfig
> config MTD_NAND_CAFE
> tristate "OLPC CAFÉ NAND controller"
>
> But when menuconfig in the same terminal displays that same text, it
> looks like "16 ~W 4 KiB sectors" and "OLPC CAF ~I NAND".
>
> On the other hand, gconfig and xconfig do render these correctly. So
> I think there's something wrong with UTF-8 in menuconfig, and we
> should fix that as well.
>
> Anyway, I'm fine with this patch as far as it goes:
>
> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
IPVS portion:
Acked-by: Simon Horman <horms@verge.net.au>
next prev parent reply other threads:[~2022-08-15 17:19 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-02 10:12 [PATCH] Kconfig: eliminate abnormal characters displayed under menuconfig Liu Song
2022-08-09 2:51 ` Bjorn Helgaas
2022-08-15 17:18 ` Simon Horman [this message]
2023-10-06 20:29 ` Bjorn Helgaas
2023-11-09 3:57 ` Randy Dunlap
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=YvqAATtWg1xLzyRs@vergenet.net \
--to=horms@verge.net.au \
--cc=apw@canonical.com \
--cc=bhelgaas@google.com \
--cc=davem@davemloft.net \
--cc=helgaas@kernel.org \
--cc=joe@perches.com \
--cc=linux-kernel@vger.kernel.org \
--cc=liusong@linux.alibaba.com \
--cc=nathan@kernel.org \
--cc=pablo@netfilter.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.