doc: Better explain GNU ld's dislike of ld64's options #18779

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:better_document_gnu_ld_issues_with_ld64_options changing 1 files +3 −1
  1. fanquake commented at 3:33 AM on April 27, 2020: member

    There's also now more than a single option being special cased for darwin. If we didn't special case these options they would still end up on the link line and the binaries produced would just segfault.

    I'm going to plug #17874 here as well, because adding -fatal-warnings to our AX_CHECK_LINK_FLAG calls would mostly prevent this sort of option mangling from happening.

    An example of the warning behaviour:

    echo "int main() {}" | g++ -x c++ -std=c++11 -Wl,-dead_strip -
    /usr/bin/ld: warning: cannot find entry symbol ad_strip; defaulting to 0000000000001040
    
    nm -C a.out
    0000000000001000 t _init
    0000000000001040 T _start
                     U ad_strip
    
  2. doc: Better explain GNU ld's dislike of ld64's options
    There's also now more than a single option being special cased for
    darwin.
    cd24f37ea9
  3. fanquake added the label Docs on Apr 27, 2020
  4. fanquake added the label Build system on Apr 27, 2020
  5. dongcarl commented at 12:03 PM on April 27, 2020: member

    ACK cd24f37ea9168bd56b8c518ea6125e242cc4213d


    Much clearer, and yikes...

  6. fanquake merged this on Apr 28, 2020
  7. fanquake closed this on Apr 28, 2020

  8. fanquake deleted the branch on Apr 28, 2020
  9. PastaPastaPasta referenced this in commit 67b55ab271 on Jun 27, 2021
  10. PastaPastaPasta referenced this in commit adb424252e on Jun 28, 2021
  11. PastaPastaPasta referenced this in commit de15f01176 on Jun 29, 2021
  12. PastaPastaPasta referenced this in commit 464fe3d89a on Jul 1, 2021
  13. PastaPastaPasta referenced this in commit 343fce2996 on Jul 1, 2021
  14. PastaPastaPasta referenced this in commit f03e6b2676 on Sep 17, 2021
  15. PastaPastaPasta referenced this in commit 3bd5b89aa5 on Sep 17, 2021
  16. PastaPastaPasta referenced this in commit 1090f1aab4 on Sep 18, 2021
  17. thelazier referenced this in commit b4f2864137 on Sep 25, 2021
  18. 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-22 18:14 UTC

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