Skip to content

ENT-13496: Use SYSTEM_SSL variable instead of hardcoded RHEL version check - #2029

Merged
larsewi merged 1 commit into
cfengine:masterfrom
larsewi:openssl
Nov 11, 2025
Merged

ENT-13496: Use SYSTEM_SSL variable instead of hardcoded RHEL version check#2029
larsewi merged 1 commit into
cfengine:masterfrom
larsewi:openssl

Conversation

@larsewi

@larsewi larsewi commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

Replace the hardcoded RHEL 8 check with the SYSTEM_SSL variable to
determine whether to use system-provided OpenSSL libraries. This makes
the build configuration more flexible and allows the same logic to be
applied across different platforms based on build configuration rather
than being tied to a specific RHEL version.

This change enables using system SSL on SuSE 15 in addition to RHEL 8.

Ticket: ENT-13496
Signed-off-by: Lars Erik Wik lars.erik.wik@northern.tech

Replace the hardcoded RHEL 8 check with the SYSTEM_SSL variable to
determine whether to use system-provided OpenSSL libraries. This makes
the build configuration more flexible and allows the same logic to be
applied across different platforms based on build configuration rather
than being tied to a specific RHEL version.

This change enables using system SSL on SuSE 15 in addition to RHEL 8.

Ticket: ENT-13496
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi

larsewi commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

Build RHEL only
Build Status
Build SuSE only
Build Status

@larsewi
larsewi marked this pull request as ready for review November 10, 2025 10:59
@larsewi

larsewi commented Nov 10, 2025

Copy link
Copy Markdown
Contributor Author

This did not solve the blocker ENT-13496 unfortunately 😞 But the change is nice either way

@larsewi
larsewi merged commit 7e0e359 into cfengine:master Nov 11, 2025
14 checks passed
@larsewi
larsewi deleted the openssl branch April 27, 2026 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants