Bug#300996: ITP: clamassassin -- simple wrapper for clamav
On Wed, 23 Mar 2005, Nick Price wrote:
> > > Description : clamassassin is a simple virus filter wrapper for
> > > clamav for use in procmail filters and similar applications.
> > >
> > > clamassassin is a simple virus filter wrapper for clamav for use
> > > inprocmail filters and similar applications. Its interface is similar
> > > to that of spamassassin, making it easy to implement for those
> > > familiar with that tool. clamassassin is designed with an emphasis on
> > > security, robustness, and simplicity.
> >
> > I was going to package this but the source tarball does not include
> > configure.ac. Please ask the author to include it, so that the source
> > is really complete.
>
> I doubt if this is necessary seeing as clamassassin is just a shell
> script. configure works as it should, and make, and make install both
> work as they should. Is there something I'm missing?
The configure script is not to be modified directly. The "preferred"
method to make modifications is by changing configure.ac and then
running autoconf. If there is no configure.ac, the source is incomplete.
Anyway, I wish you don't use the configure script directly, as it
a) produces a clamassassin script with lots of hardcoded paths here
and there, which is evil (there is a reason why we have the PATH variable).
and
b) would make the package to build-depend on clamav and procmail.
Those build-dependencies would be gratuitous.
In case it helps, this is what I was using, you may consider it public
domain:
#!/bin/sed -f
s%@CONF_TMP@%/tmp%
s%@CONF_SUBJECTHEAD@%%
s%@CONF_FORMAIL@%formail%
s%@CONF_CLAMSCANNER@%clamscan%
s%@CONF_CLAMSCANOPT@%--no-summary --stdout --mbox%
s%@CONF_ADDSCANNERFLAG@%1%
s%@CONF_SIGTOOL@%sigtool%
s%@CONF_SIGLOC@%/var/lib/clamav%
s%@CONF_SIGVERSFLAG@%0%
s%@CONF_MKTEMP@%mktemp%
s%@CONF_RM@%rm%
s%@CONF_CAT@%cat%
s%@CONF_SED@%sed%
s%@CONF_ECHO@%echo%
s%@PACKAGE_STRING@%clamassassin 1.2.2%
This is to be run on clamassassin.in to create clamassassin.
No useless build-dependencies, and no hardcoded paths like RM=/bin/rm.
Reply to: