[src/trunk]: src/share/mk In UUDECODE_FILES, always use -p to a temporary fil...

[lukem <lukem%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/238b6c787cbb
branches:  trunk
changeset: 566686:238b6c787cbb
user:      lukem <lukem%NetBSD.org@localhost>
date:      Mon May 17 12:58:17 2004 +0000

description:
In UUDECODE_FILES, always use -p to a temporary file and then rename
appropriately.  This simplifies the rule (IMHO), and removes the use
of :? which is buggy on older makes...

diffstat:

 share/mk/bsd.files.mk |  10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diffs (27 lines):

diff -r 17d07ecc963b -r 238b6c787cbb share/mk/bsd.files.mk
--- a/share/mk/bsd.files.mk     Mon May 17 12:17:18 2004 +0000
+++ b/share/mk/bsd.files.mk     Mon May 17 12:58:17 2004 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: bsd.files.mk,v 1.36 2004/05/13 13:05:53 lukem Exp $
+#      $NetBSD: bsd.files.mk,v 1.37 2004/05/17 12:58:17 lukem Exp $
 
 .if !defined(_BSD_FILES_MK_)
 _BSD_FILES_MK_=1
@@ -121,13 +121,13 @@
 
 .uue:
        ${_MKTARGET_CREATE}
-       rm -f ${.TARGET}
-       ${TOOL_UUDECODE} ${UUDECODE_FILES_RENAME_${.TARGET}:?-p:} ${.IMPSRC} ${UUDECODE_FILES_RENAME_${.TARGET}:?>:} ${UUDECODE_FILES_RENAME_${.TARGET}:U}
-       ${UUDECODE_FILES_RENAME_${.TARGET}:?touch ${.TARGET}:@true}
+       rm -f ${.TARGET} ${.TARGET}.tmp
+       ${TOOL_UUDECODE} -p ${.IMPSRC} > ${.TARGET}.tmp \
+           && mv ${.TARGET}.tmp ${UUDECODE_FILES_RENAME_${.TARGET}:U${.TARGET}}
 
 realall: ${UUDECODE_FILES}
 
-CLEANUUDECODE_FILES=${UUDECODE_FILES}
+CLEANUUDECODE_FILES=${UUDECODE_FILES} ${UUDECODE_FILES:=.tmp}
 .for i in ${UUDECODE_FILES}
 CLEANUUDECODE_FILES+=${UUDECODE_FILES_RENAME_${i}}
 .endfor