Skip to content

refactor: consistent -sys dependency naming and feature gates#374

Merged
DorianZheng merged 1 commit intomainfrom
refactor/consistent-sys-feature-names
Mar 13, 2026
Merged

refactor: consistent -sys dependency naming and feature gates#374
DorianZheng merged 1 commit intomainfrom
refactor/consistent-sys-feature-names

Conversation

@DorianZheng
Copy link
Copy Markdown
Member

Summary

  • Rename features to short, consistent names across boxlite and libkrun-sys:
    • gvproxy-backendgvproxy, libslirp-backendlibslirp, link-krunkrun
    • libkrun-sys internals: link-static + build-libkrunkrun, build-libkrunfwkrunfw
  • Make all -sys dependencies optional with feature gates:
    • e2fsprogs-syse2fsprogs feature (default-on)
    • bubblewrap-sysbubblewrap feature (default-on, Linux)
    • libkrun-syskrunfw feature (default-on)
  • cargo build --no-default-features now compiles with zero native dependencies

Test plan

  • cargo build --lib -p boxlite — defaults (krunfw + e2fsprogs + bubblewrap)
  • cargo build --lib -p boxlite --no-default-features — zero native deps
  • cargo test -p boxlite --no-default-features --lib — 567 unit tests pass
  • BOXLITE_DEPS_STUB=1 cargo clippy -p boxlite --all-features -- -D warnings — clean
  • cargo fmt --check — clean

- Rename features to short, consistent names across boxlite and libkrun-sys:
  - gvproxy-backend → gvproxy, libslirp-backend → libslirp, link-krun → krun
  - libkrun-sys: merge link-static + build-libkrun → krun, build-libkrunfw → krunfw
- Make all -sys dependencies optional with feature gates:
  - e2fsprogs-sys → e2fsprogs feature (default-on)
  - bubblewrap-sys → bubblewrap feature (default-on, Linux)
  - libkrun-sys → krunfw feature (default-on)
- cargo build --no-default-features now compiles with zero native deps
- Update build scripts, test targets, and docs to use new feature names
@DorianZheng DorianZheng merged commit c95c0f0 into main Mar 13, 2026
18 checks passed
@DorianZheng DorianZheng deleted the refactor/consistent-sys-feature-names branch March 13, 2026 07:25
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.

1 participant