X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=docs%2Fcontrib%2Fbuild_nmh;h=d8d2315c86a7ae27040a13daf84604f0ff94b867;hb=a69289ce6e12c75a4a934d966a502b3899374e29;hp=335e042168e5b986081199aebbecf7a8875f4d7a;hpb=1a28128fdeeeeb9152c3a7289c638e465853d489;p=mmh

diff --git a/docs/contrib/build_nmh b/docs/contrib/build_nmh
index 335e042..d8d2315 100755
--- a/docs/contrib/build_nmh
+++ b/docs/contrib/build_nmh
@@ -129,26 +129,27 @@ if install-mh -check >/dev/null 2>&1; then
 
   mtsconf=`dirname "$mhbin"`/etc/mts.conf
   if [ -f "$mtsconf" ]; then
-    mts_entry=`grep '^mts:' $mtsconf`
+    mts_entry=`grep '^mts:' "$mtsconf"`
     if [ "$mts_entry" ]; then
-      mts=`echo $mts_entry | sed -e 's/^mts: *//'`
+      mts=`echo "$mts_entry" | sed -e 's/^mts: *//'`
       if [ "$mts"  -a  "$mts" != smtp ]; then
         config_mts="$mts"
       fi
     fi
 
-    mtsconfservers=`grep '^servers:' $mtsconf`
+    mtsconfservers=`grep '^servers:' "$mtsconf"`
     if [ "$mtsconfservers" ]; then
-      servers=`echo $mtsconfservers | sed -e 's/^servers: *//' -e 's/ /\\\ /g'`
+      servers=`echo "$mtsconfservers" | \
+               sed -e 's/^servers: *//' -e 's/ /\\\ /g'`
       [ "$servers" ]  &&  config_smtpservers="$servers"
     fi
   fi
 
-  if $ldd $mhbin/inc | grep sasl >/dev/null; then
+  if $ldd "$mhbin/inc" | grep sasl >/dev/null; then
     config_sasl=y
   fi
 
-  if $ldd $mhbin/inc | grep ssl >/dev/null; then
+  if $ldd "$mhbin/inc" | grep ssl >/dev/null; then
     config_tls=y
   fi
 fi
@@ -173,7 +174,7 @@ if [ $yes -eq 0 ]; then
   read locking
   [ "$locking" ]  &&  config_locking="$locking"
 
-  printf 'MTS (smtp|sendmail) [%s]: ' $config_mts
+  printf 'MTS (smtp|sendmail/smtp|sendmail/pipe) [%s]: ' $config_mts
   read mts
   [ "$mts" ]  &&  config_mts="$mts"
 
@@ -202,11 +203,11 @@ if [ $yes -eq 0 ]; then
 
   printf 'Default editor [%s]: ' $config_editor
   read editor
-  [ "$editor" ]  &&  config_editor=$editor
+  [ "$editor" ]  &&  config_editor="$editor"
 
   printf 'Pager [%s]: ' $config_pager
   read pager
-  [ "$pager" ]  &&  config_pager=$pager
+  [ "$pager" ]  &&  config_pager="$pager"
 
   #### Don't confirm debug here:  obey the -d option to this script.
 fi
@@ -255,12 +256,12 @@ if [ -f Makefile ]; then
   fi
 fi
 
-/bin/rm -f $logfile
+/bin/rm -f "$logfile"
 if [ -f configure  -a  -f Makefile.in ]; then
   :
 else
   [ $verbose -ge 1 ]  &&  echo autoconfiguring . . .
-  ./autogen.sh >>$logfile 2>&1
+  ./autogen.sh >>"$logfile" 2>&1
 fi
 
 
@@ -268,59 +269,63 @@ fi
 #### Build.
 ####
 [ $verbose -ge 1 ]  &&  echo configuring . . .
-echo ./configure $config_opts ${smtpservers:+"$smtpservers"} >>$logfile 2>&1
-./configure $config_opts ${smtpservers:+"$smtpservers"} >>$logfile 2>&1
+echo ./configure $config_opts ${smtpservers:+"$smtpservers"} >>"$logfile" 2>&1
+./configure $config_opts ${smtpservers:+"$smtpservers"} >>"$logfile" 2>&1
 status=$?
 
 if [ $status -eq 0 ]; then
   [ $verbose -ge 1 ]  &&  echo building . . .
-  make >>$logfile 2>&1
+  make >>"$logfile" 2>&1
   status=$?
 
   if [ $status -eq 0 ]; then
-    if [ "${TESTS_SHELL}"x = x ]; then
+    if [ "$TESTS_SHELL"x = x ]; then
       #### Bonus:  use heirloom shell to test, if available, and if
       #### TESTS_SHELL hadn't already been set.
       heirloom_shell=/usr/lib/heirloom/5bin/sh
-      if [ -x "${heirloom_shell}" ]; then
-        TESTS_SHELL="${heirloom_shell}"; export TESTS_SHELL
+      if [ -x "$heirloom_shell" ]; then
+        TESTS_SHELL="$heirloom_shell"; export TESTS_SHELL
       fi
    fi
 
     [ $verbose -ge 1 ]  &&  echo testing . . .
-    checkoutput=`make $check 2>>$logfile`
+    checkoutput=`make $check 2>>"$logfile"`
     status=$?
 
     tests_summary=`echo "$checkoutput" | grep tests`
+    #### If multiple tests not run, that line will be caught by the
+    #### "grep tests" above.
+    test_not_run=`echo "$checkoutput" | grep 'test was not run'`
     if [ "$tests_summary" ]; then
-      echo '===================' >>$logfile
-      echo $tests_summary >>$logfile
-      echo '===================' >>$logfile
+      echo '===================' >>"$logfile"
+      echo "$tests_summary" >>"$logfile"
+      [ "$test_not_run" ]  &&  echo "$test_not_run" >>"$logfile"
+      echo '===================' >>"$logfile"
       [ "$check" = distcheck ]  &&  \
-        echo "$checkoutput" | tail -n 4 >>$logfile
+        echo "$checkoutput" | tail -n 4 >>"$logfile"
     fi
 
     if [ $status -eq 0 ]; then
       if [ $install -ge 1 ]; then
         [ $verbose -ge 1 ]  &&  echo installing . . .
-        ($installpriv make install) >/dev/null 2>>$logfile
+        ($installpriv make install) >/dev/null 2>>"$logfile"
         status=$?
       fi
 
       if [ $status -eq 0  -a  $build_rpm -ge 1 ]; then
         [ $verbose -ge 1 ]  &&  echo building rpm . . .
-        make rpm >/dev/null 2>>$logfile
+        make rpm >/dev/null 2>>"$logfile"
         status=$?
       fi
     fi
   fi
 fi
 
-grep 'Error' $logfile
-grep 'warn' $logfile
+grep 'Error' "$logfile"
+grep 'warn' "$logfile"
 if [ $status -ne 0 ]; then
   echo build failed!
-  echo build log is in $logfile
+  echo build log is in "$logfile"
 fi
 [ $status -eq 0  -a  $verbose -ge 1 ]  &&  echo build completed successfully