tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

imake templates don't stop on error



While investigating why the xview-clients package is broken, I
discovered that one of the problems is that the imake templates are
wrong. For example,

        for i in dirs ;\                                                @@\
        do \                                                            @@\
                echo verb "in $(CURRENT_DIR)/$$i..."; \                 @@\
                (cd $$i && LANG=lang $(MAKE) $(MFLAGS) flags subname);\ @@\
        done

does not stop on error; the standard for sh -e says that failing ()
constructs do not cause exits, so "|| exit $?" is required.

Investigation shows a lot of other uses of (). The patch below is
against the pkgsrc xorg version of Imake.rules. I haven't tackled any
of the other files, but I think this should cover most of the most
important cases.

(1) Can someone eyeball this for stupid mistakes? I've tested at least
the basic subdir rule, but much of this code probably only comes into
play very rarely; trying to test it all isn't really feasible.

(2) Is it ok to commit this to the pkgsrc imake templates
(x11/xorg-cf-files)?

(3) Can someone equipped to run that build deal with the xorg-in-base
version?

(4) Is it worth merging this into the old xfree tree?

[note I'm not currently reading tech-x11, so please maintain cc accordingly]


--- Imake.rules~        2005-11-08 01:33:24.000000000 -0500
+++ Imake.rules 2008-08-30 11:57:32.000000000 -0400
@@ -534,9 +534,9 @@
                set +x; \                                               @@\
        else \                                                          @@\
                if [ -h dir ]; then \                                   @@\
-                       (set -x; rm -f dir); \                          @@\
+                       (set -x; rm -f dir) || exit $?; \               @@\
                fi; \                                                   @@\
-               (set -x; $(MKDIRHIER) dir); \                           @@\
+               (set -x; $(MKDIRHIER) dir) || exit $?; \                @@\
        fi
 #endif /* MakeDir */
 
@@ -545,9 +545,9 @@
                :; \                                                    @@\
        else \                                                          @@\
                if [ -h dir ]; then \                                   @@\
-                       (set -x; rm -f dir); \                          @@\
+                       (set -x; rm -f dir) || exit $?; \               @@\
                fi; \                                                   @@\
-               (set -x; $(MKDIRHIER) dir); \                           @@\
+               (set -x; $(MKDIRHIER) dir) || exit $?; \                @@\
        fi
 #endif /* MakeDirInline */
 
@@ -571,7 +571,7 @@
        @(set -x; cd $(BUILDINCDIR) && for i in srclist; do \           @@\
                RemoveFile($$i); \                                      @@\
                $(LN) $(BUILDINCTOP)/$(CURRENT_DIR)/$$i .; \            @@\
-       done)
+       done) || exit $?
 #endif /* BuildIncludesTop */
 
 #ifndef BuildIncludes
@@ -581,7 +581,7 @@
        @(set -x; cd $(BUILDINCDIR)/dstsubdir && for i in srclist; do \ @@\
                RemoveFile($$i); \                                      @@\
                $(LN) $(BUILDINCTOP)/dstupdir/$(CURRENT_DIR)/$$i .; \   @@\
-       done)
+       done) || exit $?
 #endif /* BuildIncludes */
 
 /*
@@ -604,7 +604,7 @@
        @(set -x; cd $(BUILDINCDIR)/dstsubdir && for i in srclist; do \ @@\
                RemoveFile($$i); \                                      @@\
                $(LN) $(BUILDINCTOP)/dstupdir/$(CURRENT_DIR)/$$i .; \   @@\
-       done)
+       done) || exit $?
 #endif /* BuildInterfaces */
 
 /*
@@ -1302,7 +1302,7 @@
        RemoveFile($${TMP}); \                                          @@\
        echo .so `basename destdir`/file.$(MANSUFFIX) > $${TMP}; \      @@\
        for i in aliases; do (set -x; \                                 @@\
-       $(INSTALL) $(INSTALLFLAGS) $(INSTMANFLAGS) $${TMP} 
$(DESTDIR)destdir/$$i.$(MANSUFFIX)); \ @@\
+       $(INSTALL) $(INSTALLFLAGS) $(INSTMANFLAGS) $${TMP} 
$(DESTDIR)destdir/$$i.$(MANSUFFIX)) || exit $?; \ @@\
        done; \                                                         @@\
        RemoveFile($${TMP}))
 #endif /* InstallManPageAliases */
