Skip to content

Explicitly set the C version when compiling GMP - #1569

Merged
Tim Hutt (Timmmm) merged 1 commit into
riscv:masterfrom
Timmmm:user/timh/explicit_gmp_c_version
Feb 27, 2026
Merged

Tim Hutt (Timmmm) merged 1 commit into
riscv:masterfrom
Timmmm:user/timh/explicit_gmp_c_version

Conversation

@Timmmm

Copy link
Copy Markdown
Collaborator

This avoids compilation code due to its ./configure having buggy code that causes errors with newer C versions.

Fixes #1511

This avoids compilation code due to its `./configure` having buggy code that causes errors with newer C versions.

Fixes riscv#1511

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Would be good to add a static build to CI.

@Timmmm Tim Hutt (Timmmm) added the will be merged Scheduled to be merged soon if nobody objects label Feb 27, 2026
@github-actions

Copy link
Copy Markdown

Test Results

2 314 tests  ±0   2 314 ✅ ±0   31m 34s ⏱️ +3s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 4a789e9. ± Comparison against base commit 7f37e03.

@Timmmm
Tim Hutt (Timmmm) added this pull request to the merge queue Feb 27, 2026
Merged via the queue into riscv:master with commit 13d19ac Feb 27, 2026
15 of 16 checks passed
@Timmmm
Tim Hutt (Timmmm) deleted the user/timh/explicit_gmp_c_version branch March 2, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

will be merged Scheduled to be merged soon if nobody objects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GMP build fails with recent compilers

3 participants