Skip to content

Sxian/clt 2237/update new rust commit#6

Merged
xianshijing-lk merged 7 commits intomainfrom
sxian/CLT-2237/update_new_rust_commit
Nov 10, 2025
Merged

Sxian/clt 2237/update new rust commit#6
xianshijing-lk merged 7 commits intomainfrom
sxian/CLT-2237/update_new_rust_commit

Conversation

@xianshijing-lk
Copy link
Collaborator

This MR updated the Rust code to latest, depending on livekit/rust-sdks#781.

Copy link
Contributor

@ladvoc ladvoc left a comment

Choose a reason for hiding this comment

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

LGTM, just one comment.

SendRequest(request);
livekit_ffi_initialize(&LivekitFfiCallback,
true,
"client-sdk-cpp",
Copy link
Contributor

Choose a reason for hiding this comment

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

For this SDK, the SFU recognizes "cpp" as the identifier.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@xianshijing-lk xianshijing-lk merged commit c3f3260 into main Nov 10, 2025
3 checks passed
@xianshijing-lk xianshijing-lk deleted the sxian/CLT-2237/update_new_rust_commit branch November 10, 2025 11:10
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.

3 participants