Openembedded Core Discussions
 help / color / mirror / Atom feed
* [OE-core][scarthgap 00/29] Patch review
@ 2024-06-13 14:20 Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 01/29] cronie: upgrade 1.7.1 -> 1.7.2 Steve Sakoman
                   ` (28 more replies)
  0 siblings, 29 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

Please review this set of changes for scarthgap and have comments back by
end of day Monday, June 17

Passed a-full on autobuilder:

https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/7034

The following changes since commit 3b070fc3963d04ce8c13b5b78c3b7ae95e26c435:

  binutils: Fix aarch64 disassembly abort (2024-06-10 05:31:10 -0700)

are available in the Git repository at:

  https://git.openembedded.org/openembedded-core-contrib stable/scarthgap-nut
  https://git.openembedded.org/openembedded-core-contrib/log/?h=stable/scarthgap-nut

Alexander Kanavin (9):
  libtraceevent: submit meson.patch upstream
  serf: mark patch as inappropriate for upstream submission
  kea: remove unnecessary reproducibility patch
  expect: mark patches as Inactive-Upstream
  apr: submit 0001-Add-option-to-disable-timed-dependant-tests.patch
    upstream
  busybox: submit CVE-2022-28391 patches upstream
  xinput-calibrator: mark upstream as inactive in a patch
  bash: mark build-tests.patch as Inappropriate
  icu: add upstream submission links for fix-install-manx.patch

Alexandre Truong (2):
  oeqa/selftest/devtool: fix _test_devtool_add_git_url
  Revert "oeqa/selftest/devtool: fix test_devtool_add_git_style2"

Anton Almqvist (1):
  devtool: modify: Catch git submodule error for go code

Felix Nilsson (1):
  base-files: profile: fix error sh: 1: unknown operand

Heiko (1):
  kernel.bbclass: check, if directory exists before removing empty
    module directory

Julien Stephan (1):
  oeqa: selftest: context: run tests serially if testtools/subunit
    modules are not found

Khem Raj (1):
  oeqa/postactions: Do not use -l option with df

Marc Ferland (1):
  libinput: fix building with debug-gui option

Maxin B. John (1):
  iproute2: drop obsolete patch

Ola x Nilsson (1):
  oeqa/selftest/devtool: add test for modifying recipes using go.bbclass

Poonam Jadhav (1):
  ppp: Add RSA-MD in LICENSE

Ross Burton (1):
  procps: fix build with new glibc but old kernel headers

Vincent Kriek (1):
  devtool: sync: Fix Execution error

Wang Mingyu (4):
  cronie: upgrade 1.7.1 -> 1.7.2
  lttng-ust: upgrade 2.13.7 -> 2.13.8
  taglib: upgrade 2.0 -> 2.0.1
  mesa: upgrade 24.0.3 -> 24.0.5

Xiangyu Chen (2):
  iputils: splitting the ping6 as a package
  ltp: add iputils-ping6 to RDEPENDS

Zoltan Boszormenyi (1):
  cracklib: Modify patch to compile with GCC 14

 meta/classes-recipe/kernel.bbclass            |  2 +-
 meta/lib/oeqa/selftest/cases/devtool.py       | 31 ++++++++--
 meta/lib/oeqa/selftest/context.py             | 17 ++++-
 meta/lib/oeqa/utils/postactions.py            |  2 +-
 ...01-libc-compat.h-add-musl-workaround.patch | 39 ------------
 .../iproute2/iproute2_6.7.0.bb                |  4 +-
 ...1-kea-fix-reproducible-build-failure.patch | 62 -------------------
 meta/recipes-connectivity/kea/kea_2.4.1.bb    |  3 +-
 meta/recipes-connectivity/ppp/ppp_2.5.0.bb    |  2 +-
 .../base-files/base-files/profile             |  2 +-
 ...tr-ensure-only-printable-characters-.patch |  2 +-
 ...e-all-printed-strings-with-printable.patch |  2 +-
 ...001-Resolve-string-formatting-issues.patch |  2 +-
 .../expect/expect/0001-configure.in.patch     |  2 +-
 ...exp_main_sub.c-Use-PATH_MAX-for-path.patch |  2 +-
 ...ults-if-Tcl-is-built-with-stubs-and-.patch |  2 +-
 .../expect/expect/0002-tcl.m4.patch           |  2 +-
 .../bash/bash/build-tests.patch               |  2 +-
 ...port-dictionary-byte-order-dependent.patch |  2 +-
 .../{cronie_1.7.1.bb => cronie_1.7.2.bb}      |  2 +-
 .../iputils/iputils_20240117.bb               | 13 +++-
 meta/recipes-extended/ltp/ltp_20240129.bb     |  1 +
 .../procps/procps/pidfd.patch                 | 42 +++++++++++++
 meta/recipes-extended/procps/procps_4.0.4.bb  |  1 +
 .../{mesa-gl_24.0.3.bb => mesa-gl_24.0.5.bb}  |  0
 meta/recipes-graphics/mesa/mesa.inc           |  2 +-
 .../mesa/{mesa_24.0.3.bb => mesa_24.0.5.bb}   |  0
 .../wayland/libinput_1.25.0.bb                |  2 +-
 ...ibrator_pointercal.sh-to-be-run-as-n.patch |  2 +-
 .../libtraceevent/libtraceevent/meson.patch   |  2 +-
 ...ttng-ust_2.13.7.bb => lttng-ust_2.13.8.bb} |  2 +-
 ...ion-to-disable-timed-dependant-tests.patch |  2 +-
 .../icu/icu/fix-install-manx.patch            |  4 +-
 ...ories.without.sandbox-install.prefix.patch |  2 +-
 .../taglib/{taglib_2.0.bb => taglib_2.0.1.bb} |  2 +-
 scripts/lib/devtool/standard.py               | 18 +++++-
 36 files changed, 142 insertions(+), 137 deletions(-)
 delete mode 100644 meta/recipes-connectivity/iproute2/iproute2/0001-libc-compat.h-add-musl-workaround.patch
 delete mode 100644 meta/recipes-connectivity/kea/files/0001-kea-fix-reproducible-build-failure.patch
 rename meta/recipes-extended/cronie/{cronie_1.7.1.bb => cronie_1.7.2.bb} (97%)
 create mode 100644 meta/recipes-extended/procps/procps/pidfd.patch
 rename meta/recipes-graphics/mesa/{mesa-gl_24.0.3.bb => mesa-gl_24.0.5.bb} (100%)
 rename meta/recipes-graphics/mesa/{mesa_24.0.3.bb => mesa_24.0.5.bb} (100%)
 rename meta/recipes-kernel/lttng/{lttng-ust_2.13.7.bb => lttng-ust_2.13.8.bb} (95%)
 rename meta/recipes-support/taglib/{taglib_2.0.bb => taglib_2.0.1.bb} (95%)

-- 
2.34.1



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

* [OE-core][scarthgap 01/29] cronie: upgrade 1.7.1 -> 1.7.2
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 02/29] lttng-ust: upgrade 2.13.7 -> 2.13.8 Steve Sakoman
                   ` (27 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Wang Mingyu <wangmy@fujitsu.com>

Changelog:
============
*Revert setting the return path to <>. It is not RFC compliant.
*Inherit MAILFROM from the crond process environment.

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit e93d7ba7ea3718e396510b12726a232edaecf976)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../cronie/{cronie_1.7.1.bb => cronie_1.7.2.bb}                 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-extended/cronie/{cronie_1.7.1.bb => cronie_1.7.2.bb} (97%)

diff --git a/meta/recipes-extended/cronie/cronie_1.7.1.bb b/meta/recipes-extended/cronie/cronie_1.7.2.bb
similarity index 97%
rename from meta/recipes-extended/cronie/cronie_1.7.1.bb
rename to meta/recipes-extended/cronie/cronie_1.7.2.bb
index 854b68163c..abdaff644d 100644
--- a/meta/recipes-extended/cronie/cronie_1.7.1.bb
+++ b/meta/recipes-extended/cronie/cronie_1.7.2.bb
@@ -25,7 +25,7 @@ SRC_URI = "${GITHUB_BASE_URI}/download/cronie-${PV}/cronie-${PV}.tar.gz \
 PAM_SRC_URI = "file://crond_pam_config.patch"
 PAM_DEPS = "libpam libpam-runtime pam-plugin-access pam-plugin-loginuid"
 
-SRC_URI[sha256sum] = "78033100c24413f0c40f93e6138774d6a4f55bc31050567b90db45a2f9f1b954"
+SRC_URI[sha256sum] = "f1da374a15ba7605cf378347f96bc8b678d3d7c0765269c8242cfe5b0789c571"
 
 inherit autotools update-rc.d useradd systemd github-releases
 UPSTREAM_CHECK_REGEX = "releases/tag/cronie-(?P<pver>\d+(\.\d+)+)"
-- 
2.34.1



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

