Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order
@ 2014-12-13  8:18 Jerzy Grzegorek
  2014-12-13  8:18 ` [Buildroot] [PATCH 2/2] package/Config.in: fix " Jerzy Grzegorek
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Jerzy Grzegorek @ 2014-12-13  8:18 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
---
 package/Config.in |   22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 48be2df..7300443 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -961,6 +961,17 @@ endmenu
 
 endmenu
 
+menu "Mail"
+	source "package/dovecot/Config.in"
+	source "package/exim/Config.in"
+	source "package/fetchmail/Config.in"
+	source "package/heirloom-mailx/Config.in"
+	source "package/libesmtp/Config.in"
+	source "package/msmtp/Config.in"
+	source "package/mutt/Config.in"
+	source "package/sylpheed/Config.in"
+endmenu
+
 menu "Miscellaneous"
 	source "package/aespipe/Config.in"
 	source "package/bc/Config.in"
@@ -978,17 +989,6 @@ menu "Miscellaneous"
 	source "package/sound-theme-freedesktop/Config.in"
 endmenu
 
-menu "Mail"
-	source "package/dovecot/Config.in"
-	source "package/exim/Config.in"
-	source "package/fetchmail/Config.in"
-	source "package/heirloom-mailx/Config.in"
-	source "package/libesmtp/Config.in"
-	source "package/msmtp/Config.in"
-	source "package/mutt/Config.in"
-	source "package/sylpheed/Config.in"
-endmenu
-
 menu "Networking applications"
 	source "package/aiccu/Config.in"
 	source "package/aircrack-ng/Config.in"
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order
  2014-12-13  8:18 [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order Jerzy Grzegorek
@ 2014-12-13  8:18 ` Jerzy Grzegorek
  2014-12-16 17:46   ` Yann E. MORIN
  2014-12-16 17:45 ` [Buildroot] [PATCH 1/2] package/Config.in: fix menu " Yann E. MORIN
  2014-12-16 22:46 ` Peter Korsgaard
  2 siblings, 1 reply; 9+ messages in thread
From: Jerzy Grzegorek @ 2014-12-13  8:18 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
---
 package/Config.in |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 7300443..81bb9e3 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -303,9 +303,9 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
 endif
 	source "package/dmidecode/Config.in"
 	source "package/dmraid/Config.in"
+	source "package/dtv-scan-tables/Config.in"
 	source "package/dvb-apps/Config.in"
 	source "package/dvbsnoop/Config.in"
-	source "package/dtv-scan-tables/Config.in"
 	source "package/eeprog/Config.in"
 	source "package/eudev/Config.in"
 	source "package/evemu/Config.in"
@@ -424,7 +424,6 @@ menu "Lua libraries/modules"
 	source "package/lua-messagepack/Config.in"
 	source "package/lua-msgpack-native/Config.in"
 	source "package/lua-testmore/Config.in"
-	source "package/luasql-sqlite3/Config.in"
 	source "package/luabitop/Config.in"
 	source "package/luacrypto/Config.in"
 	source "package/luaexpat/Config.in"
@@ -435,6 +434,7 @@ menu "Lua libraries/modules"
 	source "package/luaposix/Config.in"
 	source "package/luasec/Config.in"
 	source "package/luasocket/Config.in"
+	source "package/luasql-sqlite3/Config.in"
 	source "package/lunit/Config.in"
 	source "package/lzlib/Config.in"
 	source "package/orbit/Config.in"
@@ -460,8 +460,8 @@ menu "Perl libraries/modules"
 	source "package/perl-try-tiny/Config.in"
 	source "package/perl-xml-libxml/Config.in"
 	source "package/perl-xml-namespacesupport/Config.in"
-	source "package/perl-xml-sax-base/Config.in"
 	source "package/perl-xml-sax/Config.in"
+	source "package/perl-xml-sax-base/Config.in"
 endmenu
 endif
 	source "package/php/Config.in"
@@ -837,9 +837,9 @@ menu "Networking"
 	source "package/libosip2/Config.in"
 	source "package/libpcap/Config.in"
 	source "package/librsync/Config.in"
-	source "package/libsocketcan/Config.in"
 	source "package/libshairplay/Config.in"
 	source "package/libshout/Config.in"
+	source "package/libsocketcan/Config.in"
 	source "package/libsoup/Config.in"
 	source "package/libstrophe/Config.in"
 	source "package/libtirpc/Config.in"
@@ -1236,8 +1236,8 @@ endif
 	source "package/numactl/Config.in"
 	source "package/nut/Config.in"
 	source "package/openvmtools/Config.in"
-	source "package/powerpc-utils/Config.in"
 	source "package/polkit/Config.in"
+	source "package/powerpc-utils/Config.in"
 if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
 	source "package/procps-ng/Config.in"
 	source "package/psmisc/Config.in"
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order
  2014-12-13  8:18 [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order Jerzy Grzegorek
  2014-12-13  8:18 ` [Buildroot] [PATCH 2/2] package/Config.in: fix " Jerzy Grzegorek
