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

Rustbuild regression fixes #43430

Merged
merged 2 commits into from
Jul 24, 2017
Merged

Conversation

Mark-Simulacrum
Copy link
Member

I believe this will fix #43427.

This PR also avoids running pretty tests by-default.

r? @alexcrichton

@Mark-Simulacrum
Copy link
Member Author

@bors p=100 (once this is reviewed)

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 23, 2017

📌 Commit 003bbfc has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 23, 2017

⌛ Testing commit 003bbfceb6ee9ea90eb7725d7cdcff6344402000 with merge d05e8853f3f061b1a9a08f317ecbb632b10d8cc9...

@Mark-Simulacrum
Copy link
Member Author

@bors r- retry

@Mark-Simulacrum Mark-Simulacrum deleted the rustbuild-fixes branch July 23, 2017 21:05
@Mark-Simulacrum
Copy link
Member Author

@bors r+ p=100

@bors
Copy link
Contributor

bors commented Jul 23, 2017

📌 Commit bcc610a has been approved by Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Jul 23, 2017

⌛ Testing commit bcc610a76f15a21dbcac065f1e6cdf5581141842 with merge 6e5f4acdf4346fce513f96d080e6f61ca7e79ae5...

@bors
Copy link
Contributor

bors commented Jul 24, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Jul 24, 2017

Fuchsia failed to build LLVM at stage1 (why?). Legit.

[00:52:07] thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "target `x86_64-unknown-fuchsia` is not configured as a host, only as a target"', /checkout/src/libcore/result.rs:860

@carols10cents carols10cents added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Jul 24, 2017
@Mark-Simulacrum
Copy link
Member Author

@bors r=alexcrichton

I believe the last commit makes dist and install equivalent to what the old build system ran, which should hopefully resolve our issues.

@bors
Copy link
Contributor

bors commented Jul 24, 2017

📌 Commit 9bad3dc has been approved by alexcrichton

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. 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-author Status: This is awaiting some action (such as code changes or more information) from the author. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 24, 2017
@kennytm
Copy link
Member

kennytm commented Jul 24, 2017

@Mark-Simulacrum The PR CI failed to build bootstrap.

[00:01:17]    Compiling bootstrap v0.0.0 (file:///checkout/src/bootstrap)
[00:01:18] error[E0560]: struct `dist::DebuggerScripts` has no field named `target`
[00:01:18]    --> /checkout/src/bootstrap/check.rs:538:17
[00:01:18]     |
[00:01:18] 538 |                 target: target
[00:01:18]     |                 ^^^^^^^ field does not exist - did you mean `host`?
[00:01:18] 
[00:01:19] error[E0560]: struct `dist::DebuggerScripts` has no field named `target`
[00:01:19]    --> /checkout/src/bootstrap/dist.rs:435:17
[00:01:19]     |
[00:01:19] 435 |                 target: host,
[00:01:19]     |                 ^^^^^^^ field does not exist - did you mean `host`?
[00:01:19] 
[00:01:19] error[E0560]: struct `dist::Extended` has no field named `compiler`
[00:01:19]     --> /checkout/src/bootstrap/dist.rs:1103:13
[00:01:19]      |
[00:01:19] 1103 |             compiler: run.builder.top_stage,
[00:01:19]      |             ^^^^^^^^^ field does not exist - did you mean `stage`?

@bors
Copy link
Contributor

bors commented Jul 24, 2017

⌛ Testing commit 9bad3dcb9671b6dd597d3d9e1ae529cc2775e680 with merge 3c4fc0f3f66785c410b8ee3245aa26b23d50ef38...

@bors
Copy link
Contributor

bors commented Jul 24, 2017

💔 Test failed - status-appveyor

@Mark-Simulacrum
Copy link
Member Author

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Jul 24, 2017

📌 Commit fef9b48 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 24, 2017

⌛ Testing commit fef9b48 with merge 2ad68c47023a625d693c69b88e1d9f0a476bf48b...

@bors
Copy link
Contributor

bors commented Jul 24, 2017

💔 Test failed - status-travis

@Mark-Simulacrum
Copy link
Member Author

@bors retry

[01:21:53] 
[01:21:53] failures:
[01:21:53] 
[01:21:53] ---- test::test_borrow_error stdout ----
[01:21:53] 	thread 'test::test_borrow_error' panicked at 'Hit timeout', /checkout/src/tools/rls/src/test/harness.rs:142:12
[01:21:53] 
[01:21:53] 
[01:21:53] failures:
[01:21:53]     test::test_borrow_error

bors added a commit that referenced this pull request Jul 24, 2017
Rustbuild regression fixes

I believe this will fix #43427.

This PR also avoids running pretty tests by-default.

r? @alexcrichton
@bors
Copy link
Contributor

bors commented Jul 24, 2017

⌛ Testing commit fef9b48 with merge b80e946...

@bors
Copy link
Contributor

bors commented Jul 24, 2017

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

@bors bors merged commit fef9b48 into rust-lang:master Jul 24, 2017
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.

Latest Rust nightly does not work on FreeBSD
5 participants