Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Apr 28, 2020

cargo package --list was changed in #7905 to generate Cargo.lock earlier. If there is a problem, then it would fail where previously it would succeed. This changes it so that file generation is deferred until after --list.

This also changes it so that the "dependencies must have a version" check is deferred until after --list as well.

Closes #8151

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 28, 2020
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Apr 28, 2020

📌 Commit 156c651 has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 28, 2020
@bors
Copy link
Contributor

bors commented Apr 28, 2020

⌛ Testing commit 156c651 with merge ade4c7b...

@sfackler
Copy link
Member

Thanks!

@bors
Copy link
Contributor

bors commented Apr 28, 2020

☀️ Test successful - checks-azure
Approved by: alexcrichton
Pushing ade4c7b to master...

@bors bors merged commit ade4c7b into rust-lang:master Apr 28, 2020
ehuss pushed a commit to ehuss/cargo that referenced this pull request May 6, 2020
Allow `cargo package --list` even for things that don't package.

`cargo package --list` was changed in rust-lang#7905 to generate `Cargo.lock` earlier. If there is a problem, then it would fail where previously it would succeed. This changes it so that file generation is deferred until after `--list`.

This also changes it so that the "dependencies must have a version" check is deferred until after `--list` as well.

Closes rust-lang#8151
bors added a commit that referenced this pull request May 6, 2020
[beta] Allow `cargo package --list` even for things that don't package.

Beta backport of #8175.
@ehuss ehuss modified the milestones: 1.45.0, 1.43.1 Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cargo package --list requires all dependencies to be published in 1.43.0
5 participants