diff options
| author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2023-07-06 14:29:27 +0200 |
|---|---|---|
| committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2023-07-11 14:07:11 +0200 |
| commit | 85938804a4a043c75cc4ca51a64ee789e55ba7cb (patch) | |
| tree | ee04af0ecd39e8c9697d2405edc0c318ff906fed /test/buildtool/graph_traverser/graph_traverser.test.hpp | |
| parent | bec3322cfff305f13735c5f72febfd752b49fa10 (diff) | |
| download | justbuild-85938804a4a043c75cc4ca51a64ee789e55ba7cb.tar.gz | |
filesystem: Avoid unwanted indirections...
...that std::filesystem::* calls produce. This is because existence
and type checks use almost exclusively std::filesystem::status,
which follows symbolic links, when being called with path arguments.
Instead, one should instead use these methods with the value
returned by a call of std::filesystem::symlink_status.
This commit also streamlines the FileSystemManager tests, as well
as replace bare calls to std::filesystem with their FileSystemManager
counterparts (where suitable).
Diffstat (limited to 'test/buildtool/graph_traverser/graph_traverser.test.hpp')
| -rw-r--r-- | test/buildtool/graph_traverser/graph_traverser.test.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/buildtool/graph_traverser/graph_traverser.test.hpp b/test/buildtool/graph_traverser/graph_traverser.test.hpp index c5ffb840..84b348e4 100644 --- a/test/buildtool/graph_traverser/graph_traverser.test.hpp +++ b/test/buildtool/graph_traverser/graph_traverser.test.hpp @@ -107,7 +107,7 @@ class TestProject { clargs.artifacts = entry_points; auto const comp_graph = root_dir_ / "graph_description_compatible"; if (Compatibility::IsCompatible() and - std::filesystem::exists(comp_graph)) { + FileSystemManager::Exists(comp_graph)) { clargs.graph_description = comp_graph; } else { |