mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Marc Kleine-Budde <mkl@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: mol@pengutronix.de, Marc Kleine-Budde <mkl@pengutronix.de>
Subject: [ptxdist] [PATCH v2] attr/acl: fix installation when user or group name contains a space
Date: Thu, 15 Aug 2019 14:50:42 +0200	[thread overview]
Message-ID: <20190815125042.29479-1-mkl@pengutronix.de> (raw)

Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
---
 ...install-with-domain-user-or-domain-u.patch | 229 ++++++++++++++++++
 ...dmacros-fix-install-with-user-domain.patch |  27 ---
 patches/acl-2.2.52/series                     |   4 +-
 ...tall-with-domain-user-or-domain-user.patch | 229 ++++++++++++++++++
 .../0001-fix-install-with-domain-user.patch   |  27 ---
 ...nslations-only-if-gettext-is-enabled.patch |   4 -
 patches/attr-2.4.47/series                    |   5 +-
 7 files changed, 464 insertions(+), 61 deletions(-)
 create mode 100644 patches/acl-2.2.52/0003-buildmacros-fix-install-with-domain-user-or-domain-u.patch
 delete mode 100644 patches/acl-2.2.52/0003-buildmacros-fix-install-with-user-domain.patch
 create mode 100644 patches/attr-2.4.47/0001-fix-install-with-domain-user-or-domain-user.patch
 delete mode 100644 patches/attr-2.4.47/0001-fix-install-with-domain-user.patch

