apr-util: migrate to openssl@4#280827
Conversation
Signed-off-by: Rui Chen <rui@chenrui.dev>
d9b93b6 to
5f46b6b
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f46b6bca3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| depends_on "apr" | ||
| depends_on "openssl@3" | ||
| depends_on "openssl@4" |
There was a problem hiding this comment.
Avoid migrating apr-util to openssl@4 without engine patch
apr-util 1.6.3 still builds its OpenSSL backend with ENGINE APIs (for example ENGINE_load_builtin_engines, ENGINE_by_id, and ENGINE_cleanup in crypto/apr_crypto_openssl.c), but OpenSSL 4 removed ENGINE support from the shared library. Pointing this formula at openssl@4 therefore breaks source builds in environments that compile apr-util locally, unless a downstream patch is added first to remove/guard the ENGINE code path.
Useful? React with 👍 / 👎.
|
🤖 An automated task has requested bottles to be published to this PR. Caution Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch. |
Migrates
apr-utilfromopenssl@3toopenssl@4as part of the staging migration.References: