-
Notifications
You must be signed in to change notification settings - Fork 46
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
Build fails on Windows since 0.4.8: error C2039: 'clamp': is not a member of 'std'
#126
Comments
SimonSapin
added a commit
to apollographql/apollo-rs
that referenced
this issue
Nov 25, 2024
SimonSapin
added a commit
to apollographql/apollo-rs
that referenced
this issue
Nov 25, 2024
Hm, the gcc-style flags being passed along to cl.exe seem very suspect. https://github.com/rust-fuzz/libfuzzer/blob/main/build.rs#L34 is wrong and a likely cause. Should be using the I don't have msvc around to test, though. |
github-merge-queue bot
pushed a commit
to smithy-lang/smithy-rs
that referenced
this issue
Nov 27, 2024
~If CI fails, commit the necessary fixes to this PR until all checks pass.~ ~If changes are required to [crateNameToLastKnownWorkingVersions](https://github.com/smithy-lang/smithy-rs/blob/92916b5484cdfef9ff58540ebf5e845eeeccf860/aws/sdk/build.gradle.kts#L504), revert the first commit in the PR, run `./gradlew aws:sdk:cargoUpdateAllLockfiles`, and commit the updated lockfiles.~ This PR pins `libfuzzer-sys` to 0.4.7 due to rust-fuzz/libfuzzer#126 while upgrading dependencies. --------- Co-authored-by: ysaito1001 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
CI for the apollo-rs project started failing on Windows since libfuzzer-sys 0.4.8 was released. The most relevant part of the output seems to be:
Full output from Cargo:
The text was updated successfully, but these errors were encountered: