* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting @ 2013-10-24 13:16 Maxime Hadjinlian 2013-10-24 13:16 ` [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles Maxime Hadjinlian 2013-10-24 13:36 ` [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Thomas De Schampheleire 0 siblings, 2 replies; 10+ messages in thread From: Maxime Hadjinlian @ 2013-10-24 13:16 UTC (permalink / raw) To: buildroot This patch moves : "Filesystem and flash utilities" menu "X applications" category "Misc devices firmwares" menu "JSON/XML" menu so they are in alphabetical order with the rest of the file. --- package/Config.in | 132 +++++++++++++++++++++++++++--------------------------- 1 file changed, 66 insertions(+), 66 deletions(-) diff --git a/package/Config.in b/package/Config.in index 0e6f86c..ebbdeb5 100644 --- a/package/Config.in +++ b/package/Config.in @@ -129,6 +129,32 @@ source "package/vala/Config.in" source "package/yasm/Config.in" endmenu +menu "Filesystem and flash utilities" +source "package/cifs-utils/Config.in" +source "package/cramfs/Config.in" +source "package/curlftpfs/Config.in" +source "package/dosfstools/Config.in" +source "package/e2fsprogs/Config.in" +source "package/ecryptfs-utils/Config.in" +source "package/exfat/Config.in" +source "package/exfat-utils/Config.in" +source "package/f2fs-tools/Config.in" +source "package/flashbench/Config.in" +source "package/genext2fs/Config.in" +source "package/genromfs/Config.in" +source "package/kobs-ng/Config.in" +source "package/makedevs/Config.in" +source "package/mtd/Config.in" +source "package/nfs-utils/Config.in" +source "package/ntfs-3g/Config.in" +source "package/squashfs/Config.in" +source "package/squashfs3/Config.in" +source "package/sshfs/Config.in" +source "package/sunxi-tools/Config.in" +source "package/unionfs/Config.in" +source "package/xfsprogs/Config.in" +endmenu + menu "Games" source "package/doom-wad/Config.in" source "package/gnuchess/Config.in" @@ -187,18 +213,6 @@ source "package/qt5/Config.in" source "package/x11r7/Config.in" source "package/weston/Config.in" -comment "X libraries and helper libraries" -source "package/liberation/Config.in" -source "package/libsexy/Config.in" -source "package/x11r7/xkeyboard-config/Config.in" - -comment "X Window managers" -source "package/blackbox/Config.in" -source "package/enlightenment/Config.in" -source "package/fluxbox/Config.in" -source "package/matchbox/Config.in" -source "package/metacity/Config.in" - comment "X applications" source "package/alsamixergui/Config.in" source "package/docker/Config.in" @@ -222,42 +236,19 @@ source "package/xterm/Config.in" source "package/xvkbd/Config.in" endmenu -menu "Filesystem and flash utilities" -source "package/cifs-utils/Config.in" -source "package/cramfs/Config.in" -source "package/curlftpfs/Config.in" -source "package/dosfstools/Config.in" -source "package/e2fsprogs/Config.in" -source "package/ecryptfs-utils/Config.in" -source "package/exfat/Config.in" -source "package/exfat-utils/Config.in" -source "package/f2fs-tools/Config.in" -source "package/flashbench/Config.in" -source "package/genext2fs/Config.in" -source "package/genromfs/Config.in" -source "package/kobs-ng/Config.in" -source "package/makedevs/Config.in" -source "package/mtd/Config.in" -source "package/nfs-utils/Config.in" -source "package/ntfs-3g/Config.in" -source "package/squashfs/Config.in" -source "package/squashfs3/Config.in" -source "package/sshfs/Config.in" -source "package/sunxi-tools/Config.in" -source "package/unionfs/Config.in" -source "package/xfsprogs/Config.in" -endmenu +comment "X libraries and helper libraries" +source "package/liberation/Config.in" +source "package/libsexy/Config.in" +source "package/x11r7/xkeyboard-config/Config.in" + +comment "X Window managers" +source "package/blackbox/Config.in" +source "package/enlightenment/Config.in" +source "package/fluxbox/Config.in" +source "package/matchbox/Config.in" +source "package/metacity/Config.in" menu "Hardware handling" -menu "Misc devices firmwares" -source "package/am33x-cm3/Config.in" -source "package/b43-firmware/Config.in" -source "package/linux-firmware/Config.in" -source "package/rpi-firmware/Config.in" -source "package/sunxi-boards/Config.in" -source "package/ux500-firmware/Config.in" -source "package/zd1211-firmware/Config.in" -endmenu source "package/a10disp/Config.in" source "package/acpid/Config.in" source "package/cdrkit/Config.in" @@ -304,6 +295,15 @@ source "package/mdadm/Config.in" source "package/media-ctl/Config.in" source "package/memtester/Config.in" source "package/minicom/Config.in" +menu "Misc devices firmwares" +source "package/am33x-cm3/Config.in" +source "package/b43-firmware/Config.in" +source "package/linux-firmware/Config.in" +source "package/rpi-firmware/Config.in" +source "package/sunxi-boards/Config.in" +source "package/ux500-firmware/Config.in" +source "package/zd1211-firmware/Config.in" +endmenu source "package/nanocom/Config.in" source "package/neard/Config.in" source "package/ofono/Config.in" @@ -575,6 +575,26 @@ source "package/jsmin/Config.in" source "package/json-javascript/Config.in" endmenu +menu "JSON/XML" +source "package/cjson/Config.in" +source "package/expat/Config.in" +source "package/ezxml/Config.in" +source "package/jansson/Config.in" +source "package/json-c/Config.in" +source "package/json-glib/Config.in" +source "package/libjson/Config.in" +source "package/libroxml/Config.in" +source "package/libxml-parser-perl/Config.in" +source "package/libxml2/Config.in" +source "package/libxmlpp/Config.in" +source "package/libxslt/Config.in" +source "package/libyaml/Config.in" +source "package/mxml/Config.in" +source "package/rapidjson/Config.in" +source "package/xerces/Config.in" +source "package/yajl/Config.in" +endmenu + menu "Multimedia" source "package/libdvbsi/Config.in" source "package/libdvdnav/Config.in" @@ -720,26 +740,6 @@ source "package/readline/Config.in" source "package/slang/Config.in" endmenu -menu "JSON/XML" -source "package/cjson/Config.in" -source "package/expat/Config.in" -source "package/ezxml/Config.in" -source "package/jansson/Config.in" -source "package/json-c/Config.in" -source "package/json-glib/Config.in" -source "package/libjson/Config.in" -source "package/libroxml/Config.in" -source "package/libxml-parser-perl/Config.in" -source "package/libxml2/Config.in" -source "package/libxmlpp/Config.in" -source "package/libxslt/Config.in" -source "package/libyaml/Config.in" -source "package/mxml/Config.in" -source "package/rapidjson/Config.in" -source "package/xerces/Config.in" -source "package/yajl/Config.in" -endmenu - endmenu menu "Miscellaneous" -- 1.8.4.rc3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles 2013-10-24 13:16 [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Maxime Hadjinlian @ 2013-10-24 13:16 ` Maxime Hadjinlian 2013-10-24 13:26 ` Thomas De Schampheleire 2013-10-24 13:36 ` [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Thomas De Schampheleire 1 sibling, 1 reply; 10+ messages in thread From: Maxime Hadjinlian @ 2013-10-24 13:16 UTC (permalink / raw) To: buildroot --- package/Config.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/package/Config.in b/package/Config.in index ebbdeb5..b1ceb16 100644 --- a/package/Config.in +++ b/package/Config.in @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" source "package/jhead/Config.in" source "package/rrdtool/Config.in" -comment "graphic libraries" +comment "Graphic libraries" source "package/cegui06/Config.in" source "package/directfb/Config.in" source "package/directfb-examples/Config.in" @@ -241,7 +241,7 @@ source "package/liberation/Config.in" source "package/libsexy/Config.in" source "package/x11r7/xkeyboard-config/Config.in" -comment "X Window managers" +comment "X window managers" source "package/blackbox/Config.in" source "package/enlightenment/Config.in" source "package/fluxbox/Config.in" -- 1.8.4.rc3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles 2013-10-24 13:16 ` [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles Maxime Hadjinlian @ 2013-10-24 13:26 ` Thomas De Schampheleire 2013-10-24 14:13 ` Maxime Hadjinlian 0 siblings, 1 reply; 10+ messages in thread From: Thomas De Schampheleire @ 2013-10-24 13:26 UTC (permalink / raw) To: buildroot Hi Maxime, On Thu, Oct 24, 2013 at 3:16 PM, Maxime Hadjinlian <maxime.hadjinlian@gmail.com> wrote: > --- > package/Config.in | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/package/Config.in b/package/Config.in > index ebbdeb5..b1ceb16 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" > source "package/jhead/Config.in" > source "package/rrdtool/Config.in" > > -comment "graphic libraries" > +comment "Graphic libraries" > source "package/cegui06/Config.in" > source "package/directfb/Config.in" > source "package/directfb-examples/Config.in" > @@ -241,7 +241,7 @@ source "package/liberation/Config.in" > source "package/libsexy/Config.in" > source "package/x11r7/xkeyboard-config/Config.in" > > -comment "X Window managers" > +comment "X window managers" > source "package/blackbox/Config.in" > source "package/enlightenment/Config.in" > source "package/fluxbox/Config.in" > -- There is also a 'other GUIs' entry. Any reason why you did not change that to 'Other GUIs' ? Best regards, Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles 2013-10-24 13:26 ` Thomas De Schampheleire @ 2013-10-24 14:13 ` Maxime Hadjinlian 0 siblings, 0 replies; 10+ messages in thread From: Maxime Hadjinlian @ 2013-10-24 14:13 UTC (permalink / raw) To: buildroot On Thu, Oct 24, 2013 at 3:26 PM, Thomas De Schampheleire <patrickdepinguin@gmail.com> wrote: > Hi Maxime, Hi Thomas, > > On Thu, Oct 24, 2013 at 3:16 PM, Maxime Hadjinlian > <maxime.hadjinlian@gmail.com> wrote: >> --- >> package/Config.in | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/package/Config.in b/package/Config.in >> index ebbdeb5..b1ceb16 100644 >> --- a/package/Config.in >> +++ b/package/Config.in >> @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" >> source "package/jhead/Config.in" >> source "package/rrdtool/Config.in" >> >> -comment "graphic libraries" >> +comment "Graphic libraries" >> source "package/cegui06/Config.in" >> source "package/directfb/Config.in" >> source "package/directfb-examples/Config.in" >> @@ -241,7 +241,7 @@ source "package/liberation/Config.in" >> source "package/libsexy/Config.in" >> source "package/x11r7/xkeyboard-config/Config.in" >> >> -comment "X Window managers" >> +comment "X window managers" >> source "package/blackbox/Config.in" >> source "package/enlightenment/Config.in" >> source "package/fluxbox/Config.in" >> -- > > There is also a 'other GUIs' entry. Any reason why you did not change > that to 'Other GUIs' ? > I missed it. I'll fix this and send this patch alone again. > Best regards, > Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting 2013-10-24 13:16 [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Maxime Hadjinlian 2013-10-24 13:16 ` [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles Maxime Hadjinlian @ 2013-10-24 13:36 ` Thomas De Schampheleire 2013-10-24 14:11 ` Maxime Hadjinlian 1 sibling, 1 reply; 10+ messages in thread From: Thomas De Schampheleire @ 2013-10-24 13:36 UTC (permalink / raw) To: buildroot Hi Maxime, On Thu, Oct 24, 2013 at 3:16 PM, Maxime Hadjinlian <maxime.hadjinlian@gmail.com> wrote: > This patch moves : > "Filesystem and flash utilities" menu > "X applications" category > "Misc devices firmwares" menu > "JSON/XML" menu > so they are in alphabetical order with the rest of the file. > --- > package/Config.in | 132 +++++++++++++++++++++++++++--------------------------- > 1 file changed, 66 insertions(+), 66 deletions(-) > > diff --git a/package/Config.in b/package/Config.in > index 0e6f86c..ebbdeb5 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -129,6 +129,32 @@ source "package/vala/Config.in" > source "package/yasm/Config.in" > endmenu > > +menu "Filesystem and flash utilities" > +source "package/cifs-utils/Config.in" > +source "package/cramfs/Config.in" > +source "package/curlftpfs/Config.in" > +source "package/dosfstools/Config.in" > +source "package/e2fsprogs/Config.in" > +source "package/ecryptfs-utils/Config.in" > +source "package/exfat/Config.in" > +source "package/exfat-utils/Config.in" > +source "package/f2fs-tools/Config.in" > +source "package/flashbench/Config.in" > +source "package/genext2fs/Config.in" > +source "package/genromfs/Config.in" > +source "package/kobs-ng/Config.in" > +source "package/makedevs/Config.in" > +source "package/mtd/Config.in" > +source "package/nfs-utils/Config.in" > +source "package/ntfs-3g/Config.in" > +source "package/squashfs/Config.in" > +source "package/squashfs3/Config.in" > +source "package/sshfs/Config.in" > +source "package/sunxi-tools/Config.in" > +source "package/unionfs/Config.in" > +source "package/xfsprogs/Config.in" > +endmenu > + > menu "Games" > source "package/doom-wad/Config.in" > source "package/gnuchess/Config.in" > @@ -187,18 +213,6 @@ source "package/qt5/Config.in" > source "package/x11r7/Config.in" > source "package/weston/Config.in" > > -comment "X libraries and helper libraries" > -source "package/liberation/Config.in" > -source "package/libsexy/Config.in" > -source "package/x11r7/xkeyboard-config/Config.in" > - > -comment "X Window managers" > -source "package/blackbox/Config.in" > -source "package/enlightenment/Config.in" > -source "package/fluxbox/Config.in" > -source "package/matchbox/Config.in" > -source "package/metacity/Config.in" > - > comment "X applications" > source "package/alsamixergui/Config.in" > source "package/docker/Config.in" > @@ -222,42 +236,19 @@ source "package/xterm/Config.in" > source "package/xvkbd/Config.in" > endmenu > > -menu "Filesystem and flash utilities" > -source "package/cifs-utils/Config.in" > -source "package/cramfs/Config.in" > -source "package/curlftpfs/Config.in" > -source "package/dosfstools/Config.in" > -source "package/e2fsprogs/Config.in" > -source "package/ecryptfs-utils/Config.in" > -source "package/exfat/Config.in" > -source "package/exfat-utils/Config.in" > -source "package/f2fs-tools/Config.in" > -source "package/flashbench/Config.in" > -source "package/genext2fs/Config.in" > -source "package/genromfs/Config.in" > -source "package/kobs-ng/Config.in" > -source "package/makedevs/Config.in" > -source "package/mtd/Config.in" > -source "package/nfs-utils/Config.in" > -source "package/ntfs-3g/Config.in" > -source "package/squashfs/Config.in" > -source "package/squashfs3/Config.in" > -source "package/sshfs/Config.in" > -source "package/sunxi-tools/Config.in" > -source "package/unionfs/Config.in" > -source "package/xfsprogs/Config.in" > -endmenu > +comment "X libraries and helper libraries" > +source "package/liberation/Config.in" > +source "package/libsexy/Config.in" > +source "package/x11r7/xkeyboard-config/Config.in" > + > +comment "X Window managers" > +source "package/blackbox/Config.in" > +source "package/enlightenment/Config.in" > +source "package/fluxbox/Config.in" > +source "package/matchbox/Config.in" > +source "package/metacity/Config.in" > > menu "Hardware handling" > -menu "Misc devices firmwares" > -source "package/am33x-cm3/Config.in" > -source "package/b43-firmware/Config.in" > -source "package/linux-firmware/Config.in" > -source "package/rpi-firmware/Config.in" > -source "package/sunxi-boards/Config.in" > -source "package/ux500-firmware/Config.in" > -source "package/zd1211-firmware/Config.in" > -endmenu > source "package/a10disp/Config.in" > source "package/acpid/Config.in" > source "package/cdrkit/Config.in" > @@ -304,6 +295,15 @@ source "package/mdadm/Config.in" > source "package/media-ctl/Config.in" > source "package/memtester/Config.in" > source "package/minicom/Config.in" > +menu "Misc devices firmwares" > +source "package/am33x-cm3/Config.in" > +source "package/b43-firmware/Config.in" > +source "package/linux-firmware/Config.in" > +source "package/rpi-firmware/Config.in" > +source "package/sunxi-boards/Config.in" > +source "package/ux500-firmware/Config.in" > +source "package/zd1211-firmware/Config.in" > +endmenu > source "package/nanocom/Config.in" > source "package/neard/Config.in" > source "package/ofono/Config.in" I can follow all changes, except this one: the 'Misc devices firmwares' menu is a submenu, between normal packages. Moving that in alphabetical order between normal packages seems very odd to me. Especially because the name 'Misc' does not carry a real meaning. By the way, I don't think that 'firmwares' is correct English, the '-ware' already is plural, so it should be 'Misc devices firmware' or maybe plain 'Firmware' (are there other firmware packages in buildroot that are not part of this menu?). Best regards, Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting 2013-10-24 13:36 ` [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Thomas De Schampheleire @ 2013-10-24 14:11 ` Maxime Hadjinlian 2013-10-25 6:22 ` Thomas De Schampheleire 0 siblings, 1 reply; 10+ messages in thread From: Maxime Hadjinlian @ 2013-10-24 14:11 UTC (permalink / raw) To: buildroot On Thu, Oct 24, 2013 at 3:36 PM, Thomas De Schampheleire <patrickdepinguin@gmail.com> wrote: > Hi Maxime, Hi Thomas, > > On Thu, Oct 24, 2013 at 3:16 PM, Maxime Hadjinlian > <maxime.hadjinlian@gmail.com> wrote: >> This patch moves : >> "Filesystem and flash utilities" menu >> "X applications" category >> "Misc devices firmwares" menu >> "JSON/XML" menu >> so they are in alphabetical order with the rest of the file. >> --- >> package/Config.in | 132 +++++++++++++++++++++++++++--------------------------- >> 1 file changed, 66 insertions(+), 66 deletions(-) >> >> diff --git a/package/Config.in b/package/Config.in >> index 0e6f86c..ebbdeb5 100644 >> --- a/package/Config.in >> +++ b/package/Config.in >> @@ -129,6 +129,32 @@ source "package/vala/Config.in" >> source "package/yasm/Config.in" >> endmenu >> >> +menu "Filesystem and flash utilities" >> +source "package/cifs-utils/Config.in" >> +source "package/cramfs/Config.in" >> +source "package/curlftpfs/Config.in" >> +source "package/dosfstools/Config.in" >> +source "package/e2fsprogs/Config.in" >> +source "package/ecryptfs-utils/Config.in" >> +source "package/exfat/Config.in" >> +source "package/exfat-utils/Config.in" >> +source "package/f2fs-tools/Config.in" >> +source "package/flashbench/Config.in" >> +source "package/genext2fs/Config.in" >> +source "package/genromfs/Config.in" >> +source "package/kobs-ng/Config.in" >> +source "package/makedevs/Config.in" >> +source "package/mtd/Config.in" >> +source "package/nfs-utils/Config.in" >> +source "package/ntfs-3g/Config.in" >> +source "package/squashfs/Config.in" >> +source "package/squashfs3/Config.in" >> +source "package/sshfs/Config.in" >> +source "package/sunxi-tools/Config.in" >> +source "package/unionfs/Config.in" >> +source "package/xfsprogs/Config.in" >> +endmenu >> + >> menu "Games" >> source "package/doom-wad/Config.in" >> source "package/gnuchess/Config.in" >> @@ -187,18 +213,6 @@ source "package/qt5/Config.in" >> source "package/x11r7/Config.in" >> source "package/weston/Config.in" >> >> -comment "X libraries and helper libraries" >> -source "package/liberation/Config.in" >> -source "package/libsexy/Config.in" >> -source "package/x11r7/xkeyboard-config/Config.in" >> - >> -comment "X Window managers" >> -source "package/blackbox/Config.in" >> -source "package/enlightenment/Config.in" >> -source "package/fluxbox/Config.in" >> -source "package/matchbox/Config.in" >> -source "package/metacity/Config.in" >> - >> comment "X applications" >> source "package/alsamixergui/Config.in" >> source "package/docker/Config.in" >> @@ -222,42 +236,19 @@ source "package/xterm/Config.in" >> source "package/xvkbd/Config.in" >> endmenu >> >> -menu "Filesystem and flash utilities" >> -source "package/cifs-utils/Config.in" >> -source "package/cramfs/Config.in" >> -source "package/curlftpfs/Config.in" >> -source "package/dosfstools/Config.in" >> -source "package/e2fsprogs/Config.in" >> -source "package/ecryptfs-utils/Config.in" >> -source "package/exfat/Config.in" >> -source "package/exfat-utils/Config.in" >> -source "package/f2fs-tools/Config.in" >> -source "package/flashbench/Config.in" >> -source "package/genext2fs/Config.in" >> -source "package/genromfs/Config.in" >> -source "package/kobs-ng/Config.in" >> -source "package/makedevs/Config.in" >> -source "package/mtd/Config.in" >> -source "package/nfs-utils/Config.in" >> -source "package/ntfs-3g/Config.in" >> -source "package/squashfs/Config.in" >> -source "package/squashfs3/Config.in" >> -source "package/sshfs/Config.in" >> -source "package/sunxi-tools/Config.in" >> -source "package/unionfs/Config.in" >> -source "package/xfsprogs/Config.in" >> -endmenu >> +comment "X libraries and helper libraries" >> +source "package/liberation/Config.in" >> +source "package/libsexy/Config.in" >> +source "package/x11r7/xkeyboard-config/Config.in" >> + >> +comment "X Window managers" >> +source "package/blackbox/Config.in" >> +source "package/enlightenment/Config.in" >> +source "package/fluxbox/Config.in" >> +source "package/matchbox/Config.in" >> +source "package/metacity/Config.in" >> >> menu "Hardware handling" >> -menu "Misc devices firmwares" >> -source "package/am33x-cm3/Config.in" >> -source "package/b43-firmware/Config.in" >> -source "package/linux-firmware/Config.in" >> -source "package/rpi-firmware/Config.in" >> -source "package/sunxi-boards/Config.in" >> -source "package/ux500-firmware/Config.in" >> -source "package/zd1211-firmware/Config.in" >> -endmenu >> source "package/a10disp/Config.in" >> source "package/acpid/Config.in" >> source "package/cdrkit/Config.in" >> @@ -304,6 +295,15 @@ source "package/mdadm/Config.in" >> source "package/media-ctl/Config.in" >> source "package/memtester/Config.in" >> source "package/minicom/Config.in" >> +menu "Misc devices firmwares" >> +source "package/am33x-cm3/Config.in" >> +source "package/b43-firmware/Config.in" >> +source "package/linux-firmware/Config.in" >> +source "package/rpi-firmware/Config.in" >> +source "package/sunxi-boards/Config.in" >> +source "package/ux500-firmware/Config.in" >> +source "package/zd1211-firmware/Config.in" >> +endmenu >> source "package/nanocom/Config.in" >> source "package/neard/Config.in" >> source "package/ofono/Config.in" > > I can follow all changes, except this one: the 'Misc devices > firmwares' menu is a submenu, between normal packages. Moving that in > alphabetical order between normal packages seems very odd to me. I did it after looking at the "Interpreter languages and scripting" menu where the "Lua libraries/modules" would appear in order. > Especially because the name 'Misc' does not carry a real meaning. > > By the way, I don't think that 'firmwares' is correct English, the > '-ware' already is plural, so it should be 'Misc devices firmware' or > maybe plain 'Firmware' (are there other firmware packages in buildroot > that are not part of this menu?). After a very quick search, it seems you are right about the 's' of firmware. About the naming of the menu, we already are in Hardware menu, so talking about "devices" may seem redundant, and the Misc is maybe too cumbersome. Although, a simple "Firmware" would not be very friendly. How about settling on "Firmware's Devices" ? Give or take the casing. > > > Best regards, > Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting 2013-10-24 14:11 ` Maxime Hadjinlian @ 2013-10-25 6:22 ` Thomas De Schampheleire 2013-10-25 8:32 ` Maxime Hadjinlian 0 siblings, 1 reply; 10+ messages in thread From: Thomas De Schampheleire @ 2013-10-25 6:22 UTC (permalink / raw) To: buildroot Hi Maxime, On Thu, Oct 24, 2013 at 4:11 PM, Maxime Hadjinlian <maxime.hadjinlian@gmail.com> wrote: > On Thu, Oct 24, 2013 at 3:36 PM, Thomas De Schampheleire > <patrickdepinguin@gmail.com> wrote: >> >> I can follow all changes, except this one: the 'Misc devices >> firmwares' menu is a submenu, between normal packages. Moving that in >> alphabetical order between normal packages seems very odd to me. > I did it after looking at the "Interpreter languages and scripting" > menu where the "Lua libraries/modules" would appear in order. There it makes sense: the 'Lua libraries/modules' submenu comes immediately after the 'lua' package (that appears in alpabetical order with the other packages in that menu). Moving the Lua libraries menu away from the lua package is not logical. > >> Especially because the name 'Misc' does not carry a real meaning. >> >> By the way, I don't think that 'firmwares' is correct English, the >> '-ware' already is plural, so it should be 'Misc devices firmware' or >> maybe plain 'Firmware' (are there other firmware packages in buildroot >> that are not part of this menu?). > After a very quick search, it seems you are right about the 's' of firmware. > About the naming of the menu, we already are in Hardware menu, so > talking about "devices" may seem redundant, and the Misc is maybe too > cumbersome. Although, a simple "Firmware" would not be very friendly. > How about settling on "Firmware's Devices" ? Give or take the casing. Then this should be 'Device firmware' because "Firmware's Devices" means "The devices of firmware" which is not what you intend to say. I'm not convinced there exists firmware for something else than a device though. For me 'Firmware' or 'Device firmware' are both ok. Best regards, Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting 2013-10-25 6:22 ` Thomas De Schampheleire @ 2013-10-25 8:32 ` Maxime Hadjinlian 2013-10-25 8:48 ` Thomas De Schampheleire 0 siblings, 1 reply; 10+ messages in thread From: Maxime Hadjinlian @ 2013-10-25 8:32 UTC (permalink / raw) To: buildroot On Fri, Oct 25, 2013 at 8:22 AM, Thomas De Schampheleire <patrickdepinguin@gmail.com> wrote: > Hi Maxime, > > On Thu, Oct 24, 2013 at 4:11 PM, Maxime Hadjinlian > <maxime.hadjinlian@gmail.com> wrote: >> On Thu, Oct 24, 2013 at 3:36 PM, Thomas De Schampheleire >> <patrickdepinguin@gmail.com> wrote: >>> >>> I can follow all changes, except this one: the 'Misc devices >>> firmwares' menu is a submenu, between normal packages. Moving that in >>> alphabetical order between normal packages seems very odd to me. >> I did it after looking at the "Interpreter languages and scripting" >> menu where the "Lua libraries/modules" would appear in order. > > There it makes sense: the 'Lua libraries/modules' submenu comes > immediately after the 'lua' package (that appears in alpabetical order > with the other packages in that menu). Moving the Lua libraries menu > away from the lua package is not logical. I will cancel that move then. > >> >>> Especially because the name 'Misc' does not carry a real meaning. >>> >>> By the way, I don't think that 'firmwares' is correct English, the >>> '-ware' already is plural, so it should be 'Misc devices firmware' or >>> maybe plain 'Firmware' (are there other firmware packages in buildroot >>> that are not part of this menu?). >> After a very quick search, it seems you are right about the 's' of firmware. >> About the naming of the menu, we already are in Hardware menu, so >> talking about "devices" may seem redundant, and the Misc is maybe too >> cumbersome. Although, a simple "Firmware" would not be very friendly. >> How about settling on "Firmware's Devices" ? Give or take the casing. > > Then this should be 'Device firmware' > because "Firmware's Devices" means "The devices of firmware" which is > not what you intend to say. > > I'm not convinced there exists firmware for something else than a > device though. For me 'Firmware' or 'Device firmware' are both ok. I just though "Firmware" to be a bit blunt, but I am not really good at wordings as you may have seen so I am OK with whatever is wanted. > > Best regards, > Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting 2013-10-25 8:32 ` Maxime Hadjinlian @ 2013-10-25 8:48 ` Thomas De Schampheleire 2013-10-25 9:58 ` Arnout Vandecappelle 0 siblings, 1 reply; 10+ messages in thread From: Thomas De Schampheleire @ 2013-10-25 8:48 UTC (permalink / raw) To: buildroot On Fri, Oct 25, 2013 at 10:32 AM, Maxime Hadjinlian <maxime.hadjinlian@gmail.com> wrote: > On Fri, Oct 25, 2013 at 8:22 AM, Thomas De Schampheleire >>>> By the way, I don't think that 'firmwares' is correct English, the >>>> '-ware' already is plural, so it should be 'Misc devices firmware' or >>>> maybe plain 'Firmware' (are there other firmware packages in buildroot >>>> that are not part of this menu?). >>> After a very quick search, it seems you are right about the 's' of firmware. >>> About the naming of the menu, we already are in Hardware menu, so >>> talking about "devices" may seem redundant, and the Misc is maybe too >>> cumbersome. Although, a simple "Firmware" would not be very friendly. >>> How about settling on "Firmware's Devices" ? Give or take the casing. >> >> Then this should be 'Device firmware' >> because "Firmware's Devices" means "The devices of firmware" which is >> not what you intend to say. >> >> I'm not convinced there exists firmware for something else than a >> device though. For me 'Firmware' or 'Device firmware' are both ok. > I just though "Firmware" to be a bit blunt, but I am not really good > at wordings as you may have seen so I am OK with whatever is wanted. I'm looking forward to the opinion of others. I am just one of the developers, like you. Best regards, Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting 2013-10-25 8:48 ` Thomas De Schampheleire @ 2013-10-25 9:58 ` Arnout Vandecappelle 0 siblings, 0 replies; 10+ messages in thread From: Arnout Vandecappelle @ 2013-10-25 9:58 UTC (permalink / raw) To: buildroot On 25/10/13 10:48, Thomas De Schampheleire wrote: > On Fri, Oct 25, 2013 at 10:32 AM, Maxime Hadjinlian > <maxime.hadjinlian@gmail.com> wrote: >> On Fri, Oct 25, 2013 at 8:22 AM, Thomas De Schampheleire > >>>>> By the way, I don't think that 'firmwares' is correct English, the >>>>> '-ware' already is plural, so it should be 'Misc devices firmware' or >>>>> maybe plain 'Firmware' (are there other firmware packages in buildroot >>>>> that are not part of this menu?). >>>> After a very quick search, it seems you are right about the 's' of firmware. >>>> About the naming of the menu, we already are in Hardware menu, so >>>> talking about "devices" may seem redundant, and the Misc is maybe too >>>> cumbersome. Although, a simple "Firmware" would not be very friendly. >>>> How about settling on "Firmware's Devices" ? Give or take the casing. >>> >>> Then this should be 'Device firmware' >>> because "Firmware's Devices" means "The devices of firmware" which is >>> not what you intend to say. >>> >>> I'm not convinced there exists firmware for something else than a >>> device though. For me 'Firmware' or 'Device firmware' are both ok. >> I just though "Firmware" to be a bit blunt, but I am not really good >> at wordings as you may have seen so I am OK with whatever is wanted. > > I'm looking forward to the opinion of others. I am just one of the > developers, like you. Firmware blobs? Regards, Arnout -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286500 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2013-10-25 9:58 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-10-24 13:16 [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Maxime Hadjinlian 2013-10-24 13:16 ` [Buildroot] [PATCH 2/2] packages/Config.in: Fix case on entries titles Maxime Hadjinlian 2013-10-24 13:26 ` Thomas De Schampheleire 2013-10-24 14:13 ` Maxime Hadjinlian 2013-10-24 13:36 ` [Buildroot] [PATCH 1/2] packages/Config.in: Alphabetical entries sorting Thomas De Schampheleire 2013-10-24 14:11 ` Maxime Hadjinlian 2013-10-25 6:22 ` Thomas De Schampheleire 2013-10-25 8:32 ` Maxime Hadjinlian 2013-10-25 8:48 ` Thomas De Schampheleire 2013-10-25 9:58 ` Arnout Vandecappelle
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox