Skip to content

Rework DSO API conditions and configuration option [1.1.1]#8628

Closed
levitte wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
levitte:remove-no-dso-111
Closed

Rework DSO API conditions and configuration option [1.1.1]#8628
levitte wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
levitte:remove-no-dso-111

Conversation

@levitte
Copy link
Copy Markdown
Member

@levitte levitte commented Apr 1, 2019

'no-dso' is meaningless, as it doesn't get any macro defined.
Therefore, we remove all checks of OPENSSL_NO_DSO. However, there may
be some odd platforms with no DSO scheme. For those, we generate the
internal macro DSO_NONE aand use it.


This is a backport of #8622 to the 1.1.1 branch

'no-dso' is meaningless, as it doesn't get any macro defined.
Therefore, we remove all checks of OPENSSL_NO_DSO.  However, there may
be some odd platforms with no DSO scheme.  For those, we generate the
internal macro DSO_NONE aand use it.
@levitte levitte added the branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) label Apr 1, 2019
@levitte
Copy link
Copy Markdown
Member Author

levitte commented Apr 1, 2019

Note that this change simply ignores no-dso rather than outright deprecating it. It's done with a trick in the hash table for deprecated options.

@levitte
Copy link
Copy Markdown
Member Author

levitte commented Apr 9, 2019

Ping?

@levitte
Copy link
Copy Markdown
Member Author

levitte commented Apr 10, 2019

Merged. Thank you.

5fba3af Rework DSO API conditions and configuration option

@levitte levitte closed this Apr 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants