Bug#506703: new version (0.1.22) for amavis-stats

[Gokdeniz Karadag <gokdenizk@gmail.com>]

Hi,

I have just had the time to test the package. Thanks for providing it. My
report will be rather long, sorry for it. But at the end the thing works now :D




If I remember correctly, amavis-stats prefixes itself to /usr while building.
And It seems that the package is built with --prefix=/usr.
In line 1120 of /usr/sbin/amavis-stats it looks for config file at
'/etc/amavis-stats/amavis-stats.conf'. Also, it installs
/usr/var/cache/amavis-stats and /usr/var/lib/amavis-stats directories.

There must be a configure option for the prefix.

Also it puts files in "/usr/doc", no other package does this according to
debian policy. and /usr/doc/amavis-stats.txt is in fact a man file -with
horrible formatting :(
This program really sucks but there is no ready-made alternative.


The data directories (/var/{lib,cache}/amavis-stats) must be www-data owned.
And I cannot remember what original permissions were on /var/log/mail.info, but
amavis-stats must be able to read it.


The new version defaults in a daemon-like operation. So the cron.d file is
unnecessary, instead an initscript must be installed, that will directly call
/usr/sbin/amavis-stats with start|stop etc. Be aware that process renames
itself as `Amavis-Stats`


/usr/share/amavis-stats/amavis-stats.alias.conf  is similar to the apache.conf,
with added expiration instructions, it should not be present in installed package.

In my setup, /etc/amavis-stats/apache.conf was not linked under apache's conf.d

Also I had to fix /usr/share/amavis-stats/img link, as the current package has
an extra "/usr/" prefix in all directories, in the corrected version, that link
should be pointing to correct place.

Some dependencies are missing; a dep on php and php-gd must be present for php
files. I am not very comfortable with this. People do not install web servers
on their mail servers. I would not prefer such a dep, instead I copy rrd files
 over to a apache installed machine. Can you provide a backend package that
produces rrd files, and a frontend that produces graphics ?

After all these adjustments, I managed to get graphics of virus, spam and other
activity. Yay!
Actually, the "business1" theme seems like professional thingy :)

Thanks for your efforts.

Barry deFreese demis ki::
> Barry deFreese wrote:
>> Gokdeniz Karadag wrote:
>>> Hi,
>>>
>>> I can test the package. Where can I find it ?
>>>
>>> - Gokdeniz Karadag
>>>
>>> Barry deFreese demis ki::
>>>  
>>>> Hi,
>>>>
>>>> I have a package of 0.1.22 ready to go but I am not overly familiar
>>>> with
>>>> using it.  Is there any chance you could do some testing for me?
>>>>
>>>> Thanks!
>>>>
>>>> Barry deFreese
>>>> Debian QA
>>>>     
>>>
>>>   
>> Gokdeniz,
>>
>> Thanks.  I have put the .deb here for now: 
>> http://people.debian.org/~bdefreese/amavis-stats/amavis-stats_0.1.22-1_all.deb
>>
>>
>> Unfortunately it isn't apt-gettable so you will have to wget and use
>> dpkg -i.  If you don't know how to do that, please let me know.  I'm
>> sure there is some extraneous stuff in the package since the way it is
>> configured seems to have changed significantly.  In fact the
>> apache.conf file that is shipped is probably no good anymore as well
>> as the cronjob probably needs modification.  Any help/suggestions you
>> could give there would be greatly appreciated.
>>
>> Thanks,
>>
>> Barry
>>
> Gokdeniz,
> 
> Hi, sorry to bug you, just curious if you've had a chance to test this
> version out?
> 
> Thanks!
> 
> Barry deFreese