Date: Tue, 12 Jun 2012 18:04:45 +0100 From: Chris Rees <crees@FreeBSD.org> To: Mikhail Teterin <mi@freebsd.org> Cc: cvs-ports@freebsd.org, cvs-all@freebsd.org, ports-committers@freebsd.org Subject: Re: cvs commit: ports/databases/adstudio Makefile distinfo Message-ID: <CADLo839KPaf-9wqVEb8DcnSJHkq3318jV_dna44bprfKO92gvQ@mail.gmail.com> In-Reply-To: <201206121651.q5CGpop5037088@repoman.freebsd.org> References: <201206121651.q5CGpop5037088@repoman.freebsd.org>
index | next in thread | previous in thread | raw e-mail
On 12 June 2012 17:51, Mikhail Teterin <mi@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
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo839KPaf-9wqVEb8DcnSJHkq3318jV_dna44bprfKO92gvQ>