Skip to content

Conversation

@craigcomstock
Copy link
Contributor

@craigcomstock craigcomstock commented Nov 4, 2025

@cf-bottom
Copy link

Thank you for submitting a PR! Maybe @larsewi can review this?

@craigcomstock craigcomstock changed the title Adjusted cf3_with_library to handle case where --with-feature is not defined (3.24) RedHat 10 and Debian 13 related changes Nov 5, 2025
@craigcomstock
Copy link
Contributor Author

retry, policy should be ok now Build Status

@craigcomstock craigcomstock force-pushed the ent-13016/3.24 branch 2 times, most recently from a939eca to 1e79a13 Compare November 6, 2025 13:38
@craigcomstock
Copy link
Contributor Author

fixed up openssl path to prefer vendored Build Status

@craigcomstock craigcomstock changed the title RedHat 10 and Debian 13 related changes RedHat 10 and Debian 13 related changes (3.24) Nov 6, 2025
@craigcomstock
Copy link
Contributor Author

ubuntu 20 and centos 7 complain of no valid ciphers. I will dig into this more.

@craigcomstock craigcomstock marked this pull request as draft November 6, 2025 19:03
…defined

In the case of pam and lmdb, due to checks for brew, --with-pam and --with-lmdb, if not specified by the user, are not defined.
This resulted in -R/lib being added to LMDB_LDFLAGS and PAM_LDFLAGS and by appending: CORE_LDFLAGS.

This -R/lib adds an RPATH which rhel-10 rpmbuild complains about since it is a standard path that should not be included in RPATHS.

Ticket: ENT-13016
Changelog: none
(cherry picked from commit 2264624)
@craigcomstock craigcomstock force-pushed the ent-13016/3.24 branch 2 times, most recently from ef41d95 to fdf960e Compare November 10, 2025 15:41
On RedHat 10 the cipher AES128-GCM-SHA256 is not part of the -ciphersuites TLSv1.3 defaults and so the client with defaults cannot communicate with the server.

We need to configure these tests more dynamically based on what ciphers are available in the openssl that is being used.
In this case RedHat 10 uses systemssl and happens to be openssl 3.2.2 which apparently breaks this test.

Ticket: ENT-13494
Changelog: none
@larsewi larsewi merged commit b805b28 into cfengine:3.24.x Nov 13, 2025
47 of 51 checks passed
@craigcomstock craigcomstock deleted the ent-13016/3.24 branch November 13, 2025 19:31
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.

3 participants