Closed (fixed)
Project:
IP to Country
Version:
master
Component:
Miscellaneous
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Jun 2008 at 01:13 UTC
Updated:
13 Apr 2009 at 22:00 UTC
The database http://ip-to-country.webhosting.info which is used for this module is not licensed under GPL. I consider this a critical bug, because it makes the module unusable and violates the Drupal.org CVS Usage Policy http://drupal.org/node/103704
The database license can be found at http://ip-to-country.webhosting.info/node/view/62 and http://ip-to-country.webhosting.info/node/view/63 While the database is "free", there are license restrictions on its use which conflict with GPL licensing.
Comments
Comment #1
avpadernoThe database is not included in the CVS repository, therefore it's not against Drupal policies which just talk of files included into the repository handled by Drupal.org.
I changed the category because there is nothing wrong with using a database file that has not been released under GPL, if the database file is not committed in CVS.
Comment #3
tr commentedKiam@avpnet.org: The database *is* included in the CVS repository, that's the point. It's in the form of ipdb.mysql, which is just a bunch of SQL to insert the webhosting.info data into Drupal DB tables.
mrfelton: I know you inherited this module, so this was a decision made before you got involved. But I do think it's something that needs to be addressed for the continued viability of the module.
Comment #4
mrfelton commented@TR: You're absolutely right. Actually, on my todo list was to rework this module so that the database is automatically refreshed on a periodical basis. So, as part of that, I will ensure that the database is removed from CVS and is simply downloaded on install/cron. I probably won't be able to work on this until next weekend.
Comment #5
mrfelton commentedfixed in cvs. New release will be out shortly to rectify this for good.