diff options
| author | merge-script <fanquake@gmail.com> | 2025-03-22 11:12:14 +0800 |
|---|---|---|
| committer | merge-script <fanquake@gmail.com> | 2025-03-22 11:12:14 +0800 |
| commit | af3dee0b8d456a8aa9ade9c6e7f4283a90590eae (patch) | |
| tree | c6939f2d6033a0aa3a68b05e041a80a076504561 /gitian-building | |
| parent | 804675930598ccd3102eab642d2b5aa3caa982d4 (diff) | |
| parent | fa7a40d952ad7588f45f6229aeae754b02fdfb55 (diff) | |
Merge bitcoin/bitcoin#32074: contrib: Make deterministic-coverage error messages more readableHEADmaster
fa7a40d952ad7588f45f6229aeae754b02fdfb55 contrib: Print deterministic-coverage runs (MarcoFalke)
fa751639fb667dc31a2220e33cbd134aa6b30741 contrib: Make deterministic-coverage error messages more readable (MarcoFalke)
Pull request description:
This is almost a "refactor" to tidy up the error messages. Apart from the messages, the behavior of the tools is identical.
This was requested in https://github.com/bitcoin/bitcoin/pull/31901#discussion_r1969502508.
Previously, the tool would abort the program early on some errors. Now, the tool propagates an `std::result::Result::Err` up to `main` via an early return. Getting rid of the aborts also allows to drop the `RUST_BACKTRACE` env setting.
ACKs for top commit:
hodlinator:
re-ACK fa7a40d952ad7588f45f6229aeae754b02fdfb55
janb84:
ACK [fa7a40d](https://github.com/bitcoin/bitcoin/commit/fa7a40d952ad7588f45f6229aeae754b02fdfb55)
Tree-SHA512: 6c97861306e2fececa14b2d12deafb78995fc2bcf75e4e22773cb0ab4231de78834db9f1f89b30c49d77499433b1c16c1d90b97eb4069c81855bd2a7944b554f
Diffstat (limited to 'gitian-building')
0 files changed, 0 insertions, 0 deletions