Bug 222650

Summary: net/GeoIP: download data files
Product: Ports & Packages Reporter: Max Kostikov <max>
Component: Individual Port(s)Assignee: Adam Weinberger <adamw>
Status: Closed Works As Intended    
Severity: Affects Only Me    
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   

Description Max Kostikov 2017-09-27 19:08:12 UTC
This possible bug related to mail/spamassassin bug https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=222649
Seems like p5-Geo-IP port requires two files GeoIP.dat and GeoIPv6.dat but can't find it.
Here is part of Spamassassin debug:
> сент. 27 20:04:54.169 [10746] dbg: metadata: RelayCountry: failed to load 'Geo::IP', skipping: GeoIP.dat not found at /usr/local/lib/perl5/site_perl/Mail/SpamAssassin/Plugin/RelayCountry.pm line 61.
Comment 1 Po-Chuan Hsieh freebsd_committer freebsd_triage 2017-09-28 10:18:34 UTC
net/p5-Geo-IP depends on net/GeoIP. When you install net/GeoIP, you'll see a pkg-message as follows:

GeoIP does not ship with the actual data files. You must download
them yourself! Please run:
# /usr/local/bin/geoipupdate.sh

Follow the instruction to download data files.
Comment 2 Max Kostikov 2017-09-28 12:04:48 UTC
(In reply to Po-Chuan Hsieh from comment #1)
Oh, thank you. 
Probably adding this scipt execution at ports post install stage will be a good idea.
Comment 3 Po-Chuan Hsieh freebsd_committer freebsd_triage 2018-01-28 17:43:57 UTC
Pass to maintainer of net/GeoIP.
Comment 4 Adam Weinberger freebsd_committer freebsd_triage 2018-01-28 18:37:43 UTC
I'm not sure about this. That only gets you the free versions of the databases. Maxmind sells many different databases, and for sites that use those, they won't want the free versions.
Comment 5 Adam Weinberger freebsd_committer freebsd_triage 2018-05-18 15:34:58 UTC
Closing this out. Only a subset of the people installing GeoIP will want the free database, so it doesn't make sense to me to fetch them for everybody.

Plus, there's no point in fetching and installing the databases every time poudriere pulls in GeoIP as a dependency.