Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation for installing GeoIP PECL extension misleading #7052

Closed
driehle opened this issue Jan 20, 2015 · 8 comments
Closed

Documentation for installing GeoIP PECL extension misleading #7052

driehle opened this issue Jan 20, 2015 · 8 comments
Labels
c: Website matomo.org For issues related to our matomo.org website. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.

Comments

@driehle
Copy link

driehle commented Jan 20, 2015

The documentation under http://piwik.org/faq/how-to/#faq_163 suggests to run the following command on Debian or Ubuntu machines to set up the GeoIP PECL extension for PHP:

sudo apt-get install php5-geoip php5-dev libgeoip-dev

I don't see why there should be an need to install php5-dev and libgeoip-dev if you don't want to compile the PECL extension yourself. And actually there is no need to compile that extension yourself under Debian or Ubuntu, because it is available from the main repository.

So to set up GeoIP PECL under Debian or Ubuntu, the following should be sufficient:

sudo apt-get install geoip-database-contrib php5-geoip

This installs a package which downloads all databases and even configures a cronjob for the user to update these databases every month, if the user whishes to do so.

@tsteur
Copy link
Member

tsteur commented Jan 20, 2015

Looks good to me, maybe someone else knows why those dependencies are listed there?

@tsteur tsteur added Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. c: Website matomo.org For issues related to our matomo.org website. and removed c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. labels Jan 20, 2015
@mattab mattab added this to the Piwik 2.11.0 milestone Feb 7, 2015
@mattab
Copy link
Member

mattab commented Feb 7, 2015

Thanks for the report, maybe @diosmosis you have some feedback

@diosmosis
Copy link
Member

I can't remember much from back then.

Note: If you change the FAQ to use geoip-database-contrib it should mention if it's possible to use the package w/ commercial MaxMind DBs and if not to mention Piwik's auto updater.

@mattab
Copy link
Member

mattab commented Feb 9, 2015

we should test it before changing the doc, if someone is keen please go ahead

@mattab mattab modified the milestones: Short term, Piwik 2.11.0 Feb 9, 2015
@mattab mattab modified the milestones: Mid term, Short term Apr 9, 2015
@mvhconsult
Copy link

Would be nice to get this uptodate. Also note that PHP7 is spreading fast, so an explicit referral to php5 is not useful. I would also like to see a clearer distinction between 'compile' and simply install. Use some extra header?

@mattab
Copy link
Member

mattab commented Dec 1, 2016

@mvhconsult Unfortunately we don't have time or resource right now, but if you can help we'd love to see a pull request 👍

@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016
@benwazza
Copy link

The suggested change above got GeoIP PECL working correctly for me on PHP 7/ Ubuntu 16.04. I also set geoip.custom_directory=/usr/share/GeoIPin php.ini to use the database that is updated via cron. The documented PECL install command (with php5 changed to php) only provided country level locations even though the UI reported everything was fine.

@Findus23
Copy link
Member

As Matomo now uses GeoIP2 this can be closed. The C extension isn't needed anymore and if one still wants to use it, it is explained here:
https://matomo.org/faq/how-to/faq_164/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Website matomo.org For issues related to our matomo.org website. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

No branches or pull requests

7 participants