From owner-svn-ports-head@freebsd.org Sun Jul 22 07:24:37 2018 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id EA1651042B6F; Sun, 22 Jul 2018 07:24:36 +0000 (UTC) (envelope-from se@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 989EB8AF3B; Sun, 22 Jul 2018 07:24:36 +0000 (UTC) (envelope-from se@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 75B732A05; Sun, 22 Jul 2018 07:24:36 +0000 (UTC) (envelope-from se@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id w6M7OaIw061536; Sun, 22 Jul 2018 07:24:36 GMT (envelope-from se@FreeBSD.org) Received: (from se@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id w6M7Oakb061532; Sun, 22 Jul 2018 07:24:36 GMT (envelope-from se@FreeBSD.org) Message-Id: <201807220724.w6M7Oakb061532@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: se set sender to se@FreeBSD.org using -f From: Stefan Esser Date: Sun, 22 Jul 2018 07:24:36 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r475089 - in head/ports-mgmt/portmaster: . files X-SVN-Group: ports-head X-SVN-Commit-Author: se X-SVN-Commit-Paths: in head/ports-mgmt/portmaster: . files X-SVN-Commit-Revision: 475089 X-SVN-Commit-Repository: ports MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 22 Jul 2018 07:24:37 -0000 Author: se Date: Sun Jul 22 07:24:35 2018 New Revision: 475089 URL: https://svnweb.freebsd.org/changeset/ports/475089 Log: Make some sub-routines use "$@" instead of $* to correctly pass arguments with blanks to invoked commands. This fixes clean-up of shared library backups, where only one file was removed due to a parameter list being terminated by the first blank within one parameter. Reported by: tatsuki_makino at hotmail.com (Tatsuki Makino) Approved by: antoine (implicit) Modified: head/ports-mgmt/portmaster/Makefile head/ports-mgmt/portmaster/files/patch-portmaster Modified: head/ports-mgmt/portmaster/Makefile ============================================================================== --- head/ports-mgmt/portmaster/Makefile Sun Jul 22 07:03:13 2018 (r475088) +++ head/ports-mgmt/portmaster/Makefile Sun Jul 22 07:24:35 2018 (r475089) @@ -2,7 +2,7 @@ PORTNAME= portmaster PORTVERSION= 3.19 -PORTREVISION= 13 +PORTREVISION= 14 CATEGORIES= ports-mgmt MAINTAINER= se@FreeBSD.org Modified: head/ports-mgmt/portmaster/files/patch-portmaster ============================================================================== --- head/ports-mgmt/portmaster/files/patch-portmaster Sun Jul 22 07:03:13 2018 (r475088) +++ head/ports-mgmt/portmaster/files/patch-portmaster Sun Jul 22 07:24:35 2018 (r475089) @@ -40,7 +40,7 @@ if [ -z "$PM_FIRST_PASS" ]; then echo "DISPLAY_LIST='$DISPLAY_LIST'" >> $IPC_SAVE echo "INSTALLED_LIST='$INSTALLED_LIST'" >> $IPC_SAVE -@@ -308,20 +308,19 @@ safe_exit () { +@@ -308,50 +308,49 @@ safe_exit () { exit ${1:-0} } # safe_exit() @@ -63,12 +63,18 @@ pm_isdir () { builtin test -d "$1"; } -pm_isdir_pd () { local dir=$pd/$(dir_part "$1"); - builtin test -d "$dir"; } +-pm_kill () { kill $* >/dev/null 2>/dev/null; } +pm_isdir_pd () { local dir=$(dir_part "$1"); + builtin test -n "$dir" -a -d "$pd/$dir"; } - pm_kill () { kill $* >/dev/null 2>/dev/null; } ++pm_kill () { kill "$@" >/dev/null 2>/dev/null; } pm_make () { ( unset -v CUR_DEPS INSTALLED_LIST PM_DEPTH build_l PM_URB_LIST; - /usr/bin/nice /usr/bin/make $PM_MAKE_ARGS $*; ); } -@@ -332,7 +331,7 @@ pm_mktemp () { +- /usr/bin/nice /usr/bin/make $PM_MAKE_ARGS $*; ); } +-pm_make_b () { /usr/bin/make $PM_MAKE_ARGS BEFOREPORTMK=bpm $*; } ++ /usr/bin/nice /usr/bin/make $PM_MAKE_ARGS "$@"; ); } ++pm_make_b () { /usr/bin/make $PM_MAKE_ARGS BEFOREPORTMK=bpm "$@"; } + pm_mktemp () { + pm_mktemp_file=`/usr/bin/mktemp -t f-${PM_PARENT_PID}-$1 2>&1` || + fail "mktemp for $1 failed:\n ${pm_mktemp_file#mktemp: }" } pm_unlink () { [ -e "$1" ] && /bin/unlink $1; } pm_islocked () { local locked; [ -n "$1" ] && locked=`pkg query %k "$1"` && @@ -77,6 +83,31 @@ # Superuser versions for commands that need root privileges +-pm_find_s () { $PM_SU_CMD /usr/bin/find $*; } ++pm_find_s () { $PM_SU_CMD /usr/bin/find "$@"; } + pm_install_s () { $PM_SU_CMD /usr/bin/install -p -o root -g wheel -m 644 $1 $2; } + pm_make_s () { ( unset -v CUR_DEPS INSTALLED_LIST PM_DEPTH build_l PM_URB_LIST; +- $PM_SU_CMD /usr/bin/nice /usr/bin/make $PM_MAKE_ARGS $*; ); } ++ $PM_SU_CMD /usr/bin/nice /usr/bin/make $PM_MAKE_ARGS "$@"; ); } + pm_mkdir_s () { $PM_SU_CMD /bin/mkdir -p $1; } + pm_pkg_delete_s () { +- $PM_SU_CMD /usr/local/sbin/pkg delete -yf $*; ++ $PM_SU_CMD /usr/local/sbin/pkg delete -yf "$@"; + for d in $* ; do + pm_rm_s -rf $pdb/$d + done + } +-pm_rm_s () { $PM_SU_CMD /bin/rm $*; } +-pm_rmdir_s () { $PM_SU_CMD /bin/rmdir $*; } ++pm_rm_s () { $PM_SU_CMD /bin/rm "$@"; } ++pm_rmdir_s () { $PM_SU_CMD /bin/rmdir "$@"; } + pm_unlink_s () { [ -e "$1" ] && $PM_SU_CMD /bin/unlink $1; } + +-pm_v () { [ -n "$PM_VERBOSE" ] && echo -e "$*"; } ++pm_v () { [ -n "$PM_VERBOSE" ] && echo -e "$@"; } + pm_sv () { [ -n "$PM_SU_VERBOSE" ] && echo -e "===>>> SU $*"; } + + # Do this here so we can have a reasonably good guess. @@ -562,12 +561,37 @@ find_glob_dirs () { return 1 }