Hello, This breaks all the qemu*-alt build this way:
https://autobuilder.yoctoproject.org/typhoon/#/builders/109/builds/7875/steps/14/logs/stdio On 25/05/2024 13:03:48+0800, Chen Qi via lists.openembedded.org wrote: > From: Chen Qi <[email protected]> > > License-Update: > =============== > GPL-2.0 is changed to GPL-2.0-only in README.licensing. > > The following two patches are removed as they've been there in new release: > > 0001-login-utils-include-libgen.h-for-basename-API.patch > mit-license.patch > > Other existing patches are refreshed to avoid patch-fuzz error. > > A new PACKAGECONFIG, lastlog2, is added, which serves as a switch to build > Y2038 > safe lastlog pam module. This is a new functionality added in 2.40. > > This 2.40.1 version has the problem of unnecessary remake, causing > intermmittent errors at parallel make. The do_compile:prepend is a > workaround for this. Upstream issue and PR are: > https://github.com/util-linux/util-linux/issues/3061 > https://github.com/util-linux/util-linux/pull/3064 > > do_install is changed to do_install:append, the legacy codes are > removed. They've been there since the born of oe-core. I tested > with ccache, there's no error. > > Signed-off-by: Chen Qi <[email protected]> > --- > ...2.39.3.bb => util-linux-libuuid_2.40.1.bb} | 0 > meta/recipes-core/util-linux/util-linux.inc | 8 +-- > ...ls-include-libgen.h-for-basename-API.patch | 60 ------------------- > .../util-linux/avoid_parallel_tests.patch | 5 +- > .../util-linux/configure-sbindir.patch | 7 +-- > .../display_testname_for_subtest.patch | 7 +-- > .../util-linux/util-linux/mit-license.patch | 45 -------------- > .../util-linux/util-linux/ptest.patch | 11 ++-- > ...l-linux_2.39.3.bb => util-linux_2.40.1.bb} | 21 ++++--- > 9 files changed, 29 insertions(+), 135 deletions(-) > rename meta/recipes-core/util-linux/{util-linux-libuuid_2.39.3.bb => > util-linux-libuuid_2.40.1.bb} (100%) > delete mode 100644 > meta/recipes-core/util-linux/util-linux/0001-login-utils-include-libgen.h-for-basename-API.patch > delete mode 100644 meta/recipes-core/util-linux/util-linux/mit-license.patch > rename meta/recipes-core/util-linux/{util-linux_2.39.3.bb => > util-linux_2.40.1.bb} (95%) > > diff --git a/meta/recipes-core/util-linux/util-linux-libuuid_2.39.3.bb > b/meta/recipes-core/util-linux/util-linux-libuuid_2.40.1.bb > similarity index 100% > rename from meta/recipes-core/util-linux/util-linux-libuuid_2.39.3.bb > rename to meta/recipes-core/util-linux/util-linux-libuuid_2.40.1.bb > diff --git a/meta/recipes-core/util-linux/util-linux.inc > b/meta/recipes-core/util-linux/util-linux.inc > index d506783f9a..0f5f838e84 100644 > --- a/meta/recipes-core/util-linux/util-linux.inc > +++ b/meta/recipes-core/util-linux/util-linux.inc > @@ -6,7 +6,7 @@ disk partitioning, kernel message management, filesystem > creation, and system lo > > SECTION = "base" > > -LICENSE = "GPL-1.0-or-later & GPL-2.0-or-later & LGPL-2.1-or-later & > BSD-2-Clause & BSD-3-Clause & BSD-4-Clause & MIT" > +LICENSE = "GPL-1.0-or-later & GPL-2.0-only & GPL-2.0-or-later & > LGPL-2.1-or-later & BSD-2-Clause & BSD-3-Clause & BSD-4-Clause & MIT" > LICENSE:${PN}-fcntl-lock = "MIT" > LICENSE:${PN}-fdisk = "GPL-1.0-or-later" > LICENSE:${PN}-libblkid = "LGPL-2.1-or-later" > @@ -14,7 +14,7 @@ LICENSE:${PN}-libfdisk = "LGPL-2.1-or-later" > LICENSE:${PN}-libmount = "LGPL-2.1-or-later" > LICENSE:${PN}-libsmartcols = "LGPL-2.1-or-later" > > -LIC_FILES_CHKSUM = > "file://README.licensing;md5=cc80239f106687ab39ef0271ff5cf4ba \ > +LIC_FILES_CHKSUM = > "file://README.licensing;md5=ddd58b6c94da86ff4f03e91208eb9cfc \ > file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ > > file://Documentation/licenses/COPYING.GPL-2.0-or-later;md5=b234ee4d69f5fce4486a80fdaf4a4263 > \ > > file://Documentation/licenses/COPYING.LGPL-2.1-or-later;md5=4fbd65380cdd255951079008b364516c > \ > @@ -30,7 +30,6 @@ LIC_FILES_CHKSUM = > "file://README.licensing;md5=cc80239f106687ab39ef0271ff5cf4ba > FILESEXTRAPATHS:prepend := "${THISDIR}/util-linux:" > MAJOR_VERSION = "${@'.'.join(d.getVar('PV').split('.')[0:2])}" > SRC_URI = > "${KERNELORG_MIRROR}/linux/utils/util-linux/v${MAJOR_VERSION}/util-linux-${PV}.tar.xz > \ > - file://mit-license.patch \ > file://configure-sbindir.patch \ > file://runuser.pamd \ > file://runuser-l.pamd \ > @@ -38,8 +37,7 @@ SRC_URI = > "${KERNELORG_MIRROR}/linux/utils/util-linux/v${MAJOR_VERSION}/util-lin > file://run-ptest \ > file://display_testname_for_subtest.patch \ > file://avoid_parallel_tests.patch \ > - file://0001-login-utils-include-libgen.h-for-basename-API.patch \ > file://fcntl-lock.c \ > " > > -SRC_URI[sha256sum] = > "7b6605e48d1a49f43cc4b4cfc59f313d0dd5402fa40b96810bd572e167dfed0f" > +SRC_URI[sha256sum] = > "59e676aa53ccb44b6c39f0ffe01a8fa274891c91bef1474752fad92461def24f" > diff --git > a/meta/recipes-core/util-linux/util-linux/0001-login-utils-include-libgen.h-for-basename-API.patch > > b/meta/recipes-core/util-linux/util-linux/0001-login-utils-include-libgen.h-for-basename-API.patch > deleted file mode 100644 > index 6258710e1e..0000000000 > --- > a/meta/recipes-core/util-linux/util-linux/0001-login-utils-include-libgen.h-for-basename-API.patch > +++ /dev/null > @@ -1,60 +0,0 @@ > -From d44e3ad1f6f8b5c1b3098bb7d537943a4c21d22f Mon Sep 17 00:00:00 2001 > -From: Khem Raj <[email protected]> > -Date: Sun, 3 Dec 2023 19:59:46 -0800 > -Subject: [PATCH] login-utils: include libgen.h for basename API > - > -musl has removed the non-prototype declaration of basename from string.h [1] > which now results in build errors with clang-17+ compiler > - > -include libgen.h for using the posix declaration of the funciton. > - > -Fixes > - > -../util-linux-2.39.2/login-utils/su-common.c:847:20: error: call to > undeclared function 'basename'; ISO C99 and later do not support implicit > function declarations [-Wimplicit-function-declaration] > - 847 | shell_basename = basename(shell); > - | ^ > - > -[1] > https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7 > - > -Upstream-Status: Submitted > [https://github.com/util-linux/util-linux/pull/2615] > -Signed-off-by: Khem Raj <[email protected]> > - > ---- > - login-utils/su-common.c | 10 +++++++--- > - 1 file changed, 7 insertions(+), 3 deletions(-) > - > -diff --git a/login-utils/su-common.c b/login-utils/su-common.c > -index b674920..3297c78 100644 > ---- a/login-utils/su-common.c > -+++ b/login-utils/su-common.c > -@@ -26,6 +26,7 @@ > - #include <sys/types.h> > - #include <pwd.h> > - #include <grp.h> > -+#include <libgen.h> > - #include <security/pam_appl.h> > - #ifdef HAVE_SECURITY_PAM_MISC_H > - # include <security/pam_misc.h> > -@@ -840,17 +841,20 @@ static void run_shell( > - su->simulate_login ? " login" : "", > - su->fast_startup ? " fast-start" : "")); > - > -+ char* tmp = xstrdup(shell); > - if (su->simulate_login) { > - char *arg0; > - char *shell_basename; > - > -- shell_basename = basename(shell); > -+ shell_basename = basename(tmp); > - arg0 = xmalloc(strlen(shell_basename) + 2); > - arg0[0] = '-'; > - strcpy(arg0 + 1, shell_basename); > - args[0] = arg0; > -- } else > -- args[0] = basename(shell); > -+ } else { > -+ args[0] = basename(tmp); > -+ } > -+ free(tmp); > - > - if (su->fast_startup) > - args[argno++] = "-f"; > diff --git > a/meta/recipes-core/util-linux/util-linux/avoid_parallel_tests.patch > b/meta/recipes-core/util-linux/util-linux/avoid_parallel_tests.patch > index 85ad7a5575..8ea00dbb95 100644 > --- a/meta/recipes-core/util-linux/util-linux/avoid_parallel_tests.patch > +++ b/meta/recipes-core/util-linux/util-linux/avoid_parallel_tests.patch > @@ -1,4 +1,4 @@ > -From 0b05e4695a0616badef71dfa459a00ef6ff1b521 Mon Sep 17 00:00:00 2001 > +From da58b4150f6932de36c962ea657713bcf4ea44be Mon Sep 17 00:00:00 2001 > From: Tudor Florea <[email protected]> > Date: Mon, 14 Jun 2021 14:00:31 +0200 > Subject: [PATCH] util-linux: Add ptest > @@ -9,13 +9,12 @@ Revert run.sh script accordingly to serialize running tests > > Signed-off-by: Tudor Florea <[email protected]> > Upstream-Status: Inappropriate > - > --- > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > -index 890212f..870e817 100644 > +index 651ef2b..2a8012e 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -11,7 +11,7 @@ AC_CONFIG_MACRO_DIR([m4]) > diff --git a/meta/recipes-core/util-linux/util-linux/configure-sbindir.patch > b/meta/recipes-core/util-linux/util-linux/configure-sbindir.patch > index 75adeca188..74abdc2038 100644 > --- a/meta/recipes-core/util-linux/util-linux/configure-sbindir.patch > +++ b/meta/recipes-core/util-linux/util-linux/configure-sbindir.patch > @@ -1,4 +1,4 @@ > -From c79222a9a5e3425c55e150edc0b7ac59c573aa2f Mon Sep 17 00:00:00 2001 > +From 0387d71516c978899cfd556ca512f530a212b4a9 Mon Sep 17 00:00:00 2001 > From: Phil Blundell <[email protected]> > Date: Mon, 24 Sep 2012 07:24:51 +0100 > Subject: [PATCH] util-linux: Ensure that ${sbindir} is respected > @@ -9,16 +9,15 @@ fix the test, the [ ] syntax was getting eaten by autoconf > Signed-off-by: Phil Blundell <[email protected]> > Signed-off-by: Saul Wold <[email protected] > Upstream-Status: Inappropriate [configuration] > - > --- > configure.ac | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > -index 36c24b4..890212f 100644 > +index 23594ff..651ef2b 100644 > --- a/configure.ac > +++ b/configure.ac > -@@ -102,7 +102,10 @@ AC_SUBST([runstatedir]) > +@@ -110,7 +110,10 @@ AC_SUBST([runstatedir]) > usrbin_execdir='${exec_prefix}/bin' > AC_SUBST([usrbin_execdir]) > > diff --git > a/meta/recipes-core/util-linux/util-linux/display_testname_for_subtest.patch > b/meta/recipes-core/util-linux/util-linux/display_testname_for_subtest.patch > index 815ae9c915..871b0b2d30 100644 > --- > a/meta/recipes-core/util-linux/util-linux/display_testname_for_subtest.patch > +++ > b/meta/recipes-core/util-linux/util-linux/display_testname_for_subtest.patch > @@ -1,20 +1,19 @@ > -From fc5de1de898fd1a372a2fd2fa493dc57323a029d Mon Sep 17 00:00:00 2001 > +From 45450d0339cd7ee27f53c0f2fac1cc74ed9bbfb8 Mon Sep 17 00:00:00 2001 > From: Tudor Florea <[email protected]> > Date: Thu, 3 Dec 2015 04:08:00 +0100 > Subject: [PATCH] Display testname for subtest > > Signed-off-by: Tudor Florea <[email protected]> > Upstream-Status: Pending > - > --- > tests/functions.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/functions.sh b/tests/functions.sh > -index 5a562a3..098145e 100644 > +index 5fe5ba0..a6c703e 100644 > --- a/tests/functions.sh > +++ b/tests/functions.sh > -@@ -437,7 +437,7 @@ function ts_init_subtest { > +@@ -450,7 +450,7 @@ function ts_init_subtest { > > if [ "$TS_PARSABLE" != "yes" ]; then > [ $TS_NSUBTESTS -eq 1 ] && echo > diff --git a/meta/recipes-core/util-linux/util-linux/mit-license.patch > b/meta/recipes-core/util-linux/util-linux/mit-license.patch > deleted file mode 100644 > index afbec98f18..0000000000 > --- a/meta/recipes-core/util-linux/util-linux/mit-license.patch > +++ /dev/null > @@ -1,45 +0,0 @@ > -From 5b8fab1584017d9d9be008c23b90128bba41a7b5 Mon Sep 17 00:00:00 2001 > -From: Richard Purdie <[email protected]> > -Date: Thu, 28 Mar 2024 12:16:57 +0000 > -Subject: [PATCH] README.licensing/flock: Add MIT license mention > - > -Looking at the license text, flock.c is under the MIT license (see > -https://spdx.org/licenses/MIT). > - > -Add an SPDX license identifier header and add to the list of licenses the > -source so everything is correctly listed/identified. > - > -Signed-off-by: Richard Purdie <[email protected]> > - > -Upstream-Status: Submitted > [https://github.com/util-linux/util-linux/pull/2870] > - > ---- > - README.licensing | 2 ++ > - sys-utils/flock.c | 4 +++- > - 2 files changed, 5 insertions(+), 1 deletion(-) > - > -diff --git a/README.licensing b/README.licensing > -index 4454f8392a..535ad34813 100644 > ---- a/README.licensing > -+++ b/README.licensing > -@@ -12,6 +12,8 @@ There is code under: > - > - * LGPL-2.1-or-later - GNU Lesser General Public License 2.1 or any > later version > - > -+ * MIT - MIT License > -+ > - * BSD-2-Clause - Simplified BSD License > - > - * BSD-3-Clause - BSD 3-Clause "New" or "Revised" License > -diff --git a/sys-utils/flock.c b/sys-utils/flock.c > -index fed29d7270..7d878ff810 100644 > ---- a/sys-utils/flock.c > -+++ b/sys-utils/flock.c > -@@ -1,4 +1,6 @@ > --/* Copyright 2003-2005 H. Peter Anvin - All Rights Reserved > -+/* SPDX-License-Identifier: MIT > -+ * > -+ * Copyright 2003-2005 H. Peter Anvin - All Rights Reserved > - * > - * Permission is hereby granted, free of charge, to any person > - * obtaining a copy of this software and associated documentation > diff --git a/meta/recipes-core/util-linux/util-linux/ptest.patch > b/meta/recipes-core/util-linux/util-linux/ptest.patch > index 6221de7182..7345926b88 100644 > --- a/meta/recipes-core/util-linux/util-linux/ptest.patch > +++ b/meta/recipes-core/util-linux/util-linux/ptest.patch > @@ -1,24 +1,23 @@ > -From d0a69ce80c579cbb7627a2f20e8b92e006a8d8ad Mon Sep 17 00:00:00 2001 > +From f9be73c3e2b0a4f0923950dcb058be85d3ee8ed7 Mon Sep 17 00:00:00 2001 > From: Tudor Florea <[email protected]> > Date: Thu, 3 Dec 2015 04:08:00 +0100 > Subject: [PATCH] Define TESTS variable > > Signed-off-by: Tudor Florea <[email protected]> > Upstream-Status: Pending > - > --- > Makefile.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Makefile.am b/Makefile.am > -index effbb02..7d2bd1e 100644 > +index 50ab8d0..7e2340d 100644 > --- a/Makefile.am > +++ b/Makefile.am > -@@ -57,6 +57,7 @@ systemdsystemunit_DATA = > +@@ -64,6 +64,7 @@ tmpfiles_DATA = > dist_bashcompletion_DATA = > check_PROGRAMS = > dist_check_SCRIPTS = > +TESTS = $(check_PROGRAMS) > > - PATHFILES = > - ADOCFILES_COMMON = > + BUILT_SOURCES = > + > diff --git a/meta/recipes-core/util-linux/util-linux_2.39.3.bb > b/meta/recipes-core/util-linux/util-linux_2.40.1.bb > similarity index 95% > rename from meta/recipes-core/util-linux/util-linux_2.39.3.bb > rename to meta/recipes-core/util-linux/util-linux_2.40.1.bb > index 2cd7750afc..5985ddc7dc 100644 > --- a/meta/recipes-core/util-linux/util-linux_2.39.3.bb > +++ b/meta/recipes-core/util-linux/util-linux_2.40.1.bb > @@ -90,7 +90,7 @@ EXTRA_OECONF:append = " --disable-hwclock-gplv3" > # build host versions during development > # > PACKAGECONFIG ?= "pcre2" > -PACKAGECONFIG:class-target ?= "${@bb.utils.contains('DISTRO_FEATURES', > 'pam', 'chfn-chsh pam', '', d)}" > +PACKAGECONFIG:class-target ?= "${@bb.utils.contains('DISTRO_FEATURES', > 'pam', 'chfn-chsh pam lastlog2', '', d)}" > # inherit manpages requires this to be present, however util-linux does not > have > # configuration options, and installs manpages always > PACKAGECONFIG[manpages] = "" > @@ -106,6 +106,7 @@ PACKAGECONFIG[pcre2] = ",,libpcre2" > PACKAGECONFIG[cryptsetup] = > "--with-cryptsetup,--without-cryptsetup,cryptsetup" > PACKAGECONFIG[chfn-chsh] = "--enable-chfn-chsh,--disable-chfn-chsh," > PACKAGECONFIG[selinux] = "--with-selinux,--without-selinux,libselinux" > +PACKAGECONFIG[lastlog2] = > "--enable-liblastlog2,--disable-liblastlog2,sqlite3" > > EXTRA_OEMAKE = "ARCH=${TARGET_ARCH} CPU= CPUOPT= 'OPT=${CFLAGS}'" > > @@ -117,6 +118,7 @@ FILES:${PN}-mount = "${sysconfdir}/default/mountall" > FILES:${PN}-runuser = "${sysconfdir}/pam.d/runuser*" > FILES:${PN}-su = "${sysconfdir}/pam.d/su-l" > CONFFILES:${PN}-su = "${sysconfdir}/pam.d/su-l" > +FILES:${PN}-lastlog2 += "${base_libdir}/security/pam_lastlog2.so" > FILES:${PN}-pylibmount = "${PYTHON_SITEPACKAGES_DIR}/libmount/pylibmount.so \ > ${PYTHON_SITEPACKAGES_DIR}/libmount/__init__.* \ > ${PYTHON_SITEPACKAGES_DIR}/libmount/__pycache__/*" > @@ -147,18 +149,21 @@ SYSTEMD_AUTO_ENABLE:${PN}-uuidd = "disable" > SYSTEMD_SERVICE:${PN}-fstrim = "fstrim.timer fstrim.service" > SYSTEMD_AUTO_ENABLE:${PN}-fstrim = "disable" > > +do_compile:prepend () { > + # this is a workaround for the unnecessary remake problem. Issue and PR > are as below: > + # https://github.com/util-linux/util-linux/issues/3061 > + # https://github.com/util-linux/util-linux/pull/3064 > + # When newly release tarball contains the above fix, the following > workaround could be dropped. > + [ -e ${S}/libsmartcols/src/filter-scanner.c ] && touch > ${S}/libsmartcols/src/filter-scanner.c > + [ -e ${S}/libsmartcols/src/filter-parser.c ] && touch > ${S}/libsmartcols/src/filter-parser.c > +} > + > do_compile:append () { > cp ${UNPACKDIR}/fcntl-lock.c ${S}/fcntl-lock.c > ${CC} ${CFLAGS} ${LDFLAGS} ${S}/fcntl-lock.c -o ${B}/fcntl-lock > } > > -do_install () { > - # with ccache the timestamps on compiled files may > - # end up earlier than on their inputs, this allows > - # for the resultant compilation in the install step. > - oe_runmake 'CC=${CC}' 'LD=${LD}' \ > - 'LDFLAGS=${LDFLAGS}' 'DESTDIR=${D}' install > - > +do_install:append () { > mkdir -p ${D}${base_bindir} > > sbinprogs="agetty ctrlaltdel cfdisk vipw vigr" > -- > 2.34.1 > > > > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#199977): https://lists.openembedded.org/g/openembedded-core/message/199977 Mute This Topic: https://lists.openembedded.org/mt/106294633/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
