* [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
@ 2005-05-03 0:34 Adrian Bunk
2005-05-03 9:10 ` Roman Zippel
0 siblings, 1 reply; 8+ messages in thread
From: Adrian Bunk @ 2005-05-03 0:34 UTC (permalink / raw)
To: Andrew Morton, zippel; +Cc: kbuild-devel, linux-kernel, Jesper Juhl
[-- Attachment #1: Type: text/plain, Size: 6693 bytes --]
This patch is the majority of a patch by Jesper Juhl.
This patch renames all instances of "---help---" to simply "help" in all
of the Kconfig files.
The main reason for this patch (quoting Jesper) is:
Consistency. out of ~4000 help entries in 134 Kconfig files, 747 of
those entries use "---help---" as the keyword, the rest use just "help".
So the users of "---help---" are clearly a minority and by renaming them
we make things consistent. - I hate inconsistency. :-)
My change to the patch was to drop all parts that don't apply to both
2.6.12-rc3 and 2.6.12-rc3-mm2 (for making inclusion in Linus' tree
easier).
My plan for this issue is:
- get this patch accepted
- rename all remaining "---help---" in both -mm and Linus' tree
- remove the "---help---" from the Kconfig language
---
This patch is attached gzip'ed because it's > 200 kB.
arch/alpha/Kconfig | 12 +-
arch/arm/Kconfig | 10 +-
arch/arm/mach-clps711x/Kconfig | 2
arch/arm/mach-footbridge/Kconfig | 2
arch/arm26/Kconfig | 2
arch/cris/arch-v10/drivers/Kconfig | 4
arch/h8300/Kconfig | 6 -
arch/i386/Kconfig | 26 +++---
arch/i386/kernel/cpu/cpufreq/Kconfig | 2
arch/ia64/Kconfig | 2
arch/m32r/Kconfig | 4
arch/m68k/Kconfig | 18 ++--
arch/m68knommu/Kconfig | 8 -
arch/mips/Kconfig | 12 +-
arch/parisc/Kconfig | 4
arch/ppc/Kconfig | 26 +++---
arch/ppc64/Kconfig | 6 -
arch/s390/Kconfig | 4
arch/sh/Kconfig | 8 -
arch/sh/cchips/Kconfig | 4
arch/sparc/Kconfig | 14 +--
arch/sparc64/Kconfig | 18 ++--
arch/um/Kconfig | 2
arch/x86_64/Kconfig | 8 -
drivers/acpi/Kconfig | 12 +-
drivers/atm/Kconfig | 16 +--
drivers/base/Kconfig | 2
drivers/block/Kconfig | 22 ++---
drivers/block/paride/Kconfig | 8 -
drivers/cdrom/Kconfig | 20 ++--
drivers/char/Kconfig | 50 +++++------
drivers/char/agp/Kconfig | 6 -
drivers/char/ftape/Kconfig | 18 ++--
drivers/char/tpm/Kconfig | 6 -
drivers/char/watchdog/Kconfig | 32 +++----
drivers/eisa/Kconfig | 8 -
drivers/fc4/Kconfig | 4
drivers/i2c/Kconfig | 2
drivers/ide/Kconfig | 36 ++++----
drivers/infiniband/Kconfig | 2
drivers/infiniband/hw/mthca/Kconfig | 4
drivers/infiniband/ulp/ipoib/Kconfig | 6 -
drivers/input/Kconfig | 12 +-
drivers/input/gameport/Kconfig | 2
drivers/input/joystick/Kconfig | 6 -
drivers/input/mouse/Kconfig | 4
drivers/input/serio/Kconfig | 12 +-
drivers/isdn/Kconfig | 4
drivers/isdn/hisax/Kconfig | 2
drivers/macintosh/Kconfig | 2
drivers/md/Kconfig | 28 +++---
drivers/media/Kconfig | 2
drivers/media/dvb/Kconfig | 2
drivers/media/radio/Kconfig | 30 +++----
drivers/media/video/Kconfig | 26 +++---
drivers/message/i2o/Kconfig | 2
drivers/misc/Kconfig | 2
drivers/mtd/Kconfig | 16 +--
drivers/mtd/chips/Kconfig | 2
drivers/mtd/devices/Kconfig | 10 +-
drivers/mtd/nand/Kconfig | 2
drivers/net/Kconfig | 114 +++++++++++++--------------
drivers/net/appletalk/Kconfig | 2
drivers/net/arcnet/Kconfig | 8 -
drivers/net/hamradio/Kconfig | 16 +--
drivers/net/irda/Kconfig | 8 -
drivers/net/pcmcia/Kconfig | 4
drivers/net/tokenring/Kconfig | 12 +-
drivers/net/tulip/Kconfig | 14 +--
drivers/net/wan/Kconfig | 26 +++---
drivers/net/wireless/Kconfig | 28 +++---
drivers/parport/Kconfig | 4
drivers/pci/Kconfig | 4
drivers/pci/hotplug/Kconfig | 2
drivers/pcmcia/Kconfig | 8 -
drivers/pnp/Kconfig | 2
drivers/pnp/pnpacpi/Kconfig | 2
drivers/pnp/pnpbios/Kconfig | 4
drivers/s390/Kconfig | 2
drivers/scsi/Kconfig | 90 ++++++++++-----------
drivers/scsi/qla2xxx/Kconfig | 10 +-
drivers/serial/Kconfig | 16 +--
drivers/telephony/Kconfig | 4
drivers/usb/Kconfig | 4
drivers/usb/class/Kconfig | 6 -
drivers/usb/core/Kconfig | 2
drivers/usb/host/Kconfig | 14 +--
drivers/usb/image/Kconfig | 2
drivers/usb/input/Kconfig | 16 +--
drivers/usb/media/Kconfig | 22 ++---
drivers/usb/misc/Kconfig | 4
drivers/usb/misc/sisusbvga/Kconfig | 2
drivers/usb/net/Kconfig | 10 +-
drivers/usb/serial/Kconfig | 20 ++--
drivers/usb/storage/Kconfig | 4
drivers/video/Kconfig | 50 +++++------
drivers/video/console/Kconfig | 4
drivers/video/geode/Kconfig | 4
drivers/w1/Kconfig | 2
drivers/zorro/Kconfig | 2
fs/Kconfig | 4
fs/ncpfs/Kconfig | 2
fs/nls/Kconfig | 10 +-
fs/partitions/Kconfig | 6 -
init/Kconfig | 12 +-
kernel/power/Kconfig | 6 -
net/Kconfig | 8 -
net/ax25/Kconfig | 6 -
net/decnet/Kconfig | 2
net/ipv4/Kconfig | 24 ++---
net/ipv4/ipvs/Kconfig | 36 ++++----
net/ipv4/netfilter/Kconfig | 12 +-
net/ipv6/Kconfig | 12 +-
net/ipv6/netfilter/Kconfig | 2
net/ipx/Kconfig | 2
net/irda/Kconfig | 4
net/sched/Kconfig | 48 +++++------
net/sctp/Kconfig | 2
net/xfrm/Kconfig | 2
sound/oss/Kconfig | 16 +--
120 files changed, 694 insertions(+), 694 deletions(-)
[-- Attachment #2: patch-Kconfig-help.gz --]
[-- Type: application/octet-stream, Size: 60288 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
2005-05-03 0:34 [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part) Adrian Bunk
@ 2005-05-03 9:10 ` Roman Zippel
2005-05-03 9:22 ` Adrian Bunk
0 siblings, 1 reply; 8+ messages in thread
From: Roman Zippel @ 2005-05-03 9:10 UTC (permalink / raw)
To: Adrian Bunk; +Cc: Andrew Morton, kbuild-devel, linux-kernel, Jesper Juhl
Hi,
On Tue, 3 May 2005, Adrian Bunk wrote:
> This patch is the majority of a patch by Jesper Juhl.
>
> This patch renames all instances of "---help---" to simply "help" in all
> of the Kconfig files.
>
> The main reason for this patch (quoting Jesper) is:
>
> Consistency. out of ~4000 help entries in 134 Kconfig files, 747 of
> those entries use "---help---" as the keyword, the rest use just "help".
> So the users of "---help---" are clearly a minority and by renaming them
> we make things consistent. - I hate inconsistency. :-)
This has nothing to do with consistency but with readability.
This was introduced to better separate the help in large menu entries. In
order to accept this patch, I would either like hear reasons, why this
isn't needed anymore or I'd like to see an alternative, more consistent
separator.
bye, Roman
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
2005-05-03 9:10 ` Roman Zippel
@ 2005-05-03 9:22 ` Adrian Bunk
2005-05-03 10:09 ` Roman Zippel
0 siblings, 1 reply; 8+ messages in thread
From: Adrian Bunk @ 2005-05-03 9:22 UTC (permalink / raw)
To: Roman Zippel; +Cc: Andrew Morton, kbuild-devel, linux-kernel, Jesper Juhl
On Tue, May 03, 2005 at 11:10:48AM +0200, Roman Zippel wrote:
> Hi,
>
> On Tue, 3 May 2005, Adrian Bunk wrote:
>
> > This patch is the majority of a patch by Jesper Juhl.
> >
> > This patch renames all instances of "---help---" to simply "help" in all
> > of the Kconfig files.
> >
> > The main reason for this patch (quoting Jesper) is:
> >
> > Consistency. out of ~4000 help entries in 134 Kconfig files, 747 of
> > those entries use "---help---" as the keyword, the rest use just "help".
> > So the users of "---help---" are clearly a minority and by renaming them
> > we make things consistent. - I hate inconsistency. :-)
>
> This has nothing to do with consistency but with readability.
> This was introduced to better separate the help in large menu entries. In
> order to accept this patch, I would either like hear reasons, why this
> isn't needed anymore or I'd like to see an alternative, more consistent
> separator.
The separator used for the help is to indent help texts by two
additional spaces.
IMHO, Kconfig files are quite readable due to this indentation even
though only a minority of the entries was using "---help---" even
before this patch.
> bye, Roman
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
2005-05-03 9:22 ` Adrian Bunk
@ 2005-05-03 10:09 ` Roman Zippel
2005-05-03 10:57 ` Adrian Bunk
0 siblings, 1 reply; 8+ messages in thread
From: Roman Zippel @ 2005-05-03 10:09 UTC (permalink / raw)
To: Adrian Bunk; +Cc: Andrew Morton, kbuild-devel, linux-kernel, Jesper Juhl
Hi,
On Tue, 3 May 2005, Adrian Bunk wrote:
> The separator used for the help is to indent help texts by two
> additional spaces.
Yes, that's an additional indicator.
> IMHO, Kconfig files are quite readable due to this indentation even
> though only a minority of the entries was using "---help---" even
> before this patch.
So why exactly has to be removed? Is it ugly? Does it make Kconfig worse?
Sorry, but only because it's not used that often, is not enough of a
reason for me to remove it. If it helps only a little bit to spot the help
text start easier, it's IMO worth to keep it.
bye, Roman
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
2005-05-03 10:09 ` Roman Zippel
@ 2005-05-03 10:57 ` Adrian Bunk
2005-05-03 13:47 ` Roman Zippel
0 siblings, 1 reply; 8+ messages in thread
From: Adrian Bunk @ 2005-05-03 10:57 UTC (permalink / raw)
To: Roman Zippel; +Cc: Andrew Morton, kbuild-devel, linux-kernel, Jesper Juhl
On Tue, May 03, 2005 at 12:09:37PM +0200, Roman Zippel wrote:
> Hi,
>
> On Tue, 3 May 2005, Adrian Bunk wrote:
>
> > The separator used for the help is to indent help texts by two
> > additional spaces.
>
> Yes, that's an additional indicator.
>
> > IMHO, Kconfig files are quite readable due to this indentation even
> > though only a minority of the entries was using "---help---" even
> > before this patch.
>
> So why exactly has to be removed? Is it ugly? Does it make Kconfig worse?
The ugly thing is that there are currently two different ways to express
the same thing. It only causes confusion for people who think those
different syntaxes had a different meaning.
> Sorry, but only because it's not used that often, is not enough of a
> reason for me to remove it. If it helps only a little bit to spot the help
> text start easier, it's IMO worth to keep it.
Do you or does anyone else have a problem with spotting the help text
start with "only" two additional spaces indentation?
If it's only for Aunt Tillie it's not required.
> bye, Roman
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
2005-05-03 10:57 ` Adrian Bunk
@ 2005-05-03 13:47 ` Roman Zippel
2005-05-04 11:50 ` Adrian Bunk
0 siblings, 1 reply; 8+ messages in thread
From: Roman Zippel @ 2005-05-03 13:47 UTC (permalink / raw)
To: Adrian Bunk; +Cc: Andrew Morton, kbuild-devel, linux-kernel, Jesper Juhl
Hi,
On Tue, 3 May 2005, Adrian Bunk wrote:
> > So why exactly has to be removed? Is it ugly? Does it make Kconfig worse?
>
> The ugly thing is that there are currently two different ways to express
> the same thing. It only causes confusion for people who think those
> different syntaxes had a different meaning.
Languages often have more than one way to express something, this is not
different.
Early on there was some confusion about this from people writing new
Kconfig entries (not just reading existing ones), but this was became a
non-issue since it's documented now.
> > Sorry, but only because it's not used that often, is not enough of a
> > reason for me to remove it. If it helps only a little bit to spot the help
> > text start easier, it's IMO worth to keep it.
>
> Do you or does anyone else have a problem with spotting the help text
> start with "only" two additional spaces indentation?
It is easier to spot! Is it a big help? I don't know, but I don't see any
improvement in removing them.
Personally I'd rather leave it to the maintainers of these entries, if
they don't like it, they can remove it. If this feature is one day not
used anymore, I'll remove it.
> If it's only for Aunt Tillie it's not required.
Please stop this. There is no point in arguing whether this is good or bad
for Aunt Tillie.
bye, Roman
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
2005-05-03 13:47 ` Roman Zippel
@ 2005-05-04 11:50 ` Adrian Bunk
0 siblings, 0 replies; 8+ messages in thread
From: Adrian Bunk @ 2005-05-04 11:50 UTC (permalink / raw)
To: Roman Zippel; +Cc: Andrew Morton, kbuild-devel, linux-kernel, Jesper Juhl
On Tue, May 03, 2005 at 03:47:46PM +0200, Roman Zippel wrote:
> Hi,
Hi Roman,
> On Tue, 3 May 2005, Adrian Bunk wrote:
>
> > > So why exactly has to be removed? Is it ugly? Does it make Kconfig worse?
> >
> > The ugly thing is that there are currently two different ways to express
> > the same thing. It only causes confusion for people who think those
> > different syntaxes had a different meaning.
>
> Languages often have more than one way to express something, this is not
> different.
> Early on there was some confusion about this from people writing new
> Kconfig entries (not just reading existing ones), but this was became a
> non-issue since it's documented now.
>...
Different ways for expressing the same thing is good for writers but bad
for readers.
E.g. where to place opening braces in C is a religious issue. For the
Linux kernel, CodingStyle sets the rules. And although I'd prefer to set
them different, I have to admit that the consistency indide the kernel
makes reading easier.
> bye, Roman
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <3ZTLT-3D3-1@gated-at.bofh.it>]
end of thread, other threads:[~2005-05-04 11:50 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-05-03 0:34 [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part) Adrian Bunk
2005-05-03 9:10 ` Roman Zippel
2005-05-03 9:22 ` Adrian Bunk
2005-05-03 10:09 ` Roman Zippel
2005-05-03 10:57 ` Adrian Bunk
2005-05-03 13:47 ` Roman Zippel
2005-05-04 11:50 ` Adrian Bunk
[not found] <3ZTLT-3D3-1@gated-at.bofh.it>
[not found] ` <401Js-1AH-1@gated-at.bofh.it>
[not found] ` <401Tf-1Hp-9@gated-at.bofh.it>
[not found] ` <402FB-2mN-15@gated-at.bofh.it>
2005-05-03 13:56 ` Bodo Eggert <harvested.in.lkml@posting.7eggert.dyndns.org>
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox