This removes & rebuilds the current instructions for installing Homebrew dependencies on OS X, particular the instructions around exercising the --force command from inside Homebrew to link OpenSSL. This is irresponsible and dangerous to advise, on several levels, and it’s also completely unnecessary. The patch is borrowed from Litecoin, and retains the attribution to the original author inside the document. The patch is much much safer than the current suggested alternative.
OS X Homebrew Installation #4699
pull DomT4 wants to merge 2 commits into bitcoin:master from DomT4:master changing 2 files +55 −7-
DomT4 commented at 2:32 AM on August 14, 2014: contributor
-
7d50bfca57
OS X Homebrew Installation
This removes the current instructions for installing Homebrew dependencies on OS X, particular the instructions around exercising the —force command from inside Homebrew to link OpenSSL. This is irresponsible and dangerous to advise, on several levels, and it’s also completely unnecessary. The patch is borrowed from Litecoin, and retains the attribution to the original author inside the document. The patch is much *much* safer than the current suggested alternative.
-
50d6347988
OS X Homebrew Installation
Forgot to include the -i. Technically, it should work it out without it, but better to include it now just in case.
-
fanquake commented at 2:52 AM on August 14, 2014: member
We no longer use makefiles since the switch to an autotools build system 35b8af9.
-
laanwj commented at 3:22 AM on August 14, 2014: member
Now that we've added the depends/ system, there is a better option that doesn't require homebrew or a similar system but can build all dependencies in-place - see #4592 and https://github.com/theuni/bitcoin/blob/depends-travis/depends/README.usage .
At some point the mac build document will have to be updated but I leave that up to @theuni.
Anyhow - we can't use your instructions as they are incompatible with the build system, so I'm closing this.
- laanwj closed this on Aug 14, 2014
-
DomT4 commented at 3:37 AM on August 14, 2014: contributor
Sorry, I didn't see PR 4592. It's an interesting change, and certainly a better one than mine. My proposal was largely intended to be temporary till something more permanent could be written up, so hey, I'm only too glad to see that someone's beaten me to fixing the OS X dependency issues. I thoroughly approve of the way @theuni went about doing this, It's darn impressive stuff. My apologies for my redundant PR.
-
theuni commented at 4:59 AM on August 14, 2014: member
Build instructions for OSX to use the deps are coming, but I was waiting until the new pull-tester was live and the new Gitian deps are hooked up first. If Travis isn't hooked up soon (...), I'll go ahead and PR the gitian deps so that they don't get stale.
- MarcoFalke locked this on Sep 8, 2021