mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Trent Piepho <xyzzy@speakeasy.org>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: Sam Ravnborg <sam@ravnborg.org>,
	Jan Engelhardt <jengelh@linux01.gwdg.de>,
	Mauro Carvalho Chehab <mchehab@infradead.org>,
	v4l-dvb maintainer list <v4l-dvb-maintainer@linuxtv.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [v4l-dvb-maintainer] [PATCH 35/36] Use menuconfig objects II - DVB
Date: Thu, 3 May 2007 18:40:58 -0700 (PDT)	[thread overview]
Message-ID: <Pine.LNX.4.58.0705031826360.32689@shell2.speakeasy.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0705040212080.1479@scrub.home>

On Fri, 4 May 2007, Roman Zippel wrote:
> I don't quite understand. With the menuconfig changes more menu entries
> should  appear on the left side, so I don't understand why you have to
> "drill down" to reach it.
> The rule for menu to appear on the left side is relatively simple - all
> its parents must be of menu type as well. So if a menuconfig is on the
> right side it must have a normal config entry as parent.

I think that's it.  The media tree was done with options to select the core
system module, then a menuconfig that depended on that which the drivers
were under.

> > > menuconfig XXXX
> > > if XXXX
> > > [....all the other options....]
> > > endif
> > >
> > > Into this:
> > >
> > > menuconfig XXXX
> > > [....all the other options....]
> > > endmenu
> > >
> > > The reason is that a frontend would easily be able to understand the coupling
> > > between the "menuconfig XXXX" and the "if XXXX".  It will make it easier for
> > > the frontend to see that all the options are inside and controlled by the
> > > enclosing menuconfig.
>
> If the frontend wants to change the behaviour of a menuconfig, it can
> already do that, so this doesn't require a syntax change.

How about these examples:

menuconfig FOO
if FOO
config A
	depends on FOO
endif
config B
if FOO
config C
	depends on FOO
endif

Or this:
menu FOO
menuconfig BAR
config A
menuconfig BAZ
config B
endmenu

How does it show the first one, keeping the config entries in the correct
order and put them into the menu at the same time?

And which of what should the second be show?

foo
\-bar
  \-baz

or

foo
|-bar
\-baz

