Bug 538723: run_for_effect is not portable for Bourne shell r=ted.mielczarek

build/Makefile.in

@@ -117,7 +117,7 @@ check::
           echo "TEST-UNEXPECTED-FAIL | build/ | unify failed to produce a universal binary!"; \
           false; \
         fi
-	@if ! test -f ./unify-test-universal; then \
+	@if test ! -f ./unify-test-universal; then \
           echo "TEST-UNEXPECTED-FAIL | build/ | unify failed to produce a universal binary!"; \
           false; \
         fi
@@ -136,7 +136,7 @@ check::
           echo "TEST-UNEXPECTED-FAIL | build/ | unify failed to unify a Java class file!"; \
           false; \
         fi
-	@if ! test -f ./unifytestc.class; then \
+	@if test ! -f ./unifytestc.class; then \
           echo "TEST-UNEXPECTED-FAIL | build/ | unify failed to unify a Java class file!"; \
           false; \
         fi

build/pymake/tests/include-dynamic.mk

@@ -1,5 +1,5 @@
 $(shell \
-if ! test -f include-dynamic.inc; then \
+if test ! -f include-dynamic.inc; then \
   echo "TESTVAR = oldval" > include-dynamic.inc; \
   sleep 2; \
   echo "TESTVAR = newval" > include-dynamic.inc.in; \

extensions/java/xpcom/package/Makefile.in

@@ -73,7 +73,7 @@ stage/javaconnect/.done:
 javaconnect.js: stage/javaconnect/.done
 	@echo Creating $@...
 	rm -f javaconnect.js
-	@if ! test -e "javaconnect.jst"; then             \
+	@if test ! -e "javaconnect.jst"; then             \
 		$(INSTALL) -m 644 $(srcdir)/javaconnect.jst .; \
 	fi
 	$(PERL) $(topsrcdir)/toolkit/mozapps/installer/makejs.pl javaconnect.jst \