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

build: add checks for openssl configure options #12175

Closed
wants to merge 4 commits into from
Closed

build: add checks for openssl configure options #12175

wants to merge 4 commits into from

Commits on Apr 3, 2017

  1. build: add checks for openssl configure options 

    Currently it is possible to configure using --without-ssl and
--shared-openssl/--openssl-no-asm/openssl-fips without an error
occuring.

The commit add check for these combinations:

$ ./configure --without-ssl --shared-openssl
Error: --without-ssl is incompatible with --shared-openssl

$ ./configure --without-ssl --openssl-no-asm
Error: --without-ssl is incompatible with --openssl-no-asm

$ ./configure --without-ssl --openssl-fips=dummy
Error: --without-ssl is incompatible with --openssl-fips
    @danbev
    danbev committed Apr 3, 2017
    Configuration menu
    Copy the full SHA
    c9977f4 View commit details
    Browse the repository at this point in the history

  2. remove semicolon after exit statement

    @danbev
    danbev committed Apr 3, 2017
    Configuration menu
    Copy the full SHA
    1a20f65 View commit details
    Browse the repository at this point in the history

  3. rename to without_ssl_error and move into if block

    @danbev
    danbev committed Apr 3, 2017
    Configuration menu
    Copy the full SHA
    b44ba5e View commit details
    Browse the repository at this point in the history

  4. removing leftover empty line

    @danbev
    danbev committed Apr 3, 2017
    Configuration menu
    Copy the full SHA
    8d6b9ff View commit details
    Browse the repository at this point in the history