doc: Remove outdated -fdebug-prefix-map section in dev notes #34413

pull maflcko wants to merge 2 commits into bitcoin:master from maflcko:2601-build-fix-remove changing 2 files +0 −36
  1. maflcko commented at 4:17 pm on January 26, 2026: member
    This removes some docs. See the commit messages for an explanation.
  2. doc: Remove outdated -fdebug-prefix-map section in dev notes
    The section claims to be for ccache builds, however those are already
    fixed after commit 1cc58d3a0c653ac30df04d1010a3cf84c6bc307a.
    
    If there are still any build or debug problems after that commit,
    dedicated instructions can be added back, along with exact steps to
    reproduce and test.
    fa06cd4ba7
  3. DrahtBot renamed this:
    doc: Remove outdated -fdebug-prefix-map section in dev notes
    doc: Remove outdated -fdebug-prefix-map section in dev notes
    on Jan 26, 2026
  4. DrahtBot added the label Docs on Jan 26, 2026
  5. DrahtBot commented at 4:17 pm on January 26, 2026: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34413.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK l0rinc, sedited
    Concept ACK hebasto

    If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #31723 (qa: Facilitate debugging bitcoind inside functional tests by hodlinator)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  6. hebasto commented at 4:34 pm on January 26, 2026: member
    Concept ACK.
  7. build: Remove outdated comment about -ffile-prefix-map
    The -ffile-prefix-map option is no longer used and it seems fine to
    remove the warning about it possibly breaking coverage builds.
    
    If this needs documentation, the dev notes seem like a better place,
    because it also affects other places, such as depends. C.f. commit
    407062f2ac93624f350e9e8a4f641c882a2aaf2f
    fa2e1b85dd
  8. maflcko force-pushed on Jan 26, 2026
  9. DrahtBot added the label CI failed on Jan 26, 2026
  10. DrahtBot removed the label CI failed on Jan 26, 2026
  11. l0rinc commented at 6:53 pm on January 26, 2026: contributor
    ACK fa2e1b85dd6b263a77294566bd361d7873731045
  12. DrahtBot requested review from hebasto on Jan 26, 2026
  13. maflcko commented at 8:57 am on January 27, 2026: member
    As a note in the future, if this is ever added back, could also try to add a workaround by changing the pwd: #21885 (comment)
  14. sedited approved
  15. sedited commented at 5:23 pm on January 27, 2026: contributor
    ACK fa2e1b85dd6b263a77294566bd361d7873731045
  16. sedited merged this on Jan 27, 2026
  17. sedited closed this on Jan 27, 2026

  18. maflcko deleted the branch on Jan 27, 2026
  19. fanquake referenced this in commit eda6c979bd on Jan 28, 2026
  20. fanquake referenced this in commit 4deda48fa4 on Jan 28, 2026
  21. fanquake commented at 2:33 pm on January 28, 2026: member
    Backported to 30.x in #34283.

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-03-03 09:13 UTC

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