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

Problem with current PHP 8.3.2 version #51

Closed
karlalbertknapp opened this issue Feb 14, 2024 · 6 comments
Closed

Problem with current PHP 8.3.2 version #51

karlalbertknapp opened this issue Feb 14, 2024 · 6 comments

Comments

@karlalbertknapp
Copy link

Hi Daniel,

I use the mindshape_cookie_consent extension with TYPO3 v12.4.11 and PHP 8.3.2 and I get the following error message, when I try to execute composer update:
- mindshape/mindshape-cookie-consent v3.1.2 requires php >=8.0 <=8.3 -> your php version (8.3.2) does not satisfy that requirement.

When I set the PHP version to 8.3.0 it works. Did you indicate problems with PHP 8.3.2 or is it just an adjustment in the composer.json file?

Cheers
Christopher

@featdd
Copy link
Member

featdd commented Feb 14, 2024

Hi @karlalbertknapp,

this is strange, as <=8.3 should allow all 8.3.X versions and I didn't encounter this issue yet.
Have you set some specific version in your composer.json in "config -> platform -> php"?

I'll have to test this myself again, you may please have an extra log at the output of your composer update!?

Greetings
Daniel

@karlalbertknapp
Copy link
Author

Hi Daniel,

I just had set in my composer.json:

"require": {
        "php": "8.3.*"
}

After I removed, same behaviour.

@featdd
Copy link
Member

featdd commented Feb 16, 2024

@karlalbertknapp I will try to test this myself on the weekend.

I don't know if this is a misunderstanding or a strange behaviour from composers version parser
If I test this in tools like this it works: https://jubianchi.github.io/semver-check/#/%3E%3D8.0%20%3C%3D8.3/8.3.2

The constraint "~8.0 <8.4" seems to work, can you give me feedback if it worked for you too?

I will prepare a patch next week after I made some testing myself.

Greetings
Daniel

@karlalbertknapp
Copy link
Author

Hi Daniel,

I will test it and let you know if it works for me as well.

Grettings
Christopher

@featdd
Copy link
Member

featdd commented Feb 18, 2024

@karlalbertknapp I fixed this here: 4574c9a

seems like I had config.platform set on my other machine and didn't got the error displayed...
I will prepare an update next week with some other fixes.

Greetings
Daniel

@featdd featdd closed this as completed Feb 18, 2024
@karlalbertknapp
Copy link
Author

Hi Daniel,

thank you very much for your fast action.

Greetings
Christopher

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