Patchwork Improving config navigation for mtd tools

login
register
mail settings
Submitter Charles Manning
Date Nov. 28, 2012, 12:12 a.m.
Message ID <CAE21AQqPOGLxFVdCjaFpPo1x2kXCSo3+VbwvYk=THKfRbk+-UQ@mail.gmail.com>
Download mbox | patch
Permalink /patch/202327/
State Superseded
Headers show

Comments

Charles Manning - Nov. 28, 2012, 12:12 a.m.
Hello All

The mtd utilities are part flash utilities and part file system
utilities. This makes it a bit cryptic to put them under "Filesystem
utilities".

I suggest this can be resolved by remaining the heading to "Filesystem
and flash utilities"




Regards

Charles
Arnout Vandecappelle - Nov. 29, 2012, 11:22 p.m.
On 28/11/12 01:12, Charles Manning wrote:
> Hello All
>
> The mtd utilities are part flash utilities and part file system
> utilities. This makes it a bit cryptic to put them under "Filesystem
> utilities".
>
> I suggest this can be resolved by remaining the heading to "Filesystem
> and flash utilities"

  Thank you, that's a good idea.  Could you post a proper patch?
That means:

- Write a commit message that documents the change (more
or less what you wrote above).

- Add a Signed-off-by line for yourself.  This is a short way for you to
assert that you are entitled to contribute the patch under buildroot's
GPL license.  See  http://elinux.org/Developer_Certificate_Of_Origin
for more details.

- Preferably send it with git-send-email.

  This will also add you to the list of buildroot contributors :-)

  Regards,
  Arnout

>
>
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -177,7 +177,7 @@ source "package/xterm/Config.in"
>   source "package/xvkbd/Config.in"
>   endmenu
>
> -menu "Filesystem utilities"
> +menu "Filesystem and flash utilities"
>   source "package/cifs-utils/Config.in"
>   source "package/cramfs/Config.in"
>   source "package/dosfstools/Config.in"
>
>
> Regards
>
> Charles
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>

Patch

--- a/package/Config.in
+++ b/package/Config.in
@@ -177,7 +177,7 @@  source "package/xterm/Config.in"
 source "package/xvkbd/Config.in"
 endmenu

-menu "Filesystem utilities"
+menu "Filesystem and flash utilities"
 source "package/cifs-utils/Config.in"
 source "package/cramfs/Config.in"
 source "package/dosfstools/Config.in"