Skip to content

Disable errexit for sanity checking git repo#41147

Merged
bors merged 1 commit into
rust-lang:masterfrom
aidanhs:aphs-fix-appveyor-cache
Apr 8, 2017
Merged

Disable errexit for sanity checking git repo#41147
bors merged 1 commit into
rust-lang:masterfrom
aidanhs:aphs-fix-appveyor-cache

Conversation

@aidanhs

@aidanhs aidanhs commented Apr 7, 2017

Copy link
Copy Markdown
Contributor

Unblock appveyor

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

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

@TimNN

TimNN commented Apr 7, 2017

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit b6de319 has been approved by TimNN

@TimNN

TimNN commented Apr 7, 2017

Copy link
Copy Markdown
Contributor

@bors p=1

Comment thread src/ci/init_repo.sh Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not part of this PR, but the && echo $? seems a bit suspicious to me -- since && is used $? should always be 0 here, right?

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit b6de319 with merge 5fa0023...

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-appveyor

@aidanhs aidanhs force-pushed the aphs-fix-appveyor-cache branch from b6de319 to bd4f381 Compare April 7, 2017 19:15
@TimNN

TimNN commented Apr 7, 2017

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit bd4f381 has been approved by TimNN

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit bd4f381 with merge 535ba87...

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-appveyor

@frewsxcv

frewsxcv commented Apr 7, 2017

Copy link
Copy Markdown
Contributor

Test failure seems legit.

@TimNN

TimNN commented Apr 7, 2017

Copy link
Copy Markdown
Contributor
[01:33:06] ---- test_release_ignore_panic stdout ----
[01:33:06] 	test
[01:33:06] running `C:\projects\rust\build\ct\cargo\target\debug\cargo test -v`
[01:33:06] bench
[01:33:06] running `C:\projects\rust\build\ct\cargo\target\debug\cargo bench -v`
[01:33:06] thread 'test_release_ignore_panic' panicked at '
[01:33:06] Expected: execs
[01:33:06]     but: exited with exit code: 3221225477
[01:33:06] --- stdout
[01:33:06] 
[01:33:06] --- stderr
[01:33:06]    Compiling a v0.0.1 (file:///C:/projects/rust/build/ct/cargo/target/cit/t53/foo/a)
[01:33:06]      Running `rustc --crate-name a a\src\lib.rs --crate-type lib --emit=dep-info,link -C opt-level=3 -C metadata=850c3cbe05004df2 -C extra-filename=-850c3cbe05004df2 --out-dir C:\projects\rust\build\ct\cargo\target\cit\t53\foo\target\release\deps -L dependency=C:\projects\rust\build\ct\cargo\target\cit\t53\foo\target\release\deps`
[01:33:06]    Compiling foo v0.0.1 (file:///C:/projects/rust/build/ct/cargo/target/cit/t53/foo)
[01:33:06]      Running `rustc --crate-name foo src\lib.rs --emit=dep-info,link -C opt-level=3 --test -C metadata=4f826724f99492b5 -C extra-filename=-4f826724f99492b5 --out-dir C:\projects\rust\build\ct\cargo\target\cit\t53\foo\target\release\deps -L dependency=C:\projects\rust\build\ct\cargo\target\cit\t53\foo\target\release\deps --extern a=C:\projects\rust\build\ct\cargo\target\cit\t53\foo\target\release\deps\liba-850c3cbe05004df2.rlib`
[01:33:06]     Finished release [optimized] target(s) in 0.61 secs
[01:33:06]      Running `C:\projects\rust\build\ct\cargo\target\cit\t53\foo\target\release\deps\foo-4f826724f99492b5.exe --bench`
[01:33:06] error: bench failed
[01:33:06] ', C:\Users\appveyor\.cargo\registry\src\github.com-1ecc6299db9ec823\hamcrest-0.1.1\src\core.rs:31
[01:33:06] note: Run with `RUST_BACKTRACE=1` for a backtrace.
[01:33:06] 
[01:33:06] 
[01:33:06] failures:
[01:33:06]     test_release_ignore_panic
[01:33:06] 
[01:33:06] test result: FAILED. 61 passed; 1 failed; 0 ignored; 0 measured
[01:33:06] 
[01:33:06] error: test failed
[01:33:06] thread 'main' panicked at 'tests failed for https://github.com/rust-lang/cargo', src\tools\cargotest\main.rs:98
[01:33:06] note: Run with `RUST_BACKTRACE=1` for a backtrace.
[01:33:06] 
[01:33:06] 
[01:33:06] command did not execute successfully: "C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-tools\\x86_64-pc-windows-msvc\\release\\cargotest.exe" "C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0/bin\\cargo.exe" "C:\\projects\\rust\\build\\ct"
[01:33:06] expected success, got: exit code: 101

@TimNN

TimNN commented Apr 7, 2017

Copy link
Copy Markdown
Contributor

Yeah, looks legit, but unrelated to this PR?

@bors retry

@bors

bors commented Apr 7, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit bd4f381 with merge e3693bd...

@frewsxcv

frewsxcv commented Apr 8, 2017

Copy link
Copy Markdown
Contributor

@bora retry

Homu is stuck

@frewsxcv

frewsxcv commented Apr 8, 2017

Copy link
Copy Markdown
Contributor

@bors retry

@bors

bors commented Apr 8, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit bd4f381 with merge 6b68d9a...

@bors

bors commented Apr 8, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-appveyor

@frewsxcv

frewsxcv commented Apr 8, 2017

Copy link
Copy Markdown
Contributor

@bors retry #40546

@bors

bors commented Apr 8, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit bd4f381 with merge 547c12b...

bors added a commit that referenced this pull request Apr 8, 2017
Disable errexit for sanity checking git repo

Unblock appveyor
@bors

bors commented Apr 8, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: TimNN
Pushing 547c12b to master...

@bors bors merged commit bd4f381 into rust-lang:master Apr 8, 2017
@aidanhs aidanhs deleted the aphs-fix-appveyor-cache branch April 8, 2017 07:25
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.

6 participants