Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1176486

Re: Kconfig: '+config' valid syntax?

From Stefan Hengelein <stefan.hengelein@fau.de>
Newsgroups linux.kernel
Subject Re: Kconfig: '+config' valid syntax?
Date 2015-07-03 13:00 +0200
Message-ID <pI6Rl-7tK-29@gated-at.bofh.it> (permalink)
References (4 earlier) <pI3JM-5CA-3@gated-at.bofh.it> <pI58S-6BI-27@gated-at.bofh.it> <pI5se-6JS-19@gated-at.bofh.it> <pI6HE-7qk-3@gated-at.bofh.it> <pI6Rk-7tK-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


2015-07-03 12:51 GMT+02:00 Valentin Rothberg <valentinrothberg@gmail.com>:
>>>
>>> The output looks like this:
>>> scripts/kconfig/conf  --allyesconfig Kconfig
>>> arch/x86/Kconfig:4:warning: ignoring unsupported character '-'
>>> arch/x86/Kconfig:4:warning: ignoring unsupported character '-'
>>> arch/x86/Kconfig:4:warning: ignoring unsupported character '-'
>>> init/Kconfig:222:warning: ignoring unsupported character '-'
>>> init/Kconfig:222:warning: ignoring unsupported character '-'
>>> init/Kconfig:222:warning: ignoring unsupported character '-'
>>> init/Kconfig:244:warning: ignoring unsupported character '-'
>>> init/Kconfig:244:warning: ignoring unsupported character '-'
>>> init/Kconfig:244:warning: ignoring unsupported character '-'
>>> [...]
>>>
>>> So we would need to add special treatment for '-' also in the command
>>> case, right? But that doesn't look appealing to me, more like a dirty,
>>> dirty hack around the actual problem...
>>>
>>> Regards,
>>>
>>> Andreas
>>>
>>
>> Except for scattered accidents like in the original message, which are
>> hopefully pretty rare and easy to fix, the only documented thing that depends
>> on that lexer sloppiness is the ---help--- "token".
>>
>> I'd just add "---help---" as another T_HELP alias (or get rid of it altogether,
>> but that's probably more work than it's worth). Tightening things up should be
>> safe after that.
>
> This idea has a big ACK from me.  It seems to me the cleanest way to
> solve the issue.
>

Agreed! I also wanted to suggest this solution, but Ulf was faster :)

Kind Regards,
Stefan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: Kconfig: '+config' valid syntax? Paul Bolle <pebolle@tiscali.nl> - 2015-07-02 11:10 +0200
  Re: Kconfig: '+config' valid syntax? Paul Bolle <pebolle@tiscali.nl> - 2015-07-02 11:30 +0200
  Re: Kconfig: '+config' valid syntax? Andreas Ruprecht <andreas.ruprecht@fau.de> - 2015-07-03 11:30 +0200
    Re: Kconfig: '+config' valid syntax? Ulf Magnusson <ulfalizer.lkml@gmail.com> - 2015-07-03 12:50 +0200
      Re: Kconfig: '+config' valid syntax? Valentin Rothberg <valentinrothberg@gmail.com> - 2015-07-03 13:00 +0200
        Re: Kconfig: '+config' valid syntax? Stefan Hengelein <stefan.hengelein@fau.de> - 2015-07-03 13:00 +0200
      Re: Kconfig: '+config' valid syntax? Paul Bolle <pebolle@tiscali.nl> - 2015-07-03 13:10 +0200
    Re: Kconfig: '+config' valid syntax? Paul Bolle <pebolle@tiscali.nl> - 2015-07-03 13:00 +0200

csiph-web