Skip to content

Add linked TinyBus module entrypoint - #20

Closed
senamakel wants to merge 6 commits into
tinyhumansai:mainfrom
senamakel:windows-linked-modules
Closed

senamakel wants to merge 6 commits into
tinyhumansai:mainfrom
senamakel:windows-linked-modules

Conversation

@senamakel

@senamakel senamakel commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

Adds static-link (with linked as an alias) and uses module_export_optional_static! to select the Rust-addressable module export. The normal build keeps its dynamic C ABI. TinyJevClient and TinyBus use exact version dependencies with local workspace patches. The nested TinyBus checkout is pinned to commit 433d9ed from tinyhumansai/tinybus#29.

Draft readiness condition: TinyBus PR #29 must merge, then this repository must pin its merged commit and pass CI before this PR is marked ready or merged.

Related issue

None.

API or behavior changes

With static-link, Rust hosts can call tinydesktop::linked_module() to register this module in process. linked remains an alias for static-link.

Validation

  • cargo check -p tinydesktop --locked
  • cargo check -p tinydesktop --features static-link --locked
  • cargo fmt --all -- --check
  • git diff --check

Tests

TinyBus PR #29 covers the shared module export behavior. Full repository suites were not run locally.

Documentation

The crate-root function has rustdoc; the TinyBus dependency PR documents the host registration API.

Checklist

  • The change is focused on linked module integration
  • No new #[ignore] or relaxed lint settings
  • No secrets or credentials in the change

@coderabbitai

coderabbitai Bot commented Sep 25, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@senamakel senamakel closed this Sep 27, 2026
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