kou commented on code in PR #44644:
URL: https://github.com/apache/arrow/pull/44644#discussion_r1841206177


##########
ci/scripts/install_vcpkg.sh:
##########
@@ -53,4 +53,33 @@ if [ -f "${vcpkg_ports_patch}" ]; then
   echo "Patch successfully applied to the VCPKG port files!"
 fi
 
+if [ -n "${GITHUB_TOKEN:-}" ] && \
+     [ -n "${GITHUB_REPOSITORY_OWNER:-}" ] && \
+     [ "${VCPKG_BINARY_SOURCES:-}" = "clear;nuget,GitHub,readwrite" ] ; then

Review Comment:
   If `VCPKG_BINARY_SOURCES` is not `clear;nuget,GitHub,readwrite` (including 
not present case on `manylinux2014_aarch64`), we don't configure NuGet 
automatically.
   
   Ignoring not `clear;nuget,GitHub,readwrite` `VCPKG_BINARY_SOURCES` is for 
using other cache. For example, we may want to use local file cache for local 
`archery docker run`. But it's out-of-scope of this PR and it's not tested. If 
we need it, we can work on it as a separated task.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to