All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-virtualization][PATCH] diod: Update to b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6
@ 2024-03-11 12:24 Ola x Nilsson
  2024-03-11 12:36 ` Bruce Ashfield
  0 siblings, 1 reply; 3+ messages in thread
From: Ola x Nilsson @ 2024-03-11 12:24 UTC (permalink / raw)
  To: meta-virtualization; +Cc: Ola x Nilsson

Just needs a single patch to handle the lua feature in configure when
cross-compiling.

Signed-off-by: Ola x Nilsson <olani@axis.com>
---
 recipes-extended/diod/diod_1.0.24.bb          |  13 +-
 .../0001-build-Find-lua-with-pkg-config.patch |  50 +++++++
 ...ilds-to-work-with-separate-build-dir.patch | 126 ------------------
 ...od-ops.c-add-header-file-for-makedev.patch |  37 -----
 ...drod-add-option-to-config-systemddir.patch |  51 -------
 .../0002-auto.diod.in-remove-bashisms.patch   |  47 -------
 6 files changed, 55 insertions(+), 269 deletions(-)
 create mode 100644 recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
 delete mode 100644 recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
 delete mode 100644 recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
 delete mode 100644 recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
 delete mode 100644 recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch

diff --git a/recipes-extended/diod/diod_1.0.24.bb b/recipes-extended/diod/diod_1.0.24.bb
index 2b87a8ae..5e191dc0 100644
--- a/recipes-extended/diod/diod_1.0.24.bb
+++ b/recipes-extended/diod/diod_1.0.24.bb
@@ -9,22 +9,19 @@ LICENSE = "GPL-2.0-only"
 LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552"
 
 PV = "1.0.24+git"
-SRCREV = "0ea3fe3d829b5085307cd27a512708d99ef48199"
+SRCREV = "b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6"
 SRC_URI = "git://github.com/chaos/diod.git;protocol=https;branch=master \
            file://diod \
            file://diod.conf \
-           file://0001-build-allow-builds-to-work-with-separate-build-dir.patch \
-           file://0002-auto.diod.in-remove-bashisms.patch \
-           file://0001-diod-ops.c-add-header-file-for-makedev.patch \
-           file://0001-drod-add-option-to-config-systemddir.patch \
-          "
+           file://0001-build-Find-lua-with-pkg-config.patch \
+           "
 DEPENDS = "libcap ncurses tcp-wrappers lua"
 
-EXTRA_OECONF = "--with-systemddir=${systemd_unitdir}/system"
+EXTRA_OEMAKE += "systemddir=${systemd_unitdir}/system"
 
 S = "${WORKDIR}/git"
 
-inherit autotools systemd
+inherit autotools pkgconfig systemd
 
 do_install:append () {
         # install our init based on start-stop-daemon
diff --git a/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch b/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
new file mode 100644
index 00000000..9f61792a
--- /dev/null
+++ b/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
@@ -0,0 +1,50 @@
+From 8124eec4e59222f1c3f3dc4d7e6706e13c33755c Mon Sep 17 00:00:00 2001
+From: Ola x Nilsson <olani@axis.com>
+Date: Wed, 6 Mar 2024 18:40:16 +0100
+Subject: [PATCH] build: Find lua with pkg-config
+
+ax_lua.m4 does not support cross compilation.
+
+Upstream-Status: Inappropriate (Embedded specific)
+
+---
+ configure.ac | 26 +++++++++++++++++++++++++-
+ 1 file changed, 25 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index e8fcdb2..f3ebdc5 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -121,7 +121,31 @@ AC_ARG_ENABLE([config],
+   [AS_HELP_STRING([--disable-config], [disable lua config file support])])
+ 
+ AS_IF([test "x$enable_config" != "xno"], [
+-  AX_PROG_LUA([5.1],[5.5])
++  dnl Use pkg_config to check for lua settings.  This is not standard
++  dnl as lua typically does not distribute a pc file.  The
++  dnl OpenEmbedded recipe adds one that can be used.  Reuse/replicate
++  dnl as much of AX_PROG_LUA as possible so AX_LUA_HEADERS and
++  dnl AX_LUA_LIBS are still usable.
++  PKG_CHECK_EXISTS([lua], [
++    AS_IF([
++      $PKG_CONFIG --atleast-version=5.1 lua && $PKG_CONFIG --max-version=5.5 lua
++    ],[
++      AC_ARG_VAR([LUA], [The LUA interpreter, e.g. /usr/bin/lua5.1])
++	  dnl should really check that the file exists ...
++      AC_SUBST([LUA], [${bindir}/lua])
++	  AC_CACHE_CHECK([for $LUA version], [ax_cv_lua_version], [
++	    ax_cv_lua_version=[`$PKG_CONFIG --modversion lua | $SED 's/^\([0-9]\+\.[0-9]\+\).*/\1/'`]
++      ])
++	  AS_IF([test "x$ax_cv_lua_version" = 'x'],
++       [AC_MSG_ERROR([invalid Lua version number])])
++      AC_SUBST([LUA_VERSION], [$ax_cv_lua_version])
++      AC_SUBST([LUA_SHORT_VERSION], [`echo "$LUA_VERSION" | $SED 's|\.||'`])
++    ],[
++	  AC_MSG_ERROR([Incorrect lua version])
++	])
++  ],[
++    AX_PROG_LUA([5.1],[5.5])
++  ])
+   AX_LUA_HEADERS([],[
+     AC_MSG_ERROR([Install lua headers or configure with --disable-config])
+   ])
diff --git a/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch b/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
deleted file mode 100644
index f40e0eb5..00000000
--- a/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
+++ /dev/null
@@ -1,126 +0,0 @@
-From 43403468298ef4167baa5d84de2ee2eaf7f4007a Mon Sep 17 00:00:00 2001
-From: Mark Asselstine <mark.asselstine@windriver.com>
-Date: Fri, 9 Jan 2015 18:34:04 +0000
-Subject: [PATCH 1/2] build: allow builds to work with separate build dir
-
-Remove assumptions in include paths that assume the build
-is happening in the source directories.
-
-Upstream-Status: Inappropriate [embedded specific]
-
-Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
----
- diod/Makefile.am              | 2 +-
- libdiod/Makefile.am           | 2 +-
- liblsd/Makefile.am            | 2 +-
- libnpclient/Makefile.am       | 2 +-
- scripts/Makefile.am           | 6 +++---
- tests/kern/dbench/Makefile.am | 4 ++--
- utils/Makefile.am             | 2 +-
- 7 files changed, 10 insertions(+), 10 deletions(-)
-
-diff --git a/diod/Makefile.am b/diod/Makefile.am
-index 7644be7..fdaf130 100644
---- a/diod/Makefile.am
-+++ b/diod/Makefile.am
-@@ -1,7 +1,7 @@
- AM_CFLAGS = @GCCWARN@
- 
- AM_CPPFLAGS = \
--	-I../libnpfs -I../liblsd -I../libdiod
-+	-I$(srcdir)/../libnpfs -I$(srcdir)/../liblsd -I$(srcdir)/../libdiod
- 
- sbin_PROGRAMS = diod
- 
-diff --git a/libdiod/Makefile.am b/libdiod/Makefile.am
-index 4810e14..6905cdd 100644
---- a/libdiod/Makefile.am
-+++ b/libdiod/Makefile.am
-@@ -1,7 +1,7 @@
- AM_CFLAGS = @GCCWARN@
- 
- AM_CPPFLAGS = \
--	-I../libnpfs -I../libnpclient -I../liblsd
-+	-I$(srcdir)/../libnpfs -I$(srcdir)/../libnpclient -I$(srcdir)/../liblsd
- 
- noinst_LIBRARIES = libdiod.a
- 
-diff --git a/liblsd/Makefile.am b/liblsd/Makefile.am
-index c3e5658..7e18bf8 100644
---- a/liblsd/Makefile.am
-+++ b/liblsd/Makefile.am
-@@ -1,6 +1,6 @@
- AM_CFLAGS = @GCCWARN@
- 
--AM_CPPFLAGS = -I../libdiod
-+AM_CPPFLAGS = -I$(srcdir)/../libdiod
- 
- noinst_LIBRARIES = liblsd.a
- 
-diff --git a/libnpclient/Makefile.am b/libnpclient/Makefile.am
-index 5305df9..cbaf266 100644
---- a/libnpclient/Makefile.am
-+++ b/libnpclient/Makefile.am
-@@ -1,7 +1,7 @@
- AM_CFLAGS = @GCCWARN@
- 
- AM_CPPFLAGS = \
--	-I../libnpfs
-+	-I$(srcdir)/../libnpfs
- 
- noinst_LIBRARIES = libnpclient.a
- 
-diff --git a/scripts/Makefile.am b/scripts/Makefile.am
-index 51c24a3..2aba728 100644
---- a/scripts/Makefile.am
-+++ b/scripts/Makefile.am
-@@ -1,9 +1,9 @@
--systemddir=$(sysconfdir)/systemd/system
-+systemddir=/lib/systemd/system
- 
- install-data-local:
--	$(top_srcdir)/config/install-sh -m 755 $(srcdir)/auto.diod \
-+	$(top_srcdir)/config/install-sh -m 755 ./auto.diod \
- 		$(DESTDIR)$(sysconfdir)/auto.diod
--	$(top_srcdir)/config/install-sh -m 755 $(srcdir)/diod.service \
-+	$(top_srcdir)/config/install-sh -m 644 ./diod.service \
- 		$(DESTDIR)$(systemddir)/diod.service
- 
- uninstall-local:
-diff --git a/tests/kern/dbench/Makefile.am b/tests/kern/dbench/Makefile.am
-index e0cdefa..1704f9f 100644
---- a/tests/kern/dbench/Makefile.am
-+++ b/tests/kern/dbench/Makefile.am
-@@ -2,7 +2,7 @@ AM_CFLAGS = -w
- 
- # VERSION=4.00
- 
--AM_CPPFLAGS = -DDATADIR=\"$(X_DATADIR)\"
-+AM_CPPFLAGS = -I. -DDATADIR=\"$(X_DATADIR)\"
- 
- LDADD = $(LIBPOPT)
- 
-@@ -25,6 +25,6 @@ BUILT_SOURCES = proto.h
- CLEANFILES = proto.h
- 
- proto.h: $(dbench_SOURCES) mkproto.pl
--	perl mkproto.pl $(dbench_SOURCES) > proto.h
-+	$(srcdir)/mkproto.pl $(patsubst %,$(srcdir),$(dbench_SOURCES)) > proto.h
- 
- EXTRA_DIST = mkproto.pl
-diff --git a/utils/Makefile.am b/utils/Makefile.am
-index 169b6c4..91d571d 100644
---- a/utils/Makefile.am
-+++ b/utils/Makefile.am
-@@ -1,7 +1,7 @@
- AM_CFLAGS = @GCCWARN@
- 
- AM_CPPFLAGS = \
--	-I../libnpfs -I../liblsd -I../libdiod -I../libnpclient
-+	-I$(srcdir)/../libnpfs -I$(srcdir)/../liblsd -I$(srcdir)/../libdiod -I$(srcdir)/../libnpclient
- 
- sbin_PROGRAMS = diodmount diodcat dtop diodload diodls diodshowmount dioddate
- 
--- 
-2.11.0
-
diff --git a/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch b/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
deleted file mode 100644
index 49bc507b..00000000
--- a/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
+++ /dev/null
@@ -1,37 +0,0 @@
-From f75836731d6051d85e6317eb4c12465bafb4973e Mon Sep 17 00:00:00 2001
-From: "Hongzhi.Song" <hongzhi.song@windriver.com>
-Date: Fri, 31 Aug 2018 05:47:02 -0400
-Subject: [PATCH] diod/ops.c: add header file for makedev
-
-Error:
-diod/ops.c:845: undefined reference to `makedev'
-
-Fixed:
-Glibc removes sys/sysmacros.h which defines makedev from sys/types.h
-since v2.28. [Commit ID: e16deca62e16f]
-
-And then glibc suggestions us to include <sys/sysmacros.h> directly if
-code needs it.
-
-Upstream-Status: Submitted
-
-Signed-off-by: Hongzhi.Song <hongzhi.song@windriver.com>
----
- diod/ops.c | 1 +
- 1 file changed, 1 insertion(+)
-
-diff --git a/diod/ops.c b/diod/ops.c
-index c6c8291..469a502 100644
---- a/diod/ops.c
-+++ b/diod/ops.c
-@@ -74,6 +74,7 @@
- #include <pthread.h>
- #include <errno.h>
- #include <sys/types.h>
-+#include <sys/sysmacros.h>
- 
- #ifdef __FreeBSD__
- #if !__BSD_VISIBLE
--- 
-2.8.1
-
diff --git a/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch b/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
deleted file mode 100644
index 4931a19b..00000000
--- a/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
+++ /dev/null
@@ -1,51 +0,0 @@
-From 365159534b1981dfe291ecc05d2455f32652a166 Mon Sep 17 00:00:00 2001
-From: Changqing Li <changqing.li@windriver.com>
-Date: Mon, 26 Aug 2019 18:00:39 +0800
-Subject: [PATCH] drod: add option to config systemddir
-
-Not need to send upstream, since upstream have refactor
-related code, but not released. and this version release 
-4 years ago, not proper to just backport one or more
-commits, this patch maybe could be dropped after upgrade.
-
-Upstream-Status: Inappropriate [embedded specific]
-
-Signed-off-by: Changqing Li <changqing.li@windriver.com>
----
- configure.ac        | 9 +++++++++
- scripts/Makefile.am | 2 +-
- 2 files changed, 10 insertions(+), 1 deletion(-)
-
-diff --git a/configure.ac b/configure.ac
-index 1cb8579..3e4a5ec 100644
---- a/configure.ac
-+++ b/configure.ac
-@@ -25,6 +25,15 @@ AC_DEFINE([_GNU_SOURCE], 1,
-           [Define _GNU_SOURCE so that we get all necessary prototypes])
- 
- ##
-+#Add option to config systemddir
-+##
-+AC_ARG_WITH([systemddir],
-+  [AS_HELP_STRING([--with-systemddir=DIR], [systemd dir])],
-+  [systemddir=$withval],
-+  [systemddir="/lib/systemd/system"])
-+AC_SUBST([systemddir], [$systemddir])
-+
-+##
- # Checks for programs
- ##
- AC_PROG_CC
-diff --git a/scripts/Makefile.am b/scripts/Makefile.am
-index 2aba728..40822b7 100644
---- a/scripts/Makefile.am
-+++ b/scripts/Makefile.am
-@@ -1,4 +1,4 @@
--systemddir=/lib/systemd/system
-+systemddir=@systemddir@
- 
- install-data-local:
- 	$(top_srcdir)/config/install-sh -m 755 ./auto.diod \
--- 
-2.7.4
-
diff --git a/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch b/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
deleted file mode 100644
index 13e0a86e..00000000
--- a/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
+++ /dev/null
@@ -1,47 +0,0 @@
-From 5a9e09dc5de833db11607530351cd87cecbfd17e Mon Sep 17 00:00:00 2001
-From: Roy Li <rongqing.li@windriver.com>
-Date: Thu, 22 Jun 2017 06:32:30 +0000
-Subject: [PATCH 2/2] auto.diod.in: remove bashisms
-
-Upstream-Status: Inappropriate [embedded specific]
-
-Signed-off-by: Roy Li <rongqing.li@windriver.com>
----
- scripts/auto.diod.in | 14 +++++++-------
- 1 file changed, 7 insertions(+), 7 deletions(-)
-
-diff --git a/scripts/auto.diod.in b/scripts/auto.diod.in
-index f63e004..4d5fc3a 100755
---- a/scripts/auto.diod.in
-+++ b/scripts/auto.diod.in
-@@ -1,4 +1,4 @@
--#!/bin/bash
-+#!/bin/sh
- #
- # auto.diod - executable automounter map for diod file systems
- #
-@@ -41,15 +41,15 @@ dcatopts="${DIOD_TIMEOUT:+-t $DIOD_TIMEOUT}"
- for server in $DIOD_SERVERS; do
-     $DIOD_DIODCAT -s $server $dcatopts exports | awk '{print $1}' |\
-     while read path; do
--        if [ "$path" == "/" ]; then
--            if [ "$key" == "ROOT" ]; then
-+        if [ "$path" = "/" ]; then
-+            if [ "$key" = "ROOT" ]; then
-                 echo "$prefix $server:$path"
-                 exit 0
-             fi
--        elif [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//./g')" ] \
--          || [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//_/g')" ] \
--          || [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//-/g')" ] \
--          || [ "$key" == "$(basename $path)" ]; then
-+        elif [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//./g')" ] \
-+          || [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//_/g')" ] \
-+          || [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//-/g')" ] \
-+          || [ "$key" = "$(basename $path)" ]; then
-             echo "$prefix $server:$path"
-             exit 0
-         fi
--- 
-2.11.0
-
-- 
2.39.2



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

* Re: [meta-virtualization][PATCH] diod: Update to b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6
  2024-03-11 12:24 [meta-virtualization][PATCH] diod: Update to b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6 Ola x Nilsson
@ 2024-03-11 12:36 ` Bruce Ashfield
  2024-03-11 14:57   ` Ola x Nilsson
  0 siblings, 1 reply; 3+ messages in thread
From: Bruce Ashfield @ 2024-03-11 12:36 UTC (permalink / raw)
  To: Ola x Nilsson; +Cc: meta-virtualization, Ola x Nilsson

On Mon, Mar 11, 2024 at 8:24 AM Ola x Nilsson <ola.x.nilsson@axis.com> wrote:
>
> Just needs a single patch to handle the lua feature in configure when
> cross-compiling.
>

This needs a listing of the changes between the two SRCREVs.

I haven't gotten around to updating diod for the release, so I'll consider
this once ceph is working again, but for now, it will probably wait until
my round of package bumps post release.

Bruce

> Signed-off-by: Ola x Nilsson <olani@axis.com>
> ---
>  recipes-extended/diod/diod_1.0.24.bb          |  13 +-
>  .../0001-build-Find-lua-with-pkg-config.patch |  50 +++++++
>  ...ilds-to-work-with-separate-build-dir.patch | 126 ------------------
>  ...od-ops.c-add-header-file-for-makedev.patch |  37 -----
>  ...drod-add-option-to-config-systemddir.patch |  51 -------
>  .../0002-auto.diod.in-remove-bashisms.patch   |  47 -------
>  6 files changed, 55 insertions(+), 269 deletions(-)
>  create mode 100644 recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
>  delete mode 100644 recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
>  delete mode 100644 recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
>  delete mode 100644 recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
>  delete mode 100644 recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
>
> diff --git a/recipes-extended/diod/diod_1.0.24.bb b/recipes-extended/diod/diod_1.0.24.bb
> index 2b87a8ae..5e191dc0 100644
> --- a/recipes-extended/diod/diod_1.0.24.bb
> +++ b/recipes-extended/diod/diod_1.0.24.bb
> @@ -9,22 +9,19 @@ LICENSE = "GPL-2.0-only"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552"
>
>  PV = "1.0.24+git"
> -SRCREV = "0ea3fe3d829b5085307cd27a512708d99ef48199"
> +SRCREV = "b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6"
>  SRC_URI = "git://github.com/chaos/diod.git;protocol=https;branch=master \
>             file://diod \
>             file://diod.conf \
> -           file://0001-build-allow-builds-to-work-with-separate-build-dir.patch \
> -           file://0002-auto.diod.in-remove-bashisms.patch \
> -           file://0001-diod-ops.c-add-header-file-for-makedev.patch \
> -           file://0001-drod-add-option-to-config-systemddir.patch \
> -          "
> +           file://0001-build-Find-lua-with-pkg-config.patch \
> +           "
>  DEPENDS = "libcap ncurses tcp-wrappers lua"
>
> -EXTRA_OECONF = "--with-systemddir=${systemd_unitdir}/system"
> +EXTRA_OEMAKE += "systemddir=${systemd_unitdir}/system"
>
>  S = "${WORKDIR}/git"
>
> -inherit autotools systemd
> +inherit autotools pkgconfig systemd
>
>  do_install:append () {
>          # install our init based on start-stop-daemon
> diff --git a/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch b/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
> new file mode 100644
> index 00000000..9f61792a
> --- /dev/null
> +++ b/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
> @@ -0,0 +1,50 @@
> +From 8124eec4e59222f1c3f3dc4d7e6706e13c33755c Mon Sep 17 00:00:00 2001
> +From: Ola x Nilsson <olani@axis.com>
> +Date: Wed, 6 Mar 2024 18:40:16 +0100
> +Subject: [PATCH] build: Find lua with pkg-config
> +
> +ax_lua.m4 does not support cross compilation.
> +
> +Upstream-Status: Inappropriate (Embedded specific)
> +
> +---
> + configure.ac | 26 +++++++++++++++++++++++++-
> + 1 file changed, 25 insertions(+), 1 deletion(-)
> +
> +diff --git a/configure.ac b/configure.ac
> +index e8fcdb2..f3ebdc5 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -121,7 +121,31 @@ AC_ARG_ENABLE([config],
> +   [AS_HELP_STRING([--disable-config], [disable lua config file support])])
> +
> + AS_IF([test "x$enable_config" != "xno"], [
> +-  AX_PROG_LUA([5.1],[5.5])
> ++  dnl Use pkg_config to check for lua settings.  This is not standard
> ++  dnl as lua typically does not distribute a pc file.  The
> ++  dnl OpenEmbedded recipe adds one that can be used.  Reuse/replicate
> ++  dnl as much of AX_PROG_LUA as possible so AX_LUA_HEADERS and
> ++  dnl AX_LUA_LIBS are still usable.
> ++  PKG_CHECK_EXISTS([lua], [
> ++    AS_IF([
> ++      $PKG_CONFIG --atleast-version=5.1 lua && $PKG_CONFIG --max-version=5.5 lua
> ++    ],[
> ++      AC_ARG_VAR([LUA], [The LUA interpreter, e.g. /usr/bin/lua5.1])
> ++        dnl should really check that the file exists ...
> ++      AC_SUBST([LUA], [${bindir}/lua])
> ++        AC_CACHE_CHECK([for $LUA version], [ax_cv_lua_version], [
> ++          ax_cv_lua_version=[`$PKG_CONFIG --modversion lua | $SED 's/^\([0-9]\+\.[0-9]\+\).*/\1/'`]
> ++      ])
> ++        AS_IF([test "x$ax_cv_lua_version" = 'x'],
> ++       [AC_MSG_ERROR([invalid Lua version number])])
> ++      AC_SUBST([LUA_VERSION], [$ax_cv_lua_version])
> ++      AC_SUBST([LUA_SHORT_VERSION], [`echo "$LUA_VERSION" | $SED 's|\.||'`])
> ++    ],[
> ++        AC_MSG_ERROR([Incorrect lua version])
> ++      ])
> ++  ],[
> ++    AX_PROG_LUA([5.1],[5.5])
> ++  ])
> +   AX_LUA_HEADERS([],[
> +     AC_MSG_ERROR([Install lua headers or configure with --disable-config])
> +   ])
> diff --git a/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch b/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
> deleted file mode 100644
> index f40e0eb5..00000000
> --- a/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
> +++ /dev/null
> @@ -1,126 +0,0 @@
> -From 43403468298ef4167baa5d84de2ee2eaf7f4007a Mon Sep 17 00:00:00 2001
> -From: Mark Asselstine <mark.asselstine@windriver.com>
> -Date: Fri, 9 Jan 2015 18:34:04 +0000
> -Subject: [PATCH 1/2] build: allow builds to work with separate build dir
> -
> -Remove assumptions in include paths that assume the build
> -is happening in the source directories.
> -
> -Upstream-Status: Inappropriate [embedded specific]
> -
> -Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
> ----
> - diod/Makefile.am              | 2 +-
> - libdiod/Makefile.am           | 2 +-
> - liblsd/Makefile.am            | 2 +-
> - libnpclient/Makefile.am       | 2 +-
> - scripts/Makefile.am           | 6 +++---
> - tests/kern/dbench/Makefile.am | 4 ++--
> - utils/Makefile.am             | 2 +-
> - 7 files changed, 10 insertions(+), 10 deletions(-)
> -
> -diff --git a/diod/Makefile.am b/diod/Makefile.am
> -index 7644be7..fdaf130 100644
> ---- a/diod/Makefile.am
> -+++ b/diod/Makefile.am
> -@@ -1,7 +1,7 @@
> - AM_CFLAGS = @GCCWARN@
> -
> - AM_CPPFLAGS = \
> --      -I../libnpfs -I../liblsd -I../libdiod
> -+      -I$(srcdir)/../libnpfs -I$(srcdir)/../liblsd -I$(srcdir)/../libdiod
> -
> - sbin_PROGRAMS = diod
> -
> -diff --git a/libdiod/Makefile.am b/libdiod/Makefile.am
> -index 4810e14..6905cdd 100644
> ---- a/libdiod/Makefile.am
> -+++ b/libdiod/Makefile.am
> -@@ -1,7 +1,7 @@
> - AM_CFLAGS = @GCCWARN@
> -
> - AM_CPPFLAGS = \
> --      -I../libnpfs -I../libnpclient -I../liblsd
> -+      -I$(srcdir)/../libnpfs -I$(srcdir)/../libnpclient -I$(srcdir)/../liblsd
> -
> - noinst_LIBRARIES = libdiod.a
> -
> -diff --git a/liblsd/Makefile.am b/liblsd/Makefile.am
> -index c3e5658..7e18bf8 100644
> ---- a/liblsd/Makefile.am
> -+++ b/liblsd/Makefile.am
> -@@ -1,6 +1,6 @@
> - AM_CFLAGS = @GCCWARN@
> -
> --AM_CPPFLAGS = -I../libdiod
> -+AM_CPPFLAGS = -I$(srcdir)/../libdiod
> -
> - noinst_LIBRARIES = liblsd.a
> -
> -diff --git a/libnpclient/Makefile.am b/libnpclient/Makefile.am
> -index 5305df9..cbaf266 100644
> ---- a/libnpclient/Makefile.am
> -+++ b/libnpclient/Makefile.am
> -@@ -1,7 +1,7 @@
> - AM_CFLAGS = @GCCWARN@
> -
> - AM_CPPFLAGS = \
> --      -I../libnpfs
> -+      -I$(srcdir)/../libnpfs
> -
> - noinst_LIBRARIES = libnpclient.a
> -
> -diff --git a/scripts/Makefile.am b/scripts/Makefile.am
> -index 51c24a3..2aba728 100644
> ---- a/scripts/Makefile.am
> -+++ b/scripts/Makefile.am
> -@@ -1,9 +1,9 @@
> --systemddir=$(sysconfdir)/systemd/system
> -+systemddir=/lib/systemd/system
> -
> - install-data-local:
> --      $(top_srcdir)/config/install-sh -m 755 $(srcdir)/auto.diod \
> -+      $(top_srcdir)/config/install-sh -m 755 ./auto.diod \
> -               $(DESTDIR)$(sysconfdir)/auto.diod
> --      $(top_srcdir)/config/install-sh -m 755 $(srcdir)/diod.service \
> -+      $(top_srcdir)/config/install-sh -m 644 ./diod.service \
> -               $(DESTDIR)$(systemddir)/diod.service
> -
> - uninstall-local:
> -diff --git a/tests/kern/dbench/Makefile.am b/tests/kern/dbench/Makefile.am
> -index e0cdefa..1704f9f 100644
> ---- a/tests/kern/dbench/Makefile.am
> -+++ b/tests/kern/dbench/Makefile.am
> -@@ -2,7 +2,7 @@ AM_CFLAGS = -w
> -
> - # VERSION=4.00
> -
> --AM_CPPFLAGS = -DDATADIR=\"$(X_DATADIR)\"
> -+AM_CPPFLAGS = -I. -DDATADIR=\"$(X_DATADIR)\"
> -
> - LDADD = $(LIBPOPT)
> -
> -@@ -25,6 +25,6 @@ BUILT_SOURCES = proto.h
> - CLEANFILES = proto.h
> -
> - proto.h: $(dbench_SOURCES) mkproto.pl
> --      perl mkproto.pl $(dbench_SOURCES) > proto.h
> -+      $(srcdir)/mkproto.pl $(patsubst %,$(srcdir),$(dbench_SOURCES)) > proto.h
> -
> - EXTRA_DIST = mkproto.pl
> -diff --git a/utils/Makefile.am b/utils/Makefile.am
> -index 169b6c4..91d571d 100644
> ---- a/utils/Makefile.am
> -+++ b/utils/Makefile.am
> -@@ -1,7 +1,7 @@
> - AM_CFLAGS = @GCCWARN@
> -
> - AM_CPPFLAGS = \
> --      -I../libnpfs -I../liblsd -I../libdiod -I../libnpclient
> -+      -I$(srcdir)/../libnpfs -I$(srcdir)/../liblsd -I$(srcdir)/../libdiod -I$(srcdir)/../libnpclient
> -
> - sbin_PROGRAMS = diodmount diodcat dtop diodload diodls diodshowmount dioddate
> -
> ---
> -2.11.0
> -
> diff --git a/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch b/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
> deleted file mode 100644
> index 49bc507b..00000000
> --- a/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
> +++ /dev/null
> @@ -1,37 +0,0 @@
> -From f75836731d6051d85e6317eb4c12465bafb4973e Mon Sep 17 00:00:00 2001
> -From: "Hongzhi.Song" <hongzhi.song@windriver.com>
> -Date: Fri, 31 Aug 2018 05:47:02 -0400
> -Subject: [PATCH] diod/ops.c: add header file for makedev
> -
> -Error:
> -diod/ops.c:845: undefined reference to `makedev'
> -
> -Fixed:
> -Glibc removes sys/sysmacros.h which defines makedev from sys/types.h
> -since v2.28. [Commit ID: e16deca62e16f]
> -
> -And then glibc suggestions us to include <sys/sysmacros.h> directly if
> -code needs it.
> -
> -Upstream-Status: Submitted
> -
> -Signed-off-by: Hongzhi.Song <hongzhi.song@windriver.com>
> ----
> - diod/ops.c | 1 +
> - 1 file changed, 1 insertion(+)
> -
> -diff --git a/diod/ops.c b/diod/ops.c
> -index c6c8291..469a502 100644
> ---- a/diod/ops.c
> -+++ b/diod/ops.c
> -@@ -74,6 +74,7 @@
> - #include <pthread.h>
> - #include <errno.h>
> - #include <sys/types.h>
> -+#include <sys/sysmacros.h>
> -
> - #ifdef __FreeBSD__
> - #if !__BSD_VISIBLE
> ---
> -2.8.1
> -
> diff --git a/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch b/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
> deleted file mode 100644
> index 4931a19b..00000000
> --- a/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
> +++ /dev/null
> @@ -1,51 +0,0 @@
> -From 365159534b1981dfe291ecc05d2455f32652a166 Mon Sep 17 00:00:00 2001
> -From: Changqing Li <changqing.li@windriver.com>
> -Date: Mon, 26 Aug 2019 18:00:39 +0800
> -Subject: [PATCH] drod: add option to config systemddir
> -
> -Not need to send upstream, since upstream have refactor
> -related code, but not released. and this version release
> -4 years ago, not proper to just backport one or more
> -commits, this patch maybe could be dropped after upgrade.
> -
> -Upstream-Status: Inappropriate [embedded specific]
> -
> -Signed-off-by: Changqing Li <changqing.li@windriver.com>
> ----
> - configure.ac        | 9 +++++++++
> - scripts/Makefile.am | 2 +-
> - 2 files changed, 10 insertions(+), 1 deletion(-)
> -
> -diff --git a/configure.ac b/configure.ac
> -index 1cb8579..3e4a5ec 100644
> ---- a/configure.ac
> -+++ b/configure.ac
> -@@ -25,6 +25,15 @@ AC_DEFINE([_GNU_SOURCE], 1,
> -           [Define _GNU_SOURCE so that we get all necessary prototypes])
> -
> - ##
> -+#Add option to config systemddir
> -+##
> -+AC_ARG_WITH([systemddir],
> -+  [AS_HELP_STRING([--with-systemddir=DIR], [systemd dir])],
> -+  [systemddir=$withval],
> -+  [systemddir="/lib/systemd/system"])
> -+AC_SUBST([systemddir], [$systemddir])
> -+
> -+##
> - # Checks for programs
> - ##
> - AC_PROG_CC
> -diff --git a/scripts/Makefile.am b/scripts/Makefile.am
> -index 2aba728..40822b7 100644
> ---- a/scripts/Makefile.am
> -+++ b/scripts/Makefile.am
> -@@ -1,4 +1,4 @@
> --systemddir=/lib/systemd/system
> -+systemddir=@systemddir@
> -
> - install-data-local:
> -       $(top_srcdir)/config/install-sh -m 755 ./auto.diod \
> ---
> -2.7.4
> -
> diff --git a/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch b/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
> deleted file mode 100644
> index 13e0a86e..00000000
> --- a/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
> +++ /dev/null
> @@ -1,47 +0,0 @@
> -From 5a9e09dc5de833db11607530351cd87cecbfd17e Mon Sep 17 00:00:00 2001
> -From: Roy Li <rongqing.li@windriver.com>
> -Date: Thu, 22 Jun 2017 06:32:30 +0000
> -Subject: [PATCH 2/2] auto.diod.in: remove bashisms
> -
> -Upstream-Status: Inappropriate [embedded specific]
> -
> -Signed-off-by: Roy Li <rongqing.li@windriver.com>
> ----
> - scripts/auto.diod.in | 14 +++++++-------
> - 1 file changed, 7 insertions(+), 7 deletions(-)
> -
> -diff --git a/scripts/auto.diod.in b/scripts/auto.diod.in
> -index f63e004..4d5fc3a 100755
> ---- a/scripts/auto.diod.in
> -+++ b/scripts/auto.diod.in
> -@@ -1,4 +1,4 @@
> --#!/bin/bash
> -+#!/bin/sh
> - #
> - # auto.diod - executable automounter map for diod file systems
> - #
> -@@ -41,15 +41,15 @@ dcatopts="${DIOD_TIMEOUT:+-t $DIOD_TIMEOUT}"
> - for server in $DIOD_SERVERS; do
> -     $DIOD_DIODCAT -s $server $dcatopts exports | awk '{print $1}' |\
> -     while read path; do
> --        if [ "$path" == "/" ]; then
> --            if [ "$key" == "ROOT" ]; then
> -+        if [ "$path" = "/" ]; then
> -+            if [ "$key" = "ROOT" ]; then
> -                 echo "$prefix $server:$path"
> -                 exit 0
> -             fi
> --        elif [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//./g')" ] \
> --          || [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//_/g')" ] \
> --          || [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//-/g')" ] \
> --          || [ "$key" == "$(basename $path)" ]; then
> -+        elif [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//./g')" ] \
> -+          || [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//_/g')" ] \
> -+          || [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//-/g')" ] \
> -+          || [ "$key" = "$(basename $path)" ]; then
> -             echo "$prefix $server:$path"
> -             exit 0
> -         fi
> ---
> -2.11.0
> -
> --
> 2.39.2
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#8619): https://lists.yoctoproject.org/g/meta-virtualization/message/8619
> Mute This Topic: https://lists.yoctoproject.org/mt/104861806/1050810
> Group Owner: meta-virtualization+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [bruce.ashfield@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II


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

* Re: [meta-virtualization][PATCH] diod: Update to b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6
  2024-03-11 12:36 ` Bruce Ashfield
@ 2024-03-11 14:57   ` Ola x Nilsson
  0 siblings, 0 replies; 3+ messages in thread
From: Ola x Nilsson @ 2024-03-11 14:57 UTC (permalink / raw)
  To: Bruce Ashfield; +Cc: meta-virtualization




On Mon, Mar 11 2024, Bruce Ashfield wrote:

> On Mon, Mar 11, 2024 at 8:24 AM Ola x Nilsson <ola.x.nilsson@axis.com> wrote:
>>
>> Just needs a single patch to handle the lua feature in configure when
>> cross-compiling.
>>
>
> This needs a listing of the changes between the two SRCREVs.

I've sent a v2.

> I haven't gotten around to updating diod for the release, so I'll consider
> this once ceph is working again, but for now, it will probably wait until
> my round of package bumps post release.

That's fine with me.

/Ola


>
>> Signed-off-by: Ola x Nilsson <olani@axis.com>
>> ---
>>  recipes-extended/diod/diod_1.0.24.bb          |  13 +-
>>  .../0001-build-Find-lua-with-pkg-config.patch |  50 +++++++
>>  ...ilds-to-work-with-separate-build-dir.patch | 126 ------------------
>>  ...od-ops.c-add-header-file-for-makedev.patch |  37 -----
>>  ...drod-add-option-to-config-systemddir.patch |  51 -------
>>  .../0002-auto.diod.in-remove-bashisms.patch   |  47 -------
>>  6 files changed, 55 insertions(+), 269 deletions(-)
>>  create mode 100644 recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
>>  delete mode 100644 recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
>>  delete mode 100644 recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
>>  delete mode 100644 recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
>>  delete mode 100644 recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
>>
>> diff --git a/recipes-extended/diod/diod_1.0.24.bb b/recipes-extended/diod/diod_1.0.24.bb
>> index 2b87a8ae..5e191dc0 100644
>> --- a/recipes-extended/diod/diod_1.0.24.bb
>> +++ b/recipes-extended/diod/diod_1.0.24.bb
>> @@ -9,22 +9,19 @@ LICENSE = "GPL-2.0-only"
>>  LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552"
>>
>>  PV = "1.0.24+git"
>> -SRCREV = "0ea3fe3d829b5085307cd27a512708d99ef48199"
>> +SRCREV = "b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6"
>>  SRC_URI = "git://github.com/chaos/diod.git;protocol=https;branch=master \
>>             file://diod \
>>             file://diod.conf \
>> -           file://0001-build-allow-builds-to-work-with-separate-build-dir.patch \
>> -           file://0002-auto.diod.in-remove-bashisms.patch \
>> -           file://0001-diod-ops.c-add-header-file-for-makedev.patch \
>> -           file://0001-drod-add-option-to-config-systemddir.patch \
>> -          "
>> +           file://0001-build-Find-lua-with-pkg-config.patch \
>> +           "
>>  DEPENDS = "libcap ncurses tcp-wrappers lua"
>>
>> -EXTRA_OECONF = "--with-systemddir=${systemd_unitdir}/system"
>> +EXTRA_OEMAKE += "systemddir=${systemd_unitdir}/system"
>>
>>  S = "${WORKDIR}/git"
>>
>> -inherit autotools systemd
>> +inherit autotools pkgconfig systemd
>>
>>  do_install:append () {
>>          # install our init based on start-stop-daemon
>> diff --git a/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch b/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
>> new file mode 100644
>> index 00000000..9f61792a
>> --- /dev/null
>> +++ b/recipes-extended/diod/files/0001-build-Find-lua-with-pkg-config.patch
>> @@ -0,0 +1,50 @@
>> +From 8124eec4e59222f1c3f3dc4d7e6706e13c33755c Mon Sep 17 00:00:00 2001
>> +From: Ola x Nilsson <olani@axis.com>
>> +Date: Wed, 6 Mar 2024 18:40:16 +0100
>> +Subject: [PATCH] build: Find lua with pkg-config
>> +
>> +ax_lua.m4 does not support cross compilation.
>> +
>> +Upstream-Status: Inappropriate (Embedded specific)
>> +
>> +---
>> + configure.ac | 26 +++++++++++++++++++++++++-
>> + 1 file changed, 25 insertions(+), 1 deletion(-)
>> +
>> +diff --git a/configure.ac b/configure.ac
>> +index e8fcdb2..f3ebdc5 100644
>> +--- a/configure.ac
>> ++++ b/configure.ac
>> +@@ -121,7 +121,31 @@ AC_ARG_ENABLE([config],
>> +   [AS_HELP_STRING([--disable-config], [disable lua config file support])])
>> +
>> + AS_IF([test "x$enable_config" != "xno"], [
>> +-  AX_PROG_LUA([5.1],[5.5])
>> ++  dnl Use pkg_config to check for lua settings.  This is not standard
>> ++  dnl as lua typically does not distribute a pc file.  The
>> ++  dnl OpenEmbedded recipe adds one that can be used.  Reuse/replicate
>> ++  dnl as much of AX_PROG_LUA as possible so AX_LUA_HEADERS and
>> ++  dnl AX_LUA_LIBS are still usable.
>> ++  PKG_CHECK_EXISTS([lua], [
>> ++    AS_IF([
>> ++      $PKG_CONFIG --atleast-version=5.1 lua && $PKG_CONFIG --max-version=5.5 lua
>> ++    ],[
>> ++      AC_ARG_VAR([LUA], [The LUA interpreter, e.g. /usr/bin/lua5.1])
>> ++        dnl should really check that the file exists ...
>> ++      AC_SUBST([LUA], [${bindir}/lua])
>> ++        AC_CACHE_CHECK([for $LUA version], [ax_cv_lua_version], [
>> ++          ax_cv_lua_version=[`$PKG_CONFIG --modversion lua | $SED 's/^\([0-9]\+\.[0-9]\+\).*/\1/'`]
>> ++      ])
>> ++        AS_IF([test "x$ax_cv_lua_version" = 'x'],
>> ++       [AC_MSG_ERROR([invalid Lua version number])])
>> ++      AC_SUBST([LUA_VERSION], [$ax_cv_lua_version])
>> ++      AC_SUBST([LUA_SHORT_VERSION], [`echo "$LUA_VERSION" | $SED 's|\.||'`])
>> ++    ],[
>> ++        AC_MSG_ERROR([Incorrect lua version])
>> ++      ])
>> ++  ],[
>> ++    AX_PROG_LUA([5.1],[5.5])
>> ++  ])
>> +   AX_LUA_HEADERS([],[
>> +     AC_MSG_ERROR([Install lua headers or configure with --disable-config])
>> +   ])
>> diff --git a/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch b/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
>> deleted file mode 100644
>> index f40e0eb5..00000000
>> --- a/recipes-extended/diod/files/0001-build-allow-builds-to-work-with-separate-build-dir.patch
>> +++ /dev/null
>> @@ -1,126 +0,0 @@
>> -From 43403468298ef4167baa5d84de2ee2eaf7f4007a Mon Sep 17 00:00:00 2001
>> -From: Mark Asselstine <mark.asselstine@windriver.com>
>> -Date: Fri, 9 Jan 2015 18:34:04 +0000
>> -Subject: [PATCH 1/2] build: allow builds to work with separate build dir
>> -
>> -Remove assumptions in include paths that assume the build
>> -is happening in the source directories.
>> -
>> -Upstream-Status: Inappropriate [embedded specific]
>> -
>> -Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
>> ----
>> - diod/Makefile.am              | 2 +-
>> - libdiod/Makefile.am           | 2 +-
>> - liblsd/Makefile.am            | 2 +-
>> - libnpclient/Makefile.am       | 2 +-
>> - scripts/Makefile.am           | 6 +++---
>> - tests/kern/dbench/Makefile.am | 4 ++--
>> - utils/Makefile.am             | 2 +-
>> - 7 files changed, 10 insertions(+), 10 deletions(-)
>> -
>> -diff --git a/diod/Makefile.am b/diod/Makefile.am
>> -index 7644be7..fdaf130 100644
>> ---- a/diod/Makefile.am
>> -+++ b/diod/Makefile.am
>> -@@ -1,7 +1,7 @@
>> - AM_CFLAGS = @GCCWARN@
>> -
>> - AM_CPPFLAGS = \
>> --      -I../libnpfs -I../liblsd -I../libdiod
>> -+      -I$(srcdir)/../libnpfs -I$(srcdir)/../liblsd -I$(srcdir)/../libdiod
>> -
>> - sbin_PROGRAMS = diod
>> -
>> -diff --git a/libdiod/Makefile.am b/libdiod/Makefile.am
>> -index 4810e14..6905cdd 100644
>> ---- a/libdiod/Makefile.am
>> -+++ b/libdiod/Makefile.am
>> -@@ -1,7 +1,7 @@
>> - AM_CFLAGS = @GCCWARN@
>> -
>> - AM_CPPFLAGS = \
>> --      -I../libnpfs -I../libnpclient -I../liblsd
>> -+      -I$(srcdir)/../libnpfs -I$(srcdir)/../libnpclient -I$(srcdir)/../liblsd
>> -
>> - noinst_LIBRARIES = libdiod.a
>> -
>> -diff --git a/liblsd/Makefile.am b/liblsd/Makefile.am
>> -index c3e5658..7e18bf8 100644
>> ---- a/liblsd/Makefile.am
>> -+++ b/liblsd/Makefile.am
>> -@@ -1,6 +1,6 @@
>> - AM_CFLAGS = @GCCWARN@
>> -
>> --AM_CPPFLAGS = -I../libdiod
>> -+AM_CPPFLAGS = -I$(srcdir)/../libdiod
>> -
>> - noinst_LIBRARIES = liblsd.a
>> -
>> -diff --git a/libnpclient/Makefile.am b/libnpclient/Makefile.am
>> -index 5305df9..cbaf266 100644
>> ---- a/libnpclient/Makefile.am
>> -+++ b/libnpclient/Makefile.am
>> -@@ -1,7 +1,7 @@
>> - AM_CFLAGS = @GCCWARN@
>> -
>> - AM_CPPFLAGS = \
>> --      -I../libnpfs
>> -+      -I$(srcdir)/../libnpfs
>> -
>> - noinst_LIBRARIES = libnpclient.a
>> -
>> -diff --git a/scripts/Makefile.am b/scripts/Makefile.am
>> -index 51c24a3..2aba728 100644
>> ---- a/scripts/Makefile.am
>> -+++ b/scripts/Makefile.am
>> -@@ -1,9 +1,9 @@
>> --systemddir=$(sysconfdir)/systemd/system
>> -+systemddir=/lib/systemd/system
>> -
>> - install-data-local:
>> --      $(top_srcdir)/config/install-sh -m 755 $(srcdir)/auto.diod \
>> -+      $(top_srcdir)/config/install-sh -m 755 ./auto.diod \
>> -               $(DESTDIR)$(sysconfdir)/auto.diod
>> --      $(top_srcdir)/config/install-sh -m 755 $(srcdir)/diod.service \
>> -+      $(top_srcdir)/config/install-sh -m 644 ./diod.service \
>> -               $(DESTDIR)$(systemddir)/diod.service
>> -
>> - uninstall-local:
>> -diff --git a/tests/kern/dbench/Makefile.am b/tests/kern/dbench/Makefile.am
>> -index e0cdefa..1704f9f 100644
>> ---- a/tests/kern/dbench/Makefile.am
>> -+++ b/tests/kern/dbench/Makefile.am
>> -@@ -2,7 +2,7 @@ AM_CFLAGS = -w
>> -
>> - # VERSION=4.00
>> -
>> --AM_CPPFLAGS = -DDATADIR=\"$(X_DATADIR)\"
>> -+AM_CPPFLAGS = -I. -DDATADIR=\"$(X_DATADIR)\"
>> -
>> - LDADD = $(LIBPOPT)
>> -
>> -@@ -25,6 +25,6 @@ BUILT_SOURCES = proto.h
>> - CLEANFILES = proto.h
>> -
>> - proto.h: $(dbench_SOURCES) mkproto.pl
>> --      perl mkproto.pl $(dbench_SOURCES) > proto.h
>> -+      $(srcdir)/mkproto.pl $(patsubst %,$(srcdir),$(dbench_SOURCES)) > proto.h
>> -
>> - EXTRA_DIST = mkproto.pl
>> -diff --git a/utils/Makefile.am b/utils/Makefile.am
>> -index 169b6c4..91d571d 100644
>> ---- a/utils/Makefile.am
>> -+++ b/utils/Makefile.am
>> -@@ -1,7 +1,7 @@
>> - AM_CFLAGS = @GCCWARN@
>> -
>> - AM_CPPFLAGS = \
>> --      -I../libnpfs -I../liblsd -I../libdiod -I../libnpclient
>> -+      -I$(srcdir)/../libnpfs -I$(srcdir)/../liblsd -I$(srcdir)/../libdiod -I$(srcdir)/../libnpclient
>> -
>> - sbin_PROGRAMS = diodmount diodcat dtop diodload diodls diodshowmount dioddate
>> -
>> ---
>> -2.11.0
>> -
>> diff --git a/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch b/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
>> deleted file mode 100644
>> index 49bc507b..00000000
>> --- a/recipes-extended/diod/files/0001-diod-ops.c-add-header-file-for-makedev.patch
>> +++ /dev/null
>> @@ -1,37 +0,0 @@
>> -From f75836731d6051d85e6317eb4c12465bafb4973e Mon Sep 17 00:00:00 2001
>> -From: "Hongzhi.Song" <hongzhi.song@windriver.com>
>> -Date: Fri, 31 Aug 2018 05:47:02 -0400
>> -Subject: [PATCH] diod/ops.c: add header file for makedev
>> -
>> -Error:
>> -diod/ops.c:845: undefined reference to `makedev'
>> -
>> -Fixed:
>> -Glibc removes sys/sysmacros.h which defines makedev from sys/types.h
>> -since v2.28. [Commit ID: e16deca62e16f]
>> -
>> -And then glibc suggestions us to include <sys/sysmacros.h> directly if
>> -code needs it.
>> -
>> -Upstream-Status: Submitted
>> -
>> -Signed-off-by: Hongzhi.Song <hongzhi.song@windriver.com>
>> ----
>> - diod/ops.c | 1 +
>> - 1 file changed, 1 insertion(+)
>> -
>> -diff --git a/diod/ops.c b/diod/ops.c
>> -index c6c8291..469a502 100644
>> ---- a/diod/ops.c
>> -+++ b/diod/ops.c
>> -@@ -74,6 +74,7 @@
>> - #include <pthread.h>
>> - #include <errno.h>
>> - #include <sys/types.h>
>> -+#include <sys/sysmacros.h>
>> -
>> - #ifdef __FreeBSD__
>> - #if !__BSD_VISIBLE
>> ---
>> -2.8.1
>> -
>> diff --git a/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch b/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
>> deleted file mode 100644
>> index 4931a19b..00000000
>> --- a/recipes-extended/diod/files/0001-drod-add-option-to-config-systemddir.patch
>> +++ /dev/null
>> @@ -1,51 +0,0 @@
>> -From 365159534b1981dfe291ecc05d2455f32652a166 Mon Sep 17 00:00:00 2001
>> -From: Changqing Li <changqing.li@windriver.com>
>> -Date: Mon, 26 Aug 2019 18:00:39 +0800
>> -Subject: [PATCH] drod: add option to config systemddir
>> -
>> -Not need to send upstream, since upstream have refactor
>> -related code, but not released. and this version release
>> -4 years ago, not proper to just backport one or more
>> -commits, this patch maybe could be dropped after upgrade.
>> -
>> -Upstream-Status: Inappropriate [embedded specific]
>> -
>> -Signed-off-by: Changqing Li <changqing.li@windriver.com>
>> ----
>> - configure.ac        | 9 +++++++++
>> - scripts/Makefile.am | 2 +-
>> - 2 files changed, 10 insertions(+), 1 deletion(-)
>> -
>> -diff --git a/configure.ac b/configure.ac
>> -index 1cb8579..3e4a5ec 100644
>> ---- a/configure.ac
>> -+++ b/configure.ac
>> -@@ -25,6 +25,15 @@ AC_DEFINE([_GNU_SOURCE], 1,
>> -           [Define _GNU_SOURCE so that we get all necessary prototypes])
>> -
>> - ##
>> -+#Add option to config systemddir
>> -+##
>> -+AC_ARG_WITH([systemddir],
>> -+  [AS_HELP_STRING([--with-systemddir=DIR], [systemd dir])],
>> -+  [systemddir=$withval],
>> -+  [systemddir="/lib/systemd/system"])
>> -+AC_SUBST([systemddir], [$systemddir])
>> -+
>> -+##
>> - # Checks for programs
>> - ##
>> - AC_PROG_CC
>> -diff --git a/scripts/Makefile.am b/scripts/Makefile.am
>> -index 2aba728..40822b7 100644
>> ---- a/scripts/Makefile.am
>> -+++ b/scripts/Makefile.am
>> -@@ -1,4 +1,4 @@
>> --systemddir=/lib/systemd/system
>> -+systemddir=@systemddir@
>> -
>> - install-data-local:
>> -       $(top_srcdir)/config/install-sh -m 755 ./auto.diod \
>> ---
>> -2.7.4
>> -
>> diff --git a/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch b/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
>> deleted file mode 100644
>> index 13e0a86e..00000000
>> --- a/recipes-extended/diod/files/0002-auto.diod.in-remove-bashisms.patch
>> +++ /dev/null
>> @@ -1,47 +0,0 @@
>> -From 5a9e09dc5de833db11607530351cd87cecbfd17e Mon Sep 17 00:00:00 2001
>> -From: Roy Li <rongqing.li@windriver.com>
>> -Date: Thu, 22 Jun 2017 06:32:30 +0000
>> -Subject: [PATCH 2/2] auto.diod.in: remove bashisms
>> -
>> -Upstream-Status: Inappropriate [embedded specific]
>> -
>> -Signed-off-by: Roy Li <rongqing.li@windriver.com>
>> ----
>> - scripts/auto.diod.in | 14 +++++++-------
>> - 1 file changed, 7 insertions(+), 7 deletions(-)
>> -
>> -diff --git a/scripts/auto.diod.in b/scripts/auto.diod.in
>> -index f63e004..4d5fc3a 100755
>> ---- a/scripts/auto.diod.in
>> -+++ b/scripts/auto.diod.in
>> -@@ -1,4 +1,4 @@
>> --#!/bin/bash
>> -+#!/bin/sh
>> - #
>> - # auto.diod - executable automounter map for diod file systems
>> - #
>> -@@ -41,15 +41,15 @@ dcatopts="${DIOD_TIMEOUT:+-t $DIOD_TIMEOUT}"
>> - for server in $DIOD_SERVERS; do
>> -     $DIOD_DIODCAT -s $server $dcatopts exports | awk '{print $1}' |\
>> -     while read path; do
>> --        if [ "$path" == "/" ]; then
>> --            if [ "$key" == "ROOT" ]; then
>> -+        if [ "$path" = "/" ]; then
>> -+            if [ "$key" = "ROOT" ]; then
>> -                 echo "$prefix $server:$path"
>> -                 exit 0
>> -             fi
>> --        elif [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//./g')" ] \
>> --          || [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//_/g')" ] \
>> --          || [ "$key" == "$(echo $path|sed -e's/^\///' -e's/\//-/g')" ] \
>> --          || [ "$key" == "$(basename $path)" ]; then
>> -+        elif [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//./g')" ] \
>> -+          || [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//_/g')" ] \
>> -+          || [ "$key" = "$(echo $path|sed -e's/^\///' -e's/\//-/g')" ] \
>> -+          || [ "$key" = "$(basename $path)" ]; then
>> -             echo "$prefix $server:$path"
>> -             exit 0
>> -         fi
>> ---
>> -2.11.0
>> -
>> --
>> 2.39.2
>>
>>
>> 
>>


-- 
Ola x Nilsson


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

end of thread, other threads:[~2024-03-11 14:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-11 12:24 [meta-virtualization][PATCH] diod: Update to b4b5e8e00ed11b21d7fcf05a080dc054a8eac2d6 Ola x Nilsson
2024-03-11 12:36 ` Bruce Ashfield
2024-03-11 14:57   ` Ola x Nilsson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.