-
Notifications
You must be signed in to change notification settings - Fork 1.8k
generate stderr file on 32bit pointer system #4690
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
Conversation
|
@bors r+ |
|
📌 Commit 1f96524 has been approved by |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
generate stderr file on non x86-64 system changelog: none
|
@bors r- (I think the PR author can |
0ae50af to
bef518c
Compare
|
It is unfortunate that I couldn't build for i686 target directly on x86-64 system. |
|
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 😄 |
|
The problem is that we couldn't cross-build tools: See rust-lang/rust#62558 for details. |
|
It will fail if somebody tries to run it on 64bit system other than x86_64, like Aarch64. I think you should be able to install i686 host toolchain on x86_64 system and use it to build Clippy "natively". |
|
Thanks. I will try to add build job for i688 target later. |
|
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`. |
|
@lzutao you cannot cross compile it because of Rust bug. |
bef518c to
1044787
Compare
|
I see it now. Thanks @mati865 . This now just needs a rustup. |
1044787 to
779a17f
Compare
|
Error reading rustc version. What am I doing wrong? |
|
It has something to do either with Ubuntu or Travis. |
|
I don't want to deal with this further. |
b22f10c to
c18319b
Compare
41b006f to
7ff7b14
Compare
|
☔ The latest upstream changes (presumably #4856) made this pull request unmergeable. Please resolve the merge conflicts. |
25e7532 to
4fafa36
Compare
|
The build is green: https://travis-ci.com/rust-lang/rust-clippy/jobs/261809318 |
4fafa36 to
b57b9e3
Compare
b57b9e3 to
7156aa7
Compare
|
Thanks! @bors r+ |
|
📌 Commit 7156aa7 has been approved by |
generate stderr file on 32bit pointer system changelog: none
|
☀️ Test successful - checks-travis, status-appveyor |
changelog: none