There is no question with menus, as the menu tree is clearly lexically
defined by the matching menu / endmenu pairs.  But menuconfig doesn't work
that way, and it seems like it would make more sense if it did.

  reply	other threads:[~2007-05-04  1:41 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-30 11:03 [PATCH 00/36] Use menuconfig objects II - (overview) Jan Engelhardt
2007-04-30 11:20 ` [PATCH 01/36] Use menuconfig objects I.b - block Jan Engelhardt
2007-04-30 11:20 ` [PATCH 02/36] Use menuconfig objects II - auxdisplay Jan Engelhardt
2007-04-30 11:21 ` [PATCH 03/36] Use menuconfig objects II - battery Jan Engelhardt
2007-04-30 11:22 ` [PATCH 04/36] Use menuconfig objects II - block devices Jan Engelhardt
2007-04-30 11:24 ` [PATCH 05/36] Use menuconfig objects II - EDAC Jan Engelhardt
2007-04-30 11:26 ` [PATCH 06/36] Use menuconfig objects II - fusion Jan Engelhardt
2007-04-30 11:27 ` [PATCH 07/36] Use menuconfig objects II - HID Jan Engelhardt
2007-04-30 11:28 ` [PATCH 08/36] Use menuconfig objects II - hwmon Jan Engelhardt
2007-05-03 16:58   ` Jean Delvare
2007-04-30 11:29 ` [PATCH 09/36] Use menuconfig objects II - IEEE1394 Jan Engelhardt
2007-04-30 11:32 ` [PATCH 10/36] Use menuconfig objects II - Infiniband Jan Engelhardt
2007-04-30 11:33 ` [PATCH 11/36] Use menuconfig objects II - IPMI Jan Engelhardt
2007-04-30 14:43   ` Corey Minyard
2007-04-30 11:35 ` [PATCH 12/36] Use menuconfig objects II - ISDN Jan Engelhardt
2007-04-30 11:35 ` [PATCH 13/36] Use menuconfig objects II - KVM/Virt Jan Engelhardt
2007-04-30 13:45   ` Avi Kivity
2007-04-30 11:36 ` [PATCH 14/36] Use menuconfig objects II - LED Jan Engelhardt
2007-04-30 11:37 ` [PATCH 15/36] Use menuconfig objects II - Macintosh Jan Engelhardt
2007-04-30 11:38 ` [PATCH 16/36] Use menuconfig objects II - MD Jan Engelhardt
2007-04-30 11:40 ` [PATCH 17/36] Use menuconfig objects II - misc strange dev Jan Engelhardt
2007-04-30 11:40 ` [PATCH 18/36] Use menuconfig objects II - MMC Jan Engelhardt
2007-04-30 13:51   ` Pierre Ossman
2007-05-01 15:57   ` Pierre Ossman
2007-05-01 20:37     ` Jan Engelhardt
2007-05-02  5:19       ` Pierre Ossman
2007-05-02 17:47         ` Jan Engelhardt
2007-04-30 11:41 ` [PATCH 19/36] Use menuconfig objects II - module menu Jan Engelhardt
2007-04-30 11:43 ` [PATCH 20/36] Use menuconfig objects II - netdev (general+100mbit) Jan Engelhardt
2007-04-30 11:44 ` [PATCH 21/36] Use menuconfig objects II - netdev/atm Jan Engelhardt
2007-04-30 11:45 ` [PATCH 22/36] Use menuconfig objects II - netdev/pcmcia Jan Engelhardt
2007-04-30 11:45 ` [PATCH 23/36] Use menuconfig objects II - netdev/wan Jan Engelhardt
2007-05-01 20:04   ` Krzysztof Halasa
2007-04-30 11:46 ` [PATCH 24/36] Use menuconfig objects II - oprofile Jan Engelhardt
2007-04-30 11:48 ` [PATCH 25/36] Use menuconfig objects II - (external_)power Jan Engelhardt
2007-04-30 11:49 ` [PATCH 26/36] Use menuconfig objects II - RTC Jan Engelhardt
2007-04-30 11:50 ` [PATCH 27/36] Use menuconfig objects II - SCSI Jan Engelhardt
2007-04-30 11:52 ` [PATCH 28/36] Use menuconfig objects II - sound Jan Engelhardt
2007-04-30 14:19   ` Johannes Berg
2007-04-30 14:25     ` Johannes Berg
2007-04-30 14:45     ` Jan Engelhardt
2007-04-30 14:55       ` Johannes Berg
2007-04-30 15:12     ` Jan Engelhardt
2007-04-30 15:14       ` Jan Engelhardt
2007-04-30 15:21   ` [PATCH 28/36 #2] " Jan Engelhardt
2007-04-30 15:41     ` Johannes Berg
2007-04-30 16:13       ` Jan Engelhardt
2007-04-30 11:53 ` [PATCH 29/36] Use menuconfig objects II - Telephony Jan Engelhardt
2007-04-30 11:53 ` [PATCH 30/36] Use menuconfig objects II - TPM Jan Engelhardt
2007-04-30 11:54 ` [PATCH 31/36] Use menuconfig objects II - UIO Jan Engelhardt
2007-04-30 11:55 ` [PATCH 32/36] Use menuconfig objects II - USB Jan Engelhardt
2007-04-30 11:56 ` [PATCH 33/36] Use menuconfig objects II - video/logo Jan Engelhardt
2007-05-01  0:08   ` Antonino A. Daplas
2007-04-30 11:56 ` [PATCH 34/36] Use menuconfig objects II - watchdog Jan Engelhardt
2007-04-30 12:30 ` [PATCH 35/36] Use menuconfig objects II - DVB Jan Engelhardt
2007-05-02 18:10   ` [v4l-dvb-maintainer] " Mauro Carvalho Chehab
2007-05-02 21:38     ` Trent Piepho
2007-05-03  6:56       ` Jan Engelhardt
2007-05-03 12:37         ` Mauro Carvalho Chehab
2007-05-03 12:54           ` Jan Engelhardt
2007-05-03 18:38             ` Mauro Carvalho Chehab
2007-05-03 20:39             ` Trent Piepho
2007-05-03 20:49               ` Sam Ravnborg
2007-05-04  0:28                 ` Roman Zippel
2007-05-04  1:40                   ` Trent Piepho [this message]
2007-05-04  7:46                     ` Jan Engelhardt
2007-05-03 20:50               ` Sam Ravnborg
2007-05-03 20:53               ` Jan Engelhardt
2007-04-30 12:30 ` [PATCH 36/36] Use menuconfig objects II - V4L Jan Engelhardt
2007-05-02 18:03   ` [v4l-dvb-maintainer] " Mauro Carvalho Chehab

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=Pine.LNX.4.58.0705031826360.32689@shell2.speakeasy.net \
    --to=xyzzy@speakeasy.org \
    --cc=jengelh@linux01.gwdg.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=sam@ravnborg.org \
    --cc=v4l-dvb-maintainer@linuxtv.org \
    --cc=zippel@linux-m68k.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®