Skip to content

Conversation

@dkwo
Copy link
Contributor

@dkwo dkwo commented Feb 13, 2023

  • I tested the changes in this PR: no
  • I built this PR locally for my native architecture, (x86_64-musl)

#37681 @TinfoilSubmarine

@TinfoilSubmarine
Copy link
Contributor

My preference is that we get #38536 merged. Then we can get php8.2 in repo and drop 8.0 (since 8.1+ support openssl3)

@dkwo
Copy link
Contributor Author

dkwo commented Feb 15, 2023

That PR is dropping 7.*, right? Assuming it gets merged soon, would you mind adding this patch for 8.0 there temporarily, before that gets dropped sometimes later?

@paper42
Copy link
Member

paper42 commented Mar 4, 2023

@TinfoilSubmarine I don't think we should drop php 8.0, it is still getting security fixes until the end of the year and some projects still prefer it. Are there some big issues with php 8.0 and openssl 3?

@TinfoilSubmarine
Copy link
Contributor

It is not supported, see https://www.php.net/manual/en/openssl.requirements.php and https://bugs.php.net/bug.php?id=81540&edit=1. I don't know where this patch came from, but it's not official.

@paper42
Copy link
Member

paper42 commented Mar 7, 2023

I would be in favor of building it with openssl1.1-compat then

@dkwo
Copy link
Contributor Author

dkwo commented Mar 7, 2023

@paper42 Ok, but last time I tried it conflicts with openssl, which seems to be pulled in by default in the chroot. What is the right way to do it? I can then apply that also to the other removed package.

@paper42
Copy link
Member

paper42 commented Mar 7, 2023

@paper42 Ok, but last time I tried it conflicts with openssl, which seems to be pulled in by default in the chroot. What is the right way to do it? I can then apply that also to the other removed package.

openssl1.1-devel conflicting with openssl-devel doesn't matter because only one will be installed at one time, openssl1.1-compat can probably be an empty meta-package because I think we don't need anything it provides. Also, openssl1.1-devel probably doesn't have to depend on openssl1.1-compat, but you would have to try that.

@dkwo
Copy link
Contributor Author

dkwo commented Mar 7, 2023

Hmm that worked for gcsf, but it seems something (for sure apache-devel, maybe others) in php8.0 brings in openssl-devel independently ??

@paper42
Copy link
Member

paper42 commented Mar 9, 2023

I guess we will have to remove this package with openssl 3.0 then

@dkwo dkwo closed this Mar 9, 2023
@dkwo dkwo deleted the O3php80 branch May 16, 2023 18:19
@dkwo dkwo mentioned this pull request Aug 9, 2023
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