8000 ci: pin cmake to 3.28 on linux by cpu · Pull Request #2408 · rustls/rustls · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

ci: pin cmake to 3.28 on linux #2408

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

Closed
wants to merge 1 commit into from
Closed

Conversation

cpu
Copy link
Member
@cpu cpu commented Apr 1, 2025

cmake 4.0 is not compatible w/ aws-lc FIPS builds until the forthcoming release. The runner image switched the version out from under us. Pin it back until one side or the other sorts itself out.

The runner-image-provided cmake@4.0 is in /usr/local/cmake and has higher $PATH priority. Rather than munge with the PATH and change other things we symbolic link the installed cmake@3.28 over-top.

Fixes the build failures on main.

See aws/aws-lc-rs#755, actions/runner-images#11926

cmake 4.0 is not compatible w/ aws-lc until the forthcoming release. The
runner image switched the version out from under us. Pin it back until
one side or the other sorts itself out.
@cpu cpu self-assigned this Apr 1, 2025
@cpu
Copy link
Member Author
cpu commented Apr 1, 2025

rustls / Build+test (stable, windows-latest) (pull_request)Failing after 5m

argh, I was hoping it was just the Linux runners that had the issue. I'm not sure I know how to easily remedy this on Windows 🤔

Copy link
codecov bot commented Apr 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.16%. Comparing base (bdb3036) to head (357ff28).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2408   +/-   ##
=======================================
  Coverage   96.16%   96.16%           
=======================================
  Files          94       94           
  Lines       22815    22815           
=======================================
  Hits        21939    21939           
  Misses        876      876           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cpu
Copy link
Member Author
cpu commented Apr 1, 2025

I'm not sure I know how to easily remedy this on Windows

Maybe best to exclude FIPS builds on Windows/MacOS for now?

@benj5378
Copy link
benj5378 commented Apr 1, 2025

@cpu see fix for Windows and Ubuntu here

@cpu
Copy link
Member Author
cpu commented Apr 1, 2025

Closing this - I think an aws-lc-fips-sys 0.13.5 w/ CMake 4.0 support was released. Let's take that update instead of reverting CMake.

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.

2 participants
0