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

Use updated clvm_rs. #98

Merged
merged 2 commits into from
Dec 6, 2022
Merged

Use updated clvm_rs. #98

merged 2 commits into from
Dec 6, 2022

Conversation

richardkiss
Copy link
Contributor

No description provided.

@richardkiss richardkiss marked this pull request as draft December 3, 2022 01:24
Cargo.toml Outdated Show resolved Hide resolved
fuzz/Cargo.toml Outdated Show resolved Hide resolved
wheel/Cargo.toml Outdated Show resolved Hide resolved
@richardkiss richardkiss force-pushed the bump-version branch 3 times, most recently from 97a3778 to 69218d7 Compare December 3, 2022 02:16
@richardkiss richardkiss marked this pull request as ready for review December 3, 2022 02:17
arvidn
arvidn previously approved these changes Dec 5, 2022
@arvidn
Copy link
Contributor

arvidn commented Dec 5, 2022

it looks like there's still a version conflict of clvm_rs: https://github.com/Chia-Network/chia_rs/actions/runs/3606526215/jobs/6077924242#step:9:53

@richardkiss
Copy link
Contributor Author

it looks like there's still a version conflict of clvm_rs: https://github.com/Chia-Network/chia_rs/actions/runs/3606526215/jobs/6077924242#step:9:53

I think this is because when you do cargo publish, it no longer gets sub-crates from the local file system, but instead forces a fetch from crates.io (because stuff uploaded to crates.io can only depend on other things from crates.io for transitive security). I expect a real cargo publish will succeed as long as the clvm-utils is published first.

However, I do have to update everyone's version number (or at least the things that depend on clvmr 0.20 now).

@richardkiss richardkiss merged commit e285fde into main Dec 6, 2022
@richardkiss richardkiss deleted the bump-version branch December 6, 2022 23:48
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.

2 participants