Random makefile cleanup prior to bmake import
authorJohn Marino <draco@marino.st>
Sun, 28 Oct 2012 12:32:36 +0000 (13:32 +0100)
committerJohn Marino <draco@marino.st>
Sun, 28 Oct 2012 13:32:12 +0000 (14:32 +0100)
Bmake does not like to encounter "exit 0" or "exit 1".  It fails using the
exec command.  Replace instances of "exit 1" with /usr/bin/false.  There
was one instance of "exit 0" but it was at the end the target and thus
unnecessary.

The groff/tmac makefile was improved by providing a cd command for
every installed file.

Finally, one of the test makefiles was missing a single quote in one of
the targets.

etc/Makefile
gnu/usr.bin/groff/tmac/Makefile
sys/conf/kern.post.mk
test/vkernel/Makefile
test/x86_64/Makefile
usr.bin/getconf/Makefile

index dc44175..39c6ee9 100644 (file)
@@ -87,7 +87,7 @@ remove-obsolete-files:
        @if [ -z "${TO_REMOVE}" ]; then \
                echo "Please do a 'make installworld' first."; \
                echo "See build(7) for further information."; \
-               exit 1; \
+               /usr/bin/false; \
        fi;
        @echo "===> Remove now obsolete files"
        @for item in ${TO_REMOVE:M*.info.gz}; do \
index 6a3ccbb..004e551 100644 (file)
@@ -82,16 +82,18 @@ beforeinstall:
        cd ${.CURDIR}; \
            ${INSTALL} -o ${TMACOWN} -g ${TMACGRP} -m ${TMACMODE} \
            koi8-r.tmac hyphen.ru ${DESTDIR}${TMACDIR}
-       cd ${.OBJDIR}
 .for f in ${STRIPFILES} ${SPECIALFILES} ${NORMALFILES}
+       cd ${.OBJDIR}; \
        ${INSTALL} -o ${TMACOWN} -g ${TMACGRP} -m ${TMACMODE} \
            $f-s ${DESTDIR}${TMACDIR}/$f
 .endfor
 .for f in ${MDOCFILES}
+       cd ${.OBJDIR}; \
        ${INSTALL} -o ${TMACOWN} -g ${TMACGRP} -m ${TMACMODE} \
            $f-s ${DESTDIR}${MDOCDIR}/$f
 .endfor
 .if !exists(${DESTDIR}${TMACDIR}/man.local)
+       cd ${.OBJDIR}; \
        ${INSTALL} -o ${TMACOWN} -g ${TMACGRP} -m ${TMACMODE} \
            ${DIST_DIR}/man.local ${DESTDIR}${TMACDIR}
 .endif
index b70b016..1abecf6 100644 (file)
@@ -105,7 +105,7 @@ links:
        sh makelinks && rm -f dontlink
 
 kernel-tags:
-       @[ -f .depend ] || { echo "you must make depend first"; exit 1; }
+       @[ -f .depend ] || { echo "you must make depend first"; /usr/bin/false; }
        sh $S/conf/systags.sh
        rm -f tags1
        sed -e 's,      ../,    ,' tags > tags1
@@ -119,7 +119,7 @@ kernel-tags:
 kernel-install: kernel-installable
        @if [ ! -f ${SELECTEDKERNEL} ]; then                    \
                echo "You must build a kernel first.";          \
-               exit 1;                                         \
+               /usr/bin/false;                                 \
        fi
        @if [ -f ${DESTDIR}${DESTKERNDIR}.old ]; then           \
                rm -f ${DESTDIR}${DESTKERNDIR}.old;             \
@@ -163,7 +163,7 @@ kernel-installable:
                echo "You need to make buildworld, installworld, and upgrade"; \
                echo "before you can install a new kernel, because the"; \
                echo "kernel and modules have moved to /boot"; \
-               exit 1; \
+               /usr/bin/false; \
        fi
 # Skip this step for vkernels
 .if ${MACHINE_PLATFORM} != vkernel && ${MACHINE_PLATFORM} != vkernel64