@ 2014-12-16 17:45 ` Yann E. MORIN
  2014-12-16 22:46 ` Peter Korsgaard
  2 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2014-12-16 17:45 UTC (permalink / raw)
  To: buildroot

Jerzy, All,

On 2014-12-13 09:18 +0100, Jerzy Grzegorek spake thusly:
> Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/Config.in |   22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/package/Config.in b/package/Config.in
> index 48be2df..7300443 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -961,6 +961,17 @@ endmenu
>  
>  endmenu
>  
> +menu "Mail"
> +	source "package/dovecot/Config.in"
> +	source "package/exim/Config.in"
> +	source "package/fetchmail/Config.in"
> +	source "package/heirloom-mailx/Config.in"
> +	source "package/libesmtp/Config.in"
> +	source "package/msmtp/Config.in"
> +	source "package/mutt/Config.in"
> +	source "package/sylpheed/Config.in"
> +endmenu
> +
>  menu "Miscellaneous"
>  	source "package/aespipe/Config.in"
>  	source "package/bc/Config.in"
> @@ -978,17 +989,6 @@ menu "Miscellaneous"
>  	source "package/sound-theme-freedesktop/Config.in"
>  endmenu
>  
> -menu "Mail"
> -	source "package/dovecot/Config.in"
> -	source "package/exim/Config.in"
> -	source "package/fetchmail/Config.in"
> -	source "package/heirloom-mailx/Config.in"
> -	source "package/libesmtp/Config.in"
> -	source "package/msmtp/Config.in"
> -	source "package/mutt/Config.in"
> -	source "package/sylpheed/Config.in"
> -endmenu
> -
>  menu "Networking applications"
>  	source "package/aiccu/Config.in"
>  	source "package/aircrack-ng/Config.in"
> -- 
> 1.7.9.5
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order
  2014-12-13  8:18 ` [Buildroot] [PATCH 2/2] package/Config.in: fix " Jerzy Grzegorek
@ 2014-12-16 17:46   ` Yann E. MORIN
  0 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2014-12-16 17:46 UTC (permalink / raw)
  To: buildroot

Jerzy, All,

