[Pkg-clamav-devel] package status for jessie

[Sebastian Andrzej Siewior]

On Sat, Aug 23, 2014 at 05:07:48PM +0200, Andreas Cadhalpun wrote:
> Hi Sebastian,
Hi Andreas,

> >>Perhaps it would be better to use ftruncate64.
> >
> >Yes, but this is nothing you should specify. The define should do it. So just
> >digged a little into it.
> 
> We could define it:
> #define ftruncate ftruncate64

no, since it wouldn't help.

> >I've been looking where that truncate() user is coming from and I did not find
> >a single user in the source.
> 
> Did you use codesearch? It finds 4 uses of ftruncate [1].

I search for truncate() not for ftruncate() exactly for that reason (because
there are ftruncate() users in tree).

> >Looking at the library I've found that the user is
> >the resize_file() function from llvm [0]. This is where it got interresting
> >since we should use the external library.
> >Looking at the rules file I noticed that
> >	-with-system-llvm=/usr/bin/llvm-config
> >is used. I would *assume* that --with-sys… should be used instead. However the
> >configure output is the same.
> 
> It is a long option, so it should have two dashes, but it works with one as
> well.
Interresting.

> >Anyway, it looks like we linking llvm staticly instead of using the external
> >library. This isn't on purpose right?
> 
> I think it is on purpose, because clamav upstream didn't want to ship shared
> LLVM libraries...
This makes sense as long as you use the in-tree llvm code.

> But for Debian it would be better, if it was dynamically linked. (Otherwise
> we don't benefit from bug fixes in LLVM minor versions without recompiling
> clamav.)

Good. So I guess that there is a -static switch which we are getting rid of.
Otherwise I can't explain it :)
Do you know by any chance what is the progress by upstream using external llvm?

> >The external llvm was introduced in 0.98.1+dfsg-6 and for the i386 build
> >   11074880 Apr 24 06:52 0.98.1+dfsg-5/usr/lib/libclamav.so.6.1.20
> >   11823364 May  3 06:05 0.98.1+dfsg-6/usr/lib/libclamav.so.6.1.20
> >
> >the library gained approx 730KiB. I guess this are the static llvm files.
> 
> I would say this is the size increase in the static LLVM files between
> version 2.8 and 3.3. The symbols detected by lintian as non-LFS have been
> present in 0.98.1+dfsg-5 as well.

Yup. Good old lintian :)

> Best regards,
> Andreas

Sebastian