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

zlib:x64-windows build failure #12582

Closed
Hydr4bytes opened this issue Jul 26, 2020 · 8 comments · Fixed by #12467
Closed

zlib:x64-windows build failure #12582

Hydr4bytes opened this issue Jul 26, 2020 · 8 comments · Fixed by #12467
Assignees
Labels
category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly

Comments

@Hydr4bytes
Copy link

Host Environment

  • OS: Windows
  • Compiler: revision

To Reproduce
Steps to reproduce the behavior:
./vcpkg install zlib:x64-windows

Failure logs
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
Command failed: C:/dev/vcpkg/downloads/tools/msys2/msys64/usr/bin/bash.exe --noprofile --norc -c "pacman -S --noconfirm --needed pkg-config"
Working Directory: C:/dev/vcpkg/downloads/tools/msys2
Error code: 1
See logs for more information:
C:\dev\vcpkg\buildtrees\zlib\msys-pacman-x64-windows-err.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_acquire_msys.cmake:127 (vcpkg_execute_required_process)
scripts/cmake/vcpkg_fixup_pkgconfig.cmake:281 (vcpkg_acquire_msys)
ports/zlib/portfile.cmake:46 (vcpkg_fixup_pkgconfig)
scripts/ports.cmake:79 (include)

Error: Building package zlib:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: zlib:x64-windows
Vcpkg version: 2020.06.15-nohash

@StarGate-One
Copy link
Contributor

Please attache/upload the file C:\dev\vcpkg\buildtrees\zlib\msys-pacman-x64-windows-err.log to this issue for review and possible corrective actions.

@amrosado
Copy link

I'm having a similar issue and I've been trying to figure it out. I believe the output might look like this.

error: mingw32: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" is unknown
error: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" could not be looked up remotely
error: mingw64: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" is unknown
error: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" could not be looked up remotely
error: msys: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" is unknown
error: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" could not be looked up remotely
error: database 'mingw32' is not valid (invalid or corrupted database (PGP signature))
error: database 'mingw64' is not valid (invalid or corrupted database (PGP signature))
error: database 'msys' is not valid (invalid or corrupted database (PGP signature))

@Goddard
Copy link

Goddard commented Jul 26, 2020

Also effects me.

@BenKey
Copy link
Contributor

BenKey commented Jul 26, 2020

I am getting the same issue when building ICU.

@StarGate-One
Copy link
Contributor

See #12565 - There are 2 issues:

  1. vcpkg_fixup_pkconfig.cmake needs to be reworked - PR [vcpkg_fixup_pkgconfig] Remove required in first find_program call for pkg-config #12569 is in the works but in the mean time you can make the changes shown by myself @StarGate-One to the script found in <vcpkg-root>\scripts\cmake to temporarily fix this issue.
    vcpkg_fixup_pkgconfig.cmake.txt

  2. vcpkg_acquire_msys.cmake needs to have the MSYS/mingw crypto keyrings added, @ImmaturityRules has attached a copy to this same issue which fix this issue, but a PR needs to be done with those changes. You can download dthe file and put it in the same location <vcpkg-root>\scripts\cmake to temporarily fix this issue.
    vcpkg_acquire_msys.cmake.txt

I have attached a copy of both, download, remove the .txt extension and copy to <vcpkg-root>\scripts\cmake folder and it should work.

@PhoebeHui
Copy link
Contributor

PhoebeHui commented Jul 27, 2020

We have a exist PR #12467 to fix msys issue, appreciate your effort very much.

@PhoebeHui PhoebeHui added the category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly label Jul 27, 2020
@prokotg
Copy link

prokotg commented Jul 27, 2020

Same MSYS2 thing withm Glibmm, just leting know

@kcgen
Copy link

kcgen commented Jul 31, 2020

@strega-nil , can you reopen this? This issue is still affecting CI workflows today (4 days after it was closed): dosbox-staging/dosbox-staging#524

The manual work-around posted by @StarGate-One above is a starting point to the solution, but until its tested, integrated into the production code-base, and proven to be passing by end-users, then the issue should stay open.

Because this affects zlib, the ripple effect has been quite massive - with GitHub CI workflows tanking almost everywhere that uses vcpkg. Can't wait for the fix! thank you all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants