Skip to content

Add back Manifest::targets_mut - #8494

Merged
bors merged 1 commit into
rust-lang:masterfrom
lu-zero:unbreak-cargo-c
Jul 18, 2020
Merged

Add back Manifest::targets_mut#8494
bors merged 1 commit into
rust-lang:masterfrom
lu-zero:unbreak-cargo-c

Conversation

@lu-zero

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

Copy link
Copy Markdown
Contributor

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

@rust-highfive

Copy link
Copy Markdown

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

Eh2406 commented Jul 17, 2020

Copy link
Copy Markdown
Contributor

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

ehuss commented Jul 17, 2020

Copy link
Copy Markdown
Contributor

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

lu-zero commented Jul 17, 2020

Copy link
Copy Markdown
Contributor Author

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

ehuss commented Jul 18, 2020

Copy link
Copy Markdown
Contributor

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

@bors r+

@bors

bors commented Jul 18, 2020

Copy link
Copy Markdown
Contributor

📌 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

bors commented Jul 18, 2020

Copy link
Copy Markdown
Contributor

⌛ Testing commit 9c8544a with merge c004bf9...

@bors

bors commented Jul 18, 2020

Copy link
Copy Markdown
Contributor

☀️ 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