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.

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

Mac: Use brew installed sqlite instead of the system one #434

Merged
merged 5 commits into from
Apr 27, 2016
Merged

Mac: Use brew installed sqlite instead of the system one #434

merged 5 commits into from
Apr 27, 2016

Conversation

JohanLorenzo
Copy link
Contributor

Link #414

@coveralls
Copy link

Coverage Status

Coverage remained the same at 38.441% when pulling 229d488 on JohanLorenzo:workaround-openssl-failure into e272dd1 on fxbox:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 38.441% when pulling 1cf80c4 on JohanLorenzo:workaround-openssl-failure into 3098121 on fxbox:master.

@JohanLorenzo
Copy link
Contributor Author

JohanLorenzo commented Apr 27, 2016

Mac OS hasn't failed after 3 runs on Travis (Mac OS X). 3 more runs locally. r? @julienw

brew install openssl libupnp sqlite

# openssl and sqlite libraries are already installed on OS X base system. That's
# why we need to export them explicitely.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: "that's why we need to explicitely specify the directories that brew installed"

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b9a7f07 on JohanLorenzo:workaround-openssl-failure into * on fxbox:master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling fbcda44 on JohanLorenzo:workaround-openssl-failure into * on fxbox:master*.

@julienw
Copy link
Contributor

julienw commented Apr 27, 2016

r=me ! thx !

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 45aec70 on JohanLorenzo:workaround-openssl-failure into * on fxbox:master*.

@JohanLorenzo JohanLorenzo merged commit 1aaa839 into fxbox:master Apr 27, 2016
@JohanLorenzo JohanLorenzo deleted the workaround-openssl-failure branch May 19, 2016 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants