from:"Jörg Sommer via lists . openembedded . org"

Re: [OE-core] [PATCH] classes/kernel: No symlink in postinst without KERNEL_IMAGETYPE_SYMLINK

2024-06-09 Thread Jörg Sommer via lists . openembedded . org

Jörg Sommer via lists.openembedded.org schrieb am So 09. Jun, 16:59 (GMT):
> diff --git a/meta/classes-recipe/kernel.bbclass 
> b/meta/classes-recipe/kernel.bbclass
> index d6eedf942c..89badd90f1 100644
> --- a/meta/classes-recipe/kernel.bbclass
> +++ b/meta/classes-recipe/kernel.bbclass
> @@ -115,7 +115,9 @@ python __anonymous () {
>  
>  d.setVar('PKG:%s-image-%s' % (kname,typelower), 
> '%s-image-%s-${KERNEL_VERSION_PKG_NAME}' % (kname, typelower))
>  d.setVar('ALLOW_EMPTY:%s-image-%s' % (kname, typelower), '1')
> -d.prependVar('pkg_postinst:%s-image-%s' % (kname,typelower), """set 
> +e
> +
> +if d.getVar('KERNEL_IMAGETYPE_SYMLINK') == '1':
> +d.prependVar('pkg_postinst:%s-image-%s' % (kname,typelower), 
> """set +e
>  if [ -n "$D" ]; then
>  ln -sf %s-${KERNEL_VERSION} $D/${KERNEL_IMAGEDEST}/%s > /dev/null 2>&1
>  else
> @@ -127,7 +129,7 @@ else
>  fi
>  set -e
>  """ % (type, type, type, type, type, type, type))
> -d.setVar('pkg_postrm:%s-image-%s' % (kname,typelower), """set +e
> +d.setVar('pkg_postrm:%s-image-%s' % (kname,typelower), """set +e

While backporting this patch to kirkstone I realised this uses `setVar`
instead of `prependVar` (as above). Is this right?



smime.p7s
Description: S/MIME cryptographic signature

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#200461): 
https://lists.openembedded.org/g/openembedded-core/message/200461
Mute This Topic: https://lists.openembedded.org/mt/106576203/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] classes/kernel: No symlink in postinst without KERNEL_IMAGETYPE_SYMLINK

2024-06-09 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

The commit “Use a copy of image for kernel*.rpm if fs doesn't support
symlinks” [1] added postinst and postrm scripts to the kernel package which
create a symlink after package installation. This should not happen if
`KERNEL_IMAGETYPE_SYMLINK` is not `1`.

Background: The u-boot implementation of jffs2 does not support symlinks.
Using a hardlink or removing `${KERNEL_VERSION}` from the file name fails,
because the current postinst script replaces the file with the symlink.

[1] 8b6b95106a5d4f1f6d34209ec5c475c900270ecd

Cc: Bruce Ashfield 
Cc: Richard Purdie 
Cc: Yanfei Xu 
Signed-off-by: Jörg Sommer 
---
 meta/classes-recipe/kernel.bbclass | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/meta/classes-recipe/kernel.bbclass 
b/meta/classes-recipe/kernel.bbclass
index d6eedf942c..89badd90f1 100644
--- a/meta/classes-recipe/kernel.bbclass
+++ b/meta/classes-recipe/kernel.bbclass
@@ -115,7 +115,9 @@ python __anonymous () {
 
 d.setVar('PKG:%s-image-%s' % (kname,typelower), 
'%s-image-%s-${KERNEL_VERSION_PKG_NAME}' % (kname, typelower))
 d.setVar('ALLOW_EMPTY:%s-image-%s' % (kname, typelower), '1')
-d.prependVar('pkg_postinst:%s-image-%s' % (kname,typelower), """set +e
+
+if d.getVar('KERNEL_IMAGETYPE_SYMLINK') == '1':
+d.prependVar('pkg_postinst:%s-image-%s' % (kname,typelower), 
"""set +e
 if [ -n "$D" ]; then
 ln -sf %s-${KERNEL_VERSION} $D/${KERNEL_IMAGEDEST}/%s > /dev/null 2>&1
 else
@@ -127,7 +129,7 @@ else
 fi
 set -e
 """ % (type, type, type, type, type, type, type))
-d.setVar('pkg_postrm:%s-image-%s' % (kname,typelower), """set +e
+d.setVar('pkg_postrm:%s-image-%s' % (kname,typelower), """set +e
 if [ -f "${KERNEL_IMAGEDEST}/%s" -o -L "${KERNEL_IMAGEDEST}/%s" ]; then
 rm -f ${KERNEL_IMAGEDEST}/%s  > /dev/null 2>&1
 fi
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#200460): 
https://lists.openembedded.org/g/openembedded-core/message/200460
Mute This Topic: https://lists.openembedded.org/mt/106576203/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] New to the group

2024-05-30 Thread Jörg Sommer via lists . openembedded . org

faganronan via lists.openembedded.org schrieb am Do 23. Mai, 12:58 (GMT):
> I'm brand new to the group, apologies if this is not the right spot. I've
> been working a lot with SBOMs for embedded devices lately and been playing
> around with the recipes for spdx and cve with limited success. The SPDX on
> even a minimal image creates 500ish SPDX files and then it has to be with
> the CPE data. I've had some good success with CycloneDX. Anyone else
> running in to this?

Yes, I'm using CycloneDX with DependencyTrack, and wrote a converter of spdx
to CycloneDX. The script also handles the upload to DependencyTrack, which
is not that easy, because you have to wait for the end of the processing,
before going on.

A better approach would we a cyclonedx class, and I plan to create one, but
didn't found the time to do.

Here is the current version of the script:

```
#!/bin/sh

# There will never again be support for SPDX by DependencyTrack
# https://github.com/DependencyTrack/dependency-track/issues/1053

set -e -u -C

no_upload=false
recipe_filter=\*
skip_sbom=false
skip_vex=false

while test $# -gt 0
do
case "$1" in
--no-upload) no_upload=true;;
--skip-sbom) skip_sbom=true;;
--skip-vex) skip_vex=true;;
--recipe-filter)
shift
recipe_filter=${1:?Argument for --recipe-filter missing}
;;
*) break;;
esac

shift
done

if ! $no_upload && test -z "${DT_API_KEY:-}"
then
echo 'Environment variable DT_API_KEY not set' >&2
exit 1
fi

if test $# -ne 4
then
echo "Usage: $0 [OPTIONS] HOST_URL PROJECT_NAME SPDX_TAR_ZST 
PROJECT_VERSION" >&2
echo >&2
echo 'OPTIONS:' >&2
echo '--no-uploaddo not upload, but print to stdout' >&2
echo '--recipe-filter FILTER only process recipes matched by 
wildcard FILTER' >&2
echo '--skip-sbomdo not generate the SBOM' >&2
echo '--skip-vex do not generate the VEX' >&2
exit 1
fi

dt_host_url=$1
dt_project=$2
spdx_tar=$3
dt_version=$4

api_req() (
path=${1:?API path}
shift

if out=$(curl --no-progress-meter --fail-with-body -H "X-Api-Key: 
$DT_API_KEY" \
"$@" "$dt_host_url/api/v1$path")
then
printf '%s' "$out"
else
rc=$?
echo "curl $dt_host_url/api/v1$path failed with exit code $rc: $out" >&2
return $rc
fi
)

api_req_prj() {
api_req "$@" -F "projectName=$dt_project" -F "projectVersion=$dt_version"
}

spdx_to_cydx_sbom() {
# 
https://github.com/CycloneDX/specification/blob/1.5/schema/bom-1.5.schema.json
# https://docs.dependencytrack.org/usage/cicd/

tar -x -f "$spdx_tar" --to-stdout --wildcards 
recipe-"$recipe_filter".spdx.json | \
jq --slurp 'map(.packages) |flatten
|map(select(.name |startswith("packagegroup-") |not)) | {
bomFormat: "CycloneDX",
specVersion: "1.5",
serialNumber: "urn:uuid:'"$(uuidgen)"'",
version: 1,

metadata: {
timestamp: "'"$(date --iso-8601=seconds)"'",
component: {
name: "'"$dt_project"'",
version: "'"$dt_version"'",
type: "operating-system",
},
},

components: map({
type: "application",
name,
"bom-ref": .name,
supplier: { name: .supplier },
version: .versionInfo,
description,
licenses: [{
expression: .licenseDeclared,
}],
externalReferences: ([
{ type: "vcs", url: .downloadLocation },
{ type: "website", url: .homepage }
] |map(select(.url != null and .url != "NOASSERTION"))),
} + (
.externalRefs[0].referenceLocator
| if . != null then { cpe: . } else { } end
)),
}'
}

spdx_to_cydx_vex() {
# https://github.com/CycloneDX/bom-examples/blob/master/VEX/vex.json

tar -x -f "$spdx_tar" --to-stdout --wildcards 
recipe-"$recipe_filter".spdx.json | \
jq --slurp 'map(.packages) |flatten
|map(select(.name |startswith("packagegroup-") |not))
|map(select(.sourceInfo != null)) | {
bomFormat: "CycloneDX",
specVersion: "1.5",
serialNumber: "urn:uuid:'"$(uuidgen)"'",
version: 1,
metadata: {
timestamp: "'"$(date --iso-8601=seconds)"'",
component: {
name: "'"$dt_project"'",
version: "'"$dt_version"'",
type: "operating-system",
"bom-ref": "project",
},
},

vulnerabilities: map(. as { name: $name, versionInfo: $version }
| .sourceInfo |match("CVE-\\S+"; "g") | .string
| {
id: .,

[oe] [meta-networking][PATCH] bluez-tools: New recipe for bluez5 tools

2024-03-18 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

---
 .../bluez-tools/fix-memory-leaks.patch| 768 ++
 .../obex-file-fix-null-check.patch|  41 +
 .../bluez-tools/bluez-tools_git.bb|  24 +
 3 files changed, 833 insertions(+)
 create mode 100644 
meta-networking/recipes-connectivity/bluez-tools/bluez-tools/fix-memory-leaks.patch
 create mode 100644 
meta-networking/recipes-connectivity/bluez-tools/bluez-tools/obex-file-fix-null-check.patch
 create mode 100644 
meta-networking/recipes-connectivity/bluez-tools/bluez-tools_git.bb

diff --git 
a/meta-networking/recipes-connectivity/bluez-tools/bluez-tools/fix-memory-leaks.patch
 
b/meta-networking/recipes-connectivity/bluez-tools/bluez-tools/fix-memory-leaks.patch
new file mode 100644
index 0..22e1fac85
--- /dev/null
+++ 
b/meta-networking/recipes-connectivity/bluez-tools/bluez-tools/fix-memory-leaks.patch
@@ -0,0 +1,768 @@
+Upstream-Status: Submitted [https://github.com/khvzak/bluez-tools/pull/48]
+
+From e5db2eec2591f0109f0eb7c2631055210b55f2f5 Mon Sep 17 00:00:00 2001
+Message-Id: 

+From: thatlittlegit 
+Date: Sat, 7 Nov 2020 01:07:24 -0500
+Subject: [PATCH 1/9] Remove memory leaks and overall restructure
+ manager_find_adapter
+
+---
+ src/lib/manager.c | 61 +--
+ 1 file changed, 43 insertions(+), 18 deletions(-)
+
+diff --git a/src/lib/manager.c b/src/lib/manager.c
+index 5286a3a..2263afc 100644
+--- a/src/lib/manager.c
 b/src/lib/manager.c
+@@ -136,43 +136,68 @@ const gchar *manager_find_adapter(Manager *self, const 
gchar *pattern, GError **
+ GVariant *ifaces_and_properties;
+ GVariantIter i;
+ 
++gchar *pattern_lowercase = g_ascii_strdown(pattern, -1);
++
+ g_variant_iter_init(, objects);
+-while (g_variant_iter_next(, "{@a{sa{sv}}}", _path, 
_and_properties))
++gboolean still_looking = TRUE;
++while (still_looking && g_variant_iter_next(, "{@a{sa{sv}}}", 
_path, _and_properties))
+ {
+ const gchar *interface_name;
+-GVariant *properties;
+ GVariantIter ii;
++GVariant* properties;
+ g_variant_iter_init(, ifaces_and_properties);
+ while (g_variant_iter_next(, "{@a{sv}}", _name, 
))
+ {
+-if (g_strstr_len(g_ascii_strdown(interface_name, -1), -1, 
"adapter"))
++gchar *interface_name_lowercase = g_ascii_strdown(interface_name, 
-1);
++if (strstr(interface_name_lowercase, "adapter"))
+ {
+-const gchar *object_base_name = 
g_path_get_basename(object_path);
+-if (g_strstr_len(g_ascii_strdown(object_base_name, -1), -1, 
g_ascii_strdown(pattern, -1)))
++g_free(interface_name_lowercase);
++
++gchar *object_base_name_original = 
g_path_get_basename(object_path);
++gchar *object_base_name = g_ascii_strdown(interface_name, -1);
++g_free(object_base_name_original);
++
++if (strstr(object_base_name, pattern_lowercase))
+ {
+-const gchar *retVal = g_strdup(object_path);
+-g_variant_unref(properties);
+-g_variant_unref(ifaces_and_properties);
+-g_variant_unref(objects);
+-return retVal;
++still_looking = FALSE;
++g_free(object_base_name);
++break;
+ }
+-const gchar *address = 
g_variant_get_string(g_variant_lookup_value(properties, "Address", NULL), NULL);
+-if (g_strstr_len(g_ascii_strdown(address, -1), -1, 
g_ascii_strdown(pattern, -1)))
++
++g_free(object_base_name);
++
++const gchar *address_original = 
g_variant_get_string(g_variant_lookup_value(properties, "Address", NULL), NULL);
++gchar *address = g_ascii_strdown(address_original, -1);
++
++if (strstr(address, pattern_lowercase))
+ {
+-gchar *retVal = g_strdup(object_path);
+-g_variant_unref(properties);
+-g_variant_unref(ifaces_and_properties);
+-g_variant_unref(objects);
+-return retVal;
++still_looking = FALSE;
++g_free(address);
++break;
+ }
++g_free(address);
+ }
++else
++{
++g_free(interface_name_lowercase);
++}
++
+ g_variant_unref(properties);
+ }
++
+ g_variant_unref(ifaces_and_properties);
+ }
+ g_variant_unref(objects);
++g_free(pattern_lowercase);
+ 
+-return NULL;
++if (still_looking)
++{
++return NULL;
++}
++else
++{
++return object_path;
++}
+ }
+ 
+ GPtrArray *manager_get_adapters(Manager *self)
+-- 
+2.34.1
+
+
+From

Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-16 Thread Jörg Sommer via lists . openembedded . org

I've found this bug report 
upstreamhttps://github.com/freeswitch/spandsp/issues/67. It builds fine for me 
with the older version df1282eb9af538ab1aadb6d66146e258451d4fe4.


Mit freundlichen Grüßen


Jörg Sommer

Software Developer / Programmierer

--


Navimatix GmbH

Tatzendpromenade 2

07745 Jena

T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de



Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480


From: Martin Jansa 
Sent: Saturday, 16 March 2024 09:02
To: Jörg Sommer 
Cc: Khem Raj ; openembedded-devel@lists.openembedded.org 

Subject: Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

I'm seeing the same errors with gcc :)

libtool: compile:  ccache arm-oe-linux-gnueabi-gcc -mthumb -mfpu=neon 
-mfloat-abi=softfp -mcpu=cortex-a9 -mtune=cortex-a9 -fstack-protector-strong 
-O2 -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security -Werror=format-security 
-Werror=return-type -funwind-tables -D_TIME_BITS=64 -D_FILE_OFFSET_BITS=64 
--sysroot=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/lib32-recipe-sysroot
 -DHAVE_CONFIG_H -I. -I../../git/src -I.. -DNDEBUG -Wunused-but-set-variable 
-std=gnu99 -ffast-math -Wall -Wunused-variable -Wwrite-strings 
-Wstrict-prototypes -Wmissing-prototypes -fvisibility=hidden 
-DHAVE_VISIBILITY=1 -O2 -pipe -g -feliminate-unused-debug-types 
-fcanon-prefix-map 
-fmacro-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/git=/usr/src/debug/lib32-spandsp/3.0.0+git
 
-fdebug-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/git=/usr/src/debug/lib32-spandsp/3.0.0+git
 
-fmacro-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/build=/usr/src/debug/lib32-spandsp/3.0.0+git
 
-fdebug-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/build=/usr/src/debug/lib32-spandsp/3.0.0+git
 
-fdebug-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/lib32-recipe-sysroot=
 
-fmacro-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/lib32-recipe-sysroot=
 
-fdebug-prefix-map=TOPDIR/BUILD/work/mach-oe-linux-gnueabi/lib32-spandsp/3.0.0+git/recipe-sysroot-native=
 -c ../../git/src/t38_terminal.c  -fPIC -DPIC -o .libs/t38_terminal.o
../../git/src/ademco_contactid.c:449:110: error: expected ')' before ';' token
  449 | static const int detection_threshold= 
goertzel_threshold_dbm0(GOERTZEL_SAMPLES_PER_BLOCK, -42.0f);
  | 
 ^
In file included from ../../git/src/ademco_contactid.c:66:
../../git/src/spandsp/tone_detect.h:63:55: note: to match this '('
   63 | #define goertzel_threshold_dbm0(len,thresh) (int) 
((len*len*256.0*256.0/2.0*pow(10.0, (thresh - DBM0_MAX_SINE_POWER)/10.0))
  |   ^
../../git/src/ademco_contactid.c:449:51: note: in expansion of macro 
'goertzel_threshold_dbm0'
  449 | static const int detection_threshold= 
goertzel_threshold_dbm0(GOERTZEL_SAMPLES_PER_BLOCK, -42.0f);
  |   
^~~
../../git/src/ademco_contactid.c:449:18: warning: 'detection_threshold' defined 
but not used [-Wunused-const-variable=]
  449 | static const int detection_threshold= 
goertzel_threshold_dbm0(GOERTZEL_SAMPLES_PER_BLOCK, -42.0f);
  |  ^~~
../../git/src/ademco_contactid.c:210:35: warning: 'ademco_codes' defined but 
not used [-Wunused-const-variable=]
  210 | static const struct ademco_code_s ademco_codes[] =
  |   ^~~~
make[2]: *** [Makefile:1091: ademco_contactid.lo] Error 1

This is with master-next meta-oe from yesterday in build with multilib (night 
not be important)

On Fri, Mar 15, 2024 at 8:19 AM Jörg Sommer via 
lists.openembedded.org 
mailto:navimatix...@lists.openembedded.org>>
 wrote:
Yes, I'm getting these errors, too, when compiling with clang; independent of 
the clang version up to 19. It seems to be a problem of `pow()`. gcc might have 
a built-in and can compute it at compile-time, while clang has no built-in of 
pow.




Jörg Sommer

Software Developer / Programmierer

--


Navimatix GmbH

Tatzendpromenade 2

07745 Jena

T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de



Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480


From: Khem Raj mailto:raj.k...@gmail.com>>
Sent: Friday, 15 March 2024 06:13
To: Jörg Sommer mailto:joerg.som...@navimatix.de>>
Cc:

[oe] [meta-multimedia][PATCH v3] spandsp: new telephony DSP library

2024-03-16 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Signed-off-by: Jörg Sommer 
---
 .../spandsp/spandsp/configure.patch   |  56 ++
 .../spandsp/spandsp/makefile.patch| 161 ++
 .../recipes-multimedia/spandsp/spandsp_git.bb |  50 ++
 3 files changed, 267 insertions(+)
 create mode 100644 
meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
 create mode 100644 
meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
 create mode 100644 meta-multimedia/recipes-multimedia/spandsp/spandsp_git.bb

diff --git a/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch 
b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
new file mode 100644
index 0..af504be76
--- /dev/null
+++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
@@ -0,0 +1,56 @@
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 14 Mar 2024 09:41:57 +0100
+Subject: [PATCH 2/2] configure: Replace manual search for libxml by
+ AC_CHECK_LIB
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The manual search fails if the include directories (and build environment)
+are located somewhere else.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
+---
+
+ERROR: spandsp-3.0.0+git-r0 do_configure: QA Issue: This autoconf log 
indicates errors, it looked at host include and/or library paths while 
determining system capabilities.
+
+ configure.ac | 22 +-
+ 1 file changed, 5 insertions(+), 17 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 2140837..b5138d6 100644
+--- a/configure.ac
 b/configure.ac
+@@ -185,23 +185,11 @@ then
+ fi
+ 
+ # Determine XML2 include path
+-AC_MSG_CHECKING(for libxml/xmlmemory.h)
+-
+-# Can we include headers using system include dirs?
+-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], [[int a 
= 1;]])],[XML2_INCLUDE=" "],[XML2_INCLUDE=])
+-
+-# Hunt through several possible directories to find the includes for libxml2
+-if test "x$XML2_INCLUDE" = "x"; then
+-old_CPPFLAGS="$CPPFLAGS"
+-for i in $xml2_include_dir /usr/include /usr/local/include 
/usr/include/libxml2 /usr/local/include/libxml2 ; do
+-CPPFLAGS="$old_CPPFLAGS -I$i"
+-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], 
[[int a = 1;]])],[XML2_INCLUDE="-I$i"],[XML2_INCLUDE=
+-])
+-if test "x$XML2_INCLUDE" != "x"; then
+-break;
+-fi
+-done
+-CPPFLAGS="$old_CPPFLAGS $XML2_INCLUDE"
++AC_CHECK_LIB([xml2], [xmlParseFile], [libxml2_found=yes])
++if test "$libxml2_found" = "yes" ; then
++AC_DEFINE(HAVE_LIBXML2,1,[defined when libxml2 is available])
++else
++AC_MSG_WARN([libxml2 not found. Disabling cache.])
+ fi
+ 
+ AC_CHECK_HEADERS([libxml/xmlmemory.h])
+-- 
+2.34.1
+
diff --git a/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch 
b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
new file mode 100644
index 0..d3eb047fe
--- /dev/null
+++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
@@ -0,0 +1,161 @@
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 14 Mar 2024 08:08:13 +0100
+Subject: [PATCH 1/2] configure: Use AX_PROG_CC_FOR_BUILD for cross compiling
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The macro AX_PROG_CC_FOR_BUILD [1] sets the variables CC_FOR_BUILD and
+..._FLAGS_FOR_BUILD they can be used [2] to build binaries used at compile
+time.
+
+[1]: https://www.gnu.org/software/autoconf-archive/ax_prog_cc_for_build.html
+[2]: 
https://stackoverflow.com/questions/24201260/autotools-cross-compilation-and-generated-sources/24208587#24208587
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
+---
+ configure.ac| 49 +---
+ src/Makefile.am | 54 +
+ 2 files changed, 29 insertions(+), 74 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 96fd022..2140837 100644
+--- a/configure.ac
 b/configure.ac
+@@ -55,54 +55,7 @@ AC_LANG([C])
+ 
+ AX_COMPILER_VENDOR
+ 
+-if test "${build}" != "${host}"
+-then
+-# If we are doing a Canadian Cross, in which the host and build systems
+-# are not the same, we set reasonable default values for the tools.
+-
+-CC_FOR_BUILD=${CC_FOR_BUILD-gcc}
+-CPPFLAGS_FOR_BUILD="\$(CPPFLAGS)"
+-CC=${CC-${host_alias}-gcc}
+-CFLAGS=${CFLAGS-"-g -O2"}
+-CXX=${CXX-${host_alias}-c++}
+-CXXFLAGS=${CXXFLAGS-"-g -O2"}
+-else
+-# Set reasonable default values for some tools even if not Canadian.
+-# Of course, these are different reasonable default values, originally
+-# specified directly in the Makefile.
+-# We don't export, so that autoconf can do its job.
+-# Note that all these settings are above the fragment inclusion point
+-# in

Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-15 Thread Jörg Sommer via lists . openembedded . org

Yes, I'm getting these errors, too, when compiling with clang; independent of 
the clang version up to 19. It seems to be a problem of `pow()`. gcc might have 
a built-in and can compute it at compile-time, while clang has no built-in of 
pow.




Jörg Sommer

Software Developer / Programmierer

--


Navimatix GmbH

Tatzendpromenade 2

07745 Jena

T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de



Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480


From: Khem Raj 
Sent: Friday, 15 March 2024 06:13
To: Jörg Sommer 
Cc: openembedded-devel@lists.openembedded.org 

Subject: Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

Not much to do with musl, it fails when I use clang on my desktop as
well. you can see the same errors with clang on host

CC=clang ./configure
make

On Thu, Mar 14, 2024 at 9:37 PM Khem Raj  wrote:
>
> fails to build with musl/clang
>
> https://snips.sh/f/qCR86Rwwz1
>
> On Thu, Mar 14, 2024 at 12:43 PM Jörg Sommer via
> lists.openembedded.org
>  wrote:
> >
> > From: Jörg Sommer 
> >
> > Signed-off-by: Jörg Sommer 
> > ---
> >  .../spandsp/spandsp/configure.patch   |  56 ++
> >  .../spandsp/spandsp/makefile.patch| 161 ++
> >  .../recipes-multimedia/spandsp/spandsp_git.bb |  47 +
> >  3 files changed, 264 insertions(+)
> >  create mode 100644 
> > meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
> >  create mode 100644 
> > meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
> >  create mode 100644 
> > meta-multimedia/recipes-multimedia/spandsp/spandsp_git.bb
> >
> > diff --git 
> > a/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch 
> > b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
> > new file mode 100644
> > index 0..af504be76
> > --- /dev/null
> > +++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
> > @@ -0,0 +1,56 @@
> > +From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
> > +Date: Thu, 14 Mar 2024 09:41:57 +0100
> > +Subject: [PATCH 2/2] configure: Replace manual search for libxml by
> > + AC_CHECK_LIB
> > +MIME-Version: 1.0
> > +Content-Type: text/plain; charset=UTF-8
> > +Content-Transfer-Encoding: 8bit
> > +
> > +The manual search fails if the include directories (and build environment)
> > +are located somewhere else.
> > +
> > +Signed-off-by: Jörg Sommer 
> > +Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
> > +---
> > +
> > +ERROR: spandsp-3.0.0+git-r0 do_configure: QA Issue: This autoconf log 
> > indicates errors, it looked at host include and/or library paths while 
> > determining system capabilities.
> > +
> > + configure.ac | 22 +-
> > + 1 file changed, 5 insertions(+), 17 deletions(-)
> > +
> > +diff --git a/configure.ac b/configure.ac
> > +index 2140837..b5138d6 100644
> > +--- a/configure.ac
> >  b/configure.ac
> > +@@ -185,23 +185,11 @@ then
> > + fi
> > +
> > + # Determine XML2 include path
> > +-AC_MSG_CHECKING(for libxml/xmlmemory.h)
> > +-
> > +-# Can we include headers using system include dirs?
> > +-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], 
> > [[int a = 1;]])],[XML2_INCLUDE=" "],[XML2_INCLUDE=])
> > +-
> > +-# Hunt through several possible directories to find the includes for 
> > libxml2
> > +-if test "x$XML2_INCLUDE" = "x"; then
> > +-old_CPPFLAGS="$CPPFLAGS"
> > +-for i in $xml2_include_dir /usr/include /usr/local/include 
> > /usr/include/libxml2 /usr/local/include/libxml2 ; do
> > +-CPPFLAGS="$old_CPPFLAGS -I$i"
> > +-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include 
> > ]], [[int a = 
> > 1;]])],[XML2_INCLUDE="-I$i"],[XML2_INCLUDE=
> > +-])
> > +-if test "x$XML2_INCLUDE" != "x"; then
> > +-break;
> > +-fi
> > +-done
> > +-CPPFLAGS="$old_CPPFLAGS $XML2_INCLUDE"
> > ++AC_CHECK_LIB([xml2], [xmlParseFile], [libxml2_found=yes])
> > ++if test "$libxml2_found" = "yes" ; then
> > ++AC_DEFINE(HAVE_LIBXML2,1,[defined when libxml2 is available])
> > ++else
> > ++AC_MSG_WARN([libxml2 not found. Disabling cache.])
> > + fi
> > +
> > + AC_CHECK_HEADERS([libxml/xmlmemory.h])
> > +--
> > +2.34.1
> > +
> > diff --git 
> > a/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch 
> > b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
> > new file mode 100644
> > index 0..d3eb047fe
> > --- /dev/null
> > +++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
> > @@ -0,0 +1,161 @@
> > +From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
> > +Date: Thu, 14 Mar 2024 08:08:13 +0100
> > +Subject: [PATCH 1/2] configure: Use AX_PROG_CC_FOR_BUILD for cross 
> > compiling
> > +MIME-Version: 1.0
> > +Content-Type: text/plain; charset=UTF-8
> > +Content-Transfer-Encoding: 8bit
> > +
> > +The macro AX_PROG_CC_FOR_BUILD [1]

[oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-14 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Signed-off-by: Jörg Sommer 
---
 .../spandsp/spandsp/configure.patch   |  56 ++
 .../spandsp/spandsp/makefile.patch| 161 ++
 .../recipes-multimedia/spandsp/spandsp_git.bb |  47 +
 3 files changed, 264 insertions(+)
 create mode 100644 
meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
 create mode 100644 
meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
 create mode 100644 meta-multimedia/recipes-multimedia/spandsp/spandsp_git.bb

diff --git a/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch 
b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
new file mode 100644
index 0..af504be76
--- /dev/null
+++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
@@ -0,0 +1,56 @@
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 14 Mar 2024 09:41:57 +0100
+Subject: [PATCH 2/2] configure: Replace manual search for libxml by
+ AC_CHECK_LIB
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The manual search fails if the include directories (and build environment)
+are located somewhere else.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
+---
+
+ERROR: spandsp-3.0.0+git-r0 do_configure: QA Issue: This autoconf log 
indicates errors, it looked at host include and/or library paths while 
determining system capabilities.
+
+ configure.ac | 22 +-
+ 1 file changed, 5 insertions(+), 17 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 2140837..b5138d6 100644
+--- a/configure.ac
 b/configure.ac
+@@ -185,23 +185,11 @@ then
+ fi
+ 
+ # Determine XML2 include path
+-AC_MSG_CHECKING(for libxml/xmlmemory.h)
+-
+-# Can we include headers using system include dirs?
+-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], [[int a 
= 1;]])],[XML2_INCLUDE=" "],[XML2_INCLUDE=])
+-
+-# Hunt through several possible directories to find the includes for libxml2
+-if test "x$XML2_INCLUDE" = "x"; then
+-old_CPPFLAGS="$CPPFLAGS"
+-for i in $xml2_include_dir /usr/include /usr/local/include 
/usr/include/libxml2 /usr/local/include/libxml2 ; do
+-CPPFLAGS="$old_CPPFLAGS -I$i"
+-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], 
[[int a = 1;]])],[XML2_INCLUDE="-I$i"],[XML2_INCLUDE=
+-])
+-if test "x$XML2_INCLUDE" != "x"; then
+-break;
+-fi
+-done
+-CPPFLAGS="$old_CPPFLAGS $XML2_INCLUDE"
++AC_CHECK_LIB([xml2], [xmlParseFile], [libxml2_found=yes])
++if test "$libxml2_found" = "yes" ; then
++AC_DEFINE(HAVE_LIBXML2,1,[defined when libxml2 is available])
++else
++AC_MSG_WARN([libxml2 not found. Disabling cache.])
+ fi
+ 
+ AC_CHECK_HEADERS([libxml/xmlmemory.h])
+-- 
+2.34.1
+
diff --git a/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch 
b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
new file mode 100644
index 0..d3eb047fe
--- /dev/null
+++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
@@ -0,0 +1,161 @@
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 14 Mar 2024 08:08:13 +0100
+Subject: [PATCH 1/2] configure: Use AX_PROG_CC_FOR_BUILD for cross compiling
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The macro AX_PROG_CC_FOR_BUILD [1] sets the variables CC_FOR_BUILD and
+..._FLAGS_FOR_BUILD they can be used [2] to build binaries used at compile
+time.
+
+[1]: https://www.gnu.org/software/autoconf-archive/ax_prog_cc_for_build.html
+[2]: 
https://stackoverflow.com/questions/24201260/autotools-cross-compilation-and-generated-sources/24208587#24208587
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
+---
+ configure.ac| 49 +---
+ src/Makefile.am | 54 +
+ 2 files changed, 29 insertions(+), 74 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 96fd022..2140837 100644
+--- a/configure.ac
 b/configure.ac
+@@ -55,54 +55,7 @@ AC_LANG([C])
+ 
+ AX_COMPILER_VENDOR
+ 
+-if test "${build}" != "${host}"
+-then
+-# If we are doing a Canadian Cross, in which the host and build systems
+-# are not the same, we set reasonable default values for the tools.
+-
+-CC_FOR_BUILD=${CC_FOR_BUILD-gcc}
+-CPPFLAGS_FOR_BUILD="\$(CPPFLAGS)"
+-CC=${CC-${host_alias}-gcc}
+-CFLAGS=${CFLAGS-"-g -O2"}
+-CXX=${CXX-${host_alias}-c++}
+-CXXFLAGS=${CXXFLAGS-"-g -O2"}
+-else
+-# Set reasonable default values for some tools even if not Canadian.
+-# Of course, these are different reasonable default values, originally
+-# specified directly in the Makefile.
+-# We don't export, so that autoconf can do its job.
+-# Note that all these settings are above the fragment inclusion point
+-# in

[OE-core] [PATCH] autotools: update link in comment for cross compiling

2024-03-14 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Signed-off-by: Jörg Sommer 
---
 meta/classes-recipe/autotools.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes-recipe/autotools.bbclass 
b/meta/classes-recipe/autotools.bbclass
index ca76cde0e7..9359c9b4e1 100644
--- a/meta/classes-recipe/autotools.bbclass
+++ b/meta/classes-recipe/autotools.bbclass
@@ -43,7 +43,8 @@ export lt_cv_sys_lib_dlsearch_path_spec = "${libdir} 
${base_libdir}"
 
 # When building tools for use at build-time it's recommended for the build
 # system to use these variables when cross-compiling.
-# (http://sources.redhat.com/autobook/autobook/autobook_270.html)
+# https://www.gnu.org/software/autoconf-archive/ax_prog_cc_for_build.html
+# 
https://stackoverflow.com/questions/24201260/autotools-cross-compilation-and-generated-sources/24208587#24208587
 export CPP_FOR_BUILD = "${BUILD_CPP}"
 export CPPFLAGS_FOR_BUILD = "${BUILD_CPPFLAGS}"
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#197100): 
https://lists.openembedded.org/g/openembedded-core/message/197100
Mute This Topic: https://lists.openembedded.org/mt/104923267/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] [PATCH] sngrep: new recipe for ncurses SIP Messages flow viewer

2024-03-14 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Signed-off-by: Jörg Sommer 
---
 .../recipes-support/sngrep/sngrep_1.8.0.bb| 37 +++
 1 file changed, 37 insertions(+)
 create mode 100644 meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb

diff --git a/meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb 
b/meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb
new file mode 100644
index 0..2824dde88
--- /dev/null
+++ b/meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb
@@ -0,0 +1,37 @@
+SUMMARY = "Ncurses SIP Messages flow viewer"
+DESCRIPTION = "Tool for displaying SIP calls message flows from terminal"
+HOMEPAGE = "https://github.com/irontec/sngrep;
+BUGTRACKER = "https://github.com/irontec/sngrep/issues;
+SECTION = "console/network"
+LICENSE = "GPL-3.0-or-later & OpenSSL"
+LIC_FILES_CHKSUM = "\
+file://LICENSE;md5=d32239bcb673463ab874e80d47fae504 \
+file://LICENSE.OpenSSL;md5=e39170c41c6f83de36426dbf49a03632 \
+
file://README;beginline=100;endline=124;md5=758a88cf2b27572df05996a3810066b3 \
+"
+
+DEPENDS = "\
+libpcap \
+ncurses \
+"
+
+SRC_URI = "git://github.com/irontec/sngrep.git;protocol=https;branch=master"
+SRCREV = "f7b36df3b79617892958b67cb4ad9313c6ce72d2"
+
+S = "${WORKDIR}/git"
+
+inherit cmake pkgconfig
+
+PACKAGECONFIG ?= "\
+openssl \
+${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+unicode \
+"
+
+PACKAGECONFIG[openssl] = "-DWITH_OPENSSL=ON,-DWITH_OPENSSL=OFF,openssl"
+PACKAGECONFIG[gnutls] = "-DWITH_GNUTLS=ON,-DWITH_GNUTLS=OFF,gnutls libgcrypt"
+PACKAGECONFIG[pcre] = "-DWITH_PCRE=OFF,-DWITH_PCRE=OFF,libpcre"
+PACKAGECONFIG[zlib] = "-DWITH_ZLIB=ON,-DWITH_ZLIB=OFF,zlib"
+PACKAGECONFIG[unicode] = "-DWITH_UNICODE=ON,-DWITH_UNICODE=OFF"
+PACKAGECONFIG[ipv6] = "-DUSE_IPV6=ON,-DUSE_IPV6=OFF"
+PACKAGECONFIG[eep] = "-DUSE_EEP=ON,-DUSE_EEP=OFF"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109313): 
https://lists.openembedded.org/g/openembedded-devel/message/109313
Mute This Topic: https://lists.openembedded.org/mt/104923036/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] [meta-multimedia][PATCH] spandsp: new telephony DSP library

2024-03-14 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Signed-off-by: Jörg Sommer 
---
 .../spandsp/spandsp/configure.patch   |  56 ++
 .../spandsp/spandsp/makefile.patch| 161 ++
 .../recipes-multimedia/spandsp/spandsp_git.bb |  47 +
 3 files changed, 264 insertions(+)
 create mode 100644 
meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
 create mode 100644 
meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
 create mode 100644 meta-multimedia/recipes-multimedia/spandsp/spandsp_git.bb

diff --git a/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch 
b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
new file mode 100644
index 0..af504be76
--- /dev/null
+++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
@@ -0,0 +1,56 @@
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 14 Mar 2024 09:41:57 +0100
+Subject: [PATCH 2/2] configure: Replace manual search for libxml by
+ AC_CHECK_LIB
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The manual search fails if the include directories (and build environment)
+are located somewhere else.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
+---
+
+ERROR: spandsp-3.0.0+git-r0 do_configure: QA Issue: This autoconf log 
indicates errors, it looked at host include and/or library paths while 
determining system capabilities.
+
+ configure.ac | 22 +-
+ 1 file changed, 5 insertions(+), 17 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 2140837..b5138d6 100644
+--- a/configure.ac
 b/configure.ac
+@@ -185,23 +185,11 @@ then
+ fi
+ 
+ # Determine XML2 include path
+-AC_MSG_CHECKING(for libxml/xmlmemory.h)
+-
+-# Can we include headers using system include dirs?
+-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], [[int a 
= 1;]])],[XML2_INCLUDE=" "],[XML2_INCLUDE=])
+-
+-# Hunt through several possible directories to find the includes for libxml2
+-if test "x$XML2_INCLUDE" = "x"; then
+-old_CPPFLAGS="$CPPFLAGS"
+-for i in $xml2_include_dir /usr/include /usr/local/include 
/usr/include/libxml2 /usr/local/include/libxml2 ; do
+-CPPFLAGS="$old_CPPFLAGS -I$i"
+-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], 
[[int a = 1;]])],[XML2_INCLUDE="-I$i"],[XML2_INCLUDE=
+-])
+-if test "x$XML2_INCLUDE" != "x"; then
+-break;
+-fi
+-done
+-CPPFLAGS="$old_CPPFLAGS $XML2_INCLUDE"
++AC_CHECK_LIB([xml2], [xmlParseFile], [libxml2_found=yes])
++if test "$libxml2_found" = "yes" ; then
++AC_DEFINE(HAVE_LIBXML2,1,[defined when libxml2 is available])
++else
++AC_MSG_WARN([libxml2 not found. Disabling cache.])
+ fi
+ 
+ AC_CHECK_HEADERS([libxml/xmlmemory.h])
+-- 
+2.34.1
+
diff --git a/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch 
b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
new file mode 100644
index 0..d3eb047fe
--- /dev/null
+++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
@@ -0,0 +1,161 @@
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 14 Mar 2024 08:08:13 +0100
+Subject: [PATCH 1/2] configure: Use AX_PROG_CC_FOR_BUILD for cross compiling
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The macro AX_PROG_CC_FOR_BUILD [1] sets the variables CC_FOR_BUILD and
+..._FLAGS_FOR_BUILD they can be used [2] to build binaries used at compile
+time.
+
+[1]: https://www.gnu.org/software/autoconf-archive/ax_prog_cc_for_build.html
+[2]: 
https://stackoverflow.com/questions/24201260/autotools-cross-compilation-and-generated-sources/24208587#24208587
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
+---
+ configure.ac| 49 +---
+ src/Makefile.am | 54 +
+ 2 files changed, 29 insertions(+), 74 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 96fd022..2140837 100644
+--- a/configure.ac
 b/configure.ac
+@@ -55,54 +55,7 @@ AC_LANG([C])
+ 
+ AX_COMPILER_VENDOR
+ 
+-if test "${build}" != "${host}"
+-then
+-# If we are doing a Canadian Cross, in which the host and build systems
+-# are not the same, we set reasonable default values for the tools.
+-
+-CC_FOR_BUILD=${CC_FOR_BUILD-gcc}
+-CPPFLAGS_FOR_BUILD="\$(CPPFLAGS)"
+-CC=${CC-${host_alias}-gcc}
+-CFLAGS=${CFLAGS-"-g -O2"}
+-CXX=${CXX-${host_alias}-c++}
+-CXXFLAGS=${CXXFLAGS-"-g -O2"}
+-else
+-# Set reasonable default values for some tools even if not Canadian.
+-# Of course, these are different reasonable default values, originally
+-# specified directly in the Makefile.
+-# We don't export, so that autoconf can do its job.
+-# Note that all these settings are above the fragment inclusion point
+-# in

[OE-core] [PATCH] sngrep: new recipe for ncurses SIP Messages flow viewer

2024-03-13 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Signed-off-by: Jörg Sommer 
---
 .../recipes-support/sngrep/sngrep_1.8.0.bb| 37 +++
 1 file changed, 37 insertions(+)
 create mode 100644 meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb

diff --git a/meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb 
b/meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb
new file mode 100644
index 0..2824dde88
--- /dev/null
+++ b/meta-networking/recipes-support/sngrep/sngrep_1.8.0.bb
@@ -0,0 +1,37 @@
+SUMMARY = "Ncurses SIP Messages flow viewer"
+DESCRIPTION = "Tool for displaying SIP calls message flows from terminal"
+HOMEPAGE = "https://github.com/irontec/sngrep;
+BUGTRACKER = "https://github.com/irontec/sngrep/issues;
+SECTION = "console/network"
+LICENSE = "GPL-3.0-or-later & OpenSSL"
+LIC_FILES_CHKSUM = "\
+file://LICENSE;md5=d32239bcb673463ab874e80d47fae504 \
+file://LICENSE.OpenSSL;md5=e39170c41c6f83de36426dbf49a03632 \
+
file://README;beginline=100;endline=124;md5=758a88cf2b27572df05996a3810066b3 \
+"
+
+DEPENDS = "\
+libpcap \
+ncurses \
+"
+
+SRC_URI = "git://github.com/irontec/sngrep.git;protocol=https;branch=master"
+SRCREV = "f7b36df3b79617892958b67cb4ad9313c6ce72d2"
+
+S = "${WORKDIR}/git"
+
+inherit cmake pkgconfig
+
+PACKAGECONFIG ?= "\
+openssl \
+${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+unicode \
+"
+
+PACKAGECONFIG[openssl] = "-DWITH_OPENSSL=ON,-DWITH_OPENSSL=OFF,openssl"
+PACKAGECONFIG[gnutls] = "-DWITH_GNUTLS=ON,-DWITH_GNUTLS=OFF,gnutls libgcrypt"
+PACKAGECONFIG[pcre] = "-DWITH_PCRE=OFF,-DWITH_PCRE=OFF,libpcre"
+PACKAGECONFIG[zlib] = "-DWITH_ZLIB=ON,-DWITH_ZLIB=OFF,zlib"
+PACKAGECONFIG[unicode] = "-DWITH_UNICODE=ON,-DWITH_UNICODE=OFF"
+PACKAGECONFIG[ipv6] = "-DUSE_IPV6=ON,-DUSE_IPV6=OFF"
+PACKAGECONFIG[eep] = "-DUSE_EEP=ON,-DUSE_EEP=OFF"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#197083): 
https://lists.openembedded.org/g/openembedded-core/message/197083
Mute This Topic: https://lists.openembedded.org/mt/104914270/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] [meta-oe][PATCH v5] bonnie++: New recipe for version 2.0

2024-01-28 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
 .../makefile-use-link-for-helper.patch|  24 +++
 .../bonnie/bonnie++_2.00a.bb  |  33 
 4 files changed, 279 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/makefile-use-link-for-helper.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 0..af20acdcd
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,39 @@
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
+
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 0..4b37b8d65
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,183 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@

Re: [OE-core] [PATCH] uboot-sign: support to load optee-os and TFA images

2024-01-16 Thread Jörg Sommer via lists . openembedded . org

On 15.01.24 08:54, Jamin Lin via lists.openembedded.org wrote:
> Currently, u-boot FIT image only support to load u-boot image.
> To support optee-os and trusted-firmware-a, update ITS file generation
> scripts, so users are able to use u-boot FIT image to load
> u-boot, optee-os and treustred-firmware-a images
> 
> Add a variable "UBOOT_FIT_ARM_TRUSTED_FIRMWARE_A" to
> enable trusted-firmware-a image and it is disable by default.
> 
> Add a variable "UBOOT_FIT_OPTEE_OS" to enable optee-os image
> and it is disable by default.
> 
> The ITS file creation loos like as following.


Is here a K missing?


Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#193859): 
https://lists.openembedded.org/g/openembedded-core/message/193859
Mute This Topic: https://lists.openembedded.org/mt/103734859/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] [meta-oe][PATCH] i2cdev: Fix MUSL build

2024-01-12 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Building the recipe with MUSL fails, because it uses error.h which isn't
supported by MUSL. Because the usage is only in one expression, it's easy to
rewrite this.

Signed-off-by: Jörg Sommer 
---
 .../recipes-bsp/i2cdev/i2cdev/fix-musl.patch  | 122 ++
 meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb  |   1 +
 2 files changed, 123 insertions(+)
 create mode 100644 meta-oe/recipes-bsp/i2cdev/i2cdev/fix-musl.patch

diff --git a/meta-oe/recipes-bsp/i2cdev/i2cdev/fix-musl.patch 
b/meta-oe/recipes-bsp/i2cdev/i2cdev/fix-musl.patch
new file mode 100644
index 0..59fd379de
--- /dev/null
+++ b/meta-oe/recipes-bsp/i2cdev/i2cdev/fix-musl.patch
@@ -0,0 +1,122 @@
+From ce3affeb45a65649dda1edc9a4f0586e9db47ada Mon Sep 17 00:00:00 2001
+Message-Id: 

+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Fri, 12 Jan 2024 13:59:51 +0100
+Subject: [PATCH] lsi2c: Replace error() by fprintf, drop error.h
+
+The MUSL C library doesn't support error.h. Because the only usage of this
+is the *error* function in lsi2c.c, this gets replaced by a *fprintf*. This
+doesn't print the program name, but keeps the message and the error
+description.
+
+Upstream-Status: Submitted [https://github.com/costad2/i2cdev/pull/5]
+---
+ libi2cdev/access.c | 1 -
+ libi2cdev/i2c-bus-parser.c | 1 -
+ libi2cdev/i2c-dev-path.c   | 1 -
+ libi2cdev/i2c-error.c  | 1 -
+ libi2cdev/init.c   | 2 --
+ libi2cdev/sysfs.c  | 1 -
+ lsi2c/lsi2c.c  | 4 ++--
+ 7 files changed, 2 insertions(+), 9 deletions(-)
+
+diff --git a/libi2cdev/access.c b/libi2cdev/access.c
+index 62a3f59..2e77659 100644
+--- a/libi2cdev/access.c
 b/libi2cdev/access.c
+@@ -16,7 +16,6 @@
+ 
+ #include "busses.h"
+ #include "data.h"
+-#include "error.h"
+ #include "sysfs.h"
+ 
+ #include "i2cdiscov.h"
+diff --git a/libi2cdev/i2c-bus-parser.c b/libi2cdev/i2c-bus-parser.c
+index c4b8688..8c760e0 100644
+--- a/libi2cdev/i2c-bus-parser.c
 b/libi2cdev/i2c-bus-parser.c
+@@ -27,7 +27,6 @@
+ #include 
+ #include 
+ #include 
+-#include 
+ #include 
+ #include 
+ 
+diff --git a/libi2cdev/i2c-dev-path.c b/libi2cdev/i2c-dev-path.c
+index b156db7..361761c 100644
+--- a/libi2cdev/i2c-dev-path.c
 b/libi2cdev/i2c-dev-path.c
+@@ -11,7 +11,6 @@
+ #include 
+ #include 
+ #include 
+-#include 
+ 
+ #include 
+ 
+diff --git a/libi2cdev/i2c-error.c b/libi2cdev/i2c-error.c
+index f92fb6b..540c112 100644
+--- a/libi2cdev/i2c-error.c
 b/libi2cdev/i2c-error.c
+@@ -9,7 +9,6 @@
+ 
+ #include 
+ #include 
+-#include 
+ #include 
+ #include 
+ #include 
+diff --git a/libi2cdev/init.c b/libi2cdev/init.c
+index 99a7edd..dfc4090 100644
+--- a/libi2cdev/init.c
 b/libi2cdev/init.c
+@@ -16,10 +16,8 @@
+ #include 
+ #include 
+ #include 
+-#include 
+ #include 
+ #include 
+-#include 
+ #include 
+ #include 
+ 
+diff --git a/libi2cdev/sysfs.c b/libi2cdev/sysfs.c
+index 2811500..a7e13a8 100644
+--- a/libi2cdev/sysfs.c
 b/libi2cdev/sysfs.c
+@@ -17,7 +17,6 @@
+ #include 
+ #include 
+ #include 
+-#include 
+ #include 
+ 
+ #include 
+diff --git a/lsi2c/lsi2c.c b/lsi2c/lsi2c.c
+index 7af5313..34c6225 100644
+--- a/lsi2c/lsi2c.c
 b/lsi2c/lsi2c.c
+@@ -38,7 +38,6 @@
+ #include 
+ #include 
+ #include 
+-#include 
+ #include 
+ #include 
+ 
+@@ -205,7 +204,8 @@ static int read_config_file(const char *config_file_name)
+ if (err < 0) {
+ err = -err;
+ }
+-error(0, err, "Failed to initialize i2cdevices");
++fflush(stdout);
++fprintf(stderr, "Failed to initialize i2cdevices: %s", strerror(err));
+ if (config_file) {
+ fclose(config_file);
+ }
+-- 
+2.34.1
+
diff --git a/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb 
b/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb
index b41ecffae..b2b5fda09 100644
--- a/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb
+++ b/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb
@@ -17,6 +17,7 @@ PR = "git${SRCPV}"
 SRC_URI = "\
 git://github.com/costad2/i2cdev.git;protocol=https;branch=master \
 file://fix-lsi2c-makefile.patch \
+file://fix-musl.patch \
 "
 SRCREV = "ed9ad777d842880e7ac6ca5e0de4bd2d3b4d02dc"
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#108242): 
https://lists.openembedded.org/g/openembedded-devel/message/108242
Mute This Topic: https://lists.openembedded.org/mt/103683188/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] [meta-oe][PATCH v4] bonnie++: New recipe for version 2.0

2024-01-09 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
 .../makefile-use-link-for-helper.patch|  24 +++
 .../bonnie/bonnie++_2.00a.bb  |  33 
 4 files changed, 279 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/makefile-use-link-for-helper.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 0..af20acdcd
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,39 @@
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
+
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 0..4b37b8d65
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,183 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@

[oe] [meta-oe][PATCH v3] bonnie++: New recipe for version 2.0

2024-01-08 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
 .../makefile-use-link-for-helper.patch|  24 +++
 .../bonnie/bonnie++_2.00a.bb  |  33 
 4 files changed, 279 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/makefile-use-link-for-helper.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 0..af20acdcd
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,39 @@
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
+
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 0..4b37b8d65
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,183 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@

[oe] [meta-oe][PATCH v3] bonnie++: New recipe for version 2.0

2023-12-30 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
 .../makefile-use-link-for-helper.patch|  24 +++
 .../bonnie/bonnie++_2.00a.bb  |  33 
 4 files changed, 279 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/makefile-use-link-for-helper.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 0..af20acdcd
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,39 @@
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
+
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 0..4b37b8d65
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,183 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@

Re: [oe] [meta-oe][PATCH v2] bonnie++: New recipe for version 2.0

2023-12-30 Thread Jörg Sommer via lists . openembedded . org

Are the build logs available anywhere? I would like to see do_configure and 
do_compile. I suspect that configure picks the wrong compiler.

Kind regards


Jörg Sommer

Software Developer / Programmierer

--


Navimatix GmbH

Tatzendpromenade 2

07745 Jena

T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de



Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480


From: Khem Raj 
Sent: Friday, 29 December 2023 17:58
To: Jörg Sommer 
Cc: openembedded-devel@lists.openembedded.org 

Subject: Re: [oe] [meta-oe][PATCH v2] bonnie++: New recipe for version 2.0

Here are some build issues seen while building for x86

https://errors.yoctoproject.org/Errors/Details/748405/

On Fri, Dec 29, 2023 at 2:19 AM Jörg Sommer via lists.openembedded.org
 wrote:
>
> From: Jörg Sommer 
>
> Newer versions of bonnie get published on
> . Unfortunately, the new version
> doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
> configure fails due to cross compilation which gets fixed
> with *fix-configure-lfs.patch*
>
> Signed-off-by: Jörg Sommer 
> ---
>  .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
>  .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
>  .../bonnie/bonnie++_2.00a.bb  |  26 +++
>  3 files changed, 248 insertions(+)
>  create mode 100644 
> meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
>  create mode 100644 
> meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
>  create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb
>
> diff --git 
> a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
> b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
> new file mode 100644
> index 0..af20acdcd
> --- /dev/null
> +++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
> @@ -0,0 +1,39 @@
> +Upstream-Status: Submitted 
> [https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
> +
> +diff --git i/configure.in w/configure.in
> +index 080e40c..f2a2bbe 100644
> +--- i/configure.in
>  w/configure.in
> +@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
> +   , thread_ldflags="-lpthread"
> +   , thread_ldflags="-pthread")
> +
> +-AC_SUBST(large_file)
> +-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
> ++AC_ARG_ENABLE(lfs,
> ++  [  --disable-lfs  disable large file support],
> ++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
> ++
> ++if test "$LFS_CHOICE" = yes; then
> ++   bonniepp_cv_large_file=yes
> ++elif test "$LFS_CHOICE" = check; then
> ++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
> bonniepp_cv_large_file,
> ++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
> + #define _LARGEFILE64_SOURCE
> + #endif
> + #include 
> +@@ -118,8 +125,12 @@ int main () {
> +   }
> +   close(fd);
> +   return 0;
> +-}], large_file="yes")
> +-if [[ -n "$large_file" ]]; then
> ++}], bonniepp_cv_large_file="yes"))
> ++fi
> ++
> ++AC_SUBST(large_file)
> ++
> ++if [[ -n "$bonniepp_cv_large_file" ]]; then
> +large_file="#define _LARGEFILE64_SOURCE"
> + fi
> +
> diff --git 
> a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
> b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
> new file mode 100644
> index 0..4b37b8d65
> --- /dev/null
> +++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
> @@ -0,0 +1,183 @@
> +commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
> +Author: Jörg Sommer 
> +Date:   Mon Jun 26 12:38:30 2023 +0200
> +
> +csv2html: Explicitly reference data in top level
> +
> +With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
> +needed to explicitly address the variable from the top level scope.
> +
> +[1] https://en.cppreference.com/w/cpp/iterator/data
> +
> +Upstream-Status: Submitted 
> [https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
> +
> +diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
> +index e9d9c50..652e330 100644
> +--- a/bon_csv2html.cpp
>  b/bon_csv2html.cpp
> +@@ -87,8 +87,8 @@ int main(int argc, char **argv)
> + read_in(buf);
> +   }
> +
> +-  props = new PPCCHAR[data.size()];
> +-  for(i = 0; i < data.size(); i++)
> ++  props = new PPCCHAR[::data.size()];
> ++  for(i = 0; i < ::data.size(); i++)
> +   {
> + props[i] = new PCCHAR[MAX_ITEMS];
> + props[i][0] = NULL;
> +@@ -109,7 +109,7 @@ int main(int argc, char **argv)
> +   }
> +   calc_vals();
> +   int mid_width = header();
> +-  for(i = 0; i < data.size(); i++)
> ++  for(i = 0; i < ::data.size(); i++)
> +   {
> + // First print the average speed line
> + printf("");
> +@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
> +
> + void

[oe] [meta-oe][PATCH v2] bonnie++: New recipe for version 2.0

2023-12-29 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
 .../bonnie/bonnie++_2.00a.bb  |  26 +++
 3 files changed, 248 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 0..af20acdcd
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,39 @@
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
+
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 0..4b37b8d65
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,183 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@ void calc_vals()
+ break;
+ case eCPU:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind

[oe] [meta-oe][PATCH] bonnie++: New recipe for version 2.0

2023-12-28 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  39 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 183 ++
 .../bonnie/bonnie++_2.00a.bb  |  33 
 3 files changed, 255 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 0..af20acdcd
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,39 @@
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=4ffece51791ba75ddca2e664cdce726cc40c92d3]
+
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 0..4b37b8d65
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,183 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+Upstream-Status: Submitted 
[https://salsa.debian.org/etbe/bonnie/-/merge_requests/3/diffs?commit_id=fb13a71d56dab8aaa39233fcaaedfb0ba4ad647d]
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@ void calc_vals()
+ break;
+ case eCPU:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int

[oe] [meta-oe][PATCH] i2cdev: New recipe with i2c tools

2023-12-27 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

The main part is the tool lsi2c to scan the i2c bus, like lspci and lsusb
do.

This recipe also ships a static library *libi2cdev.a*. Because this library
contains the file *smbus.c* which is *GPL v2+,* and a static library can't
be linked dynamically to satisfy the LGPL, the whole recipe is marked
as *GPL-2.0-or-later.*

Signed-off-by: Jörg Sommer 
---
 .../i2cdev/i2cdev/fix-lsi2c-makefile.patch| 28 +++
 meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb  | 26 +
 2 files changed, 54 insertions(+)
 create mode 100644 meta-oe/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch
 create mode 100644 meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb

diff --git a/meta-oe/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch 
b/meta-oe/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch
new file mode 100644
index 0..820d4bb44
--- /dev/null
+++ b/meta-oe/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch
@@ -0,0 +1,28 @@
+From 57c09727220e00ab961325a2c85f5611bd1770d6 Mon Sep 17 00:00:00 2001
+Message-Id: 
<57c09727220e00ab961325a2c85f5611bd1770d6.1701984646.git.joerg.som...@navimatix.de>
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 7 Dec 2023 22:29:40 +0100
+Subject: [PATCH] lsi2c/Makefile: Use builddir to refer to libi2cdev.a
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted 
[https://github.com/costad2/i2cdev/pull/2/commits/43f15e97d869797dbfeaacafa13216aaaf353426]
+---
+ lsi2c/Makefile.am | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/lsi2c/Makefile.am b/lsi2c/Makefile.am
+index 6defc1f..0d81d01 100644
+--- a/lsi2c/Makefile.am
 b/lsi2c/Makefile.am
+@@ -14,7 +14,7 @@ bin_PROGRAMS=lsi2c
+ lsi2c_SOURCES = lsi2c.c
+ 
+ # Linker options for lsi2c
+-lsi2c_LDADD = $(top_srcdir)/libi2cdev/libi2cdev.a
++lsi2c_LDADD = $(top_builddir)/libi2cdev/libi2cdev.a
+ 
+ # Compiler options for lsi2c
+ lsi2c_CFLAGS = -I$(top_srcdir)/include -std=c99 -fPIC
+-- 
+2.34.1
+
diff --git a/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb 
b/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb
new file mode 100644
index 0..b1f8a7f80
--- /dev/null
+++ b/meta-oe/recipes-bsp/i2cdev/i2cdev_git.bb
@@ -0,0 +1,26 @@
+SUMMARY = "i2c dev tools for Linux"
+DESCRIPTION = "\
+This package contains an I2C dev library and the i2c bus scanning \
+utility lsi2c. \
+"
+AUTHOR = "Danielle Costantino"
+HOMEPAGE = "https://github.com/costad2/i2cdev;
+LICENSE = "GPL-2.0-or-later"
+LIC_FILES_CHKSUM = "\
+file://COPYING;md5=768997ba510a952bef1775c50bc22b00 \
+
file://include/libi2cdev.h;beginline=12;endline=25;md5=72486a5e192d6ac5c7e55a4a95e380a6
 \
+
file://libi2cdev/smbus.c;beginline=9;endline=22;md5=d9a0de5a611b960fa75912ded6c60096
 \
+
file://lsi2c/lsi2c.c;beginline=11;endline=24;md5=72486a5e192d6ac5c7e55a4a95e380a6
 \
+"
+
+PR = "git${SRCPV}"
+
+SRC_URI = "\
+git://github.com/costad2/i2cdev.git;protocol=https;branch=master \
+file://fix-lsi2c-makefile.patch \
+"
+SRCREV = "ed9ad777d842880e7ac6ca5e0de4bd2d3b4d02dc"
+
+S = "${WORKDIR}/git"
+
+inherit autotools
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#107869): 
https://lists.openembedded.org/g/openembedded-devel/message/107869
Mute This Topic: https://lists.openembedded.org/mt/103387468/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] i2cdev: New recipe with i2c tools

2023-12-27 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

The main part is the tool lsi2c to scan the i2c bus, like lspci and lsusb
do.

This recipe also ships a static library *libi2cdev.a*. Because this library
contains the file *smbus.c* which is *GPL v2+,* and a static library can't
be linked dynamically to satisfy the LGPL, the whole recipe is marked
as *GPL-2.0-or-later.*

Signed-off-by: Jörg Sommer 
---
 .../i2cdev/i2cdev/fix-lsi2c-makefile.patch| 28 +++
 meta/recipes-bsp/i2cdev/i2cdev_git.bb | 26 +
 2 files changed, 54 insertions(+)
 create mode 100644 meta/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch
 create mode 100644 meta/recipes-bsp/i2cdev/i2cdev_git.bb

diff --git a/meta/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch 
b/meta/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch
new file mode 100644
index 00..4df7527205
--- /dev/null
+++ b/meta/recipes-bsp/i2cdev/i2cdev/fix-lsi2c-makefile.patch
@@ -0,0 +1,28 @@
+From 57c09727220e00ab961325a2c85f5611bd1770d6 Mon Sep 17 00:00:00 2001
+Message-Id: 
<57c09727220e00ab961325a2c85f5611bd1770d6.1701984646.git.joerg.som...@navimatix.de>
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 7 Dec 2023 22:29:40 +0100
+Subject: [PATCH] lsi2c/Makefile: Use builddir to refer to libi2cdev.a
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted 
https://github.com/costad2/i2cdev/pull/2/commits/43f15e97d869797dbfeaacafa13216aaaf353426
+---
+ lsi2c/Makefile.am | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/lsi2c/Makefile.am b/lsi2c/Makefile.am
+index 6defc1f..0d81d01 100644
+--- a/lsi2c/Makefile.am
 b/lsi2c/Makefile.am
+@@ -14,7 +14,7 @@ bin_PROGRAMS=lsi2c
+ lsi2c_SOURCES = lsi2c.c
+ 
+ # Linker options for lsi2c
+-lsi2c_LDADD = $(top_srcdir)/libi2cdev/libi2cdev.a
++lsi2c_LDADD = $(top_builddir)/libi2cdev/libi2cdev.a
+ 
+ # Compiler options for lsi2c
+ lsi2c_CFLAGS = -I$(top_srcdir)/include -std=c99 -fPIC
+-- 
+2.34.1
+
diff --git a/meta/recipes-bsp/i2cdev/i2cdev_git.bb 
b/meta/recipes-bsp/i2cdev/i2cdev_git.bb
new file mode 100644
index 00..b1f8a7f805
--- /dev/null
+++ b/meta/recipes-bsp/i2cdev/i2cdev_git.bb
@@ -0,0 +1,26 @@
+SUMMARY = "i2c dev tools for Linux"
+DESCRIPTION = "\
+This package contains an I2C dev library and the i2c bus scanning \
+utility lsi2c. \
+"
+AUTHOR = "Danielle Costantino"
+HOMEPAGE = "https://github.com/costad2/i2cdev;
+LICENSE = "GPL-2.0-or-later"
+LIC_FILES_CHKSUM = "\
+file://COPYING;md5=768997ba510a952bef1775c50bc22b00 \
+
file://include/libi2cdev.h;beginline=12;endline=25;md5=72486a5e192d6ac5c7e55a4a95e380a6
 \
+
file://libi2cdev/smbus.c;beginline=9;endline=22;md5=d9a0de5a611b960fa75912ded6c60096
 \
+
file://lsi2c/lsi2c.c;beginline=11;endline=24;md5=72486a5e192d6ac5c7e55a4a95e380a6
 \
+"
+
+PR = "git${SRCPV}"
+
+SRC_URI = "\
+git://github.com/costad2/i2cdev.git;protocol=https;branch=master \
+file://fix-lsi2c-makefile.patch \
+"
+SRCREV = "ed9ad777d842880e7ac6ca5e0de4bd2d3b4d02dc"
+
+S = "${WORKDIR}/git"
+
+inherit autotools
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#192939): 
https://lists.openembedded.org/g/openembedded-core/message/192939
Mute This Topic: https://lists.openembedded.org/mt/103383581/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] wic/DirectPlugin: don't update fstab if --no-fstab-update

2023-12-06 Thread Jörg Sommer via lists . openembedded . org

From: Jörg Sommer 

The function `update_fstab` should not touch the fstab for partitions with
`--no-fstab-update`.

Signed-off-by: Jörg Sommer 
---
 scripts/lib/wic/plugins/imager/direct.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/lib/wic/plugins/imager/direct.py 
b/scripts/lib/wic/plugins/imager/direct.py
index 9b619e41c1..013aa4255f 100644
--- a/scripts/lib/wic/plugins/imager/direct.py
+++ b/scripts/lib/wic/plugins/imager/direct.py
@@ -116,7 +116,7 @@ class DirectPlugin(ImagerPlugin):
 
 updated = False
 for part in self.parts:
-if not part.realnum or not part.mountpoint \
+if not part.realnum or not part.mountpoint or part.no_fstab_update 
\
or part.mountpoint == "/" or not 
(part.mountpoint.startswith('/') or part.mountpoint == "swap"):
 continue
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#191906): 
https://lists.openembedded.org/g/openembedded-core/message/191906
Mute This Topic: https://lists.openembedded.org/mt/103018986/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [oe] [PATCH] [meta classes] sed -i destroys symlinks

2023-11-08 Thread Jörg Sommer via lists . openembedded . org

On 07.11.23 21:24, Joakim Tjernlund via lists.openembedded.org wrote:
> In /etc/passwd is a symlink, sed -i on same file will replace the

I suspect the »In« should be an »If«

> symlink with a new file. Prevent that by expanding the file with realpath
> before giving it to sed.

Using `sed -i` could be used in many places; e.g. /etc/hosts. Maybe the
replacement with a symlink should happen after all other scripts are
done. Would this be possible?


Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#106471): 
https://lists.openembedded.org/g/openembedded-devel/message/106471
Mute This Topic: https://lists.openembedded.org/mt/102450721/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [oe] openssl-native.rm_work runs before kernel-module.install -> no signature

2023-11-03 Thread Jörg Sommer via lists . openembedded . org

On 30.10.23 14:24, Jörg Sommer via lists.openembedded.org wrote:
> On 30.10.23 13:29, Bruce Ashfield wrote:
>> On Mon, Oct 30, 2023 at 4:40 AM Jörg Sommer  
>> wrote:
>>> On 26.10.23 17:25, Bruce Ashfield wrote:
> 
 I ran into issues with some kernel versions, but we may have moved on
 far enough that it is no longer an issue.
>>>
>>> Do you remember what kind of issues these were? Possibly I can check if
>>> they still exist.
>>
>> It was kernels older than 5.10 failing to take the r-path changes and
>> hence failing when doing the signing step.
> 
> This could be solved by setting LD_LIBRARY_PATH. Would this be okay?
> 
> ```
> diff --git i/meta/classes/module.bbclass w/meta/classes/module.bbclass
> index a09ec3ed1e..38e1c95216 100644
> --- i/meta/classes/module.bbclass
> +++ w/meta/classes/module.bbclass
> @@ -51,6 +51,7 @@ module_do_install() {
>INSTALL_FW_PATH="${D}${nonarch_base_libdir}/firmware" \
>CC="${KERNEL_CC}" LD="${KERNEL_LD}" \
>O=${STAGING_KERNEL_BUILDDIR} \
> +  
> LD_LIBRARY_PATH="/build/tmp/sysroots-components/x86_64/openssl-native/usr/lib${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH}"
>  \
>${MODULES_INSTALL_TARGET}
>  
> if [ ! -e "${B}/${MODULES_MODULE_SYMVERS_LOCATION}/Module.symvers" ] 
> ; then
> ```

Hi Bruce,

I would like to here your opinion to this issue, because you've provided
the most to the discussion. I would like to submit a patch to fix
problem; either the BUILD_LDFLAGS one or the LD_LIBRARY_PATH.

Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105851): 
https://lists.openembedded.org/g/openembedded-devel/message/105851
Mute This Topic: https://lists.openembedded.org/mt/102174355/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [oe] openssl-native.rm_work runs before kernel-module.install -> no signature

2023-10-30 Thread Jörg Sommer via lists . openembedded . org

On 30.10.23 13:29, Bruce Ashfield wrote:
> On Mon, Oct 30, 2023 at 4:40 AM Jörg Sommer  wrote:
>> On 26.10.23 17:25, Bruce Ashfield wrote:

>>> I ran into issues with some kernel versions, but we may have moved on
>>> far enough that it is no longer an issue.
>>
>> Do you remember what kind of issues these were? Possibly I can check if
>> they still exist.
> 
> It was kernels older than 5.10 failing to take the r-path changes and
> hence failing when doing the signing step.

This could be solved by setting LD_LIBRARY_PATH. Would this be okay?

```
diff --git i/meta/classes/module.bbclass w/meta/classes/module.bbclass
index a09ec3ed1e..38e1c95216 100644
--- i/meta/classes/module.bbclass
+++ w/meta/classes/module.bbclass
@@ -51,6 +51,7 @@ module_do_install() {
   INSTALL_FW_PATH="${D}${nonarch_base_libdir}/firmware" \
   CC="${KERNEL_CC}" LD="${KERNEL_LD}" \
   O=${STAGING_KERNEL_BUILDDIR} \
+  
LD_LIBRARY_PATH="/build/tmp/sysroots-components/x86_64/openssl-native/usr/lib${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH}"
 \
   ${MODULES_INSTALL_TARGET}
 
if [ ! -e "${B}/${MODULES_MODULE_SYMVERS_LOCATION}/Module.symvers" ] ; 
then
```


Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena  
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105793): 
https://lists.openembedded.org/g/openembedded-devel/message/105793
Mute This Topic: https://lists.openembedded.org/mt/102174355/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [oe] openssl-native.rm_work runs before kernel-module.install -> no signature

2023-10-30 Thread Jörg Sommer via lists . openembedded . org

On 26.10.23 17:25, Bruce Ashfield wrote:
> On Thu, Oct 26, 2023 at 7:16 AM Jörg Sommer  wrote:
>>
>> On 26.10.23 10:34, Jose Quaresma wrote:
>>>
>>>
>>> Khem Raj mailto:raj.k...@gmail.com>> escreveu no
>>> dia quarta, 25/10/2023 à(s) 18:18:
>>>
>>> I wonder if something like below would help
>>>
>>> diff --git
>>> a/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
>>> 
>>> b/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
>>> 
>>> index 2cff48c39c3..6133721334c 100644
>>> --- a/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
>>> 
>>> +++ b/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
>>> 
>>> @@ -12,7 +12,7 @@ S = "${WORKDIR}"
>>>
>>>  do_configure[depends] += "virtual/kernel:do_shared_workdir
>>> openssl-native:do_populate_sysroot"
>>>  do_compile[depends] += "virtual/kernel:do_compile_kernelmodules"
>>> -
>>> +do_populate_sysroot[depends] += "openssl-native:do_populate_sysroot"
>>>
>>>
>>> This doesn't solve the problem because at compile time the openssl is
>>> already there on native sysroot
>>
>> How about a patch like this:
>>
> 
> I proposed something similar in the earlier threads on this topic (it
> has been around for quite some time, and is in bugzilla for oe-core)/

Can you give me a link to the bugzilla entry? I didn't found it.

> I ran into issues with some kernel versions, but we may have moved on
> far enough that it is no longer an issue.

Do you remember what kind of issues these were? Possibly I can check if
they still exist.

Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105790): 
https://lists.openembedded.org/g/openembedded-devel/message/105790
Mute This Topic: https://lists.openembedded.org/mt/102174355/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [oe] openssl-native.rm_work runs before kernel-module.install -> no signature

2023-10-26 Thread Jörg Sommer via lists . openembedded . org

On 26.10.23 10:34, Jose Quaresma wrote:
> 
> 
> Khem Raj mailto:raj.k...@gmail.com>> escreveu no
> dia quarta, 25/10/2023 à(s) 18:18:
> 
> I wonder if something like below would help
> 
> diff --git
> a/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
> 
> b/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
> 
> index 2cff48c39c3..6133721334c 100644
> --- a/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
> 
> +++ b/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
> 
> @@ -12,7 +12,7 @@ S = "${WORKDIR}"
> 
>  do_configure[depends] += "virtual/kernel:do_shared_workdir
> openssl-native:do_populate_sysroot"
>  do_compile[depends] += "virtual/kernel:do_compile_kernelmodules"
> -
> +do_populate_sysroot[depends] += "openssl-native:do_populate_sysroot"
> 
> 
> This doesn't solve the problem because at compile time the openssl is
> already there on native sysroot

How about a patch like this:

```
diff --git i/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb 
w/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
index 0e420a25d9..182abd0233 100644
--- i/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
+++ w/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
@@ -18,6 +18,10 @@ RDEPENDS:${PN}-dev = ""
 DEPENDS += "bc-native bison-native"
 DEPENDS += "gmp-native"
 
+BUILD_LDFLAGS =+ "-L 
/build/tmp/sysroots-components/x86_64/openssl-native/usr/lib \
+   
-Wl,-rpath-link,/build/tmp/sysroots-components/x86_64/openssl-native/usr/lib \
+   -Wl,-rpath,/build/tmp/sysroots-components/x86_64/openssl-native/usr/lib"
+
 EXTRA_OEMAKE = " HOSTCC="${BUILD_CC} ${BUILD_CFLAGS} ${BUILD_LDFLAGS}" 
HOSTCPP="${BUILD_CPP}""
 EXTRA_OEMAKE += " HOSTCXX="${BUILD_CXX} ${BUILD_CXXFLAGS} ${BUILD_LDFLAGS}" 
CROSS_COMPILE=${TARGET_PREFIX}"
 
```

Regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena  
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105741): 
https://lists.openembedded.org/g/openembedded-devel/message/105741
Mute This Topic: https://lists.openembedded.org/mt/102174355/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] [PATCH] openssh: Don't hardcode the dir in sshd.service

2023-10-26 Thread Jörg Sommer via lists . openembedded . org

On 26.10.23 09:04, Yu, Mingli via lists.openembedded.org wrote:
> From: Mingli Yu 
> 
> Don't hardcode the directory of the binary in sshd.service.
> 
> Signed-off-by: Mingli Yu 
> ---
>  meta/recipes-connectivity/openssh/openssh/sshd.service | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-connectivity/openssh/openssh/sshd.service 
> b/meta/recipes-connectivity/openssh/openssh/sshd.service
> index 6ace67d8ae..2a997b656a 100644
> --- a/meta/recipes-connectivity/openssh/openssh/sshd.service
> +++ b/meta/recipes-connectivity/openssh/openssh/sshd.service
> @@ -6,7 +6,7 @@ After=sshdgenkeys.service
>  [Service]
>  Environment="SSHD_OPTS="
>  EnvironmentFile=-/etc/default/ssh
> -ExecStartPre=/usr/bin/mkdir -p /var/run/sshd
> +ExecStartPre=@BASE_BINDIR@/mkdir -p /var/run/sshd

How about using `RuntimeDirectory=sshd` or don't hardcode any path and
call `ExecStartPre=mkdir -p …`

>  ExecStart=-@SBINDIR@/sshd -D $SSHD_OPTS
>  ExecReload=@BASE_BINDIR@/kill -HUP $MAINPID

Same here. This would match the example in the Manual page
systemd.service(5):

```
ExecReload=
Commands to execute to trigger a configuration reload in the
service. This argument takes multiple command lines, following the same
scheme as described
for ExecStart= above. Use of this setting is optional. Specifier and
environment variable substitution is supported here following the same
scheme as for
ExecStart=.

One additional, special environment variable is set: if known,
$MAINPID is set to the main process of the daemon, and may be used for
command lines like
the following:

ExecReload=kill -HUP $MAINPID
```

>  KillMode=process


Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189704): 
https://lists.openembedded.org/g/openembedded-core/message/189704
Mute This Topic: https://lists.openembedded.org/mt/102195446/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] openssl-native.rm_work runs before kernel-module.install -> no signature

2023-10-25 Thread Jörg Sommer via lists . openembedded . org

Hi,

I'm having an odd problem building the external kernel module
kernel-module-imx-gpu-viv [1]. I'm using kirkstone and have rm_work
enabled.

[1]  https://layers.openembedded.org/layerindex/recipe/40267/

The module builds fine, but the install step contains the module signing
which **silently** fails with *libcrypto.so.3* not found:

```
| make -C /build/tmp/work-shared/ziborium/kernel-source
M=/build/tmp/work/machine-pyxis-linux/kernel-module-imx-gpu-viv/6.4.3.p4.2-r0/git/src
modules_install
| make[1]: Entering directory '/build/tmp/work-shared/machine/kernel-source'
| make -C /build/tmp/work-shared/machine/kernel-build-artifacts -f
/build/tmp/work-shared/machine/kernel-source/Makefile modules_install
| make[2]: Entering directory
'/build/tmp/work-shared/machine/kernel-build-artifacts'
| test -e include/generated/autoconf.h -a -e include/config/auto.conf ||
(   \
| echo >&2; \
| echo >&2 "  ERROR: Kernel configuration is invalid."; \
| echo >&2 " include/generated/autoconf.h or
include/config/auto.conf are missing.";\
| echo >&2 " Run 'make oldconfig && make prepare' on kernel src
to fix it.";\
| echo >&2 ;\
| /bin/false)
| make -f
/build/tmp/work-shared/machine/kernel-source/scripts/Makefile.modinst
|   mkdir -p
/build/tmp/work/machine-dist-linux/kernel-module-imx-gpu-viv/6.4.3.p4.2-r0/image/usr/lib/modules/5.15.60-imx8mm+g343e81c0e39e/extra/;
cp
/build/tmp/work/machine-dist-linux/kernel-module-imx-gpu-viv/6.4.3.p4.2-r0/git/src/galcore.ko
/build/tmp/work/machine-dist-linux/kernel-module-imx-gpu-viv/6.4.3.p4.2-r0/image/usr/lib/modules/5.15.60-imx8mm+g343e81c0e39e/extra/galcore.ko
|   :
|   scripts/sign-file "sha1" "certs/signing_key.pem"
certs/signing_key.x509
/build/tmp/work/machine-dist-linux/kernel-module-imx-gpu-viv/6.4.3.p4.2-r0/image/usr/lib/modules/5.15.60-imx8mm+g343e81c0e39e/extra/galcore.ko
|| true
| scripts/sign-file: error while loading shared libraries:
libcrypto.so.3: cannot open shared object file: No such file or directory
|   sh /build/tmp/work-shared/machine/kernel-source/scripts/depmod.sh
echo 5.15.60-imx8mm+g343e81c0e39e
| Warning: modules_install: missing 'System.map' file. Skipping depmod.
| make[2]: Leaving directory
'/build/tmp/work-shared/machine/kernel-build-artifacts'
| make[1]: Leaving directory '/build/tmp/work-shared/machine/kernel-source'
```

I noticed that there are special dependency definitions in make-mod-scripts

```
do_configure[depends] += "virtual/kernel:do_shared_workdir
openssl-native:do_populate_sysroot"
do_compile[depends] += "virtual/kernel:do_compile_kernelmodules"
```

and module-base.bbclass (inherited by module.bbclass)

```
# We do the dependency this way because the output is not preserved
# in sstate, so we must force do_compile to run (once).
do_configure[depends] += "make-mod-scripts:do_compile"
```

Is there an additional dependency needed to address rm_work?

Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
D-07745 Jena
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105726): 
https://lists.openembedded.org/g/openembedded-devel/message/105726
Mute This Topic: https://lists.openembedded.org/mt/102174355/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] package_qa_check_rdepends: Allow /usr/bin/sh if usrmerge

2023-10-20 Thread Jörg Sommer via lists . openembedded . org

If the distro feature usrmerge is set, all files from /bin are moved to
/usr/bin, i.e. /usr/bin/sh is the same as /bin/sh and should be allowed be
ignored, because it's always present.

Signed-off-by: Jörg Sommer 
---
 meta/classes-global/insane.bbclass | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/classes-global/insane.bbclass 
b/meta/classes-global/insane.bbclass
index 6f3cd3026d..ecc868a4e0 100644
--- a/meta/classes-global/insane.bbclass
+++ b/meta/classes-global/insane.bbclass
@@ -923,8 +923,12 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, 
taskdeps, packages, d):
 
 if "file-rdeps" not in skip:
 ignored_file_rdeps = set(['/bin/sh', '/usr/bin/env', 
'rtld(GNU_HASH)'])
+if bb.utils.contains('DISTRO_FEATURES', 'usrmerge', True, False, 
d):
+ignored_file_rdeps |= set(['/usr/bin/sh'])
 if bb.data.inherits_class('nativesdk', d):
 ignored_file_rdeps |= set(['/bin/bash', '/usr/bin/perl', 
'perl'])
+if bb.utils.contains('DISTRO_FEATURES', 'usrmerge', True, 
False, d):
+ignored_file_rdeps |= set(['/usr/bin/bash'])
 # For Saving the FILERDEPENDS
 filerdepends = {}
 rdep_data = oe.packagedata.read_subpkgdata(pkg, d)
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189500): 
https://lists.openembedded.org/g/openembedded-core/message/189500
Mute This Topic: https://lists.openembedded.org/mt/102076957/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH v4] libtirpc: Support ipv6 in DISTRO_FEATURES

2023-10-16 Thread Jörg Sommer via lists . openembedded . org

If the ipv6 feature for the distribution is not set, the package should not
contain settings for ipv6. This makes rpcbind doesn't try to bind to a IPv6
socket, and complain that this fails.

Signed-off-by: Jörg Sommer 
---
 .../libtirpc/libtirpc/ipv6.patch  | 52 +++
 .../libtirpc/libtirpc_1.3.3.bb|  9 +++-
 2 files changed, 60 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-extended/libtirpc/libtirpc/ipv6.patch

diff --git a/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch 
b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
new file mode 100644
index 00..f746f986f4
--- /dev/null
+++ b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
@@ -0,0 +1,52 @@
+From 077bbd32e8b7474dc5f153997732e1e6aec7fad6 Mon Sep 17 00:00:00 2001
+Message-Id: 
<077bbd32e8b7474dc5f153997732e1e6aec7fad6.1697120796.git.joerg.som...@navimatix.de>
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 12 Oct 2023 16:22:59 +0200
+Subject: [PATCH] netconfig: remove tcp6, udp6 on --disable-ipv6
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+If the configuration for IPv6 is disabled, the netconfig should not contain
+settings for tcp6 and udp6.
+
+The test for the configure option didn't work, because it check the wrong
+variable.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [libtirpc-de...@lists.sourceforge.net]
+Upstream-Status: Submitted [linux-...@vger.kernel.org]
+---
+ configure.ac| 2 +-
+ doc/Makefile.am | 5 +
+ 2 files changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index fe6c517..b687f8d 100644
+--- a/configure.ac
 b/configure.ac
+@@ -64,7 +64,7 @@ fi
+ AC_ARG_ENABLE(ipv6,
+   [AC_HELP_STRING([--disable-ipv6], [Disable IPv6 support 
@<:@default=no@:>@])],
+   [],[enable_ipv6=yes])
+-AM_CONDITIONAL(INET6, test "x$disable_ipv6" != xno)
++AM_CONDITIONAL(INET6, test "x$enable_ipv6" != xno)
+ if test "x$enable_ipv6" != xno; then
+   AC_DEFINE(INET6, 1, [Define to 1 if IPv6 is available])
+ fi
+diff --git a/doc/Makefile.am b/doc/Makefile.am
+index d42ab90..b9678f6 100644
+--- a/doc/Makefile.am
 b/doc/Makefile.am
+@@ -2,3 +2,8 @@ dist_sysconf_DATA  = netconfig bindresvport.blacklist
+ 
+ CLEANFILES   = cscope.* *~
+ DISTCLEANFILES   = Makefile.in
++
++if ! INET6
++install-exec-hook:
++  $(SED) -i '/^tcp6\|^udp6/d' "$(DESTDIR)$(sysconfdir)"/netconfig
++endif
+-- 
+2.34.1
+
diff --git a/meta/recipes-extended/libtirpc/libtirpc_1.3.3.bb 
b/meta/recipes-extended/libtirpc/libtirpc_1.3.3.bb
index b27c302460..898a952a8b 100644
--- a/meta/recipes-extended/libtirpc/libtirpc_1.3.3.bb
+++ b/meta/recipes-extended/libtirpc/libtirpc_1.3.3.bb
@@ -9,7 +9,9 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=f835cce8852481e4b2bbbdd23b5e47f3 \
 
 PROVIDES = "virtual/librpc"
 
-SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.bz2"
+SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.bz2 \
+   file://ipv6.patch \
+"
 UPSTREAM_CHECK_URI = 
"https://sourceforge.net/projects/libtirpc/files/libtirpc/;
 UPSTREAM_CHECK_REGEX = "(?P\d+(\.\d+)+)/"
 SRC_URI[sha256sum] = 
"6474e98851d9f6f33871957ddee9714fdcd9d8a5ee9abb5a98d63ea2e60e12f3"
@@ -21,6 +23,11 @@ inherit autotools pkgconfig
 PACKAGECONFIG ??= ""
 PACKAGECONFIG[gssapi] = "--enable-gssapi,--disable-gssapi,krb5"
 
+PACKAGECONFIG ??= "\
+   ${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+"
+PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6"
+
 do_install:append() {
test -e ${D}${sysconfdir}/netconfig && chown root:root 
${D}${sysconfdir}/netconfig
 }
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189283): 
https://lists.openembedded.org/g/openembedded-core/message/189283
Mute This Topic: https://lists.openembedded.org/mt/101992881/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] base-files: Remove localhost ::1 from hosts if ipv6 missing

2023-10-16 Thread Jörg Sommer via lists . openembedded . org

If a distribution doesn't provide IPv6, the mapping of localhost and ::1 has
to be removed.

Signed-off-by: Jörg Sommer 
---
 meta/recipes-core/base-files/base-files_3.0.14.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb 
b/meta/recipes-core/base-files/base-files_3.0.14.bb
index 6ba3971e32..4d246126a2 100644
--- a/meta/recipes-core/base-files/base-files_3.0.14.bb
+++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
@@ -136,6 +136,10 @@ do_install () {
echo ${hostname} > ${D}${sysconfdir}/hostname
echo "127.0.1.1 ${hostname}" >> ${D}${sysconfdir}/hosts
fi
+
+   if ${@bb.utils.contains('DISTRO_FEATURES', 'ipv6', 'false', 'true', 
d)}; then
+   sed -i '/^::1/s/ localhost//' ${D}${sysconfdir}/hosts
+   fi
 }
 
 do_install:append:libc-glibc () {
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189282): 
https://lists.openembedded.org/g/openembedded-core/message/189282
Mute This Topic: https://lists.openembedded.org/mt/101992845/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] [PATCH v3] libtirpc: Support ipv6 in DISTRO_FEATURES

2023-10-13 Thread Jörg Sommer via lists . openembedded . org

On 12 October 2023 21:13, Dan McGregor wrote:
> On Thu, 12 Oct 2023 at 11:10, Jörg Sommer via lists.openembedded.org
>  wrote:
> >
> > This is only a minor change, because oelint-adv had warned about the space 
> > after " of PACKAGECONFIG.
> >
> > 
> > From: openembedded-core@lists.openembedded.org 
> >  on behalf of Jörg Sommer via 
> > lists.openembedded.org 
> > Sent: Thursday, 12 October 2023 18:34
> > To: openembedded-core@lists.openembedded.org 
> > 
> > Cc: Jörg Sommer 
> > Subject: [OE-core] [PATCH v3] libtirpc: Support ipv6 in DISTRO_FEATURES
> >
> > If the ipv6 feature for the distribution is not set, the package should not
> > contain settings for ipv6. This makes rpcbind doesn't try to bind to a IPv6
> > socket, and complain that this fails.

> > +PACKAGECONFIG ??= "\
> > +    ${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
> > +"
> > +PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6"
> 
> Looks like this will conflict with recent changes I made to master, so
> hopefully those can be resolved automatically.

I'm happy to provide a new version that applies on master, if it's needed. Let 
me know if I should create it.

-- 
Navimatix GmbH
Tatzendpromenade 2
07745 Jena  
Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189057): 
https://lists.openembedded.org/g/openembedded-core/message/189057
Mute This Topic: https://lists.openembedded.org/mt/101921995/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] [PATCH v3] libtirpc: Support ipv6 in DISTRO_FEATURES

2023-10-12 Thread Jörg Sommer via lists . openembedded . org

This is only a minor change, because oelint-adv had warned about the space 
after " of PACKAGECONFIG.


From: openembedded-core@lists.openembedded.org 
 on behalf of Jörg Sommer via 
lists.openembedded.org 
Sent: Thursday, 12 October 2023 18:34
To: openembedded-core@lists.openembedded.org 

Cc: Jörg Sommer 
Subject: [OE-core] [PATCH v3] libtirpc: Support ipv6 in DISTRO_FEATURES

If the ipv6 feature for the distribution is not set, the package should not
contain settings for ipv6. This makes rpcbind doesn't try to bind to a IPv6
socket, and complain that this fails.

Signed-off-by: Jörg Sommer 
---
 .../libtirpc/libtirpc/ipv6.patch  | 52 +++
 .../libtirpc/libtirpc_1.3.2.bb|  6 +++
 2 files changed, 58 insertions(+)
 create mode 100644 meta/recipes-extended/libtirpc/libtirpc/ipv6.patch

diff --git a/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch 
b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
new file mode 100644
index 00..f746f986f4
--- /dev/null
+++ b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
@@ -0,0 +1,52 @@
+From 077bbd32e8b7474dc5f153997732e1e6aec7fad6 Mon Sep 17 00:00:00 2001
+Message-Id: 
<077bbd32e8b7474dc5f153997732e1e6aec7fad6.1697120796.git.joerg.som...@navimatix.de>
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 12 Oct 2023 16:22:59 +0200
+Subject: [PATCH] netconfig: remove tcp6, udp6 on --disable-ipv6
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+If the configuration for IPv6 is disabled, the netconfig should not contain
+settings for tcp6 and udp6.
+
+The test for the configure option didn't work, because it check the wrong
+variable.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [libtirpc-de...@lists.sourceforge.net]
+Upstream-Status: Submitted [linux-...@vger.kernel.org]
+---
+ configure.ac| 2 +-
+ doc/Makefile.am | 5 +
+ 2 files changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index fe6c517..b687f8d 100644
+--- a/configure.ac
 b/configure.ac
+@@ -64,7 +64,7 @@ fi
+ AC_ARG_ENABLE(ipv6,
+[AC_HELP_STRING([--disable-ipv6], [Disable IPv6 support 
@<:@default=no@:>@])],
+[],[enable_ipv6=yes])
+-AM_CONDITIONAL(INET6, test "x$disable_ipv6" != xno)
++AM_CONDITIONAL(INET6, test "x$enable_ipv6" != xno)
+ if test "x$enable_ipv6" != xno; then
+AC_DEFINE(INET6, 1, [Define to 1 if IPv6 is available])
+ fi
+diff --git a/doc/Makefile.am b/doc/Makefile.am
+index d42ab90..b9678f6 100644
+--- a/doc/Makefile.am
 b/doc/Makefile.am
+@@ -2,3 +2,8 @@ dist_sysconf_DATA  = netconfig bindresvport.blacklist
+
+ CLEANFILES   = cscope.* *~
+ DISTCLEANFILES   = Makefile.in
++
++if ! INET6
++install-exec-hook:
++  $(SED) -i '/^tcp6\|^udp6/d' "$(DESTDIR)$(sysconfdir)"/netconfig
++endif
+--
+2.34.1
+
diff --git a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb 
b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
index 6980135a92..edb98082f2 100644
--- a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
+++ b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
@@ -11,6 +11,7 @@ PROVIDES = "virtual/librpc"

 SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.bz2 \
file://CVE-2021-46828.patch \
+  file://ipv6.patch \
   "
 UPSTREAM_CHECK_URI = 
"https://sourceforge.net/projects/libtirpc/files/libtirpc/;
 UPSTREAM_CHECK_REGEX = "(?P\d+(\.\d+)+)/"
@@ -20,6 +21,11 @@ inherit autotools pkgconfig

 EXTRA_OECONF = "--disable-gssapi"

+PACKAGECONFIG ??= "\
+${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+"
+PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6"
+
 do_install:append() {
 test -e ${D}${sysconfdir}/netconfig && chown root:root 
${D}${sysconfdir}/netconfig
 }
--
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189009): 
https://lists.openembedded.org/g/openembedded-core/message/189009
Mute This Topic: https://lists.openembedded.org/mt/101921995/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH v3] libtirpc: Support ipv6 in DISTRO_FEATURES

2023-10-12 Thread Jörg Sommer via lists . openembedded . org

If the ipv6 feature for the distribution is not set, the package should not
contain settings for ipv6. This makes rpcbind doesn't try to bind to a IPv6
socket, and complain that this fails.

Signed-off-by: Jörg Sommer 
---
 .../libtirpc/libtirpc/ipv6.patch  | 52 +++
 .../libtirpc/libtirpc_1.3.2.bb|  6 +++
 2 files changed, 58 insertions(+)
 create mode 100644 meta/recipes-extended/libtirpc/libtirpc/ipv6.patch

diff --git a/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch 
b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
new file mode 100644
index 00..f746f986f4
--- /dev/null
+++ b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
@@ -0,0 +1,52 @@
+From 077bbd32e8b7474dc5f153997732e1e6aec7fad6 Mon Sep 17 00:00:00 2001
+Message-Id: 
<077bbd32e8b7474dc5f153997732e1e6aec7fad6.1697120796.git.joerg.som...@navimatix.de>
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 12 Oct 2023 16:22:59 +0200
+Subject: [PATCH] netconfig: remove tcp6, udp6 on --disable-ipv6
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+If the configuration for IPv6 is disabled, the netconfig should not contain
+settings for tcp6 and udp6.
+
+The test for the configure option didn't work, because it check the wrong
+variable.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [libtirpc-de...@lists.sourceforge.net]
+Upstream-Status: Submitted [linux-...@vger.kernel.org]
+---
+ configure.ac| 2 +-
+ doc/Makefile.am | 5 +
+ 2 files changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index fe6c517..b687f8d 100644
+--- a/configure.ac
 b/configure.ac
+@@ -64,7 +64,7 @@ fi
+ AC_ARG_ENABLE(ipv6,
+   [AC_HELP_STRING([--disable-ipv6], [Disable IPv6 support 
@<:@default=no@:>@])],
+   [],[enable_ipv6=yes])
+-AM_CONDITIONAL(INET6, test "x$disable_ipv6" != xno)
++AM_CONDITIONAL(INET6, test "x$enable_ipv6" != xno)
+ if test "x$enable_ipv6" != xno; then
+   AC_DEFINE(INET6, 1, [Define to 1 if IPv6 is available])
+ fi
+diff --git a/doc/Makefile.am b/doc/Makefile.am
+index d42ab90..b9678f6 100644
+--- a/doc/Makefile.am
 b/doc/Makefile.am
+@@ -2,3 +2,8 @@ dist_sysconf_DATA  = netconfig bindresvport.blacklist
+ 
+ CLEANFILES   = cscope.* *~
+ DISTCLEANFILES   = Makefile.in
++
++if ! INET6
++install-exec-hook:
++  $(SED) -i '/^tcp6\|^udp6/d' "$(DESTDIR)$(sysconfdir)"/netconfig
++endif
+-- 
+2.34.1
+
diff --git a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb 
b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
index 6980135a92..edb98082f2 100644
--- a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
+++ b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
@@ -11,6 +11,7 @@ PROVIDES = "virtual/librpc"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.bz2 \
   file://CVE-2021-46828.patch \
+  file://ipv6.patch \
  "
 UPSTREAM_CHECK_URI = 
"https://sourceforge.net/projects/libtirpc/files/libtirpc/;
 UPSTREAM_CHECK_REGEX = "(?P\d+(\.\d+)+)/"
@@ -20,6 +21,11 @@ inherit autotools pkgconfig
 
 EXTRA_OECONF = "--disable-gssapi"
 
+PACKAGECONFIG ??= "\
+${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+"
+PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6"
+
 do_install:append() {
test -e ${D}${sysconfdir}/netconfig && chown root:root 
${D}${sysconfdir}/netconfig
 }
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189008): 
https://lists.openembedded.org/g/openembedded-core/message/189008
Mute This Topic: https://lists.openembedded.org/mt/101921995/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH v2] libtirpc: Support ipv6 in DISTRO_FEATURES

2023-10-12 Thread Jörg Sommer via lists . openembedded . org

If the ipv6 feature for the distribution is not set, the package should not
contain settings for ipv6. This makes rpcbind doesn't try to bind to a IPv6
socket, and complain that this fails.

Signed-off-by: Jörg Sommer 
---
 .../libtirpc/libtirpc/ipv6.patch  | 52 +++
 .../libtirpc/libtirpc_1.3.2.bb|  6 +++
 2 files changed, 58 insertions(+)
 create mode 100644 meta/recipes-extended/libtirpc/libtirpc/ipv6.patch

diff --git a/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch 
b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
new file mode 100644
index 00..f746f986f4
--- /dev/null
+++ b/meta/recipes-extended/libtirpc/libtirpc/ipv6.patch
@@ -0,0 +1,52 @@
+From 077bbd32e8b7474dc5f153997732e1e6aec7fad6 Mon Sep 17 00:00:00 2001
+Message-Id: 
<077bbd32e8b7474dc5f153997732e1e6aec7fad6.1697120796.git.joerg.som...@navimatix.de>
+From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
+Date: Thu, 12 Oct 2023 16:22:59 +0200
+Subject: [PATCH] netconfig: remove tcp6, udp6 on --disable-ipv6
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+If the configuration for IPv6 is disabled, the netconfig should not contain
+settings for tcp6 and udp6.
+
+The test for the configure option didn't work, because it check the wrong
+variable.
+
+Signed-off-by: Jörg Sommer 
+Upstream-Status: Submitted [libtirpc-de...@lists.sourceforge.net]
+Upstream-Status: Submitted [linux-...@vger.kernel.org]
+---
+ configure.ac| 2 +-
+ doc/Makefile.am | 5 +
+ 2 files changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index fe6c517..b687f8d 100644
+--- a/configure.ac
 b/configure.ac
+@@ -64,7 +64,7 @@ fi
+ AC_ARG_ENABLE(ipv6,
+   [AC_HELP_STRING([--disable-ipv6], [Disable IPv6 support 
@<:@default=no@:>@])],
+   [],[enable_ipv6=yes])
+-AM_CONDITIONAL(INET6, test "x$disable_ipv6" != xno)
++AM_CONDITIONAL(INET6, test "x$enable_ipv6" != xno)
+ if test "x$enable_ipv6" != xno; then
+   AC_DEFINE(INET6, 1, [Define to 1 if IPv6 is available])
+ fi
+diff --git a/doc/Makefile.am b/doc/Makefile.am
+index d42ab90..b9678f6 100644
+--- a/doc/Makefile.am
 b/doc/Makefile.am
+@@ -2,3 +2,8 @@ dist_sysconf_DATA  = netconfig bindresvport.blacklist
+ 
+ CLEANFILES   = cscope.* *~
+ DISTCLEANFILES   = Makefile.in
++
++if ! INET6
++install-exec-hook:
++  $(SED) -i '/^tcp6\|^udp6/d' "$(DESTDIR)$(sysconfdir)"/netconfig
++endif
+-- 
+2.34.1
+
diff --git a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb 
b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
index 6980135a92..dca5a964a8 100644
--- a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
+++ b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
@@ -11,6 +11,7 @@ PROVIDES = "virtual/librpc"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.bz2 \
   file://CVE-2021-46828.patch \
+  file://ipv6.patch \
  "
 UPSTREAM_CHECK_URI = 
"https://sourceforge.net/projects/libtirpc/files/libtirpc/;
 UPSTREAM_CHECK_REGEX = "(?P\d+(\.\d+)+)/"
@@ -20,6 +21,11 @@ inherit autotools pkgconfig
 
 EXTRA_OECONF = "--disable-gssapi"
 
+PACKAGECONFIG ??= " \
+${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+"
+PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6"
+
 do_install:append() {
test -e ${D}${sysconfdir}/netconfig && chown root:root 
${D}${sysconfdir}/netconfig
 }
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189004): 
https://lists.openembedded.org/g/openembedded-core/message/189004
Mute This Topic: https://lists.openembedded.org/mt/101919607/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] libtirpc: Support ipv6 in DISTRO_FEATURES

2023-10-12 Thread Jörg Sommer via lists . openembedded . org

If the ipv6 feature for the distribution is not set, the package should not
contain settings for ipv6. This makes rpcbind doesn't try to bind to a IPv6
socket, and complain that this fails.

Signed-off-by: Jörg Sommer 
---
 meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb | 14 +-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb 
b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
index 6980135a92..14db4a5eda 100644
--- a/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
+++ b/meta/recipes-extended/libtirpc/libtirpc_1.3.2.bb
@@ -20,8 +20,20 @@ inherit autotools pkgconfig
 
 EXTRA_OECONF = "--disable-gssapi"
 
+PACKAGECONFIG ??= " \
+${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
+"
+PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6"
+
 do_install:append() {
-   test -e ${D}${sysconfdir}/netconfig && chown root:root 
${D}${sysconfdir}/netconfig
+   if test -e ${D}${sysconfdir}/netconfig
+   then
+   if ${@bb.utils.contains('DISTRO_FEATURES', 'ipv6', 'false', 
'true', d)}
+   then
+   sed -i '/^tcp6\|^udp6/d' ${D}${sysconfdir}/netconfig
+   fi
+   chown root:root ${D}${sysconfdir}/netconfig
+   fi
 }
 
 BBCLASSEXTEND = "native nativesdk"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188993): 
https://lists.openembedded.org/g/openembedded-core/message/188993
Mute This Topic: https://lists.openembedded.org/mt/101916628/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[oe] [meta-oe][PATCH] collectd: Use https in SRC_URI, add HOMEPAGE

2023-10-05 Thread Jörg Sommer via lists . openembedded . org

Signed-off-by: Jörg Sommer 
---
 meta-oe/recipes-extended/collectd/collectd_5.12.0.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta-oe/recipes-extended/collectd/collectd_5.12.0.bb 
b/meta-oe/recipes-extended/collectd/collectd_5.12.0.bb
index 479c12d6a..bd4a5b3e8 100644
--- a/meta-oe/recipes-extended/collectd/collectd_5.12.0.bb
+++ b/meta-oe/recipes-extended/collectd/collectd_5.12.0.bb
@@ -1,11 +1,12 @@
 SUMMARY = "Collects and summarises system performance statistics"
 DESCRIPTION = "collectd is a daemon which collects system performance 
statistics periodically and provides mechanisms to store the values in a 
variety of ways, for example in RRD files."
+HOMEPAGE = "https://collectd.org/;
 LICENSE = "GPL-2.0-only & MIT"
 LIC_FILES_CHKSUM = "file://COPYING;md5=1bd21f19f7f0c61a7be8ecacb0e28854"
 
 DEPENDS = "rrdtool curl libpcap libxml2 yajl libgcrypt libtool lvm2"
 
-SRC_URI = "http://collectd.org/files/collectd-${PV}.tar.bz2 \
+SRC_URI = "https://collectd.org/files/collectd-${PV}.tar.bz2 \
file://collectd.init \
file://collectd.service \
file://no-gcrypt-badpath.patch \
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105354): 
https://lists.openembedded.org/g/openembedded-devel/message/105354
Mute This Topic: https://lists.openembedded.org/mt/101774188/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [kirkstone cherry-pick] dbus: Specify runstatedir configure option

2023-09-15 Thread Jörg Sommer via lists . openembedded . org

From: Pavel Zhukov 

Without specifing runstatedir tmpfiles.d is configured to use /var/run
for dbus and this causes deprecation warnings in system logs.

(From OE-Core rev: 4df1a16e5c38d0fb724f63d37cc032aa37fa122f)

Signed-off-by: Pavel Zhukov 
Signed-off-by: Luca Ceresoli 
Signed-off-by: Richard Purdie 
---
 meta/recipes-core/dbus/dbus_1.14.8.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/dbus/dbus_1.14.8.bb 
b/meta/recipes-core/dbus/dbus_1.14.8.bb
index 2ba56bf782..4da307ecb3 100644
--- a/meta/recipes-core/dbus/dbus_1.14.8.bb
+++ b/meta/recipes-core/dbus/dbus_1.14.8.bb
@@ -25,6 +25,7 @@ EXTRA_OECONF = "--disable-xml-docs \
 --enable-tests \
 --enable-checks \
 --enable-asserts \
+--runstatedir=/run \
 "
 EXTRA_OECONF:append:class-target = " SYSTEMCTL=${base_bindir}/systemctl"
 
@@ -132,7 +133,7 @@ do_install() {
sed 's:@bindir@:${bindir}:' < ${WORKDIR}/dbus-1.init 
>${WORKDIR}/dbus-1.init.sh
install -m 0755 ${WORKDIR}/dbus-1.init.sh 
${D}${sysconfdir}/init.d/dbus-1
install -d ${D}${sysconfdir}/default/volatiles
-   echo "d messagebus messagebus 0755 ${localstatedir}/run/dbus 
none" \
+   echo "d messagebus messagebus 0755 /run/dbus none" \
 > ${D}${sysconfdir}/default/volatiles/99_dbus
fi
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#187665): 
https://lists.openembedded.org/g/openembedded-core/message/187665
Mute This Topic: https://lists.openembedded.org/mt/101376711/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] dbus patch 1916cb6 in master branch, but not in kirkstone

2023-09-15 Thread Jörg Sommer via lists . openembedded . org

Hello,

the patch »dbus: Specify runstatedir configure option« [1] is in the master 
branch, but not in the kirkstone branch. Is it possible to get it applied 
there, too? Whom can I ping for this?

[1] 
https://git.yoctoproject.org/poky/commit/meta/recipes-core/dbus?id=1916cb69980dbe1de79c3809f50280567e85792b

Kind regards

Jörg Sommer
-- 
Navimatix GmbH
Tatzendpromenade 2
07745 Jena

Geschäftsführer: Steffen Späthe, Jan Rommeley
Registergericht: Amtsgericht Jena, HRB 501480

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#187662): 
https://lists.openembedded.org/g/openembedded-core/message/187662
Mute This Topic: https://lists.openembedded.org/mt/101376323/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH] bonnie++: New recipe for version 2.0

2023-06-26 Thread Jörg Sommer via lists . openembedded . org

Newer versions of bonnie get published on
. Unfortunately, the new version
doesn't compile with g++ 11 which requires *fix-csv2html-data.patch* and
configure fails due to cross compilation which gets fixed
with *fix-configure-lfs.patch*

Signed-off-by: Jörg Sommer 
---
 .../bonnie/bonnie++/fix-configure-lfs.patch   |  37 
 .../bonnie/bonnie++/fix-csv2html-data.patch   | 181 ++
 .../bonnie/bonnie++_2.00a.bb  |  33 
 3 files changed, 251 insertions(+)
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
 create mode 100644 
meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
 create mode 100644 meta-oe/recipes-benchmark/bonnie/bonnie++_2.00a.bb

diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
new file mode 100644
index 00..d28e28658c
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-configure-lfs.patch
@@ -0,0 +1,37 @@
+diff --git i/configure.in w/configure.in
+index 080e40c..f2a2bbe 100644
+--- i/configure.in
 w/configure.in
+@@ -82,8 +82,15 @@ void * thread_func(void * param) { return NULL; }
+   , thread_ldflags="-lpthread"
+   , thread_ldflags="-pthread")
+ 
+-AC_SUBST(large_file)
+-AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
++AC_ARG_ENABLE(lfs,
++  [  --disable-lfs  disable large file support],
++  LFS_CHOICE=$enableval, LFS_CHOICE=check)
++
++if test "$LFS_CHOICE" = yes; then
++   bonniepp_cv_large_file=yes
++elif test "$LFS_CHOICE" = check; then
++   AC_CACHE_CHECK([whether to enable -D_LARGEFILE64_SOURCE], 
bonniepp_cv_large_file,
++  AC_TRY_RUN([#ifndef _LARGEFILE64_SOURCE
+ #define _LARGEFILE64_SOURCE
+ #endif
+ #include 
+@@ -118,8 +125,12 @@ int main () {
+   }
+   close(fd);
+   return 0;
+-}], large_file="yes")
+-if [[ -n "$large_file" ]]; then
++}], bonniepp_cv_large_file="yes"))
++fi
++
++AC_SUBST(large_file)
++
++if [[ -n "$bonniepp_cv_large_file" ]]; then
+large_file="#define _LARGEFILE64_SOURCE"
+ fi
+ 
diff --git a/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch 
b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
new file mode 100644
index 00..78ac347aa8
--- /dev/null
+++ b/meta-oe/recipes-benchmark/bonnie/bonnie++/fix-csv2html-data.patch
@@ -0,0 +1,181 @@
+commit 7e9433a56f22426b11cbc9bd80e0debca67c893b
+Author: Jörg Sommer 
+Date:   Mon Jun 26 12:38:30 2023 +0200
+
+csv2html: Explicitly reference data in top level
+
+With g++ 11 *data* became ambiguous with [std::data][1]. Therefore it's
+needed to explicitly address the variable from the top level scope.
+
+[1] https://en.cppreference.com/w/cpp/iterator/data
+
+diff --git a/bon_csv2html.cpp b/bon_csv2html.cpp
+index e9d9c50..652e330 100644
+--- a/bon_csv2html.cpp
 b/bon_csv2html.cpp
+@@ -87,8 +87,8 @@ int main(int argc, char **argv)
+ read_in(buf);
+   }
+ 
+-  props = new PPCCHAR[data.size()];
+-  for(i = 0; i < data.size(); i++)
++  props = new PPCCHAR[::data.size()];
++  for(i = 0; i < ::data.size(); i++)
+   {
+ props[i] = new PCCHAR[MAX_ITEMS];
+ props[i][0] = NULL;
+@@ -109,7 +109,7 @@ int main(int argc, char **argv)
+   }
+   calc_vals();
+   int mid_width = header();
+-  for(i = 0; i < data.size(); i++)
++  for(i = 0; i < ::data.size(); i++)
+   {
+ // First print the average speed line
+ printf("");
+@@ -171,23 +171,23 @@ int compar(const void *a, const void *b)
+ 
+ void calc_vals()
+ {
+-  ITEM *arr = new ITEM[data.size()];
++  ITEM *arr = new ITEM[::data.size()];
+   for(unsigned int column_ind = 0; column_ind < MAX_ITEMS; column_ind++)
+   {
+ switch(vals[column_ind])
+ {
+ case eNoCols:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ if(column_ind == COL_CONCURRENCY)
+ {
+-  if(data[row_ind][column_ind] && strcmp("1", 
data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && strcmp("1", 
::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+ else
+ {
+-  if(data[row_ind][column_ind] && strlen(data[row_ind][column_ind]))
++  if(::data[row_ind][column_ind] && 
strlen(::data[row_ind][column_ind]))
+ col_used[column_ind] = true;
+ }
+   }
+@@ -195,22 +195,22 @@ void calc_vals()
+ break;
+ case eCPU:
+ {
+-  for(unsigned int row_ind = 0; row_ind < data.size(); row_ind++)
++  for(unsigned int row_ind = 0; row_ind < ::data.size(); row_ind++)
+   {
+ double work, cpu;
+ arr[row_ind].val = 0.0;
+-if(data[row_ind].size() > column_ind
+- && sscanf(data[row_ind][column_ind - 1], "%lf", ) == 1
+- && sscanf(data[row_ind][column_ind], "%lf", ) == 1)
++

Re: [OE-core] [PATCH 4/9] runqemu-ifup: remove only our taps

2023-06-23 Thread Jörg Sommer via lists . openembedded . org

On 22 June 2023 19:01, openembedded-core@lists.openembedded.org wrote:
> If there are other tap interfaces than the interfaces created by the
> runqemu-* scripts, these interfaces are not ignored. This is now fixed
> by filtering the interfaces for a specific prefix in the interface name.
>
> Signed-off-by: Adrian Freihofer 
> ---
>  scripts/runqemu-ifup | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/runqemu-ifup b/scripts/runqemu-ifup
> index fe4492e78b7..c65ceaf1c83 100755
> --- a/scripts/runqemu-ifup
> +++ b/scripts/runqemu-ifup
> @@ -45,7 +45,7 @@ if [ -z "$OE_TAP_NAME" ]; then
>  fi
>
>  if taps=$(ip tuntap list 2>/dev/null); then
> -   tap_no_last=$(echo "$taps" |cut -f 1 -d ":" |sed "s/$OE_TAP_NAME//g" 
> | sort -rn | head -n 1)
> +   tap_no_last=$(echo "$taps" |cut -f 1 -d ":" |grep -E 
> "^$OE_TAP_NAME.*" |sed "s/$OE_TAP_NAME//g" | sort -rn | head -n 1)

You can combine the cut+grep+sed to `sed "/^$OE_TAP_NAME/!d; s///; s/:.*//"`


Regards,

Jörg Sommer

Software Developer / Programmierer
--

Navimatix GmbH

Tatzendpromenade 2

07745 Jena
  

T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de
  



Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#183315): 
https://lists.openembedded.org/g/openembedded-core/message/183315
Mute This Topic: https://lists.openembedded.org/mt/99702217/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [OE-core] [PATCH 1/9] runqemu-ifup: remove uid parameter

2023-06-23 Thread Jörg Sommer via lists . openembedded . org

On 22 June 2023 19:01, openembedded-core@lists.openembedded.org wrote:
> ip tuntap does not need the uid, it was an unused variable/parameter.
> Backward compatibility should be fine.
>
> Signed-off-by: Adrian Freihofer 
> ---
> scripts/runqemu-ifup | 13 -
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/scripts/runqemu-ifup b/scripts/runqemu-ifup
> index 5dc765dee82..26714518020 100755
> --- a/scripts/runqemu-ifup
> +++ b/scripts/runqemu-ifup
> @@ -21,7 +21,7 @@
> #
>
> usage() {
> -   echo "sudo $(basename $0)  "
> +   echo "sudo $(basename $0) "
> }
>
> if [ $EUID -ne 0 ]; then
> @@ -29,17 +29,20 @@ if [ $EUID -ne 0 ]; then
> exit 1
> fi
>
> -if [ $# -ne 2 ]; then
> +if [ $# -eq 2 ]; then
> +   echo "Warning: uid parameter is ignored. It is no longer needed."

Would it be better to send this message to stderr (use `>&2`)?

Regards

Jörg Sommer

Software Developer / Programmierer
--

Navimatix GmbH

Tatzendpromenade 2

07745 Jena


T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de




Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#183312): 
https://lists.openembedded.org/g/openembedded-core/message/183312
Mute This Topic: https://lists.openembedded.org/mt/99702214/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCH 2/2] runqemu-ifupdown/get-tapdevs: Add support for ip tuntap

2023-06-09 Thread Jörg Sommer via lists . openembedded . org

The *ip* command supports the creation and destruction of TAP devices since
2009 and might be more likely installed on systems then *tunctl*. Therefore
it should be tried to setup or teardown the TAP interface with *ip* before
falling back to *tunctl*.

https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=580fbd88f75cc9eea0d28a48c025b090eb9419a7

Signed-off-by: Jörg Sommer 
---
 scripts/runqemu-gen-tapdevs | 26 +-
 scripts/runqemu-ifdown  | 14 --
 scripts/runqemu-ifup| 31 +++
 3 files changed, 44 insertions(+), 27 deletions(-)

diff --git a/scripts/runqemu-gen-tapdevs b/scripts/runqemu-gen-tapdevs
index f2d6cc39c2..ffb82adce6 100755
--- a/scripts/runqemu-gen-tapdevs
+++ b/scripts/runqemu-gen-tapdevs
@@ -50,12 +50,6 @@ if ! [ $COUNT -ge 0 ]; then
exit 1
 fi
 
-TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
-if [[ ! -x "$TUNCTL" || -d "$TUNCTL" ]]; then
-   echo "Error: $TUNCTL is not an executable"
-   usage
-fi
-
 if [ $EUID -ne 0 ]; then
echo "Error: This script must be run with root privileges"
exit
@@ -68,15 +62,29 @@ if [ ! -x "$RUNQEMU_IFUP" ]; then
exit 1
 fi
 
-if ! interfaces=`ip link` 2>/dev/null; then
+TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
+ip_supports_tuntap=false
+if interfaces=`ip tuntap list` 2>/dev/null; then
+   ip_supports_tuntap=true
+   interfaces=`echo "$interfaces |cut -f1 -d:`
+elif [[ ! -x "$TUNCTL" || -d "$TUNCTL" ]]; then
+   echo "Error: $TUNCTL is not an executable"
+   usage
+elif interfaces=`ip link` 2>/dev/null; then
+   interfaces=`echo "$interfaces" | sed '/^[0-9]\+: 
\(docker[0-9]\+\):.*/!d; s//\1/'`
+else
echo "Failed to call 'ip link'" >&2
exit 1
 fi
 
 # Ensure we start with a clean slate
-for tap in `echo "$interfaces" | sed '/^[0-9]\+: \(docker[0-9]\+\):.*/!d; 
s//\1/'`; do
+for tap in $interfaces; do
echo "Note: Destroying pre-existing tap interface $tap..."
-   $TUNCTL -d $tap
+   if $ip_supports_tuntap; then
+   ip tuntap del $tap mode tap
+   else
+   $TUNCTL -d $tap
+   fi
 done
 rm -f /etc/runqemu-nosudo
 
diff --git a/scripts/runqemu-ifdown b/scripts/runqemu-ifdown
index e0eb5344c6..f72166b32b 100755
--- a/scripts/runqemu-ifdown
+++ b/scripts/runqemu-ifdown
@@ -33,13 +33,15 @@ fi
 TAP=$1
 STAGING_BINDIR_NATIVE=$2
 
-TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
-if [ ! -e "$TUNCTL" ]; then
-   echo "Error: Unable to find tunctl binary in '$STAGING_BINDIR_NATIVE', 
please bitbake qemu-helper-native"
-   exit 1
-fi
+if !ip tuntap del $TAP mode tap 2>/dev/null; then
+   TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
+   if [ ! -e "$TUNCTL" ]; then
+   echo "Error: Unable to find tunctl binary in 
'$STAGING_BINDIR_NATIVE', please bitbake qemu-helper-native"
+   exit 1
+   fi
 
-$TUNCTL -d $TAP
+   $TUNCTL -d $TAP
+fi
 
 IFCONFIG=`which ip 2> /dev/null`
 if [ "x$IFCONFIG" = "x" ]; then
diff --git a/scripts/runqemu-ifup b/scripts/runqemu-ifup
index bb661740c5..5fdcddeeda 100755
--- a/scripts/runqemu-ifup
+++ b/scripts/runqemu-ifup
@@ -41,22 +41,29 @@ USERID="-u $1"
 GROUP="-g $2"
 STAGING_BINDIR_NATIVE=$3
 
-TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
-if [ ! -x "$TUNCTL" ]; then
-   echo "Error: Unable to find tunctl binary in '$STAGING_BINDIR_NATIVE', 
please bitbake qemu-helper-native"
-   exit 1
+if taps=$(ip tuntap list 2>/dev/null); then
+   tap_no=$(( $(echo "$taps" |sort -r |sed 's/^tap//; s/:.*//; q') + 1 ))
+   ip tuntap add tap$tap_no mode tap group $2 && TAP=tap$tap_no
 fi
 
-TAP=`$TUNCTL -b $GROUP 2>&1`
-STATUS=$?
-if [ $STATUS -ne 0 ]; then
-# If tunctl -g fails, try using tunctl -u, for older host kernels 
-# which do not support the TUNSETGROUP ioctl
-   TAP=`$TUNCTL -b $USERID 2>&1`
+if [ -z $TAP ]; then
+   TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
+   if [ ! -x "$TUNCTL" ]; then
+   echo "Error: Unable to find tunctl binary in 
'$STAGING_BINDIR_NATIVE', please bitbake qemu-helper-native"
+   exit 1
+   fi
+
+   TAP=`$TUNCTL -b $GROUP 2>&1`
STATUS=$?
if [ $STATUS -ne 0 ]; then
-   echo "tunctl failed:"
-   exit 1
+   # If tunctl -g fails, try using tunctl -u, for older host kernels
+   # which do not support the TUNSETGROUP ioctl
+   TAP=`$TUNCTL -b $USERID 2>&1`
+   STATUS=$?
+   if [ $STATUS -ne 0 ]; then
+   echo "tunctl failed:"
+   exit 1
+   fi
fi
 fi
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#182547): 
https://lists.openembedded.org/g/openembedded-core/message/182547
Mute This Topic: https://lists.openembedded.org/mt/99424585/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe:

[OE-core] [PATCH 1/2] runqemu-gen-tapdevs: Refactoring

2023-06-09 Thread Jörg Sommer via lists . openembedded . org

The changes are mostly about early exit which causes indentation changes;
check with `git diff -w`. Another change is the check for ip by simply
calling it and deciding upon the exit code, if it's fine or not.

Signed-off-by: Jörg Sommer 
---
 scripts/runqemu-gen-tapdevs | 77 ++---
 1 file changed, 37 insertions(+), 40 deletions(-)

diff --git a/scripts/runqemu-gen-tapdevs b/scripts/runqemu-gen-tapdevs
index a6ee4517da..f2d6cc39c2 100755
--- a/scripts/runqemu-gen-tapdevs
+++ b/scripts/runqemu-gen-tapdevs
@@ -44,6 +44,12 @@ GID=$2
 COUNT=$3
 STAGING_BINDIR_NATIVE=$4
 
+# check if COUNT is a number and >= 0
+if ! [ $COUNT -ge 0 ]; then
+   echo "Error: Incorrect count: $COUNT"
+   exit 1
+fi
+
 TUNCTL=$STAGING_BINDIR_NATIVE/tunctl
 if [[ ! -x "$TUNCTL" || -d "$TUNCTL" ]]; then
echo "Error: $TUNCTL is not an executable"
@@ -62,48 +68,39 @@ if [ ! -x "$RUNQEMU_IFUP" ]; then
exit 1
 fi
 
-IFCONFIG=`which ip 2> /dev/null`
-if [ -z "$IFCONFIG" ]; then
-   # Is it ever anywhere else?
-   IFCONFIG=/sbin/ip
-fi
-if [ ! -x "$IFCONFIG" ]; then
-   echo "$IFCONFIG cannot be executed"
-   exit 1
-fi
-
-if [ $COUNT -ge 0 ]; then
-   # Ensure we start with a clean slate
-   for tap in `$IFCONFIG link | grep tap | awk '{ print \$2 }' | sed 
s/://`; do
-   echo "Note: Destroying pre-existing tap interface $tap..."
-   $TUNCTL -d $tap
-   done
-   rm -f /etc/runqemu-nosudo
-else
-   echo "Error: Incorrect count: $COUNT"
+if ! interfaces=`ip link` 2>/dev/null; then
+   echo "Failed to call 'ip link'" >&2
exit 1
 fi
 
-if [ $COUNT -gt 0 ]; then
-   echo "Creating $COUNT tap devices for UID: $TUID GID: $GID..."
-   for ((index=0; index < $COUNT; index++)); do
-   echo "Creating tap$index"
-   ifup=`$RUNQEMU_IFUP $TUID $GID $STAGING_BINDIR_NATIVE 2>&1`
-   if [ $? -ne 0 ]; then
-   echo "Error running tunctl: $ifup"
-   exit 1
-   fi
-   done
+# Ensure we start with a clean slate
+for tap in `echo "$interfaces" | sed '/^[0-9]\+: \(docker[0-9]\+\):.*/!d; 
s//\1/'`; do
+   echo "Note: Destroying pre-existing tap interface $tap..."
+   $TUNCTL -d $tap
+done
+rm -f /etc/runqemu-nosudo
 
-   echo "Note: For systems running NetworkManager, it's recommended"
-   echo "Note: that the tap devices be set as unmanaged in the"
-   echo "Note: NetworkManager.conf file. Add the following lines to"
-   echo "Note: /etc/NetworkManager/NetworkManager.conf"
-   echo "[keyfile]"
-   echo "unmanaged-devices=interface-name:tap*"
-
-   # The runqemu script will check for this file, and if it exists,
-   # will use the existing bank of tap devices without creating
-   # additional ones via sudo.
-   touch /etc/runqemu-nosudo
+if [ $COUNT -eq 0 ]; then
+   exit 0
 fi
+
+echo "Creating $COUNT tap devices for UID: $TUID GID: $GID..."
+for ((index=0; index < $COUNT; index++)); do
+   echo "Creating tap$index"
+   if ! ifup=`$RUNQEMU_IFUP $TUID $GID $STAGING_BINDIR_NATIVE 2>&1`; then
+   echo "Error running tunctl: $ifup"
+   exit 1
+   fi
+done
+
+echo "Note: For systems running NetworkManager, it's recommended"
+echo "Note: that the tap devices be set as unmanaged in the"
+echo "Note: NetworkManager.conf file. Add the following lines to"
+echo "Note: /etc/NetworkManager/NetworkManager.conf"
+echo "[keyfile]"
+echo "unmanaged-devices=interface-name:tap*"
+
+# The runqemu script will check for this file, and if it exists,
+# will use the existing bank of tap devices without creating
+# additional ones via sudo.
+touch /etc/runqemu-nosudo
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#182546): 
https://lists.openembedded.org/g/openembedded-core/message/182546
Mute This Topic: https://lists.openembedded.org/mt/99424584/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] Fw: [poky] [PATCH] runqemu-ifupdown: Add support for ip tuntap

2023-06-09 Thread Jörg Sommer via lists . openembedded . org

Sorry, I have to resend this message, because I wasn't subscribed to oe-core.


From: Jörg Sommer 
Sent: Friday, 9 June 2023 09:30
To: Richard Purdie ; 
p...@lists.yoctoproject.org ; 
openembedded-core@lists.openembedded.org 

Subject: Re: [poky] [PATCH] runqemu-ifupdown: Add support for ip tuntap

@openembedded: I have proposed a patch to runqemu-ifup/down to use `ip tuntap` 
as an alternative to tunctl for setting up the tap interface. Now the question 
came up if tunctl could be fully dropped.

On 8 June 2023 22:18, Richard Purdie wrote:
> On Thu, 2023-06-08 at 15:07 +0200, Jörg Sommer via
> lists.yoctoproject.org wrote:
> > The *ip* command supports the creation and destruction of TAP devices since
> > 2009 and might be more likely installed on systems then *tunctl*. Therefore
> > it should be tried to setup or teardown the TAP interface with *ip* before
> > falling back to *tunctl*.
> >
> > https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=580fbd88f75cc9eea0d28a48c025b090eb9419a7
> >
> > Signed-off-by: Jörg Sommer 
> > ---
> >  scripts/runqemu-ifdown | 14 --
> >  scripts/runqemu-ifup   | 31 +++
> >  2 files changed, 27 insertions(+), 18 deletions(-)
>
> This does make me wonder if we could just drop tunctl now?

I think so. But do all systems support ip, now? If so, the part for ifconfig 
could be dropped.

> We originally had this as ifconfig couldn't do what we needed and ip
> was comparatively rare on systems. Things have changed and moved on!
>
> Did the gen-tap-devs script also need updating?

Yeah, you're right. I forgot about it.

> Also, this patch does need to go to the openembedded-core list as it is
> changing that repository which poky is built from.

Thanks for the pointing.

Mit freundlichen Grüßen


Jörg Sommer

Software Developer / Programmierer



--

Navimatix GmbH

Tatzendpromenade 2

07745 Jena


T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de






Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#182543): 
https://lists.openembedded.org/g/openembedded-core/message/182543
Mute This Topic: https://lists.openembedded.org/mt/99423892/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

48 matches

Mail list logo