Skip to content

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Oct 18, 2019

changelog: none

@phansch
Copy link
Contributor

phansch commented Oct 18, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Oct 18, 2019

📌 Commit 1f96524 has been approved by phansch

@tesuji

This comment has been minimized.

@tesuji

This comment has been minimized.

@tesuji tesuji closed this Oct 18, 2019
@tesuji tesuji reopened this Oct 18, 2019
@tesuji tesuji closed this Oct 18, 2019
@tesuji tesuji reopened this Oct 18, 2019
@bors
Copy link
Contributor

bors commented Oct 18, 2019

⌛ Testing commit 1f96524 with merge ebcf8b2...

bors added a commit that referenced this pull request Oct 18, 2019
generate stderr file on non x86-64 system

changelog: none
@tesuji tesuji closed this Oct 18, 2019
@tesuji tesuji reopened this Oct 18, 2019
@flip1995
Copy link
Member

@bors r-

(I think the PR author can r- bors. At least you can do this in rust-lang/rust)

@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch 2 times, most recently from 0ae50af to bef518c Compare October 18, 2019 12:31
@tesuji tesuji changed the title generate stderr file on non x86-64 system generate stderr file on 32bit pointer system Oct 18, 2019
@tesuji
Copy link
Contributor Author

tesuji commented Oct 18, 2019

It is unfortunate that I couldn't build for i686 target directly on x86-64 system.
I kept getting error missing crate rustc.

@flip1995
Copy link
Member

It seems to work though. We don't test this in travis anyway, since travis also runs on 64bit. I think this is more an alibi test 😄

@tesuji
Copy link
Contributor Author

tesuji commented Oct 21, 2019

The problem is that we couldn't cross-build tools: See rust-lang/rust#62558 for details.

@mati865
Copy link
Member

mati865 commented Oct 21, 2019

It will fail if somebody tries to run it on 64bit system other than x86_64, like Aarch64.
In that case lesser evil would be to run this test only on x86.

I think you should be able to install i686 host toolchain on x86_64 system and use it to build Clippy "natively".

@tesuji
Copy link
Contributor Author

tesuji commented Oct 21, 2019

Thanks. I will try to add build job for i688 target later.

@tesuji
Copy link
Contributor Author

tesuji commented Oct 22, 2019

So I got the following errors on my x86_64 linux system:

% cargo check --all-targets --workspace --target i686-unknown-linux-gnu                    
   Compiling cargo_metadata v0.8.2
    Checking clippy_lints v0.0.212 (/home/lzutao/fork/rust/clippy/clippy_lints)
    Checking compiletest_rs v0.3.25
error[E0463]: can't find crate for `rustc_macros` which `rustc` depends on
  --> /home/lzutao/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.25/src/lib.rs:19:1
   |
19 | extern crate rustc;
   | ^^^^^^^^^^^^^^^^^^^ can't find crate

error: aborting due to previous error

For more information about this error, try `rustc --explain E0463`.

@mati865
Copy link
Member

mati865 commented Oct 22, 2019

@lzutao you cannot cross compile it because of Rust bug.
You need i686 host toolchain.

@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch from bef518c to 1044787 Compare October 22, 2019 09:05
@tesuji
Copy link
Contributor Author

tesuji commented Oct 22, 2019

I see it now. Thanks @mati865 . This now just needs a rustup.

@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch from 1044787 to 779a17f Compare October 22, 2019 09:17
@tesuji
Copy link
Contributor Author

tesuji commented Oct 22, 2019

Error reading rustc version. What am I doing wrong?
https://travis-ci.com/rust-lang/rust-clippy/jobs/248209462#L174

@mati865
Copy link
Member

mati865 commented Oct 22, 2019

It has something to do either with Ubuntu or Travis.
On my x86_64 Arch Linux:

$ rustc +nightly-2019-10-20-i686-unknown-linux-gnu -vV                                                                                                                                  
rustc 1.40.0-nightly (c23a7aa77 2019-10-19)
binary: rustc
commit-hash: c23a7aa778b0dfeffbf83b099bdf971242c1e1ac
commit-date: 2019-10-19
host: i686-unknown-linux-gnu
release: 1.40.0-nightly
LLVM version: 9.0

@tesuji
Copy link
Contributor Author

tesuji commented Oct 22, 2019

I don't want to deal with this further.
Let's block this PR on #4714 .

@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch from b22f10c to c18319b Compare November 26, 2019 14:53
@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch 3 times, most recently from 41b006f to 7ff7b14 Compare November 28, 2019 03:51
@tesuji tesuji closed this Nov 28, 2019
@tesuji tesuji reopened this Nov 28, 2019
@flip1995 flip1995 added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work labels Nov 28, 2019
@bors
Copy link
Contributor

bors commented Nov 29, 2019

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

@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch 4 times, most recently from 25e7532 to 4fafa36 Compare November 30, 2019 17:35
@tesuji
Copy link
Contributor Author

tesuji commented Nov 30, 2019

@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch from 4fafa36 to b57b9e3 Compare November 30, 2019 18:00
@tesuji tesuji force-pushed the fix-test-on-non-amd64 branch from b57b9e3 to 7156aa7 Compare December 1, 2019 12:06
@flip1995
Copy link
Member

flip1995 commented Dec 1, 2019

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Dec 1, 2019

📌 Commit 7156aa7 has been approved by flip1995

bors added a commit that referenced this pull request Dec 1, 2019
generate stderr file on 32bit pointer system

changelog: none
@bors
Copy link
Contributor

bors commented Dec 1, 2019

⌛ Testing commit 7156aa7 with merge 45196ce...

@bors
Copy link
Contributor

bors commented Dec 1, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 45196ce to master...

@bors bors merged commit 7156aa7 into rust-lang:master Dec 1, 2019
@tesuji tesuji deleted the fix-test-on-non-amd64 branch December 1, 2019 19:15
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 from the author. (Use `@rustbot ready` to update this status)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants