Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [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 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 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 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