Bug 538723 run_for_effect is not portable for Bourne shell r=ted.mielczarek
authorGinn Chen <ginn.chen@sun.com>
Tue, 02 Mar 2010 13:56:46 +0800
changeset 38836 ff5a35f0a2cfe8b7972c4ef0c662c778a7c59132
parent 38835 52187d722adcb53ba77dee6af06bed0df8fc510c
child 38837 b4f8880a80c2eb0aeaf2459ff056afd7a1d9c3cb
push id11888
push userginn.chen@sun.com
push dateTue, 02 Mar 2010 06:01:26 +0000
treeherdermozilla-central@b4f8880a80c2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersted.mielczarek
bugs538723
milestone1.9.3a3pre
Bug 538723 run_for_effect is not portable for Bourne shell r=ted.mielczarek
toolkit/locales/l10n.mk
--- a/toolkit/locales/l10n.mk
+++ b/toolkit/locales/l10n.mk
@@ -57,17 +57,17 @@
 #   to initially clobber the locale staging area, and then to build the
 #   language pack and zip package.
 #   Other targets like windows installers might be listed, too, and should
 #   be defined in the including makefile.
 # The including makefile should provide values for the variables
 #   MOZ_APP_VERSION and MOZ_LANGPACK_EID.
 
 
-run_for_effects := $(shell if ! test -d $(DIST); then $(NSINSTALL) -D $(DIST); fi)
+run_for_effects := $(shell if test ! -d $(DIST); then $(NSINSTALL) -D $(DIST); fi)
 _ABS_DIST := $(shell cd $(DIST) && pwd)
 
 
 # This makefile uses variable overrides from the libs-% target to
 # build non-default locales to non-default dist/ locations. Be aware!
 
 AB = $(firstword $(subst -, ,$(AB_CD)))