Removes the unnecessary directions that encourage people to force install openssl into /usr/local with Homebrew. Unnecessary, and potentially quite risky. @theuni okayed this removal here.
Build-osx: Force Fix. #4753
pull DomT4 wants to merge 2 commits into bitcoin:master from DomT4:patch-1 changing 1 files +0 −14-
DomT4 commented at 3:15 AM on August 23, 2014: contributor
-
5a61553a14
Build-osx: Force Fix.
Removes the unnecessary directions that encourage people to force install openssl into /usr/local with Homebrew. Unnecessary, and potentially quite risky. @theuni okayed this removal [here](https://github.com/bitcoin/bitcoin/pull/4740#issuecomment-53076840).
-
theuni commented at 3:30 AM on August 23, 2014: member
The "Make sure..." line should go as well. We don't actually need/run the openssl program, so that hint is just confusing.
-
df920c6d49
Build-osx: Force Fix.
Kills the "make" line as well.
-
DomT4 commented at 3:36 AM on August 23, 2014: contributor
Killed that line as well in the latest commit :thumbsup:.
-
BitcoinPullTester commented at 3:49 AM on August 23, 2014: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4753_df920c6d49137a667dbca5dd0763cb4e7de3a3b1/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
- laanwj merged this on Aug 23, 2014
- laanwj closed this on Aug 23, 2014
- laanwj referenced this in commit 0a670bf5d7 on Aug 23, 2014
- DomT4 deleted the branch on Aug 23, 2014
- MarcoFalke locked this on Sep 8, 2021
Contributors
Linked (view graph)