On 2014-12-13 09:18 +0100, Jerzy Grzegorek spake thusly:
> Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/Config.in |   10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/package/Config.in b/package/Config.in
> index 7300443..81bb9e3 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -303,9 +303,9 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
>  endif
>  	source "package/dmidecode/Config.in"
>  	source "package/dmraid/Config.in"
> +	source "package/dtv-scan-tables/Config.in"
>  	source "package/dvb-apps/Config.in"
>  	source "package/dvbsnoop/Config.in"
> -	source "package/dtv-scan-tables/Config.in"
>  	source "package/eeprog/Config.in"
>  	source "package/eudev/Config.in"
>  	source "package/evemu/Config.in"
> @@ -424,7 +424,6 @@ menu "Lua libraries/modules"
>  	source "package/lua-messagepack/Config.in"
>  	source "package/lua-msgpack-native/Config.in"
>  	source "package/lua-testmore/Config.in"
> -	source "package/luasql-sqlite3/Config.in"
>  	source "package/luabitop/Config.in"
>  	source "package/luacrypto/Config.in"
>  	source "package/luaexpat/Config.in"
> @@ -435,6 +434,7 @@ menu "Lua libraries/modules"
>  	source "package/luaposix/Config.in"
>  	source "package/luasec/Config.in"
>  	source "package/luasocket/Config.in"
> +	source "package/luasql-sqlite3/Config.in"
>  	source "package/lunit/Config.in"
>  	source "package/lzlib/Config.in"
>  	source "package/orbit/Config.in"
> @@ -460,8 +460,8 @@ menu "Perl libraries/modules"
>  	source "package/perl-try-tiny/Config.in"
>  	source "package/perl-xml-libxml/Config.in"
>  	source "package/perl-xml-namespacesupport/Config.in"
> -	source "package/perl-xml-sax-base/Config.in"
>  	source "package/perl-xml-sax/Config.in"
> +	source "package/perl-xml-sax-base/Config.in"
>  endmenu
>  endif
>  	source "package/php/Config.in"
> @@ -837,9 +837,9 @@ menu "Networking"
>  	source "package/libosip2/Config.in"
>  	source "package/libpcap/Config.in"
>  	source "package/librsync/Config.in"
> -	source "package/libsocketcan/Config.in"
>  	source "package/libshairplay/Config.in"
>  	source "package/libshout/Config.in"
> +	source "package/libsocketcan/Config.in"
>  	source "package/libsoup/Config.in"
>  	source "package/libstrophe/Config.in"
>  	source "package/libtirpc/Config.in"
> @@ -1236,8 +1236,8 @@ endif
>  	source "package/numactl/Config.in"
>  	source "package/nut/Config.in"
>  	source "package/openvmtools/Config.in"
> -	source "package/powerpc-utils/Config.in"
>  	source "package/polkit/Config.in"
> +	source "package/powerpc-utils/Config.in"
>  if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
>  	source "package/procps-ng/Config.in"
>  	source "package/psmisc/Config.in"
> -- 
> 1.7.9.5
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order
  2014-12-13  8:18 [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order Jerzy Grzegorek
  2014-12-13  8:18 ` [Buildroot] [PATCH 2/2] package/Config.in: fix " Jerzy Grzegorek
  2014-12-16 17:45 ` [Buildroot] [PATCH 1/2] package/Config.in: fix menu " Yann E. MORIN
@ 2014-12-16 22:46 ` Peter Korsgaard
  2 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2014-12-16 22:46 UTC (permalink / raw)
  To: buildroot

>>>>> "Jerzy" == Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net> writes:

 > Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>

Committed both, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order
  2015-10-10  7:39 [Buildroot] [PATCH 1/2] package/Config.in: correct capitalization of menu items Jerzy Grzegorek
@ 2015-10-10  7:39 ` Jerzy Grzegorek
  0 siblings, 0 replies; 9+ messages in thread
From: Jerzy Grzegorek @ 2015-10-10  7:39 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
---
 package/Config.in | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 8672ec4..e049ffc 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1,7 +1,7 @@
 menu "Target packages"
 
-	source "package/skeleton/Config.in"
 	source "package/busybox/Config.in"
+	source "package/skeleton/Config.in"
 
 menu "Audio and video applications"
 	source "package/alsa-utils/Config.in"
@@ -171,8 +171,8 @@ menu "Filesystem and flash utilities"
 	source "package/nfs-utils/Config.in"
 	source "package/ntfs-3g/Config.in"
 	source "package/simicsfs/Config.in"
-	source "package/squashfs/Config.in"
 	source "package/sp-oops-extract/Config.in"
+	source "package/squashfs/Config.in"
 	source "package/sshfs/Config.in"
 	source "package/sunxi-tools/Config.in"
 	source "package/unionfs/Config.in"
@@ -621,8 +621,8 @@ menu "External python modules"
 	source "package/python-keyring/Config.in"
 	source "package/python-libconfig/Config.in"
 	source "package/python-lxml/Config.in"
-	source "package/python-mako/Config.in"
 	source "package/python-mad/Config.in"
+	source "package/python-mako/Config.in"
 	source "package/python-markdown/Config.in"
 	source "package/python-markupsafe/Config.in"
 	source "package/python-meld3/Config.in"
@@ -733,8 +733,8 @@ endmenu
 
 menu "Compression and decompression"
 	source "package/libarchive/Config.in"
-	source "package/libzip/Config.in"
 	source "package/libsquish/Config.in"
+	source "package/libzip/Config.in"
 	source "package/lzo/Config.in"
 	source "package/snappy/Config.in"
 	source "package/szip/Config.in"
@@ -984,8 +984,8 @@ menu "Networking"
 	source "package/czmq/Config.in"
 	source "package/filemq/Config.in"
 	source "package/flickcurl/Config.in"
-	source "package/freeradius-client/Config.in"
 	source "package/fmlib/Config.in"
+	source "package/freeradius-client/Config.in"
 	source "package/geoip/Config.in"
 	source "package/glib-networking/Config.in"
 	source "package/gssdp/Config.in"
@@ -1010,7 +1010,6 @@ menu "Networking"
 	source "package/libmnl/Config.in"
 	source "package/libmodbus/Config.in"
 	source "package/libndp/Config.in"
-	source "package/libnice/Config.in"
 	source "package/libnetfilter_acct/Config.in"
 	source "package/libnetfilter_conntrack/Config.in"
 	source "package/libnetfilter_cthelper/Config.in"
@@ -1019,6 +1018,7 @@ menu "Networking"
 	source "package/libnetfilter_queue/Config.in"
 	source "package/libnfnetlink/Config.in"
 	source "package/libnftnl/Config.in"
+	source "package/libnice/Config.in"
 	source "package/libnl/Config.in"
 	source "package/liboauth/Config.in"
 	source "package/liboping/Config.in"
@@ -1192,8 +1192,8 @@ menu "Miscellaneous"
 	source "package/haveged/Config.in"
 	source "package/mcrypt/Config.in"
 	source "package/mobile-broadband-provider-info/Config.in"
-	source "package/qpdf/Config.in"
 	source "package/qemu/Config.in"
+	source "package/qpdf/Config.in"
 	source "package/shared-mime-info/Config.in"
 	source "package/snowball-init/Config.in"
 	source "package/wine/Config.in"
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order
  2019-05-29 15:05 [Buildroot] [PATCH 1/2] checkpackagelib/lib_config.py: check packages alphabetical order in {Config.in, Config.in.host} Jerzy Grzegorek
@ 2019-05-29 15:05 ` Jerzy Grzegorek
  2019-05-29 21:42   ` Arnout Vandecappelle
  0 siblings, 1 reply; 9+ messages in thread
From: Jerzy Grzegorek @ 2019-05-29 15:05 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Jerzy Grzegorek <jerzy.m.grzegorek@gmail.com>
---
 package/Config.in | 44 ++++++++++++++++++++++----------------------
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index e89d0cfc8b..aec91bfdcd 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -287,10 +287,10 @@ comment "Graphic libraries"
 	source "package/directfb/Config.in"
 	source "package/directfb-examples/Config.in"
 	source "package/efl/Config.in"
+	source "package/fb-test-app/Config.in"
 	source "package/fbdump/Config.in"
 	source "package/fbgrab/Config.in"
 	source "package/fbset/Config.in"
-	source "package/fb-test-app/Config.in"
 	source "package/fbterm/Config.in"
 	source "package/fbv/Config.in"
 	source "package/freerdp/Config.in"
@@ -302,23 +302,23 @@ comment "Graphic libraries"
 	source "package/ocrad/Config.in"
 	source "package/psplash/Config.in"
 	source "package/sdl/Config.in"
-	source "package/sdl_gfx/Config.in"
-	source "package/sdl_image/Config.in"
-	source "package/sdl_mixer/Config.in"
-	source "package/sdl_net/Config.in"
-	source "package/sdl_sound/Config.in"
-	source "package/sdl_ttf/Config.in"
 	source "package/sdl2/Config.in"
 	source "package/sdl2_gfx/Config.in"
 	source "package/sdl2_image/Config.in"
 	source "package/sdl2_mixer/Config.in"
 	source "package/sdl2_net/Config.in"
 	source "package/sdl2_ttf/Config.in"
+	source "package/sdl_gfx/Config.in"
+	source "package/sdl_image/Config.in"
+	source "package/sdl_mixer/Config.in"
+	source "package/sdl_net/Config.in"
+	source "package/sdl_sound/Config.in"
+	source "package/sdl_ttf/Config.in"
 	source "package/tk/Config.in"
 
 comment "Other GUIs"
-	source "package/qt5/Config.in"
 	source "package/kf5/Config.in"
+	source "package/qt5/Config.in"
 if BR2_PACKAGE_QT5
 comment "QT libraries and helper libraries"
 	source "package/cutelyst/Config.in"
@@ -459,8 +459,8 @@ endmenu
 	source "package/kbd/Config.in"
 	source "package/lcdproc/Config.in"
 	source "package/libuio/Config.in"
-	source "package/linuxconsoletools/Config.in"
 	source "package/linux-backports/Config.in"
+	source "package/linuxconsoletools/Config.in"
 	source "package/lirc-tools/Config.in"
 	source "package/lm-sensors/Config.in"
 	source "package/lshw/Config.in"
@@ -629,12 +629,15 @@ menu "Lua libraries/modules"
 	source "package/lua-messagepack/Config.in"
 	source "package/lua-msgpack-native/Config.in"
 	source "package/lua-periphery/Config.in"
+	source "package/lua-resty-http/Config.in"
+	source "package/lua-sailor/Config.in"
 	source "package/lua-sdl2/Config.in"
 	source "package/lua-std-debug/Config.in"
 	source "package/lua-std-normalize/Config.in"
 	source "package/lua-stdlib/Config.in"
 	source "package/lua-testmore/Config.in"
 	source "package/lua-utf8/Config.in"
+	source "package/lua-valua/Config.in"
 	source "package/luabitop/Config.in"
 	source "package/luadbi/Config.in"
 	source "package/luadbi-sqlite3/Config.in"
@@ -645,12 +648,9 @@ menu "Lua libraries/modules"
 	source "package/lualogging/Config.in"
 	source "package/luaossl/Config.in"
 	source "package/luaposix/Config.in"
-	source "package/lua-resty-http/Config.in"
-	source "package/lua-sailor/Config.in"
 	source "package/luasec/Config.in"
 	source "package/luasocket/Config.in"
 	source "package/luasql-sqlite3/Config.in"
-	source "package/lua-valua/Config.in"
 	source "package/lutok/Config.in"
 	source "package/luv/Config.in"
 	source "package/luvi/Config.in"
@@ -739,13 +739,13 @@ menu "Perl libraries/modules"
 	source "package/perl-lwp-mediatypes/Config.in"
 	source "package/perl-mail-dkim/Config.in"
 	source "package/perl-mailtools/Config.in"
-	source "package/perl-mime-base64-urlsafe/Config.in"
 	source "package/perl-mime-base64/Config.in"
+	source "package/perl-mime-base64-urlsafe/Config.in"
 	source "package/perl-mime-tools/Config.in"
 	source "package/perl-module-implementation/Config.in"
 	source "package/perl-module-runtime/Config.in"
-	source "package/perl-mojolicious-plugin-authentication/Config.in"
 	source "package/perl-mojolicious/Config.in"
+	source "package/perl-mojolicious-plugin-authentication/Config.in"
 	source "package/perl-moo/Config.in"
 	source "package/perl-net-dns/Config.in"
 	source "package/perl-net-http/Config.in"
@@ -764,8 +764,8 @@ menu "Perl libraries/modules"
 	source "package/perl-role-tiny/Config.in"
 	source "package/perl-stream-buffered/Config.in"
 	source "package/perl-sub-exporter-progressive/Config.in"
-	source "package/perl-sub-quote/Config.in"
 	source "package/perl-sub-install/Config.in"
+	source "package/perl-sub-quote/Config.in"
 	source "package/perl-sys-cpu/Config.in"
 	source "package/perl-sys-meminfo/Config.in"
 	source "package/perl-sys-mmap/Config.in"
@@ -816,9 +816,9 @@ menu "External python modules"
 	source "package/python-aiohttp-security/Config.in"
 	source "package/python-aiohttp-session/Config.in"
 	source "package/python-aiohttp-sse/Config.in"
+	source "package/python-aiojobs/Config.in"
 	source "package/python-aiomonitor/Config.in"
 	source "package/python-aiorwlock/Config.in"
-	source "package/python-aiojobs/Config.in"
 	source "package/python-alsaaudio/Config.in"
 	source "package/python-argh/Config.in"
 	source "package/python-arrow/Config.in"
@@ -869,8 +869,8 @@ menu "External python modules"
 	source "package/python-dicttoxml/Config.in"
 	source "package/python-django/Config.in"
 	source "package/python-docker/Config.in"
-	source "package/python-dockerpty/Config.in"
 	source "package/python-docker-pycreds/Config.in"
+	source "package/python-dockerpty/Config.in"
 	source "package/python-docopt/Config.in"
 	source "package/python-docutils/Config.in"
 	source "package/python-dominate/Config.in"
@@ -882,8 +882,8 @@ menu "External python modules"
 	source "package/python-falcon/Config.in"
 	source "package/python-fire/Config.in"
 	source "package/python-flask/Config.in"
-	source "package/python-flask-cors/Config.in"
 	source "package/python-flask-babel/Config.in"
+	source "package/python-flask-cors/Config.in"
 	source "package/python-flask-jsonrpc/Config.in"
 	source "package/python-flask-login/Config.in"
 	source "package/python-flask-sqlalchemy/Config.in"
@@ -916,9 +916,9 @@ menu "External python modules"
 	source "package/python-jaraco-classes/Config.in"
 	source "package/python-jedi/Config.in"
 	source "package/python-jinja2/Config.in"
+	source "package/python-json-schema-validator/Config.in"
 	source "package/python-jsonmodels/Config.in"
 	source "package/python-jsonschema/Config.in"
-	source "package/python-json-schema-validator/Config.in"
 	source "package/python-keyring/Config.in"
 	source "package/python-libconfig/Config.in"
 	source "package/python-libusb1/Config.in"
@@ -1263,8 +1263,8 @@ menu "Graphics"
 	source "package/cairo/Config.in"
 	source "package/cairomm/Config.in"
 	source "package/chipmunk/Config.in"
-	source "package/exiv2/Config.in"
 	source "package/exempi/Config.in"
+	source "package/exiv2/Config.in"
 	source "package/fltk/Config.in"
 	source "package/fontconfig/Config.in"
 	source "package/freetype/Config.in"
@@ -1845,8 +1845,8 @@ menu "Networking applications"
 	source "package/bind/Config.in"
 	source "package/bird/Config.in"
 	source "package/bluez-tools/Config.in"
-	source "package/bluez_utils/Config.in"
 	source "package/bluez5_utils/Config.in"
+	source "package/bluez_utils/Config.in"
 	source "package/bmon/Config.in"
 	source "package/boa/Config.in"
 	source "package/boinc/Config.in"
@@ -1956,13 +1956,13 @@ menu "Networking applications"
 	source "package/nbd/Config.in"
 	source "package/ncftp/Config.in"
 	source "package/ndisc6/Config.in"
+	source "package/net-tools/Config.in"
 	source "package/netatalk/Config.in"
 	source "package/netcat/Config.in"
 	source "package/netcat-openbsd/Config.in"
 	source "package/netplug/Config.in"
 	source "package/netsnmp/Config.in"
 	source "package/netstat-nat/Config.in"
-	source "package/net-tools/Config.in"
 	source "package/network-manager/Config.in"
 	source "package/nfacct/Config.in"
 	source "package/nftables/Config.in"
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order
  2019-05-29 15:05 ` [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order Jerzy Grzegorek
@ 2019-05-29 21:42   ` Arnout Vandecappelle
  2019-05-30  6:37     ` Jerzy Grzegorek
  0 siblings, 1 reply; 9+ messages in thread
From: Arnout Vandecappelle @ 2019-05-29 21:42 UTC (permalink / raw)
  To: buildroot

 Hi Jerzy,

On 29/05/2019 17:05, Jerzy Grzegorek wrote:

> @@ -302,23 +302,23 @@ comment "Graphic libraries"
>  	source "package/ocrad/Config.in"
>  	source "package/psplash/Config.in"
>  	source "package/sdl/Config.in"
> -	source "package/sdl_gfx/Config.in"
> -	source "package/sdl_image/Config.in"
> -	source "package/sdl_mixer/Config.in"
> -	source "package/sdl_net/Config.in"
> -	source "package/sdl_sound/Config.in"
> -	source "package/sdl_ttf/Config.in"
>  	source "package/sdl2/Config.in"
>  	source "package/sdl2_gfx/Config.in"
>  	source "package/sdl2_image/Config.in"
>  	source "package/sdl2_mixer/Config.in"
>  	source "package/sdl2_net/Config.in"
>  	source "package/sdl2_ttf/Config.in"
> +	source "package/sdl_gfx/Config.in"
> +	source "package/sdl_image/Config.in"
> +	source "package/sdl_mixer/Config.in"
> +	source "package/sdl_net/Config.in"
> +	source "package/sdl_sound/Config.in"
> +	source "package/sdl_ttf/Config.in"

 Although I think it's great to check the alphabetical ordering in
check_package, I think that the menus should still follow a logical ordering. It
makes more sense for all sdl packages to come before sdl2, an not:

sdl
sdl2
sdl2_bar
sdl2_foo
sdl_bar
sdl_foo

like you made it now.

 Therefore, I think you should just put exclusion markers on those lines.

>  	source "package/tk/Config.in"
>  
>  comment "Other GUIs"
> -	source "package/qt5/Config.in"
>  	source "package/kf5/Config.in"
> +	source "package/qt5/Config.in"

 Here also: kf5 depends on qt5, so it's very unnatural to swap the order.

>  if BR2_PACKAGE_QT5

 Actually, it would make sense for kf5 to move under this condition, which would
solve the issue :-)

>  comment "QT libraries and helper libraries"
>  	source "package/cutelyst/Config.in"
[snip]
> @@ -1845,8 +1845,8 @@ menu "Networking applications"
>  	source "package/bind/Config.in"
>  	source "package/bird/Config.in"
>  	source "package/bluez-tools/Config.in"
> -	source "package/bluez_utils/Config.in"
>  	source "package/bluez5_utils/Config.in"
> +	source "package/bluez_utils/Config.in"

 And again, putting bluez after bluez5 is weird.


 Also, this patch should be first, the one that updates checkpackagelib second.
This makes it clear that the checkpackagelib update shouldn't be applied until
package/Config.in is fixed.

 Regards,
 Arnout

>  	source "package/bmon/Config.in"
>  	source "package/boa/Config.in"
>  	source "package/boinc/Config.in"
> @@ -1956,13 +1956,13 @@ menu "Networking applications"
>  	source "package/nbd/Config.in"
>  	source "package/ncftp/Config.in"
>  	source "package/ndisc6/Config.in"
> +	source "package/net-tools/Config.in"
>  	source "package/netatalk/Config.in"
>  	source "package/netcat/Config.in"
>  	source "package/netcat-openbsd/Config.in"
>  	source "package/netplug/Config.in"
>  	source "package/netsnmp/Config.in"
>  	source "package/netstat-nat/Config.in"
> -	source "package/net-tools/Config.in"
>  	source "package/network-manager/Config.in"
>  	source "package/nfacct/Config.in"
>  	source "package/nftables/Config.in"
> 

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order
  2019-05-29 21:42   ` Arnout Vandecappelle
@ 2019-05-30  6:37     ` Jerzy Grzegorek
  0 siblings, 0 replies; 9+ messages in thread
From: Jerzy Grzegorek @ 2019-05-30  6:37 UTC (permalink / raw)
  To: buildroot

Hi Arnout,


>   Hi Jerzy,
>
> On 29/05/2019 17:05, Jerzy Grzegorek wrote:
>
>> @@ -302,23 +302,23 @@ comment "Graphic libraries"
>>   	source "package/ocrad/Config.in"
>>   	source "package/psplash/Config.in"
>>   	source "package/sdl/Config.in"
>> -	source "package/sdl_gfx/Config.in"
>> -	source "package/sdl_image/Config.in"
>> -	source "package/sdl_mixer/Config.in"
>> -	source "package/sdl_net/Config.in"
>> -	source "package/sdl_sound/Config.in"
>> -	source "package/sdl_ttf/Config.in"
>>   	source "package/sdl2/Config.in"
>>   	source "package/sdl2_gfx/Config.in"
>>   	source "package/sdl2_image/Config.in"
>>   	source "package/sdl2_mixer/Config.in"
>>   	source "package/sdl2_net/Config.in"
>>   	source "package/sdl2_ttf/Config.in"
>> +	source "package/sdl_gfx/Config.in"
>> +	source "package/sdl_image/Config.in"
>> +	source "package/sdl_mixer/Config.in"
>> +	source "package/sdl_net/Config.in"
>> +	source "package/sdl_sound/Config.in"
>> +	source "package/sdl_ttf/Config.in"
>   Although I think it's great to check the alphabetical ordering in
> check_package, I think that the menus should still follow a logical ordering. It
> makes more sense for all sdl packages to come before sdl2, an not:
>
> sdl
> sdl2
> sdl2_bar
> sdl2_foo
> sdl_bar
> sdl_foo
>
> like you made it now.
>
>   Therefore, I think you should just put exclusion markers on those lines.
>
>>   	source "package/tk/Config.in"
>>   
>>   comment "Other GUIs"
>> -	source "package/qt5/Config.in"
>>   	source "package/kf5/Config.in"
>> +	source "package/qt5/Config.in"
>   Here also: kf5 depends on qt5, so it's very unnatural to swap the order.
>
>>   if BR2_PACKAGE_QT5
>   Actually, it would make sense for kf5 to move under this condition, which would
> solve the issue :-)
>
>>   comment "QT libraries and helper libraries"
>>   	source "package/cutelyst/Config.in"
> [snip]
>> @@ -1845,8 +1845,8 @@ menu "Networking applications"
>>   	source "package/bind/Config.in"
>>   	source "package/bird/Config.in"
>>   	source "package/bluez-tools/Config.in"
>> -	source "package/bluez_utils/Config.in"
>>   	source "package/bluez5_utils/Config.in"
>> +	source "package/bluez_utils/Config.in"
>   And again, putting bluez after bluez5 is weird.
>
>
>   Also, this patch should be first, the one that updates checkpackagelib second.
> This makes it clear that the checkpackagelib update shouldn't be applied until
> package/Config.in is fixed.


Thanks for your review and the pointers. I'll take them into consideration.
Before respin I'll wait a couple of days for other comments.

Regards,
Jerzy


>
>   Regards,
>   Arnout
>
>>   	source "package/bmon/Config.in"
>>   	source "package/boa/Config.in"
>>   	source "package/boinc/Config.in"
>> @@ -1956,13 +1956,13 @@ menu "Networking applications"
>>   	source "package/nbd/Config.in"
>>   	source "package/ncftp/Config.in"
>>   	source "package/ndisc6/Config.in"
>> +	source "package/net-tools/Config.in"
>>   	source "package/netatalk/Config.in"
>>   	source "package/netcat/Config.in"
>>   	source "package/netcat-openbsd/Config.in"
>>   	source "package/netplug/Config.in"
>>   	source "package/netsnmp/Config.in"
>>   	source "package/netstat-nat/Config.in"
>> -	source "package/net-tools/Config.in"
>>   	source "package/network-manager/Config.in"
>>   	source "package/nfacct/Config.in"
>>   	source "package/nftables/Config.in"
>>

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2019-05-30  6:37 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-13  8:18 [Buildroot] [PATCH 1/2] package/Config.in: fix menu alphabetical order Jerzy Grzegorek
2014-12-13  8:18 ` [Buildroot] [PATCH 2/2] package/Config.in: fix " Jerzy Grzegorek
2014-12-16 17:46   ` Yann E. MORIN
2014-12-16 17:45 ` [Buildroot] [PATCH 1/2] package/Config.in: fix menu " Yann E. MORIN
2014-12-16 22:46 ` Peter Korsgaard
  -- strict thread matches above, loose matches on Subject: below --
2015-10-10  7:39 [Buildroot] [PATCH 1/2] package/Config.in: correct capitalization of menu items Jerzy Grzegorek
2015-10-10  7:39 ` [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order Jerzy Grzegorek
2019-05-29 15:05 [Buildroot] [PATCH 1/2] checkpackagelib/lib_config.py: check packages alphabetical order in {Config.in, Config.in.host} Jerzy Grzegorek
2019-05-29 15:05 ` [Buildroot] [PATCH 2/2] package/Config.in: fix alphabetical order Jerzy Grzegorek
2019-05-29 21:42   ` Arnout Vandecappelle
2019-05-30  6:37     ` Jerzy Grzegorek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox