Skip to content

travis: Enable rust-analysis package for more targets#41267

Merged
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:fix-analysis-dist
Apr 13, 2017
Merged

travis: Enable rust-analysis package for more targets#41267
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:fix-analysis-dist

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

This commit enables the rust-analysis package to be produced for all targets
that are part of the dist-* suite of docker images on Travis. Currently
these packages are showing up with available = false in the
channel-rust-nightly.toml manifest where we'd prefer to have them show up for
all targets.

Unfortunately rustup isn't handling the available = false section well right
now, so this should also inadvertently fix the nightly regression.

This commit enables the `rust-analysis` package to be produced for all targets
that are part of the `dist-*` suite of docker images on Travis. Currently
these packages are showing up with `available = false` in the
`channel-rust-nightly.toml` manifest where we'd prefer to have them show up for
all targets.

Unfortunately rustup isn't handling the `available = false` section well right
now, so this should also inadvertently fix the nightly regression.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

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

@alexcrichton

Copy link
Copy Markdown
Member Author

@bors: r+ p=10

fixing a nightly regression in rust-lang/rustup#1062

@bors

bors commented Apr 13, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit cdedecb has been approved by alexcrichton

@bors

bors commented Apr 13, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit cdedecb with merge 7bdd8fb...

@bors

bors commented Apr 13, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-appveyor

@arielb1

arielb1 commented Apr 13, 2017

Copy link
Copy Markdown
Contributor

@bors

bors commented Apr 13, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit cdedecb with merge 1f59c7e...

bors added a commit that referenced this pull request Apr 13, 2017
travis: Enable rust-analysis package for more targets

This commit enables the `rust-analysis` package to be produced for all targets
that are part of the `dist-*` suite of docker images on Travis. Currently
these packages are showing up with `available = false` in the
`channel-rust-nightly.toml` manifest where we'd prefer to have them show up for
all targets.

Unfortunately rustup isn't handling the `available = false` section well right
now, so this should also inadvertently fix the nightly regression.
@bors

bors commented Apr 13, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 1f59c7e to master...

@bors bors merged commit cdedecb into rust-lang:master Apr 13, 2017
@alexcrichton alexcrichton deleted the fix-analysis-dist branch April 19, 2017 16:05
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.

5 participants