Fix pkg-config issues for 0.13 #8314

pull theuni wants to merge 2 commits into bitcoin:master from theuni:fix-pkg-config changing 2 files +34 −13
  1. theuni commented at 6:39 pm on July 7, 2016: member

    For 0.13, I’d prefer to keep the requirements the same as before: All platforms require pkg-config except for Windows.

    For now, I believe I’ve fixed up the current issues and regressions. Fixes: #8228, replaces #8242.

    Post-0.13, I believe the consensus is to treat Windows like the others and require pkg-config across the board. We can drop all of the non-pkg-config paths, and simply AC_REQUIRE(PKG_PROG_PKG_CONFIG)

  2. build: Fix Qt5PlatformSupport check without pkg-config
    The non-pkg-config case can't use pkg-config to check the version.
    
    Also, make sure that the check is properly guarded in the case of missing
    pkg-config macros.
    0c928cb13c
  3. build: fix Windows builds without pkg-config
    - guard PKG_PROG_PKG_CONFIG with an m4_ifdef. If not building for windows,
      require it
    - add nops as necessary in case the ifdef reduces the if/then to nothing
    - AC_SUBST some missing _LIBS. These were split out over time, but not all were
      properly substituted. They continued to work if pkg-config is installed
      because it does the AC_SUBST itself
    b556beda26
  4. MarcoFalke added this to the milestone 0.13.0 on Jul 7, 2016
  5. MarcoFalke added the label Build system on Jul 7, 2016
  6. jonasschnelli commented at 6:52 am on July 8, 2016: contributor
    Gitian was happy with this PR: https://bitcoin.jonasschnelli.ch/pulls/8314/
  7. laanwj commented at 1:07 pm on July 8, 2016: member

    Works for me - retried #8228 and get an error now, as expected:

    0configure: error: PKG_PROG_PKG_CONFIG macro not found. Please install pkg-config and re-run autogen.sh.
    
  8. laanwj merged this on Jul 8, 2016
  9. laanwj closed this on Jul 8, 2016

  10. laanwj referenced this in commit 67caef6730 on Jul 8, 2016
  11. codablock referenced this in commit ef8043b9c9 on Sep 19, 2017
  12. zkbot referenced this in commit 75604363cc on Dec 1, 2017
  13. zkbot referenced this in commit 6aef4033a7 on Dec 1, 2017
  14. zkbot referenced this in commit 83af270002 on Dec 15, 2017
  15. codablock referenced this in commit 4d6c2d9894 on Dec 27, 2017
  16. codablock referenced this in commit 1670472976 on Dec 28, 2017
  17. kotodev referenced this in commit c8a979fc92 on Jan 25, 2018
  18. renium9 referenced this in commit 23640da445 on Feb 6, 2018
  19. andvgal referenced this in commit 303ab7162c on Jan 6, 2019
  20. fanquake referenced this in commit 265492723a on Jun 13, 2020
  21. sidhujag referenced this in commit c8addbe7ea on Jun 13, 2020
  22. xdustinface referenced this in commit e2d3235eb2 on Feb 17, 2021
  23. gades referenced this in commit ac855be7fd on Jun 27, 2021
  24. DrahtBot locked this on Sep 8, 2021


theuni jonasschnelli laanwj

Labels
Build system

Milestone
0.13.0


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: 2024-12-25 15:12 UTC

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