Skip to content

Make code buildable with AWS-LC - #11219

Merged
maskit merged 5 commits into
apache:masterfrom
maskit:awslc
Jun 7, 2024
Merged

Make code buildable with AWS-LC#11219
maskit merged 5 commits into
apache:masterfrom
maskit:awslc

Conversation

@maskit

@maskit maskit commented Apr 5, 2024

Copy link
Copy Markdown
Member

Nothing is tested with AWS-LC, but it builds.

@maskit maskit added Build work related to build configuration or environment TLS labels Apr 5, 2024
@maskit maskit added this to the 10.1.0 milestone Apr 5, 2024
@maskit maskit self-assigned this Apr 5, 2024
@bryancall
bryancall self-requested a review April 8, 2024 22:17
@bryancall

Copy link
Copy Markdown
Contributor

[approve ci]

@maskit

maskit commented Jun 4, 2024

Copy link
Copy Markdown
Member Author

It's not 10.0 blocker, but I'd like to merge this before it gets conflicts.

@maskit
maskit merged commit a966bc4 into apache:master Jun 7, 2024
@cmcfarlen cmcfarlen modified the milestones: 10.1.0, 10.0.0 Jun 10, 2024
@cmcfarlen

Copy link
Copy Markdown
Contributor

Cherry-picked to v10.0.x

cmcfarlen pushed a commit that referenced this pull request Jun 10, 2024
* Make code buildable with AWS-LC

* Add CheckOpenSSLIsAwsLc.cmake

* Add cmakedefine for HAVE_SSL_GET_SHARED_CURVE

* Revert an unnecessary name change

* Fix a build condition for HKDF

(cherry picked from commit a966bc4)
@maskit
maskit deleted the awslc branch September 18, 2024 16:15
@JosiahWI

JosiahWI commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

This PR accidentally removed ENGINE support because the HAVE_ENGINE_XXX CMake variables do not become preprocessor macros. Do not backport to 9.x or earlier without an accompanying fix to re-enable it. This change happened to first last in 10.0.0, but we likely did not document removing ENGINE support because it was an accident.

JosiahWI added a commit that referenced this pull request Aug 3, 2026
* Remove OpenSSL ENGINE code

Commit a966bc4 (#11219) accidentally disabled OpenSSL ENGINE support
entirely. Although it was unintentional, it seems clear no one is using
that API by this point (no one has reported it was broken), and the
API is gone in recent OpenSSL versions. This patch removes the dead logic.
cmcfarlen pushed a commit that referenced this pull request Aug 4, 2026
* Remove OpenSSL ENGINE code

Commit a966bc4 (#11219) accidentally disabled OpenSSL ENGINE support
entirely. Although it was unintentional, it seems clear no one is using
that API by this point (no one has reported it was broken), and the
API is gone in recent OpenSSL versions. This patch removes the dead logic.

(cherry picked from commit e7eb77a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build work related to build configuration or environment Incompatible TLS

Projects

Status: picked-10.0.0

Development

Successfully merging this pull request may close these issues.

4 participants