Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows build instructions in the README are wrong (OpenSSL 3) #203

Open
mbakhoff opened this issue Jun 9, 2022 · 1 comment
Open

Windows build instructions in the README are wrong (OpenSSL 3) #203

mbakhoff opened this issue Jun 9, 2022 · 1 comment

Comments

@mbakhoff
Copy link

mbakhoff commented Jun 9, 2022

I tried building the app using the build instructions from readme, but it fails.

Installed some dependencies:

.\vcpkg install --recurse --triplet x64-windows --clean-after-build gtest openssl
..other stuff..
cmake --build . --config ${BUILD_TYPE} --target installer

C:\Users\User\web-eid-app\install\web-eid.wxs(91): error LGHT0103: The system cannot find the file 'C:/vcpkg/installed/x64-windows/bin\libcrypto-1_1-x64.dll'. [C:\Users\User\web-eid-app\build\src\app\installer.vcxproj]

Looks like vcpkg now installs openssl 3.x, see microsoft/vcpkg@407bc79#diff-d274f14813540f3a0420c68884476dc2d6117b0af37bb9d43e48176c5f6b8ed9

Checking out microsoft/vcpkg@a69b65229b before running .\vcpkg install ... is a workaround.

Another note: "Building and testing in Windows" should say

-Use Powershell to run the following commands
+Use Developer PowerShell for VS 2019 to run the following commands

Otherwise cmake is not found

@mrts
Copy link
Member

mrts commented Jun 10, 2022

Thank you for bringing this up! We will update the README accordingly after v2.2.0 is out.

@mrts mrts changed the title build with windows fails (openssl 3) Windows build instructions in the README are wrong (OpenSSL 3) Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants