build: Drop macports support #15175

pull Empact wants to merge 1 commits into bitcoin:master from Empact:drop-macports changing 2 files +0 −17
  1. Empact commented at 9:09 AM on January 16, 2019: member

    It's unmaintained, according to @theuni. https://github.com/bitcoin/bitcoin/pull/14920/files#r246964938

    Alternative is to put it under CI. I don't have a strong opinion on this, opened for separate consideration.

  2. Empact cross-referenced this on Jan 16, 2019 from issue Build: enable -Wdocumentation via isystem by Empact
  3. fanquake added the label Build system on Jan 16, 2019
  4. laanwj commented at 10:20 AM on January 16, 2019: member

    Concept ACK. If it's unmaintained it should be removed, simple as that. Unless someone cares enough and promises to pick this up.

  5. DrahtBot commented at 11:10 AM on January 16, 2019: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #15112 (build: Optionally enable -Wzero-as-null-pointer-constant by Empact)
    • #14920 (Build: enable -Wdocumentation via isystem by Empact)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  6. fanquake commented at 11:34 AM on January 16, 2019: member

    Concept ACK. Doing this has been mentioned many times. Macports instructions were dropped from the macOS build readme in #4939 (>4 years ago, discussion in #4917), and I don't think anyone has complained/asked to have them re-added.

  7. practicalswift commented at 3:52 PM on January 16, 2019: contributor

    Concept ACK

    Unsupported stuff should go.

  8. MarcoFalke added the label Needs gitian build on Jan 16, 2019
  9. MarcoFalke added this to the milestone 0.18.0 on Jan 16, 2019
  10. promag commented at 4:06 PM on January 16, 2019: member

    Concept ACK and change LGTM.

  11. laanwj commented at 5:08 PM on January 16, 2019: member

    "AppVeyor was unable to build non-mergeable pull request " huh

  12. build: Drop macports support
    It's untested / unmaintained, according to theuni.
    https://github.com/bitcoin/bitcoin/pull/14920/files#r246964938
    4e81438f65
  13. Empact force-pushed on Jan 16, 2019
  14. Empact commented at 11:21 PM on January 16, 2019: member

    "AppVeyor was unable to build non-mergeable pull request " huh

    Rebasing cleared this up.

  15. jonasschnelli commented at 3:09 AM on January 17, 2019: contributor

    Tested ACK 4e81438f6556d059923f07409264c11c41cc4a0b

  16. Empact commented at 3:18 AM on January 17, 2019: member

    This also removes the last remaining references to qt4 :P

  17. DrahtBot commented at 11:15 AM on January 17, 2019: contributor

    <!--a722867cd34abeea1fadc8d60700f111-->

    Gitian builds for commit fcb6694a9945d2a02f40587e18bd395ef64048e0 (master):

    Gitian builds for commit 823929563fbd52b8c9aca6af2d479c8ca183f51a (master and this pull):

  18. DrahtBot removed the label Needs gitian build on Jan 17, 2019
  19. IlyasRidhuan commented at 5:42 AM on January 18, 2019: none

    utAck

  20. fanquake commented at 5:43 AM on January 18, 2019: member

    utACK 4e81438

  21. theuni commented at 10:37 PM on January 18, 2019: member

    utACK 4e81438f6556d059923f07409264c11c41cc4a0b. Thanks!

  22. jonasschnelli merged this on Jan 19, 2019
  23. jonasschnelli closed this on Jan 19, 2019

  24. jonasschnelli referenced this in commit f55808ad69 on Jan 19, 2019
  25. rex4539 commented at 8:08 AM on January 19, 2019: contributor

    For the poor soul that may come here in the future, trying to compile using MacPorts.

    Add the following to your Bash .profile:

    export BDB_CFLAGS=/opt/local/include/db48/
    export CPPFLAGS="$CPPFLAGS -isystem /opt/local/include"
    export LIBS="$LIBS -L/opt/local/lib"
    export CPPFLAGS="$CPPFLAGS -I/opt/local/include/db48"
    export LIBS="$LIBS -L/opt/local/lib/db48"
    

    Restart your shell.

    Bitcoin Core will now compile.

  26. Empact deleted the branch on Jan 19, 2019
  27. Empact cross-referenced this on Jan 20, 2019 from issue build: Optionally enable -Wzero-as-null-pointer-constant by Empact
  28. jasonbcox referenced this in commit fa51353081 on Oct 17, 2020
  29. Fuzzbawls cross-referenced this on May 25, 2021 from issue [Build] [Depends] Unify macOS deploy target by Fuzzbawls
  30. random-zebra referenced this in commit fbebade1e4 on Jun 11, 2021
  31. PastaPastaPasta referenced this in commit e203d8127f on Jun 26, 2021
  32. PastaPastaPasta referenced this in commit 5f9fe2d5d2 on Jun 26, 2021
  33. PastaPastaPasta referenced this in commit 12fe79134c on Jun 27, 2021
  34. PastaPastaPasta referenced this in commit 57813fa9a6 on Jun 28, 2021
  35. lyricidal cross-referenced this on Nov 1, 2021 from issue [Upstream] [Build] [Depends] Unify macOS deploy target by lyricidal
  36. fanquake referenced this in commit 05cdceb19d on Nov 25, 2021
  37. fanquake cross-referenced this on Nov 25, 2021 from issue build: don't set PORT=no in config.site by fanquake
  38. fanquake referenced this in commit 111c3e06b3 on Nov 26, 2021
  39. sidhujag referenced this in commit e3472d86d5 on Nov 26, 2021
  40. bitcoin locked this on Dec 16, 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-05-20 06:54 UTC