On Fri, Oct 26, 2018 at 09:34:51AM +0000, andrej.gantv...@wago.com wrote: > Hello, > > > > The white-spaces are broken in this patch, so it does not apply. Maybe send > > it as an attachment instead. > > Here it is!
It applies now, but you did not address my other review. Michael > From 0a062f0179ae647bec338161aacf1a8a29414083 Mon Sep 17 00:00:00 2001 > From: Andrej Gantvorg <andrej.gantv...@wago.com> > Date: Fri, 5 Oct 2018 09:22:13 +0200 > Subject: [PTXDIST PATCH] ptxd_install_resolve_usr_grp: skip empty params. > To: ptxdist@pengutronix.de > > Empty parameters may be passed i.e. when calling install_archive with > UID/GID set to '-'. According to documentation this should preserve the > owner/group stored in the archive. Without this patch, empty strings are > mapped to the first entry in /etc/{passwd,group} (usually 0) instead, > thus breaking the documented behaviour. > > Signed-off-by: Andrej Gantvorg <andrej.gantv...@wago.com> > --- > scripts/lib/ptxd_make_xpkg_pkg.sh | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/scripts/lib/ptxd_make_xpkg_pkg.sh > b/scripts/lib/ptxd_make_xpkg_pkg.sh > index b3f2f20f3..0ec1c9b2a 100644 > --- a/scripts/lib/ptxd_make_xpkg_pkg.sh > +++ b/scripts/lib/ptxd_make_xpkg_pkg.sh > @@ -79,12 +79,12 @@ export -f ptxd_install_getent_id > # convert usr/grp that contain names into numeric values > # > ptxd_install_resolve_usr_grp() { > - if ! [ 0 -le $usr ] 2>/dev/null; then > + if ! [ 0 -le "$usr" ] 2>/dev/null && ! [ -z "$usr" ] ; then > ptxd_install_getent_id usr || return > else > unset usr_name > fi > - if ! [ 0 -le $grp ] 2>/dev/null; then > + if ! [ 0 -le "$grp" ] 2>/dev/null && ! [ -z "$grp" ] ; then > ptxd_install_getent_id grp || return > else > unset grp_name > -- > 2.11.3 > > _______________________________________________ > ptxdist mailing list > ptxdist@pengutronix.de -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de