Ebuild is replacing values in awstats.model.conf via sed, but the following value should NOT be replaced, it needs to be boolean. Reproducible: Always Steps to Reproduce: 1. emerge awstats Actual Results: # Possible values: 0 or 1 # Default: 0 # PurgeLogFile="/var/log/apache/access_log" # This parameter is not used if PurgeLogFile="/var/log/apache/access_log" # Change : Effective for new updates only # Possible values: 0 or 1 # Default: 0 # ArchiveLogRecords=0 Expected Results: # Possible values: 0 or 1 # Default: 0 # PurgeLogFile=0 # When PurgeLogFile is setup to 1, AWStats will clean your log file after # processing it. You can however keep an archive file (saved in "DirData") of # all processed log records by setting this to 1 (For example if you want to # use another log analyzer). # This parameter is not used if PurgeLogFile=0
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.
