svn commit: r490700 - in head: . www/geolizer www/webalizer www/webalizer/files

[Craig Leres]

On 1/19/19 8:22 PM, Adam Weinberger wrote:
> On Sat, Jan 19, 2019 at 2:10 PM Craig Leres <leres at freebsd.org> wrote:
>>
>> On 1/18/19 11:08 PM, Dirk Meyer wrote:
>>> +20190119:
>>> +  AFFECTS: users of german/webalizer2, japanese/webalizer, polish/webalizer2, portuguese/webalizer-pt_BR, portuguese/webalizer-pt_PT, ukrainian/webalizer, www/geolizer, www/webalizer
>>> +  AUTHOR:dinoex at FreeBSD.org
>>> +
>>> +  The option GEOIP has beeen changed to used the new datebase.
>>> +  Either copy GeoLite2-Country.mmdb in the working directory,
>>> +  or set the GeoIPDatabase option to the location of the file.
>>
>> Thanks for bringing geoip support back to webalizer!
>>
>> Would it be a good idea for UPDATING to also mention that
>> net/geoipupdate is used to download the GeoLite2 version of the
>> database?
> 
> It's really net/libmaxminddb that should probably have that message,
> because that is true for all GeoIP 2 users (Cc: sunpoet for this).

Concur. But I also think putting something in UPDATING would be really 
helpful to people migrating from GeoIP 1 to 2. Certainly it took me a 
few minutes to figure out net/geoipupdate was what I needed to download 
the new format database files.

>> Perhaps even include the new webalizer.conf GeoIPDatabase line:
>>
>>       GeoIPDatabase /usr/local/share/GeoIP/GeoLite2-Country.mmdb
> 
> It would make sense to me for that to be the default for webalizer.

I agree with this as well.

		Craig