[ITP] geoip -- IP lookup command line tools to show country information

[Yaakov S (Cygwin Ports)]

Jari Aalto wrote:
>     http://cygwin.cante.net/geoip/setup.hint \
>     http://cygwin.cante.net/geoip/geoip-1.3.8-1.tar.bz2.sig \
>     http://cygwin.cante.net/geoip/geoip-1.3.8-1.tar.bz2 \
>     http://cygwin.cante.net/geoip/geoip-1.3.8-1-src.tar.bz2.sig \
>     http://cygwin.cante.net/geoip/geoip-1.3.8-1-src.tar.bz2

Is there a reason why you're specifically packaging such an old version? 
  I know that 1.3.8 is in Debian stable, but 1.3.14 is in Debian 
testing, 1.3.10 in Ubuntu breezy,

Can we please get (yet another) clarification on this policy:
Does a packages presence in some distribution's stable repo (for which a 
Cygwin package would be exempt from voting) also limit the package 
*version* to whatever is found stable there?  Or once the package itself 
is found to be in stable, indicating that it's a mainstream package, is 
the choice of which version to package at the (potential) packager's 
discretion?

Package issues:

1) Shared libraries are not built due to a lack of '-no-undefined' in 
LDFLAGS.  Patch libGeoIP/Makefile.am as follows (from 1.3.14):

-libGeoIP_la_LDFLAGS = -version-info @GEOIP_VERSION_INFO@
+libGeoIP_la_LDFLAGS = -version-info @GEOIP_VERSION_INFO@ -no-undefined

-libGeoIPUpdate_la_LIBADD = -lz
+libGeoIPUpdate_la_LDFLAGS = -no-undefined
+libGeoIPUpdate_la_LIBADD = libGeoIP.la -lz

Then run autoreconf before configure, and shared libraries will be built 
as well.

With 1.3.8, it's a little trickier; the following line needs to be added 
to libGeoIP/Makefile.am as well:

+libGeoIPBitReader_la_LDFLAGS = -no-undefined

and you need to force the use of autoconf-2.5x instead of that version's 
default of 2.13.

2) The CYGWIN-PATCHES directory is extracted under geoip-1.3.8-orig, not 
geoip-1.3.8, causing the install step to bail out.


Yaakov