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 8 pull requests #65769

Closed
wants to merge 24 commits into from
Closed

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Oct 24, 2019

Successful merges:

Failed merges:

r? @ghost

varkor and others added 24 commits October 22, 2019 12:26
Add secondary span labels with no text to make it clear when there's a
mismatch bewteen the positional arguments in a format string and the
arguments to the macro. This shouldn't affect experienced users, but it
should make it easier for newcomers to more clearly understand how
`format!()` and `println!()` are supposed to be used.

```
error: 2 positional arguments in format string, but there is 1 argument
 --> file8.rs:2:14
  |
2 |     format!("{} {}", 1);
  |              ^^ ^^   -
```

instead of

```
error: 2 positional arguments in format string, but there is 1 argument
 --> file8.rs:2:14
  |
2 |     format!("{} {}", 1);
  |              ^^ ^^
```
Pulls in rust-lang/hashbrown#119 which should be a good improvement for
compile times of hashmap-heavy crates.
…-E0740, r=Dylan-DPC

Create new error E0741 and add long error explanation

Part of rust-lang#61137.

Creates E0740 error code and add its long error explanation.
…ructural_match, r=petrochenkov

Forbid non-`structural_match` types in const generics

Fixes rust-lang#60286.
Update cargo

6 commits in 3a9abe3f065554a7fbc59f440df2baba4a6e47ee..3ba5f27170db10af7a92f2b682e049397197b8fa
2019-10-15 15:55:35 +0000 to 2019-10-22 15:05:18 +0000
- Fix typo in `cargo install --profile` help (rust-lang/cargo#7532)
- Use stricter -Z flag parsing. (rust-lang/cargo#7531)
- Set timestamp on generated files in archive to now (rust-lang/cargo#7523)
- Support rustc's `-Z panic-abort-tests` in Cargo (rust-lang/cargo#7460)
- rustfmt for nightly changes. (rust-lang/cargo#7526)
- Allow --all-features in root of virtual workspace. (rust-lang/cargo#7525)
Tweak format string error to point at arguments always

Add secondary span labels with no text to make it clear when there's a
mismatch bewteen the positional arguments in a format string and the
arguments to the macro. This shouldn't affect experienced users, but it
should make it easier for newcomers to more clearly understand how
`format!()` and `println!()` are supposed to be used.

```
error: 2 positional arguments in format string, but there is 1 argument
 --> file8.rs:2:14
  |
2 |     format!("{} {}", 1);
  |              ^^ ^^   -
```

instead of

```
error: 2 positional arguments in format string, but there is 1 argument
 --> file8.rs:2:14
  |
2 |     format!("{} {}", 1);
  |              ^^ ^^
```

r? @Centril
Don't assert for different instance on impl trait alias

Closes rust-lang#65679
r? @Centril @nikomatsakis
… r=Mark-Simulacrum

Update hashbrown to 0.6.2

Pulls in rust-lang/hashbrown#119 which should be a good improvement for
compile times of hashmap-heavy crates.
@Centril
Copy link
Contributor Author

Centril commented Oct 24, 2019

@bors r+ p=8 rollup=never

@bors
Copy link
Contributor

bors commented Oct 24, 2019

📌 Commit a0cf897 has been approved by Centril

@bors
Copy link
Contributor

bors commented Oct 24, 2019

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

@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 Oct 24, 2019
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
@Centril Centril closed this Oct 24, 2019
@rust-highfive
Copy link
Collaborator

The job x86_64-gnu-llvm-6.0 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-10-24T17:47:01.8280673Z ##[command]git remote add origin https://github.com/rust-lang/rust
2019-10-24T17:47:01.8497584Z ##[command]git config gc.auto 0
2019-10-24T17:47:01.8585716Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2019-10-24T17:47:01.8653839Z ##[command]git config --get-all http.proxy
2019-10-24T17:47:01.8816740Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/65769/merge:refs/remotes/pull/65769/merge
---
2019-10-24T17:53:51.2492446Z    Compiling serde_json v1.0.40
2019-10-24T17:53:53.1167116Z    Compiling tidy v0.1.0 (/checkout/src/tools/tidy)
2019-10-24T17:54:04.8524782Z     Finished release [optimized] target(s) in 1m 32s
2019-10-24T17:54:04.8613404Z tidy check
2019-10-24T17:54:06.1735850Z tidy error: duplicate error code: 741
2019-10-24T17:54:06.1737103Z tidy error: /checkout/src/librustc_resolve/error_codes.rs:1910: E0741: r##"
2019-10-24T17:54:06.1737520Z tidy error: /checkout/src/librustc_typeck/error_codes.rs:5008: E0741: r##"
2019-10-24T17:54:07.5201896Z Found 484 error codes
2019-10-24T17:54:07.5202044Z Found 0 error codes with no tests
2019-10-24T17:54:07.5202161Z Done!
2019-10-24T17:54:07.5202206Z some tidy checks failed
2019-10-24T17:54:07.5202206Z some tidy checks failed
2019-10-24T17:54:07.5202275Z 
2019-10-24T17:54:07.5203587Z 
2019-10-24T17:54:07.5204923Z command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0-tools-bin/tidy" "/checkout/src" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0/bin/cargo" "--no-vendor"
2019-10-24T17:54:07.5205962Z 
2019-10-24T17:54:07.5206007Z 
2019-10-24T17:54:07.5220856Z failed to run: /checkout/obj/build/bootstrap/debug/bootstrap test src/tools/tidy
2019-10-24T17:54:07.5220960Z Build completed unsuccessfully in 0:01:35
2019-10-24T17:54:07.5220960Z Build completed unsuccessfully in 0:01:35
2019-10-24T17:54:07.5270918Z == clock drift check ==
2019-10-24T17:54:07.5279195Z   local time: Thu Oct 24 17:54:07 UTC 2019
2019-10-24T17:54:07.6860660Z   network time: Thu, 24 Oct 2019 17:54:07 GMT
2019-10-24T17:54:07.6860807Z == end clock drift check ==
2019-10-24T17:54:09.1237031Z 
2019-10-24T17:54:09.1348628Z ##[error]Bash exited with code '1'.
2019-10-24T17:54:09.1383326Z ##[section]Starting: Checkout
2019-10-24T17:54:09.1384979Z ==============================================================================
2019-10-24T17:54:09.1385028Z Task         : Get sources
2019-10-24T17:54:09.1385066Z 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)

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.

None yet

9 participants