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

Rollup of 7 pull requests #67426

Closed
wants to merge 23 commits into from
Closed

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Dec 19, 2019

Successful merges:

Failed merges:

r? @ghost

estebank and others added 23 commits December 11, 2019 17:42
This code was added to avoid UB in LLVM 6 and earlier, but we no longer support those LLVM versions.
Since https://reviews.llvm.org/D47807 (released in LLVM 7), uitofp does exactly what we need.

Closes rust-lang#51872
Normalize ident

Perform unicode normalization on identifiers. Resolving the first bullet point in rust-lang#55467.
…=varkor

Use structured suggestion for disambiguating method calls

Fix rust-lang#65635.
…xcrichton

add string.insert benchmarks

This adds benchmarks for `String::insert` and `String::insert_str`
…atthewjasper

Remove now-redundant range check on u128 -> f32 casts

This code was added to avoid UB in LLVM 6 and earlier, but we no longer support those LLVM versions.
Since https://reviews.llvm.org/D47807 (released in LLVM 7), uitofp does exactly what we need.

Closes rust-lang#51872
Fix unresolved type span inside async object

Closes rust-lang#65180
r? @estebank
It's hard to create a minimal repro for that issue, [decided](https://rust-lang.zulipchat.com/#narrow/stream/187312-wg-async-foundations/topic/meeting.202019.2E12.2E17/near/183675659) to give up finding mcve.
cc [previous take](rust-lang#65668)
@Centril
Copy link
Contributor Author

Centril commented Dec 19, 2019

@bors r+ p=7 rollup=never

@bors
Copy link
Contributor

bors commented Dec 19, 2019

📌 Commit 4ff4d6c has been approved by Centril

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 19, 2019
@Centril Centril added the rollup A PR which is a rollup label Dec 19, 2019
@bors
Copy link
Contributor

bors commented Dec 19, 2019

⌛ Testing commit 4ff4d6c with merge d353e5e70db245d34b75d1841f4032ad57f61d0d...

@rust-highfive
Copy link
Collaborator

The job mingw-check of 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-19T18:38:59.1668888Z Build completed successfully in 0:01:41
2019-12-19T18:38:59.1679313Z + /scripts/validate-toolstate.sh
2019-12-19T18:38:59.1740667Z Cloning into 'rust-toolstate'...
2019-12-19T18:38:59.7015712Z Traceback (most recent call last):
2019-12-19T18:38:59.7015911Z   File "../../src/tools/publish_toolstate.py", line 303, in <module>
2019-12-19T18:38:59.7015990Z     cur_datetime
2019-12-19T18:38:59.7016081Z   File "../../src/tools/publish_toolstate.py", line 182, in update_latest
2019-12-19T18:38:59.7016872Z     for os in ['windows', 'linux']
2019-12-19T18:38:59.7016976Z   File "../../src/tools/publish_toolstate.py", line 182, in <dictcomp>
2019-12-19T18:38:59.7017245Z     for os in ['windows', 'linux']
2019-12-19T18:38:59.7017327Z   File "../../src/tools/publish_toolstate.py", line 111, in read_current_status
2019-12-19T18:38:59.7017900Z     (commit, status) = line.split('\t', 1)
2019-12-19T18:38:59.7018004Z ValueError: need more than 1 value to unpack
2019-12-19T18:38:59.7052713Z   local time: Thu Dec 19 18:38:59 UTC 2019
2019-12-19T18:39:00.2254762Z   network time: Thu, 19 Dec 2019 18:39:00 GMT
2019-12-19T18:39:00.2259016Z == end clock drift check ==
2019-12-19T18:39:02.2252179Z 
2019-12-19T18:39:02.2252179Z 
2019-12-19T18:39:02.2426917Z ##[error]Bash exited with code '1'.
2019-12-19T18:39:02.2463725Z ##[section]Starting: Checkout
2019-12-19T18:39:02.2466195Z ==============================================================================
2019-12-19T18:39:02.2466466Z Task         : Get sources
2019-12-19T18:39:02.2466572Z 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 19, 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 19, 2019
@Centril
Copy link
Contributor Author

Centril commented Dec 19, 2019

Infra stuff are broken ^--, @bors treeclosed=100

@Centril
Copy link
Contributor Author

Centril commented Dec 19, 2019

@bors retry

@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 19, 2019
Centril added a commit to Centril/rust that referenced this pull request Dec 19, 2019
Rollup of 7 pull requests

Successful merges:

 - rust-lang#66670 (Normalize ident)
 - rust-lang#66755 (Remove a const-if-hack in RawVec)
 - rust-lang#67127 (Use structured suggestion for disambiguating method calls)
 - rust-lang#67281 (add string.insert benchmarks)
 - rust-lang#67328 (Remove now-redundant range check on u128 -> f32 casts)
 - rust-lang#67392 (Fix unresolved type span inside async object)
 - rust-lang#67421 (Fix internal documentation typo)

Failed merges:

r? @ghost
Centril added a commit to Centril/rust that referenced this pull request Dec 19, 2019
Rollup of 7 pull requests

Successful merges:

 - rust-lang#66670 (Normalize ident)
 - rust-lang#66755 (Remove a const-if-hack in RawVec)
 - rust-lang#67127 (Use structured suggestion for disambiguating method calls)
 - rust-lang#67281 (add string.insert benchmarks)
 - rust-lang#67328 (Remove now-redundant range check on u128 -> f32 casts)
 - rust-lang#67392 (Fix unresolved type span inside async object)
 - rust-lang#67421 (Fix internal documentation typo)

Failed merges:

r? @ghost
bors added a commit that referenced this pull request Dec 19, 2019
Rollup of 3 pull requests

Successful merges:

 - #67351 (Set release channel on non-dist builders)
 - #67426 (Rollup of 7 pull requests)
 - #67432 (Fix toolstate history format)

Failed merges:

r? @ghost
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Dec 19, 2019
Rollup of 7 pull requests

Successful merges:

 - rust-lang#66670 (Normalize ident)
 - rust-lang#66755 (Remove a const-if-hack in RawVec)
 - rust-lang#67127 (Use structured suggestion for disambiguating method calls)
 - rust-lang#67281 (add string.insert benchmarks)
 - rust-lang#67328 (Remove now-redundant range check on u128 -> f32 casts)
 - rust-lang#67392 (Fix unresolved type span inside async object)
 - rust-lang#67421 (Fix internal documentation typo)

Failed merges:

r? @ghost
bors added a commit that referenced this pull request Dec 19, 2019
Rollup of 3 pull requests

Successful merges:

 - #67351 (Set release channel on non-dist builders)
 - #67426 (Rollup of 7 pull requests)
 - #67432 (Fix toolstate history format)

Failed merges:

r? @ghost
@Centril Centril deleted the rollup-rswp4rf branch December 23, 2019 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup 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.

10 participants