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

aws/aws-sdk-php v~2 dependency #426

Closed
eduardoconceicao opened this issue Sep 8, 2016 · 3 comments
Closed

aws/aws-sdk-php v~2 dependency #426

eduardoconceicao opened this issue Sep 8, 2016 · 3 comments

Comments

@eduardoconceicao
Copy link

eduardoconceicao commented Sep 8, 2016

Is there any special reason why this bundle depends on version 2 of aws/aws-sdk-php.

I'm questioning because the version 2 of aws/aws-sdk-php depends on guzzle/guzzle which is on abandoned project. On the other hand the new versions of aws/aws-sdk-php (3.19.5 atm) depends not on guzzle/guzzle but on guzzlehttp/guzzle.

Can this dependency version be updated?

Thanks,
Eduardo

@redthor
Copy link

redthor commented Nov 8, 2016

This is now quite important because quite a few packages have upgraded to guzzlehttp/guzzle and now we're getting errors because of classes appearing twice and inconsistencies between those classes. It'd be great to get this dependency updated.

@redthor
Copy link

redthor commented Nov 8, 2016

Actually this appears a bit harder than just updating the composer.json file.

rackspace/php-opencloud also includes guzzle/guzzle and on it's homepage it says:

this codebase which you are now looking at will enter end-of-life soon. For Rackspace Cloud users, a v2 is actively being worked on.

but actually, I cannot see a v2 branch for rackspace/php-opencloud. There is this issue:

rackspace/php-opencloud#586

@jamiehannaford may know more about the status of updating php-opencloud to guzzlehttp

However, the real solution is to break Gaufrette into multiple packages:
http://php-and-symfony.matthiasnoback.nl/2014/04/theres-no-such-thing-as-an-optional-dependency/#the-solution

Then we could confine this message to the knplabs/gaufrette-aws and knplabs/gaufrette-opencloud packages and the impact would be minimal.

@eduardoconceicao
Copy link
Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants