[vcpkg] Merge the vcpkg metadata uploader into the vcpkg binary

docs/tool-maintainers/layout.md

@@ -36,16 +36,6 @@ We have six files in this directory -- one `.clang-format` file, one
       the IDE.
     - `vcpkg.sln`: The solution file is how one opens the project in the VS IDE.
 
-### `vcpkg`, `vcpkglib`, `vcpkgmetricsuploader`, and `vcpkgtest`
-
-These four contain exactly one `<name>.vcxproj` and one
-`<name>.vcxproj.filters`. The `<name>.vcxproj` file contains the source files
-and the `<name>.vcxproj.filters` contains information on how Visual Studio
-should lay out the project's source files in the IDE's project view.
-
-`vcpkgtest` should not be touched. It's likely that it will be deleted soon. If
-you want to test your code, use the cmake build system.
-
 ## Source Files
 
 If you're modifying the project, it's likely that these are the directories that
@@ -72,8 +62,7 @@ There are three directories:
 ### `src`
 
 The source files live here. `pch.cpp` is the source file for the
-[precompiled header]; `vcpkg.cpp` is where the `vcpkg` binary lives; and
-`vcpkgmetricsuploader.cpp` is where the metrics uploader lives.
+[precompiled header]; `vcpkg.cpp` is where the `vcpkg` binary lives.
 
 The interesting files live in the `vcpkg` and `vcpkg-test` directories. In
 `vcpkg`, you have the implementation for the interfaces that live in

scripts/bootstrap.ps1

