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 /.cirrus.yml | |
| 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 '.cirrus.yml')
| -rw-r--r-- | .cirrus.yml | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/.cirrus.yml b/.cirrus.yml index e5a2f36f4d..5c5942e2d5 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -84,6 +84,9 @@ task: memory: 1G # For faster CI feedback, immediately schedule the linters << : *CREDITS_TEMPLATE + test_runner_cache: + folder: "/lint_test_runner" + fingerprint_script: echo $CIRRUS_TASK_NAME $(git rev-parse HEAD:test/lint/test_runner) python_cache: folder: "/python_build" fingerprint_script: cat .python-version /etc/os-release |