Skip to content

uniffi: Update to latest version of uniffi (0.31) - #143

Merged
heinrich5991 merged 2 commits into
heinrich5991:masterfrom
ChillerDragon:pr_uniffi31
Jun 12, 2026
Merged

uniffi: Update to latest version of uniffi (0.31)#143
heinrich5991 merged 2 commits into
heinrich5991:masterfrom
ChillerDragon:pr_uniffi31

Conversation

@ChillerDragon

@ChillerDragon ChillerDragon commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Closes #142
Closes #130

I could reproduce the issue #142 locally by running python -m build in the ./uniffi/huffman directory. And updating uniffi did resolve the issue.

@heinrich5991 heinrich5991 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for fixing the uniffi stuff. :)

It seems you've run cargo update. Please revert Cargo.lock changes and re-generate Cargo.lock by runnning something like cargo check.

Closed #142
Closed #130

Please use present tense in the commit message.

If you don't want to adjust your PR, you can also close it, I already reproduced the changes locally.

@ChillerDragon

Copy link
Copy Markdown
Contributor Author

Yes I intentionally deleted Cargo.lock to update all the dependencies. cargo check does not do that for me.

Closes heinrich5991#142
Closes heinrich5991#130

I could reproduce the issue heinrich5991#142 locally by running `python -m build` in the `./uniffi/huffman` directory. And updating uniffi did resolve the issue.
@ChillerDragon

Copy link
Copy Markdown
Contributor Author

Ah okay if I change the Cargo.toml uniffi version and then cargo check it does something. Well.. that reduced the diff from 500 lines to 400.

@heinrich5991 heinrich5991 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks. :)

@heinrich5991
heinrich5991 merged commit 5b69a88 into heinrich5991:master Jun 12, 2026
8 checks passed
@ChillerDragon

Copy link
Copy Markdown
Contributor Author

@heinrich5991 could you do a new pypi release so my pip install works again? :)

@heinrich5991

Copy link
Copy Markdown
Owner

Done, libtw2-huffman 0.2.2 released on PyPI.

ChillerDragon added a commit to ChillerDragon/teeworlds-web-traffic-decoder that referenced this pull request Jun 13, 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.

huffman: python package fails on cargo metadata returned 2 packages for crate name libtw2_huffman CI: Build uniffi fails

2 participants