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.

Sign up for GitHub

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

Jump to bottom

Fix the configure.py TOML field for a couple LLVM options #67286

Merged
merged 1 commit into from
Dec 19, 2019
Merged

Fix the configure.py TOML field for a couple LLVM options #67286

merged 1 commit into from
Dec 19, 2019

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Dec 13, 2019

The actual fields in config.toml.example have dashes, not underscores.

@cuviper
Fix the configure.py TOML field for a couple LLVM options
6c7c512 
The actual fields in `config.toml.example` have dashes, not underscores.
@rust-highfive
Copy link
Collaborator

r? @alexcrichton

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 13, 2019
@Dylan-DPC-zz
Copy link

@bors r+

@bors
Copy link
Contributor

bors commented Dec 14, 2019

📌 Commit 6c7c512 has been approved by Dylan-DPC

@bors bors 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 Dec 14, 2019
@bors
Copy link
Contributor

bors commented Dec 16, 2019

⌛ Testing commit 6c7c512 with merge 075bf232f748586d08c1b17a925b7c87fceb5daa...

@rust-highfive
Copy link
Collaborator

Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log. 
2019-12-16T07:48:11.2787825Z CommonProgramW6432=C:\Program Files\Common Files
2019-12-16T07:48:11.2787909Z DEPLOY_BUCKET=rust-lang-ci2
2019-12-16T07:48:11.2787992Z ENDPOINT_URL_SYSTEMVSSCONNECTION=https://dev.azure.com/rust-lang/
2019-12-16T07:48:11.2788085Z EXEPATH=C:\Program Files\Git\bin
2019-12-16T07:48:11.2788177Z Fix the configure.py TOML field for a couple LLVM options
2019-12-16T07:48:11.2788332Z GIT_TERMINAL_PROMPT=0
2019-12-16T07:48:11.2788420Z GOROOT=C:\Go1.12.7
2019-12-16T07:48:11.2788542Z GOROOT_1_10_X64=C:\Go1.10.8
2019-12-16T07:48:11.2788606Z GOROOT_1_11_X64=C:\Go1.11.12
---
2019-12-16T07:48:11.2801111Z TMP=/tmp
2019-12-16T07:48:11.2801200Z TOOLSTATE_ISSUES_API_URL=https://api.github.com/repos/rust-lang/rust/issues
2019-12-16T07:48:11.2801301Z TOOLSTATE_PUBLISH=1
2019-12-16T07:48:11.2801400Z TOOLSTATE_REPO=https://github.com/rust-lang-nursery/rust-toolstate
2019-12-16T07:48:11.2801499Z The actual fields in `config.toml.example` have dashes, not underscores.
2019-12-16T07:48:11.2801681Z USERDOMAIN=fv-az379
2019-12-16T07:48:11.2801768Z USERDOMAIN_ROAMINGPROFILE=fv-az379
2019-12-16T07:48:11.2801841Z USERNAME=VssAdministrator
2019-12-16T07:48:11.2801931Z USERPROFILE=C:\Users\VssAdministrator
---
2019-12-16T07:48:24.4808891Z Chocolatey v0.10.15
2019-12-16T07:49:21.5785834Z Installing the following packages:
2019-12-16T07:49:21.5790006Z msys2
2019-12-16T07:49:21.5799451Z By installing you accept licenses for the packages.
2019-12-16T07:49:26.0671584Z msys2 not installed. An error occurred during installation:
2019-12-16T07:49:26.0672226Z  The remote server returned an error: (520) Origin Error. Origin Error
2019-12-16T07:49:26.1023674Z The install of msys2 was NOT successful.
2019-12-16T07:49:26.1024130Z msys2 not installed. An error occurred during installation:
2019-12-16T07:49:26.1024130Z msys2 not installed. An error occurred during installation:
2019-12-16T07:49:26.1024414Z  The remote server returned an error: (520) Origin Error. Origin Error
2019-12-16T07:49:26.1163172Z Chocolatey installed 0/1 packages. 1 packages failed.
2019-12-16T07:49:26.1163672Z  See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
2019-12-16T07:49:26.1167456Z 
2019-12-16T07:49:26.1172834Z Failures
2019-12-16T07:49:26.1172834Z Failures
2019-12-16T07:49:26.1179063Z  - msys2 (exited 1) - msys2 not installed. An error occurred during installation:
2019-12-16T07:49:26.1179422Z  The remote server returned an error: (520) Origin Error. Origin Error
2019-12-16T07:49:26.6317173Z 
2019-12-16T07:49:26.6416187Z ##[error]Bash exited with code '1'.
2019-12-16T07:49:26.6560745Z ##[section]Starting: Checkout
2019-12-16T07:49:26.6677061Z ==============================================================================
2019-12-16T07:49:26.6677174Z Task         : Get sources
2019-12-16T07:49:26.6677301Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@bors
Copy link
Contributor

