[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: update for wip/libdv
On Thu, 10 Jun 2004, Min Sik Kim wrote:
> > +PKGCONFIG_OVERRIDE+= ${WRKSRC}/libdv.pc.in
>
> I think relative paths are preferred for PKGCONFIG_OVERRIDE.
I got that example as generated from url2pkg script which has:
echo "PKGCONFIG_OVERRIDE+= \${WRKSRC}/$i"
Many examples in pkgsrc do not use WRKSRC, but mk/bsd.pkg.mk has:
SUBST_FILES.pkgconfig= ${PKGCONFIG_OVERRIDE:S/^${WRKSRC}\///}
so I guess it was (or is still) alright.
> > +CONFIGURE_ARGS+= --prefix=${PREFIX}
>
> Isn't this redundant?
Sorry, I forgot to mention that my patch changes the formatting. Sorry for
so many changes in one patch. I added some tabs.
The "CONFIGURE_ARGS+= --prefix=${PREFIX}" was in the original Makefile
before I patched it.
Yes, it is redundant.
Jeremy C. Reed
technical support & remote administration
http://www.pugetsoundtechnology.com/