diff --git a/patches/acl-2.2.52/0003-buildmacros-fix-install-with-domain-user-or-domain-u.patch b/patches/acl-2.2.52/0003-buildmacros-fix-install-with-domain-user-or-domain-u.patch
new file mode 100644
index 000000000000..cec5eb2f62f2
--- /dev/null
+++ b/patches/acl-2.2.52/0003-buildmacros-fix-install-with-domain-user-or-domain-u.patch
@@ -0,0 +1,229 @@
+From: Jon Ringle <jon@ringle.org>
+Date: Mon, 23 May 2011 10:30:59 -0400
+Subject: [PATCH] buildmacros: fix install with domain\user or "domain user"
+
+If current user is domain\user the \ gets interpreted as an escape char
+resulting in user not found. This also breaks if the user or group contains a
+space.
+
+Enclose "$(PKG_USER)" and "$(PKG_GROUP)" in quotes to fix.
+
+Signed-off-by: Jon Ringle <jon@ringle.org>
+Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
+---
+ include/buildmacros |  2 +-
+ include/install-sh  | 26 +++++++++++++-------------
+ install-sh          | 26 +++++++++++++-------------
+ 3 files changed, 27 insertions(+), 27 deletions(-)
+
+diff --git a/include/buildmacros b/include/buildmacros
+index 805ef64d8d57..29e937e077c8 100644
+--- a/include/buildmacros
++++ b/include/buildmacros
+@@ -40,7 +40,7 @@ OBJECTS = $(ASFILES:.s=.o) \
+ 	  $(LFILES:.l=.o) \
+ 	  $(YFILES:%.y=%.tab.o)
+ 
+-INSTALL	= $(TOPDIR)/include/install-sh -o $(PKG_USER) -g $(PKG_GROUP)
++INSTALL	= $(TOPDIR)/include/install-sh -o "$(PKG_USER)" -g "$(PKG_GROUP)"
+ 
+ IMAGES_DIR = $(TOPDIR)/all-images
+ DIST_DIR = $(TOPDIR)/dist
+diff --git a/include/install-sh b/include/install-sh
+index 3c9d49be8c68..5fdeb525aab0 100755
+--- a/include/install-sh
++++ b/include/install-sh
+@@ -56,7 +56,7 @@ _chown ()
+ {
+     _st=255
+     if [ $# -eq 3 ] ; then
+-	chown $1:$2 $3
++	chown "$1":"$2" $3
+ 	_st=$?
+ 	if [ $_st -ne 0 ] ; then
+ 	    if [ $REAL_UID != '0' ] ; then
+@@ -115,7 +115,7 @@ fi
+ 
+ [ -n "$DIST_ROOT" -a $REAL_UID -ne 0 ] && CHOWN=true
+ 
+-while getopts "Dcm:d:S:o:g:T:" c $*
++while getopts "Dcm:d:S:o:g:T:" c "$@"
+ do
+    case $c in
+    c)
+@@ -168,10 +168,10 @@ then
+     fi
+     if [ $status -eq 0 ]
+     then
+-	$CHOWN $OWNER $GROUP $dir
++	$CHOWN "$OWNER" "$GROUP" $dir
+ 	status=$?
+     fi
+-    $MANIFEST d $DIRMODE $OWNER $GROUP ${dir#$DIST_ROOT}
++    $MANIFEST d $DIRMODE "$OWNER" "$GROUP" ${dir#$DIST_ROOT}
+ elif $Sflag
+ then
+     #
+@@ -217,7 +217,7 @@ then
+ 			install_name=$target/$solib
+ 			$CP $solib $install_name
+ 			status=$?
+-			$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$solib ${install_name#$DIST_ROOT}
++			$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$solib ${install_name#$DIST_ROOT}
+ 			break
+ 		fi
+ 	done
+@@ -268,7 +268,7 @@ then
+ 	install_name=$target/$old_library
+ 	$CP $old_library $install_name
+ 	status=$?
+-	$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$old_library ${install_name#$DIST_ROOT}
++	$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$old_library ${install_name#$DIST_ROOT}
+ 	;;
+     *)
+ 	echo "$prog: -T $lt_install invalid"
+@@ -281,7 +281,7 @@ then
+ 	if [ $status -eq 0 ]
+ 	then
+ 		$CHMOD $FILEMODE $install_name
+-		$CHOWN $OWNER $GROUP $install_name
++		$CHOWN "$OWNER" "$GROUP" $install_name
+ 	fi
+ 	;;
+     esac
+@@ -310,19 +310,19 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    else
+ 		$CHMOD $FILEMODE $dir
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir
++		    $CHOWN "$OWNER" "$GROUP" $dir
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$dir ${dir#$DIST_ROOT}
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$dir ${dir#$DIST_ROOT}
+ 	    fi
+ 	fi
+     else
+@@ -352,10 +352,10 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    fi
+ 	    [ $status -ne 0 ] && break
+ 	done
+diff --git a/install-sh b/install-sh
+index 3c9d49be8c68..5fdeb525aab0 100755
+--- a/install-sh
++++ b/install-sh
+@@ -56,7 +56,7 @@ _chown ()
+ {
+     _st=255
+     if [ $# -eq 3 ] ; then
+-	chown $1:$2 $3
++	chown "$1":"$2" $3
+ 	_st=$?
+ 	if [ $_st -ne 0 ] ; then
+ 	    if [ $REAL_UID != '0' ] ; then
+@@ -115,7 +115,7 @@ fi
+ 
+ [ -n "$DIST_ROOT" -a $REAL_UID -ne 0 ] && CHOWN=true
+ 
+-while getopts "Dcm:d:S:o:g:T:" c $*
++while getopts "Dcm:d:S:o:g:T:" c "$@"
+ do
+    case $c in
+    c)
+@@ -168,10 +168,10 @@ then
+     fi
+     if [ $status -eq 0 ]
+     then
+-	$CHOWN $OWNER $GROUP $dir
++	$CHOWN "$OWNER" "$GROUP" $dir
+ 	status=$?
+     fi
+-    $MANIFEST d $DIRMODE $OWNER $GROUP ${dir#$DIST_ROOT}
++    $MANIFEST d $DIRMODE "$OWNER" "$GROUP" ${dir#$DIST_ROOT}
+ elif $Sflag
+ then
+     #
+@@ -217,7 +217,7 @@ then
+ 			install_name=$target/$solib
+ 			$CP $solib $install_name
+ 			status=$?
+-			$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$solib ${install_name#$DIST_ROOT}
++			$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$solib ${install_name#$DIST_ROOT}
+ 			break
+ 		fi
+ 	done
+@@ -268,7 +268,7 @@ then
+ 	install_name=$target/$old_library
+ 	$CP $old_library $install_name
+ 	status=$?
+-	$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$old_library ${install_name#$DIST_ROOT}
++	$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$old_library ${install_name#$DIST_ROOT}
+ 	;;
+     *)
+ 	echo "$prog: -T $lt_install invalid"
+@@ -281,7 +281,7 @@ then
+ 	if [ $status -eq 0 ]
+ 	then
+ 		$CHMOD $FILEMODE $install_name
+-		$CHOWN $OWNER $GROUP $install_name
++		$CHOWN "$OWNER" "$GROUP" $install_name
+ 	fi
+ 	;;
+     esac
+@@ -310,19 +310,19 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    else
+ 		$CHMOD $FILEMODE $dir
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir
++		    $CHOWN "$OWNER" "$GROUP" $dir
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$dir ${dir#$DIST_ROOT}
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$dir ${dir#$DIST_ROOT}
+ 	    fi
+ 	fi
+     else
+@@ -352,10 +352,10 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    fi
+ 	    [ $status -ne 0 ] && break
+ 	done
diff --git a/patches/acl-2.2.52/0003-buildmacros-fix-install-with-user-domain.patch b/patches/acl-2.2.52/0003-buildmacros-fix-install-with-user-domain.patch
deleted file mode 100644
index dfa3c9593daf..000000000000
--- a/patches/acl-2.2.52/0003-buildmacros-fix-install-with-user-domain.patch
+++ /dev/null
@@ -1,27 +0,0 @@
-From: Jon Ringle <jon@ringle.org>
-Date: Mon, 23 May 2011 10:30:59 -0400
-Subject: [PATCH] buildmacros: fix install with user\domain
-
-If current user is domain\user the \ gets interpreted as an escape char
-resulting in user not found.
-
-Enclose "$(PKG_USER)" and "$(PKG_GROUP)" in quotes to fix.
-
-Signed-off-by: Jon Ringle <jon@ringle.org>
----
- include/buildmacros | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/include/buildmacros b/include/buildmacros
-index 805ef64d8d57..29e937e077c8 100644
---- a/include/buildmacros
-+++ b/include/buildmacros
-@@ -40,7 +40,7 @@ OBJECTS = $(ASFILES:.s=.o) \
- 	  $(LFILES:.l=.o) \
- 	  $(YFILES:%.y=%.tab.o)
- 
--INSTALL	= $(TOPDIR)/include/install-sh -o $(PKG_USER) -g $(PKG_GROUP)
-+INSTALL	= $(TOPDIR)/include/install-sh -o "$(PKG_USER)" -g "$(PKG_GROUP)"
- 
- IMAGES_DIR = $(TOPDIR)/all-images
- DIST_DIR = $(TOPDIR)/dist
diff --git a/patches/acl-2.2.52/series b/patches/acl-2.2.52/series
index d073142537ea..6816cae02ebd 100644
--- a/patches/acl-2.2.52/series
+++ b/patches/acl-2.2.52/series
@@ -2,5 +2,5 @@
 #tag:base --start-number 1
 0001-builddefs-save-C-and-CPPFLAGS-from-configure-and-use.patch
 0002-buildmacros-add-tag-CC-to-libtool-invocation.patch
-0003-buildmacros-fix-install-with-user-domain.patch
-# 7f7779678dde859863b19dcae5e93fb1  - git-ptx-patches magic
+0003-buildmacros-fix-install-with-domain-user-or-domain-u.patch
+# 842c7b9ceac477f4c23e59ce8ade0f20  - git-ptx-patches magic
diff --git a/patches/attr-2.4.47/0001-fix-install-with-domain-user-or-domain-user.patch b/patches/attr-2.4.47/0001-fix-install-with-domain-user-or-domain-user.patch
new file mode 100644
index 000000000000..5e12725d1664
--- /dev/null
+++ b/patches/attr-2.4.47/0001-fix-install-with-domain-user-or-domain-user.patch
@@ -0,0 +1,229 @@
+From: Jon Ringle <jon@ringle.org>
+Date: Mon, 23 May 2011 09:27:46 -0400
+Subject: [PATCH] fix install with domain\user or "domain user"
+
+If current user is domain\user the \ gets interpreted as an escape char
+resulting in user not found. This also breaks if the user or group contains a
+space.
+
+Enclose "$(PKG_USER)" and "$(PKG_GROUP)" in quotes to fix.
+
+Signed-off-by: Jon Ringle <jon@ringle.org>
+Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
+---
+ include/buildmacros |  2 +-
+ include/install-sh  | 26 +++++++++++++-------------
+ install-sh          | 26 +++++++++++++-------------
+ 3 files changed, 27 insertions(+), 27 deletions(-)
+
+diff --git a/include/buildmacros b/include/buildmacros
+index 9e81bb83d609..939f39b80087 100644
+--- a/include/buildmacros
++++ b/include/buildmacros
+@@ -40,7 +40,7 @@ OBJECTS = $(ASFILES:.s=.o) \
+ 	  $(LFILES:.l=.o) \
+ 	  $(YFILES:%.y=%.tab.o)
+ 
+-INSTALL	= $(TOPDIR)/include/install-sh -o $(PKG_USER) -g $(PKG_GROUP)
++INSTALL	= $(TOPDIR)/include/install-sh -o "$(PKG_USER)" -g "$(PKG_GROUP)"
+ 
+ IMAGES_DIR = $(TOPDIR)/all-images
+ DIST_DIR = $(TOPDIR)/dist
+diff --git a/include/install-sh b/include/install-sh
+index fb4997a3b013..3c70b31a3b2c 100755
+--- a/include/install-sh
++++ b/include/install-sh
+@@ -56,7 +56,7 @@ _chown ()
+ {
+     _st=255
+     if [ $# -eq 3 ] ; then
+-	chown $1:$2 $3
++	chown "$1":"$2" $3
+ 	_st=$?
+ 	if [ $_st -ne 0 ] ; then
+ 	    if [ $REAL_UID != '0' ] ; then
+@@ -115,7 +115,7 @@ fi
+ 
+ [ -n "$DIST_ROOT" -a $REAL_UID -ne 0 ] && CHOWN=true
+ 
+-while getopts "Dcm:d:S:o:g:T:" c $*
++while getopts "Dcm:d:S:o:g:T:" c "$@"
+ do
+    case $c in
+    c)
+@@ -168,10 +168,10 @@ then
+     fi
+     if [ $status -eq 0 ]
+     then
+-	$CHOWN $OWNER $GROUP $dir
++	$CHOWN "$OWNER" "$GROUP" $dir
+ 	status=$?
+     fi
+-    $MANIFEST d $DIRMODE $OWNER $GROUP ${dir#$DIST_ROOT}
++    $MANIFEST d $DIRMODE "$OWNER" "$GROUP" ${dir#$DIST_ROOT}
+ elif $Sflag
+ then
+     #
+@@ -217,7 +217,7 @@ then
+ 			install_name=$target/$solib
+ 			$CP $solib $install_name
+ 			status=$?
+-			$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$solib ${install_name#$DIST_ROOT}
++			$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$solib ${install_name#$DIST_ROOT}
+ 			break
+ 		fi
+ 	done
+@@ -268,7 +268,7 @@ then
+ 	install_name=$target/$old_library
+ 	$CP $old_library $install_name
+ 	status=$?
+-	$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$old_library ${install_name#$DIST_ROOT}
++	$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$old_library ${install_name#$DIST_ROOT}
+ 	;;
+     *)
+ 	echo "$prog: -T $lt_install invalid"
+@@ -281,7 +281,7 @@ then
+ 	if [ $status -eq 0 ]
+ 	then
+ 		$CHMOD $FILEMODE $install_name
+-		$CHOWN $OWNER $GROUP $install_name
++		$CHOWN "$OWNER" "$GROUP" $install_name
+ 	fi
+ 	;;
+     esac
+@@ -310,19 +310,19 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    else
+ 		$CHMOD $FILEMODE $dir
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir
++		    $CHOWN "$OWNER" "$GROUP" $dir
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$dir ${dir#$DIST_ROOT}
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$dir ${dir#$DIST_ROOT}
+ 	    fi
+ 	fi
+     else
+@@ -352,10 +352,10 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    fi
+ 	    [ $status -ne 0 ] && break
+ 	done
+diff --git a/install-sh b/install-sh
+index fb4997a3b013..3c70b31a3b2c 100755
+--- a/install-sh
++++ b/install-sh
+@@ -56,7 +56,7 @@ _chown ()
+ {
+     _st=255
+     if [ $# -eq 3 ] ; then
+-	chown $1:$2 $3
++	chown "$1":"$2" $3
+ 	_st=$?
+ 	if [ $_st -ne 0 ] ; then
+ 	    if [ $REAL_UID != '0' ] ; then
+@@ -115,7 +115,7 @@ fi
+ 
+ [ -n "$DIST_ROOT" -a $REAL_UID -ne 0 ] && CHOWN=true
+ 
+-while getopts "Dcm:d:S:o:g:T:" c $*
++while getopts "Dcm:d:S:o:g:T:" c "$@"
+ do
+    case $c in
+    c)
+@@ -168,10 +168,10 @@ then
+     fi
+     if [ $status -eq 0 ]
+     then
+-	$CHOWN $OWNER $GROUP $dir
++	$CHOWN "$OWNER" "$GROUP" $dir
+ 	status=$?
+     fi
+-    $MANIFEST d $DIRMODE $OWNER $GROUP ${dir#$DIST_ROOT}
++    $MANIFEST d $DIRMODE "$OWNER" "$GROUP" ${dir#$DIST_ROOT}
+ elif $Sflag
+ then
+     #
+@@ -217,7 +217,7 @@ then
+ 			install_name=$target/$solib
+ 			$CP $solib $install_name
+ 			status=$?
+-			$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$solib ${install_name#$DIST_ROOT}
++			$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$solib ${install_name#$DIST_ROOT}
+ 			break
+ 		fi
+ 	done
+@@ -268,7 +268,7 @@ then
+ 	install_name=$target/$old_library
+ 	$CP $old_library $install_name
+ 	status=$?
+-	$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$old_library ${install_name#$DIST_ROOT}
++	$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$old_library ${install_name#$DIST_ROOT}
+ 	;;
+     *)
+ 	echo "$prog: -T $lt_install invalid"
+@@ -281,7 +281,7 @@ then
+ 	if [ $status -eq 0 ]
+ 	then
+ 		$CHMOD $FILEMODE $install_name
+-		$CHOWN $OWNER $GROUP $install_name
++		$CHOWN "$OWNER" "$GROUP" $install_name
+ 	fi
+ 	;;
+     esac
+@@ -310,19 +310,19 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    else
+ 		$CHMOD $FILEMODE $dir
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir
++		    $CHOWN "$OWNER" "$GROUP" $dir
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$dir ${dir#$DIST_ROOT}
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$dir ${dir#$DIST_ROOT}
+ 	    fi
+ 	fi
+     else
+@@ -352,10 +352,10 @@ else
+ 		status=$?
+ 		if [ $status -eq 0 ]
+ 		then
+-		    $CHOWN $OWNER $GROUP $dir/$f
++		    $CHOWN "$OWNER" "$GROUP" $dir/$f
+ 		    status=$?
+ 		fi
+-		$MANIFEST f $FILEMODE $OWNER $GROUP $HERE/$f ${dir#$DIST_ROOT}/$f
++		$MANIFEST f $FILEMODE "$OWNER" "$GROUP" $HERE/$f ${dir#$DIST_ROOT}/$f
+ 	    fi
+ 	    [ $status -ne 0 ] && break
+ 	done
diff --git a/patches/attr-2.4.47/0001-fix-install-with-domain-user.patch b/patches/attr-2.4.47/0001-fix-install-with-domain-user.patch
deleted file mode 100644
index 893f766304d6..000000000000
--- a/patches/attr-2.4.47/0001-fix-install-with-domain-user.patch
+++ /dev/null
@@ -1,27 +0,0 @@
-From: Jon Ringle <jon@ringle.org>
-Date: Mon, 23 May 2011 09:27:46 -0400
-Subject: [PATCH] fix install with domain\user
-
-If current user is domain\user the \ gets interpreted as an escape char
-resulting in user not found.
-
-Enclose "$(PKG_USER)" and "$(PKG_GROUP)" in quotes to fix.
-
-Signed-off-by: Jon Ringle <jon@ringle.org>
----
- include/buildmacros | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/include/buildmacros b/include/buildmacros
-index 9e81bb8..939f39b 100644
---- a/include/buildmacros
-+++ b/include/buildmacros
-@@ -40,7 +40,7 @@ OBJECTS = $(ASFILES:.s=.o) \
- 	  $(LFILES:.l=.o) \
- 	  $(YFILES:%.y=%.tab.o)
- 
--INSTALL	= $(TOPDIR)/include/install-sh -o $(PKG_USER) -g $(PKG_GROUP)
-+INSTALL	= $(TOPDIR)/include/install-sh -o "$(PKG_USER)" -g "$(PKG_GROUP)"
- 
- IMAGES_DIR = $(TOPDIR)/all-images
- DIST_DIR = $(TOPDIR)/dist
diff --git a/patches/attr-2.4.47/0002-build-translations-only-if-gettext-is-enabled.patch b/patches/attr-2.4.47/0002-build-translations-only-if-gettext-is-enabled.patch
index c5125ef95fa1..af062437a665 100644
--- a/patches/attr-2.4.47/0002-build-translations-only-if-gettext-is-enabled.patch
+++ b/patches/attr-2.4.47/0002-build-translations-only-if-gettext-is-enabled.patch
@@ -1,4 +1,3 @@
-From 06d3c2ddb6ef0a9f53c36538ef018d6cca937e51 Mon Sep 17 00:00:00 2001
 From: Philipp Zabel <p.zabel@pengutronix.de>
 Date: Thu, 13 Jul 2017 11:25:35 +0200
 Subject: [PATCH] build translations only if gettext is enabled
@@ -31,6 +30,3 @@ index 4443fe092f37..896186e1e073 100644
  
  SUBDIRS = $(LIB_SUBDIRS) $(TOOL_SUBDIRS)
  
--- 
-2.11.0
-
diff --git a/patches/attr-2.4.47/series b/patches/attr-2.4.47/series
index 883e46360f47..2dc7b2624805 100644
--- a/patches/attr-2.4.47/series
+++ b/patches/attr-2.4.47/series
@@ -1,2 +1,5 @@
-0001-fix-install-with-domain-user.patch
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-fix-install-with-domain-user-or-domain-user.patch
 0002-build-translations-only-if-gettext-is-enabled.patch
+# 871964aae99308a74a3b9e9146294e20  - git-ptx-patches magic
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

             reply	other threads:[~2019-08-15 12:50 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-15 12:50 Marc Kleine-Budde [this message]
2019-08-15 13:39 ` Michael Olbrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190815125042.29479-1-mkl@pengutronix.de \
    --to=mkl@pengutronix.de \
    --cc=mol@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox