maxmind / GeoIP2-php

PHP API for GeoIP2 webservice client and database reader
https://maxmind.github.io/GeoIP2-php/
Apache License 2.0
2.33k stars 276 forks source link

Replace Guzzle with actual version: 4 #29

Closed shkarupa-alex closed 10 years ago

shkarupa-alex commented 10 years ago

Now when installing over composer it requires old (3) version of Guzzle which itself requires part of Symfony library (that does't used by GeoIP2)

oschwald commented 10 years ago

Guzzle 4 does not support PHP 5.3, which we do support. We will be staying on 3 for the foreseeable future.

shkarupa-alex commented 10 years ago

I use GeoIP2-php as part of https://github.com/geocoder-php/Geocoder which has some more adapters like cUrl and https://github.com/kriswallsmith/Buzz

May you look at them instead Guzzle?

oschwald commented 10 years ago

What is the underlying problem that you are trying to solve? A dependency conflict?

shkarupa-alex commented 10 years ago

I don't like dependency to symfony/event-dispatcher (from Guzzle) which does't used by any other package.