-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[beta] Fix mode generated in maybe_lib
#5523
Conversation
The new `mode` for the library dependency is dependent on the library target rather than the target which is the reason for the dependency on the library! Closes rust-lang/rust#50640
r? @matklad (rust_highfive has picked a reviewer for you, use r? to override) |
@bors: r+ |
📌 Commit cfeb768 has been approved by |
[beta] Fix mode generated in `maybe_lib` Beta backport of #5520
☀️ Test successful - status-appveyor, status-travis |
@alexcrichton Let me know if you'd like me to post PRs to the rust repo to pick up these changes. Not sure how tedious that is for you, but I'm happy to help. |
@ehuss sure yeah if you wanna take care of that feel free! |
Regression fixes: - rust-lang/cargo#5510 `cargo rustc` broken for tests in project with bins - rust-lang/cargo#5523 (rust-lang#50640) shared proc-macro dependency built incorrectly
[BETA] Update Cargo Regression fixes: - rust-lang/cargo#5510 `cargo rustc` broken for tests in project with bins - rust-lang/cargo#5523 (#50640) shared proc-macro dependency built incorrectly
Beta backport of #5520