This removes some docs. See the commit messages for an explanation.
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-
maflcko commented at 4:17 PM on January 26, 2026: member
-
fa06cd4ba7
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.
- 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 - DrahtBot added the label Docs on Jan 26, 2026
-
DrahtBot commented at 4:17 PM on January 26, 2026: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34413.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--174a7506f384e20aa4161008e828411d-->
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.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
hebasto commented at 4:34 PM on January 26, 2026: member
Concept ACK.
-
fa2e1b85dd
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
- maflcko force-pushed on Jan 26, 2026
- DrahtBot added the label CI failed on Jan 26, 2026
- DrahtBot removed the label CI failed on Jan 26, 2026
-
l0rinc commented at 6:53 PM on January 26, 2026: contributor
ACK fa2e1b85dd6b263a77294566bd361d7873731045
- DrahtBot requested review from hebasto on Jan 26, 2026
-
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)
- sedited approved
-
sedited commented at 5:23 PM on January 27, 2026: contributor
ACK fa2e1b85dd6b263a77294566bd361d7873731045
- sedited merged this on Jan 27, 2026
- sedited closed this on Jan 27, 2026
- maflcko deleted the branch on Jan 27, 2026
- fanquake referenced this in commit eda6c979bd on Jan 28, 2026
- fanquake referenced this in commit 4deda48fa4 on Jan 28, 2026