* [OE-core][scarthgap 02/29] lttng-ust: upgrade 2.13.7 -> 2.13.8
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 01/29] cronie: upgrade 1.7.1 -> 1.7.2 Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 03/29] taglib: upgrade 2.0 -> 2.0.1 Steve Sakoman
                   ` (26 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Wang Mingyu <wangmy@fujitsu.com>

Changelog:
============
* Add close_range wrapper to liblttng-ust-fd.so
* ust-tracepoint-event: Add static check of sequences length type
* lttng-ust(3): Fix wrong len_type for sequence
* Fix: libc wrapper: use initial-exec for malloc_nesting TLS

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 3dd02f741a151bcc1fe46e7fd6d585ca92c86c60)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../lttng/{lttng-ust_2.13.7.bb => lttng-ust_2.13.8.bb}          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-kernel/lttng/{lttng-ust_2.13.7.bb => lttng-ust_2.13.8.bb} (95%)

diff --git a/meta/recipes-kernel/lttng/lttng-ust_2.13.7.bb b/meta/recipes-kernel/lttng/lttng-ust_2.13.8.bb
similarity index 95%
rename from meta/recipes-kernel/lttng/lttng-ust_2.13.7.bb
rename to meta/recipes-kernel/lttng/lttng-ust_2.13.8.bb
index 9509185bad..dddd3a5004 100644
--- a/meta/recipes-kernel/lttng/lttng-ust_2.13.7.bb
+++ b/meta/recipes-kernel/lttng/lttng-ust_2.13.8.bb
@@ -34,7 +34,7 @@ SRC_URI = "https://lttng.org/files/lttng-ust/lttng-ust-${PV}.tar.bz2 \
            file://0001-Makefile.am-update-rpath-link.patch \
            "
 
-SRC_URI[sha256sum] = "5fb4f17c307c8c1b79c68561e89be9562d07e7425bf40e728c4d66755342a5eb"
+SRC_URI[sha256sum] = "d4ef98dab9a37ad4f524ccafdfd50af4f266039b528dd5afabce78e49024d937"
 
 CVE_PRODUCT = "ust"
 
-- 
2.34.1



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

* [OE-core][scarthgap 03/29] taglib: upgrade 2.0 -> 2.0.1
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 01/29] cronie: upgrade 1.7.1 -> 1.7.2 Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 02/29] lttng-ust: upgrade 2.13.7 -> 2.13.8 Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 04/29] mesa: upgrade 24.0.3 -> 24.0.5 Steve Sakoman
                   ` (25 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Wang Mingyu <wangmy@fujitsu.com>

Changelog:
===========
* Fix aborting when _GLIBCXX_ASSERTIONS are enabled.
* Fall back to utf8cpp header detection in the case that its CMake
  configuration is removed.
* Improve compatibility with the SWIG interface compiler.
* Build system fixes for testing without bindings, Emscripten and Illumos.
* C bindings: Fix setting UTF-8 encoded property values.
* Windows: Fix opening long paths.

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit d3438bd2698dff19722123d7b2b5674ba40107eb)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-support/taglib/{taglib_2.0.bb => taglib_2.0.1.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-support/taglib/{taglib_2.0.bb => taglib_2.0.1.bb} (95%)

diff --git a/meta/recipes-support/taglib/taglib_2.0.bb b/meta/recipes-support/taglib/taglib_2.0.1.bb
similarity index 95%
rename from meta/recipes-support/taglib/taglib_2.0.bb
rename to meta/recipes-support/taglib/taglib_2.0.1.bb
index 4bf9be15ae..14f99aabbc 100644
--- a/meta/recipes-support/taglib/taglib_2.0.bb
+++ b/meta/recipes-support/taglib/taglib_2.0.1.bb
@@ -11,7 +11,7 @@ DEPENDS = "zlib utfcpp"
 
 SRC_URI = "http://taglib.github.io/releases/${BP}.tar.gz"
 
-SRC_URI[sha256sum] = "e36ea877a6370810b97d84cf8f72b1e4ed205149ab3ac8232d44c850f38a2859"
+SRC_URI[sha256sum] = "08c0a27b96aa5c4e23060fe0b6f93102ee9091a9385257b9d0ddcf467de0d925"
 
 UPSTREAM_CHECK_URI = "https://taglib.org/"
 UPSTREAM_CHECK_REGEX = "taglib-(?P<pver>\d+(\.\d+)+)\.tar"
-- 
2.34.1



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

* [OE-core][scarthgap 04/29] mesa: upgrade 24.0.3 -> 24.0.5
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (2 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 03/29] taglib: upgrade 2.0 -> 2.0.1 Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 05/29] procps: fix build with new glibc but old kernel headers Steve Sakoman
                   ` (24 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Wang Mingyu <wangmy@fujitsu.com>

Changelog:
 https://docs.mesa3d.org/relnotes/24.0.5.html

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 5933d099c339596f62e1237d4e738dbe9f386b10)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../mesa/{mesa-gl_24.0.3.bb => mesa-gl_24.0.5.bb}               | 0
 meta/recipes-graphics/mesa/mesa.inc                             | 2 +-
 meta/recipes-graphics/mesa/{mesa_24.0.3.bb => mesa_24.0.5.bb}   | 0
 3 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-graphics/mesa/{mesa-gl_24.0.3.bb => mesa-gl_24.0.5.bb} (100%)
 rename meta/recipes-graphics/mesa/{mesa_24.0.3.bb => mesa_24.0.5.bb} (100%)

diff --git a/meta/recipes-graphics/mesa/mesa-gl_24.0.3.bb b/meta/recipes-graphics/mesa/mesa-gl_24.0.5.bb
similarity index 100%
rename from meta/recipes-graphics/mesa/mesa-gl_24.0.3.bb
rename to meta/recipes-graphics/mesa/mesa-gl_24.0.5.bb
diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
index 1c9fa66c72..77e9c80fcb 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -22,7 +22,7 @@ SRC_URI = "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
            file://0001-Revert-meson-do-not-pull-in-clc-for-clover.patch \
 "
 
-SRC_URI[sha256sum] = "77aec9a2a37b7d3596ea1640b3cc53d0b5d9b3b52abed89de07e3717e91bfdbe"
+SRC_URI[sha256sum] = "38cc245ca8faa3c69da6d2687f8906377001f63365348a62cc6f7fafb1e8c018"
 
 UPSTREAM_CHECK_GITTAGREGEX = "mesa-(?P<pver>\d+(\.\d+)+)"
 
diff --git a/meta/recipes-graphics/mesa/mesa_24.0.3.bb b/meta/recipes-graphics/mesa/mesa_24.0.5.bb
similarity index 100%
rename from meta/recipes-graphics/mesa/mesa_24.0.3.bb
rename to meta/recipes-graphics/mesa/mesa_24.0.5.bb
-- 
2.34.1



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

* [OE-core][scarthgap 05/29] procps: fix build with new glibc but old kernel headers
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (3 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 04/29] mesa: upgrade 24.0.3 -> 24.0.5 Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 06/29] devtool: modify: Catch git submodule error for go code Steve Sakoman
                   ` (23 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Ross Burton <ross.burton@arm.com>

If you're building procps with a newer glibc (with pidfd_open()) but
older kernel headers (say 4.x, before __NR_pidfd_open) then procps will
fail to build because of a typo in configure.ac.

Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../procps/procps/pidfd.patch                 | 42 +++++++++++++++++++
 meta/recipes-extended/procps/procps_4.0.4.bb  |  1 +
 2 files changed, 43 insertions(+)
 create mode 100644 meta/recipes-extended/procps/procps/pidfd.patch

diff --git a/meta/recipes-extended/procps/procps/pidfd.patch b/meta/recipes-extended/procps/procps/pidfd.patch
new file mode 100644
index 0000000000..23d1a3cd3e
--- /dev/null
+++ b/meta/recipes-extended/procps/procps/pidfd.patch
@@ -0,0 +1,42 @@
+From c8f625e085b8249cc009e8b19c3a19100217eb35 Mon Sep 17 00:00:00 2001
+From: Ross Burton <ross.burton@arm.com>
+Date: Thu, 25 Apr 2024 13:33:15 +0000
+Subject: [PATCH] Fix pidfd_open detection
+
+This check for pidfd_open uses AC_CHECK_FUNC which just runs the specified code, but 
+src/pgrep.c checks HAVE_PIDFD_OPEN which will only be defined by AC_CHECK_FUNCS.
+
+Also pidfd_open is defined in sys/pidfd.h so that needs including.
+
+Upstream-Status: Submitted [https://gitlab.com/procps-ng/procps/-/merge_requests/229]
+Signed-off-by: Ross Burton <ross.burton@arm.com>
+---
+
+diff --git a/configure.ac b/configure.ac
+index fec27e3f..024731c7 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -170,7 +170,7 @@ AC_TRY_COMPILE([#include <errno.h>],
+ 		AC_MSG_RESULT(yes),
+ 		AC_MSG_RESULT(no))
+ 
+-AC_CHECK_FUNC([pidfd_open], [enable_pidwait=yes], [
++AC_CHECK_FUNCS([pidfd_open], [enable_pidwait=yes], [
+   AC_MSG_CHECKING([for __NR_pidfd_open])
+   AC_COMPILE_IFELSE([AC_LANG_SOURCE([
+ #include <sys/syscall.h>
+diff --git a/src/pgrep.c b/src/pgrep.c
+index d8e57dff..c5211aec 100644
+--- a/src/pgrep.c
++++ b/src/pgrep.c
+@@ -44,7 +44,9 @@
+
+ #ifdef ENABLE_PIDWAIT
+ #include <sys/epoll.h>
+-#ifndef HAVE_PIDFD_OPEN
++#ifdef HAVE_PIDFD_OPEN
++#include <sys/pidfd.h>
++#else
+ #include <sys/syscall.h>
+ #endif /* !HAVE_PIDFD_OPEN */
+ #endif
diff --git a/meta/recipes-extended/procps/procps_4.0.4.bb b/meta/recipes-extended/procps/procps_4.0.4.bb
index 800384f22f..ec8c4b0261 100644
--- a/meta/recipes-extended/procps/procps_4.0.4.bb
+++ b/meta/recipes-extended/procps/procps_4.0.4.bb
@@ -14,6 +14,7 @@ inherit autotools gettext pkgconfig update-alternatives
 
 SRC_URI = "git://gitlab.com/procps-ng/procps.git;protocol=https;branch=master \
            file://sysctl.conf \
+           file://pidfd.patch \
            "
 SRCREV = "4ddcef2fd843170c8e2d59a83042978f41037a2b"
 
-- 
2.34.1



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

* [OE-core][scarthgap 06/29] devtool: modify: Catch git submodule error for go code
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (4 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 05/29] procps: fix build with new glibc but old kernel headers Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 07/29] oeqa/selftest/devtool: add test for modifying recipes using go.bbclass Steve Sakoman
                   ` (22 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Anton Almqvist <antonal@axis.com>

One of the git submodule commands failed for source extracted for
recipes using go.bbclass.  The root cause is probably the path set up
by go_do_unpack which makes S and gitroot not match.

This patch does not fix the root problem, but at least it is no worse
than before the git submodule support.

The extracted source will still have two .git folders, one in S
created by devtool and one in the go path which will contain the tru
git history.

[ YOCTO #15483 ]

Signed-off-by: Anton Almqvist <antonal@axis.com>
Signed-off-by: Ola x Nilsson <olani@axis.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit fe242408af40dd1f6e47d9b2b232bdc76756c80a)
Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 scripts/lib/devtool/standard.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index 10d0cd3b7c..2beb058de8 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -893,7 +893,10 @@ def modify(args, config, basepath, workspace):
                 (stdout, _) = bb.process.run('git rev-list --reverse %s..HEAD' % initial_revs["."], cwd=srctree)
                 commits["."] = stdout.split()
                 check_commits = True
-                (stdout, _) = bb.process.run('git submodule --quiet foreach --recursive  \'echo `git rev-parse devtool-base` $PWD\'', cwd=srctree)
+                try:
+                    (stdout, _) = bb.process.run('git submodule --quiet foreach --recursive  \'echo `git rev-parse devtool-base` $PWD\'', cwd=srctree)
+                except bb.process.ExecutionError:
+                    stdout = ""
                 for line in stdout.splitlines():
                     (rev, submodule_path) = line.split()
                     submodule = os.path.relpath(submodule_path, srctree)
-- 
2.34.1



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

* [OE-core][scarthgap 07/29] oeqa/selftest/devtool: add test for modifying recipes using go.bbclass
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (5 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 06/29] devtool: modify: Catch git submodule error for go code Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 08/29] base-files: profile: fix error sh: 1: unknown operand Steve Sakoman
                   ` (21 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Ola x Nilsson <olani@axis.com>

go.bbclass uses a special do_unpack function that causes the git root
to be different from S.  Verify that it unpacks as expected.

[ YOCTO #15483 ]

Signed-off-by: Ola x Nilsson <olani@axis.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit fab0c737b95b8d0c0bbf58336bc308776c956406)
Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/lib/oeqa/selftest/cases/devtool.py | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py
index 51949e3c93..cc4cbec5ae 100644
--- a/meta/lib/oeqa/selftest/cases/devtool.py
+++ b/meta/lib/oeqa/selftest/cases/devtool.py
@@ -749,6 +749,25 @@ class DevtoolModifyTests(DevtoolBase):
         result = runCmd('devtool status')
         self.assertNotIn('mdadm', result.output)
 
+    def test_devtool_modify_go(self):
+        import oe.path
+        from tempfile import TemporaryDirectory
+        with TemporaryDirectory(prefix='devtoolqa') as tempdir:
+            self.track_for_cleanup(self.workspacedir)
+            self.add_command_to_tearDown('bitbake -c clean go-helloworld')
+            self.add_command_to_tearDown('bitbake-layers remove-layer */workspace')
+            result = runCmd('devtool modify go-helloworld -x %s' % tempdir)
+            self.assertExists(
+                oe.path.join(tempdir, 'src', 'golang.org', 'x', 'example', 'go.mod'),
+                             'Extracted source could not be found'
+            )
+            self.assertExists(
+                oe.path.join(self.workspacedir, 'conf', 'layer.conf'),
+                'Workspace directory not created'
+            )
+            matches = glob.glob(oe.path.join(self.workspacedir, 'appends', 'go-helloworld_*.bbappend'))
+            self.assertTrue(matches, 'bbappend not created %s' % result.output)
+
     def test_devtool_buildclean(self):
         def assertFile(path, *paths):
             f = os.path.join(path, *paths)
-- 
2.34.1



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

* [OE-core][scarthgap 08/29] base-files: profile: fix error sh: 1: unknown operand
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (6 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 07/29] oeqa/selftest/devtool: add test for modifying recipes using go.bbclass Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 09/29] oeqa/postactions: Do not use -l option with df Steve Sakoman
                   ` (20 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Felix Nilsson <felixn@axis.com>

Handle errors when SHLVL isn't set.

Signed-off-by: Felix Nilsson <felixn@axis.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 5df53fcfe3b70a5312fced3fcc1ba6290f2ee794)
Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-core/base-files/base-files/profile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/base-files/base-files/profile b/meta/recipes-core/base-files/base-files/profile
index bded3757cc..5e8393c91c 100644
--- a/meta/recipes-core/base-files/base-files/profile
+++ b/meta/recipes-core/base-files/base-files/profile
@@ -58,7 +58,7 @@ resize() {
 	fi
 	# only do this for /dev/tty[A-z] which are typically
 	# serial ports
-	if [ $FIRSTTIMESETUP -eq 1 -a $SHLVL -eq 1 ] ; then
+	if [ $FIRSTTIMESETUP -eq 1 -a ${SHLVL:-1} -eq 1 ] ; then
 		case $(tty 2>/dev/null) in
 			/dev/tty[A-z]*) resize >/dev/null;;
 		esac
-- 
2.34.1



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

* [OE-core][scarthgap 09/29] oeqa/postactions: Do not use -l option with df
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (7 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 08/29] base-files: profile: fix error sh: 1: unknown operand Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 10/29] oeqa: selftest: context: run tests serially if testtools/subunit modules are not found Steve Sakoman
                   ` (19 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Khem Raj <raj.khem@gmail.com>

-l option is specific to df provided by coreutils, if df
applet from busybox is used then it does not work and fails
like below

Fixes
   df: invalid option -- 'l'
   BusyBox v1.36.1 () multi-call binary.

   Usage: df [-PkmhT] [-t TYPE] [FILESYSTEM]...
   DEBUG: [Command returned '1' after 0.71 seconds]
   DEBUG: Command: df -hl
   Status: 1 Output:  df: invalid option -- 'l'

it seems worth a compromise to show remote mounted filesystems if any
during ptests and it works with both df implementations

Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
(cherry picked from commit 75cc1ea4348a2294fdc5ab20530fcff27056ff06)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/lib/oeqa/utils/postactions.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/utils/postactions.py b/meta/lib/oeqa/utils/postactions.py
index 8104400ac2..ecdddd2d40 100644
--- a/meta/lib/oeqa/utils/postactions.py
+++ b/meta/lib/oeqa/utils/postactions.py
@@ -25,7 +25,7 @@ def create_artifacts_directory(d, tc):
 def get_target_disk_usage(d, tc):
     output_file = os.path.join(get_json_result_dir(d), "artifacts", "target_disk_usage.txt")
     try:
-        (status, output) = tc.target.run('df -hl')
+        (status, output) = tc.target.run('df -h')
         with open(output_file, 'w') as f:
             f.write(output)
             f.write("\n")
-- 
2.34.1



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

* [OE-core][scarthgap 10/29] oeqa: selftest: context: run tests serially if testtools/subunit modules are not found
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (8 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 09/29] oeqa/postactions: Do not use -l option with df Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 11/29] libtraceevent: submit meson.patch upstream Steve Sakoman
                   ` (18 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Julien Stephan <jstephan@baylibre.com>

If testtools and/or subunit modules are not found we get the following backtrace
(example for testtools):

  NOTE: Starting bitbake server...
  Traceback (most recent call last):
    File "<..>/poky/scripts/oe-selftest", line 60, in
  <module>
      ret = main()
    File "<..>/poky/scripts/oe-selftest", line 47, in main
      results = args.func(logger, args)
    File "<..>/poky/meta/lib/oeqa/selftest/context.py",
  line 391, in run
      rc = self._internal_run(logger, args)
    File "<..>/poky/meta/lib/oeqa/selftest/context.py",
  line 377, in _internal_run
      rc = self.tc.runTests(**self.tc_kwargs['run'])
    File "<..>/poky/meta/lib/oeqa/selftest/context.py",
  line 161, in runTests
      return super(OESelftestTestContext, self).runTests(processes, skips)
    File "<..>/poky/meta/lib/oeqa/core/context.py", line
  91, in runTests
      result = self.runner.run(self.prepareSuite(self.suites, processes))
    File "<..>/poky/meta/lib/oeqa/selftest/context.py",
  line 154, in prepareSuite
      from oeqa.core.utils.concurrencytest import ConcurrentTestSuite
    File
  "<..>/poky/meta/lib/oeqa/core/utils/concurrencytest.py",
  line 22, in <module>
      import testtools
  ModuleNotFoundError: No module named 'testtools'

Fix this by adding a custom callback on -j/--num-processes parameter to
check testtools and subunit modules. Fallback to serial testing if
missing. This strategy is already used in sdk/context.py

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
(cherry picked from commit 35284404473b2c2d9f69594582868ed66ef3525e)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/lib/oeqa/selftest/context.py | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/selftest/context.py b/meta/lib/oeqa/selftest/context.py
index 57844b289a..99186175e5 100644
--- a/meta/lib/oeqa/selftest/context.py
+++ b/meta/lib/oeqa/selftest/context.py
@@ -194,8 +194,23 @@ class OESelftestTestContextExecutor(OETestContextExecutor):
         parser.add_argument('-R', '--skip-tests', required=False, action='store',
                 nargs='+', dest="skips", default=None,
                 help='Skip the tests specified. Format should be <module>[.<class>[.<test_method>]]')
+
+        def check_parallel_support(parameter):
+            if not parameter.isdigit():
+                import argparse
+                raise argparse.ArgumentTypeError("argument -j/--num-processes: invalid int value: '%s' " % str(parameter))
+
+            processes = int(parameter)
+            if processes:
+                try:
+                    import testtools, subunit
+                except ImportError:
+                    print("Failed to import testtools or subunit, the testcases will run serially")
+                    processes = None
+            return processes
+
         parser.add_argument('-j', '--num-processes', dest='processes', action='store',
-                type=int, help="number of processes to execute in parallel with")
+                type=check_parallel_support, help="number of processes to execute in parallel with")
 
         parser.add_argument('-t', '--select-tag', dest="select_tags",
                 action='append', default=None,
-- 
2.34.1



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

* [OE-core][scarthgap 11/29] libtraceevent: submit meson.patch upstream
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (9 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 10/29] oeqa: selftest: context: run tests serially if testtools/subunit modules are not found Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 12/29] devtool: sync: Fix Execution error Steve Sakoman
                   ` (17 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

[RP: Add url to submission]
Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit d00306620aa890a361f42516a064d64dd7c68aa7)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-kernel/libtraceevent/libtraceevent/meson.patch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/libtraceevent/libtraceevent/meson.patch b/meta/recipes-kernel/libtraceevent/libtraceevent/meson.patch
index 38b6107136..6e96ba2167 100644
--- a/meta/recipes-kernel/libtraceevent/libtraceevent/meson.patch
+++ b/meta/recipes-kernel/libtraceevent/libtraceevent/meson.patch
@@ -4,7 +4,7 @@ Fixes for the Meson build of libtraceevent:
 - Install the plugins as modules not static and versioned shared libraries
 - Add an option to disable building the documentation (needs asciidoc and xmlto)
 
-Upstream-Status: Pending
+Upstream-Status: Submitted [https://lore.kernel.org/linux-trace-devel/20240311111140.1789879-1-alex@linutronix.de/T/#u]
 Signed-off-by: Ross Burton <ross.burton@arm.com>
 
 diff --git a/meson.build b/meson.build
-- 
2.34.1



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

* [OE-core][scarthgap 12/29] devtool: sync: Fix Execution error
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (10 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 11/29] libtraceevent: submit meson.patch upstream Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 13/29] serf: mark patch as inappropriate for upstream submission Steve Sakoman
                   ` (16 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Vincent Kriek <vincent@coelebs.dev>

When executing devtool sync on a recipe that was extract with devtool
extract earlier the following error occured:

    Traceback (most recent call last):
    [...]
    bb.process.ExecutionError: Execution of 'git fetch file:///home/vin/projects/poky/build/tmp/work/all-poky-linux/netbase/6.4/devtooltmp-figt1jmr/workdir/netbase devtool:devtool' failed with exit code 128:
    fatal: refusing to fetch into branch 'refs/heads/devtool' checked out at '/home/vin/projects/poky/build/netbase-src'

Fix this by adding --update-head-ok and --force to git fetch so it will
override the current head even if it is checked out and has changes.
Possible existing changes in the devtool branch can be retrieved by
checking out the devtool.bak branch

Signed-off-by: Vincent Kriek <vincent@coelebs.dev>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
(cherry picked from commit 4c89b061003a0f4143c6d3838e39fc2643edc8d6)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 scripts/lib/devtool/standard.py | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index 2beb058de8..05161942b7 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -661,7 +661,18 @@ def _extract_source(srctree, keep_temp, devbranch, sync, config, basepath, works
         srctree_localdir = os.path.join(srctree, 'oe-local-files')
 
         if sync:
-            bb.process.run('git fetch file://' + srcsubdir + ' ' + devbranch + ':' + devbranch, cwd=srctree)
+            try:
+                logger.info('Backing up current %s branch as branch: %s.bak' % (devbranch, devbranch))
+                bb.process.run('git branch -f ' + devbranch + '.bak', cwd=srctree)
+
+                # Use git fetch to update the source with the current recipe
+                # To be able to update the currently checked out branch with
+                # possibly new history (no fast-forward) git needs to be told
+                # that's ok
+                logger.info('Syncing source files including patches to git branch: %s' % devbranch)
+                bb.process.run('git fetch --update-head-ok --force file://' + srcsubdir + ' ' + devbranch + ':' + devbranch, cwd=srctree)
+            except bb.process.ExecutionError as e:
+                raise DevtoolError("Error when syncing source files to local checkout: %s" % str(e))
 
             # Move the oe-local-files directory to srctree.
             # As oe-local-files is not part of the constructed git tree,
-- 
2.34.1



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

* [OE-core][scarthgap 13/29] serf: mark patch as inappropriate for upstream submission
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (11 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 12/29] devtool: sync: Fix Execution error Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 14/29] kea: remove unnecessary reproducibility patch Steve Sakoman
                   ` (15 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

What the patch does is not suitable, as upstream would surely
want the issue with the code fixed, not removal of that code altogether.

I'd also note that serf is still semi-ummaintained, and the only consumer
of it is subversion, which is slowly but steadily fading away in favor of
git and artifact-specific solutions.

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 634717817e94740ac8d4e8cc82cb68bfd15d8dbd)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 ...op.creating.directories.without.sandbox-install.prefix.patch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch b/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
index 91640d6044..c8e6eddfec 100644
--- a/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
+++ b/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
@@ -31,7 +31,7 @@ ERROR: scons install execution failed.
   and the installed paths (including the paths inside libserf*.pc)
   look correct
 
-Upstream-Status: Pending
+Upstream-Status: Inappropriate [removes block of code rather than fixing the problem in that block]
 
 Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
 
-- 
2.34.1



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

* [OE-core][scarthgap 14/29] kea: remove unnecessary reproducibility patch
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (12 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 13/29] serf: mark patch as inappropriate for upstream submission Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 15/29] expect: mark patches as Inactive-Upstream Steve Sakoman
                   ` (14 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

Rather, adjust the sed invocation to do the correct thing directly.

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 804afb2eb9cc2b5650faa80816f377fa5b72fb38)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 ...1-kea-fix-reproducible-build-failure.patch | 62 -------------------
 meta/recipes-connectivity/kea/kea_2.4.1.bb    |  3 +-
 2 files changed, 1 insertion(+), 64 deletions(-)
 delete mode 100644 meta/recipes-connectivity/kea/files/0001-kea-fix-reproducible-build-failure.patch

diff --git a/meta/recipes-connectivity/kea/files/0001-kea-fix-reproducible-build-failure.patch b/meta/recipes-connectivity/kea/files/0001-kea-fix-reproducible-build-failure.patch
deleted file mode 100644
index 8a5bd00302..0000000000
--- a/meta/recipes-connectivity/kea/files/0001-kea-fix-reproducible-build-failure.patch
+++ /dev/null
@@ -1,62 +0,0 @@
-From f9bcfed5a1d44d9211c5f6eba403a9898c8c9057 Mon Sep 17 00:00:00 2001
-From: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
-Date: Tue, 8 Aug 2023 19:03:13 +0100
-Subject: [PATCH] kea: fix reproducible build failure
-
-New version of Kea has started using path of build-dir instead of
-src-dir which results in reproducible builds failure.
-Use src-dir as is used in v2.2.0
-
-Upstream-Status: Pending
-https://gitlab.isc.org/isc-projects/kea/-/issues/3007
-
-Upstream has confirmed the patch will not be accepted but discussions
-with upstream is still going on, we might have a proper solution later.
-
-Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
----
- src/bin/admin/kea-admin.in | 8 ++++----
- 1 file changed, 4 insertions(+), 4 deletions(-)
-
-diff --git a/src/bin/admin/kea-admin.in b/src/bin/admin/kea-admin.in
-index 034a0ee..8ab11ab 100644
---- a/src/bin/admin/kea-admin.in
-+++ b/src/bin/admin/kea-admin.in
-@@ -51,14 +51,14 @@ dump_qry=""
- if test -f "@datarootdir@/@PACKAGE_NAME@/scripts/admin-utils.sh"; then
-     . "@datarootdir@/@PACKAGE_NAME@/scripts/admin-utils.sh"
- else
--    . "@abs_top_builddir@/src/bin/admin/admin-utils.sh"
-+    . "@abs_top_srcdir@/src/bin/admin/admin-utils.sh"
- fi
- 
- # Find the installed kea-lfc if available. Fallback to sources otherwise.
- if test -x "@sbindir@/kea-lfc"; then
-     kea_lfc="@sbindir@/kea-lfc"
- else
--    kea_lfc="@abs_top_builddir@/src/bin/lfc/kea-lfc"
-+    kea_lfc="@abs_top_srcdir@/src/bin/lfc/kea-lfc"
- fi
- 
- # Prints out usage version.
-@@ -355,7 +355,7 @@ mysql_upgrade() {
-     # Check if there are any files in it
-     num_files=$(find "${upgrade_scripts_dir}" -name 'upgrade*.sh' -type f | wc -l)
-     if [ "$num_files" -eq 0 ]; then
--        upgrade_scripts_dir=@abs_top_builddir@/src/share/database/scripts/mysql
-+        upgrade_scripts_dir=@abs_top_srcdir@/src/share/database/scripts/mysql
- 
-         # Check if the scripts directory exists at all.
-         if [ ! -d ${upgrade_scripts_dir} ]; then
-@@ -405,7 +405,7 @@ pgsql_upgrade() {
-     # Check if there are any files in it
-     num_files=$(find "${upgrade_scripts_dir}" -name 'upgrade*.sh' -type f | wc -l)
-     if [ "$num_files" -eq 0 ]; then
--        upgrade_scripts_dir=@abs_top_builddir@/src/share/database/scripts/pgsql
-+        upgrade_scripts_dir=@abs_top_srcdir@/src/share/database/scripts/pgsql
- 
-         # Check if the scripts directory exists at all.
-         if [ ! -d ${upgrade_scripts_dir} ]; then
--- 
-2.39.2
-
diff --git a/meta/recipes-connectivity/kea/kea_2.4.1.bb b/meta/recipes-connectivity/kea/kea_2.4.1.bb
index c3aa4dc8f0..9f8758f379 100644
--- a/meta/recipes-connectivity/kea/kea_2.4.1.bb
+++ b/meta/recipes-connectivity/kea/kea_2.4.1.bb
@@ -17,7 +17,6 @@ SRC_URI = "http://ftp.isc.org/isc/kea/${PV}/${BP}.tar.gz \
            file://fix-multilib-conflict.patch \
            file://fix_pid_keactrl.patch \
            file://0001-src-lib-log-logger_unittest_support.cc-do-not-write-.patch \
-           file://0001-kea-fix-reproducible-build-failure.patch \
            "
 SRC_URI[sha256sum] = "815c61f5c271caa4a1db31dd656eb50a7f6ea973da3690f7c8581408e180131a"
 
@@ -47,7 +46,7 @@ do_configure:prepend() {
     # replace abs_top_builddir to avoid introducing the build path
     # don't expand the abs_top_builddir on the target as the abs_top_builddir is meanlingless on the target
     find ${S} -type f -name *.sh.in | xargs sed -i  "s:@abs_top_builddir@:@abs_top_builddir_placeholder@:g"
-    sed -i "s:@abs_top_srcdir@:@abs_top_srcdir_placeholder@:g" ${S}/src/bin/admin/kea-admin.in
+    sed -i "s:@abs_top_builddir@:@abs_top_builddir_placeholder@:g" ${S}/src/bin/admin/kea-admin.in
 }
 
 # patch out build host paths for reproducibility
-- 
2.34.1



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

* [OE-core][scarthgap 15/29] expect: mark patches as Inactive-Upstream
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (13 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 14/29] kea: remove unnecessary reproducibility patch Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 16/29] apr: submit 0001-Add-option-to-disable-timed-dependant-tests.patch upstream Steve Sakoman
                   ` (13 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

There's every indication that this is an ex-expect:
last release in 2018, stale tickets, cvs server gone.

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 0fcfd2cb51428a3f35c0f78634bff0a16ae654e9)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../expect/expect/0001-Resolve-string-formatting-issues.patch   | 2 +-
 meta/recipes-devtools/expect/expect/0001-configure.in.patch     | 2 +-
 .../expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch      | 2 +-
 ...1-expect-Fix-segfaults-if-Tcl-is-built-with-stubs-and-.patch | 2 +-
 meta/recipes-devtools/expect/expect/0002-tcl.m4.patch           | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-devtools/expect/expect/0001-Resolve-string-formatting-issues.patch b/meta/recipes-devtools/expect/expect/0001-Resolve-string-formatting-issues.patch
index af1d8c626c..bfb6dcd89a 100644
--- a/meta/recipes-devtools/expect/expect/0001-Resolve-string-formatting-issues.patch
+++ b/meta/recipes-devtools/expect/expect/0001-Resolve-string-formatting-issues.patch
@@ -3,7 +3,7 @@ From: Alexander Kanavin <alex.kanavin@gmail.com>
 Date: Thu, 23 Mar 2017 13:44:41 +0200
 Subject: [PATCH] Resolve string formatting issues.
 
-Upstream-Status: Inappropriate [upstream seems dead]
+Upstream-Status: Inactive-Upstream [no activity since 2018; cvs server went read-only]
 Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
 ---
  exp_clib.c | 4 ++--
diff --git a/meta/recipes-devtools/expect/expect/0001-configure.in.patch b/meta/recipes-devtools/expect/expect/0001-configure.in.patch
index 7595a254a8..618c4bee8e 100644
--- a/meta/recipes-devtools/expect/expect/0001-configure.in.patch
+++ b/meta/recipes-devtools/expect/expect/0001-configure.in.patch
@@ -1,7 +1,7 @@
 Allow cross compiling.
 
 Signed-off-by: Anders Roxell <anders.roxell@enea.com>
-Upstream-Status: Pending
+Upstream-Status: Inactive-Upstream [no activity since 2018; cvs server went read-only]
 ---
 diff -uNr a/configure.in b/configure.in
 --- a/configure.in	2012-12-14 15:31:32.623180450 +0100
diff --git a/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch b/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch
index 37512fb9bc..d73f4c3421 100644
--- a/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch
+++ b/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch
@@ -10,7 +10,7 @@ Aborted (core dumped)
 
 Use PATH_MAX to fix the problem.
 
-Upstream-Status: Pending [Upstream seems dead]
+Upstream-Status: Inactive-Upstream [no activity since 2018; cvs server went read-only]
 
 Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
 ---
diff --git a/meta/recipes-devtools/expect/expect/0001-expect-Fix-segfaults-if-Tcl-is-built-with-stubs-and-.patch b/meta/recipes-devtools/expect/expect/0001-expect-Fix-segfaults-if-Tcl-is-built-with-stubs-and-.patch
index b1d322d5c9..40f7f3bd85 100644
--- a/meta/recipes-devtools/expect/expect/0001-expect-Fix-segfaults-if-Tcl-is-built-with-stubs-and-.patch
+++ b/meta/recipes-devtools/expect/expect/0001-expect-Fix-segfaults-if-Tcl-is-built-with-stubs-and-.patch
@@ -42,7 +42,7 @@ Example:
  }
 Author: Sergei Golovan <sgolovan@debian.org>
 
-Upstream-Status: Pending
+Upstream-Status: Inactive-Upstream [no activity since 2018; cvs server went read-only]
 This patch is backported from fedora changes for expect:
 http://pkgs.fedoraproject.org/cgit/rpms/expect.git/commit/
 ?h=master&id=b6737eed550be93182f2ed194e836a6cbbcf4fa3
diff --git a/meta/recipes-devtools/expect/expect/0002-tcl.m4.patch b/meta/recipes-devtools/expect/expect/0002-tcl.m4.patch
index dc4c6ba406..eb178f462a 100644
--- a/meta/recipes-devtools/expect/expect/0002-tcl.m4.patch
+++ b/meta/recipes-devtools/expect/expect/0002-tcl.m4.patch
@@ -1,7 +1,7 @@
 Use proper -L path when cross compiling.
 
 Signed-off-by: Anders Roxell <anders.roxell@enea.com>
-Upstream-Status: Pending
+Upstream-Status: Inactive-Upstream [no activity since 2018; cvs server went read-only]
 ---
 diff -uNr a/tclconfig/tcl.m4 b/tclconfig/tcl.m4
 --- a/tclconfig/tcl.m4	2012-12-14 09:16:58.789861281 +0100
-- 
2.34.1



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

* [OE-core][scarthgap 16/29] apr: submit 0001-Add-option-to-disable-timed-dependant-tests.patch upstream
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (14 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 15/29] expect: mark patches as Inactive-Upstream Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 17/29] busybox: submit CVE-2022-28391 patches upstream Steve Sakoman
                   ` (12 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 1bfe386e69d7a688815a78bd2d5766a4b0665e7f)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../apr/0001-Add-option-to-disable-timed-dependant-tests.patch  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/apr/apr/0001-Add-option-to-disable-timed-dependant-tests.patch b/meta/recipes-support/apr/apr/0001-Add-option-to-disable-timed-dependant-tests.patch
index a274f3a16e..b46dc76a86 100644
--- a/meta/recipes-support/apr/apr/0001-Add-option-to-disable-timed-dependant-tests.patch
+++ b/meta/recipes-support/apr/apr/0001-Add-option-to-disable-timed-dependant-tests.patch
@@ -7,7 +7,7 @@ The disabled tests rely on timing to pass correctly. On a virtualized
 system under heavy load, these tests randomly fail because they miss
 a timer or other timing related issues.
 
-Upstream-Status: Pending
+Upstream-Status: Submitted [https://github.com/apache/apr/pull/54]
 Signed-off-by: Jeremy Puhlman <jpuhlman@mvista.com>
 
 ---
-- 
2.34.1



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

* [OE-core][scarthgap 17/29] busybox: submit CVE-2022-28391 patches upstream
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (15 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 16/29] apr: submit 0001-Add-option-to-disable-timed-dependant-tests.patch upstream Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 18/29] xinput-calibrator: mark upstream as inactive in a patch Steve Sakoman
                   ` (11 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 64c026024b1c86797de338760b9f29d7e949926c)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 ...1-libbb-sockaddr2str-ensure-only-printable-characters-.patch | 2 +-
 ...2-nslookup-sanitize-all-printed-strings-with-printable.patch | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/busybox/busybox/0001-libbb-sockaddr2str-ensure-only-printable-characters-.patch b/meta/recipes-core/busybox/busybox/0001-libbb-sockaddr2str-ensure-only-printable-characters-.patch
index 4635250170..ceb3ad7250 100644
--- a/meta/recipes-core/busybox/busybox/0001-libbb-sockaddr2str-ensure-only-printable-characters-.patch
+++ b/meta/recipes-core/busybox/busybox/0001-libbb-sockaddr2str-ensure-only-printable-characters-.patch
@@ -5,7 +5,7 @@ Subject: [PATCH 1/2] libbb: sockaddr2str: ensure only printable characters are
  returned for the hostname part
 
 CVE: CVE-2022-28391
-Upstream-Status: Pending
+Upstream-Status: Submitted [https://bugs.busybox.net/show_bug.cgi?id=15001]
 Signed-off-by: Ariadne Conill <ariadne@dereferenced.org>
 Signed-off-by: Steve Sakoman <steve@sakoman.com>
 ---
diff --git a/meta/recipes-core/busybox/busybox/0002-nslookup-sanitize-all-printed-strings-with-printable.patch b/meta/recipes-core/busybox/busybox/0002-nslookup-sanitize-all-printed-strings-with-printable.patch
index 0d7409ddc3..1dbc3388a4 100644
--- a/meta/recipes-core/busybox/busybox/0002-nslookup-sanitize-all-printed-strings-with-printable.patch
+++ b/meta/recipes-core/busybox/busybox/0002-nslookup-sanitize-all-printed-strings-with-printable.patch
@@ -8,7 +8,7 @@ Otherwise, terminal sequences can be injected, which enables various terminal in
 attacks from DNS results.
 
 CVE: CVE-2022-28391
-Upstream-Status: Pending
+Upstream-Status: Submitted [https://bugs.busybox.net/show_bug.cgi?id=15001]
 Signed-off-by: Ariadne Conill <ariadne@dereferenced.org>
 Signed-off-by: Steve Sakoman <steve@sakoman.com>
 ---
-- 
2.34.1



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

* [OE-core][scarthgap 18/29] xinput-calibrator: mark upstream as inactive in a patch
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (16 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 17/29] busybox: submit CVE-2022-28391 patches upstream Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 19/29] bash: mark build-tests.patch as Inappropriate Steve Sakoman
                   ` (10 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 959591615d18f636768671e1e14e5945a6cbb010)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 .../Allow-xinput_calibrator_pointercal.sh-to-be-run-as-n.patch  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/xinput-calibrator/xinput-calibrator/Allow-xinput_calibrator_pointercal.sh-to-be-run-as-n.patch b/meta/recipes-graphics/xinput-calibrator/xinput-calibrator/Allow-xinput_calibrator_pointercal.sh-to-be-run-as-n.patch
index 86982924a8..de0862599d 100644
--- a/meta/recipes-graphics/xinput-calibrator/xinput-calibrator/Allow-xinput_calibrator_pointercal.sh-to-be-run-as-n.patch
+++ b/meta/recipes-graphics/xinput-calibrator/xinput-calibrator/Allow-xinput_calibrator_pointercal.sh-to-be-run-as-n.patch
@@ -1,4 +1,4 @@
-Upstream-Status: Pending
+Upstream-Status: Inactive-Upstream [last commits over a decade ago]
 
 From 14734a93bd3fc323325459e24b04795422e395e6 Mon Sep 17 00:00:00 2001
 From: Laurentiu Palcu <laurentiu.palcu@intel.com>
-- 
2.34.1



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

* [OE-core][scarthgap 19/29] bash: mark build-tests.patch as Inappropriate
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (17 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 18/29] xinput-calibrator: mark upstream as inactive in a patch Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 20/29] icu: add upstream submission links for fix-install-manx.patch Steve Sakoman
                   ` (9 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 3d82d44ed6822b7a276df355311e384c02d36c43)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-extended/bash/bash/build-tests.patch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/bash/bash/build-tests.patch b/meta/recipes-extended/bash/bash/build-tests.patch
index ea38bace9b..c1b9b8261f 100644
--- a/meta/recipes-extended/bash/bash/build-tests.patch
+++ b/meta/recipes-extended/bash/bash/build-tests.patch
@@ -4,7 +4,7 @@ Date: Wed, 19 Dec 2012 17:18:31 +0100
 Subject: [PATCH] Add 'ptest' target to Makefile, to run tests without checking
  dependencies.
 
-Upstream-Status: Pending
+Upstream-Status: Inappropriate [ptest specific]
 Signed-off-by: Anders Roxell <anders.roxell@enea.com>
 
 Rebase to 5.0
-- 
2.34.1



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

* [OE-core][scarthgap 20/29] icu: add upstream submission links for fix-install-manx.patch
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (18 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 19/29] bash: mark build-tests.patch as Inappropriate Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 21/29] ppp: Add RSA-MD in LICENSE Steve Sakoman
                   ` (8 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexander Kanavin <alex.kanavin@gmail.com>

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 269a1a65d2526be5622b4c1f552b6f5541e0cb86)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-support/icu/icu/fix-install-manx.patch | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-support/icu/icu/fix-install-manx.patch b/meta/recipes-support/icu/icu/fix-install-manx.patch
index 7526bde370..16cabc8264 100644
--- a/meta/recipes-support/icu/icu/fix-install-manx.patch
+++ b/meta/recipes-support/icu/icu/fix-install-manx.patch
@@ -11,7 +11,9 @@ and one process tries to chown a file that the other process has just deleted.
 Also install-manx should be a phony target, and for clarity use $^ instead of $?
 in the install command.
 
-Upstream-Status: Pending
+Upstream ticket: https://unicode-org.atlassian.net/jira/software/c/projects/ICU/issues/ICU-21172
+
+Upstream-Status: Submitted [https://github.com/unicode-org/icu/pull/2966]
 Signed-off-by: Ross Burton <ross.burton@intel.com>
 ---
  Makefile.in | 8 ++++----
-- 
2.34.1



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

* [OE-core][scarthgap 21/29] ppp: Add RSA-MD in LICENSE
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (19 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 20/29] icu: add upstream submission links for fix-install-manx.patch Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 22/29] iproute2: drop obsolete patch Steve Sakoman
                   ` (7 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Poonam Jadhav <ppjadhav456@gmail.com>

ppp package has "RSA Data Security" license text in
Message-Digest Algorithm source file ppp-md5.c and ppp-md4.c
Add RSA-MD in LICENSE field for ppp package

Signed-off-by: Poonam Jadhav <poonam.jadhav@kpit.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 9c58193c898b0ea28c3931a57dca5bb829a44051)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-connectivity/ppp/ppp_2.5.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/ppp/ppp_2.5.0.bb b/meta/recipes-connectivity/ppp/ppp_2.5.0.bb
index 4b052f8ed9..5f0c75de83 100644
--- a/meta/recipes-connectivity/ppp/ppp_2.5.0.bb
+++ b/meta/recipes-connectivity/ppp/ppp_2.5.0.bb
@@ -5,7 +5,7 @@ SECTION = "console/network"
 HOMEPAGE = "http://samba.org/ppp/"
 BUGTRACKER = "http://ppp.samba.org/cgi-bin/ppp-bugs"
 DEPENDS = "libpcap openssl virtual/crypt"
-LICENSE = "BSD-3-Clause & BSD-3-Clause-Attribution & GPL-2.0-or-later & LGPL-2.0-or-later & PD"
+LICENSE = "BSD-3-Clause & BSD-3-Clause-Attribution & GPL-2.0-or-later & LGPL-2.0-or-later & PD & RSA-MD"
 LIC_FILES_CHKSUM = "file://pppd/ccp.c;beginline=1;endline=29;md5=e2c43fe6e81ff77d87dc9c290a424dea \
                     file://pppd/plugins/passprompt.c;beginline=1;endline=10;md5=3bcbcdbf0e369c9a3e0b8c8275b065d8 \
                     file://pppd/tdb.c;beginline=1;endline=27;md5=4ca3a9991b011038d085d6675ae7c4e6 \
-- 
2.34.1



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

* [OE-core][scarthgap 22/29] iproute2: drop obsolete patch
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (20 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 21/29] ppp: Add RSA-MD in LICENSE Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 23/29] kernel.bbclass: check, if directory exists before removing empty module directory Steve Sakoman
                   ` (6 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: "Maxin B. John" <maxin.john@gmail.com>

libc-compat.h fix for musl was obsolete after 4.16.0 release of iproute2.
Drop it.

Signed-off-by: Maxin John <maxin.john@gmail.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit ad57a1e124a1de7af7ef8479ed779230e923d3af)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 ...01-libc-compat.h-add-musl-workaround.patch | 39 -------------------
 .../iproute2/iproute2_6.7.0.bb                |  4 +-
 2 files changed, 1 insertion(+), 42 deletions(-)
 delete mode 100644 meta/recipes-connectivity/iproute2/iproute2/0001-libc-compat.h-add-musl-workaround.patch

diff --git a/meta/recipes-connectivity/iproute2/iproute2/0001-libc-compat.h-add-musl-workaround.patch b/meta/recipes-connectivity/iproute2/iproute2/0001-libc-compat.h-add-musl-workaround.patch
deleted file mode 100644
index 74e3de1ce9..0000000000
--- a/meta/recipes-connectivity/iproute2/iproute2/0001-libc-compat.h-add-musl-workaround.patch
+++ /dev/null
@@ -1,39 +0,0 @@
-From c25f8d1f7a6203dfeb10b39f80ffd314bb84a58d Mon Sep 17 00:00:00 2001
-From: Baruch Siach <baruch@tkos.co.il>
-Date: Thu, 22 Dec 2016 15:26:30 +0200
-Subject: [PATCH] libc-compat.h: add musl workaround
-
-The libc-compat.h kernel header uses glibc specific macros (__GLIBC__ and
-__USE_MISC) to solve conflicts with libc provided headers. This patch makes
-libc-compat.h work for musl libc as well.
-
-Upstream-Status: Pending
-
-Taken From:
-https://git.buildroot.net/buildroot/tree/package/iproute2/0001-Add-the-musl-workaround-to-the-libc-compat.h-copy.patch
-
-Signed-off-by: Baruch Siach <baruch@tkos.co.il>
-Signed-off-by: Maxin B. John <maxin.john@intel.com>
-
----
- include/uapi/linux/libc-compat.h | 4 +++-
- 1 file changed, 3 insertions(+), 1 deletion(-)
-
-diff --git a/include/uapi/linux/libc-compat.h b/include/uapi/linux/libc-compat.h
-index a159991..22198fa 100644
---- a/include/uapi/linux/libc-compat.h
-+++ b/include/uapi/linux/libc-compat.h
-@@ -50,10 +50,12 @@
- #define _LIBC_COMPAT_H
- 
- /* We have included glibc headers... */
--#if defined(__GLIBC__)
-+#if 1
-+#define __USE_MISC
- 
- /* Coordinate with glibc net/if.h header. */
- #if defined(_NET_IF_H) && defined(__USE_MISC)
-+#define __UAPI_DEF_IF_NET_DEVICE_FLAGS_LOWER_UP_DORMANT_ECHO 0
- 
- /* GLIBC headers included first so don't define anything
-  * that would already be defined. */
diff --git a/meta/recipes-connectivity/iproute2/iproute2_6.7.0.bb b/meta/recipes-connectivity/iproute2/iproute2_6.7.0.bb
index 640b3013f1..442c6bdd4f 100644
--- a/meta/recipes-connectivity/iproute2/iproute2_6.7.0.bb
+++ b/meta/recipes-connectivity/iproute2/iproute2_6.7.0.bb
@@ -11,9 +11,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=eb723b61539feef013de476e68b5c50a \
 
 DEPENDS = "flex-native bison-native iptables libcap"
 
-SRC_URI = "${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BP}.tar.xz \
-           file://0001-libc-compat.h-add-musl-workaround.patch \
-           "
+SRC_URI = "${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BP}.tar.xz"
 
 SRC_URI[sha256sum] = "ff942dd9828d7d1f867f61fe72ce433078c31e5d8e4a78e20f02cb5892e8841d"
 
-- 
2.34.1



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

* [OE-core][scarthgap 23/29] kernel.bbclass: check, if directory exists before removing empty module directory
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (21 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 22/29] iproute2: drop obsolete patch Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 24/29] iputils: splitting the ping6 as a package Steve Sakoman
                   ` (5 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Heiko <heiko.thole@entwicklung.eq-3.de>

If the kernel folder does not exist, find will result in an error.
This can occur if the kernel has no modules but, for example, custom modules are created.

Add check before deleting.

Signed-off-by: Heiko Thole <heiko.thole@entwicklung.eq-3.de>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 7ef767d84d56b25498e45db83bb8f9d9caebeaf9)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/classes-recipe/kernel.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes-recipe/kernel.bbclass b/meta/classes-recipe/kernel.bbclass
index b084d6d69d..c0a2056fec 100644
--- a/meta/classes-recipe/kernel.bbclass
+++ b/meta/classes-recipe/kernel.bbclass
@@ -463,7 +463,7 @@ kernel_do_install() {
 		rm -f "${D}${nonarch_base_libdir}/modules/${KERNEL_VERSION}/build"
 		rm -f "${D}${nonarch_base_libdir}/modules/${KERNEL_VERSION}/source"
 		# Remove empty module directories to prevent QA issues
-		find "${D}${nonarch_base_libdir}/modules/${KERNEL_VERSION}/kernel" -type d -empty -delete
+		[ -d "${D}${nonarch_base_libdir}/modules/${KERNEL_VERSION}/kernel" ] && find "${D}${nonarch_base_libdir}/modules/${KERNEL_VERSION}/kernel" -type d -empty -delete
 	else
 		bbnote "no modules to install"
 	fi
-- 
2.34.1



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

* [OE-core][scarthgap 24/29] iputils: splitting the ping6 as a package
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (22 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 23/29] kernel.bbclass: check, if directory exists before removing empty module directory Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 25/29] ltp: add iputils-ping6 to RDEPENDS Steve Sakoman
                   ` (4 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Xiangyu Chen <xiangyu.chen@windriver.com>

The iputils merged the ping4 and ping6 command into the ping since the
commit ebad35fe[1], currently, iputils ping support being called using
"ping -6" or "ping6" symlinks.

In oe-core, there are two packages provide ipv6 ping, inetutils and iputils,
the inetutils-ping6 doesn't support "-I" option to bind the interface, that
cause some of scripts(e.g. LTP test[2]) which was based on iputils-ping6 fails.

If someone or a package needs iputils-ping6, they can enable it in local.conf
or add it into rdepends directly.

Ref:
[1] https://github.com/iputils/iputils/commit/e3de851b809c7b72ccc654a72b6af61d
[2] https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/ \
    containers/netns/netns_comm.sh#L25

Signed-off-by: Xiangyu Chen <xiangyu.chen@windriver.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
(cherry picked from commit 0b89bc2041f670af625829d061ce79e0840ac116)
Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-extended/iputils/iputils_20240117.bb | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/iputils/iputils_20240117.bb b/meta/recipes-extended/iputils/iputils_20240117.bb
index 5a5e15528e..3880689742 100644
--- a/meta/recipes-extended/iputils/iputils_20240117.bb
+++ b/meta/recipes-extended/iputils/iputils_20240117.bb
@@ -35,7 +35,11 @@ ALTERNATIVE_PRIORITY = "100"
 ALTERNATIVE:${PN}-ping = "ping"
 ALTERNATIVE_LINK_NAME[ping] = "${base_bindir}/ping"
 
-SPLITPKGS = "${PN}-ping ${PN}-arping ${PN}-tracepath ${PN}-clockdiff"
+ALTERNATIVE:${PN}-ping6 = "ping6"
+ALTERNATIVE_LINK_NAME[ping6] = "${base_bindir}/ping6"
+
+SPLITPKGS = "${PN}-ping ${PN}-arping ${PN}-tracepath ${PN}-clockdiff \
+             ${@bb.utils.contains('DISTRO_FEATURES', 'ipv6', '${PN}-ping6', '', d)}"
 PACKAGES += "${SPLITPKGS}"
 
 ALLOW_EMPTY:${PN} = "1"
@@ -43,6 +47,13 @@ RDEPENDS:${PN} += "${SPLITPKGS}"
 
 FILES:${PN} = ""
 FILES:${PN}-ping = "${base_bindir}/ping.${BPN}"
+FILES:${PN}-ping6 = "${base_bindir}/ping6.${BPN}"
 FILES:${PN}-arping = "${base_bindir}/arping"
 FILES:${PN}-tracepath = "${base_bindir}/tracepath"
 FILES:${PN}-clockdiff = "${base_bindir}/clockdiff"
+
+do_install:append() {
+	if ${@bb.utils.contains('DISTRO_FEATURES', 'ipv6', 'true', 'false', d)}; then
+		ln -sf ping ${D}/${base_bindir}/ping6
+	fi
+}
-- 
2.34.1



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

* [OE-core][scarthgap 25/29] ltp: add iputils-ping6 to RDEPENDS
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (23 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 24/29] iputils: splitting the ping6 as a package Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 26/29] libinput: fix building with debug-gui option Steve Sakoman
                   ` (3 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Xiangyu Chen <xiangyu.chen@windriver.com>

According to the LTP documents[1], it using ping6 from iputils which
support "-I" option to bind the interface[2].

Ref:
[1] https://github.com/linux-test-project/ltp/commit/6f97789c
[2] https://github.com/linux-test-project/ltp/commit/64b11656

Signed-off-by: Xiangyu Chen <xiangyu.chen@windriver.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 05107ec5e4b81315d53efef23f2a570d520ee4c7)
Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-extended/ltp/ltp_20240129.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-extended/ltp/ltp_20240129.bb b/meta/recipes-extended/ltp/ltp_20240129.bb
index 3e896957d1..e88e931a68 100644
--- a/meta/recipes-extended/ltp/ltp_20240129.bb
+++ b/meta/recipes-extended/ltp/ltp_20240129.bb
@@ -98,6 +98,7 @@ RDEPENDS:${PN} = "\
     gdb \
     gzip \
     iproute2 \
+    ${@bb.utils.contains('DISTRO_FEATURES', 'ipv6', 'iputils-ping6', '', d)} \
     ldd \
     libaio \
     logrotate \
-- 
2.34.1



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

* [OE-core][scarthgap 26/29] libinput: fix building with debug-gui option
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (24 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 25/29] ltp: add iputils-ping6 to RDEPENDS Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 27/29] oeqa/selftest/devtool: fix _test_devtool_add_git_url Steve Sakoman
                   ` (2 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Marc Ferland <marc.ferland@sonatest.com>

When building libinput with the 'gui' PACKAGECONFIG option, it fails
with:

| Run-time dependency x11 found: YES 1.8.9
| Run-time dependency wayland-client found: YES 1.22.0
| Run-time dependency wayland-protocols found: YES 1.34
| Program wayland-scanner found: NO
|
| ../git/meson.build:578:20: ERROR: Program 'wayland-scanner' not found or not executable
|
| A full log can be found at /home/marc/mnt/yocto-latest/build/tmp/work/core2-64-poky-linux/libinput/1.25.0/build/meson-logs/meson-log.txt
| ERROR: meson failed
| WARNING: exit code 1 from a shell command.

Adding a build dependency on wayland-native fixes the issue.

Signed-off-by: Marc Ferland <marc.ferland@sonatest.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 0e2d18e6267d26870ccbe45734bfccbc02744357)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-graphics/wayland/libinput_1.25.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/wayland/libinput_1.25.0.bb b/meta/recipes-graphics/wayland/libinput_1.25.0.bb
index 517b247fed..894858e361 100644
--- a/meta/recipes-graphics/wayland/libinput_1.25.0.bb
+++ b/meta/recipes-graphics/wayland/libinput_1.25.0.bb
@@ -32,7 +32,7 @@ do_configure:append() {
 
 PACKAGECONFIG ??= "${@bb.utils.contains('PTEST_ENABLED', '1', 'tests', '', d)}"
 PACKAGECONFIG[libwacom] = "-Dlibwacom=true,-Dlibwacom=false,libwacom"
-PACKAGECONFIG[gui] = "-Ddebug-gui=true,-Ddebug-gui=false,cairo gtk+3"
+PACKAGECONFIG[gui] = "-Ddebug-gui=true,-Ddebug-gui=false,cairo gtk+3 wayland-native"
 PACKAGECONFIG[tests] = "-Dtests=true -Dinstall-tests=true,-Dtests=false -Dinstall-tests=false,libcheck"
 
 UDEVDIR = "`pkg-config --variable=udevdir udev`"
-- 
2.34.1



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

* [OE-core][scarthgap 27/29] oeqa/selftest/devtool: fix _test_devtool_add_git_url
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (25 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 26/29] libinput: fix building with debug-gui option Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 28/29] Revert "oeqa/selftest/devtool: fix test_devtool_add_git_style2" Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 29/29] cracklib: Modify patch to compile with GCC 14 Steve Sakoman
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexandre Truong <alexandre.truong@smile.fr>

This patch is a follow-up to the bug#15466.

As a reminder, the bug was about devtool’s submodule detection
checking for HEAD when a version is being passed.

As Vincent Kriek pointed out:
the --version that is being passed to devtool is only used for
setting the PV value in the recipe. To take into account the tag,
we add --srcrev to the command:
devtool add --srcrev v3.1.0 --version v3.1.0 mbedtls git://git@github.com/ARMmbed/mbedtls.git;protocol=https

Changes to _test_devtool_add_git_url have been made to take
into account the srcrev. srcrev will be passed as an optional
parameter because the test_devtool_add_git_style1
does not need the srcrev contrary to test_devtool_add_git_style2

Fixes [YOCTO #15466]

Signed-off-by: Alexandre Truong <alexandre.truong@smile.fr>
Reported-by: Alexandre Truong <alexandre.truong@smile.fr>
Suggested-by: Vincent Kriek <vincent@coelebs.dev>
Reviewed-by: Yoann Congal <yoann.congal@smile.fr>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit a8686f3641e4407dee3d807898ffd620e2732b78)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/lib/oeqa/selftest/cases/devtool.py | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py
index cc4cbec5ae..1597aeedff 100644
--- a/meta/lib/oeqa/selftest/cases/devtool.py
+++ b/meta/lib/oeqa/selftest/cases/devtool.py
@@ -286,10 +286,13 @@ class DevtoolTestCase(OESelftestTestCase):
         else:
             self.skipTest('No tap devices found - you must set up tap devices with scripts/runqemu-gen-tapdevs before running this test')
 
-    def _test_devtool_add_git_url(self, git_url, version, pn, resulting_src_uri):
+    def _test_devtool_add_git_url(self, git_url, version, pn, resulting_src_uri, srcrev=None):
         self.track_for_cleanup(self.workspacedir)
         self.add_command_to_tearDown('bitbake-layers remove-layer */workspace')
-        result = runCmd('devtool add --version %s %s %s' % (version, pn, git_url))
+        command = 'devtool add --version %s %s %s' % (version, pn, git_url)
+        if srcrev :
+            command += ' --srcrev %s' %srcrev
+        result = runCmd(command)
         self.assertExists(os.path.join(self.workspacedir, 'conf', 'layer.conf'), 'Workspace directory not created')
         # Check the recipe name is correct
         recipefile = get_bb_var('FILE', pn)
@@ -479,11 +482,12 @@ class DevtoolAddTests(DevtoolBase):
 
     def test_devtool_add_git_style2(self):
         version = 'v3.1.0'
+        srcrev = 'v3.1.0'
         pn = 'mbedtls'
         # this will trigger reformat_git_uri with branch parameter in url
         git_url = "'git://git@github.com/ARMmbed/mbedtls.git;protocol=https'"
         resulting_src_uri = "gitsm://git@github.com/ARMmbed/mbedtls.git;protocol=https;branch=master"
-        self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri)
+        self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri, srcrev)
 
     def test_devtool_add_library(self):
         # Fetch source
-- 
2.34.1



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

* [OE-core][scarthgap 28/29] Revert "oeqa/selftest/devtool: fix test_devtool_add_git_style2"
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (26 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 27/29] oeqa/selftest/devtool: fix _test_devtool_add_git_url Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  2024-06-13 14:20 ` [OE-core][scarthgap 29/29] cracklib: Modify patch to compile with GCC 14 Steve Sakoman
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Alexandre Truong <alexandre.truong@smile.fr>

This reverts commit ab6d3e3d645ffc343f434bf731339fa237df027a
from poky repository.

The previous reverted commit was a workaround. The fix
"oeqa/selftest/devtool: fix _test_devtool_add_git_url"
tackle the issue. So, the workaround is not needed anymore.

Signed-off-by: Alexandre Truong <alexandre.truong@smile.fr>
Reviewed-by: Yoann Congal <yoann.congal@smile.fr>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 731f47ecfd8ad6558aac629806810789c623986b)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/lib/oeqa/selftest/cases/devtool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py
index 1597aeedff..44a2a50f2e 100644
--- a/meta/lib/oeqa/selftest/cases/devtool.py
+++ b/meta/lib/oeqa/selftest/cases/devtool.py
@@ -486,7 +486,7 @@ class DevtoolAddTests(DevtoolBase):
         pn = 'mbedtls'
         # this will trigger reformat_git_uri with branch parameter in url
         git_url = "'git://git@github.com/ARMmbed/mbedtls.git;protocol=https'"
-        resulting_src_uri = "gitsm://git@github.com/ARMmbed/mbedtls.git;protocol=https;branch=master"
+        resulting_src_uri = "git://git@github.com/ARMmbed/mbedtls.git;protocol=https;branch=master"
         self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri, srcrev)
 
     def test_devtool_add_library(self):
-- 
2.34.1



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

* [OE-core][scarthgap 29/29] cracklib: Modify patch to compile with GCC 14
  2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
                   ` (27 preceding siblings ...)
  2024-06-13 14:20 ` [OE-core][scarthgap 28/29] Revert "oeqa/selftest/devtool: fix test_devtool_add_git_style2" Steve Sakoman
@ 2024-06-13 14:20 ` Steve Sakoman
  28 siblings, 0 replies; 30+ messages in thread
From: Steve Sakoman @ 2024-06-13 14:20 UTC (permalink / raw)
  To: openembedded-core

From: Zoltan Boszormenyi <zboszor@gmail.com>

GCC 14 implicitly turns a warning into a compiler error:

| ../../git/src/lib/packlib.c: In function ‘PWClose’:
| ../../git/src/lib/packlib.c:554:40: error: passing argument 1 of ‘HwmsHostToBigEndian’ from incompatible pointer type [-Wincompatible-pointer-types]
|   554 |             HwmsHostToBigEndian(tmp_pwp.hwms, sizeof(tmp_pwp.hwms), en_is32);
|       |                                 ~~~~~~~^~~~~
|       |                                        |
|       |                                        uint32_t * {aka unsigned int *}
| ../../git/src/lib/packlib.c:142:27: note: expected ‘char *’ but argument is of type ‘uint32_t *’ {aka ‘unsigned int *’}
|   142 | HwmsHostToBigEndian(char *pHwms, int nLen,int nBitType)
|       |                     ~~~~~~^~~~~

Add the cast to (char *) to silence it.

Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 914128f6bd988cde278e087fb9457a0c70c7e5ec)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 ...0001-packlib.c-support-dictionary-byte-order-dependent.patch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/cracklib/cracklib/0001-packlib.c-support-dictionary-byte-order-dependent.patch b/meta/recipes-extended/cracklib/cracklib/0001-packlib.c-support-dictionary-byte-order-dependent.patch
index 20572b55c4..35229ae890 100644
--- a/meta/recipes-extended/cracklib/cracklib/0001-packlib.c-support-dictionary-byte-order-dependent.patch
+++ b/meta/recipes-extended/cracklib/cracklib/0001-packlib.c-support-dictionary-byte-order-dependent.patch
@@ -303,7 +303,7 @@ index 9396e1d..d0bb181 100644
 +	    PWDICT tmp_pwp;
 +
 +	    memcpy(&tmp_pwp, pwp, sizeof(PWDICT));
-+	    HwmsHostToBigEndian(tmp_pwp.hwms, sizeof(tmp_pwp.hwms), en_is32);
++	    HwmsHostToBigEndian((char *)tmp_pwp.hwms, sizeof(tmp_pwp.hwms), en_is32);
 +	    fwrite(tmp_pwp.hwms, 1, sizeof(tmp_pwp.hwms), pwp->wfp);
  	}
      }
-- 
2.34.1



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

end of thread, other threads:[~2024-06-13 14:21 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-13 14:20 [OE-core][scarthgap 00/29] Patch review Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 01/29] cronie: upgrade 1.7.1 -> 1.7.2 Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 02/29] lttng-ust: upgrade 2.13.7 -> 2.13.8 Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 03/29] taglib: upgrade 2.0 -> 2.0.1 Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 04/29] mesa: upgrade 24.0.3 -> 24.0.5 Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 05/29] procps: fix build with new glibc but old kernel headers Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 06/29] devtool: modify: Catch git submodule error for go code Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 07/29] oeqa/selftest/devtool: add test for modifying recipes using go.bbclass Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 08/29] base-files: profile: fix error sh: 1: unknown operand Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 09/29] oeqa/postactions: Do not use -l option with df Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 10/29] oeqa: selftest: context: run tests serially if testtools/subunit modules are not found Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 11/29] libtraceevent: submit meson.patch upstream Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 12/29] devtool: sync: Fix Execution error Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 13/29] serf: mark patch as inappropriate for upstream submission Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 14/29] kea: remove unnecessary reproducibility patch Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 15/29] expect: mark patches as Inactive-Upstream Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 16/29] apr: submit 0001-Add-option-to-disable-timed-dependant-tests.patch upstream Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 17/29] busybox: submit CVE-2022-28391 patches upstream Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 18/29] xinput-calibrator: mark upstream as inactive in a patch Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 19/29] bash: mark build-tests.patch as Inappropriate Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 20/29] icu: add upstream submission links for fix-install-manx.patch Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 21/29] ppp: Add RSA-MD in LICENSE Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 22/29] iproute2: drop obsolete patch Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 23/29] kernel.bbclass: check, if directory exists before removing empty module directory Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 24/29] iputils: splitting the ping6 as a package Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 25/29] ltp: add iputils-ping6 to RDEPENDS Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 26/29] libinput: fix building with debug-gui option Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 27/29] oeqa/selftest/devtool: fix _test_devtool_add_git_url Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 28/29] Revert "oeqa/selftest/devtool: fix test_devtool_add_git_style2" Steve Sakoman
2024-06-13 14:20 ` [OE-core][scarthgap 29/29] cracklib: Modify patch to compile with GCC 14 Steve Sakoman

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