This appears to break buildworld:

make[4]: "/usr/home/smh/freebsd/base/head/share/mk/bsd.links.mk" line 10: Wrong number of words (1) in .for substitution list with 2 vars
make[4]: Fatal errors encountered -- cannot continue
make[4]: stopped in /usr/home/smh/freebsd/base/head/usr.bin/mandoc
*** [cleandir_subdir_mandoc] Error code 1

On 15/03/2015 21:50, Baptiste Daroussin wrote:
Author: bapt
Date: Sun Mar 15 21:50:58 2015
New Revision: 280122
URL: https://svnweb.freebsd.org/changeset/base/280122

Log:
   Symplify links installation by using multi variable for loop
Using multi variable for loop not only simplify the code, it also ensures that
   the LINKS and SYMLINKS input have the right number of words
Differential Revision: https://reviews.freebsd.org/D2069
   Reviewed by: imp

Modified:
   head/share/mk/bsd.links.mk

Modified: head/share/mk/bsd.links.mk
==============================================================================
--- head/share/mk/bsd.links.mk  Sun Mar 15 21:43:43 2015        (r280121)
+++ head/share/mk/bsd.links.mk  Sun Mar 15 21:50:58 2015        (r280122)
@@ -7,25 +7,11 @@
  afterinstall: _installlinks
  .ORDER: realinstall _installlinks
  _installlinks:
-.if defined(LINKS) && !empty(LINKS)
-       @set ${LINKS}; \
-       while test $$# -ge 2; do \
-               l=${DESTDIR}$$1; \
-               shift; \
-               t=${DESTDIR}$$1; \
-               shift; \
-               ${ECHO} $$t -\> $$l; \
-               ${INSTALL_LINK} $$l $$t; \
-       done; true
-.endif
-.if defined(SYMLINKS) && !empty(SYMLINKS)
-       @set ${SYMLINKS}; \
-       while test $$# -ge 2; do \
-               l=$$1; \
-               shift; \
-               t=${DESTDIR}$$1; \
-               shift; \
-               ${ECHO} $$t -\> $$l; \
-               ${INSTALL_SYMLINK} $$l $$t; \
-       done; true
-.endif
+.for s t in ${LINKS}
+       @${ECHO} "$t -> $s" ;\
+       ${INSTALL_LINK} $s $t
+.endfor
+.for s t in ${SYMLINKS}
+       @${ECHO} "$t -> $s" ;\
+       ${INSTALL_SYMLINK} $s $t
+.endfor


_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "[email protected]"

Reply via email to