Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Don't include engine.h when OPENSSL_NO_ENGINE is defined #11328

Closed
wants to merge 1 commit into from
Closed

Don't include engine.h when OPENSSL_NO_ENGINE is defined #11328

wants to merge 1 commit into from

Commits on Jul 22, 2024

  1. Don't include engine.h when OPENSSL_NO_ENGINE is defined 

    Fedora 41 and RHEL 10 are deprecating and phasing out OpenSSL ENGINE
support. Downstream has moved `openssl/engine.h` into a separate RPM
package and is recompiling packages with `-DOPENSSL_NO_ENGINE=1`. The
compiler flag disables PyCA cryptography's ENGINE support successfully.
We also like to build the downstream package without the `engine.h`
header file present.

This commit makes the include conditional. The `ENGINE` type is
defined in `openssl/types.h`.

See: https://src.fedoraproject.org/rpms/openssl/c/e67e9d9c40cd2cb9547e539c658e2b63f2736762?branch=rawhide
See: https://issues.redhat.com/browse/RHEL-33747
Signed-off-by: Christian Heimes <[email protected]>
    @tiran
    tiran committed Jul 22, 2024
    Configuration menu
    Copy the full SHA
    7a1927b View commit details
    Browse the repository at this point in the history