@@ -379,7 +379,7 @@ $arguments = (
 "/verbosity:minimal",
 "/m",
 "/nologo",
-"`"$vcpkgBootstrapPath\dirs.proj`"") -join " "
+"`"$vcpkgBootstrapPath\vcpkg.vcxproj`"") -join " "
 
 function vcpkgInvokeCommandClean()
 {
@@ -432,9 +432,4 @@ Write-Verbose "Placing vcpkg.exe in the correct location"
 
 Copy-Item "$vcpkgReleaseDir\vcpkg.exe" "$vcpkgRootDir\vcpkg.exe"
 
-if (-not $disableMetrics)
-{
-    Copy-Item "$vcpkgReleaseDir\vcpkgmetricsuploader.exe" "$vcpkgRootDir\scripts\vcpkgmetricsuploader.exe"
-}
-
 Remove-Item "$vcpkgReleaseDir" -Force -Recurse -ErrorAction SilentlyContinue

toolsrc/CMakeLists.txt

@@ -32,14 +32,16 @@ endif()
 # =============
 
 file(GLOB VCPKGLIB_SOURCES CONFIGURE_DEPENDS src/vcpkg/*.cpp)
+if (VCPKG_DISABLE_METRICS OR NOT WIN32)
+    list(REMOVE_ITEM VCPKGLIB_SOURCES ${CMAKE_CURRENT_SOURCE_DIR}/src/commands.upload-metrics.cpp)
+endif()
+
 file(GLOB VCPKGLIB_BASE_SOURCES CONFIGURE_DEPENDS src/vcpkg/base/*.cpp)
 file(GLOB VCPKGLIB_INCLUDES CONFIGURE_DEPENDS include/vcpkg/*.h include/vcpkg/fwd/*.h)
 file(GLOB VCPKGLIB_BASE_INCLUDES CONFIGURE_DEPENDS include/vcpkg/base/*.h include/vcpkg/base/fwd/*.h)
 
 set(VCPKG_SOURCES ${CMAKE_CURRENT_SOURCE_DIR}/src/vcpkg.cpp)
 
-set(VCPKGMETRICSUPLOADER_SOURCES ${CMAKE_CURRENT_SOURCE_DIR}/src/vcpkgmetricsuploader.cpp)
-
 file(GLOB VCPKG_TEST_SOURCES CONFIGURE_DEPENDS src/vcpkg-test/*.cpp)
 file(GLOB VCPKG_TEST_INCLUDES CONFIGURE_DEPENDS include/vcpkg-test/*.h)
 
@@ -116,14 +118,6 @@ add_executable(vcpkg ${VCPKG_SOURCES})
 target_link_libraries(vcpkg PRIVATE vcpkglib)
 vcpkg_target_add_warning_options(vcpkg)
 
-# === Target: vcpkgmetricsuploader ===
-
-if(WIN32 AND NOT VCPKG_DISABLE_METRICS)
-    add_executable(vcpkgmetricsuploader WIN32 ${VCPKGMETRICSUPLOADER_SOURCES})
-    target_link_libraries(vcpkgmetricsuploader PRIVATE vcpkglib)
-    vcpkg_target_add_warning_options(vcpkgmetricsuploader)
-endif()
-
 # === Target: vcpkg-test ===
 
 if (BUILD_TESTING)
@@ -164,7 +158,6 @@ if(CLANG_FORMAT)
         COMMAND ${CLANG_FORMAT} -i -verbose ${VCPKGLIB_INCLUDES}
 
         COMMAND ${CLANG_FORMAT} -i -verbose ${VCPKG_SOURCES}
-        COMMAND ${CLANG_FORMAT} -i -verbose ${VCPKGMETRICSUPLOADER_SOURCES}
 
         COMMAND ${CLANG_FORMAT} -i -verbose ${VCPKG_TEST_SOURCES}
         COMMAND ${CLANG_FORMAT} -i -verbose ${VCPKG_TEST_INCLUDES}

toolsrc/include/vcpkg/commands.upload-metrics.h

@@ -0,0 +1,12 @@
+#pragma once
+
+#include <vcpkg/commands.interface.h>
+
+namespace vcpkg::Commands::UploadMetrics
+{
+    extern const CommandStructure COMMAND_STRUCTURE;
+    struct UploadMetricsCommand : BasicCommand
+    {
+        virtual void perform_and_exit(const VcpkgCmdArguments& args, Files::Filesystem& fs) const override;
+    };
+}

toolsrc/src/vcpkg-test/commands.cpp

@@ -2,6 +2,7 @@
 
 #include <vcpkg/commands.contact.h>
 #include <vcpkg/commands.h>
+#include <vcpkg/commands.upload-metrics.h>
 #include <vcpkg/commands.version.h>
 
 using namespace vcpkg;
@@ -10,12 +11,20 @@ TEST_CASE ("test commands are constructible", "[commands]")
 {
     Commands::Contact::ContactCommand contact{};
     Commands::Version::VersionCommand version{};
+#if !VCPKG_DISABLE_METRICS && defined(_WIN32)
+    Commands::UploadMetrics::UploadMetricsCommand upload_metrics{};
+#endif // !VCPKG_DISABLE_METRICS && defined(_WIN32)
 }
 
 TEST_CASE ("get_available_basic_commands works", "[commands]")
 {
     auto commands_list = Commands::get_available_basic_commands();
+#if !VCPKG_DISABLE_METRICS && defined(_WIN32)
+    CHECK(commands_list.size() == 3);
+    CHECK(Commands::find("x-upload-metrics", commands_list) != nullptr);
+#else  // ^^^ !VCPKG_DISABLE_METRICS && defined(_WIN32) // VCPKG_DISABLE_METRICS || !defined(_WIN32) vvv
     CHECK(commands_list.size() == 2);
+#endif // ^^^ VCPKG_DISABLE_METRICS || !defined(_WIN32)
     CHECK(Commands::find("version", commands_list) != nullptr);
     CHECK(Commands::find("contact", commands_list) != nullptr);
     CHECK(Commands::find("aang", commands_list) == nullptr);

toolsrc/src/vcpkg/base/system.process.cpp

@@ -469,6 +469,8 @@ namespace vcpkg
         STARTUPINFOW startup_info;
         memset(&startup_info, 0, sizeof(STARTUPINFOW));
         startup_info.cb = sizeof(STARTUPINFOW);
+        startup_info.dwFlags = STARTF_USESHOWWINDOW;
+        startup_info.wShowWindow = SW_HIDE;
 
         return windows_create_process(cmd_line, env, dwCreationFlags, startup_info);
     }
@@ -547,7 +549,8 @@ namespace vcpkg
     {
         auto timer = Chrono::ElapsedTimer::create_started();
 
-        auto process_info = windows_create_process(cmd_line, {}, DETACHED_PROCESS | CREATE_BREAKAWAY_FROM_JOB);
+        auto process_info =
+            windows_create_process(cmd_line, {}, CREATE_NEW_CONSOLE | CREATE_NO_WINDOW | CREATE_BREAKAWAY_FROM_JOB);
         if (!process_info.get())
         {
             Debug::print("cmd_execute_no_wait() failed with error code ", process_info.error(), "\n");

toolsrc/src/vcpkg/commands.cpp

@@ -24,6 +24,7 @@
 #include <vcpkg/commands.search.h>
 #include <vcpkg/commands.setinstalled.h>
 #include <vcpkg/commands.upgrade.h>
+#include <vcpkg/commands.upload-metrics.h>
 #include <vcpkg/commands.version.h>
 #include <vcpkg/commands.xvsinstances.h>
 #include <vcpkg/export.h>
@@ -38,9 +39,16 @@ namespace vcpkg::Commands
     {
         static const Version::VersionCommand version{};
         static const Contact::ContactCommand contact{};
+#if !VCPKG_DISABLE_METRICS && defined(_WIN32)
+        static const UploadMetrics::UploadMetricsCommand upload_metrics{};
+#endif // !VCPKG_DISABLE_METRICS && defined(_WIN32)
+
         static std::vector<PackageNameAndFunction<const BasicCommand*>> t = {
             {"version", &version},
             {"contact", &contact},
+#if !VCPKG_DISABLE_METRICS && defined(_WIN32)
+            {"x-upload-metrics", &upload_metrics},
+#endif // !VCPKG_DISABLE_METRICS && defined(_WIN32)
         };
         return t;
     }

toolsrc/src/vcpkg/commands.upload-metrics.cpp

@@ -0,0 +1,30 @@
+#include <vcpkg/base/checks.h>
+#include <vcpkg/base/files.h>
+
+#include <vcpkg/commands.upload-metrics.h>
+#include <vcpkg/metrics.h>
+#include <vcpkg/vcpkgcmdarguments.h>
+
+using namespace vcpkg;
+
+#if VCPKG_DISABLE_METRICS
+#error Attempt to build commands.upload-metrics.cpp when metrics are disabled.
+#endif // VCPKG_DISABLE_METRICS
+
+namespace vcpkg::Commands::UploadMetrics
+{
+    const CommandStructure COMMAND_STRUCTURE = {
+        create_example_string("x-upload-metrics metrics.txt"),
+        1,
+        1,
+    };
+
+    void UploadMetricsCommand::perform_and_exit(const VcpkgCmdArguments& args, Files::Filesystem& fs) const
+    {
+        (void)args.parse_arguments(COMMAND_STRUCTURE);
+        const auto& payload_path = args.command_arguments[0];
+        auto payload = fs.read_contents(payload_path).value_or_exit(VCPKG_LINE_INFO);
+        Metrics::g_metrics.lock()->upload(payload);
+        Checks::exit_success(VCPKG_LINE_INFO);
+    }
+}

toolsrc/src/vcpkg/metrics.cpp

@@ -491,41 +491,27 @@ namespace vcpkg::Metrics
 
         const fs::path temp_folder_path = fs::path(temp_folder) / "vcpkg";
         const fs::path temp_folder_path_exe =
-            temp_folder_path / Strings::format("vcpkgmetricsuploader-%s.exe", Commands::Version::base_version());
+            temp_folder_path / Strings::format("vcpkg-%s.exe", Commands::Version::base_version());
 #endif
 
-#if defined(_WIN32)
-
-        const fs::path exe_path = [&fs]() -> fs::path {
-            auto vcpkgdir = System::get_exe_path_of_current_process().parent_path();
-            auto path = vcpkgdir / "vcpkgmetricsuploader.exe";
-            if (fs.exists(path)) return path;
-
-            path = vcpkgdir / "scripts" / "vcpkgmetricsuploader.exe";
-            if (fs.exists(path)) return path;
-
-            return "";
-        }();
-
         std::error_code ec;
+#if defined(_WIN32)
         fs.create_directories(temp_folder_path, ec);
         if (ec) return;
-        fs.copy_file(exe_path, temp_folder_path_exe, fs::copy_options::skip_existing, ec);
+        fs.copy_file(
+            System::get_exe_path_of_current_process(), temp_folder_path_exe, fs::copy_options::skip_existing, ec);
         if (ec) return;
 #else
         if (!fs.exists("/tmp")) return;
         const fs::path temp_folder_path = "/tmp/vcpkg";
-        std::error_code ec;
-        fs.create_directory(temp_folder_path, ec);
-        // ignore error
-        ec.clear();
+        fs.create_directory(temp_folder_path, ignore_errors);
 #endif
         const fs::path vcpkg_metrics_txt_path = temp_folder_path / ("vcpkg" + generate_random_UUID() + ".txt");
         fs.write_contents(vcpkg_metrics_txt_path, payload, ec);
         if (ec) return;
 
 #if defined(_WIN32)
-        const std::string cmd_line = Strings::format("cmd /c \"start \"vcpkgmetricsuploader.exe\" \"%s\" \"%s\"\"",
+        const std::string cmd_line = Strings::format("\"%s\" x-upload-metrics \"%s\"\"",
                                                      fs::u8string(temp_folder_path_exe),
                                                      fs::u8string(vcpkg_metrics_txt_path));
         System::cmd_execute_no_wait(cmd_line);