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

Fix required-features using renamed dependencies. #7855

Merged
merged 1 commit into from
Feb 5, 2020

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Feb 2, 2020

The dep_name/feat_name syntax in required-features should use the "name in toml" for dep_name, not the actual package name. Otherwise, it is impossible to actually enable the feature (because the --features flag expects a "name in toml").

The dep_name/feat_name syntax in required-features should use the
"name in toml" for dep_name, not the actual package name.
@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 Feb 2, 2020
@ehuss ehuss mentioned this pull request Feb 2, 2020
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Feb 4, 2020

📌 Commit cafec11 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 Feb 4, 2020
@bors
Copy link
Collaborator

bors commented Feb 4, 2020

⌛ Testing commit cafec11 with merge ad7015086a75a19f7fabc6bcc72b1fbbcf053d5e...

@bors
Copy link
Collaborator

bors commented Feb 4, 2020

💔 Test failed - checks-azure

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

ehuss commented Feb 5, 2020

CI failure was #7865.

@bors retry

@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 Feb 5, 2020
@bors
Copy link
Collaborator

bors commented Feb 5, 2020

⌛ Testing commit cafec11 with merge 06834dc...

bors added a commit that referenced this pull request Feb 5, 2020
…hton

Fix required-features using renamed dependencies.

The dep_name/feat_name syntax in required-features should use the "name in toml" for dep_name, not the actual package name.  Otherwise, it is impossible to actually enable the feature (because the `--features` flag expects a "name in toml").
@bors
Copy link
Collaborator

bors commented Feb 5, 2020

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

@bors bors merged commit cafec11 into rust-lang:master Feb 5, 2020
@ehuss ehuss added this to the 1.43.0 milestone 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.

4 participants