Skip to content

Filter out missing components from manifests - #49705

Merged
alexcrichton merged 1 commit into
rust-lang:masterfrom
alexcrichton:less-manifest-docs
Apr 5, 2018
Merged

Filter out missing components from manifests#49705
alexcrichton merged 1 commit into
rust-lang:masterfrom
alexcrichton:less-manifest-docs

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

This commit updates our manifest generation for rustup to filter out any
components/extensions which are actually missing. This is intended to help
mitigate #49462 by making the manifests reflect reality, that many targets now
are missing a rust-docs component rather than requiring it exists.

This commit updates our manifest generation for rustup to filter out any
components/extensions which are actually missing. This is intended to help
mitigate rust-lang#49462 by making the manifests reflect reality, that many targets now
are missing a `rust-docs` component rather than requiring it exists.
@alexcrichton

Copy link
Copy Markdown
MemberAuthor

r? @kennytm

@kennytm

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Apr 5, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 59059f2 has been approved by kennytm

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 5, 2018
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Apr 5, 2018
bors added a commit that referenced this pull request Apr 5, 2018
Rollup of 8 pull requests
Successful merges:
- #49045 (Make queries thread safe)
- #49350 (Expand macros in `extern {}` blocks)
- #49497 (Chalkify - Tweak `Clause` definition and HRTBs)
- #49597 (proc_macro: Reorganize public API)
- #49686 (typos)
- #49621
- #49697
- #49705
Failed merges:
@Hoverbear

Copy link
Copy Markdown
Contributor

Would this allow for the use of the previously broken builds, or only new builds?

@alexcrichton

Copy link
Copy Markdown
MemberAuthor

No, this'd only fix builds going forward

@alexcrichton
alexcrichton merged commit 59059f2 into rust-lang:masterApr 5, 2018
@alexcrichton
alexcrichton deleted the less-manifest-docs branch April 20, 2018 06:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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

@alexcrichton@kennytm@bors@Hoverbear