Merge pull request 'fix: -DHAVE_CURL for el_runtime OTLP — resolves emit_metric linker error' (#8 ) from fix/have-curl-define into stage

fix: add -DHAVE_CURL to el_runtime.c compilation, restore el_runtime.o for soul-demo
Root cause: the staged el_runtime.c (from el.git) wraps the entire OTLP observability section (emit_metric, emit_log, trace_span_start/end) in #ifdef HAVE_CURL. Without -DHAVE_CURL, those symbols are compiled out, causing the undefined reference linker errors. libcurl IS available (installed via libcurl4-openssl-dev), so -DHAVE_CURL correctly enables the OTLP code path. Also reverts the soul-demo compile to use el_runtime.o (the pre-compiled cached object) now that the object will contain the correct symbols.
2026-05-07 02:35:32 +00:00 · 2026-05-06 21:35:16 -05:00 · 2026-05-07 02:30:46 +00:00 · 2026-05-06 21:30:25 -05:00 · 2026-05-07 02:02:50 +00:00
2 changed files with 125 additions and 123 deletions
@@ -28,7 +28,10 @@ COPY runtime/el_runtime.c runtime/el_runtime.h ./
 # Pre-compile el_runtime as a separate cached layer.
 # el_runtime.c changes rarely; main.c changes every run.
 # Splitting this out means el_runtime.o is cached across builds when only main.c changes.
-RUN cc -O2 -c el_runtime.c -I. -o el_runtime.o
+# -DHAVE_CURL: the staged el_runtime.c (from el.git) guards the OTLP observability
+# section (emit_metric, emit_log, trace_span_*) behind #ifdef HAVE_CURL.
+# libcurl IS installed above, so define HAVE_CURL to enable those functions.
+RUN cc -O2 -DHAVE_CURL -c el_runtime.c -I. -o el_runtime.o

 # ── Build neuron-web ──────────────────────────────────────────────────────────
 #
Author	SHA1	Message	Date
will.anderson	1cf2ef8835	Merge pull request 'fix: -DHAVE_CURL for el_runtime OTLP — resolves emit_metric linker error' (#8 ) from fix/have-curl-define into stage Stage — Build, push & deploy to marketing-stage / deploy-stage (push) Successful in 3m12s Details	2026-05-07 02:35:32 +00:00
will.anderson	f0a6b55a13	fix: add -DHAVE_CURL to el_runtime.c compilation, restore el_runtime.o for soul-demo Root cause: the staged el_runtime.c (from el.git) wraps the entire OTLP observability section (emit_metric, emit_log, trace_span_start/end) in #ifdef HAVE_CURL. Without -DHAVE_CURL, those symbols are compiled out, causing the undefined reference linker errors. libcurl IS available (installed via libcurl4-openssl-dev), so -DHAVE_CURL correctly enables the OTLP code path. Also reverts the soul-demo compile to use el_runtime.o (the pre-compiled cached object) now that the object will contain the correct symbols.	2026-05-06 21:35:16 -05:00
will.anderson	843b6e07a7	Merge pull request 'fix: soul-demo emit_metric linker error — rebuild from source, compile with el_runtime.c' (#7 ) from fix/soul-demo-emit-metric into stage Stage — Build, push & deploy to marketing-stage / deploy-stage (push) Failing after 2m26s Details	2026-05-07 02:30:46 +00:00
will.anderson	0202b09d37	fix: rebuild soul-demo.c from source, compile against el_runtime.c directly soul-demo.c was previously an older compiled artifact that triggered an undefined reference to emit_metric/emit_log/trace_span_* at link time in CI. Two fixes: 1. Rebuild soul-demo.c from soul-demo.el using current elc — cleaner codegen (no double-wrapped el_from_float), same logic, unix_timestamp collision with el_runtime.c removed. 2. Dockerfile.stage: compile soul-demo against el_runtime.c directly (not el_runtime.o) so all runtime symbols are always resolved from the staged source, bypassing any Docker layer cache divergence on el_runtime.o.	2026-05-06 21:30:25 -05:00
will.anderson	f19403ba68	Merge pull request 'fix: security hardening from pentest findings' (#6 ) from fix/pentest-security-hardening into stage Stage — Build, push & deploy to marketing-stage / deploy-stage (push) Failing after 2m37s Details	2026-05-07 02:02:50 +00:00