On 2013/06/21 11:02, Brian Callahan wrote:
> On 6/21/2013 10:53 AM, Brian Callahan wrote:
> >
> >Almost there...
> >If DISTNAME=PKGNAME, then you don't need PKGNAME. Also, you don't really
> >need V since you only use it one, so you can compress those three lines
> >into one:
> >DISTNAME =    the_silver_searcher-0.14
> >
> >You forgot to replace devel/autoconf/2.59 and devel/automake/1.9 with
> >their respective ${MODGNU_AUTOCONF_DEPENDS} and
> >${MODGNU_AUTOMAKE_DEPENDS} as per sthen@

why are these still needed if the rolled tarball includes a configure script?


> >I also like tedu@'s suggestion of renaming it to silver-searcher, but if
> >you're dead set against it I'm not going to argue.
> >
> >FWIW, with those changes it works for me on amd64, with admittedly light
> >testing.
> 
> On second thought, I'm not entirely sure pcre is a BUILD_DEP.
> 

that should just be in LIB_DEPENDS.

Reply via email to