◐ Shell
reader mode source ↗
Skip to content

bpo-36577: setup.py reports missing OpenSSL again#12746

Merged
miss-islington merged 1 commit into
python:masterfrom
tiran:bpo-36577-setup-ssl
Apr 9, 2019
Merged

bpo-36577: setup.py reports missing OpenSSL again#12746
miss-islington merged 1 commit into
python:masterfrom
tiran:bpo-36577-setup-ssl

Conversation

@tiran

@tiran tiran commented Apr 9, 2019

Copy link
Copy Markdown
Member

bpo-36146 introduced another regression. In case of missing OpenSSL
libraries or headers, setup.py no longer reported _hashlib and _ssl to
be missing.

Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue36577

bpo-36146 introduced another regression. In case of missing OpenSSL
libraries or headers, setup.py no longer reported _hashlib and _ssl to
be missing.

Signed-off-by: Christian Heimes <christian@python.org>
@tiran tiran force-pushed the bpo-36577-setup-ssl branch from ac404df to d2f50f5 Compare April 9, 2019 15:37
@tiran

tiran commented Apr 9, 2019

Copy link
Copy Markdown
Member Author

@xdegaye This is another regression. I found while I was verifying your PR. Thanks again!

@vstinner

vstinner commented Apr 9, 2019

Copy link
Copy Markdown
Member

@tiran: Oh. We found the same bug and I proposed a similar fix :-) PR #12750.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

LGTM.

@vstinner

vstinner commented Apr 9, 2019

Copy link
Copy Markdown
Member

@tiran: I closed my PR. Your PR LGTM.

@tiran

tiran commented Apr 9, 2019

Copy link
Copy Markdown
Member Author

Thanks @vstinner 👍

@tiran tiran deleted the bpo-36577-setup-ssl branch April 9, 2019 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants