Skip to content

Conversation

@tiran
Copy link
Member

@tiran tiran commented Jan 14, 2018

Use an improved version of multissl test helper to compile a local copy
of OpenSSL 1.1.0g.

Signed-off-by: Christian Heimes [email protected]

https://bugs.python.org/issue32549

@tiran tiran force-pushed the travis_openssl branch 2 times, most recently from 31cac11 to 0857d9c Compare January 14, 2018 10:42
@tiran tiran changed the title Compile OpenSSL 1.1.0 for Travis CI bpo-32549: Compile OpenSSL 1.1.0 on Travis CI Jan 14, 2018
@tiran tiran force-pushed the travis_openssl branch 2 times, most recently from 1a2dc69 to 88ae122 Compare January 14, 2018 10:51
Use an improved version of multissl test helper to compile a local copy
of OpenSSL 1.1.0g.

Signed-off-by: Christian Heimes <[email protected]>
@tiran
Copy link
Member Author

tiran commented Jan 16, 2018

@brettcannon @vstinner

This PR is a prerequisite for several of my ssl improvements. Do you agree with my approach? I'm going to remove the hak ss soon as Travis has OpenSSL 1.0.2+ in a base image.

Also see travis-ci/travis-ci#9069

Copy link
Member

@brettcannon brettcannon left a comment

Choose a reason for hiding this comment

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

LGTM; whatever works to unblock you is basically fine by me. 😄

@tiran
Copy link
Member Author

tiran commented Jan 16, 2018

@brettcannon thanks! I'm also changing travis CI settings from monthly to daily cron job on master. This will populate the cache tonight. Further runs should reuse pre-cached openssl builds.

@tiran tiran merged commit ced9cb5 into python:master Jan 16, 2018
@tiran tiran deleted the travis_openssl branch January 16, 2018 20:02
@tiran
Copy link
Member Author

tiran commented Jan 16, 2018

s/change/add/g

We didn't have cronjob runs at all. I misunderstood the UI.

exit
fi
if [ "${TESTING}" != "docs" ]; then
# clang complains about unused-parameter a lot, redirect stderr
Copy link
Member

Choose a reason for hiding this comment

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

At some point, perhaps the before_script should be factored out instead of duplicating all its contents?

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