Skip to content

Conversation

@rjindael
Copy link
Contributor

Fixes #47536

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

This pull request merges the following two commits from the GMP upstream repository:

These two revisions are necessary for producing a proper configure on x64-linux, but have yet to be part of a release tarball which vcpkg can download from. Therefore, this pull request includes a pre-install patch that applies them.

@rjindael
Copy link
Contributor Author

@microsoft-github-policy-service agree

@JavierMatosD
Copy link
Contributor

@rjindael, I'm not seeing this port fail for x64-linux in our CI. Can you give me the repro steps?

@JavierMatosD JavierMatosD marked this pull request as draft September 29, 2025 20:15
@rjindael
Copy link
Contributor Author

rjindael commented Sep 29, 2025

@rjindael, I'm not seeing this port fail for x64-linux in our CI. Can you give me the repro steps?

Hi, full reproduction steps are available at this issue, with logs and compiler info: #47536. I am running Arch Linux with the latest Clang/LLVM.

Please note this was already patched in upstream GMP. Microsoft CI may not be using a compiler that uses C23, and therefore it will not fail with this conftest, seeing as it has a method that fails because that form of declaration became deprecated.

@JavierMatosD JavierMatosD marked this pull request as ready for review September 29, 2025 20:50
@JavierMatosD JavierMatosD merged commit 4ed8479 into microsoft:master Sep 29, 2025
18 checks passed
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

Successfully merging this pull request may close these issues.

[gmp] build error on x64-linux

2 participants