PR comments WIP

Author: vicroms

include/vcpkg/base/contractual-constants.h

@@ -207,7 +207,6 @@ namespace vcpkg
     inline constexpr StringLiteral SwitchDebug = "debug";
     inline constexpr StringLiteral SwitchDebugBin = "debug-bin";
     inline constexpr StringLiteral SwitchDebugEnv = "debug-env";
-    inline constexpr StringLiteral SwitchDeleteFileAfterUpload = "delete-file-after-upload";
     inline constexpr StringLiteral SwitchDgml = "dgml";
     inline constexpr StringLiteral SwitchDisableMetrics = "disable-metrics";
     inline constexpr StringLiteral SwitchDot = "dot";

include/vcpkg/base/curl.h

@@ -4,8 +4,9 @@
 
 VCPKG_MSVC_WARNING(push)
 // note: disable warning triggered by curl headers
-// ws2tcpip.h(968): warning C6101: Returning uninitialized memory '*Mtu':  A successful path through the function does not set the named _Out_ parameter.
-VCPKG_MSVC_WARNING(disable: 6101)
+// ws2tcpip.h(968): warning C6101: Returning uninitialized memory '*Mtu':  A successful path through the function does
+// not set the named _Out_ parameter.
+VCPKG_MSVC_WARNING(disable : 6101)
 #include <curl/curl.h>
 #include <curl/multi.h>
 VCPKG_MSVC_WARNING(pop)

include/vcpkg/base/message-data.inc.h

@@ -863,7 +863,6 @@ DECLARE_MESSAGE(CmdUpdateRegistryAllOrTargets,
                 (),
                 "",
                 "Update registry requires either a list of artifact registry names or URiIs to update, or --all.")
-DECLARE_MESSAGE(CmdUploadMetricsDeleteFileAfterUpload, (), "", "Delete metrics payload file after upload")
 DECLARE_MESSAGE(CmdUpgradeOptNoDryRun, (), "", "Actually upgrade")
 DECLARE_MESSAGE(CmdUpgradeOptNoKeepGoing, (), "", "Stop installing packages on failure")
 DECLARE_MESSAGE(CmdUseExample1,
@@ -964,17 +963,17 @@ DECLARE_MESSAGE(CreatingNuGetPackage, (), "", "Creating NuGet package...")
 DECLARE_MESSAGE(CreatingZipArchive, (), "", "Creating zip archive...")
 DECLARE_MESSAGE(CreationFailed, (msg::path), "", "Creating {path} failed.")
 DECLARE_MESSAGE(CurlFailedGeneric,
-                (msg::exit_code, msg::error_msg),
+                (msg::exit_code),
                 "curl is the name of a program, see curl.se.",
-                "curl operation failed with error code {exit_code} ({error_msg}).")
+                "curl operation failed with error code {exit_code}")
 DECLARE_MESSAGE(CurlFailedGenericNoRetryAddendum,
                 (msg::url),
                 "curl is the name of a program, see curl.se.",
                 "This is a non-transient error, won't retry downloading from {url}.")
