Skip to content

[ci] Use pip to install cmake and ninja on Windows - #27677

Merged
kripken merged 4 commits into
emscripten-core:mainfrom
sbc100:pip-cmake-ninja
Sep 8, 2026
Merged

[ci] Use pip to install cmake and ninja on Windows#27677
kripken merged 4 commits into
emscripten-core:mainfrom
sbc100:pip-cmake-ninja

Conversation

@sbc100

@sbc100 sbc100 commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Chocolatey's community feed is prone to rate-limiting, 503 Service Unavailable, and gateway timeouts on CI runners. Installing cmake and ninja wheels via pip is significantly faster and avoids dependency on community.chocolatey.org.

Its also much faster it seems: 9s vs 38s in the run I compared.

See: #27511

@sbc100
sbc100 requested review from dschuff and kripken September 5, 2026 01:19
@sbc100
sbc100 requested a review from kleisauke September 5, 2026 16:31
Chocolatey's community feed is prone to rate-limiting, 503 Service
Unavailable, and gateway timeouts on CI runners. Installing cmake and
ninja wheels via `pip` is significantly faster and avoids dependency
on community.chocolatey.org.

See: emscripten-core#27511
Comment thread .circleci/config.yml Outdated

@kripken kripken left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm % comments

dschuff and others added 2 commits September 8, 2026 15:12
@dschuff

dschuff commented Sep 8, 2026

Copy link
Copy Markdown
Member

@kripken since Sam is out this week, maybe we can go ahead and apply this to improve reliability?

@kripken kripken left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sounds good, some more typo fixes and let's land this.

Comment thread .circleci/config.yml Outdated
@kripken
kripken enabled auto-merge (squash) September 8, 2026 22:22
@kripken
kripken merged commit 05fbfb2 into emscripten-core:main Sep 8, 2026
42 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.

3 participants