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

appveyor: Upgrade to gcc for mingw 6.3.0 #41420

Merged
merged 1 commit into from
Apr 26, 2017

Conversation

alexcrichton
Copy link
Member

This commit sort of brings back #40777 by upgrading back to 6.3.0. While
investigating #40546 it was discovered that 6.3.0 appears to not spurious
fail in the same way that 6.2.0 does (which we're currently using). The
workaround for #40184 contained in #40777 did not work so this commit also
contains a different workaround for the gdb issue. We will not download the
6.2.0 version of gdb and use that instead of the default version that comes with
6.3.0.

I'm going to optimistically say...

Closes #40546

@rust-highfive
Copy link
Collaborator

r? @brson

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

@shepmaster shepmaster added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 21, 2017
@shepmaster
Copy link
Member

Thanks for the PR, @alexcrichton! We'll rustle up a review from @brson for you!

@TimNN
Copy link
Contributor

TimNN commented Apr 22, 2017

@alexcrichton: Can you upload the respective posix threads versions as well?

@alexcrichton
Copy link
Member Author

@TimNN already done :)

I actually thought we had already upgraded and when I got around to actually making a PR I noticed I didn't have to...

We should have win32/pthread 6.2.0/6.3.0 all uploaded

@bors
Copy link
Contributor

bors commented Apr 25, 2017

☔ The latest upstream changes (presumably #40123) made this pull request unmergeable. Please resolve the merge conflicts.

@alexcrichton
Copy link
Member Author

Updated

@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Apr 25, 2017

📌 Commit 3b33913 has been approved by petrochenkov

@arielb1 arielb1 added 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-review Status: Awaiting review from the assignee but also interested parties. labels Apr 25, 2017
@alexcrichton
Copy link
Member Author

@bors: p=1

(hopefully fixing a lot of spurious failures with this)

arielb1 pushed a commit to arielb1/rust that referenced this pull request Apr 25, 2017
…enkov

appveyor: Upgrade to gcc for mingw 6.3.0

This commit sort of brings back rust-lang#40777 by upgrading back to 6.3.0. While
investigating rust-lang#40546 it was discovered that 6.3.0 appears to not spurious
fail in the same way that 6.2.0 does (which we're currently using). The
workaround for rust-lang#40184 contained in rust-lang#40777 did not work so this commit also
contains a different workaround for the gdb issue. We will not download the
6.2.0 version of gdb and use that instead of the default version that comes with
6.3.0.

I'm going to optimistically say...

Closes rust-lang#40546
@bors
Copy link
Contributor

bors commented Apr 25, 2017

⌛ Testing commit 3b33913 with merge 9a8b446...

@bors
Copy link
Contributor

bors commented Apr 25, 2017

💔 Test failed - status-appveyor

@Mark-Simulacrum
Copy link
Member

Non-spurious failure. @alexcrichton said on IRC that they probably forgot to upload artifacts.

@alexcrichton
Copy link
Member Author

@bors: r=petrochenkov

@bors
Copy link
Contributor

bors commented Apr 25, 2017

📌 Commit d7d2e5c has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Apr 25, 2017

⌛ Testing commit d7d2e5c with merge 690233d...

@bors
Copy link
Contributor

bors commented Apr 25, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member Author

@bors: r=petrochenkov

@bors
Copy link
Contributor

bors commented Apr 26, 2017

📌 Commit e239e47 has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Apr 26, 2017

⌛ Testing commit e239e47 with merge e28a4f6...

@bors
Copy link
Contributor

bors commented Apr 26, 2017

💔 Test failed - status-appveyor

@Mark-Simulacrum
Copy link
Member

Legitimate failure:

mv: cannot stat '2017-04-20-32bits-gdborig.exe': No such file or directory

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Apr 26, 2017
appveyor.yml Outdated
# Note that we don't literally overwrite the gdb.exe binary because it appears
# to just use gdborig.exe, so that's the binary we deal with instead.
- if defined MINGW_URL appveyor-retry appveyor DownloadFile %MINGW_URL%/2017-04-20-%MSYS_BITS%bit-gdborig.exe
- if defined MINGW_URL mv 2017-04-20-%MSYS_BITS%bits-gdborig.exe %MINGW_DIR%\bin\gdborig.exe
Copy link
Member

Choose a reason for hiding this comment

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

bits needs to be bit here.

This commit sort of brings back rust-lang#40777 by upgrading back to 6.3.0. While
investigating rust-lang#40546 it was discovered that 6.3.0 appears to not spurious
fail in the same way that 6.2.0 does (which we're currently using). The
workaround for rust-lang#40184 contained in rust-lang#40777 did not work so this commit also
contains a different workaround for the gdb issue. We will not download the
6.2.0 version of gdb and use that instead of the default version that comes with
6.3.0.

I'm going to optimistically say...

Closes rust-lang#40546
@alexcrichton
Copy link
Member Author

@bors: r=petrochenkov

I sure wish there was like an appveyor-simulate command ...

@bors
Copy link
Contributor

bors commented Apr 26, 2017

📌 Commit d1db74b has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Apr 26, 2017

⌛ Testing commit d1db74b with merge 0369832...

bors added a commit that referenced this pull request Apr 26, 2017
appveyor: Upgrade to gcc for mingw 6.3.0

This commit sort of brings back #40777 by upgrading back to 6.3.0. While
investigating #40546 it was discovered that 6.3.0 appears to not spurious
fail in the same way that 6.2.0 does (which we're currently using). The
workaround for #40184 contained in #40777 did not work so this commit also
contains a different workaround for the gdb issue. We will not download the
6.2.0 version of gdb and use that instead of the default version that comes with
6.3.0.

I'm going to optimistically say...

Closes #40546
@alexcrichton
Copy link
Member Author

Homu's having trouble, but tests passed, so merging

@alexcrichton alexcrichton merged commit d1db74b into rust-lang:master Apr 26, 2017
@alexcrichton
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants