Summary: | net-www/awstats ebuild erroneously replaces default value in awstats.model.conf.file | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Jakub Moc (RETIRED) <jakub> |
Component: | New packages | Assignee: | Aaron Walker (RETIRED) <ka0ttic> |
Status: | RESOLVED FIXED | ||
Severity: | trivial | ||
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Jakub Moc (RETIRED)
2005-03-03 10:29:22 UTC
While we're looking at this ebuild, I noticed that the ebuild did not correctly execute line 50. The configuration file erroneously points to /var/log/apache/access_log instead of /var/log/apache2/access_log. Line 50 uses a variable APACHEVER which may not exist: john # echo $APACHEVER john # Maybe I'm doing something wrong? I tried googleing APACHEVER and all it comes up with is makefiles which aren't that helpful. John: Yes, that John: Yes, that´s another error (ebuild does not detect installed Apache version correctly). I did not notice this one. ;-) Anyway, PurgeLogFile takes boolean only, i.e. 0/1. Yeah the problem was the first sed expression should've used ^LogFile instead of just LogFile. It was replacing both LogFile and PurgeLogFile. Also, thanks for pointing out ${APACHEVER}. I think it is left over from the mess that was (or still is mostly) awstats prior to me taking over maintainership. Fixed in both 6.3-r2 and 6.4. |