mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Randy Dunlap <rdunlap@infradead.org>
To: Ulf Magnusson <ulfalizer@gmail.com>
Cc: Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [ANNOUNCE] Kconfiglib menuconfig implementation
Date: Sun, 20 May 2018 20:47:15 -0700	[thread overview]
Message-ID: <4310659d-e36f-63d3-88b3-e1c5235679b2@infradead.org> (raw)
In-Reply-To: <CAFkk2KTrC7DEe7dU4Jq13YBNWGDSa8KmMm0mpFvxokrKeMzCFg@mail.gmail.com>

On 05/20/2018 08:33 PM, Ulf Magnusson wrote:
> On Mon, May 21, 2018 at 4:45 AM, Randy Dunlap <rdunlap@infradead.org> wrote:
>> On 05/19/2018 09:47 PM, Ulf Magnusson wrote:
>>> On Sun, May 20, 2018 at 6:03 AM, Randy Dunlap <rdunlap@infradead.org> wrote:
>>>> On 05/19/2018 08:45 PM, Randy Dunlap wrote:
>>>>> On 05/08/2018 09:59 AM, Ulf Magnusson wrote:
>>>>>> Hello,
>>>>>>
>>>>>> I've added incremental search for jumping directly to a symbol now.
>>>>>> Regular expressions are supported as well.
>>>>>>
>>>>>> Some screenshots below:
>>>>>>
>>>>>> https://raw.githubusercontent.com/ulfalizer/Kconfiglib/screenshots/screenshots/ss10.png
>>>>>> https://raw.githubusercontent.com/ulfalizer/Kconfiglib/screenshots/screenshots/ss11.png
>>>>>> https://raw.githubusercontent.com/ulfalizer/Kconfiglib/screenshots/screenshots/ss13.png
>>>>>>
>>>>>> The last screenshot shows how things might look after you jump to a
>>>>>> symbol. The jumped-to symbol wasn't visible in this case, so show-all
>>>>>> mode was turned on automatically.
>>>>>
>>>>> Hi Ulf,
>>>>>
>>>>>
>>>>>
>>>>> Hm, OK, I used the trick that you supplied a few weeks ago and I have the UI now.
>>>>>
>>>>>     ARCH=x86 SRCARCH=x86 KERNELVERSION=`make kernelversion` \
>>>>>         Kconfiglib/menuconfig.py
>>>>>
>>>>> so yes, a real Makefile target would be nice. :)
>>>>
>>>> I want to see all kconfig symbols that end with "_DEBUG" (so excluding
>>>> _DEBUGFS).  Using:
>>>>
>>>> /.*_DEBUG$
>>>>
>>>> shows me 6 symbols:
>>>> ATH9K_COMMON_DEBUG
>>>> DVB_B2C2_FLEXCOP_DEBUG
>>>> HAVE_DMA_API_DEBUG
>>>> IP_DCCP_TFRC_DEBUG
>>>> NFS_DEBUG
>>>> PM_SLEEP_DEBUG
>>>>
>>>> so where are the other (approx.) 176?
>>>> see:
>>>> $ find . -name Kconfig\* | xargs grep "config.*_DEBUG$" | grep -v \.orig | wc
>>>>     182     364    9179
>>>>
>>>>
>>>> Anyway, something for you to look at. :)
>>>>
>>>> --
>>>> ~Randy
>>>
>>> It's related to how the search is done. Searching for "DEBUG\b" (word
>>> boundary) instead will find all of them.
>>>
>>> A string is generated for each symbol. For symbols with prompts, this
>>> string includes the prompt as well, so you get e.g.
>>>
>>>     AB8500_DEBUG "Enable debug info via debugfs"
>>>
>>> Those strings are then searched/displayed.
>>>
>>> What you got with your search was all symbols whose names end in
>>> "_DEBUG" that don't have a prompt.
>>>
>>> The nice thing about searching both the name and the prompt is that
>>> e.g. "debugfs ab8500" will find the symbol above. I'm also planning to
>>> add menus to the search, by generating 'menu: "menu title"' strings
>>> for them.
>>>
>>> It's a bit awkward/unintuitive that what you tried doesn't work
>>> though. Maybe the symbol name could be searched separately from the
>>> prompt, though I'd be a bit sad to abandon the super simple
>>> single-string-per-entry implementation approach. :)
>>
>> Hi Ulf,
>>
>> What else are you planning to do with /(search)?
> 
> Here's the stuff I've thought of so far:
> 
>   - Display invisible symbols in red (to match show-all mode)
> 
>   - Search prompts of menus and comments as well
> 
>   - Have [F1] show the help display without canceling the search,
>     so you can quickly scan through a bunch of symbols
> 
> Maybe you could get fancy with stuff like finding symbols that are
> related to other symbols in particular ways too...
> 
>> /syscall lists 20 or so symbols. It would be nice if each one of those
>> showed its current setting [y,n,maybe^Wm].
> 
> Nice idea. I added it:
> https://github.com/ulfalizer/Kconfiglib/commit/3161affaa5bb8350a39bfd305c56171a25b02151
> 
> Works for strings/int/hex too.

Thanks.

>> I like Toggle show-all mode.  When I use xconfig, I usually turn on most
>> of its options, including one like that.
> 
> IIRC, xconfig doesn't make it very clear which symbols are invisible
> in show-all mode. That's why I made them red.

Yes, that's helpful.


-- 
~Randy

  reply	other threads:[~2018-05-21  3:47 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-01  0:57 Ulf Magnusson
2018-05-01 17:56 ` Randy Dunlap
2018-05-01 18:13   ` Randy Dunlap
2018-05-01 18:52     ` Randy Dunlap
2018-05-01 19:33       ` Ulf Magnusson
2018-05-03  2:47       ` Ulf Magnusson
2018-05-03 20:31         ` Randy Dunlap
2018-05-03 21:26           ` Ulf Magnusson
2018-05-01 19:12   ` Ulf Magnusson
2018-05-01 19:47     ` Ulf Magnusson
2018-05-01 21:07       ` Ulf Magnusson
2018-05-03 20:28         ` Randy Dunlap
2018-05-03 22:48           ` Ulf Magnusson
2018-05-04  0:05             ` Randy Dunlap
2018-05-08 16:59           ` Ulf Magnusson
2018-05-11  1:22             ` Ulf Magnusson
2018-05-20  3:45             ` Randy Dunlap
2018-05-20  3:51               ` Randy Dunlap
2018-05-20  4:24                 ` Ulf Magnusson
2018-05-20  5:50                   ` Ulf Magnusson
2018-05-20  4:03               ` Randy Dunlap
2018-05-20  4:47                 ` Ulf Magnusson
2018-05-20  5:13                   ` Ulf Magnusson
2018-05-21  2:45                   ` Randy Dunlap
2018-05-21  3:33                     ` Ulf Magnusson
2018-05-21  3:47                       ` Randy Dunlap [this message]
2018-05-22 10:18                         ` Ulf Magnusson
2018-05-25 22:06                           ` Randy Dunlap
2018-05-20  7:51                 ` Ulf Magnusson
2018-05-21  2:22                   ` Randy Dunlap
2018-05-20  6:03               ` Ulf Magnusson
2018-05-14 16:58     ` Pavel Machek
2018-05-14 18:12       ` Ulf Magnusson

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=4310659d-e36f-63d3-88b3-e1c5235679b2@infradead.org \
    --to=rdunlap@infradead.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=ulfalizer@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

all inboxes | Powered by JetHome®