Skip to content

travis: install openssl explicitly - #4266

Merged
ethomson merged 1 commit into
masterfrom
ethomson/travis-explicit-openssl
Jun 12, 2017
Merged

travis: install openssl explicitly#4266
ethomson merged 1 commit into
masterfrom
ethomson/travis-explicit-openssl

Conversation

@ethomson

@ethomson ethomson commented Jun 12, 2017

Copy link
Copy Markdown
Member

I can't entirely understand why this is failing, but it is. Harumpfh.

@ethomson

ethomson commented Jun 12, 2017

Copy link
Copy Markdown
Member Author

🤷‍♂️

@ethomson
ethomson merged commit d9914fb into master Jun 12, 2017
@pks-t

pks-t commented Jun 13, 2017

Copy link
Copy Markdown
Member

What exactly does this change fix? I'm a bit confused, as it doesn't really change anything at all in our macOS builds:

+brew install openssl
Error: openssl 1.0.2j is already installed
To upgrade to 1.0.2l, run `brew upgrade openssl`

@ethomson

ethomson commented Jun 13, 2017

Copy link
Copy Markdown
Member Author

Huh. Prior builds seemed to fail when trying to install libssl - it looked like a warning during the install of the prerequisite OpenSSL was being upgraded to a failure.

So I tried to install OpenSSL manually in the hopes that its warning would stay a warning.

So, I'm not sure. Possible that my interpretation of the prior build failures is wrong and that the Travis Mac builds were flakey and I waited them out, or it's possible that there's some magic here.

@ethomson

ethomson commented Jun 13, 2017

Copy link
Copy Markdown
Member Author

Happy to dig in later when I have some free time.

@ethomson
ethomson deleted the ethomson/travis-explicit-openssl branch January 9, 2019 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ethomson @pks-t