Subject: pkg/37148: _CHECKSUM_CMD not defined if NO_CHECKSUM="yes"
To: None <pkg-manager@netbsd.org, gnats-admin@netbsd.org,>
From: None <lacombar@gmail.com>
List: pkgsrc-bugs
Date: 10/18/2007 23:35:00
>Number:         37148
>Category:       pkg
>Synopsis:       _CHECKSUM_CMD not defined if NO_CHECKSUM="yes"
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Oct 18 23:35:00 +0000 2007
>Originator:     Arnaud Lacombe
>Release:        -current
>Organization:
>Environment:
4.99.32
>Description:
if NO_CHECKSUM="yes" is defined in Makefile and that patch exists, the call to ${_CHECKSUM_CMD} in `../patch/patch.mk:300` will result in:
-p: command not found
>How-To-Repeat:
try to install pkg which defines NO_CHECKSUM and have patch
>Fix:
define _CHECKSUM_CMD whether or not NO_CHECKSUM is defined:

cvs diff: Diffing .
Index: bsd.checksum-vars.mk
===================================================================
RCS file: /data/netbsd/cvsroot/pkgsrc/mk/checksum/bsd.checksum-vars.mk,v
retrieving revision 1.1
diff -u -r1.1 bsd.checksum-vars.mk
--- bsd.checksum-vars.mk        13 Jul 2006 14:02:34 -0000      1.1
+++ bsd.checksum-vars.mk        18 Oct 2007 22:31:11 -0000
@@ -10,3 +10,10 @@
 #
 
 DISTINFO_FILE?=                ${PKGDIR}/distinfo+
+_CHECKSUM_CMD=                                                        \
+       ${SETENV} DIGEST=${TOOLS_DIGEST:Q} CAT=${TOOLS_CAT:Q}          \
+               ECHO=${TOOLS_ECHO:Q} SED=${TOOLS_SED:Q}                \
+               TEST=${TOOLS_TEST:Q}                                   \
+       ${SH} ${PKGSRCDIR}/mk/checksum/checksum                        \
+
Index: checksum.mk
===================================================================
RCS file: /data/netbsd/cvsroot/pkgsrc/mk/checksum/checksum.mk,v
retrieving revision 1.12
diff -u -r1.12 checksum.mk
--- checksum.mk 9 Oct 2007 19:19:13 -0000       1.12
+++ checksum.mk 18 Oct 2007 22:28:31 -0000
@@ -26,12 +26,6 @@
 #
 _COOKIE.checksum=      ${_COOKIE.extract}
 
-_CHECKSUM_CMD=                                                        \
-       ${SETENV} DIGEST=${TOOLS_DIGEST:Q} CAT=${TOOLS_CAT:Q}          \
-               ECHO=${TOOLS_ECHO:Q} SED=${TOOLS_SED:Q}                \
-               TEST=${TOOLS_TEST:Q}                                   \
-       ${SH} ${PKGSRCDIR}/mk/checksum/checksum                        \
- 
checksum checksum-phase:
        ${RUN} set -e;                                                 \
        case ${.TARGET:Q} in                                           \