@@ -1337,7 +1337,7 @@
        RemoveFile($${TMP}); \                                          @@\
        echo .so `basename destdir`/file.suffix > $${TMP}; \            @@\
        for i in aliases; do (set -x; \                                 @@\
-       $(INSTALL) $(INSTALLFLAGS) $(INSTMANFLAGS) $${TMP} 
$(DESTDIR)destdir/$$i.suffix); \ @@\
+       $(INSTALL) $(INSTALLFLAGS) $(INSTMANFLAGS) $${TMP} 
$(DESTDIR)destdir/$$i.suffix) || exit $?; \ @@\
        done; \                                                         @@\
        RemoveFile($${TMP}))
 #endif /* InstallGenManPageAliases */
@@ -1397,8 +1397,8 @@
        for i in aliases; do (set -x; \                                 @@\
        $(RM) $(DESTDIR)destdir/$$i.*; \                                @@\
        (cd $(DESTDIR)destdir; $(LN) file.$${SUFFIX} \                  @@\
-       $$i.$${SUFFIX})); \                                             @@\
-       done)
+       $$i.$${SUFFIX}) || exit $?) || exit $?; \                       @@\
+       done) || exit $?
 #endif
 
 #ifndef InstallManPageAliases
@@ -1499,7 +1499,8 @@
        for i in aliases; do (set -x; \                                 @@\
          RemoveFile($(DESTDIR)$(DOCHTMLDIR)/$$i.$$SUF.html); \         @@\
          (cd $(DESTDIR)$(DOCHTMLDIR); \                                @@\
-           $(LN) file.$$SUF.html $$i.$$SUF.html)); \                   @@\
+           $(LN) file.$$SUF.html $$i.$$SUF.html) || exit $? \          @@\
+       ) || exit $?; \                                                 @@\
        done
 #else
 #define InstallHtmlManPageAliases(file,aliases,suffix) /**/
@@ -1681,7 +1682,7 @@
 #ifndef MakeNamedTargetSubdir
 #define MakeNamedTargetSubdir(dir,flags,subname)\
                (cd dir && $(MAKE) $(MFLAGS) $(PARALLELMFLAGS) \        @@\
-                       flags subname)
+                       flags subname) || exit $?
 #endif
 
 /*
@@ -1692,7 +1693,7 @@
 step::                                                                 @@\
        @MakeFlagsToShellFlags(i,set +e); \                             @@\
        echo "  cd" dir; cd dir && \                                    @@\
-       for i in list; do (set -x; $(LN) -f sub/$$i .); done
+       for i in list; do (set -x; $(LN) -f sub/$$i .) || exit $?; done
 #endif
 
 
@@ -1809,7 +1810,7 @@
        MakeDir($(DESTDIR)dest)                                         @@\
        @MakeFlagsToShellFlags(i,set +e); \                             @@\
        for i in list; do \                                             @@\
-               (set -x; $(INSTALL) $(INSTALLFLAGS) flags $$i $(DESTDIR)dest); 
\        @@\
+               (set -x; $(INSTALL) $(INSTALLFLAGS) flags $$i $(DESTDIR)dest) 
|| exit $?; \     @@\
        done
 #endif /* InstallMultipleDestFlags */
 
@@ -1876,7 +1877,7 @@
        MakeDir($(DESTDIR)dest)                                         @@\
        @MakeFlagsToShellFlags(i,set +e); \                             @@\
        for i in list; do \                                             @@\
-         (set -x; $(INSTALL) $(INSTALLFLAGS) $(INSTMANFLAGS) $$i.suffix 
$(DESTDIR)dest/$$i.$(MANSUFFIX)); \    @@\
+         (set -x; $(INSTALL) $(INSTALLFLAGS) $(INSTMANFLAGS) $$i.suffix 
$(DESTDIR)dest/$$i.$(MANSUFFIX)) || exit $?; \ @@\
        done
 #endif /* InstallMultipleManSuffix */
 
@@ -2601,7 +2602,8 @@
 #define LinkBuildLibraryMaybe(lib,doit) MakeDir($(BUILDLIBDIR))                
