Re: CVS commit: pkgsrc/pkgtools/pkg_install

[Jonathan Perkin <jperkin%joyent.com@localhost>]

* On 2015-01-21 at 23:12 GMT, Jonathan Perkin wrote:

> * On 2015-01-20 at 19:15 GMT, Adam Ciarcinski wrote:
> 
> > Module Name:        pkgsrc
> > Committed By:       adam
> > Date:               Tue Jan 20 19:15:56 UTC 2015
> > 
> > Modified Files:
> >     pkgsrc/pkgtools/pkg_install: Makefile
> > 
> > Log Message:
> > Fix building with libarchive v3; cosmetics
> 
> This change isn't correct.  -larchive is now added for all builds,
> which of course doesn't work when building the inplace libarchive:
> 
>   checking whether the C compiler works... no
>   configure: error: in `/tmp/pkgsrc-upstream-trunk64/wrk/pkgtools/pkg_install/work/libarchive':
>   configure: error: C compiler cannot create executables
>   See `config.log' for more details
> 
>   ld: fatal: library -larchive: not found
> 
> Could you test changes to any bootstrap packages with the full
> compliment of builtin vs inplace/pkgsrc options to ensure that none of
> the bootstrap types are broken.

This patch works for me (on SunOS with no builtin libarchive):

  https://us-east.manta.joyent.com/jperkin/public/patches/pkg_install-vs-libarchive.diff

Testers welcome,

-- 
Jonathan Perkin  -  Joyent, Inc.  -  www.joyent.com