top level:
authorthatcher <thatcher@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Dec 2006 19:30:36 +0000 (19:30 +0000)
committerthatcher <thatcher@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Dec 2006 19:30:36 +0000 (19:30 +0000)
        Reviewed by Brady.

        * Makefile.shared: use $PIPESTATUS[0] and a sub-shell to exit with xcodebuild's exit status

WebKitTools:

        Reviewed by Brady.

        * Makefile.shared: use $PIPESTATUS[0] and a sub-shell to exit with xcodebuild's exit status

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@18216 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Makefile.shared
WebKitTools/ChangeLog
WebKitTools/Makefile.shared

index 69588e6..6172428 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-12-14  Timothy Hatcher  <timothy@apple.com>
+
+        Reviewed by Brady.
+
+        * Makefile.shared: use $PIPESTATUS[0] and a sub-shell to exit with xcodebuild's exit status
+
 2006-12-13  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Brady and Anders.
index 03404cb..df3389c 100644 (file)
@@ -1,20 +1,20 @@
 XCODE_OPTIONS = `perl -I../WebKitTools/Scripts -Mwebkitdirs -e 'print XcodeOptionString()'` $(ARGS)
 
 all:
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 debug d development dev develop: force
        ../WebKitTools/Scripts/set-webkit-configuration --debug
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 release r deployment dep deploy: force
        ../WebKitTools/Scripts/set-webkit-configuration --release
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 universal u: force
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) 'ARCHS=ppc i386' | grep -v setenv
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) 'ARCHS=ppc i386' | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 clean:
-       xcodebuild -alltargets clean $(XCODE_OPTIONS) | grep -v setenv
+       ( xcodebuild -alltargets clean $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 force: ;
index ce8129a..840257c 100644 (file)
@@ -1,3 +1,9 @@
+2006-12-14  Timothy Hatcher  <timothy@apple.com>
+
+        Reviewed by Brady.
+
+        * Makefile.shared: use $PIPESTATUS[0] and a sub-shell to exit with xcodebuild's exit status
+
 2006-12-13  Zack Rusin  <zack@kde.org>
 
         Reviewed by rwlbuis
index 0f8c148..8a42f12 100644 (file)
@@ -1,20 +1,20 @@
 XCODE_OPTIONS = `perl -I../Scripts -Mwebkitdirs -e 'print XcodeOptionString()'` $(ARGS)
 
 all:
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS)
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 debug d development dev develop: force
        ../Scripts/set-webkit-configuration --debug
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS)
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 release r deployment dep deploy: force
        ../Scripts/set-webkit-configuration --release
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS)
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 universal u: force
-       xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) 'ARCHS=ppc i386'
+       ( xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) 'ARCHS=ppc i386' | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 clean:
-       xcodebuild -alltargets clean $(XCODE_OPTIONS)
+       ( xcodebuild -alltargets clean $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 force: ;