Ver código fonte

geoip: extend docs after discussion with Francesco Castellano

Henning Westerholt 14 anos atrás
pai
commit
b280f89b3a
2 arquivos alterados com 18 adições e 1 exclusões
  1. 8 0
      modules/geoip/README
  2. 10 1
      modules/geoip/doc/geoip_admin.xml

+ 8 - 0
modules/geoip/README

@@ -82,6 +82,14 @@ Chapter 1. Admin Guide
    and its derivatives as libgeoip, and probably by other distributions as
    and its derivatives as libgeoip, and probably by other distributions as
    well.
    well.
 
 
+   Debian Linux squeeze includes already a database as dependency, but as
+   this contain the wrong data, it will not work correctly with the
+   module. More acurate, the module expect the GeoIP City Edition, and
+   will not work with the GeoIP Country Edition. In newer Debian Linux
+   releases the package geoip-database-contrib should contain the
+   necessary database. You can download the Lite edition of the DB from
+   here.
+
    This module exports a new class of pseudo-variables - $gip(pvc=>key) -
    This module exports a new class of pseudo-variables - $gip(pvc=>key) -
    to enable access to the results of a query to the database.
    to enable access to the results of a query to the database.
 
 

+ 10 - 1
modules/geoip/doc/geoip_admin.xml

@@ -28,13 +28,22 @@
 	</para>
 	</para>
 	<para>
 	<para>
 		This database itself can be obtained on a free or commercial basis 
 		This database itself can be obtained on a free or commercial basis 
-		<ulink url="http://www.maxmind.com/app/ip-location">here</ulink>.  The 
+		<ulink url="http://www.maxmind.com/app/ip-location">here</ulink>. The 
 		library that interfaces with the Max Mind API, as well as scripts to
 		library that interfaces with the Max Mind API, as well as scripts to
 		automate downloading of the on-disk version of the open-source 
 		automate downloading of the on-disk version of the open-source 
 		database is also packaged by the Debian Linux distribution and 
 		database is also packaged by the Debian Linux distribution and 
 		its derivatives as <emphasis>libgeoip</emphasis>, and probably by
 		its derivatives as <emphasis>libgeoip</emphasis>, and probably by
 		other distributions as well.
 		other distributions as well.
 	</para>
 	</para>
+	<para>
+		Debian Linux squeeze includes already a database as dependency, but as
+		this contain the wrong data, it will not work correctly with the module.
+		More acurate, the module expect the <emphasis>GeoIP City Edition</emphasis>,
+		and will not work with the <emphasis>GeoIP Country Edition</emphasis>. In
+		newer Debian Linux releases the package <emphasis>geoip-database-contrib</emphasis>
+		should contain the necessary database. You can download the Lite edition
+		of the DB from <ulink url="http://www.maxmind.com/app/geolitecity">here</ulink>.
+	</para>
 	<para>
 	<para>
 		This module exports a new class of pseudo-variables -
 		This module exports a new class of pseudo-variables -
 		$gip(pvc=&gt;key) - to enable access to the results of a query to the
 		$gip(pvc=&gt;key) - to enable access to the results of a query to the