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 league dependencies to PHP 8 compatible versions #31177

Closed
sivaschenko opened this issue Dec 4, 2020 · 7 comments
Closed

Update league dependencies to PHP 8 compatible versions #31177

sivaschenko opened this issue Dec 4, 2020 · 7 comments
Assignees
Labels
Fixed in 2.4.x The issue has been fixed in 2.4-develop branch Priority: P2 A defect with this priority could have functionality issues which are not to expectations. Progress: done Project: PHP8

Comments

@sivaschenko
Copy link
Member

sivaschenko commented Dec 4, 2020

In scope of the user story MC-41382 we need to the update league/flysystem, league/flysystem-aws-s3-v3 and league/flysystem-cached-adapter dependency versions from "1.0" to "2.0"

 

In scope of this spice we need to estimate the effort required to provide that upgrade.

 

@m2-assistant
Copy link

m2-assistant bot commented Dec 4, 2020

Hi @sivaschenko. Thank you for your report.
To help us process this issue please make sure that you provided the following information:

  • Summary of the issue
  • Information on your environment
  • Steps to reproduce
  • Expected and actual results

Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:

@magento give me 2.4-develop instance - upcoming 2.4.x release

For more details, please, review the Magento Contributor Assistant documentation.

Please, add a comment to assign the issue: @magento I am working on this


⚠️ According to the Magento Contribution requirements, all issues must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting.

🕙 You can find the schedule on the Magento Community Calendar page.

📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket.

🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel

✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel

@sergeynezbritskiy
Copy link
Contributor

@magento I am working on this

@sergeynezbritskiy
Copy link
Contributor

sergeynezbritskiy commented Jan 30, 2021

Hi @sivaschenko I checked the ability to upgrade league packages
Here is what I've found:

  1. package league/flysystem-cached-adapter has the highest tag 1.1.0 at the moment, and this the version we use right now, so nothing to update here
  2. at the same time this package has a constraint on league/flysystem ~1.0, which makes impossible to update this package to 2.0
  3. current version of league/flysystem is 1.1.3 which already does support php8, so no need to update this package itself in terms of compatibility with php8
  4. Package league/flysystem-aws-s3-v3 needs to be updates but versions 2.0.0 and higher require league/flysystem ^2.0.0 and this conflicts with our current version of flysystem

Currently it looks like we just need package league/flysystem-cached-adapter to support league/flysystem:^2.0. So should I just wait while the new version of league/flysystem-cached-adapter will be released ?

I'm a bit confused how to deal with it. please suggest

Thanks in advance

@sergeynezbritskiy
Copy link
Contributor

FYI: created internal issue for league/flysystem-cached-adapter to track the progress https://github.com/thephpleague/flysystem-cached-adapter/issues/41
cc: @sivaschenko

@sivaschenko
Copy link
Member Author

Thanks for the detailed investigation @sergeynezbritskiy ! Yes, we have to wait until league/flysystem-cached-adapter version compatible with league/flysystem:^2.0 will be released to update all 3 packages.

@github-jira-sync-bot github-jira-sync-bot added the Priority: P2 A defect with this priority could have functionality issues which are not to expectations. label Mar 16, 2021
@magento-engcom-team magento-engcom-team added the Fixed in 2.4.x The issue has been fixed in 2.4-develop branch label Apr 15, 2021
@magento-engcom-team
Copy link
Contributor

Hi @sivaschenko, @sergeynezbritskiy, @engcom-Kilo.

Thank you for your report and collaboration!

The related internal Jira ticket MC-39646 was closed as Complete.

The fix will be available with the upcoming 2.4.3 release.

@magento-engcom-team
Copy link
Contributor

Hi @sivaschenko, @sergeynezbritskiy, @engcom-Kilo.

Thank you for your report and collaboration!

The related internal Jira ticket MC-41385 was closed as Fixed.

The fix will be available with the upcoming 2.4.3 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in 2.4.x The issue has been fixed in 2.4-develop branch Priority: P2 A defect with this priority could have functionality issues which are not to expectations. Progress: done Project: PHP8
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants