build: remove -Qunused-arguments workaround for clang + ccache #18535

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:dont_quash_unused_driver_arguments changing 1 files +0 −3
  1. fanquake commented at 8:01 AM on April 6, 2020: member

    This was added in 386efb7695debaf5f0f328a2e4c9abc342161665 to address spammy Clang warnings when building with ccache.

    The issue was addressed in ccache 3.2, and from a look at most major distros, it's only Debian Jessie that has a version of ccache older than that (3.1).

    Therefore I think it's acceptable to drop this workaround, and re-enable warnings for unused driver arguments (when compiling using Clang and ccache).

  2. build: remove -Qunused-arguments workaround for clang + ccache
    This was added in 386efb7695debaf5f0f328a2e4c9abc342161665 to
    address spammy Clang warnings when building with ccache.
    
    The issue was addressed in ccache 3.2
    (https://bugzilla.samba.org/show_bug.cgi?id=8118, Nov 2014),
    and from a look at all major distros, it's only Debian Jessie
    that has a version of ccache older than that (3.1).
    
    Therefore I think it's acceptable to drop this workaround, and
    re-enable warnings for unused driver arguments (when compiling
    using Clang).
    a029805f57
  3. fanquake added the label Build system on Apr 6, 2020
  4. MarcoFalke added the label Needs gitian build on Apr 6, 2020
  5. practicalswift commented at 1:34 PM on April 6, 2020: contributor

    Concept ACK

    Thanks for cleaning up!

  6. DrahtBot commented at 11:38 AM on April 7, 2020: member

    <!--a722867cd34abeea1fadc8d60700f111-->

    Gitian builds

    File commit adac12ae73e896e405e75b02a0a49c575d115b6e<br>(master) commit a309ee60eaa9cac91a85a1fe03bb7e2977f5a2bf<br>(master and this pull)
    bitcoin-0.19.99-aarch64-linux-gnu-debug.tar.gz d5b53a3f89f72ead... 456c6b38bccd15de...
    bitcoin-0.19.99-aarch64-linux-gnu.tar.gz 067812d8fbd9918e... c9912cfc75aae509...
    bitcoin-0.19.99-arm-linux-gnueabihf-debug.tar.gz 8be7ab33ba6e6075... 927185dfdf398f01...
    bitcoin-0.19.99-arm-linux-gnueabihf.tar.gz 194ab99a0758ca25... ead7a87d54202249...
    bitcoin-0.19.99-osx-unsigned.dmg e076c26306f27f2c... 8c2e8003737fcb84...
    bitcoin-0.19.99-osx64.tar.gz cbc6f79d1cd0fe5b... b08e8aa424981a8a...
    bitcoin-0.19.99-riscv64-linux-gnu-debug.tar.gz cf5a067865c9558e... e6ba44bdf9318a5d...
    bitcoin-0.19.99-riscv64-linux-gnu.tar.gz fd751d3483c555e7... 45491cf2bffef95c...
    bitcoin-0.19.99-win64-debug.zip 37604981ab8d9fde... 2e869426cfd25cef...
    bitcoin-0.19.99-win64-setup-unsigned.exe ddb2a875e5748c9b... a7015f91d6fbe082...
    bitcoin-0.19.99-win64.zip fe934b412baf6fb3... b83bf9dc27c91351...
    bitcoin-0.19.99-x86_64-linux-gnu-debug.tar.gz 0d4c783eb986d57b... 5fee80283d704cd5...
    bitcoin-0.19.99-x86_64-linux-gnu.tar.gz 067d79d34ba518b5... 3f5a6e07616f8055...
    bitcoin-0.19.99.tar.gz 59e74c33a2c18358... 21488168441cdc28...
    bitcoin-core-linux-0.20-res.yml 100f56d6448f4b18... 12dfd443304cffb7...
    bitcoin-core-osx-0.20-res.yml 0714155380d848a8... b951a13af418ea8d...
    bitcoin-core-win-0.20-res.yml 1ef3d9373bdb5c0e... bfa3a469f43be5fe...
    linux-build.log 382a1ae618392852... a952c21cc5c6ca9d...
    osx-build.log a02b5e5ada7fbc28... 826112ec65871bd3...
    win-build.log 5ba854bca82ac9b9... f69d7df8d78bbf99...
    bitcoin-core-linux-0.20-res.yml.diff 3c473283aa38dd15...
    bitcoin-core-osx-0.20-res.yml.diff 47528d87816c704c...
    bitcoin-core-win-0.20-res.yml.diff 09711d4599a2f94f...
    linux-build.log.diff 4b9da86097cd5513...
    osx-build.log.diff 1f31dcb56b1fc9aa...
    win-build.log.diff 2889a982136d1ad9...
  7. DrahtBot removed the label Needs gitian build on Apr 7, 2020
  8. hebasto approved
  9. hebasto commented at 8:05 PM on April 15, 2020: member

    ACK a029805f57fa9a4ab9867c0d1e865675d57537c7.

    Both master and master+PR produce the same two warnings:

    clang: warning: argument unused during compilation: '-pthread' [-Wunused-command-line-argument]
    clang: warning: argument unused during compilation: '-pthread' [-Wunused-command-line-argument]
    

    that are not related to ccache.

  10. vasild commented at 7:29 AM on May 6, 2020: member

    utACK a029805f57fa9a4ab9867c0d1e865675d57537c7

  11. vasild approved
  12. fanquake merged this on May 7, 2020
  13. fanquake closed this on May 7, 2020

  14. fanquake deleted the branch on May 7, 2020
  15. sidhujag referenced this in commit 2af5055891 on May 12, 2020
  16. PastaPastaPasta referenced this in commit 35c63532f5 on Jun 27, 2021
  17. PastaPastaPasta referenced this in commit d3cc4dedac on Jun 28, 2021
  18. PastaPastaPasta referenced this in commit 500a08b003 on Jun 29, 2021
  19. PastaPastaPasta referenced this in commit b51f7f01b2 on Jul 1, 2021
  20. PastaPastaPasta referenced this in commit 7a467f7356 on Jul 1, 2021
  21. PastaPastaPasta referenced this in commit c0531c304b on Jul 14, 2021
  22. PastaPastaPasta referenced this in commit ab19552f31 on Jul 15, 2021
  23. 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: 2026-04-26 06:14 UTC

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