diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-09-05 17:12:35 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-09-05 17:12:37 +0200 |
| commit | 91e07cc50db675b97fee324a8a293bc6b353d991 (patch) | |
| tree | e19061570653e86402bf55cc7809101f3aaf4f73 /.cirrus.yml | |
| parent | 6490a3ef6c4c6525d169818d18a99050f780c5b7 (diff) | |
| parent | 2b071265c37da22f15769945fd159b50a14792a3 (diff) | |
| download | bitcoin-91e07cc50db675b97fee324a8a293bc6b353d991.tar.xz | |
Merge bitcoin/bitcoin#22591: Util: error if settings json exists, but is unreadable
2b071265c37da22f15769945fd159b50a14792a3 error if settings.json exists, but is unreadable (Tyler Chambers)
Pull request description:
If settings.json exists, but is unreadable, we should error instead of overwriting.
Fixes #22571
ACKs for top commit:
Zero-1729:
tACK 2b071265c37da22f15769945fd159b50a14792a3
ShaMan239:
tACK 2b071265c37da22f15769945fd159b50a14792a3
prayank23:
tACK https://github.com/bitcoin/bitcoin/pull/22591/commits/2b071265c37da22f15769945fd159b50a14792a3
ryanofsky:
Code review ACK 2b071265c37da22f15769945fd159b50a14792a3. Thanks for the fix! Note that PR https://github.com/bitcoin-core/gui/pull/379 will change the appearance of dialogs shown in screenshots above. So it could be interesting to test the two PRs together (but current testing seems more than sufficient)
theStack:
ACK 2b071265c37da22f15769945fd159b50a14792a3 📁
Tree-SHA512: 6f7f96ce8a13213d0335198a2245d127264495c877105058d1503252435915b332a6e55068ac21088f4c0c017d564689f4956213328d5bdee81d73711efc5511
Diffstat (limited to '.cirrus.yml')
0 files changed, 0 insertions, 0 deletions