Skip to content

rustpkg: Add a test case with multiple workspaces #7241

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Closed
catamorphism opened this issue Jun 19, 2013 · 2 comments
Closed

rustpkg: Add a test case with multiple workspaces #7241

catamorphism opened this issue Jun 19, 2013 · 2 comments

Comments

@catamorphism
Copy link
Contributor

No description provided.

@catamorphism catamorphism mentioned this issue Jun 19, 2013
Closed
@catamorphism
Copy link
Contributor Author

Sub-bug of #5677

@catamorphism
Copy link
Contributor Author

I don't remember what I meant by this, but I think I meant: add a test where the same package ID exists in multiple workspaces in the RUST_PATH; ensure rustpkg uses the first one instead of complaining.

I will add the test for that.

@catamorphism catamorphism mentioned this issue Aug 23, 2013
Closed
bors added a commit that referenced this issue Aug 26, 2013
@bors
auto merge of #8712 : catamorphism/rust/rustpkg-issue-7241, r=pcwalton
1ac7d80 
r? @brson ...multiple workspaces

The test checks that rustpkg uses the first one, rather than complaining
about multiple matches.

Closes #7241
flip1995 pushed a commit to flip1995/rust that referenced this issue May 20, 2021
@bors
Auto merge of rust-lang#7241 - flip1995:warn-deny-warnings, r=camsteffen
98cddc5 
Deny warnings in every main sub-crate

Pointed out by `@xFrednet` in rust-lang/rust-clippy#7229 (comment)

This enables the same (rustc) lints in every main sub-crate:

- `clippy`
- `clippy_lints`
- `clippy_utils`
- `clippy_dev`

In addition it forwards the `deny-warnings` feature to those sub-crates, so we don't miss warnings that then become a problem during sync. (I wanted to fix that before, but forgot about it, so thanks for pointing it out `@xFrednet!)`

changelog: none
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

rustpkg: Test that different copies of the same package ID can exist in ... catamorphism/rust
1 participant
@catamorphism