Subject: Re: CVS commit: pkgsrc/devel/patch
To: Thomas Klausner <wiz@danbala.ifoer.tuwien.ac.at>
From: grant beattie <grant@netbsd.org>
List: pkgsrc-changes
Date: 01/07/2003 12:32:14
On Tue, Jan 07, 2003 at 02:27:01AM +0100, Thomas Klausner wrote:
> > Correctly honour GNU_PROGRAM_PREFIX on non-NetBSD platforms for
> > consistency (hi wiz!)
> >
> > bump PKGREVISION.
>
> mk/defs.SunOS.mk, if /usr/bin/gpatch does not exist, uses
> ${LOCALBASE}/bin/patch -- that was the original reason why I
> didn't do it this way.
Yep, which I've now fixed, because...
> Of course, we could let the package install ${LOCALBASE}/bin/patch
> anyway, or ${LOCALBASE}/bin/gpatch, similar to what gmake does.
It now installs as gpatch by default. The change you made (if NetBSD
&& defined(GNU_PROGRAM_PREFIX)) meant that the PLIST was incorrect. I
just blew away /usr/pkg on my Solaris box, otherwise I wouldn't have
noticed.
Your intentions were obviously good, so thanks :)
grant.