Skip to content

Conversation

0xPoe
Copy link
Member

@0xPoe 0xPoe commented Nov 15, 2022

What does this PR try to resolve?

close #11315

Add warning when cargo tree -i <spec> can not find packages.

How should we test and review this PR?

Please run the unit test.

@rustbot
Copy link
Collaborator

rustbot commented Nov 15, 2022

r? @weihanglo

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 15, 2022
Copy link
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution. It generally looks great to me!

@0xPoe 0xPoe requested a review from weihanglo November 17, 2022 01:01
Copy link
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

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

The rest looks pretty good!

@0xPoe 0xPoe force-pushed the rustin-patch-warning-tree branch from c5cc0d9 to c5eb618 Compare November 20, 2022 08:39
@0xPoe 0xPoe requested a review from weihanglo November 20, 2022 08:39
@weihanglo
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Nov 20, 2022

📌 Commit c5eb618 has been approved by weihanglo

It is now in the queue for this repository.

@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 Nov 20, 2022
@bors
Copy link
Contributor

bors commented Nov 20, 2022

⌛ Testing commit c5eb618 with merge 63fdd75...

@bors
Copy link
Contributor

bors commented Nov 20, 2022

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing 63fdd75 to master...

@bors bors merged commit 63fdd75 into rust-lang:master Nov 20, 2022
ws.config().shell().warn(
"nothing to print.\n\n\
To find dependencies that require specific target platforms, \
try use option `--target all` first, and then narrow your search scope accordingly.",
Copy link
Member

Choose a reason for hiding this comment

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

I think the grammar is wrong here, should be "try to use" or "try using"

Copy link
Member Author

Choose a reason for hiding this comment

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

aha, I'll fix it later. Thanks!

bors added a commit that referenced this pull request Nov 20, 2022
Fix typo `try use` -> `try to use`

See #11377 (comment)

r? `@weihanglo`
weihanglo added a commit to weihanglo/rust that referenced this pull request Nov 23, 2022
7 commits in eb5d35917b2395194593c9ca70c3778f60c1573b..ba607b23db8398723d659249d9abf5536bc322e5
2022-11-17 22:08:43 +0000 to 2022-11-22 20:52:39 +0000

- Fix failure to parse rustc's JSON output if it is too nested (rust-lang/cargo#11368)
- Add suggestions when `cargo add` multiple packages (rust-lang/cargo#11186)
- Update mod.rs (rust-lang/cargo#11395)
- Fix typo `try use` -> `try to use` (rust-lang/cargo#11394)
- Add warning when `cargo tree -i <spec>` can not find packages (rust-lang/cargo#11377)
- Clean profile, patch, and replace in cargo remove (rust-lang/cargo#11194)
- chore: Upgrade miow (rust-lang/cargo#11391)
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 23, 2022
Update cargo

7 commits in eb5d35917b2395194593c9ca70c3778f60c1573b..ba607b23db8398723d659249d9abf5536bc322e5 2022-11-17 22:08:43 +0000 to 2022-11-22 20:52:39 +0000

- Fix failure to parse rustc's JSON output if it is too nested (rust-lang/cargo#11368)
- Add suggestions when `cargo add` multiple packages (rust-lang/cargo#11186)
- Update mod.rs (rust-lang/cargo#11395)
- Fix typo `try use` -> `try to use` (rust-lang/cargo#11394)
- Add warning when `cargo tree -i <spec>` can not find packages (rust-lang/cargo#11377)
- Clean profile, patch, and replace in cargo remove (rust-lang/cargo#11194)
- chore: Upgrade miow (rust-lang/cargo#11391)
@ehuss ehuss added this to the 1.67.0 milestone Dec 14, 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 tree -i <package>' is empty for windows-only dependency
6 participants