Skip to content

Commit 53e7c30

Browse files
committed
Revert "fix(boringssl-ci): install NASM on Windows runners"
This reverts commit 10bb89f.
1 parent 8509ed7 commit 53e7c30

1 file changed

Lines changed: 0 additions & 12 deletions

File tree

.github/workflows/boringssl.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -394,18 +394,6 @@ jobs:
394394
steps.restore-checkpoint.outputs.build-required == 'true')
395395
uses: ./.github/actions/install-llvm-mingw
396396

397-
- name: Install NASM (Windows)
398-
# BoringSSL's CMakeLists.txt unconditionally calls
399-
# enable_language(ASM_NASM) on Windows regardless of target arch.
400-
# Linux/macOS runners have nasm pre-installed; Windows runners do
401-
# not. Install on every Windows job (~2MB) so configure succeeds;
402-
# arm64 builds skip any NASM-compiled translation units anyway.
403-
if: |
404-
matrix.os == 'windows' &&
405-
((steps.boringssl-checkpoint-cache.outputs.cache-hit != 'true' || steps.validate-cache.outputs.valid != 'true') ||
406-
steps.restore-checkpoint.outputs.build-required == 'true')
407-
uses: ilammy/setup-nasm@72793074d3c8cdda771dba85f6deafe00623038b # v1.5.2 (2025-02-24)
408-
409397
- name: Build BoringSSL native (macOS/Windows)
410398
if: |
411399
matrix.os != 'linux' &&

0 commit comments

Comments
 (0)