Skip to content

Commit dae1d31

Browse files
committed
ci: replace host nasm with one-time sandbox index refresh (mcpp#232)
mcpp 0.0.96 self-bootstraps nasm, but its release archive vendors an index snapshot cut before xim-pkgindex#398 fixed the nasm descriptor, and the bootstrap is offline-first — it would install the broken vendored nasm.lua as an empty payload. A single 'xlings update' against the sandbox home (~/.mcpp/registry) before the workspace run lets the lazy bootstrap resolve the fixed descriptor. Drop the step once a mcpp release vendors a post-#398 snapshot.
1 parent bd99e1d commit dae1d31

1 file changed

Lines changed: 9 additions & 6 deletions

File tree

‎.github/workflows/validate.yml‎

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -213,14 +213,17 @@ jobs:
213213
# `xlings install nasm`), lazily, only when the plan has .asm units —
214214
# same self-bootstrap contract as the llvm toolchain. The sandbox copy
215215
# lands in ~/.mcpp/registry, so the cache carries it across runs.
216-
# TEMPORARY belt-and-suspenders for compat.ffmpeg's NASM sources: the
217-
# 0.0.95 lazy nasm self-bootstrap declares the build dead BEFORE the
218-
# one-time bootstrap finishes (mcpp#232, 2x reproduced cold). PATH-first
219-
# resolution makes this deterministic; drop once #232 is fixed.
220-
- name: Install nasm (Linux)
216+
# The refresh below is needed because release archives vendor an index
217+
# snapshot and the bootstrap is offline-first: 0.0.96 shipped with a
218+
# nasm.lua deployed engines can't resolve — it "installs" empty
219+
# (mcpp#232; fixed in xim-pkgindex#398 after the release was cut).
220+
# One `xlings update` against the sandbox home lets the bootstrap see
221+
# the fixed descriptor. Drop once a mcpp release vendors a post-#398
222+
# index snapshot.
223+
- name: Refresh sandbox package index (nasm bootstrap, mcpp#232)
221224
if: runner.os == 'Linux'
222225
shell: bash
223-
run: sudo apt-get update && sudo apt-get install -y nasm
226+
run: XLINGS_HOME="$HOME/.mcpp/registry" "$MCPP_VENDORED_XLINGS" update
224227

225228
- name: mcpp test --workspace
226229
shell: bash

0 commit comments

Comments
 (0)