Re: security/clamav: Makefile patch (dependencies for archivers/xz, converters/libiconv, textproc/libxml2)

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

* On 2014-05-07 at 18:45 BST, Matthias Ferdinand wrote:

> On Wed, May 07, 2014 at 05:25:18PM +0100, Jonathan Perkin wrote:
> > * On 2014-05-07 at 17:20 BST, Matthias Ferdinand wrote:
> > 
> > > the Makefile for security/clamav is missing some buildlink-includes for
> > > dependencies. This can lead to failure on systems with no native
> > > libxml2, when e.g. pkgin does not pull in libxml2 from pkgsrc.
> > 
> > Could you show the failures please?  I do not see this on SmartOS,
> > where we do not provide any native implementations of these libraries,
> > so I'd like to know if there is some functionality we are missing out
> > on.
> 
> Then you probably have a current libxml2 from pkgsrc (or maybe clamav
> does only use libxml2 if it can find it on the build host?).

Right, it appears to be completely optional which is why I didn't see
it, so I've added it to avoid these build host vs production host
issues.

The libxml2 buildlink pulls in xz and libiconv itself, so there's no
need to add those to the clamav Makefile too.

Thanks!

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