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

submodules: update clippy from 164310dd to dc69a5c0 #62943

Merged
merged 1 commit into from
Jul 28, 2019

Conversation

matthiaskrgr
Copy link
Member

@matthiaskrgr matthiaskrgr commented Jul 24, 2019

Changes:

ci: temporarily disable rustfmt checks/tetss since it's broken for nightly
rustup https://github.com/rust-lang/rust/pull/62964
Bump version of clippy_dummy
update test stderr, not sure which rustc pull request caused this.
rustup https://github.com/rust-lang/rust/pull/62859
Fix tests for edition 2018 compatibility
Revert "Revert global fmt config and use `rustfmt::skip`"
Fix breakage due to rust-lang/rust#60913
Fix breakage due to rust-lang/rust#62705
Revert global fmt config and use `rustfmt::skip`
Fix fmt
rustup https://github.com/rust-lang/rust/pull/62679/
Update pulldown-cmark to 0.5.3
rustup https://github.com/rust-lang/rust/pull/62764
Add test
Format code
Decrease maximum length for stderr files
Improved imports
Fix "unkown clippy lint" error in UI test.
Corrections for PR review.
Implement lint for inherent to_string() method.
UI Test Cleanup: Extract match_ref_pats tests
Update UI tests
Allow no_effect lint
Remove comment
cargo fmt
UI Test Cleanup: Split up checked_unwrap tests
Removed lintining on never type.
UI Test Cleanup: Split out out_of_bounds_indexing
false positives fixes of `implicit_return`
Ignore generated fresh lifetimes in elision check.

fixes clippy toolstate
r? @Manishearth

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 24, 2019
@Manishearth
Copy link
Member

@bors r+ p=2

@bors
Copy link
Contributor

bors commented Jul 24, 2019

📌 Commit 1bf17ba has been approved by Manishearth

@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 Jul 24, 2019
@rust-highfive
Copy link
Collaborator

