build: Fix Qt link issue for macOS target with DEBUG=1 #18117

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:20200211-macos-debug changing 1 files +1 −0
  1. hebasto commented at 10:21 am on February 11, 2020: member

    Fix #16391

    With this PR for macOS target, building depends with DEBUG=1 does not cause a linker error for Qt static plugins on the following configure script run.

    Qt docs refs:

    The obvious downside is the increased compilation time for macOS targets with DEBUG=1.

  2. build: Fix Qt link issue on macOS with DEBUG=1 5bd374aeb3
  3. fanquake added the label Build system on Feb 11, 2020
  4. MarcoFalke added the label Needs gitian build on Feb 11, 2020
  5. DrahtBot commented at 9:23 am on February 12, 2020: member

    Gitian builds

    File commit 98264e2ccb177a6603fe957f7e263fb413b50d04(master) commit edc6265fddefc3735a16a45d03732402a2f7b6eb(master and this pull)
    bitcoin-0.19.99-aarch64-linux-gnu-debug.tar.gz d23e69b2f698aced... 146e16d598cf9ef5...
    bitcoin-0.19.99-aarch64-linux-gnu.tar.gz e837295a96c27d09... 489326f21eb1acc5...
    bitcoin-0.19.99-arm-linux-gnueabihf-debug.tar.gz 8c7f60315568f1ac... 97df0285353b774a...
    bitcoin-0.19.99-arm-linux-gnueabihf.tar.gz 016bdc6abea558fc... 2b5c3e6333367456...
    bitcoin-0.19.99-osx-unsigned.dmg d7a01b0abcfed7d7... b96460e8ce500b33...
    bitcoin-0.19.99-osx64.tar.gz b7364a11f2d2205d... 986eaa3e7cb413d2...
    bitcoin-0.19.99-riscv64-linux-gnu-debug.tar.gz 7dbe057003731da1... 8b2ab0eaff647cc1...
    bitcoin-0.19.99-riscv64-linux-gnu.tar.gz e82dac1f44f8992a... 7f74879e1375b491...
    bitcoin-0.19.99-win64-debug.zip 68a16d902e1b374a... 66b3ef7f5f819f9e...
    bitcoin-0.19.99-win64-setup-unsigned.exe 9e334ca3cbaa6401... 235fe66498ab08c8...
    bitcoin-0.19.99-win64.zip 102307164336285c... 0beb1438b78a840e...
    bitcoin-0.19.99-x86_64-linux-gnu-debug.tar.gz 45e9703d1a54d9fc... e3b8684eb7d8cdd7...
    bitcoin-0.19.99-x86_64-linux-gnu.tar.gz fe19b9e55e3764b5... 5bffb04fbac78bfc...
    bitcoin-0.19.99.tar.gz ff2633d71780094b... 725986e69adad2c6...
    bitcoin-core-linux-0.20-res.yml 8414d0032adae892... bf1b943c1208ddb1...
    bitcoin-core-osx-0.20-res.yml 425e4d6afd46d6ed... 2d2686cfaf0d9c25...
    bitcoin-core-win-0.20-res.yml 052e223dce02a5c8... be4a0d7abf76f7fe...
    linux-build.log 5b46b3f211f100da... 9f94419a1e724fbe...
    osx-build.log be05bc3a65338eb3... 3d84f2bc4e062438...
    win-build.log 6f9323a7794d7918... 713a0e67b11ee45b...
    bitcoin-core-linux-0.20-res.yml.diff e65b12a0cbe73240...
    bitcoin-core-osx-0.20-res.yml.diff 763ff351a1d70cfa...
    bitcoin-core-win-0.20-res.yml.diff dd0e9b74e017e2f1...
    linux-build.log.diff e9f60c7b3aae2e27...
    osx-build.log.diff ad0052e5e9885844...
    win-build.log.diff e975a83b9f7feb1a...
  6. DrahtBot removed the label Needs gitian build on Feb 12, 2020
  7. hebasto requested review from fanquake on Feb 16, 2020
  8. laanwj commented at 6:45 pm on February 28, 2020: member

    Can you explain why this is needed? Why is only building the debug libraries in debug mode not good enough for MacOS?

    The obvious downside is the increased compilation time for macOS targets with DEBUG=1.

    I mean, this is not really a downside if it doesn’t work otherwise!

  9. hebasto commented at 2:17 pm on March 3, 2020: member

    @laanwj

    Can you explain why this is needed? Why is only building the debug libraries in debug mode not good enough for MacOS?

    For macOS targets with DEBUG=1 Qt adds the _debug suffix to its libs, for example:

    0.../plugins/platforms/libqminimal_debug.a
    

    This causes that the following checks https://github.com/bitcoin/bitcoin/blob/ac5c5d0162a963be6fbaa53261c01705436a65f3/build-aux/m4/bitcoin_qt.m4#L330-L342 to fail due to the linker error:

    0ld: library not found for -lqminimal
    

    With this PR both lib versions, _debug and non-_debug, become available.

  10. hebasto commented at 12:59 pm on March 9, 2020: member
    Closed in favor of #18298
  11. hebasto closed this on Mar 9, 2020

  12. hebasto deleted the branch on Jun 13, 2020
  13. fanquake referenced this in commit fca3e98f64 on Mar 3, 2021
  14. DrahtBot locked this on Feb 15, 2022

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-07-05 19:13 UTC

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