-DECLARE_MESSAGE(CurlFailedGenericWithRetry,
-                (msg::exit_code, msg::error_msg, msg::count, msg::value),
-                "curl is the name of a program, see curl.se. {value} is the maximum amount of retries.",
-                "curl operation failed with error code {exit_code} ({error_msg}) retry {count} of {value}.")
+DECLARE_MESSAGE(CurlFailedTimeoutRetry,
+                (msg::count, msg::value),
+                "{count} is the current retry atttempt, {value} is the maximum amount of retries.",
+                "The operation timed out retry {count} of {value}.")
 DECLARE_MESSAGE(CurlFailedHttpResponse,
                 (msg::exit_code),
                 "curl is the name of a program, see curl.se.",
@@ -983,10 +982,10 @@ DECLARE_MESSAGE(CurlFailedHttpResponseWithRetry,
                 (msg::exit_code, msg::count, msg::value),
                 "curl is the name of a program, see curl.se. {value} is the maximum amount of retries.",
                 "curl operation failed with HTTP response code {exit_code} retry {count} of {value}.")
-DECLARE_MESSAGE(CurlFailedToPutHttp,
-                (msg::exit_code, msg::error_msg, msg::url, msg::value),
+DECLARE_MESSAGE(CurlFailedToPut,
+                (msg::url, msg::value),
                 "curl is the name of a program, see curl.se. {value} is an HTTP status code",
-                "curl failed to put file to {url} with exit code {exit_code} ({error_msg}) and http code {value}.")
+                "curl failed to PUT file to {url} with response code {value}.")
 DECLARE_MESSAGE(CurrentCommitBaseline,
                 (msg::commit_sha),
                 "",

locales/messages.json

@@ -502,7 +502,6 @@
   "CmdUpdateRegistrySynopsis": "Re-downloads an artifact registry",
   "CmdUpgradeOptNoDryRun": "Actually upgrade",
   "CmdUpgradeOptNoKeepGoing": "Stop installing packages on failure",
-  "CmdUploadMetricsDeleteFileAfterUpload": "Delete metrics payload file after upload",
   "CmdUseExample1": "vcpkg use <artifact name>",
   "_CmdUseExample1.comment": "This is a command line, only the <artifact name> part should be localized.",
   "CmdUseSynopsis": "Activate a single artifact in this shell",
@@ -561,18 +560,18 @@
   "CreatingZipArchive": "Creating zip archive...",
   "CreationFailed": "Creating {path} failed.",
   "_CreationFailed.comment": "An example of {path} is /foo/bar.",
-  "CurlFailedGeneric": "curl operation failed with error code {exit_code} ({error_msg}).",
-  "_CurlFailedGeneric.comment": "curl is the name of a program, see curl.se. An example of {exit_code} is 127. An example of {error_msg} is File Not Found.",
+  "CurlFailedGeneric": "curl operation failed with error code {exit_code}",
+  "_CurlFailedGeneric.comment": "curl is the name of a program, see curl.se. An example of {exit_code} is 127.",
   "CurlFailedGenericNoRetryAddendum": "This is a non-transient error, won't retry downloading from {url}.",
   "_CurlFailedGenericNoRetryAddendum.comment": "curl is the name of a program, see curl.se. An example of {url} is https://github.com/microsoft/vcpkg.",
-  "CurlFailedGenericWithRetry": "curl operation failed with error code {exit_code} ({error_msg}) retry {count} of {value}.",
-  "_CurlFailedGenericWithRetry.comment": "curl is the name of a program, see curl.se. {value} is the maximum amount of retries. An example of {exit_code} is 127. An example of {error_msg} is File Not Found. An example of {count} is 42.",
   "CurlFailedHttpResponse": "curl operation failed with HTTP response code {exit_code}.",
   "_CurlFailedHttpResponse.comment": "curl is the name of a program, see curl.se. An example of {exit_code} is 127.",
   "CurlFailedHttpResponseWithRetry": "curl operation failed with HTTP response code {exit_code} retry {count} of {value}.",
   "_CurlFailedHttpResponseWithRetry.comment": "curl is the name of a program, see curl.se. {value} is the maximum amount of retries. An example of {exit_code} is 127. An example of {count} is 42.",
-  "CurlFailedToPutHttp": "curl failed to put file to {url} with exit code {exit_code} ({error_msg}) and http code {value}.",
-  "_CurlFailedToPutHttp.comment": "curl is the name of a program, see curl.se. {value} is an HTTP status code An example of {exit_code} is 127. An example of {error_msg} is File Not Found. An example of {url} is https://github.com/microsoft/vcpkg.",
+  "CurlFailedTimeoutRetry": "The operation timed out retry {count} of {value}.",
+  "_CurlFailedTimeoutRetry.comment": "{count} is the current retry atttempt, {value} is the maximum amount of retries. An example of {count} is 42.",
+  "CurlFailedToPut": "curl failed to PUT file to {url} with response code {value}.",
+  "_CurlFailedToPut.comment": "curl is the name of a program, see curl.se. {value} is an HTTP status code An example of {url} is https://github.com/microsoft/vcpkg.",
   "CurrentCommitBaseline": "You can use the current commit as a baseline, which is:\n\t\"builtin-baseline\": \"{commit_sha}\"",
   "_CurrentCommitBaseline.comment": "An example of {commit_sha} is 7cfad47ae9f68b183983090afd6337cd60fd4949.",
   "CycleDetectedDuring": "cycle detected during {spec}:",

src/vcpkg.cpp

@@ -136,12 +136,6 @@ namespace
 
     void inner(const Filesystem& fs, const VcpkgCmdArguments& args, const BundleSettings& bundle)
     {
-        // Ensure that we call curl_global_init only once in the program.
-        if (CURLE_OK != curl_global_init_status())
-        {
-            Debug::println("Failed to initialize CURL");
-        }
-
         // track version on each invocation
         get_global_metrics_collector().track_string(StringMetric::VcpkgVersion, vcpkg_executable_version);