bors commented Dec 16, 2019

💔 Test failed - checks-azure

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 16, 2019
@cuviper
Copy link
Member Author

cuviper commented Dec 16, 2019

The install of msys2 was NOT successful.

Hopefully spurious? This PR should be trivial for rollup too, as the configure options in question are not used in CI -- couldn't even if we wanted, as they were totally broken.

@bors retry
@bors rollup

@bors bors 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 Dec 16, 2019
@mati865
Copy link
Contributor

mati865 commented Dec 17, 2019

Hopefully spurious?

Chocolatey broke (again).

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Dec 17, 2019
@Mark-Simulacrum
Rollup merge of rust-lang#67286 - cuviper:configure-llvm, r=Dylan-DPC
ae1bab2 
Fix the configure.py TOML field for a couple LLVM options

The actual fields in `config.toml.example` have dashes, not underscores.
@Mark-Simulacrum Mark-Simulacrum mentioned this pull request Dec 17, 2019
Closed
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Dec 18, 2019
@Mark-Simulacrum
Rollup merge of rust-lang#67286 - cuviper:configure-llvm, r=Dylan-DPC
743869b 
Fix the configure.py TOML field for a couple LLVM options

The actual fields in `config.toml.example` have dashes, not underscores.
@Mark-Simulacrum Mark-Simulacrum mentioned this pull request Dec 18, 2019
Closed
tmandry added a commit to tmandry/rust that referenced this pull request Dec 18, 2019
@tmandry
Rollup merge of rust-lang#67286 - cuviper:configure-llvm, r=Dylan-DPC
0ad4e67 
Fix the configure.py TOML field for a couple LLVM options

The actual fields in `config.toml.example` have dashes, not underscores.
@tmandry tmandry mentioned this pull request Dec 18, 2019
Closed
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Dec 18, 2019
@Mark-Simulacrum
Rollup merge of rust-lang#67286 - cuviper:configure-llvm, r=Dylan-DPC
ce2b3fc 
Fix the configure.py TOML field for a couple LLVM options

The actual fields in `config.toml.example` have dashes, not underscores.
@Mark-Simulacrum Mark-Simulacrum mentioned this pull request Dec 18, 2019
Closed
bors added a commit that referenced this pull request Dec 18, 2019
@bors
Auto merge of #67396 - Mark-Simulacrum:rollup-85lxz7h, r=Mark-Simulacrum
b746a58 
Rollup of 5 pull requests

Successful merges:

 - #66716 (Implement `DebugStruct::non_exhaustive`.)
 - #67286 (Fix the configure.py TOML field for a couple LLVM options)
 - #67321 (make htons const fn)
 - #67351 (Set release channel on non-dist builders)
 - #67382 (Remove some unnecessary `ATTR_*` constants.)

Failed merges:

r? @ghost
bors added a commit that referenced this pull request Dec 18, 2019
@bors
Auto merge of #67396 - Mark-Simulacrum:rollup-85lxz7h, r=Mark-Simulacrum
eb07728 
Rollup of 5 pull requests

Successful merges:

 - #66716 (Implement `DebugStruct::non_exhaustive`.)
 - #67286 (Fix the configure.py TOML field for a couple LLVM options)
 - #67321 (make htons const fn)
 - #67351 (Set release channel on non-dist builders)
 - #67382 (Remove some unnecessary `ATTR_*` constants.)

Failed merges:

r? @ghost
Centril added a commit to Centril/rust that referenced this pull request Dec 19, 2019
@Centril
Rollup merge of rust-lang#67286 - cuviper:configure-llvm, r=Dylan-DPC
0f19977 
Fix the configure.py TOML field for a couple LLVM options

The actual fields in `config.toml.example` have dashes, not underscores.
@Centril Centril mentioned this pull request Dec 19, 2019
Merged
bors added a commit that referenced this pull request Dec 19, 2019
@bors
Auto merge of #67419 - Centril:rollup-v7b0ypv, r=Centril
0de96d3 
Rollup of 8 pull requests

Successful merges:

 - #67189 (Unify binop wording)
 - #67270 (std: Implement `LineWriter::write_vectored`)
 - #67286 (Fix the configure.py TOML field for a couple LLVM options)
 - #67321 (make htons const fn)
 - #67382 (Remove some unnecessary `ATTR_*` constants.)
 - #67389 (Remove `SO_NOSIGPIPE` dummy variable on platforms that don't use it.)
 - #67394 (Remove outdated references to @t from comments)
 - #67406 (Suggest associated type when the specified one cannot be found)

Failed merges:

r? @ghost
@bors bors merged commit 6c7c512 into rust-lang:master Dec 19, 2019
@cuviper cuviper deleted the configure-llvm branch April 3, 2020 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexcrichton alexcrichton

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@cuviper @rust-highfive @Dylan-DPC-zz @bors @mati865 @alexcrichton