@@ -173,10 +173,9 @@ kernel-installable:
                echo "into a subdirectory along with their modules."; \
                echo "You can do this with a buildworld / installworld"; \
                echo "sequence."; \
-               exit 1; \
+               /usr/bin/false; \
        fi
 .endif
-       @exit 0
 
 .if !defined(MODULES_WITH_WORLD) && !defined(NO_MODULES)
 all:   modules
index 4bcf5a2..2ea451e 100644 (file)
@@ -132,7 +132,7 @@ install: mount
 
 leaf: mount
        echo 'ifconfig_vke0="up"' > ${VKDIR}/root/etc/rc.conf
-       echo 'defaultrouter="10.26.0.1" >> ${VKDIR}/root/etc/rc.conf
+       echo 'defaultrouter="10.26.0.1"' >> ${VKDIR}/root/etc/rc.conf
 
 # Quick reinstall - just install a new kernel on top of an existing image
 #
@@ -179,7 +179,7 @@ run_nfsroot: check
 #
 checkq:
 .if !defined(VKDIR)
-       @(echo "must specify VKDIR=target or as an environment variable"; exit 1)
+       @(echo "must specify VKDIR=target or as an environment variable"; /usr/bin/false)
 .endif
 .if exists(${VKDIR})
        @echo "${VKDIR} found"
@@ -194,8 +194,8 @@ check: checkq
        rm -f ${VKDIR}/vn.which
 .endif
        (vnconfig -l | fgrep "not in use" > /dev/null) || \
-           (echo "Cannot find unused VN"; exit 1)
+           (echo "Cannot find unused VN"; /usr/bin/false)
        vnconfig -l | fgrep "not in use" | \
            cut -f 1 -d : | head -1 > ${VKDIR}/vn.which
        egrep '^vn' ${VKDIR}/vn.which > /dev/null || \
-           (echo "VN device selection is bad"; exit 1)
+           (echo "VN device selection is bad"; /usr/bin/false)
index bed4697..0dc0e71 100644 (file)
@@ -223,7 +223,7 @@ run: check
 
 xrun:
 .if !defined(DISPLAY)
-       @(echo "Variable DISPLAY not set. Is X running?"; exit 1)
+       @(echo "Variable DISPLAY not set. Is X running?"; /usr/bin/false)
 .endif
        cd ${QEMUDIR} && qemu-system-x86_64 -hda ${QEMUDIR}/root.img \
                                            -m 128 -no-fd-bootchk
@@ -245,7 +245,7 @@ gdb64:
 #
 checkq:
 .if !defined(QEMUDIR)
-       @(echo "must specify QEMUDIR=target or as an environment variable"; exit 1)
+       @(echo "must specify QEMUDIR=target or as an environment variable"; /usr/bin/false)
 .endif
 .if exists(${QEMUDIR})
        @echo "${QEMUDIR} found"
@@ -260,8 +260,8 @@ check: checkq
        rm -f ${QEMUDIR}/vn.which
 .endif
        (vnconfig -l | fgrep "not in use" > /dev/null) || \
-           (echo "Cannot find unused VN"; exit 1)
+           (echo "Cannot find unused VN"; /usr/bin/false)
        vnconfig -l | fgrep "not in use" | \
            cut -f 1 -d : | head -1 > ${QEMUDIR}/vn.which
        egrep '^vn' ${QEMUDIR}/vn.which > /dev/null || \
-           (echo "VN device selection is bad"; exit 1)
+           (echo "VN device selection is bad"; /usr/bin/false)
index 11b52d7..2e2f0d1 100644 (file)
@@ -25,7 +25,7 @@ all:  conflicts
 conflicts: conflicting.names unique.names
        @if test `wc -l <conflicting.names` != `wc -l <unique.names`; then \
                echo "Name conflicts found!" >&2; \
-               exit 1; \
+               /usr/bin/false; \
        fi
 
 # pathconf.names is not included here because pathconf names are