jduo commented on code in PR #13532:
URL: https://github.com/apache/arrow/pull/13532#discussion_r918134626


##########
.github/workflows/cpp.yml:
##########
@@ -197,12 +197,21 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os:
-          - windows-2019
         include:
           - os: windows-2019
-            name: Windows 2019
+            name: Windows 2019 x64
             generator: Visual Studio 16 2019
+            platform: x64
+            openssl-n-bits: 64

Review Comment:
   openssl-n-bits is newly created. Previously the build looked for OpenSSL in 
the hardcoded directory C:\Program Files\OpenSSL-Win64 but it needs to be 
C:\Program Files (x86)\OpenSSL-Win32 when compiling the 32-bit version.
   This is from where choclatey installs OpenSSL:
   I'm not sure if the install location is documented.



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to