PR comments WIP

Author: vicroms

azure-pipelines/end-to-end-tests-dir/asset-caching.ps1

@@ -188,26 +188,10 @@ if (-not ($actual -match $expected)) {
 }
 
 # ... also with multiple authoritative URLs
-if ($IsWindows) {
-    Refresh-TestRoot
-    $expected = @(
-    "^Downloading example3\.html, trying https://nonexistent\.example\.com",
-    "Trying https://raw\.githubusercontent\.com/microsoft/vcpkg-tool/1767aaee7b229c609f7ad5cf2f57b6a6cc309fb8/LICENSE\.txt",
-    "Successfully downloaded example3\.html",
-    "$"
-    ) -join "`n"
-
-    $actual = Run-VcpkgAndCaptureOutput @commonArgs x-download "$TestDownloadsRoot/example3.html" --sha512 65077997890f66f6041bb3284bb7b88e27631411ccbc253201ca4e00c4bcc58c0d77edffda4975498797cc10772c7fd68fbeb13cc4ac493a3471a9d49e5b6f24 --url https://nonexistent.example.com --url https://raw.githubusercontent.com/microsoft/vcpkg-tool/1767aaee7b229c609f7ad5cf2f57b6a6cc309fb8/LICENSE.txt
-    Throw-IfFailed
-    if (-not ($actual -match $expected)) {
-        throw "Failure: azurl (no), x-block-origin (no), asset-cache (n/a), download (succeed)"
-    }
-}
-
-# Force curl with --header
 Refresh-TestRoot
 $expected = @(
 "^Downloading example3\.html, trying https://nonexistent\.example\.com",
+"error: curl operation failed with error code 6 \(Couldn't resolve host name\)\. This is a non-transient error, won't retry downloading from https://nonexistent\.example\.com",
 "Trying https://raw\.githubusercontent\.com/microsoft/vcpkg-tool/1767aaee7b229c609f7ad5cf2f57b6a6cc309fb8/LICENSE\.txt"
 "Successfully downloaded example3\.html",
 "$"

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

@@ -1,15 +1,15 @@
+#pragma once
 
-#ifdef _MSC_VER
-#pragma warning(push)           // Save current warning state
-#pragma warning(disable : 6101) // Disable specific warning (e.g., warning 4996)
-#endif
+#include <vcpkg/base/pragmas.h>
 
+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)
 #include <curl/curl.h>
 #include <curl/multi.h>
-
-#ifdef _MSC_VER
-#pragma warning(pop)
-#endif
+VCPKG_MSVC_WARNING(pop)
 
 namespace vcpkg
 {

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,13 +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}).")
-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}.")
+                "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(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.",
@@ -979,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),
                 "",

include/vcpkg/metrics.h

@@ -198,5 +198,5 @@ namespace vcpkg
     extern std::atomic<bool> g_should_send_metrics;
 
     void flush_global_metrics(const Filesystem&);
-    bool curl_upload_metrics(StringView payload);
+    bool curl_upload_metrics(const std::string& payload);
 }

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,16 +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.",
-  "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.",
+  "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.",
   "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);