Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename crate to finalfusion-python #33

Merged
merged 1 commit into from
Jul 24, 2019
Merged

Rename crate to finalfusion-python #33

merged 1 commit into from
Jul 24, 2019

Conversation

danieldk
Copy link
Member

This avoids that there are two crates named finalfusion.

No problems building this locally, let's see if CI is happy as well.

@danieldk danieldk added the bug Something isn't working label Jul 23, 2019
@danieldk danieldk requested a review from sebpuetz July 23, 2019 18:47
@sebpuetz
Copy link
Member

Changes look fine but CI fails because of TryFrom in a dependency

@sebpuetz
Copy link
Member

sebpuetz commented Jul 23, 2019

Frommi/miniz_oxide@7bcf211

Seems to be the commit that messed up things for us.

@danieldk
Copy link
Member Author

The problem is that pyo3-pack from crates.io does not contain Cargo.lock and will thus pin dependencies to the latest version that the constraints in Cargo.toml permit.

This avoids that there are two crates named finalfusion.
@danieldk
Copy link
Member Author

Should be fixed now, we'll hear from CI soon ;). The problem seems to be that cargo does not include Cargo.lock, even not for binary crates :(:

rust-lang/cargo#2263

Learned something new today.

@danieldk
Copy link
Member Author

See the release notes of Cargo 1.37, which will include functionality to add/use lock files for binary crates:

https://github.com/rust-lang/cargo/blob/bee04877ff8cd624f9ffcf158347c2df49863c97/CHANGELOG.md#cargo-137-2019-08-15

@danieldk danieldk merged commit 5562836 into master Jul 24, 2019
@danieldk danieldk deleted the rename-crate branch July 24, 2019 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

2 participants