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

Update Guzzlehttp to v7 #29

Open
dhildreth opened this issue Sep 9, 2020 · 3 comments
Open

Update Guzzlehttp to v7 #29

dhildreth opened this issue Sep 9, 2020 · 3 comments

Comments

@dhildreth
Copy link

Laravel released version 8 recently. It requires guzzlehttp/guzzle 7.0.1 (upgrade docs). It appears v3.1 of freshdesk-php-sdk requires >=4.0,<7.0:

$ composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: don't install guzzlehttp/guzzle 7.0.1
    - Conclusion: don't install guzzlehttp/guzzle 7.1.x-dev
    - mpclarkson/freshdesk-php-sdk 0.3.1 requires guzzlehttp/guzzle >=4.0,<7.0 -> satisfiable by guzzlehttp/guzzle[4.x-dev, 5.3.x-dev, 6.5.x-dev].
    - mpclarkson/freshdesk-php-sdk 0.3.1 requires guzzlehttp/guzzle >=4.0,<7.0 -> satisfiable by guzzlehttp/guzzle[4.x-dev, 5.3.x-dev, 6.5.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.0.x-dev, 5.3.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.0.x-dev, 6.5.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.0.x-dev, 4.x-dev].
    - Installation request for guzzlehttp/guzzle ^7.0.1 -> satisfiable by guzzlehttp/guzzle[7.0.1, 7.0.x-dev, 7.1.x-dev].
    - Installation request for mpclarkson/freshdesk-php-sdk ^0.3.1 -> satisfiable by mpclarkson/freshdesk-php-sdk[0.3.1].

Similar to issue #8 , can we get updates for guzzle 7.0.1 please?

@patrocle
Copy link

patrocle commented Dec 8, 2020

Hi,

I just change the composer.json and it works.

PR here

cheers

@dhildreth
Copy link
Author

Is there anything else we can do to help get guzzle > 7.0.1 working with this package?

vagrant@homestead:~/www.embeddedarm.com$ composer require mpclarkson/freshdesk-php-sdk
Using version ^0.3.1 for mpclarkson/freshdesk-php-sdk
./composer.json has been updated
Running composer update mpclarkson/freshdesk-php-sdk
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires mpclarkson/freshdesk-php-sdk ^0.3.1 -> satisfiable by mpclarkson/freshdesk-php-sdk[0.3.1].
    - mpclarkson/freshdesk-php-sdk 0.3.1 requires guzzlehttp/guzzle >=4.0,<7.0 -> found guzzlehttp/guzzle[4.0.0-rc.1, ..., 4.x-dev, 5.0.0, ..., 5.3.x-dev, 6.0.0, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

@dhildreth
Copy link
Author

It appears there are some forks that are more up to date. For example, celery-payroll/freshdesk-php-sdk appears to be a company-backed fork with support for Guzzle < 8.0. To use it, add the following to your composer.json file:

    "repositories": [{
        "type": "vcs",
        "url": "https://github.com/celery-payroll/freshdesk-php-sdk"
    }]

Then, run composer require mpclarkson/freshdesk-php-sdk. Yes, this looks strange, but it will actually install the correct version (in my case v1.0.1, which allows for Guzzle < 8.0.

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

2 participants