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

Add support for mcrypt on PHP 8.4 #1038

Closed
wants to merge 1 commit into from

Conversation

chrootlogin
Copy link

Enables mcrypt on PHP 8.4 because with version 1.0.7 mcrypt is supported.

@mlocati
Copy link
Owner

mlocati commented Jan 3, 2025

It seems that mcrypt doesn't support php versions greater than 8.4.0

@tarlepp
Copy link

tarlepp commented Jan 3, 2025

It seems that mcrypt doesn't support php versions greater than 8.4.0

Hmm, isn't that odd - just because php 8.4.0 doesn't even exist first version was 8.4.1

@mlocati
Copy link
Owner

mlocati commented Jan 3, 2025

Yep, rather odd. Maybe they meant PHP < 8.4.0, but in the package published on PECL we currently have <= 8.4.0

@tarlepp
Copy link

tarlepp commented Jan 4, 2025

What I see, it's that people should move out of mcrypt and use eg. openssl instead - sure this is not "easy" way out, but in the long run I think it's the correct path.

@chrootlogin
Copy link
Author

@tarlepp Thanks for your insight. I'm little bit out of the php coding space. Thats why I thought it will work. I mostly use php for our custom docker images packaging nextcloud etc.
Now I found out it's an optional dependency of nextcloud. But still creates the question why such big projects are using it.
But anyway I saw mcrypt is not maintained anymore on PECL. So I will close this.

@chrootlogin chrootlogin closed this Jan 6, 2025
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

Successfully merging this pull request may close these issues.

3 participants