[dunfell] cifs-utils: add patch for upstream build bug

Message ID 20220606183317.706-1-danielrparks@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [dunfell] cifs-utils: add patch for upstream build bug | expand

Commit Message

Daniel Parks June 6, 2022, 6:33 p.m. UTC
cifs-utils upstream has a bug that causes our build to fail when
PARALLEL_MAKE is set to "-j1". This adds an overlay with the patch for
the bug, which I have also submitted upstream.

It only changes the order of the build process, so rebuilding already
built packages is not necessary.

Signed-off-by: Daniel Parks <danielrparks@ti.com>
---
 ...automake-treat-sbin-as-exec-not-data.patch | 34 +++++++++++++++++++
 .../cifs/cifs-utils_6.10.bbappend             |  4 +++
 2 files changed, 38 insertions(+)
 create mode 100644 meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch
 create mode 100644 meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend

Comments

Denys Dmytriyenko June 6, 2022, 7:44 p.m. UTC | #1
On Mon, Jun 06, 2022 at 01:33:17PM -0500, Daniel Parks via lists.yoctoproject.org wrote:
> cifs-utils upstream has a bug that causes our build to fail when
> PARALLEL_MAKE is set to "-j1". This adds an overlay with the patch for
> the bug, which I have also submitted upstream.
> 
> It only changes the order of the build process, so rebuilding already
> built packages is not necessary.
> 
> Signed-off-by: Daniel Parks <danielrparks@ti.com>
> ---
>  ...automake-treat-sbin-as-exec-not-data.patch | 34 +++++++++++++++++++
>  .../cifs/cifs-utils_6.10.bbappend             |  4 +++
>  2 files changed, 38 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch
>  create mode 100644 meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend
> 
> diff --git a/meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch b/meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch
> new file mode 100644
> index 00000000..5a7bdd20
> --- /dev/null
> +++ b/meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch
> @@ -0,0 +1,34 @@
> +From fd5f395514256572c451127006a68483106cf2b8 Mon Sep 17 00:00:00 2001
> +From: Daniel Parks <danielrparks@ti.com>
> +Date: Fri, 3 Jun 2022 15:34:59 -0500
> +Subject: [PATCH] cifs-utils: Make automake treat /sbin as exec, not data
> +
> +Otherwise, $(DESTDIR)/sbin doesn't get created until install-data on a
> +-j1 build and install-exec-hook can fail because it might not exist.
> +
> +Steps to reproduce this bug:
> +$ autoreconf -i
> +$ ./configure
> +$ mkdir image
> +$ make DESTDIR=image install -j1

Would be nice to have an Upstream-Status: tag here.


> +Signed-off-by: Daniel Parks <danielrparks@ti.com>
> +---
> + Makefile.am | 4 ++--
> + 1 file changed, 2 insertions(+), 2 deletions(-)
> +
> +diff --git a/Makefile.am b/Makefile.am
> +index e9be6b3..edb1b7f 100644
> +--- a/Makefile.am
> ++++ b/Makefile.am
> +@@ -1,8 +1,8 @@
> + AM_CFLAGS = -Wall -Wextra -D_FORTIFY_SOURCE=2 $(PIE_CFLAGS) $(RELRO_CFLAGS)
> + ACLOCAL_AMFLAGS = -I aclocal
> + 
> +-root_sbindir = $(ROOTSBINDIR)
> +-root_sbin_PROGRAMS = mount.cifs
> ++root_exec_sbindir = $(ROOTSBINDIR)
> ++root_exec_sbin_PROGRAMS = mount.cifs
> + mount_cifs_SOURCES = mount.cifs.c mtab.c resolve_host.c util.c
> + mount_cifs_LDADD = $(LIBCAP) $(CAPNG_LDADD) $(RT_LDADD)
> + include_HEADERS = cifsidmap.h
> diff --git a/meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend b/meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend
> new file mode 100644
> index 00000000..33030b87
> --- /dev/null
> +++ b/meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend
> @@ -0,0 +1,4 @@
> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> +
> +SRC_URI += "file://0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch"
> +
> -- 
> 2.17.1

Patch

diff --git a/meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch b/meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch
new file mode 100644
index 00000000..5a7bdd20
--- /dev/null
+++ b/meta-arago-distro/recipes-support/cifs/cifs-utils/0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch
@@ -0,0 +1,34 @@ 
+From fd5f395514256572c451127006a68483106cf2b8 Mon Sep 17 00:00:00 2001
+From: Daniel Parks <danielrparks@ti.com>
+Date: Fri, 3 Jun 2022 15:34:59 -0500
+Subject: [PATCH] cifs-utils: Make automake treat /sbin as exec, not data
+
+Otherwise, $(DESTDIR)/sbin doesn't get created until install-data on a
+-j1 build and install-exec-hook can fail because it might not exist.
+
+Steps to reproduce this bug:
+$ autoreconf -i
+$ ./configure
+$ mkdir image
+$ make DESTDIR=image install -j1
+
+Signed-off-by: Daniel Parks <danielrparks@ti.com>
+---
+ Makefile.am | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/Makefile.am b/Makefile.am
+index e9be6b3..edb1b7f 100644
+--- a/Makefile.am
++++ b/Makefile.am
+@@ -1,8 +1,8 @@
+ AM_CFLAGS = -Wall -Wextra -D_FORTIFY_SOURCE=2 $(PIE_CFLAGS) $(RELRO_CFLAGS)
+ ACLOCAL_AMFLAGS = -I aclocal
+ 
+-root_sbindir = $(ROOTSBINDIR)
+-root_sbin_PROGRAMS = mount.cifs
++root_exec_sbindir = $(ROOTSBINDIR)
++root_exec_sbin_PROGRAMS = mount.cifs
+ mount_cifs_SOURCES = mount.cifs.c mtab.c resolve_host.c util.c
+ mount_cifs_LDADD = $(LIBCAP) $(CAPNG_LDADD) $(RT_LDADD)
+ include_HEADERS = cifsidmap.h
diff --git a/meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend b/meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend
new file mode 100644
index 00000000..33030b87
--- /dev/null
+++ b/meta-arago-distro/recipes-support/cifs/cifs-utils_6.10.bbappend
@@ -0,0 +1,4 @@ 
+FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
+
+SRC_URI += "file://0001-cifs-utils-Make-automake-treat-sbin-as-exec-not-data.patch"
+