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: http://svn.webkit.org/repository/webkit/trunk@18216 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index 69588e66c90e4968f31023f30c9590e8a7f9b8dd..617242893720ee2193a1deddb848ce3ed70fadd1 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.
 2006-12-13  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Brady and Anders.
index 03404cb01fe5a756a85af02553996210615f342b..df3389ca3b000372508d3f65f980cbb1616a2b60 100644 (file)
@@ -1,20 +1,20 @@
 XCODE_OPTIONS = `perl -I../WebKitTools/Scripts -Mwebkitdirs -e 'print XcodeOptionString()'` $(ARGS)
 
 all:
 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
 
 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
 
 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
 
 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:
 
 clean:
-       xcodebuild -alltargets clean $(XCODE_OPTIONS) | grep -v setenv
+       ( xcodebuild -alltargets clean $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 force: ;
 
 force: ;
index ce8129ad13acaa01e96668ab35f8b59c25b36935..840257c40ffa42f82644efcdefa993a47b6f2de8 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
 2006-12-13  Zack Rusin  <zack@kde.org>
 
         Reviewed by rwlbuis
index 0f8c148dbc0cf5ba46b7e8ca9644613f203e878b..8a42f12a5dae8a22eaaf9b3742bacbafb658b908 100644 (file)
@@ -1,20 +1,20 @@
 XCODE_OPTIONS = `perl -I../Scripts -Mwebkitdirs -e 'print XcodeOptionString()'` $(ARGS)
 
 all:
 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
 
 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
 
 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
 
 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:
 
 clean:
-       xcodebuild -alltargets clean $(XCODE_OPTIONS)
+       ( xcodebuild -alltargets clean $(XCODE_OPTIONS) | grep -v setenv && exit $${PIPESTATUS[0]} )
 
 force: ;
 
 force: ;