diff options
| author | fanquake <fanquake@gmail.com> | 2023-11-13 13:46:57 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2023-11-13 14:10:54 +0000 |
| commit | 63423480723de8f4da67e9f4a715cca15498a4ca (patch) | |
| tree | 9afc80788c7c0618d11b9a2cd68d33b3c52bd41e /share | |
| parent | 29c2c903621f7daae26113dd2902c016b56929d4 (diff) | |
| parent | bbbbdb0cd57d75a06357d2811363d30a498f4499 (diff) | |
Merge bitcoin/bitcoin#28076: util: Replace std::filesystem with util/fs.h
bbbbdb0cd57d75a06357d2811363d30a498f4499 ci: Add filesystem lint check (MarcoFalke)
fada2f91108a56cc5c447bd6b6fac411e4d5cdca refactor: Replace <filesystem> with <util/fs.h> (MarcoFalke)
Pull request description:
Using `std::filesystem` is problematic:
* There is a `fs` namespace wrapper for it. So having two ways to achieve the same is confusing.
* Not using the `fs` wrapper is dangerous and buggy, because it disables known bugs by deleting problematic functions.
Fix all issues by removing use of it and adding a linter to avoid using it again in the future.
ACKs for top commit:
TheCharlatan:
ACK bbbbdb0cd57d75a06357d2811363d30a498f4499
fanquake:
ACK bbbbdb0cd57d75a06357d2811363d30a498f4499 🦀
Tree-SHA512: 0e2d49742b08eb2635e6fce41485277cb9c40fe20b81017c391d3472a43787db1278a236825714ca1e41c9d2f59913865cfb0c649e3c8ab1fb598c849f80c660
Diffstat (limited to 'share')
0 files changed, 0 insertions, 0 deletions