Description of problem:
python-GeoIP doesn't expose the countrycode->continent or
countrycode->countryname mappings which are present in GeoIP. Mirrormanager
would like to make use of these mappings.
Version-Release number of selected component (if applicable):
Created attachment 186711 [details]
Patch to add country code info
Ignacio, you're a prince among men! (my C-fu is not that strong..)
I'll try a local compile with the patch you've submitted and report how it goes.
amen, much thanks to Ignacio. I built and tested this on rawhide i386 and it
works beautifully. I've also sent the patch to firstname.lastname@example.org, and asked
them to relicense python-GeoIP under the LGPL (instead of the GPLv2+ it is now).
I'm kind of annoyed that libgeoip is licensed LGPL, but some of their language
bindings to it are licensed GPLv2.
Wow, they re-licensed it! Yea!
Date: Wed, 05 Sep 2007 09:50:55 -0400
From: William Stevenson <email@example.com>
To: Matt Domsch <firstname.lastname@example.org>
Subject: Re: python-GeoIP license, enhancement patch
Thanks for writing and thanks for integrating us. The code has been
checked into the sourceforge CVS under the new license.
If you have any more questions, please let me know.
Business Development Associate
I've checked in the patch and tagged it ready to go for devel (buildroots are
plauing up right now, I'll wait).
Anyone for a backport to earlier Fedoras and/or EPEL? I would imagine the latter
group would appreciate it.
yes please, at least for Fedora 6 and 7, as that's what Fedora Infrastructure
has in production for this app, though it may move to RHEL5 at some point.
Due to push to F7-updates-testing, building for devel, also should be built for
FC6 and EPEL (4&5)
I'll close the bug once I've confirmed they've hit the repos.
python-GeoIP-1.2.1-8.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report.
python-GeoIP-1.2.1-8.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report.