Reviewed by Darin.

        Change Makefiles to return non-zero when module make fails.

        * Makefile:

WebKitTools:
        Change Makefiles to return non-zero when module make fails.
        Fix bug where if xcode options are not set, modules can build in the
        wrong directory.

        * Makefile:
        * Scripts/webkitdirs.pm:

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

diff --git a/ChangeLog b/ChangeLog
index 9d66689..31db14c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2006-10-30  Stephanie Lewis  <slewis@apple.com>
+
+        Reviewed by Darin.
+
+        Change Makefiles to return non-zero when module make fails.
+
+        * Makefile:
+
 2006-10-27  Brady Eidson  <beidson@apple.com>
 
         Rubber stamped by Tim Hatcher

diff --git a/Makefile b/Makefile
index 42283bb..c9b0d90 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,16 +1,21 @@
 MODULES = JavaScriptCore JavaScriptGlue WebCore WebKit WebKitTools
 
 all:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
 debug d development dev develop:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
-
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+       
 release r deployment dep deploy:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
-
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+       
 universal u:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
 
 clean:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+

diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
index b5bdf20..c38b144 100644 (file)
--- a/WebKitTools/ChangeLog
+++ b/WebKitTools/ChangeLog
@@ -1,3 +1,14 @@
+2006-10-30  Stephanie Lewis  <slewis@apple.com>
+
+        Reviewed by Darin.
+        
+        Change Makefiles to return non-zero when module make fails.
+        Fix bug where if xcode options are not set, modules can build in the
+        wrong directory.
+
+        * Makefile:
+        * Scripts/webkitdirs.pm:
+
 2006-10-30  Matt Lilek  <pewtermoose@gmail.com>
 
         Reviewed by Tim H.

diff --git a/WebKitTools/Makefile b/WebKitTools/Makefile
index 0562424..a0f946a 100644 (file)
--- a/WebKitTools/Makefile
+++ b/WebKitTools/Makefile
@@ -1,16 +1,21 @@
 MODULES = Drosera DumpRenderTree
 
 all:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
 debug d development dev develop:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
-
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+       
 release r deployment dep deploy:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
 universal u:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
 
 clean:
-       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; if [ $$? -ne 0 ]; then break; fi; done
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+

diff --git a/WebKitTools/Scripts/webkitdirs.pm b/WebKitTools/Scripts/webkitdirs.pm
index 8e9b2ce..6a7f671 100644 (file)
--- a/WebKitTools/Scripts/webkitdirs.pm
+++ b/WebKitTools/Scripts/webkitdirs.pm
@@ -57,15 +57,21 @@ sub determineSourceDir
 {
     return if $sourceDir;
     $sourceDir = $FindBin::Bin;
-    if ($sourceDir !~ s|/[^/]+/[^/]+$||) {
-        die "Could not find two levels above source directory using FindBin.\n";
+    
+    # walks up path checking each directory to see if it is the main WebKit project dir, 
+    # defined by containing JavaScriptCore, WebCore, and WebKit
+    until (-d "$sourceDir/JavaScriptCore" && -d "$sourceDir/WebCore" && -d "$sourceDir/WebKit")
+    {
+        if ($sourceDir !~ s|/[^/]+$||) {
+            die "Could not find top level webkit directory above source directory using FindBin.\n";
+        }
     }
 }
 
 # used for scripts which are stored in a non-standard location
 sub setSourceDir($)
 {
-    $sourceDir = $_;
+    ($sourceDir) = @_;
 }
 
 sub determineBaseProductDir