diff options
| author | Alberto Sartori <alberto.sartori@huawei.com> | 2022-06-28 18:26:06 +0200 |
|---|---|---|
| committer | Alberto Sartori <alberto.sartori@huawei.com> | 2022-06-28 19:13:04 +0200 |
| commit | 684cf178ef455ee21a03e2458cdf59afa584498f (patch) | |
| tree | e2d63bf176d8bae619c06c331a5574499fffc09b /src/buildtool/build_engine/target_map/target_map.cpp | |
| parent | dbc4eca7e08674a1158e48f40c7731cc7b501557 (diff) | |
| download | justbuild-684cf178ef455ee21a03e2458cdf59afa584498f.tar.gz | |
sort and deduplicate outputs before creating the action
Diffstat (limited to 'src/buildtool/build_engine/target_map/target_map.cpp')
| -rw-r--r-- | src/buildtool/build_engine/target_map/target_map.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/buildtool/build_engine/target_map/target_map.cpp b/src/buildtool/build_engine/target_map/target_map.cpp index 815f1432..d15c1af4 100644 --- a/src/buildtool/build_engine/target_map/target_map.cpp +++ b/src/buildtool/build_engine/target_map/target_map.cpp @@ -15,6 +15,7 @@ #include "src/buildtool/build_engine/target_map/built_in_rules.hpp" #include "src/buildtool/build_engine/target_map/utils.hpp" #include "src/utils/cpp/path.hpp" +#include "src/utils/cpp/vector.hpp" namespace { @@ -396,8 +397,10 @@ void withDependencies( "either outs or out_dirs must be specified for ACTION"}; } - std::sort(outputs.begin(), outputs.end()); - std::sort(output_dirs.begin(), output_dirs.end()); + sort_and_deduplicate(&outputs); + sort_and_deduplicate(&output_dirs); + + // find entries present on both fields std::vector<std::string> dups{}; std::set_intersection(outputs.begin(), outputs.end(), |