Re: svn commit: r494093 - in /spamassassin/trunk: INSTALL MANIFEST lib/Mail/SpamAssassin/Util/DependencyInfo.pm spamc/config.h.in spamc/configure spamc/configure.in spamc/libspamc.c spamc/libspamc.h spamc/spamc.c spamc/spamc.pod spamd/spamd.raw t/spamc_z.t

[Justin Mason <jm...@jmason.org>]

Theo Van Dinter writes:
> On Mon, Jan 08, 2007 at 03:33:54PM -0000, jm@apache.org wrote:
> > +  - Compress::Zlib (from CPAN)
> > +
> > +    If you wish to use the optional zlib compression for communication
> > +    between spamc and spamd (the -z option to spamc), useful for
> > +    long-distance use of spamc over the internet, you need to install
> > +    this module.
> 
> Just a fwiw, we already use IO::Zlib in sa-update and mass-check.  It may be
> good to be consistent about which of Compress or IO that we use.

oops. good point.

for what it's worth, it looks like IO::Zlib is just a wrapper around
Compress::Zlib, and is explicitly 'an interface to gzip/zlib compressed
files'.  I recall looking into it and deciding against it since it
seemed entirely oriented to files -- not in-memory buffers.

There may be some trick to use in-memory "files" with IO::Zlib, but
for now, I'd be more comfortable sticking with Compress::Zlib for
the spamd usage.

--j.