Use repo runtime dir for EL_RUNTIME in push builds #22

Merged
will.anderson merged 1 commits from fix/stage-source-check into dev 2026-05-09 23:17:51 +00:00
Owner

Push builds were setting EL_RUNTIME to /opt/el/el-compiler/runtime (from ci-base), bypassing the web repo's runtime/el_runtime.h which contains the forward declarations for the 12 web stub functions. This caused 12 implicit declaration warnings in dist/main.c on every push-to-dev run, causing the smoke test to fail.

Fix: point EL_RUNTIME at $GITHUB_WORKSPACE/runtime so both PR and push builds use the same header.

Push builds were setting EL_RUNTIME to /opt/el/el-compiler/runtime (from ci-base), bypassing the web repo's runtime/el_runtime.h which contains the forward declarations for the 12 web stub functions. This caused 12 implicit declaration warnings in dist/main.c on every push-to-dev run, causing the smoke test to fail. Fix: point EL_RUNTIME at $GITHUB_WORKSPACE/runtime so both PR and push builds use the same header.
will.anderson added 1 commit 2026-05-09 23:15:30 +00:00
Use repo runtime dir for EL_RUNTIME in push builds
Dev — Build & local smoke test / build-smoke (pull_request) Successful in 1m56s
e7c1c922f7
ci-base's el-compiler/runtime doesn't have the web-specific forward
declarations added to runtime/el_runtime.h. Point EL_RUNTIME at the
workspace runtime/ so push builds pick up the same header as PR builds.
will.anderson merged commit 17e14a9fda into dev 2026-05-09 23:17:51 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: neuron-technologies/neuron-web#22