Error consistently when running recursive make
authorDan Nicholson <dbn.lists@gmail.com>
Tue, 6 May 2008 19:29:53 +0000 (12:29 -0700)
committerDan Nicholson <dbn.lists@gmail.com>
Tue, 6 May 2008 19:29:53 +0000 (12:29 -0700)
When changing directories and running a sub-make, ensure that both the
cd and make commands propagate errors to the parent make.

Makefile
src/egl/Makefile
src/glu/Makefile
src/glx/Makefile
src/mesa/Makefile
src/mesa/drivers/beos/Makefile

index 1a0c6093b1326a70a45e5fffb92b53a9db9114ea..62ec0709796f72d70116c68c2943bbeb7fda6b71 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -14,7 +14,7 @@ default: $(TOP)/configs/current
 
 
 doxygen:
-       (cd doxygen ; make) ; \
+       cd doxygen && $(MAKE)
 
 clean:
        @for dir in $(SUBDIRS) ; do \
@@ -52,10 +52,10 @@ linux-directfb-install:
 
 # Xserver GLcore module
 glcore:
-       cd src/mesa/drivers/xorg ; $(MAKE)
+       cd src/mesa/drivers/xorg && $(MAKE)
 
 glcore-install:
-       cd src/mesa/drivers/xorg ; $(MAKE) install
+       cd src/mesa/drivers/xorg && $(MAKE) install
 
 .PHONY: default doxygen clean realclean install linux-directfb-install \
        glcore glcore-install
index 931e9d0cb8621f2ab29eb420ddd2bea18c1232eb..036bf1f01a17527f143bbd39e157908e9edc9f5e 100644 (file)
@@ -11,7 +11,7 @@ default: subdirs
 subdirs:
        @for dir in $(SUBDIRS) ; do \
                if [ -d $$dir ] ; then \
-                       (cd $$dir ; $(MAKE)) || exit 1 ; \
+                       (cd $$dir && $(MAKE)) || exit 1 ; \
                fi \
        done
 
@@ -19,6 +19,6 @@ subdirs:
 clean:
        @for dir in $(SUBDIRS) ; do \
                if [ -d $$dir ] ; then \
-                       (cd $$dir ; $(MAKE) clean) ; \
+                       (cd $$dir && $(MAKE) clean) ; \
                fi \
        done
index 836baa684cb730e16a822938baded5d84bcf8c60..3ee5cfddd7a10afe283f0d6bc87c443e1781c4b4 100644 (file)
@@ -10,7 +10,7 @@ SUBDIRS = $(GLU_DIRS)
 
 default: $(TOP)/configs/current
        @for dir in $(SUBDIRS) ; do \
-               (cd $$dir ; $(MAKE)) ; \
+               (cd $$dir && $(MAKE)) || exit 1 ; \
        done
 
 # GLU pkg-config file
@@ -29,5 +29,5 @@ install: glu.pc
 
 clean:
        @for dir in $(SUBDIRS) ; do \
-               (cd $$dir ; $(MAKE) clean) ; \
+               (cd $$dir && $(MAKE) clean) ; \
        done
index bd486cf81b333735529cf758d0d04c7a5e39ea55..a96e0dadaa79204e063f27612d2adeab360c4c19 100644 (file)
@@ -4,9 +4,9 @@ include $(TOP)/configs/current
 
 
 default:
-       cd mini ; $(MAKE)
+       cd mini && $(MAKE)
 
 
 clean:
-       cd mini ; $(MAKE) clean
+       cd mini && $(MAKE) clean
 
index 633bfb19a3bc69ecf2c213c08e1c41929c0e196b..5f45db1d24da0f42c12aead3bae1ea7e9517f2d8 100644 (file)
@@ -42,7 +42,7 @@ default:
 # BeOS driver target
 
 beos: depend subdirs libmesa.a
-       cd drivers/beos; $(MAKE)
+       cd drivers/beos && $(MAKE)
 
 
 ######################################################################
@@ -56,7 +56,7 @@ libmesa.a: $(SOLO_OBJECTS)
        fi
 
 linux-solo: depend subdirs libmesa.a
-       cd drivers/dri ; $(MAKE)
+       cd drivers/dri && $(MAKE)
 
 
 #####################################################################
@@ -69,7 +69,7 @@ libgl-core: $(CORE_OBJECTS)
                $(GL_LIB_DEPS)
 
 directfb: depend subdirs libgl-core
-       cd drivers/directfb ; $(MAKE)
+       cd drivers/directfb && $(MAKE)
 
 
 #####################################################################
@@ -145,11 +145,11 @@ depend: $(ALL_SOURCES)
 
 subdirs:
        @ if echo "$(ASM_FLAGS)" | grep -q USE_X86_ASM ; then \
-               (cd x86 ; $(MAKE)) ; \
+               (cd x86 && $(MAKE)) || exit 1 ; \
        fi
        @ if echo "$(ASM_FLAGS)" | grep -q USE_X86_64_ASM ; then \
-               (cd x86 ; $(MAKE)) ; \
-               (cd x86-64 ; $(MAKE)) ; \
+               (cd x86 && $(MAKE)) || exit 1 ; \
+               (cd x86-64 && $(MAKE)) || exit 1 ; \
        fi
 
 pcedit = sed \
@@ -180,7 +180,7 @@ install-osmesa:
 install-drivers:
        @for target in $(DRIVER_DIRS); do \
          case "$$target" in \
-           dri) cd drivers/dri ; $(MAKE) install ;; \
+           dri) (cd drivers/dri && $(MAKE) install) || exit 1 ;; \
          esac; \
        done
 
index 9c7d6affc3b162c103c9e3430b1286f7e7155933..07d3d344f17a22a429795b542660b9aa2c547e83 100644 (file)
@@ -170,10 +170,10 @@ OBJECTS := $(DRIVER_OBJECTS:.cpp=.o)
 default: depend $(TOP)/$(LIB_DIR) $(TOP)/$(LIB_DIR)/$(GL_LIB_NAME)
 
 $(MESA_MODULES):
-       cd $(TOP)/src/mesa; $(MAKE) mesa.a ;
+       cd $(TOP)/src/mesa && $(MAKE) mesa.a ;
 
 $(GLU_MODULES):
-       cd $(GLU_DIR); $(MAKE) $(subst $(GLU_DIR)/,,$(GLU_MODULES)) ; 
+       cd $(GLU_DIR) && $(MAKE) $(subst $(GLU_DIR)/,,$(GLU_MODULES)) ;
 
 $(TOP)/$(LIB_DIR):
        mkdir $(TOP)/$(LIB_DIR)
@@ -184,7 +184,7 @@ $(TOP)/$(LIB_DIR)/$(GL_LIB_NAME): $(OBJECTS) $(MESA_MODULES) $(GLU_MODULES)
                $(MKLIB_OPTIONS) $(GL_LIB_DEPS) $(OBJECTS) $(MESA_MODULES) $(GLU_MODULES)
 
 # $(GLU_OBJECTS):
-#      cd $(GLU_DIR); $(MAKE) $< ;
+#      cd $(GLU_DIR) && $(MAKE) $< ;
 
 depend: $(DRIVER_SOURCES) $(GLU_SOURCES)
        touch depend