Skip to content

Conversation

@cipolleschi
Copy link
Contributor

@cipolleschi cipolleschi commented Apr 3, 2025

Summary:

Windows executor has been updated by github and they now ship with CMake 4.0. (actions/runner-images#11926)

This is not compatible with building HermesC on windows machines. This change attempts to downgrade cmake to 3.31.6, the same version we used to use before.

Changelog:

[Internal] - downgrage Cmake to 3.31.6

Test Plan:

GHA

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Apr 3, 2025
@cipolleschi cipolleschi changed the base branch from main to 0.76-stable April 3, 2025 06:41
@cipolleschi cipolleschi changed the title cipolleschi/downgrade cmake windows [CI][Windows] Downgrade CMake to 3.31.6 Apr 3, 2025
@cipolleschi cipolleschi force-pushed the cipolleschi/downgrade-cmake-windows branch from 94bb1c8 to 0880282 Compare April 3, 2025 06:46
@github-actions
Copy link

github-actions bot commented Apr 3, 2025

Job Summary for Gradle

Test All :: build_android
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
react-native-github publishAllToMavenTempLocal build 8.11.1 Build Scan not published

@cipolleschi cipolleschi marked this pull request as ready for review April 3, 2025 06:56
@cipolleschi cipolleschi force-pushed the cipolleschi/downgrade-cmake-windows branch from 0880282 to 13a0a7c Compare April 3, 2025 07:23
@cipolleschi cipolleschi requested review from cortinico and fabriziocucci and removed request for fabriziocucci April 3, 2025 07:26
@cipolleschi cipolleschi merged commit 187f9c4 into 0.76-stable Apr 3, 2025
47 checks passed
@cipolleschi cipolleschi deleted the cipolleschi/downgrade-cmake-windows branch April 4, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants