Skip to content

Commit 4af9d22

Browse files
avi-starkwareclaude
andcommitted
ci,scripts: install LLVM before cargo tools
Reorder bootstrap and install_build_tools.sh so LLVM 19 is installed before cargo tools. This is needed because install_cargo_tools.sh will install starknet-native-compile which requires LLVM to build. Safe change: LLVM installation does not depend on Rust. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent fce8efa commit 4af9d22

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

.github/actions/bootstrap/action.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@ runs:
1212
- name: Setup Namespace Cache
1313
uses: ./.github/actions/namespace_cache
1414

15+
- name: Install cairo native.
16+
uses: ./.github/actions/setup_native_deps
17+
1518
- name: Install rust.
1619
uses: ./.github/actions/install_rust
1720
with:
1821
github_token: ${{ inputs.github_token }}
19-
20-
- name: Install cairo native.
21-
uses: ./.github/actions/setup_native_deps

scripts/install_build_tools.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,10 +129,11 @@ fi
129129
popd > /dev/null
130130
log_step "install_build_tools" "Project Rust toolchain ready: $(rustc --version)"
131131

132+
log_step "install_build_tools" "Running dependencies.sh..."
133+
./dependencies.sh
134+
log_step "install_build_tools" "dependencies.sh completed"
132135
log_step "install_build_tools" "Running install_cargo_tools.sh..."
133136
${SCRIPT_DIR}/install_cargo_tools.sh
134137
log_step "install_build_tools" "install_cargo_tools.sh completed"
135-
log_step "install_build_tools" "Running dependencies.sh..."
136-
./dependencies.sh
137138

138139
log_step "install_build_tools" "All build tools installed successfully!"

0 commit comments

Comments
 (0)