@@\
        @if doit; then (set -x; \                                       @@\
          RemoveFile($(BUILDLIBDIR)/lib); \                             @@\
-         cd $(BUILDLIBDIR) && $(LN) $(BUILDLIBTOP)/$(CURRENT_DIR)/lib .); fi
+         cd $(BUILDLIBDIR) && \                                        @@\
+           $(LN) $(BUILDLIBTOP)/$(CURRENT_DIR)/lib .) || exit $?; fi
 #endif
 #endif /* LinkBuildLibraryMaybe */
 
@@ -2615,7 +2617,9 @@
 #else
 #define LinkBuildSonameLibrary(lib) MakeDirInline($(BUILDLIBDIR));\    @@\
        (set -x; RemoveFile($(BUILDLIBDIR)/lib);\                       @@\
-       (cd $(BUILDLIBDIR) && $(LN) $(BUILDLIBTOP)/$(CURRENT_DIR)/lib .))
+         (cd $(BUILDLIBDIR) && \                                       @@\
+           $(LN) $(BUILDLIBTOP)/$(CURRENT_DIR)/lib .) || exit $? \     @@\
+       ) || exit $?
 #endif
 #endif /* LinkBuildSonameLibrary */
 
@@ -2713,9 +2717,9 @@
                                                                        @@\
 LibraryTargetName(libname): objlist1 objlist2 objlist3 $(EXTRALIBRARYDEPS)     
@@\
        RemoveFile($@)                                                  @@\
-       (cd down && MakeLibrary(up/$@,objlist1))                        @@\
-       (cd down && MakeLibrary(up/$@,objlist2))                        @@\
-       (cd down && MakeLibrary(up/$@,objlist3))                        @@\
+       (cd down && MakeLibrary(up/$@,objlist1)) || exit $?             @@\
+       (cd down && MakeLibrary(up/$@,objlist2)) || exit $?             @@\
+       (cd down && MakeLibrary(up/$@,objlist3)) || exit $?             @@\
        RanLibrary($@)                                                  @@\
        _LinkBuildLibrary($@)
 #endif /* UnsharedLibraryTarget3 */
@@ -2952,7 +2956,7 @@
        @-(list=`echo srclist | sed -e 's/[^ ]*\///g'`; \               @@\
                set -x; cd dir && RemoveFiles($$list))                  @@\
        @for i in srclist; do \                                         @@\
-               (set -x; cd dir && $(LN) ../$$i .); \                   @@\
+               (set -x; cd dir && $(LN) ../$$i .) || exit $?; \        @@\
        done                                                            @@\
                                                                        @@\
 dir::                                                                  @@\
@@ -2974,8 +2978,9 @@
        @MakeFlagsToShellFlags(ik,set +e); \                            @@\
        for i in dirs ;\                                                @@\
        do \                                                            @@\
-               echo verb "in $(CURRENT_DIR)/$$i..."; \                 @@\
-               (cd $$i && LANG=lang $(MAKE) $(MFLAGS) flags subname);\ @@\
+           echo verb "in $(CURRENT_DIR)/$$i..."; \                     @@\
+           (cd $$i && \                                                @@\
+              LANG=lang $(MAKE) $(MFLAGS) flags subname) || exit $?; \ @@\
        done
 #endif
 
@@ -3384,7 +3389,7 @@
 step::                                                                 @@\
        DirFailPrefix@MakeFlagsToShellFlags(i,set +e); \                @@\
        for i in dirs; do if [ -d $(DESTDIR)$$i ]; then \               @@\
-               set +x; else (set -x; $(MKDIRHIER) $(DESTDIR)$$i); fi; \ @@\
+               set +x; else (set -x; $(MKDIRHIER) $(DESTDIR)$$i) || exit $?; 
fi; \ @@\
        done
 #endif /* MakeDirectories */
 
@@ -3610,7 +3615,7 @@
        @$(CP) Concat3(libpath,/lib,libname.a) tmp                      @@\
        @(cd tmp; set -x; ArExtCmd Concat(lib,libname.a); \             @@\
          $(LD) $(LDCOMBINEFLAGS) *.Osuf -o ../$@; \                    @@\
-         RemoveFiles(*.Osuf))                                          @@\
+         RemoveFiles(*.Osuf)) || exit $?                               @@\
        @$(RM) -r tmp                                                   @@\
                                                                        @@\
 clean::                                                                        
@@\


-- 
David A. Holland
dholland%netbsd.org@localhost


Home | Main Index | Thread Index | Old Index