CVS commit: pkgsrc/bootstrap

["Roland Illig" <rillig%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   rillig
Date:           Sun Feb 23 18:32:46 UTC 2020

Modified Files:
        pkgsrc/bootstrap: bootstrap

Log Message:
bootstrap: use $* instead of $@ where appropriate

This runs the echo command with fewer arguments (1 in most cases).

The observable behavior is still the same, except for directory names
containing spaces, which are not supported anyway.


To generate a diff of this commit:
cvs rdiff -u -r1.273 -r1.274 pkgsrc/bootstrap/bootstrap

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/bootstrap/bootstrap
diff -u pkgsrc/bootstrap/bootstrap:1.273 pkgsrc/bootstrap/bootstrap:1.274
--- pkgsrc/bootstrap/bootstrap:1.273    Thu Dec  5 22:19:56 2019
+++ pkgsrc/bootstrap/bootstrap  Sun Feb 23 18:32:46 2020
@@ -1,6 +1,6 @@
 #! /bin/sh
 
-# $NetBSD: bootstrap,v 1.273 2019/12/05 22:19:56 sevan Exp $
+# $NetBSD: bootstrap,v 1.274 2020/02/23 18:32:46 rillig Exp $
 #
 # Copyright (c) 2001-2011 Alistair Crooks <agc%NetBSD.org@localhost>
 # All rights reserved.
@@ -123,13 +123,13 @@ mkbinarykit_tgz()
 
 die()
 {
-       echo >&2 "$@"
+       echo >&2 "$*"
        exit 1
 }
 
 echo_msg()
 {
-       echo "===> $@"
+       echo "===> $*"
 }
 
 # see if we're using gcc.  If so, set $compiler_is_gnu to '1'.
@@ -293,7 +293,7 @@ is_root()
 # run a command, abort if it fails
 run_cmd()
 {
-       echo_msg "running: $@"
+       echo_msg "running: $*"
        eval "$@"
        ret=$?
         if [ $ret -ne 0 ]; then
@@ -306,7 +306,7 @@ run_cmd()
 # install-sh wrapper instead.
 mkdir_p()
 {
-       for dir in $@; do
+       for dir in "$@"; do
                run_cmd "$install_sh -d -o $user -g $group $dir"
        done
 }
@@ -424,7 +424,7 @@ if [ -n "$PKG_PATH" ]; then
 fi
 
 build_start=`date`
-echo_msg "bootstrap command: $0 $@"
+echo_msg "bootstrap command: $0 $*"
 echo_msg "bootstrap started: $build_start"
 
 # ensure system locations are empty; we will set them later when we know