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

Add back Manifest::targets_mut #8494

Merged
merged 1 commit into from
Jul 18, 2020
Merged

Conversation

lu-zero
Copy link
Contributor

@lu-zero lu-zero commented Jul 17, 2020

It is needed by cargo-c, it was removed in df5cb70

@rust-highfive
Copy link

r? @Eh2406

(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 Jul 17, 2020
lu-zero added a commit to lu-zero/cargo-c that referenced this pull request Jul 17, 2020
@Eh2406
Copy link
Contributor

Eh2406 commented Jul 17, 2020

r? @ehuss, how best to add this back in after your reorganization?

@rust-highfive rust-highfive assigned ehuss and unassigned Eh2406 Jul 17, 2020
@ehuss
Copy link
Contributor

ehuss commented Jul 17, 2020

Can you add a comment that this is used by external tools? Otherwise it will likely get removed again in the future.
Also, just to forewarn, this will be more difficult to use because it is behind an Arc.

It is needed by cargo-c, it was removed in df5cb70
@lu-zero
Copy link
Contributor Author

lu-zero commented Jul 17, 2020

I tested it and it seems working luckily, I need a mean to update the library target, if there is a simpler way would be great but I couldn't find it.

@ehuss
Copy link
Contributor

ehuss commented Jul 18, 2020

I don't think there is any other way to modify the target.

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 18, 2020

📌 Commit 9c8544a has been approved by ehuss

@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 Jul 18, 2020
@bors
Copy link
Collaborator

bors commented Jul 18, 2020

⌛ Testing commit 9c8544a with merge c004bf9...

@bors
Copy link
Collaborator

bors commented Jul 18, 2020

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing c004bf9 to master...

@bors bors merged commit c004bf9 into rust-lang:master Jul 18, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 23, 2020
Update cargo

21 commits in 43cf77395cad5b79887b20b7cf19d418bbd703a9..aa6872140ab0fa10f641ab0b981d5330d419e927
2020-07-13 17:35:42 +0000 to 2020-07-23 13:46:27 +0000
- Update features set in CI. (rust-lang/cargo#8530)
- Stabilize -Z crate-versions (rust-lang/cargo#8509)
- Fix typo in docs (rust-lang/cargo#8529)
- Remove unused CompileMode::all_modes (rust-lang/cargo#8526)
- Mask out system core.autocrlf settings before resetting git repos (rust-lang/cargo#8523)
- Flag git zlib errors as spurious errors (rust-lang/cargo#8520)
- Fix the help display for the target-triple option (rust-lang/cargo#8515)
- Check workspace member existence as dir. (rust-lang/cargo#8511)
- Bump to 0.48.0, update changelog (rust-lang/cargo#8508)
- Apply workspace.exclude to workspace.default-members. (rust-lang/cargo#8485)
- Fix nightly tests for intra-doc links. (rust-lang/cargo#8528)
- doc: Replace "regenerate" with "revoke" for API tokens (rust-lang/cargo#8510)
- Add back Manifest::targets_mut (rust-lang/cargo#8494)
- Build host dependencies with opt-level 0 by default (rust-lang/cargo#8500)
- Fix freshness checks for build scripts on renamed dirs (rust-lang/cargo#8497)
- Add a `-Zbuild-std-features` flag (rust-lang/cargo#8490)
- clippy cleanups (rust-lang/cargo#8495)
- Fix self-publish script. (rust-lang/cargo#8492)
- Ensure `unstable.build-std` works like `-Zbuild-std` (rust-lang/cargo#8491)
- Make `cargo metadata` output deterministic (rust-lang/cargo#8489)
- Switch to github actions (rust-lang/cargo#8467)
lu-zero added a commit to lu-zero/cargo-c that referenced this pull request Sep 7, 2020
@ehuss ehuss added this to the 1.47.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.

5 participants