Skip to content

fix(build): re-run logos-delivery build script when the native lib apears - #197

Merged
mchenani merged 2 commits into
mainfrom
mch/fix/logos-delivery-build-order
Aug 4, 2026
Merged

fix(build): re-run logos-delivery build script when the native lib apears#197
mchenani merged 2 commits into
mainfrom
mch/fix/logos-delivery-build-order

Conversation

@mchenani

@mchenani mchenani commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Fixes chat-cli failing to link with undefined _logosdelivery_* symbols when the native lib is built after the first cargo build. The build script now watches the LOGOS_DELIVERY_LIB_DIR path so it re-runs once the lib appears, making build order irrelevant; the fallback nix build also enables flakes inline, and the README leads with nix develop.

@mchenani
mchenani requested a review from jazzz August 4, 2026 17:08

@jazzz jazzz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 - Great fix.

This area has become out of date after #176. But as the first place new contributors look, it needs to be a clean example. Thanks for the help!

@mchenani
mchenani force-pushed the mch/fix/logos-delivery-build-order branch from c2dcd8e to 2d86c94 Compare August 4, 2026 18:23
@mchenani
mchenani merged commit 57cd494 into main Aug 4, 2026
5 checks passed
@mchenani
mchenani deleted the mch/fix/logos-delivery-build-order branch August 4, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants