Skip to content
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

Copy --all-features request to all workspace members #5556

Merged
merged 1 commit into from
May 24, 2018

Conversation

alexcrichton
Copy link
Member

This fixes an accidental regression introduced in #5012 where the
--all-features CLI flag was only propagated to the "main crate" as opposed to
all workspace packages. This behavior has already been deemed as
"basically not what you want", but for now it's best to avoid the regression.

Closes #5518

This fixes an accidental regression introduced in rust-lang#5012 where the
`--all-features` CLI flag was only propagated to the "main crate" as opposed to
all workspace packages. This behavior has [already been deemed][pr] as
"basically not what you want", but for now it's best to avoid the regression.

Closes rust-lang#5518

[pr]: rust-lang#5353
@rust-highfive
Copy link

r? @matklad

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

@matklad
Copy link
Member

matklad commented May 24, 2018

r=me. #5012 is in beta, but not in stable, right? Than we should beta-backport?

@alexcrichton
Copy link
Member Author

I believe so!

@bors: r+

@bors
Copy link
Collaborator

bors commented May 24, 2018

📌 Commit a70d519 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented May 24, 2018

⌛ Testing commit a70d519 with merge 4138e50bec8b7895993d7d29397e642b0fc1dc26...

@alexcrichton
Copy link
Member Author

er,

@bors: r=matklad

@bors
Copy link
Collaborator

bors commented May 24, 2018

💡 This pull request was already approved, no need to approve it again.

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

@bors
Copy link
Collaborator

bors commented May 24, 2018

📌 Commit a70d519 has been approved by matklad

@bors
Copy link
Collaborator

bors commented May 24, 2018

⌛ Testing commit a70d519 with merge 22c0f22...

bors added a commit that referenced this pull request May 24, 2018
Copy `--all-features` request to all workspace members

This fixes an accidental regression introduced in #5012 where the
`--all-features` CLI flag was only propagated to the "main crate" as opposed to
all workspace packages. This behavior has [already been deemed][pr] as
"basically not what you want", but for now it's best to avoid the regression.

Closes #5518

[pr]: #5353
@bors
Copy link
Collaborator

bors commented May 24, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 22c0f22 to master...

@bors bors merged commit a70d519 into rust-lang:master May 24, 2018
@alexcrichton alexcrichton deleted the fix branch May 24, 2018 19:06
bors added a commit that referenced this pull request May 25, 2018
[beta] Copy `--all-features` request to all workspace members

Backport of #5556
@ehuss ehuss added this to the 1.28.0 milestone Feb 6, 2022
@ehuss ehuss modified the milestones: 1.28.0, 1.27.0 Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cargo build --all --all-features in a non-virtual workspace ignores features not present in the root crate
5 participants