The job LinuxTools of your PR failed (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-07-24T17:51:59.2640384Z ##[command]git remote add origin https://github.com/rust-lang/rust
2019-07-24T17:51:59.2855053Z ##[command]git config gc.auto 0
2019-07-24T17:51:59.2953070Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2019-07-24T17:51:59.3036207Z ##[command]git config --get-all http.proxy
2019-07-24T17:51:59.3179539Z ##[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/62943/merge:refs/remotes/pull/62943/merge
---
2019-07-24T17:52:35.8084690Z do so (now or later) by using -b with the checkout command again. Example:
2019-07-24T17:52:35.8084724Z 
2019-07-24T17:52:35.8085359Z   git checkout -b <new-branch-name>
2019-07-24T17:52:35.8085401Z 
2019-07-24T17:52:35.8085497Z HEAD is now at 45493b41d Merge 1bf17ba95500eb3afd15d437b89d1b07344202d4 into 27a6a304e2baaabca88059753f020377f2476978
2019-07-24T17:52:35.8225455Z ##[section]Finishing: Checkout
2019-07-24T17:52:35.8233031Z ##[section]Starting: Decide whether to run this job
2019-07-24T17:52:35.8236130Z Task         : Bash
2019-07-24T17:52:35.8236217Z Description  : Run a Bash script on macOS, Linux, or Windows
2019-07-24T17:52:35.8236268Z Version      : 3.151.3
2019-07-24T17:52:35.8236317Z Author       : Microsoft Corporation
2019-07-24T17:52:35.8236317Z Author       : Microsoft Corporation
2019-07-24T17:52:35.8236395Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/bash
2019-07-24T17:52:35.8236453Z ==============================================================================
2019-07-24T17:52:35.9663313Z Generating script.
2019-07-24T17:52:35.9692048Z ========================== Starting Command Output ===========================
2019-07-24T17:52:35.9712173Z [command]/bin/bash --noprofile --norc /home/vsts/work/_temp/4b5404bb-a480-4bf4-91e8-cadbeb4157d5.sh
2019-07-24T17:52:36.3096830Z Executing the job since submodules are updated
2019-07-24T17:52:36.3201462Z ##[section]Finishing: Decide whether to run this job
2019-07-24T17:52:36.3211887Z ==============================================================================
2019-07-24T17:52:36.3212045Z Task         : Bash
2019-07-24T17:52:36.3212093Z Description  : Run a Bash script on macOS, Linux, or Windows
2019-07-24T17:52:36.3212139Z Version      : 3.151.3
---
2019-07-24T20:09:59.4263418Z The state of "clippy-driver" has changed from "build-fail" to "test-pass"
2019-07-24T20:09:59.4263812Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:09:59.4566507Z [master 2e4521b] (linux CI update)
2019-07-24T20:09:59.4566750Z  1 file changed, 1 insertion(+)
2019-07-24T20:10:00.1422241Z remote: Invalid username or password.
2019-07-24T20:10:00.1424100Z fatal: Authentication failed for 'https://github.com/rust-lang-nursery/rust-toolstate.git/'
2019-07-24T20:10:00.4787502Z  * branch            master     -> FETCH_HEAD
2019-07-24T20:10:00.4998472Z HEAD is now at 2a2a469 (windows CI update)
2019-07-24T20:10:00.5138637Z The state of "clippy-driver" has changed from "build-fail" to "test-pass"
2019-07-24T20:10:00.5139136Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:00.5139136Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:00.5358355Z [master d754d27] (linux CI update)
2019-07-24T20:10:00.5358550Z  1 file changed, 1 insertion(+)
2019-07-24T20:10:00.8512397Z fatal: could not read Username for 'https://github.com': No such device or address
2019-07-24T20:10:02.7012764Z  * branch            master     -> FETCH_HEAD
2019-07-24T20:10:02.7173586Z HEAD is now at 2a2a469 (windows CI update)
2019-07-24T20:10:02.7306800Z The state of "clippy-driver" has changed from "build-fail" to "test-pass"
2019-07-24T20:10:02.7307231Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:02.7307231Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:02.7518963Z [master 1df5082] (linux CI update)
2019-07-24T20:10:02.7519182Z  1 file changed, 1 insertion(+)
2019-07-24T20:10:03.1089951Z fatal: could not read Username for 'https://github.com': No such device or address
2019-07-24T20:10:04.1649389Z  * branch            master     -> FETCH_HEAD
2019-07-24T20:10:04.1751853Z HEAD is now at 2a2a469 (windows CI update)
2019-07-24T20:10:04.1889245Z The state of "clippy-driver" has changed from "build-fail" to "test-pass"
2019-07-24T20:10:04.1889592Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:04.1889592Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:04.2097342Z [master 916498a] (linux CI update)
2019-07-24T20:10:04.2097495Z  1 file changed, 1 insertion(+)
2019-07-24T20:10:05.1758532Z fatal: could not read Username for 'https://github.com': No such device or address
2019-07-24T20:10:06.0618531Z  * branch            master     -> FETCH_HEAD
2019-07-24T20:10:06.0784926Z HEAD is now at 2a2a469 (windows CI update)
2019-07-24T20:10:06.0918137Z The state of "clippy-driver" has changed from "build-fail" to "test-pass"
2019-07-24T20:10:06.0918978Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:06.0918978Z The state of "rustc-guide" has changed from "test-pass" to ""
2019-07-24T20:10:06.1117904Z [master 3a465de] (linux CI update)
2019-07-24T20:10:06.1117999Z  1 file changed, 1 insertion(+)
2019-07-24T20:10:06.7886643Z fatal: could not read Username for 'https://github.com': No such device or address
2019-07-24T20:10:07.4809065Z  * branch            master     -> FETCH_HEAD
2019-07-24T20:10:07.4974662Z HEAD is now at 2a2a469 (windows CI update)
2019-07-24T20:10:07.4974662Z HEAD is now at 2a2a469 (windows CI update)
2019-07-24T20:10:08.5722626Z ##[error]Bash exited with code '1'.
2019-07-24T20:10:08.5764723Z ##[section]Starting: Checkout
2019-07-24T20:10:08.5766780Z ==============================================================================
2019-07-24T20:10:08.5766844Z Task         : Get sources
2019-07-24T20:10:08.5766897Z 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 Jul 24, 2019

⌛ Testing commit 1bf17ba with merge 67dd66c...

bors added a commit that referenced this pull request Jul 24, 2019
submodules: update clippy from 164310d to f8e04ff

Changes:
````
update test stderr, not sure which rustc pull request caused this.
rustup #62859
Fix tests for edition 2018 compatibility
Revert "Revert global fmt config and use `rustfmt::skip`"
Fix breakage due to #60913
Fix breakage due to #62705
Revert global fmt config and use `rustfmt::skip`
Fix fmt
rustup #62679
Update pulldown-cmark to 0.5.3
rustup #62764
Add test
Format code
Decrease maximum length for stderr files
Improved imports
Fix "unkown clippy lint" error in UI test.
Corrections for PR review.
Implement lint for inherent to_string() method.
UI Test Cleanup: Extract match_ref_pats tests
Update UI tests
Allow no_effect lint
Remove comment
cargo fmt
UI Test Cleanup: Split up checked_unwrap tests
Removed lintining on never type.
UI Test Cleanup: Split out out_of_bounds_indexing
false positives fixes of `implicit_return`
Ignore generated fresh lifetimes in elision check.
````

fixes clippy toolstate
r? @Manishearth
@bors
Copy link
Contributor

bors commented Jul 24, 2019

💔 Test failed - checks-azure

@rust-highfive
Copy link
Collaborator

The job i686-msvc-1 of your PR failed (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-07-24T22:00:39.7155013Z [551/2435] Linking CXX executable bin\llvm-tblgen.exe
2019-07-24T22:00:39.8882652Z [552/2435] Building Options.inc...
2019-07-24T22:00:39.8981288Z [553/2435] Building Options.inc...
2019-07-24T22:00:40.2638159Z [554/2435] Building CXX object lib\ToolDrivers\llvm-lib\CMakeFiles\LLVMLibDriver.dir\LibDriver.cpp.obj
2019-07-24T22:00:40.2638607Z FAILED: lib/ToolDrivers/llvm-lib/CMakeFiles/LLVMLibDriver.dir/LibDriver.cpp.obj 
2019-07-24T22:00:40.2639133Z D:\a\1\s\build\bootstrap\debug\sccache-plus-cl.exe  /nologo -TP -DGTEST_HAS_RTTI=0 -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_DEBUG_POINTER_IMPL="" -D_FILE_OFFSET_BITS=64 -D_HAS_EXCEPTIONS=0 -D_LARGEFILE_SOURCE -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Ilib\ToolDrivers\llvm-lib -ID:\a\1\s\src\llvm-project\llvm\lib\ToolDrivers\llvm-lib -Iinclude -ID:\a\1\s\src\llvm-project\llvm\include /nologo /MT /Zc:inline /Zc:strictStrings /Oi /Zc:rvalueCast /Brepro /W4  -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -Wno-noexcept-type -Wno-comment /MT /O2 /Ob2   -UNDEBUG  /EHs-c- /GR- /showIncludes /Folib\ToolDrivers\llvm-lib\CMakeFiles\LLVMLibDriver.dir\LibDriver.cpp.obj /Fdlib\ToolDrivers\llvm-lib\CMakeFiles\LLVMLibDriver.dir\LLVMLibDriver.pdb -c D:\a\1\s\src\llvm-project\llvm\lib\ToolDrivers\llvm-lib\LibDriver.cpp
2019-07-24T22:00:40.2639628Z clang-cl.exe: warning: argument unused during compilation: '-mno-incremental-linker-compatible' [-Wunused-command-line-argument]
2019-07-24T22:00:40.2639780Z In file included from D:\a\1\s\src\llvm-project\llvm\lib\ToolDrivers\llvm-lib\LibDriver.cpp:18:
2019-07-24T22:00:40.2639922Z In file included from D:\a\1\s\src\llvm-project\llvm\include\llvm/Bitcode/BitcodeReader.h:19:
2019-07-24T22:00:40.2640040Z In file included from D:\a\1\s\src\llvm-project\llvm\include\llvm/IR/ModuleSummaryIndex.h:27:
2019-07-24T22:00:40.2640189Z In file included from D:\a\1\s\src\llvm-project\llvm\include\llvm/IR/Module.h:23:
2019-07-24T22:00:40.2640317Z D:\a\1\s\src\llvm-project\llvm\include\llvm/IR/Attributes.h(73,14):  fatal error: 'llvm/IR/Attributes.inc' file not found
2019-07-24T22:00:40.2640438Z     #include "llvm/IR/Attributes.inc"
2019-07-24T22:00:40.2640620Z 1 error generated.
2019-07-24T22:00:40.3378436Z [555/2435] Building CXX object lib\ToolDrivers\llvm-dlltool\CMakeFiles\LLVMDlltoolDriver.dir\DlltoolDriver.cpp.obj
2019-07-24T22:00:40.3378436Z [555/2435] Building CXX object lib\ToolDrivers\llvm-dlltool\CMakeFiles\LLVMDlltoolDriver.dir\DlltoolDriver.cpp.obj
2019-07-24T22:00:40.3378636Z ninja: build stopped: subcommand failed.
2019-07-24T22:00:40.3476204Z command did not execute successfully, got: exit code: 1
2019-07-24T22:00:40.3476374Z 
2019-07-24T22:00:40.3476374Z 
2019-07-24T22:00:40.3476751Z build script failed, must exit now', C:\Users\VssAdministrator\.cargo\registry\src\github.com-1ecc6299db9ec823\cmake-0.1.38\src\lib.rs:813:5
2019-07-24T22:00:40.3488962Z  finished in 187.706
2019-07-24T22:00:40.3488962Z  finished in 187.706
2019-07-24T22:00:40.3538479Z failed to run: D:\a\1\s\build\bootstrap\debug\bootstrap test --exclude src/test/ui --exclude src/test/run-pass --exclude src/test/compile-fail --exclude src/test/run-pass-fulldeps --exclude src/tools/linkchecker
2019-07-24T22:00:40.3538662Z Build completed unsuccessfully in 0:49:10
2019-07-24T22:00:40.3747306Z make: *** [Makefile:84: ci-subset-1] Error 1
2019-07-24T22:00:40.4031445Z ##[error]Bash exited with code '2'.
2019-07-24T22:00:40.4292437Z ##[section]Starting: Upload CPU usage statistics
2019-07-24T22:00:40.4419146Z ==============================================================================
2019-07-24T22:00:40.4419556Z Task         : Bash
2019-07-24T22:00:40.4419670Z Description  : Run a Bash script on macOS, Linux, or Windows

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 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 Jul 24, 2019
@matthiaskrgr
Copy link
Member Author

Looks spurious, can someone retry?

@Centril
Copy link
Contributor

Centril commented Jul 26, 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 Jul 26, 2019
@bors
Copy link
Contributor

bors commented Jul 26, 2019

⌛ Testing commit 1bf17ba with merge fdfd0728dee2361cff15b02ee7bd16b6ac4364b9...

@matthiaskrgr
Copy link
Member Author

This won't merge because the last pullrequest broke clippy again.

@Centril
Copy link
Contributor

Centril commented Jul 26, 2019

@bors r- retry

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 26, 2019
Changes:
````
ci: temporarily disable rustfmt checks/tetss since it's broken for nightly
rustup rust-lang#62964
Bump version of clippy_dummy
update test stderr, not sure which rustc pull request caused this.
rustup rust-lang#62859
Fix tests for edition 2018 compatibility
Revert "Revert global fmt config and use `rustfmt::skip`"
Fix breakage due to rust-lang#60913
Fix breakage due to rust-lang#62705
Revert global fmt config and use `rustfmt::skip`
Fix fmt
rustup rust-lang#62679
Update pulldown-cmark to 0.5.3
rustup rust-lang#62764
Add test
Format code
Decrease maximum length for stderr files
Improved imports
Fix "unkown clippy lint" error in UI test.
Corrections for PR review.
Implement lint for inherent to_string() method.
UI Test Cleanup: Extract match_ref_pats tests
Update UI tests
Allow no_effect lint
Remove comment
cargo fmt
UI Test Cleanup: Split up checked_unwrap tests
Removed lintining on never type.
UI Test Cleanup: Split out out_of_bounds_indexing
false positives fixes of `implicit_return`
Ignore generated fresh lifetimes in elision check.
````
@matthiaskrgr
Copy link
Member Author

rebased, should work again (for now..)

@matthiaskrgr matthiaskrgr changed the title submodules: update clippy from 164310dd to f8e04ff3 submodules: update clippy from 164310dd to dc69a5c0 Jul 28, 2019
@matthiaskrgr
Copy link
Member Author

r? @Manishearth

@Centril
Copy link
Contributor

Centril commented Jul 28, 2019

@bors delegate+

@bors
Copy link
Contributor

bors commented Jul 28, 2019

✌️ @matthiaskrgr can now approve this pull request

@matthiaskrgr
Copy link
Member Author

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Jul 28, 2019

📌 Commit 79e6123 has been approved by matthiaskrgr

@bors
Copy link
Contributor

bors commented Jul 28, 2019

⌛ Testing commit 79e6123 with merge 023525d...

bors added a commit that referenced this pull request Jul 28, 2019
submodules: update clippy from 164310d to dc69a5c

Changes:
````
ci: temporarily disable rustfmt checks/tetss since it's broken for nightly
rustup #62964
Bump version of clippy_dummy
update test stderr, not sure which rustc pull request caused this.
rustup #62859
Fix tests for edition 2018 compatibility
Revert "Revert global fmt config and use `rustfmt::skip`"
Fix breakage due to #60913
Fix breakage due to #62705
Revert global fmt config and use `rustfmt::skip`
Fix fmt
rustup #62679
Update pulldown-cmark to 0.5.3
rustup #62764
Add test
Format code
Decrease maximum length for stderr files
Improved imports
Fix "unkown clippy lint" error in UI test.
Corrections for PR review.
Implement lint for inherent to_string() method.
UI Test Cleanup: Extract match_ref_pats tests
Update UI tests
Allow no_effect lint
Remove comment
cargo fmt
UI Test Cleanup: Split up checked_unwrap tests
Removed lintining on never type.
UI Test Cleanup: Split out out_of_bounds_indexing
false positives fixes of `implicit_return`
Ignore generated fresh lifetimes in elision check.
````

fixes clippy toolstate
r? @Manishearth
@bors
Copy link
Contributor

bors commented Jul 28, 2019

☀️ Test successful - checks-azure
Approved by: matthiaskrgr
Pushing 023525d to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jul 28, 2019
@bors bors merged commit 79e6123 into rust-lang:master Jul 28, 2019
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #62943!

Tested on commit 023525d.
Direct link to PR: #62943

🎉 clippy-driver on windows: build-fail → test-pass (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Jul 28, 2019
Tested on commit rust-lang/rust@023525d.
Direct link to PR: <rust-lang/rust#62943>

🎉 clippy-driver on windows: build-fail → test-pass (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).
@matthiaskrgr
Copy link
Member Author

Looks like clippy was not even built on linux 😭

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

5 participants