-
Notifications
You must be signed in to change notification settings - Fork 23
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
Magento 2.4.0 Version contraint problem #15
Comments
I feel really dumb by asking this. But shouldn't composer simply download the various versions of this package, see that version 4.0 is compatible with Magento 2.4.0, version 4.1 is compatible with Magento 2.4.1, and then pick 4.0 if you are running Magento 2.4.1? Assuming of course that the composer requirements for 4.0 would be |
Composer is not that smart. If you install the package, the version in the composer.json is "^4.0". That means according to https://semver.org/ : take any version between 4.0.0 and below 5.0.0. Your 4.1.0 is in that range. The version for m2.4.1 should be 5.0.0 (major version bump) so that composer update on a m2.4.0 stays on 4.0.3 and every user has to update by hand to 5.0.0 if the magento version is updated to 2.4.1. |
Thanks for explaining. What a lot of people seem to forget is that composer replacements are optional for composer. If you are replacing Semantic versioning would say that I would need to wait with releasing an actual In short, I'm going to pick up on this a little bit differently: The change made earlier to the This should make sure that we are ready for the release of Magento 2.4.1 later today. |
Sounds good and even makes sense according to semver because the actual code works for both magento versions. |
I'll keep this issue open for now, until we have actually upgraded to 2.4.1 and confirmed all is fine ;) |
@jissereitsma any updates on this one? |
The current branch has this fixed already: https://github.com/yireo/magento2-replace-bundled/blob/magento-2.4.1/composer.json#L5 I'm closing this issue because of that. If you are asking for the updates, because of a specific issue you are running into, could you post that issue? |
Module version 4.1.0 (branch: magento-2.4.1) has magento-ce dependency set to ">=2.4.1 <2.4.2". Because the module will most likely be required in projects with "^4.0" composer tries to install 4.1.0 and fails with magento 2.4.0.
Could be resolved in 2 ways:
The text was updated successfully, but these errors were encountered: