depends: fix Boost 1.55 build on GCC 5 #6280

pull welshjf wants to merge 1 commits into bitcoin:master from welshjf:boost-cxx11 changing 2 files +39 −1
  1. welshjf commented at 10:08 PM on June 13, 2015: contributor

    Boost assumes variadic templates are always available in GCC 4.4+, but they aren't since we don't build with -std=c++11.

    This applies the patch that fixed the issue in boost 1.57: https://github.com/boostorg/config/commit/eec808554936ae068b23df07ab54d4dc6302a695

    See also: https://svn.boost.org/trac/boost/ticket/10500

    (Or we could just bump the Boost version... I dunno how risky that is, so this is the conservative fix.)

  2. depends: fix Boost 1.55 build on GCC 5
    Boost assumes variadic templates are always available in GCC 4.4+, but
    they aren't since we don't build with -std=c++11.
    
    This applies the patch that fixed the issue in boost 1.57:
    https://github.com/boostorg/config/commit/eec808554936ae068b23df07ab54d4dc6302a695
    
    See also: https://svn.boost.org/trac/boost/ticket/10500
    b19a88b2a0
  3. luke-jr commented at 12:25 AM on June 15, 2015: member

    NACK, we shouldn't be patching dependencies here since we need to support building with system dependencies too. Doesn't gitian already succeed? That's all that should use depends/ ...

  4. laanwj commented at 5:57 AM on June 15, 2015: member

    At least for 0.12/master I think we should boost the boost version instead of piling up more local patches.

    For backporting this seems fine to me.

  5. laanwj added the label Build system on Jun 15, 2015
  6. laanwj commented at 10:12 AM on June 15, 2015: member

    @luke-jr the depends system does not only exist for gitian, it's also there for doing cross-compilation, or building when the necessary dependencies don't exist for the distro. Unfortunately it doesn't contain a gcc version, so it has to work with user's gcc which comes from a wider version range than just that used by gitian.

  7. welshjf commented at 1:16 PM on June 15, 2015: contributor

    To be clear, this isn't a failure to build Bitcoin with system dependencies, it's a failure to build Boost. My own need was for working on the depends build itself (namely bumping Qt) without going through the whole gitian setup. I didn't see this as the common case which is why I didn't base against 0.11. The gitian environment will be affected too if it's ever updated. Feel free to close if bumping Boost is the preferred way to go; I just wanted to make the problem and one solution known.

  8. theuni commented at 5:54 PM on June 15, 2015: member

    Agreed with bumping boost. There were problems with some versions after 1.55, but iirc 1.58 is relatively solid and a good candidate for us.

  9. laanwj commented at 12:39 PM on June 16, 2015: member

    @theuni Also bump boost for releases? We could use this as a more minimal patch on 0.11 branch before the next rc.

  10. theuni commented at 3:10 AM on June 17, 2015: member

    @laanwj I meant for master. This looks good for 0.11, I think it's a bit late for such a big bump there.

  11. laanwj commented at 7:08 AM on June 17, 2015: member

    OK. Going ahead and merging this then, we can remove the patch again when the bump happens.

  12. laanwj merged this on Jun 17, 2015
  13. laanwj closed this on Jun 17, 2015

  14. laanwj referenced this in commit 9849c663b1 on Jun 17, 2015
  15. laanwj referenced this in commit 25c2216168 on Jun 17, 2015
  16. 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-21 21:15 UTC

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