cvs commit: ports/databases/adstudio Makefile distinfo

Chris Rees crees at FreeBSD.org
Tue Jun 12 17:05:17 UTC 2012


On 12 June 2012 17:51, Mikhail Teterin <mi at freebsd.org> wrote:
> mi          2012-06-12 16:51:50 UTC
>
>  FreeBSD ports repository
>
>  Modified files:
>    databases/adstudio   Makefile distinfo
>  Log:
>  Upgrade from 10.0.7 to 10.0.11. Unbreak.
>
>  Revision  Changes    Path
>  1.51      +9 -6      ports/databases/adstudio/Makefile
>  1.43      +4 -2      ports/databases/adstudio/distinfo
>


-	${TAR} -C ${DATADIR} -xpf ${DISTDIR}/${_DISTFILES}	\
+	# Extracting the base version:
+	${TAR} -C ${DATADIR} -xpf ${DISTDIR}/${EXTRACT_ONLY}	\
 		${INSTFILES:S|^|./datastudio/|}

This looks very fragile to me, since by definition EXTRACT_ONLY is a list.

It'd be better off wrapped in a .for, or use ${EXTRACT_ONLY:S/^/${DISTDIR}/}

Chris


More information about the cvs-all mailing list