Skip to content

fix: restore runtime backend compile path and binary resolution#281

Merged
DorianZheng merged 4 commits intomainfrom
codex/fix-layout-snapshot-brace
Feb 19, 2026
Merged

fix: restore runtime backend compile path and binary resolution#281
DorianZheng merged 4 commits intomainfrom
codex/fix-layout-snapshot-brace

Conversation

@DorianZheng
Copy link
Copy Markdown
Member

Summary

  • fix local-only clone/export/snapshot/portability paths after backend trait abstraction
  • add backend/runtime helpers to fail explicitly on non-local backends
  • fix runtime binary lookup priority to prefer target/boxlite-runtime in local dev/tests (prevents unsigned shim selection on macOS)
  • harden fd-closing tests by running destructive fd operations in forked child processes
  • align jailer path-access tests with current shared/mounts layout behavior
  • update lifecycle integration tests to explicitly start() before asserting active/stopped states

Verification

  • make runtime-debug
  • cargo check -p boxlite
  • make test:rust
  • cargo test -p boxlite --test execution_shutdown test_all_waits_return_on_stop -- --test-threads=1 --nocapture

@uran0sH
Copy link
Copy Markdown
Contributor

uran0sH commented Feb 18, 2026

The command to run integration tests now is: BOXLITE_RUNTIME_DIR=$(pwd)/target/boxlite-runtime \ LD_LIBRARY_PATH=$(pwd)/target/boxlite-runtime:$LD_LIBRARY_PATH \ cargo test -p boxlite --test execution_shutdown test_wait_behavior_on_box_stop -- --test-threads=1 --nocapture. After merging this PR, is it no longer necessary to configure environment variables?

@DorianZheng
Copy link
Copy Markdown
Member Author

The command to run integration tests now is: BOXLITE_RUNTIME_DIR=$(pwd)/target/boxlite-runtime \ LD_LIBRARY_PATH=$(pwd)/target/boxlite-runtime:$LD_LIBRARY_PATH \ cargo test -p boxlite --test execution_shutdown test_wait_behavior_on_box_stop -- --test-threads=1 --nocapture. After merging this PR, is it no longer necessary to configure environment variables?

Hi @uran0sH. The integration tests issue will be fixed ASAP. The root cause is I accidentally set BOXLITE_RUNTIME_DIR to OUT_DIR=runtime. However, OUT_DIR=runtime doesn't contain boxlite-guest which leads to binary not found issue. This PR haven't fixed it yet. Will file another PR for the test

@DorianZheng DorianZheng merged commit b27a5c6 into main Feb 19, 2026
21 checks passed
@DorianZheng DorianZheng deleted the codex/fix-layout-snapshot-brace branch February 19, 2026 03:56
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