Re: [PATCH 1/5] kconfig: add an option to determine a menu's visibility

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Fri, Nov 26, 2010 at 11:15 AM, Michal Marek <mmarek@xxxxxxx> wrote:
> So the patches look OK to me, I added your patches to
> kbuild-2.6.git#menu-visibility and merged the branch to for-next. The
> new syntax should be documented in
> Documentation/kbuild/kconfig-language.txt, below is a first attempt at
> it. If the patches work fine in linux-next _and_ you give me permission
> to push them to Linus, I'll move them to rc-fixes and send a pull
> request.
>
last nits, could you maybe use commits from Mauro's branch at:

git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-next.git
kconfig_fixes

They contains one more menu fix and his "Acked" tag.

> Subject: [PATCH] kconfig: Document the new "visible if" syntax
>
> Signed-off-by: Michal Marek <mmarek@xxxxxxx>
>
> diff --git a/Documentation/kbuild/kconfig-language.txt b/Documentation/kbuild/kconfig-language.txt
> index 2fe93ca..2522cca 100644
> --- a/Documentation/kbuild/kconfig-language.txt
> +++ b/Documentation/kbuild/kconfig-language.txt
> @@ -114,6 +114,13 @@ applicable everywhere (see syntax).
>        the illegal configurations all over.
>        kconfig should one day warn about such things.
>
> +- limiting menu display: "visible if" <expr>
> +  This attribute is only applicable to menu blocks, if the condition is
> +  false, the menu block is not displayed to the user (the symbols
> +  contained there can still be selected by other symbols, though). It is
> +  similar to a conditional "prompt" attribude for individual menu
> +  entries.
> +
>  - numerical ranges: "range" <symbol> <symbol> ["if" <expr>]
>   This allows to limit the range of possible input values for int
>   and hex symbols. The user can only input a value which is larger than
> @@ -300,7 +307,8 @@ menu:
>        "endmenu"
>
>  This defines a menu block, see "Menu structure" above for more
> -information. The only possible options are dependencies.
> +information. The only possible options are dependencies and "visible"
> +attributes.
>
Thanks for having done that part!

regards,
 - Arnaud
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux