diff options
| author | fanquake <fanquake@gmail.com> | 2022-01-20 13:04:39 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-01-20 13:06:04 +0800 |
| commit | a541e5d5198893c3af0d2d1164a15082fb87e006 (patch) | |
| tree | a1bd2d279faae8bd1aa91668f3cbf2fc1128e52b /contrib/signet | |
| parent | 7102f7d6f3c4a3d08e22ae34a0b3460503a919cf (diff) | |
| parent | dc5d6b0d4793ca978f71f69ef7d6b818794676c2 (diff) | |
Merge bitcoin/bitcoin#24104: fs: Make compatible with boost 1.78
dc5d6b0d4793ca978f71f69ef7d6b818794676c2 fs: Make compatible with boost 1.78 (Andrew Chow)
Pull request description:
Boost 1.78 removed `operator+` in a way that breaks our usage of it in a subclass. A [proposed workaround](https://github.com/boostorg/filesystem/issues/223#issuecomment-1000230207) for this is to cast the argument to `boost::filesystem::path`, and this is backwards compatible with older versions of boost.
Additionally, it appears that `fs::canonical` no longer removes trailing slashes. This was causing a test to fail. The solution is to explicitly remove the trailing separator in the one place that `fs::canonical` is used.
Lastly, `fs::create_directories` now has an error message saying `create_directories` instead of `create_directory`. This caused wallet_multiwallet.py to fail. The error message check has been updated to be able accept either string.
Fixes #23846
ACKs for top commit:
ryanofsky:
Code review ACK dc5d6b0d4793ca978f71f69ef7d6b818794676c2
vincenzopalazzo:
ACK https://github.com/bitcoin/bitcoin/pull/24104/commits/dc5d6b0d4793ca978f71f69ef7d6b818794676c2
Tree-SHA512: d4d8e7b49b8dfbf0ced9bfe9a2b3827841227fc755fc799f19159076b0ccf882432cc8b6ad93cdeda98fb58b942b9ba50a9e0a6b4f6b1e0097e80f1074ae5682
Diffstat (limited to 'contrib/signet')
0 files changed, 0 insertions, 0 deletions