git: 5bdfcb07f38a - stable/12 - genassym.sh: Fix two minor issues found by shellcheck

Warner Losh imp at FreeBSD.org
Sun Sep 12 18:32:44 UTC 2021


The branch stable/12 has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=5bdfcb07f38a51ba09147da6074b0009f53861c4

commit 5bdfcb07f38a51ba09147da6074b0009f53861c4
Author:     Warner Losh <imp at FreeBSD.org>
AuthorDate: 2021-07-28 19:42:29 +0000
Commit:     Warner Losh <imp at FreeBSD.org>
CommitDate: 2021-09-12 16:33:13 +0000

    genassym.sh: Fix two minor issues found by shellcheck
    
    o Remove redunant $ in $(( )) expression.
    o Quote arg passed to work so paths with spaces, etc will work.
    
    MFC After:              2 weeks
    Reviewed by:            kib
    Sponsored by:           Netflix
    Differential Revision:  https://reviews.freebsd.org/D31335
    
    (cherry picked from commit 46dd3ef0338ba2ad24d05bab2a614410cf17b017)
---
 sys/kern/genassym.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/sys/kern/genassym.sh b/sys/kern/genassym.sh
index 0374c8844b9a..999090f22d0e 100644
--- a/sys/kern/genassym.sh
+++ b/sys/kern/genassym.sh
@@ -57,7 +57,7 @@ do
 	*)	usage;;
 	esac
 done
-shift $(($OPTIND - 1))
+shift $((OPTIND - 1))
 case $# in
 1)	;;
 *)	usage;;
@@ -65,8 +65,8 @@ esac
 
 if [ "$use_outfile" = "yes" ]
 then
-	work $1  3>"$outfile" >&3 3>&-
+	work "$1"  3>"$outfile" >&3 3>&-
 else
-	work $1
+	work "$1"
 fi
 


More information about the dev-commits-src-all mailing list