Show towns

[GoogleCodeExporter]

If GeoIPCity.dat and GeoIPCityv6.dat are installed, it is possible to show 
towns for peers.

Original issue reported on code.google.com by Nickollai on 23 Jan 2014 at 7:55

[GoogleCodeExporter]

I've tried adding city support, not city v6 yet. Can you let me know if it 
works?

You'll need to close and reopen the window after you add the column. I started 
adding some necessary bits to fix that, but will probably do it another time.

Original comment by a...@eth0.org.uk on 2 Feb 2014 at 12:40

[GoogleCodeExporter]

Have implemented the auto loading of these columns when they're enabled.

Original comment by a...@eth0.org.uk on 2 Feb 2014 at 4:18

• Changed state: Fixed

[GoogleCodeExporter]

Segfaults:
http://bpaste.net/raw/Ws2uURQMTNyQVDfA6uw1/

Original comment by Nickollai on 2 Feb 2014 at 7:53

[GoogleCodeExporter]

Both b598a43dfe3123c3cec1022e59558b09ffe4dac5 and 
e7a058f0e239fa112676c8e3d5fb73279f2dee13 segfault when trying to display geoip 
city info.

Original comment by Nickollai on 2 Feb 2014 at 7:56

[GoogleCodeExporter]

Can you install the debug symbols for GeoIP?

There's something going wrong in the GeoIP library which I can not reproduce, 
and there's not enough information in the backtrace. 

Original comment by a...@eth0.org.uk on 3 Feb 2014 at 10:04

[GoogleCodeExporter]

http://bpaste.net/raw/Xuw2nLJvvdDbOmUFJ9zy/

Is this bt fine? Or i need to rebuild something with different CFLAGS?

Original comment by Nickollai on 3 Feb 2014 at 11:09

[GoogleCodeExporter]

Yeah, that's more useful thanks.

It seems to be that the GeoIP database isn't initialised, which is probably 
because GeoLiteCity.dat does not exist, which is what it is for me on Fedora.

Noticed you mentioned GeoIPCity.dat, so I've made that an alternative name to 
check, and also improved things so that the column won't show if neither of 
these exist.

Original comment by a...@eth0.org.uk on 3 Feb 2014 at 3:44

[GoogleCodeExporter]

Works fine now, thanks. But you forgot to add file for ipv6: GeoIPCityv6.dat

Also it would be more correct to call "_ALT_" paths with "Lite" in name, not 
vice-versa:
https://github.com/maxmind/geoip-api-c/blob/master/README.md
"Free users should create symlinks for the GeoIP databases. For example:

cd /usr/local/share/GeoIP
ln -s GeoLiteCity.dat GeoIPCity.dat
ln -s GeoLiteCountry.dat GeoIPCountry.dat
ln -s GeoLiteASNum.dat GeoIPASNum.dat
"

Gentoo geoip package does this:
http://sources.gentoo.org/viewcvs.py/gentoo-x86/dev-libs/geoip/geoip-1.5.0.ebuil
d?rev=1.15&view=markup
$ equery f geoip
 * Searching for geoip ...
 * Contents of dev-libs/geoip-1.5.0:
/etc
/etc/GeoIP.conf
/usr
/usr/bin
/usr/bin/geoiplookup
/usr/bin/geoiplookup6
/usr/bin/geoipupdate
/usr/include
/usr/include/GeoIP.h
/usr/include/GeoIPCity.h
/usr/include/GeoIPUpdate.h
/usr/lib64
/usr/lib64/libGeoIP.so -> libGeoIP.so.1.5.0
/usr/lib64/libGeoIP.so.1 -> libGeoIP.so.1.5.0
/usr/lib64/libGeoIP.so.1.5.0
/usr/lib64/libGeoIPUpdate.so -> libGeoIPUpdate.so.0.0.0
/usr/lib64/libGeoIPUpdate.so.0 -> libGeoIPUpdate.so.0.0.0
/usr/lib64/libGeoIPUpdate.so.0.0.0
/usr/lib64/pkgconfig
/usr/lib64/pkgconfig/geoip.pc
/usr/sbin
/usr/sbin/geoipupdate.sh
/usr/share
/usr/share/GeoIP
/usr/share/GeoIP/GeoIP.dat
/usr/share/GeoIP/GeoIPASNum.dat
/usr/share/GeoIP/GeoIPCity.dat
/usr/share/GeoIP/GeoIPCityv6.dat
/usr/share/GeoIP/GeoIPv6.dat
/usr/share/doc
/usr/share/doc/geoip-1.5.0
/usr/share/doc/geoip-1.5.0/AUTHORS.bz2
/usr/share/doc/geoip-1.5.0/ChangeLog.bz2
/usr/share/doc/geoip-1.5.0/GeoIP.conf.default.bz2
/usr/share/doc/geoip-1.5.0/README.MinGW.bz2
/usr/share/doc/geoip-1.5.0/README.OSX.bz2
/usr/share/doc/geoip-1.5.0/README.bz2
/usr/share/doc/geoip-1.5.0/READMEwin32.txt.bz2
/usr/share/doc/geoip-1.5.0/READMEwin32static.txt.bz2
/usr/share/doc/geoip-1.5.0/TODO.bz2
/usr/share/man
/usr/share/man/man1
/usr/share/man/man1/geoiplookup.1.bz2
/usr/share/man/man1/geoiplookup6.1.bz2
/usr/share/man/man1/geoipupdate.1.bz2

Original comment by Nickollai on 3 Feb 2014 at 4:35

[GoogleCodeExporter]

ping

Original comment by Nickollai on 7 Feb 2014 at 7:51