Re: CVS commit: pkgsrc/mk

[David Holland <dholland-pkgchanges%netbsd.org@localhost>]

On Sun, Jul 29, 2012 at 03:24:34PM +0100, Jonathan Perkin wrote:
 > >  > Modified Files:
 > >  >         pkgsrc/mk/fetch: bsd.fetch-vars.mk fetch.mk
 > >  >         pkgsrc/mk/tools: replace.mk tools.SunOS.mk
 > >  > 
 > >  > Log Message:
 > >  > Make curl a tool.  Add support for /usr/bin/curl, if available, on 
 > > Solaris.
 > >  > 
 > >  > SmartOS and OmniOS users can now set FETCH_USING=curl and no longer 
 > > require
 > >  > pkgsrc dependencies for downloading distfiles.
 > > 
 > > Shouldn't you also add a builtin.mk to www/curl?
 > 
 > For what purpose?  If for libcurl then arguably it should be a separate
 > package first.

Well, there are a number of packages that depend on it. It seems like
a bad idea to have two copies installed and use one for some things
and the other for others.

I have no idea to what extent these packages link with libcurl vs.
invoke the binary, but I'm not convinced it matters.

-- 
David A. Holland
dholland%netbsd.org@localhost