* [OE-core][PATCH] vim: add option to disable NLS support
@ 2021-08-23 10:12 Andrej Valek
2021-08-23 11:23 ` [OE-core][PATCH v2] busybox: 1.33.1 -> 1.34.0 Andrej Valek
` (2 more replies)
0 siblings, 3 replies; 15+ messages in thread
From: Andrej Valek @ 2021-08-23 10:12 UTC (permalink / raw)
To: openembedded-core; +Cc: Andrej Valek
- Some distributions with UTF-8 locale have problem when National Language
Support is enabled. Add there an option to disable it.
- refresh options based on configure.ac
Signed-off-by: Andrej Valek <andrej.valek@siemens.com>
---
meta/recipes-support/vim/vim.inc | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes-support/vim/vim.inc
index 17d1c24a7c..7cc47884f2 100644
--- a/meta/recipes-support/vim/vim.inc
+++ b/meta/recipes-support/vim/vim.inc
@@ -54,19 +54,21 @@ do_compile() {
autotools_do_compile
}
-#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny
+#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny, selinux, elfutils, nls
PACKAGECONFIG ??= ""
PACKAGECONFIG += " \
${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', d)} \
+ nls \
"
PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3"
-PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl,"
+PACKAGECONFIG[acl] = "enable_acl="yes",--disable-acl,acl,"
PACKAGECONFIG[x11] = "--with-x,--without-x,xt,"
PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,,"
-PACKAGECONFIG[selinux] = "--enable-selinux,--disable-selinux,libselinux,"
+PACKAGECONFIG[selinux] = "enable_selinux="yes",--disable-selinux,libselinux,"
PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils,"
+PACKAGECONFIG[nls] = "enable_nls="yes",--disable-nls,,"
EXTRA_OECONF = " \
--disable-gpm \
--
2.11.0
^ permalink raw reply related [flat|nested] 15+ messages in thread* [OE-core][PATCH v2] busybox: 1.33.1 -> 1.34.0 2021-08-23 10:12 [OE-core][PATCH] vim: add option to disable NLS support Andrej Valek @ 2021-08-23 11:23 ` Andrej Valek 2021-08-23 16:12 ` [OE-core][PATCH] vim: add option to disable NLS support Khem Raj 2021-08-26 13:15 ` [OE-core][PATCH v2] " Andrej Valek 2 siblings, 0 replies; 15+ messages in thread From: Andrej Valek @ 2021-08-23 11:23 UTC (permalink / raw) To: openembedded-core; +Cc: Andrej Valek - update to next stable version 1.34.0 - refresh defconfig - remove and refresh already merged patches Signed-off-by: Andrej Valek <andrej.valek@siemens.com> --- ...inittab_1.33.0.bb => busybox-inittab_1.34.0.bb} | 0 ...iles-Use-C-locale-when-calling-sed-on-glo.patch | 28 ------ ...-testsuite-check-uudecode-before-using-it.patch | 6 +- .../busybox/busybox-udhcpc-no_deconfig.patch | 102 +++++---------------- meta/recipes-core/busybox/busybox/defconfig | 12 ++- .../{busybox_1.33.1.bb => busybox_1.34.0.bb} | 3 +- 6 files changed, 36 insertions(+), 115 deletions(-) rename meta/recipes-core/busybox/{busybox-inittab_1.33.0.bb => busybox-inittab_1.34.0.bb} (100%) delete mode 100644 meta/recipes-core/busybox/busybox/0001-gen_build_files-Use-C-locale-when-calling-sed-on-glo.patch rename meta/recipes-core/busybox/{busybox_1.33.1.bb => busybox_1.34.0.bb} (92%) diff --git a/meta/recipes-core/busybox/busybox-inittab_1.33.0.bb b/meta/recipes-core/busybox/busybox-inittab_1.34.0.bb similarity index 100% rename from meta/recipes-core/busybox/busybox-inittab_1.33.0.bb rename to meta/recipes-core/busybox/busybox-inittab_1.34.0.bb diff --git a/meta/recipes-core/busybox/busybox/0001-gen_build_files-Use-C-locale-when-calling-sed-on-glo.patch b/meta/recipes-core/busybox/busybox/0001-gen_build_files-Use-C-locale-when-calling-sed-on-glo.patch deleted file mode 100644 index e0a22c5bb3..0000000000 --- a/meta/recipes-core/busybox/busybox/0001-gen_build_files-Use-C-locale-when-calling-sed-on-glo.patch +++ /dev/null @@ -1,28 +0,0 @@ -From bff7f16f7f41de8df67beb03722f235828ef2249 Mon Sep 17 00:00:00 2001 -From: Khem Raj <raj.khem@gmail.com> -Date: Mon, 3 May 2021 15:48:19 -0700 -Subject: [PATCH] gen_build_files: Use C locale when calling sed on globbed files - -sort order is different based on chosen locale and also default shell -being bash or dash - -This sets the environment variable LC_ALL to the value C, which will -enforce bytewise sorting, irrespective of the shell - -Upstream-Status: Pending -Signed-off-by: Khem Raj <raj.khem@gmail.com> ---- - scripts/gen_build_files.sh | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - ---- a/scripts/gen_build_files.sh -+++ b/scripts/gen_build_files.sh -@@ -4,6 +4,8 @@ - # but users complain that many sed implementations - # are misinterpreting --. - -+export LC_ALL=C -+ - test $# -ge 2 || { echo "Syntax: $0 SRCTREE OBJTREE"; exit 1; } - - # cd to objtree diff --git a/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch b/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch index 25472f0bbd..d4bda3c12f 100644 --- a/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch +++ b/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch @@ -18,13 +18,13 @@ diff --git a/testsuite/tar.tests b/testsuite/tar.tests index d71a349..8c88567 100755 --- a/testsuite/tar.tests +++ b/testsuite/tar.tests -@@ -336,7 +336,7 @@ SKIP= - cd .. || exit 1; rm -rf tar.tempdir 2>/dev/null +@@ -339,7 +339,7 @@ cd .. || exit 1; rm -rf tar.tempdir 2>/d + fi mkdir tar.tempdir && cd tar.tempdir || exit 1 -optional FEATURE_SEAMLESS_BZ2 FEATURE_TAR_AUTODETECT LS +optional FEATURE_SEAMLESS_BZ2 FEATURE_TAR_AUTODETECT LS UUDECODE - testing "Symlink attack: create symlink and then write through it" '\ + testing "tar Symlink attack: create symlink and then write through it" '\ exec 2>&1 uudecode -o input && tar xvf input; echo $? diff --git a/testsuite/unzip.tests b/testsuite/unzip.tests diff --git a/meta/recipes-core/busybox/busybox/busybox-udhcpc-no_deconfig.patch b/meta/recipes-core/busybox/busybox/busybox-udhcpc-no_deconfig.patch index 35e981d6a2..948932a3e8 100644 --- a/meta/recipes-core/busybox/busybox/busybox-udhcpc-no_deconfig.patch +++ b/meta/recipes-core/busybox/busybox/busybox-udhcpc-no_deconfig.patch @@ -31,11 +31,11 @@ Signed-off-by: Andreas Oberritter <obi@opendreambox.org> networking/udhcp/dhcpc.c | 29 ++++++++++++++++------ 1 file changed, 21 insertions(+), 8 deletions(-) -Index: busybox-1.32.0/networking/udhcp/dhcpc.c +Index: busybox-1.34.0/networking/udhcp/dhcpc.c =================================================================== ---- busybox-1.32.0.orig/networking/udhcp/dhcpc.c -+++ busybox-1.32.0/networking/udhcp/dhcpc.c -@@ -48,6 +48,8 @@ struct tpacket_auxdata { +--- busybox-1.34.0.orig/networking/udhcp/dhcpc.c ++++ busybox-1.34.0/networking/udhcp/dhcpc.c +@@ -48,6 +48,8 @@ }; #endif @@ -44,47 +44,37 @@ Index: busybox-1.32.0/networking/udhcp/dhcpc.c /* "struct client_data_t client_data" is in bb_common_bufsiz1 */ -@@ -103,8 +105,10 @@ - OPT_x = 1 << 18, - OPT_f = 1 << 19, - OPT_B = 1 << 20, -+ OPT_D = 1 << 21, +@@ -100,8 +102,10 @@ + OPT_x = 1 << 16, + OPT_f = 1 << 17, + OPT_B = 1 << 18, ++ OPT_D = 1 << 19, /* The rest has variable bit positions, need to be clever */ - OPTBIT_B = 20, -+ OPTBIT_D = 21, + OPTBIT_B = 18, ++ OPTBIT_D = 19, USE_FOR_MMU( OPTBIT_b,) IF_FEATURE_UDHCPC_ARPING(OPTBIT_a,) IF_FEATURE_UDHCP_PORT( OPTBIT_P,) -@@ -1084,7 +1088,8 @@ - client_data.state = RENEW_REQUESTED; - break; - case RENEW_REQUESTED: /* impatient are we? fine, square 1 */ -- udhcp_run_script(NULL, "deconfig"); -+ if (allow_deconfig) -+ udhcp_run_script(NULL, "deconfig"); - case REQUESTING: - case RELEASED: - change_listen_mode(LISTEN_RAW); -@@ -1120,7 +1125,8 @@ static void perform_release(uint32_t server_addr, uint32_t requested_ip) - * Users requested to be notified in all cases, even if not in one - * of the states above. - */ -- udhcp_run_script(NULL, "deconfig"); +@@ -587,7 +591,8 @@ + + static void d4_run_script_deconfig(void) + { +- d4_run_script(NULL, "deconfig"); + if (allow_deconfig) -+ udhcp_run_script(NULL, "deconfig"); ++ d4_run_script(NULL, "deconfig"); + } - change_listen_mode(LISTEN_NONE); - client_data.state = RELEASED; -@@ -1238,7 +1244,7 @@ + /*** Sending/receiving packets ***/ +@@ -1244,7 +1249,7 @@ /* Parse command line */ opt = getopt32long(argv, "^" /* O,x: list; -T,-t,-A take numeric param */ -- "CV:H:h:F:i:np:qRr:s:T:+t:+SA:+O:*ox:*fB" -+ "CV:H:h:F:i:np:qRr:s:T:+t:+SA:+O:*ox:*fBD" +- "CV:F:i:np:qRr:s:T:+t:+SA:+O:*ox:*fB" ++ "CV:F:i:np:qRr:s:T:+t:+SA:+O:*ox:*fBD" USE_FOR_MMU("b") IF_FEATURE_UDHCPC_ARPING("a::") IF_FEATURE_UDHCP_PORT("P:") -@@ -1349,6 +1355,10 @@ +@@ -1361,6 +1366,10 @@ logmode |= LOGMODE_SYSLOG; } @@ -95,49 +85,3 @@ Index: busybox-1.32.0/networking/udhcp/dhcpc.c /* Create pidfile */ write_pidfile(client_data.pidfile); /* Goes to stdout (unless NOMMU) and possibly syslog */ -@@ -1357,7 +1367,8 @@ - srand(monotonic_us()); - - client_data.state = INIT_SELECTING; -- udhcp_run_script(NULL, "deconfig"); -+ if (allow_deconfig) -+ udhcp_run_script(NULL, "deconfig"); - change_listen_mode(LISTEN_RAW); - packet_num = 0; - timeout = 0; -@@ -1530,7 +1541,8 @@ - } - /* Timed out, enter init state */ - bb_simple_info_msg("lease lost, entering init state"); -- udhcp_run_script(NULL, "deconfig"); -+ if (allow_deconfig) -+ udhcp_run_script(NULL, "deconfig"); - client_data.state = INIT_SELECTING; - client_data.first_secs = 0; /* make secs field count from 0 */ - /*timeout = 0; - already is */ -@@ -1722,8 +1734,10 @@ - "(got ARP reply), declining"); - send_decline(/*xid,*/ server_addr, packet.yiaddr); - -- if (client_data.state != REQUESTING) -- udhcp_run_script(NULL, "deconfig"); -+ if (client_data.state != REQUESTING) { -+ if (allow_deconfig) -+ udhcp_run_script(NULL, "deconfig"); -+ } - change_listen_mode(LISTEN_RAW); - client_data.state = INIT_SELECTING; - client_data.first_secs = 0; /* make secs field count from 0 */ -@@ -1792,8 +1806,10 @@ - /* return to init state */ - bb_info_msg("received %s", "DHCP NAK"); - udhcp_run_script(&packet, "nak"); -- if (client_data.state != REQUESTING) -- udhcp_run_script(NULL, "deconfig"); -+ if (client_data.state != REQUESTING) { -+ if (allow_deconfig) -+ udhcp_run_script(NULL, "deconfig"); -+ } - change_listen_mode(LISTEN_RAW); - sleep(3); /* avoid excessive network traffic */ - client_data.state = INIT_SELECTING; diff --git a/meta/recipes-core/busybox/busybox/defconfig b/meta/recipes-core/busybox/busybox/defconfig index 701d48d625..16c61a84b2 100644 --- a/meta/recipes-core/busybox/busybox/defconfig +++ b/meta/recipes-core/busybox/busybox/defconfig @@ -1,7 +1,7 @@ # # Automatically generated make config: don't edit -# Busybox version: 1.33.0 -# Wed Mar 10 13:29:25 2021 +# Busybox version: 1.34.0 +# Wed Aug 23 09:07:25 2021 # CONFIG_HAVE_DOT_CONFIG=y @@ -207,11 +207,13 @@ CONFIG_CHOWN=y # CONFIG_FEATURE_CHOWN_LONG_OPTIONS is not set CONFIG_CHROOT=y # CONFIG_CKSUM is not set +CONFIG_CRC32=y # CONFIG_COMM is not set CONFIG_CP=y # CONFIG_FEATURE_CP_LONG_OPTIONS is not set # CONFIG_FEATURE_CP_REFLINK is not set CONFIG_CUT=y +CONFIG_FEATURE_CUT_REGEX=y CONFIG_DATE=y # CONFIG_FEATURE_DATE_ISOFMT is not set # CONFIG_FEATURE_DATE_NANO is not set @@ -316,7 +318,6 @@ CONFIG_TEST2=y CONFIG_FEATURE_TEST_64=y # CONFIG_TIMEOUT is not set CONFIG_TOUCH=y -CONFIG_FEATURE_TOUCH_NODEREF=y CONFIG_FEATURE_TOUCH_SUSV3=y CONFIG_TR=y CONFIG_FEATURE_TR_CLASSES=y @@ -424,6 +425,7 @@ CONFIG_VI=y CONFIG_FEATURE_VI_MAX_LEN=1024 CONFIG_FEATURE_VI_8BIT=y CONFIG_FEATURE_VI_COLON=y +CONFIG_FEATURE_VI_COLON_EXPAND=y CONFIG_FEATURE_VI_YANKMARK=y CONFIG_FEATURE_VI_SEARCH=y # CONFIG_FEATURE_VI_REGEX_SEARCH is not set @@ -437,6 +439,7 @@ CONFIG_FEATURE_VI_WIN_RESIZE=y CONFIG_FEATURE_VI_UNDO=y CONFIG_FEATURE_VI_UNDO_QUEUE=y CONFIG_FEATURE_VI_UNDO_QUEUE_MAX=256 +CONFIG_FEATURE_VI_VERBOSE_STATUS=y CONFIG_FEATURE_ALLOW_EXEC=y # @@ -743,6 +746,7 @@ CONFIG_FEATURE_VOLUMEID_SQUASHFS=y # Miscellaneous Utilities # # CONFIG_ADJTIMEX is not set +CONFIG_ASCII=y # CONFIG_BBCONFIG is not set # CONFIG_FEATURE_COMPRESS_BBCONFIG is not set # CONFIG_BC is not set @@ -838,6 +842,7 @@ CONFIG_TS=y # CONFIG_UBIRENAME is not set # CONFIG_VOLNAME is not set # CONFIG_WATCHDOG is not set +# CONFIG_FEATURE_WATCHDOG_OPEN_TWICE is not set # # Networking Utilities @@ -976,6 +981,7 @@ CONFIG_TRACEROUTE=y CONFIG_WGET=y CONFIG_FEATURE_WGET_LONG_OPTIONS=y CONFIG_FEATURE_WGET_STATUSBAR=y +CONFIG_FEATURE_WGET_FTP=y CONFIG_FEATURE_WGET_AUTHENTICATION=y CONFIG_FEATURE_WGET_TIMEOUT=y CONFIG_FEATURE_WGET_HTTPS=y diff --git a/meta/recipes-core/busybox/busybox_1.33.1.bb b/meta/recipes-core/busybox/busybox_1.34.0.bb similarity index 92% rename from meta/recipes-core/busybox/busybox_1.33.1.bb rename to meta/recipes-core/busybox/busybox_1.34.0.bb index 1e3e34e2a4..51df1df05f 100644 --- a/meta/recipes-core/busybox/busybox_1.33.1.bb +++ b/meta/recipes-core/busybox/busybox_1.34.0.bb @@ -47,9 +47,8 @@ SRC_URI = "https://busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \ file://0001-testsuite-use-www.example.org-for-wget-test-cases.patch \ file://0001-du-l-works-fix-to-use-145-instead-of-144.patch \ file://0001-sysctl-ignore-EIO-of-stable_secret-below-proc-sys-ne.patch \ - file://0001-gen_build_files-Use-C-locale-when-calling-sed-on-glo.patch \ file://0001-mktemp-add-tmpdir-option.patch \ " SRC_URI:append:libc-musl = " file://musl.cfg " -SRC_URI[tarball.sha256sum] = "12cec6bd2b16d8a9446dd16130f2b92982f1819f6e1c5f5887b6db03f5660d28" +SRC_URI[tarball.sha256sum] = "ec8d1615edb045b83b81966604759c4d4ac921434ab4011da604f629c06074ce" -- 2.11.0 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-23 10:12 [OE-core][PATCH] vim: add option to disable NLS support Andrej Valek 2021-08-23 11:23 ` [OE-core][PATCH v2] busybox: 1.33.1 -> 1.34.0 Andrej Valek @ 2021-08-23 16:12 ` Khem Raj 2021-08-23 17:12 ` Andrej Valek 2021-08-26 13:15 ` [OE-core][PATCH v2] " Andrej Valek 2 siblings, 1 reply; 15+ messages in thread From: Khem Raj @ 2021-08-23 16:12 UTC (permalink / raw) To: Andrej Valek, openembedded-core On 8/23/21 3:12 AM, Andrej Valek wrote: > - Some distributions with UTF-8 locale have problem when National Language > Support is enabled. Add there an option to disable it. > - refresh options based on configure.ac > > Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > --- > meta/recipes-support/vim/vim.inc | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes-support/vim/vim.inc > index 17d1c24a7c..7cc47884f2 100644 > --- a/meta/recipes-support/vim/vim.inc > +++ b/meta/recipes-support/vim/vim.inc > @@ -54,19 +54,21 @@ do_compile() { > autotools_do_compile > } > > -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny, selinux, elfutils, nls > PACKAGECONFIG ??= "" > PACKAGECONFIG += " \ > ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ > ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', d)} \ > + nls \ > " > > PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" > -PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl," > +PACKAGECONFIG[acl] = "enable_acl="yes",--disable-acl,acl," is 'yes' needed to be explicit ? I thought --enable-XYZ meant it implicitly > PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," > -PACKAGECONFIG[selinux] = "--enable-selinux,--disable-selinux,libselinux," > +PACKAGECONFIG[selinux] = "enable_selinux="yes",--disable-selinux,libselinux," > PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > +PACKAGECONFIG[nls] = "enable_nls="yes",--disable-nls,," > > EXTRA_OECONF = " \ > --disable-gpm \ > > > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-23 16:12 ` [OE-core][PATCH] vim: add option to disable NLS support Khem Raj @ 2021-08-23 17:12 ` Andrej Valek 2021-08-23 22:14 ` Andre McCurdy 0 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-08-23 17:12 UTC (permalink / raw) To: Khem Raj, openembedded-core Hello Khem, I looked exactly into configure.ac which arguments are expecting for those options. So I think, it has to be mentioned explicitly. Regards, Andrej > On 8/23/21 3:12 AM, Andrej Valek wrote: >> - Some distributions with UTF-8 locale have problem when National Language >> Support is enabled. Add there an option to disable it. >> - refresh options based on configure.ac >> >> Signed-off-by: Andrej Valek <andrej.valek@siemens.com> >> --- >> meta/recipes-support/vim/vim.inc | 8 +++++--- >> 1 file changed, 5 insertions(+), 3 deletions(-) >> >> diff --git a/meta/recipes-support/vim/vim.inc >> b/meta/recipes-support/vim/vim.inc >> index 17d1c24a7c..7cc47884f2 100644 >> --- a/meta/recipes-support/vim/vim.inc >> +++ b/meta/recipes-support/vim/vim.inc >> @@ -54,19 +54,21 @@ do_compile() { >> autotools_do_compile >> } >> >> -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny >> +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny, selinux, >> +elfutils, nls >> PACKAGECONFIG ??= "" >> PACKAGECONFIG += " \ >> ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ >> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', >> d)} \ >> + nls \ >> " >> >> PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" >> -PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl," >> +PACKAGECONFIG[acl] = "enable_acl="yes",--disable-acl,acl," > >is 'yes' needed to be explicit ? I thought --enable-XYZ meant it implicitly > >> PACKAGECONFIG[x11] = "--with-x,--without-x,xt," >> PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," >> -PACKAGECONFIG[selinux] = "--enable-selinux,--disable-selinux,libselinux," >> +PACKAGECONFIG[selinux] = "enable_selinux="yes",--disable-selinux,libselinux," >> PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," >> +PACKAGECONFIG[nls] = "enable_nls="yes",--disable-nls,," >> >> EXTRA_OECONF = " \ >> --disable-gpm \ ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-23 17:12 ` Andrej Valek @ 2021-08-23 22:14 ` Andre McCurdy 2021-08-24 6:56 ` Andrej Valek 0 siblings, 1 reply; 15+ messages in thread From: Andre McCurdy @ 2021-08-23 22:14 UTC (permalink / raw) To: Andrej Valek; +Cc: Khem Raj, openembedded-core On Mon, Aug 23, 2021 at 10:12 AM Andrej Valek <andrej.valek@siemens.com> wrote: > > Hello Khem, > > I looked exactly into configure.ac which arguments are expecting for those options. So I think, it has to be mentioned explicitly. Assuming configure.ac is based around AC_ARG_ENABLE / AC_ARG_WITH then an explicit option is not required. A default value of "yes" will be set for --enable-foo / --with-foo and a default value of "no" will be set for --disable-foo / --without-foo. However, apart from that, you've also dropped the leaving "--" from various --enable-foo options and converted dashes to underscores, all of which looks wrong. How were the changes tested? > Regards, > Andrej > > > On 8/23/21 3:12 AM, Andrej Valek wrote: > >> - Some distributions with UTF-8 locale have problem when National Language > >> Support is enabled. Add there an option to disable it. > >> - refresh options based on configure.ac > >> > >> Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > >> --- > >> meta/recipes-support/vim/vim.inc | 8 +++++--- > >> 1 file changed, 5 insertions(+), 3 deletions(-) > >> > >> diff --git a/meta/recipes-support/vim/vim.inc > >> b/meta/recipes-support/vim/vim.inc > >> index 17d1c24a7c..7cc47884f2 100644 > >> --- a/meta/recipes-support/vim/vim.inc > >> +++ b/meta/recipes-support/vim/vim.inc > >> @@ -54,19 +54,21 @@ do_compile() { > >> autotools_do_compile > >> } > >> > >> -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > >> +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny, selinux, > >> +elfutils, nls > >> PACKAGECONFIG ??= "" > >> PACKAGECONFIG += " \ > >> ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ > >> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', > >> d)} \ > >> + nls \ > >> " > >> > >> PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" > >> -PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl," > >> +PACKAGECONFIG[acl] = "enable_acl="yes",--disable-acl,acl," > > > >is 'yes' needed to be explicit ? I thought --enable-XYZ meant it implicitly > > > >> PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > >> PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," > >> -PACKAGECONFIG[selinux] = "--enable-selinux,--disable-selinux,libselinux," > >> +PACKAGECONFIG[selinux] = "enable_selinux="yes",--disable-selinux,libselinux," > >> PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > >> +PACKAGECONFIG[nls] = "enable_nls="yes",--disable-nls,," > >> > >> EXTRA_OECONF = " \ > >> --disable-gpm \ > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-23 22:14 ` Andre McCurdy @ 2021-08-24 6:56 ` Andrej Valek 2021-08-26 9:58 ` Andrej Valek 0 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-08-24 6:56 UTC (permalink / raw) To: Andre McCurdy; +Cc: Khem Raj, openembedded-core Hi all, Of course, that I tried it and it works. ... configure.ac:4517: You should run autoupdate. NOTE: Running ./configure --build=x86_64-linux --host=x86_64-poky-linux ... --disable-static enable_acl=yes --enable-gui=gtk3 enable_nls=yes --disable-selinux --with-features=big --with-x configure: WARNING: unrecognized options: --disable-silent-rules, --disable-dependency-tracking, --with-libtool-sysroot, --disable-static configure: loading site script /home/projects/yocto/qemux86-64/layers/build/meta/site/endian-little configure: loading site script /home/projects/yocto/qemux86-64/layers/build/meta/site/common-linux ... checking how to run man with a section nr... man checking --disable-nls argument... no checking for msgfmt... msgfmt checking for NLS... gettext() works checking for bind_textdomain_codeset... yes ... So does it mean, that I should use the --enable instead anyway? Andrej > On Mon, Aug 23, 2021 at 10:12 AM Andrej Valek <andrej.valek@siemens.com> wrote: >> >> Hello Khem, >> >> I looked exactly into configure.ac which arguments are expecting for those options. So I think, it has to be mentioned explicitly. > > Assuming configure.ac is based around AC_ARG_ENABLE / AC_ARG_WITH then an explicit option is not required. A default value of "yes" will be set for --enable-foo / --with-foo and a default value of "no" will be set for --disable-foo / --without-foo. > > However, apart from that, you've also dropped the leaving "--" from various --enable-foo options and converted dashes to underscores, all of which looks wrong. How were the changes tested? > >> Regards, >> Andrej >> >> > On 8/23/21 3:12 AM, Andrej Valek wrote: >> >> - Some distributions with UTF-8 locale have problem when National Language >> >> Support is enabled. Add there an option to disable it. >> >> - refresh options based on configure.ac >> >> >> >> Signed-off-by: Andrej Valek <andrej.valek@siemens.com> >> >> --- >> >> meta/recipes-support/vim/vim.inc | 8 +++++--- >> >> 1 file changed, 5 insertions(+), 3 deletions(-) >> >> >> >> diff --git a/meta/recipes-support/vim/vim.inc >> >> b/meta/recipes-support/vim/vim.inc >> >> index 17d1c24a7c..7cc47884f2 100644 >> >> --- a/meta/recipes-support/vim/vim.inc >> >> +++ b/meta/recipes-support/vim/vim.inc >> >> @@ -54,19 +54,21 @@ do_compile() { >> >> autotools_do_compile >> >> } >> >> >> >> -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny >> >> +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny, >> >> +selinux, elfutils, nls >> >> PACKAGECONFIG ??= "" >> >> PACKAGECONFIG += " \ >> >> ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ >> >> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', >> >> '', d)} \ >> >> + nls \ >> >> " >> >> >> >> PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" >> >> -PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl," >> >> +PACKAGECONFIG[acl] = "enable_acl="yes",--disable-acl,acl," >> > >> >is 'yes' needed to be explicit ? I thought --enable-XYZ meant it >> >implicitly >> > >> >> PACKAGECONFIG[x11] = "--with-x,--without-x,xt," >> >> PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," >> >> -PACKAGECONFIG[selinux] = "--enable-selinux,--disable-selinux,libselinux," >> >> +PACKAGECONFIG[selinux] = "enable_selinux="yes",--disable-selinux,libselinux," >> >> PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," >> >> +PACKAGECONFIG[nls] = "enable_nls="yes",--disable-nls,," >> >> >> >> EXTRA_OECONF = " \ >> >> --disable-gpm \ >> >> >> ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-24 6:56 ` Andrej Valek @ 2021-08-26 9:58 ` Andrej Valek 2021-08-26 10:59 ` Richard Purdie 0 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-08-26 9:58 UTC (permalink / raw) To: armccurdy, steve; +Cc: openembedded-core, richard.purdie, raj.khem Hello, Is there any problem, which should I fix? I would like to include this feature into dunfell if possible. Thanks, Andrej On Tue, 2021-08-24 at 06:56 +0000, Valek, Andrej (ADV D EU SK SI-BP1) wrote: > Hi all, > > Of course, that I tried it and it works. > ... > configure.ac:4517: You should run autoupdate. > NOTE: Running ./configure --build=x86_64-linux --host=x86_64-poky- > linux ... --disable-static enable_acl=yes --enable-gui=gtk3 > enable_nls=yes --disable-selinux --with-features=big --with-x > configure: WARNING: unrecognized options: --disable-silent-rules, -- > disable-dependency-tracking, --with-libtool-sysroot, --disable-static > configure: loading site script /home/projects/yocto/qemux86- > 64/layers/build/meta/site/endian-little > configure: loading site script /home/projects/yocto/qemux86- > 64/layers/build/meta/site/common-linux > ... > checking how to run man with a section nr... man > checking --disable-nls argument... no > checking for msgfmt... msgfmt > checking for NLS... gettext() works > checking for bind_textdomain_codeset... yes > ... > > So does it mean, that I should use the --enable instead anyway? > > Andrej > > > On Mon, Aug 23, 2021 at 10:12 AM Andrej Valek > > <andrej.valek@siemens.com> wrote: > > > > > > Hello Khem, > > > > > > I looked exactly into configure.ac which arguments are expecting > > > for those options. So I think, it has to be mentioned explicitly. > > > > Assuming configure.ac is based around AC_ARG_ENABLE / AC_ARG_WITH > > then an explicit option is not required. A default value of "yes" > > will be set for --enable-foo / --with-foo and a default value of > > "no" will be set for --disable-foo / --without-foo. > > > > However, apart from that, you've also dropped the leaving "--" from > > various --enable-foo options and converted dashes to underscores, > > all of which looks wrong. How were the changes tested? > > > > > Regards, > > > Andrej > > > > > > > On 8/23/21 3:12 AM, Andrej Valek wrote: > > > > > - Some distributions with UTF-8 locale have problem when > > > > > National Language > > > > > Support is enabled. Add there an option to disable it. > > > > > - refresh options based on configure.ac > > > > > > > > > > Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > > > > > --- > > > > > meta/recipes-support/vim/vim.inc | 8 +++++--- > > > > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > > > > > > > diff --git a/meta/recipes-support/vim/vim.inc > > > > > b/meta/recipes-support/vim/vim.inc > > > > > index 17d1c24a7c..7cc47884f2 100644 > > > > > --- a/meta/recipes-support/vim/vim.inc > > > > > +++ b/meta/recipes-support/vim/vim.inc > > > > > @@ -54,19 +54,21 @@ do_compile() { > > > > > autotools_do_compile > > > > > } > > > > > > > > > > -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > > > > > +selinux, elfutils, nls > > > > > PACKAGECONFIG ??= "" > > > > > PACKAGECONFIG += " \ > > > > > ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', > > > > > d)} \ > > > > > ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 > > > > > gtkgui', > > > > > '', d)} \ > > > > > + nls \ > > > > > " > > > > > > > > > > PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable- > > > > > gui=no,gtk+3" > > > > > -PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl," > > > > > +PACKAGECONFIG[acl] = "enable_acl="yes",--disable-acl,acl," > > > > > > > > is 'yes' needed to be explicit ? I thought --enable-XYZ meant > > > > it > > > > implicitly > > > > > > > > > PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > > > > > PACKAGECONFIG[tiny] = "--with-features=tiny,--with- > > > > > features=big,," > > > > > -PACKAGECONFIG[selinux] = "--enable-selinux,--disable- > > > > > selinux,libselinux," > > > > > +PACKAGECONFIG[selinux] = "enable_selinux="yes",--disable- > > > > > selinux,libselinux," > > > > > PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > > > > > +PACKAGECONFIG[nls] = "enable_nls="yes",--disable-nls,," > > > > > > > > > > EXTRA_OECONF = " \ > > > > > --disable-gpm \ > > > > > > > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-26 9:58 ` Andrej Valek @ 2021-08-26 10:59 ` Richard Purdie 2021-08-26 11:24 ` Andrej Valek 0 siblings, 1 reply; 15+ messages in thread From: Richard Purdie @ 2021-08-26 10:59 UTC (permalink / raw) To: Valek, Andrej, armccurdy, steve; +Cc: openembedded-core, raj.khem On Thu, 2021-08-26 at 09:58 +0000, Valek, Andrej wrote: > Hello, > > Is there any problem, which should I fix? > I would like to include this feature into dunfell if possible. Autotools usually has a very specific option format and this isn't following it. I know others expressed some surprise and I share that. It can happen if the software is doing something unusual so it is possible but I wanted to go and double check this as it looks odd. It may be both formats can be used and if that were the case, I'd much prefer we used the standard format we use everywhere else. So basically this is held pending me doing some due diligence as it worries me. I am however struggling to do everything needed at the moment. The other issue is that nls is an option many other recipes have but we've always handled it at the core level in the gettext bbclass. native recipes explictly pass --disable-nls for example. I also wanted to check if the recipe uses the gextext class (or should?) and if instead we should be exposing the PACKAGECONFIG differently through that class. Cheers, Richard ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-26 10:59 ` Richard Purdie @ 2021-08-26 11:24 ` Andrej Valek 2021-08-26 12:00 ` Richard Purdie 0 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-08-26 11:24 UTC (permalink / raw) To: armccurdy, richard.purdie, steve; +Cc: openembedded-core, raj.khem Hello Richard, If it isn't a big deal, I can use the "--enable" variant. If we keep -- enable-nls like a default, we will brake nothing I guess. (It's enabled by default anyway). So I'm adding there only a feature to disable it with PACKAGEOPTION. Regards, Andrej On Thu, 2021-08-26 at 11:59 +0100, Richard Purdie wrote: > On Thu, 2021-08-26 at 09:58 +0000, Valek, Andrej wrote: > > Hello, > > > > Is there any problem, which should I fix? > > I would like to include this feature into dunfell if possible. > > Autotools usually has a very specific option format and this isn't > following it. > I know others expressed some surprise and I share that. It can happen > if the > software is doing something unusual so it is possible but I wanted to > go and > double check this as it looks odd. It may be both formats can be used > and if > that were the case, I'd much prefer we used the standard format we > use > everywhere else. > > So basically this is held pending me doing some due diligence as it > worries me. > I am however struggling to do everything needed at the moment. > > The other issue is that nls is an option many other recipes have but > we've > always handled it at the core level in the gettext bbclass. native > recipes > explictly pass --disable-nls for example. > > I also wanted to check if the recipe uses the gextext class (or > should?) and if > instead we should be exposing the PACKAGECONFIG differently through > that class. > > Cheers, > > Richard > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH] vim: add option to disable NLS support 2021-08-26 11:24 ` Andrej Valek @ 2021-08-26 12:00 ` Richard Purdie 0 siblings, 0 replies; 15+ messages in thread From: Richard Purdie @ 2021-08-26 12:00 UTC (permalink / raw) To: Valek, Andrej, armccurdy, steve; +Cc: openembedded-core, raj.khem On Thu, 2021-08-26 at 11:24 +0000, Valek, Andrej wrote: > Hello Richard, > > If it isn't a big deal, I can use the "--enable" variant. If we keep -- > enable-nls like a default, we will brake nothing I guess. (It's enabled > by default anyway). So I'm adding there only a feature to disable it > with PACKAGEOPTION. Yes, lets use --enable please since that is the way the rest of the world/code works. I'm a bit worried about the nls option since the recipe does have BBCLASSEXTEND for native/nativesdk and we do disable nls for native by default. As you say, the patch probably doesn't change anything but it is a bit of an oddity. The code does look to be using gettext and there is a gettext-native dependency but the recipe does not use the gettext class. Why? Not sure. I'll put the patch in for testing if we can use the standard format though (which should work since I checked and configure is using the standard autoconf macros). Cheers, Richard ^ permalink raw reply [flat|nested] 15+ messages in thread
* [OE-core][PATCH v2] vim: add option to disable NLS support 2021-08-23 10:12 [OE-core][PATCH] vim: add option to disable NLS support Andrej Valek 2021-08-23 11:23 ` [OE-core][PATCH v2] busybox: 1.33.1 -> 1.34.0 Andrej Valek 2021-08-23 16:12 ` [OE-core][PATCH] vim: add option to disable NLS support Khem Raj @ 2021-08-26 13:15 ` Andrej Valek 2021-09-01 19:04 ` Andrej Valek 2 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-08-26 13:15 UTC (permalink / raw) To: openembedded-core; +Cc: Andrej Valek - Some distributions with UTF-8 locale have problem when National Language Support is enabled. Add there an option to disable it. Signed-off-by: Andrej Valek <andrej.valek@siemens.com> --- meta/recipes-support/vim/vim.inc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes-support/vim/vim.inc index 17d1c24a7c..860fd24863 100644 --- a/meta/recipes-support/vim/vim.inc +++ b/meta/recipes-support/vim/vim.inc @@ -54,11 +54,12 @@ do_compile() { autotools_do_compile } -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny selinux, elfutils, nls PACKAGECONFIG ??= "" PACKAGECONFIG += " \ ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', d)} \ + nls \ " PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" @@ -67,6 +68,7 @@ PACKAGECONFIG[x11] = "--with-x,--without-x,xt," PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," PACKAGECONFIG[selinux] = "--enable-selinux,--disable-selinux,libselinux," PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," +PACKAGECONFIG[nls] = "--enable-nls,--disable-nls,," EXTRA_OECONF = " \ --disable-gpm \ -- 2.31.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH v2] vim: add option to disable NLS support 2021-08-26 13:15 ` [OE-core][PATCH v2] " Andrej Valek @ 2021-09-01 19:04 ` Andrej Valek 2021-09-01 22:35 ` Steve Sakoman 0 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-09-01 19:04 UTC (permalink / raw) To: steve; +Cc: richard.purdie, openembedded-core Hello Steve, Could you please cherry-pick this patch into dunfell branch too? Thank you, Andrej On Thu, 2021-08-26 at 15:15 +0200, Andrej Valek wrote: > - Some distributions with UTF-8 locale have problem when National > Language > Support is enabled. Add there an option to disable it. > > Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > --- > meta/recipes-support/vim/vim.inc | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes- > support/vim/vim.inc > index 17d1c24a7c..860fd24863 100644 > --- a/meta/recipes-support/vim/vim.inc > +++ b/meta/recipes-support/vim/vim.inc > @@ -54,11 +54,12 @@ do_compile() { > autotools_do_compile > } > > -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny selinux, > elfutils, nls > PACKAGECONFIG ??= "" > PACKAGECONFIG += " \ > ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ > ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', > d)} \ > + nls \ > " > > PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" > @@ -67,6 +68,7 @@ PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," > PACKAGECONFIG[selinux] = "--enable-selinux,--disable- > selinux,libselinux," > PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > +PACKAGECONFIG[nls] = "--enable-nls,--disable-nls,," > > EXTRA_OECONF = " \ > --disable-gpm \ ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH v2] vim: add option to disable NLS support 2021-09-01 19:04 ` Andrej Valek @ 2021-09-01 22:35 ` Steve Sakoman 2021-09-08 7:49 ` Andrej Valek 0 siblings, 1 reply; 15+ messages in thread From: Steve Sakoman @ 2021-09-01 22:35 UTC (permalink / raw) To: Andrej Valek; +Cc: richard.purdie, openembedded-core On Wed, Sep 1, 2021 at 9:04 AM Andrej Valek <andrej.valek@siemens.com> wrote: > > Hello Steve, > > Could you please cherry-pick this patch into dunfell branch too? OK, will add this to my test queue. Steve > > Thank you, > Andrej > > On Thu, 2021-08-26 at 15:15 +0200, Andrej Valek wrote: > > - Some distributions with UTF-8 locale have problem when National > > Language > > Support is enabled. Add there an option to disable it. > > > > Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > > --- > > meta/recipes-support/vim/vim.inc | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes- > > support/vim/vim.inc > > index 17d1c24a7c..860fd24863 100644 > > --- a/meta/recipes-support/vim/vim.inc > > +++ b/meta/recipes-support/vim/vim.inc > > @@ -54,11 +54,12 @@ do_compile() { > > autotools_do_compile > > } > > > > -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > > +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny selinux, > > elfutils, nls > > PACKAGECONFIG ??= "" > > PACKAGECONFIG += " \ > > ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ > > ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', '', > > d)} \ > > + nls \ > > " > > > > PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" > > @@ -67,6 +68,7 @@ PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > > PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," > > PACKAGECONFIG[selinux] = "--enable-selinux,--disable- > > selinux,libselinux," > > PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > > +PACKAGECONFIG[nls] = "--enable-nls,--disable-nls,," > > > > EXTRA_OECONF = " \ > > --disable-gpm \ > > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH v2] vim: add option to disable NLS support 2021-09-01 22:35 ` Steve Sakoman @ 2021-09-08 7:49 ` Andrej Valek 2021-09-08 14:26 ` Steve Sakoman 0 siblings, 1 reply; 15+ messages in thread From: Andrej Valek @ 2021-09-08 7:49 UTC (permalink / raw) To: sakoman; +Cc: richard.purdie, openembedded-core Hello Steve, What's the current status in this case? I see, that changes are in dunfell-nut branch http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=stable/dunfell-nut . But you bumped the 3.1.11 version already (http://git.yoctoproject.org/cgit/cgit.cgi/poky/log/?h=dunfell ) where the changes are not there. Regards, Andrej On Wed, 2021-09-01 at 12:35 -1000, Steve Sakoman wrote: > On Wed, Sep 1, 2021 at 9:04 AM Andrej Valek <andrej.valek@siemens.com> > wrote: > > > > Hello Steve, > > > > Could you please cherry-pick this patch into dunfell branch too? > > OK, will add this to my test queue. > > Steve > > > > > Thank you, > > Andrej > > > > On Thu, 2021-08-26 at 15:15 +0200, Andrej Valek wrote: > > > - Some distributions with UTF-8 locale have problem when National > > > Language > > > Support is enabled. Add there an option to disable it. > > > > > > Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > > > --- > > > meta/recipes-support/vim/vim.inc | 4 +++- > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes- > > > support/vim/vim.inc > > > index 17d1c24a7c..860fd24863 100644 > > > --- a/meta/recipes-support/vim/vim.inc > > > +++ b/meta/recipes-support/vim/vim.inc > > > @@ -54,11 +54,12 @@ do_compile() { > > > autotools_do_compile > > > } > > > > > > -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > > > +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > > > selinux, > > > elfutils, nls > > > PACKAGECONFIG ??= "" > > > PACKAGECONFIG += " \ > > > ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ > > > ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', > > > '', > > > d)} \ > > > + nls \ > > > " > > > > > > PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" > > > @@ -67,6 +68,7 @@ PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > > > PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," > > > PACKAGECONFIG[selinux] = "--enable-selinux,--disable- > > > selinux,libselinux," > > > PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > > > +PACKAGECONFIG[nls] = "--enable-nls,--disable-nls,," > > > > > > EXTRA_OECONF = " \ > > > --disable-gpm \ > > > > > > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [OE-core][PATCH v2] vim: add option to disable NLS support 2021-09-08 7:49 ` Andrej Valek @ 2021-09-08 14:26 ` Steve Sakoman 0 siblings, 0 replies; 15+ messages in thread From: Steve Sakoman @ 2021-09-08 14:26 UTC (permalink / raw) To: Andrej Valek; +Cc: sakoman, richard.purdie, openembedded-core On Tue, Sep 7, 2021 at 9:49 PM Andrej Valek <andrej.valek@siemens.com> wrote: > > Hello Steve, > > What's the current status in this case? It is in my most recent pull request for openembedded-core: https://lists.openembedded.org/g/openembedded-core/message/155775 > I see, that changes are in dunfell-nut branch > http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=stable/dunfell-nut Patches start there for testing, then move to stable/dunfell-next when ready for a pull request. > . But you bumped the 3.1.11 version already > (http://git.yoctoproject.org/cgit/cgit.cgi/poky/log/?h=dunfell ) where > the changes are not there. Richard took the version bump patches from yocto-docs and meta-yocto prior to the final openembedded-core patches. Not a problem, we plan to build and test 3.1.11 next week and will create the 3.1.11 tag after successful completion of testing. Steve > On Wed, 2021-09-01 at 12:35 -1000, Steve Sakoman wrote: > > On Wed, Sep 1, 2021 at 9:04 AM Andrej Valek <andrej.valek@siemens.com> > > wrote: > > > > > > Hello Steve, > > > > > > Could you please cherry-pick this patch into dunfell branch too? > > > > OK, will add this to my test queue. > > > > Steve > > > > > > > > Thank you, > > > Andrej > > > > > > On Thu, 2021-08-26 at 15:15 +0200, Andrej Valek wrote: > > > > - Some distributions with UTF-8 locale have problem when National > > > > Language > > > > Support is enabled. Add there an option to disable it. > > > > > > > > Signed-off-by: Andrej Valek <andrej.valek@siemens.com> > > > > --- > > > > meta/recipes-support/vim/vim.inc | 4 +++- > > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes- > > > > support/vim/vim.inc > > > > index 17d1c24a7c..860fd24863 100644 > > > > --- a/meta/recipes-support/vim/vim.inc > > > > +++ b/meta/recipes-support/vim/vim.inc > > > > @@ -54,11 +54,12 @@ do_compile() { > > > > autotools_do_compile > > > > } > > > > > > > > -#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > > > > +#Available PACKAGECONFIG options are gtkgui, acl, x11, tiny > > > > selinux, > > > > elfutils, nls > > > > PACKAGECONFIG ??= "" > > > > PACKAGECONFIG += " \ > > > > ${@bb.utils.filter('DISTRO_FEATURES', 'acl selinux', d)} \ > > > > ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 gtkgui', > > > > '', > > > > d)} \ > > > > + nls \ > > > > " > > > > > > > > PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3" > > > > @@ -67,6 +68,7 @@ PACKAGECONFIG[x11] = "--with-x,--without-x,xt," > > > > PACKAGECONFIG[tiny] = "--with-features=tiny,--with-features=big,," > > > > PACKAGECONFIG[selinux] = "--enable-selinux,--disable- > > > > selinux,libselinux," > > > > PACKAGECONFIG[elfutils] = "--enable-elf-check,,elfutils," > > > > +PACKAGECONFIG[nls] = "--enable-nls,--disable-nls,," > > > > > > > > EXTRA_OECONF = " \ > > > > --disable-gpm \ > > > > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2021-09-08 14:26 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-23 10:12 [OE-core][PATCH] vim: add option to disable NLS support Andrej Valek 2021-08-23 11:23 ` [OE-core][PATCH v2] busybox: 1.33.1 -> 1.34.0 Andrej Valek 2021-08-23 16:12 ` [OE-core][PATCH] vim: add option to disable NLS support Khem Raj 2021-08-23 17:12 ` Andrej Valek 2021-08-23 22:14 ` Andre McCurdy 2021-08-24 6:56 ` Andrej Valek 2021-08-26 9:58 ` Andrej Valek 2021-08-26 10:59 ` Richard Purdie 2021-08-26 11:24 ` Andrej Valek 2021-08-26 12:00 ` Richard Purdie 2021-08-26 13:15 ` [OE-core][PATCH v2] " Andrej Valek 2021-09-01 19:04 ` Andrej Valek 2021-09-01 22:35 ` Steve Sakoman 2021-09-08 7:49 ` Andrej Valek 2021-09-08 14:26 ` Steve Sakoman
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.