Revert "[makexpi] Run $VALIDATE in bg"

jsha · jsha · commit 67413bcea05f · 2014-10-15T13:17:12.000-04:00
This reverts commit e0c5cad. Putting validate in the background means that when there is a failure and we call `die`, the `exit 1` gets run in a subshell and so error status does not get returned to the caller of makexpi.sh (i.e. test.sh). Fixes EFForg#671
diff --git a/makexpi.sh b/makexpi.sh
@@ -90,7 +90,7 @@ if [ "$1" != "--fast" ] ; then
     echo >&2
   else
     die "Validation of rulesets failed."
-  fi &
+  fi
 
   # Check for xmllint.
   type xmllint >/dev/null || die "xmllint not available"
@@ -108,7 +108,6 @@ if [ "$1" != "--fast" ] ; then
     }
     if validate_grammar 2>/dev/null
     then
-      wait;
       echo Validation of rulesets against $GRAMMAR succeeded. >&2
     else
       validate_grammar 2>&1 | grep -v validates
