Complete revert of accidental merge b1b7ec5c0b

This commit is contained in:
Robert Schumacher 2019-04-09 16:04:37 -07:00
parent 1db72cd0ea
commit 705f9f4919
2 changed files with 14 additions and 25 deletions

View File

@ -479,13 +479,13 @@ namespace vcpkg::Build
const int max_port_file_count = 100; const int max_port_file_count = 100;
// the order of recursive_directory_iterator is undefined so save the names to sort // the order of recursive_directory_iterator is undefined so save the names to sort
std::vector<fs::path> hashed_files; std::vector<fs::path> port_files;
for (auto &port_file : fs::stdfs::recursive_directory_iterator(config.port_dir)) for (auto &port_file : fs::stdfs::recursive_directory_iterator(config.port_dir))
{ {
if (fs::is_regular_file(status(port_file))) if (fs::is_regular_file(status(port_file)))
{ {
hashed_files.push_back(port_file); port_files.push_back(port_file);
if (hashed_files.size() > max_port_file_count) if (port_files.size() > max_port_file_count)
{ {
abi_tag_entries.emplace_back(AbiEntry{ "no_hash_max_portfile", "" }); abi_tag_entries.emplace_back(AbiEntry{ "no_hash_max_portfile", "" });
break; break;
@ -493,20 +493,12 @@ namespace vcpkg::Build
} }
} }
if (hashed_files.size() <= max_port_file_count) if (port_files.size() <= max_port_file_count)
{ {
for (auto &script_file : fs::stdfs::recursive_directory_iterator(paths.scripts)) std::sort(port_files.begin(), port_files.end());
{
if (fs::is_regular_file(status(script_file)))
{
hashed_files.push_back(script_file);
}
}
std::sort(hashed_files.begin(), hashed_files.end());
int counter = 0; int counter = 0;
for (auto & hashed_file : hashed_files) for (auto & port_file : port_files)
{ {
// When vcpkg takes a dependency on C++17 it can use fs::relative, // When vcpkg takes a dependency on C++17 it can use fs::relative,
// which will give a stable ordering and better names in the key entry. // which will give a stable ordering and better names in the key entry.
@ -516,13 +508,13 @@ namespace vcpkg::Build
{ {
System::print2("[DEBUG] mapping ", key, " from ", port_file.u8string(), "\n"); System::print2("[DEBUG] mapping ", key, " from ", port_file.u8string(), "\n");
} }
abi_tag_entries.emplace_back(AbiEntry{ key, vcpkg::Hash::get_file_hash(fs, hashed_file, "SHA1") }); abi_tag_entries.emplace_back(AbiEntry{ key, vcpkg::Hash::get_file_hash(fs, port_file, "SHA1") });
} }
} }
abi_tag_entries.emplace_back(AbiEntry{ abi_tag_entries.emplace_back(AbiEntry{
"vcpkg", "vcpkg_fixup_cmake_targets",
vcpkg::Hash::get_file_hash(fs, paths.scripts / "buildsystems" / "vcpkg.cmake", "SHA1")}); vcpkg::Hash::get_file_hash(fs, paths.scripts / "cmake" / "vcpkg_fixup_cmake_targets.cmake", "SHA1")});
abi_tag_entries.emplace_back(AbiEntry{"triplet", pre_build_info.triplet_abi_tag}); abi_tag_entries.emplace_back(AbiEntry{"triplet", pre_build_info.triplet_abi_tag});

View File

@ -209,14 +209,11 @@ namespace vcpkg::Install
if (!intersection.empty()) if (!intersection.empty())
{ {
const fs::path triplet_install_path = paths.installed / triplet.canonical_name(); const fs::path triplet_install_path = paths.installed / triplet.canonical_name();
System::printf(System::Color::error,
System::println(System::Color::error, "The following files are already installed in %s and are in conflict with %s\n",
"The following files are already installed in %s and are in conflict with %s", triplet_install_path.generic_string(),
triplet_install_path.generic_string(), bcf.core_paragraph.spec);
bcf.core_paragraph.spec); System::print2("\n ", Strings::join("\n ", intersection), "\n\n");
System::print("\n ");
System::println(Strings::join("\n ", intersection));
System::println();
return InstallResult::FILE_CONFLICTS; return InstallResult::FILE_CONFLICTS;
} }