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: Remove --enable-llvm-clean-rebuild #43562

Merged
merged 1 commit into from
Jul 31, 2017

Conversation

alexcrichton
Copy link
Member

This was intended for bots back in the day where we'd persist caches of LLVM
builds across runs, but nowadays we don't do that on any of the bots so this
option is no longer necessary

This was intended for bots back in the day where we'd persist caches of LLVM
builds across runs, but nowadays we don't do that on any of the bots so this
option is no longer necessary
@alexcrichton
Copy link
Member Author

r? @Mark-Simulacrum

@rust-highfive
Copy link
Collaborator

r? @brson

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

@petrochenkov
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jul 30, 2017

📌 Commit eb1c44b has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Jul 31, 2017

⌛ Testing commit eb1c44b with merge f3ff38af86bbb873cc089e603ccff95c999671e0...

@bors
Copy link
Contributor

bors commented Jul 31, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Jul 31, 2017

@bors retry

#43402 in x86_64-apple-darwin check

[01:38:10] failures:
[01:38:10] 
[01:38:10] ---- [run-make] run-make/issue-26092 stdout ----
[01:38:10] 	
[01:38:10] error: make failed
[01:38:10] status: exit code: 2
[01:38:10] command: "make"
[01:38:10] stdout:
[01:38:10] ------------------------------------------
[01:38:10] DYLD_LIBRARY_PATH="/Users/travis/build/rust-lang/rust/build/x86_64-apple-darwin/test/run-make/issue-26092.stage2-x86_64-apple-darwin:/Users/travis/build/rust-lang/rust/build/x86_64-apple-darwin/stage2/lib:" '/Users/travis/build/rust-lang/rust/build/x86_64-apple-darwin/stage2/bin/rustc' --out-dir /Users/travis/build/rust-lang/rust/build/x86_64-apple-darwin/test/run-make/issue-26092.stage2-x86_64-apple-darwin -L /Users/travis/build/rust-lang/rust/build/x86_64-apple-darwin/test/run-make/issue-26092.stage2-x86_64-apple-darwin  -o "" blank.rs 2>&1 | \
[01:38:10] 			grep -i 'No such file or directory'
[01:38:10] 
[01:38:10] ------------------------------------------
[01:38:10] stderr:
[01:38:10] ------------------------------------------
[01:38:10] make[1]: *** [all] Error 1
[01:38:10] 
[01:38:10] ------------------------------------------
[01:38:10] 
[01:38:10] thread '[run-make] run-make/issue-26092' panicked at 'explicit panic', src/tools/compiletest/src/runtest.rs:2499:8
[01:38:10] note: Run with `RUST_BACKTRACE=1` for a backtrace.
[01:38:10] 
[01:38:10] 
[01:38:10] failures:
[01:38:10]     [run-make] run-make/issue-26092
[01:38:10] 
[01:38:10] test result: �[31mFAILED�(B�[m. 156 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out
[01:38:10] 
[01:38:10] thread 'main' panicked at 'Some tests failed', src/tools/compiletest/src/main.rs:323:21

@bors
Copy link
Contributor

bors commented Jul 31, 2017

⌛ Testing commit eb1c44b with merge beec479106a999be5f4b7834f3116b0819493a13...

@bors
Copy link
Contributor

bors commented Jul 31, 2017

⌛ Testing commit eb1c44b with merge 2a6828e...

bors added a commit that referenced this pull request Jul 31, 2017
rustbuild: Remove `--enable-llvm-clean-rebuild`

This was intended for bots back in the day where we'd persist caches of LLVM
builds across runs, but nowadays we don't do that on any of the bots so this
option is no longer necessary
@bors
Copy link
Contributor

bors commented Jul 31, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: petrochenkov
Pushing 2a6828e to master...

@bors bors merged commit eb1c44b into rust-lang:master Jul 31, 2017
@alexcrichton alexcrichton deleted the no-clean-rebuild branch August 22, 2017 19:44
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.

7 participants