STYLE: allow '=' option separators in some bin/tools scripts

bin/tools/README

@@ -1,2 +1,2 @@
-Misc. tools, scripts, templates that are useful (eg, for building applications)
+Miscellaneous tools, scripts, templates that are useful
-but don't need to be in the PATH.
+(eg, for building applications) but don't need to be in the PATH.

bin/tools/foamConfigurePaths

@@ -209,8 +209,7 @@ replaceCsh()
     done
 }
 
-# Get the option's value (argument).
+# Get the option's value (argument), or die on missing or empty argument
-# Die if the argument doesn't exist or is empty
 # $1 option
 # $2 value
 getOptionValue()

bin/tools/foamCreateManpage

@@ -26,11 +26,11 @@ usage() {
 
 Usage: ${0##*/} [OPTION] [appName .. [appNameN]]
 options:
-  -dir DIR          Directory to process
+  -dir=DIR          Input directory to process
-  -output DIR       Write to alternative output directory
+  -output=DIR       Write to alternative output directory
   -pdf              Process as nroff man content and pass to ps2pdf
   -gz | -gzip       Compress manpage output
-  -version VER      Specify an alternative version
+  -version=VER      Specify an alternative version
   -h | -help        Print the usage
 
 Query OpenFOAM applications with -help-man for their manpage content
@@ -58,6 +58,15 @@ die()
     exit 1
 }
 
+# Get the option's value (argument), or die on missing or empty argument
+# $1 option
+# $2 value
+getOptionValue()
+{
+   [ -n "$2" ] || die "'$1' option requires an argument"
+   echo "$2"
+}
+
 #-------------------------------------------------------------------------------
 searchDirs="$FOAM_APPBIN"
 unset sedFilter outputDir outputType
@@ -68,9 +77,12 @@ do
     -h | -help*)
         usage
         ;;
+    -dir=*)
+        searchDirs="${1#*=}"
+        [ -d "$searchDirs" ] || die "directory not found '$searchDirs'"
+        ;;
     -dir)
-        [ "$#" -ge 2 ] || die "'$1' option requires an argument"
+        searchDirs=$(getOptionValue "$@")
-        searchDirs="$2"
         [ -d "$searchDirs" ] || die "directory not found '$searchDirs'"
         shift
         ;;
@@ -80,15 +92,20 @@ do
     -pdf)
         outputType="pdf"
         ;;
+    -version=*)
+        version="${1#*=}"
+        sedFilter='s/OpenFOAM-[^\"]*/OpenFOAM-'"$version/"
+        ;;
     -version)
-        [ "$#" -ge 2 ] || die "'$1' option requires an argument"
+        version=$(getOptionValue "$@")
-        version="$2"
         sedFilter='s/OpenFOAM-[^\"]*/OpenFOAM-'"$version/"
         shift
         ;;
+    -output=*)
+        outputDir="${1#*=}"
+        ;;
     -output)
-        [ "$#" -ge 2 ] || die "'$1' option requires an argument"
+        outputDir=$(getOptionValue "$@")
-        outputDir="$2"
         shift
         ;;
     -*)
@@ -101,7 +118,7 @@ do
     shift
 done
 
-: ${outputDir:=$defaultOutputDir}
+: "${outputDir:=$defaultOutputDir}"
 
 # Verify that output is writeable
 if [ -e "$outputDir" ]

bin/tools/foamPackRelease

@@ -28,7 +28,7 @@
 # Description
 #     Simple script generator for packing OpenFOAM sources and submodules
 #
-#       $ foamPackRelease -output some/path origin/master > create-tar
+#       $ foamPackRelease -output=some/path origin/master > create-tar
 #       $ bash ./create-tar
 #
 #     Or directly:
@@ -46,7 +46,7 @@ cat <<USAGE
 
 Usage: ${0##*/} [OPTION] commit-ish
 options:
-   -output DIR      Specify alternative output directory
+   -output=dir      Write to alternative output directory
    -no-modules      Exclude submodules
    -no-patch        Ignore _patch number for the output tar-file
    -compress=TYPE   Compress with specified type
@@ -78,6 +78,7 @@ die()
     exit 1
 }
 
+
 #-------------------------------------------------------------------------------
 outputDir="."
 unset skipModules skipPatchNum
@@ -94,6 +95,9 @@ do
         outputDir="$2"
         shift
         ;;
+    -output=*)
+        outputDir="${1#*=}"
+        ;;
     -no-modules)
         skipModules=true
         ;;
@@ -291,7 +295,7 @@ case "$compress" in
 
     (tgz)
     echo "Using gzip compression with tgz ending" 1>&2
-    echo 'gzip -c9 "$outputDir/$tarName.tar" > "$outputDir/$tarName.tgz"'
+    echo 'gzip -c -9 "$outputDir/$tarName.tar" > "$outputDir/$tarName.tgz"'
     ;;
 
     (bz | bzip | bzip2)

bin/tools/git-find-trailingspace

@@ -4,10 +4,10 @@
 #     git-find-trailingspace
 #
 # Description
-#     Use git grep to search for files with trailing whitespace
+#     Use git grep to find files with trailing whitespace
 #
 #------------------------------------------------------------------------------
 
-git grep -c -P '\s+$' -- $@
+git grep --count -P '\s+$' -- $@
 
 #------------------------------------------------------------------------------