Add support for AWS LC [#74]#77
Open
izzyleung wants to merge 5 commits into
Open
Conversation
Contributor
Author
|
Verified working on ARM64 with the following testing build, some outputs were removed for brevity: |
izzyleung
commented
Dec 27, 2025
|
|
||
| %build | ||
| %if 0%{?_use_awslc} | ||
| env BUILDSSL_DESTDIR=/usr AWS_LC_VERSION=%{_awslc_version} scripts/build-ssl.sh |
Contributor
Author
There was a problem hiding this comment.
izzyleung
commented
Dec 27, 2025
|
|
||
| %{__make} -j$RPM_BUILD_NCPUS %{?_smp_mflags} ${USE_LUA} CPU="${CPU}" TARGET="linux-glibc" ${systemd_opts} ${pcre_opts} USE_OPENSSL=1 USE_ZLIB=1 ${regparm_opts} ADDINC="$CFLAGS" USE_LINUX_TPROXY=1 USE_THREAD=1 USE_TFO=${USE_TFO} USE_NS=${USE_NS} ${USE_PROMEX} ADDLIB="%{__global_ldflags}" | ||
| %if 0%{?_use_awslc} | ||
| OPENSSL_ARGS="USE_OPENSSL_AWSLC=1 SSL_LIB=/usr/lib SSL_INC=/usr/include" |
Contributor
Author
There was a problem hiding this comment.
izzyleung
commented
Dec 27, 2025
| fi | ||
| %endif | ||
|
|
||
| # The file "README" is available < 3.2, while "README.md" is available >= 3.2; if we cannot find either, don't include such file |
izzyleung
commented
Dec 27, 2025
| %files | ||
| %defattr(-,root,root) | ||
| %doc CHANGELOG README examples/*.cfg doc/architecture.txt doc/configuration.txt doc/intro.txt doc/management.txt doc/proxy-protocol.txt | ||
| %doc CHANGELOG README* examples/*.cfg doc/configuration.txt doc/intro.txt doc/management.txt doc/proxy-protocol.txt |
Contributor
Author
|
Verified working on AMD64 with the following testing build, some outputs were removed for brevity: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR completes #74
Manual verification
python3 -m http.server, taking requests onhttp://0.0.0.0:8000haproxy -db -f haproxy.cfghttps://localhost:8443SAN via:Current limitations:
opensslif installed, so better build this in a container/tmp/download-cache, it's not cleaned up after building RPM, we can add logic to clean them up but if we run things in containers, we don't need to bothercmake3rather thancmaketo make things work. I am lazy, and this OS is going out of support soon, so probably won't do this for the sake of keeping it aliveopenssl, it makes no sense. Maybe we should add a-awslcsuffix to let user know RPM is complied against AWS LC?libssl.soinfo to RPM, when installing built RPM, we will getBut with
We can verify HAProxy is working, but I think this is more to do with AWS LC, less to do with the RPM