Skip to content

Commit

Permalink
Auto merge of rust-lang#38639 - xen0n:nightly-dist-hotfix, r=brson
Browse files Browse the repository at this point in the history
rustbuild: Hotfix to unbreak nightly

Fixes an oversight unnoticed in rust-lang#38468 that eventually broke nightly packaging. I didn't realize this until some moments ago, when I finally found out the failure is actually deterministic. Many apologies for eating 3 nightlies during the holidays.

r? @alexcrichton
  • Loading branch information
bors committed Dec 28, 2016
2 parents 371f4d6 + cf89453 commit 02b22ec
Showing 1 changed file with 13 additions and 6 deletions.
19 changes: 13 additions & 6 deletions src/bootstrap/step.rs
Original file line number Diff line number Diff line change
Expand Up @@ -838,14 +838,21 @@ invalid rule dependency graph detected, was a rule added and maybe typo'd?
} else {
&self.build.config.target
};
// If --target was specified but --host wasn't specified, don't run
// any host-only tests
// Determine the actual targets participating in this rule.
// NOTE: We should keep the full projection from build triple to
// the hosts for the dist steps, now that the hosts array above is
// truncated to avoid duplication of work in that case. Therefore
// the original non-shadowed hosts array is used below.
let arr = if rule.host {
if self.build.flags.target.len() > 0 &&
self.build.flags.host.len() == 0 {
&hosts[..0]
// If --target was specified but --host wasn't specified,
// don't run any host-only tests. Also, respect any `--host`
// overrides as done for `hosts`.
if self.build.flags.host.len() > 0 {
&self.build.flags.host[..]
} else if self.build.flags.target.len() > 0 {
&[]
} else {
hosts
&self.build.config.host[..]
}
} else {
targets
Expand Down

0 comments on commit 02b22ec

Please sign in to comment.