-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
don't build/run doctests when target != host #2485
Conversation
⌛ Testing commit fa34134 with merge 38bc8a3... |
💔 Test failed - cargo-win-msvc-64 |
Pushed a commit fixing the cross_tests test |
(I don't think I have r+ rights here, but worth a shot) @bors: r=alexcrichton |
@bors: r+ |
📌 Commit d37d38c has been approved by |
don't build/run doctests when target != host fixes rust-lang/rust#31907 r? @alexcrichton
💔 Test failed - cargo-mac-32 |
Pushed a commit fixing my previous fix |
but who fixes the fix of the fix?! @bors: r+ |
📌 Commit ac46a1b has been approved by |
don't build/run doctests when target != host fixes rust-lang/rust#31907 r? @alexcrichton
☀️ Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64 |
It seems doctests are disabled when cross-compiling as of rust-lang/cargo#2485. Thus, we don't need this workaround any more.
fixes rust-lang/rust#31907
r? @alexcrichton