Skip to content

Fix Windows CI: Use pre-installed 7zip on GHA runners - #127

Merged
sosukesuzuki merged 1 commit into
mainfrom
claude/fix-windows-ci-external-7zip
Jan 5, 2026
Merged

sosukesuzuki merged 1 commit into
mainfrom
claude/fix-windows-ci-external-7zip

Conversation

@sosukesuzuki

Copy link
Copy Markdown
Member

Problem

Windows CI is still failing because LLVM installation requires 7zip to extract the installer, and downloads from 7-zip.org fail with SSL errors on GitHub Actions runners.

Solution

GitHub Actions Windows runners have 7zip pre-installed at C:\Program Files\7-Zip. Configure scoop to use this external 7zip:

"C:\Program Files\7-Zip" >> $Env:GITHUB_PATH
scoop config use_external_7zip true
scoop install ninja llvm@19.1.7

This also simplifies the LLVM installation by using the standard llvm@19.1.7 syntax instead of a pinned manifest URL.

Downloads from 7-zip.org fail with SSL errors on GitHub Actions runners.
Configure scoop to use the pre-installed 7zip at 'C:\Program Files\7-Zip'
instead of trying to download it.

This also simplifies the LLVM installation by using the standard
'llvm@19.1.7' syntax instead of a pinned manifest URL.

Claude-Generated-By: Claude Code (cli/claude-opus-4-5=100%)
Claude-Steers: 3
Claude-Permission-Prompts: 1
Claude-Escapes: 0
@sosukesuzuki
sosukesuzuki merged commit d6c402c into main Jan 5, 2026
1 check passed
@coderabbitai

coderabbitai Bot commented Jan 5, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

The Windows installation flow for LLVM and Ninja in the build workflow was replaced with a Scoop-based approach. It now uses the pre-installed 7-zip on Windows runners, installs LLVM 19.1.7 and Ninja via Scoop, and updates the PATH configuration accordingly.

Changes

Cohort / File(s) Change Summary
GitHub Actions Workflow
.github/workflows/build-reusable.yml
Updated Windows installation flow for LLVM and Ninja to use Scoop with pre-installed 7-zip, avoiding SSL-pinned downloads. Configures Scoop to use external 7-zip, installs LLVM 19.1.7 and Ninja via Scoop, and updates PATH to include 7-zip and LLVM binaries.

📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 6513903 and 006e195.

📒 Files selected for processing (1)
  • .github/workflows/build-reusable.yml

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant