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

[gmp,nettle,hogweed] Add new ports #10613

Merged
merged 36 commits into from
May 8, 2020

Conversation

Neumann-A
Copy link
Contributor

@Neumann-A Neumann-A commented Mar 30, 2020

Describe the pull request
New ports: gmp, nettle (including hogweed)

requires (and includes for CI testing) #9861 (this PR starts at 9307095)

closes #9021

Felt like these should be separated from the X window PR

CI seems to not pass env YASMPATH correctly to cl.
Since YASM is on PATH -> remove $(VCInstallDir) as the default for YasmPath
@Neumann-A
Copy link
Contributor Author

Hmm:

Starting package 2/2: gmp:x64-windows
Building package gmp[core]:x64-windows...
-- Using cached D:/vcpkg_common/downloads/ShiftMediaProject-gmp-e140dfc8668e96d7e56cbd46467945adcc6b3cc4.tar.gz
-- Extracting source D:/vcpkg_common/downloads/ShiftMediaProject-gmp-e140dfc8668e96d7e56cbd46467945adcc6b3cc4.tar.gz
-- Applying patch vs.build.patch
-- Using source at D:/qt/buildtrees/gmp/src/adcc6b3cc4-a9dac07c3d
-- Building SMP/libgmp.sln for Release
-- Building SMP/libgmp.sln for Debug
-- Installing: D:/qt/packages/gmp_x64-windows/share/gmp/copyright
-- Fixing pkgconfig
-- Fixing pkgconfig - release
-- Checking file: D:/qt/packages/gmp_x64-windows/lib/pkgconfig/gmp.pc
-- Checking file: D:/qt/packages/gmp_x64-windows/lib/pkgconfig/gmpxx.pc
-- Fixing pkgconfig - debug
-- Checking file: D:/qt/packages/gmp_x64-windows/debug/lib/pkgconfig/gmp.pc
-- Checking file: D:/qt/packages/gmp_x64-windows/debug/lib/pkgconfig/gmpxx.pc
-- Fixing pkgconfig --- finished
-- Performing post-build validation
-- Performing post-build validation done
Building package gmp[core]:x64-windows... done
Installing package gmp[core]:x64-windows...
Installing package gmp[core]:x64-windows... done

without failure logs from CI I won't know what went wrong.

@MVoz
Copy link
Contributor

MVoz commented Mar 31, 2020

@Neumann-A version msvc ? maybe does not find yasm.exe

problem msvc 2019
#10523

@Neumann-A Neumann-A marked this pull request as draft April 9, 2020 18:24
@Neumann-A
Copy link
Contributor Author

switched to draft until #9861 is merged

@JackBoosY JackBoosY removed the info:reviewed Pull Request changes follow basic guidelines label Apr 10, 2020
@Neumann-A Neumann-A marked this pull request as ready for review April 29, 2020 12:31
@Neumann-A
Copy link
Contributor Author

Requesting a clean rebuild. 3ba8f9f was green and I did not touch anything important since then. The UWP log also looks like an old failure.

@JackBoosY
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label May 8, 2020
@strega-nil
Copy link
Contributor

Cool, thanks much @Neumann-A :D

@strega-nil strega-nil merged commit 18ab200 into microsoft:master May 8, 2020
@Neumann-A Neumann-A deleted the add_gmp_nettle branch May 8, 2020 21:02
@JackBoosY
Copy link
Contributor

When building gmp:x86-windows:

    2>F:\vcpkg\installed\x86-windows\share\vs-yasm\yasm.targets(50,5): error MSB3721: The command ""F:/vcpkg/downloads/tools/yasm/1.3.0.6/"yasm.exe -Xvc -f win32 -o "F:\vcpkg\buildtrees\gmp\x86-windows-rel\adcc6b3cc4-2b6258d695\SMP\obj\ReleaseDLL\Win32\libgmp\addmul_1.obj" -rraw -pgas  "F:\vcpkg\buildtrees\gmp\x86-windows-rel\adcc6b3cc4-2b6258d695\SMP\mpn\x86\addmul_1.s"" exited with code -1073741819. [F:\vcpkg\buildtrees\gmp\x86-windows-rel\adcc6b3cc4-2b6258d695\SMP\libgmp.vcxproj]

@Neumann-A
Copy link
Contributor Author

Elapsed time for package vs-yasm:x86-windows: 1.119 s
Starting package 2/2: gmp:x86-windows
Building package gmp[core]:x86-windows...
-- Using cached D:/vcpkg_common/downloads/ShiftMediaProject-gmp-e140dfc8668e96d7e56cbd46467945adcc6b3cc4.tar.gz
-- Using source at D:/vpkg/buildtrees/gmp/src/adcc6b3cc4-2b6258d695
-- Building SMP/libgmp.sln for Release
-- Building SMP/libgmp.sln for Debug
-- Installing: D:/vpkg/packages/gmp_x86-windows/share/gmp/copyright
-- Fixing pkgconfig
-- Fixing pkgconfig - release
-- Checking file: D:/vpkg/packages/gmp_x86-windows/lib/pkgconfig/gmp.pc
-- Checking file: D:/vpkg/packages/gmp_x86-windows/lib/pkgconfig/gmpxx.pc
-- Fixing pkgconfig - debug
-- Checking file: D:/vpkg/packages/gmp_x86-windows/debug/lib/pkgconfig/gmp.pc
-- REL PATH: ../../
-- Checking file: D:/vpkg/packages/gmp_x86-windows/debug/lib/pkgconfig/gmpxx.pc
-- REL PATH: ../../
-- Fixing pkgconfig --- finished
-- Performing post-build validation
-- Performing post-build validation done
Building package gmp[core]:x86-windows... done
Installing package gmp[core]:x86-windows...
Installing package gmp[core]:x86-windows... done
Elapsed time for package gmp:x86-windows: 1.297 min

Total elapsed time: 1.316 min

No issue here from a clean vcpkg directory. Please delete the old buildtree?.

@JackBoosY
Copy link
Contributor

@Neumann-A Clone vcpkg in the new directory and run the same command result: same error.

@strega-nil
Copy link
Contributor

@Neumann-A I'm having the same error, fwiw

@strega-nil
Copy link
Contributor

@Neumann-A looks like yasm is segfaulting

@Neumann-A
Copy link
Contributor Author

Hmmm interesting. I went to my Laptop and had the same problem.
I then updated Visual Studio 2019, Git for Windows and Windows itself with the latest update, restarted and now the failure is gone.
So maybe update your tools? Cannot really help here since I only could reproduce the failure once.

@strega-nil
Copy link
Contributor

that is... really odd. However, it did work after updating my tools.

@JackBoosY
Copy link
Contributor

This issue blocked my PR #10767.

@Neumann-A
Copy link
Contributor Author

Which VS version is running on CI?

@JackBoosY
Copy link
Contributor

When using Visual Studio 2019 to build gmp, the error disappeared.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[New Port Request] Nettle
5 participants