[Serdev] [PATCH] stop build on error

[Andrei Pelinescu-Onciul]

On May 21, 2008 at 09:57, Marcus Better <marcus at better.se> wrote:
> Make sure the build stops on compile errors.

Thanks, but I'm not going to apply it to 2.0 since its frozen (I wouldn't want
to retest it) and 2.1 already has it (it can be even turned off with make
err_fail=0).

Andrei

> 
> --- a/Makefile
> +++ b/Makefile
> @@ -279,7 +279,7 @@ print-modules:
>  
>  .PHONY: modules
>  modules:
> -       - at for r in $(modules) "" ; do \
> +       @set -e; for r in $(modules) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         echo  "" ; \
>                         echo  "" ; \
> @@ -289,7 +289,7 @@ modules:
>  
>  $(extra_objs):
>         - at echo "Extra objs: $(extra_objs)" 
> -       - at for r in $(static_modules_path) "" ; do \
> +       @set -e; for r in $(static_modules_path) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         echo  "" ; \
>                         echo  "Making static module $r" ; \
> @@ -299,7 +299,7 @@ $(extra_objs):
>  
>  .PHONY: utils
>  utils:
> -       - at for r in $(utils_compile) "" ; do \
> +       @set -e; for r in $(utils_compile) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         echo  "" ; \
>                         echo  "" ; \
> @@ -382,7 +382,7 @@ sunpkg:
>  
>  .PHONY: modules-doc
>  modules-doc:
> -       - at for r in $(modules) "" ; do \
> +       @set -e; for r in $(modules) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         echo  "" ; \
>                         echo  "" ; \
> @@ -446,7 +446,7 @@ install-bin: $(bin-prefix)/$(bin-dir) 
>                 $(INSTALL-BIN) ser $(bin-prefix)/$(bin-dir)
>  
>  install-modules: modules $(modules-prefix)/$(modules-dir)
> -       - at for r in $(modules_full_path) "" ; do \
> +       @set -e; for r in $(modules_full_path) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         if [ -f "$$r" ]; then \
>                                 $(INSTALL-TOUCH) \
> @@ -459,7 +459,7 @@ install-modules: modules $(modules-prefi
>         done 
>  
>  install-utils: utils $(bin-prefix)/$(bin-dir)
> -       - at for r in $(utils_install) "" ; do \
> +       @set -e; for r in $(utils_install) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         if [ -f "$$r" ]; then \
>                                 $(INSTALL-TOUCH) \
> @@ -490,7 +490,7 @@ install-doc: $(doc-prefix)/$(doc-dir) in
>  
>  
>  install-modules-doc: $(doc-prefix)/$(doc-dir)
> -       - at for r in $(modules_basenames) "" ; do \
> +       @set -e; for r in $(modules_basenames) "" ; do \
>                 if [ -n "$$r" ]; then \
>                         if [ -f modules/"$$r"/README ]; then \
>                                 $(INSTALL-TOUCH)  $(doc-prefix)/$(doc-dir)/README ; \
> 
> 
> _______________________________________________
> Serdev mailing list
> Serdev at lists.iptel.org
> http://lists.iptel.org/mailman/listinfo/serdev