[sr-dev] git:master: geoip: extend docs after discussion with Francesco Castellano

[Daniel-Constantin Mierla]

On 5/16/11 3:49 PM, Henning Westerholt wrote:
> Module: sip-router
> Branch: master
> Commit: b280f89b3ac4f97c3cad1487961509bae893f656
> URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b280f89b3ac4f97c3cad1487961509bae893f656
>
> Author: Henning Westerholt<henning.westerholt at 1und1.de>
> Committer: Henning Westerholt<henning.westerholt at 1und1.de>
> Date:   Mon May 16 15:48:53 2011 +0200
>
> geoip: extend docs after discussion with Francesco Castellano
>
> ---
>
>   modules/geoip/README              |    8 ++++++++
>   modules/geoip/doc/geoip_admin.xml |   11 ++++++++++-
>   2 files changed, 18 insertions(+), 1 deletions(-)
>
> diff --git a/modules/geoip/README b/modules/geoip/README
> index 016d0d8..595b5df 100644
> --- a/modules/geoip/README
> +++ b/modules/geoip/README
> @@ -82,6 +82,14 @@ Chapter 1. Admin Guide
>      and its derivatives as libgeoip, and probably by other distributions as
>      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.
probably full link value should be given instead of word 'here' so that 
the text format for README has it as well.

Cheers,
Daniel

-- 
Daniel-Constantin Mierla
http://www.asipto.com