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.