Skip to content

Commit 0394019

Browse files
hyperpolymathclaude
andcommitted
chore(chapel-ci): add --print-commands + smoke-run to surface linker errors
chpl swallows linker errors as a terse `error: Make Binary - Linking` with no underlying ld output. Add `--print-commands` so the synthesised linker invocation lands in the log, and also run the produced binary under LD_LIBRARY_PATH=$PROVEN_LIB_PATH to confirm it actually executes (instead of only checking that it compiled). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 96ca502 commit 0394019

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

.github/workflows/chapel-ci.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,14 +206,20 @@ jobs:
206206
# exercises the same typechecker without the broken gen-path
207207
# resolution, AND additionally proves the binding links — which is
208208
# what we actually care about.
209+
# --print-commands surfaces the linker invocation chpl synthesises;
210+
# without it, link failures appear only as "error: Make Binary -
211+
# Linking" with no actionable detail.
209212
chpl -o /tmp/_chapel_typecheck_driver \
210213
smoke/hello_smoke.chpl \
211214
-M src/ \
212215
-I "$PROVEN_INCLUDE_PATH" \
213216
-L "$PROVEN_LIB_PATH" -lproven \
214217
--ldflags "-Wl,-rpath,$PROVEN_LIB_PATH" \
218+
--print-commands \
215219
--print-callstack-on-error
216220
ls -la /tmp/_chapel_typecheck_driver
221+
# Sanity check: the binary actually runs against the test fixture.
222+
LD_LIBRARY_PATH="$PROVEN_LIB_PATH" /tmp/_chapel_typecheck_driver
217223
echo "[chapel-build] typecheck via real link: OK"
218224
219225
- name: Upload libproven.so artefact

0 commit comments

Comments
 (0)