diff options
| author | rillig <rillig> | 2006-10-07 12:22:06 +0000 |
|---|---|---|
| committer | rillig <rillig> | 2006-10-07 12:22:06 +0000 |
| commit | dd16ee8a172110e7e5a3d1d30eaeb25a5e8f8e31 (patch) | |
| tree | 121a7014dd1aec378cd46462bcc9ef212d49ea15 /mk/fetch | |
| parent | bc9c7286e05e52abd097c14274824fdd147d06ce (diff) | |
| download | pkgsrc-dd16ee8a172110e7e5a3d1d30eaeb25a5e8f8e31.tar.gz | |
Meta-packages that had DIST_SUBDIR set (for whatever reason) tried to
fetch files with an empty name, for example "geda/".
The problem was that the expression ${LIST:S,^,${DIST_SUBDIR}/,} results
in ${DIST_SUBDIR}/ when ${LIST} is empty. This is surprising but matches
the documentation in the manual page, so this cannot be called a bug.
The proper fix is to use ${LIST:@f@${DIST_SUBDIR}/${f}@} instead.
Noticed by Don Woodstock on #netbsd-code.
Diffstat (limited to 'mk/fetch')
| -rw-r--r-- | mk/fetch/fetch.mk | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/mk/fetch/fetch.mk b/mk/fetch/fetch.mk index 4bc443f3b8a..25b365004a7 100644 --- a/mk/fetch/fetch.mk +++ b/mk/fetch/fetch.mk @@ -1,4 +1,4 @@ -# $NetBSD: fetch.mk,v 1.20 2006/07/27 18:48:03 jlam Exp $ +# $NetBSD: fetch.mk,v 1.21 2006/10/07 12:22:06 rillig Exp $ _MASTER_SITE_BACKUP= ${MASTER_SITE_BACKUP:=${DIST_SUBDIR}${DIST_SUBDIR:D/}} _MASTER_SITE_OVERRIDE= ${MASTER_SITE_OVERRIDE:=${DIST_SUBDIR}${DIST_SUBDIR:D/}} @@ -15,10 +15,10 @@ CKSUMFILES:= ${CKSUMFILES:N${__tmp__}} # List of all files, with ${DIST_SUBDIR} in front. Used for fetch and checksum. .if defined(DIST_SUBDIR) -_CKSUMFILES?= ${CKSUMFILES:S/^/${DIST_SUBDIR}\//} -_DISTFILES?= ${DISTFILES:S/^/${DIST_SUBDIR}\//} -_IGNOREFILES?= ${IGNOREFILES:S/^/${DIST_SUBDIR}\//} -_PATCHFILES?= ${PATCHFILES:S/^/${DIST_SUBDIR}\//} +_CKSUMFILES?= ${CKSUMFILES:@.f.@${DIST_SUBDIR}/${.f.}@} +_DISTFILES?= ${DISTFILES:@.f.@${DIST_SUBDIR}/${.f.}@} +_IGNOREFILES?= ${IGNOREFILES:@.f.@${DIST_SUBDIR}/${.f.}@} +_PATCHFILES?= ${PATCHFILES:@.f.@${DIST_SUBDIR}/${.f.}@} .else _CKSUMFILES?= ${CKSUMFILES} _DISTFILES?= ${DISTFILES} |