OS X Homebrew Installation #4699

pull DomT4 wants to merge 2 commits into bitcoin:master from DomT4:master changing 2 files +55 −7
  1. DomT4 commented at 2:32 AM on August 14, 2014: contributor

    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.

  2. 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.
    7d50bfca57
  3. 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.
    50d6347988
  4. 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.

  5. 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.

  6. laanwj closed this on Aug 14, 2014

  7. 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.

  8. 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.

  9. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-18 21:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me