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
Releasing a workspace does not update local path dependencies #223
Comments
For anyone else wanting more details, the crate is at https://github.com/algbio/practical-omnitigs/tree/master/implementation |
The only thing I can think of is #206 where we update dependents in a workspace when doing the regular version bump but then don't during the post-release version bump. Does that look like your issue? |
Hmm, looks like #209 was included in 0.13.4 |
Hm, I just executed `cargo release rc`, using cargo-release version 0.13.5. It again forgot to update the versions.
The dependencies are still at alpha, even though I released the whole workspace at once. See here for example: https://crates.io/crates/genome-graph/0.1.0-rc.1
…________________________________
From: Ed Page <notifications@github.com>
Sent: 23 July 2020 17:32
To: sunng87/cargo-release <cargo-release@noreply.github.com>
Cc: Schmidt, Sebastian S <sebastian.schmidt@helsinki.fi>; Author <author@noreply.github.com>
Subject: Re: [sunng87/cargo-release] Releasing a workspace does not update local path dependencies (#223)
Hmm, looks like #209<#209> was included in 0.13.4
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#223 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AORQGR5G7HVMYBALE364J2DR5BCWFANCNFSM4PEKRTDA>.
|
Ah, so what's happening seems to be that the dependencies within the workspace are updated only after release, and not before. But they would need to be updated before release, such that the release crates depend on the correct versions.
…________________________________
From: Schmidt, Sebastian S <sebastian.schmidt@helsinki.fi>
Sent: 04 August 2020 09:41
To: sunng87/cargo-release <reply@reply.github.com>
Subject: Re: [sunng87/cargo-release] Releasing a workspace does not update local path dependencies (#223)
Hm, I just executed `cargo release rc`, using cargo-release version 0.13.5. It again forgot to update the versions.
The dependencies are still at alpha, even though I released the whole workspace at once. See here for example: https://crates.io/crates/genome-graph/0.1.0-rc.1
________________________________
From: Ed Page <notifications@github.com>
Sent: 23 July 2020 17:32
To: sunng87/cargo-release <cargo-release@noreply.github.com>
Cc: Schmidt, Sebastian S <sebastian.schmidt@helsinki.fi>; Author <author@noreply.github.com>
Subject: Re: [sunng87/cargo-release] Releasing a workspace does not update local path dependencies (#223)
Hmm, looks like #209<#209> was included in 0.13.4
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#223 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AORQGR5G7HVMYBALE364J2DR5BCWFANCNFSM4PEKRTDA>.
|
And - when I fix the version numbers before executing `cargo release`, it complains that the given versions don't exist. Hm, so no way to release with correct dependencies at the moment except manually. It would really be super nice and help me a lot if that would get fixed!
…________________________________
From: Schmidt, Sebastian S <sebastian.schmidt@helsinki.fi>
Sent: 04 August 2020 09:43
To: sunng87/cargo-release <reply@reply.github.com>
Subject: Re: [sunng87/cargo-release] Releasing a workspace does not update local path dependencies (#223)
Ah, so what's happening seems to be that the dependencies within the workspace are updated only after release, and not before. But they would need to be updated before release, such that the release crates depend on the correct versions.
________________________________
From: Schmidt, Sebastian S <sebastian.schmidt@helsinki.fi>
Sent: 04 August 2020 09:41
To: sunng87/cargo-release <reply@reply.github.com>
Subject: Re: [sunng87/cargo-release] Releasing a workspace does not update local path dependencies (#223)
Hm, I just executed `cargo release rc`, using cargo-release version 0.13.5. It again forgot to update the versions.
The dependencies are still at alpha, even though I released the whole workspace at once. See here for example: https://crates.io/crates/genome-graph/0.1.0-rc.1
________________________________
From: Ed Page <notifications@github.com>
Sent: 23 July 2020 17:32
To: sunng87/cargo-release <cargo-release@noreply.github.com>
Cc: Schmidt, Sebastian S <sebastian.schmidt@helsinki.fi>; Author <author@noreply.github.com>
Subject: Re: [sunng87/cargo-release] Releasing a workspace does not update local path dependencies (#223)
Hmm, looks like #209<#209> was included in 0.13.4
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#223 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AORQGR5G7HVMYBALE364J2DR5BCWFANCNFSM4PEKRTDA>.
|
I'm also running into this right now |
Hitting this issue on an internal project. Many workspaces are interdependent, so this seems pretty crucial. Temporarily I am just going to remove the versions. For projects that are public (in my case the Rust CV org |
Goling ahead and closing |
I have a workspace with multiple crates that depend on each other, with the dependency specified via a path. Before release, I had the following dependencies of the
genome-graph
crate:The version of
bigraph
in the workspace was0.1.0-alpha.4
.I then executed
cargo release alpha
on the workspace root, which bumped the versions of all crates and released them as expected. But it did NOT bump the versions in the[dependencies]
sections. As a result, the releasedgenome-graph
crate has version0.1.0-alpha.5
, but still points tobigraph
version0.1.0-alpha.4
, which is wrong.How do I get
cargo-release
to bump also the versions in the declaractions of[dependencies]
?The text was updated successfully, but these errors were encountered: