Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: webalizer


https://bugzilla.redhat.com/show_bug.cgi?id=226536


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |ASSIGNED
               Flag|needinfo?                   |




------- Additional Comments From [EMAIL PROTECTED]  2008-02-09 06:18 EST -------
Not really.

I had a more precise look at the code/output, and it seems that
the 2 images shipped in the package are not used. webalizer.png
is not used at all, while msfree.png is used as an example in a
commented out section of the config file. So I think that they should
definitively be in %doc if shipped at all.

The issue of webalizer running as soon as it is installed is still
an issue in my opinion. Just like webalizer installing /var/www/usage
and filling it as soon as it is installed. There are many ways to fix
that. I see 2 possibilities:

* have a separate webalizer-cron package which owns /var/www/usage
  and /etc/cron.daily/00webalizer /etc/httpd/conf.d/webalizer.conf
  with a very clear %description stating that it modifies httpd
  configuration and starts automatically webalizer.
* have /etc/cron.daily/00webalizer read /etc/sysconfig/webalizer and
  have a variable in /etc/sysconfig/webalizer that enables webalizer,
  and ship /etc/httpd/conf.d/webalizer.conf in %doc.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to