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

Check in CI that the crate builds on targets without libstd #110

Merged
merged 5 commits into from
Aug 13, 2019

Conversation

Amanieu
Copy link
Member

@Amanieu Amanieu commented Aug 5, 2019

Fixes #109

@Amanieu
Copy link
Member Author

Amanieu commented Aug 5, 2019

cc @tkaitchuck: This is blocked on const_random adding #![no_std].

@Amanieu
Copy link
Member Author

Amanieu commented Aug 6, 2019

@tkaitchuck It seems that public dependencies (public = true) are only supported on nightly.

@Amanieu
Copy link
Member Author

Amanieu commented Aug 6, 2019

Hmm I'm not sure why this worked before though...

@Amanieu
Copy link
Member Author

Amanieu commented Aug 6, 2019

Oh it's because const-random 0.1.3 didn't use public = true.

@Amanieu
Copy link
Member Author

Amanieu commented Aug 6, 2019

@tkaitchuck Can you please remove public = true from the const-random crate's Cargo.toml? And check that AHash compiles on stable.

@tkaitchuck
Copy link
Contributor

Done. Const_random 0.1.6.

@Amanieu
Copy link
Member Author

Amanieu commented Aug 6, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Aug 6, 2019

📌 Commit 8a3654c has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Aug 6, 2019

⌛ Testing commit 8a3654c with merge 6c17fc5...

bors added a commit that referenced this pull request Aug 6, 2019
Check in CI that the crate builds on targets without libstd

Fixes #109
@bors
Copy link
Contributor

bors commented Aug 6, 2019

💔 Test failed - checks-travis

@Amanieu
Copy link
Member Author

Amanieu commented Aug 7, 2019

@bors retry

@bors
Copy link
Contributor

bors commented Aug 7, 2019

⌛ Testing commit 8a3654c with merge 2aa1c47...

bors added a commit that referenced this pull request Aug 7, 2019
Check in CI that the crate builds on targets without libstd

Fixes #109
@bors
Copy link
Contributor

bors commented Aug 7, 2019

💔 Test failed - checks-travis

@Amanieu
Copy link
Member Author

Amanieu commented Aug 7, 2019

@tkaitchuck no_panic seems to be causing build failures on Windows, can you investigate?

@Amanieu
Copy link
Member Author

Amanieu commented Aug 7, 2019

@tkaitchuck Also, ahash seems to be importing the alloc crate, which is not actually used: https://travis-ci.com/rust-lang/hashbrown/jobs/223101557

@Amanieu
Copy link
Member Author

Amanieu commented Aug 7, 2019

The no_panic failure can be reproduced with:

cargo test --target x86_64-pc-windows-gnu --no-run

Make sure you have the mingw linker properly set up in ~/.cargo/config

@tkaitchuck
Copy link
Contributor

I saw two issues with running that command the first was:
rust-lang/rust#33465
Which appears to still be an unresolved issue.

But I assume based on your comment about the config that this is not what you are referring to. The second issue was due to the optimizer not being aggressive enough. I've fixed this is 0.2.10. Hopefully that resolves the issue

@Amanieu
Copy link
Member Author

Amanieu commented Aug 9, 2019

@tkaitchuck Windows builds are still failing, I think you need to mark the #[no_panic] test with #[cfg(test)].

@tkaitchuck
Copy link
Contributor

tkaitchuck commented Aug 9, 2019

Ok, done.
But I am confused, was it building it with the 'dev' profile?

@Amanieu
Copy link
Member Author

Amanieu commented Aug 9, 2019

The builds seem to be fixed now. I'm just waiting on rust-lang/rust#63302 to land on nightly for the thumbv6m fix.

By the way, no-panic should be a dev-dependency now so it doesn't get pulled in outside of tests.

@Amanieu
Copy link
Member Author

Amanieu commented Aug 9, 2019

But I am confused, was it building it with the 'dev' profile?

The reason it was still failing is because the profile flags in Cargo.toml are only taken into account when building that crate directly. They are ignored when a crate is built as a dependency.

@Amanieu
Copy link
Member Author

Amanieu commented Aug 12, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Aug 12, 2019

📌 Commit 3239c2b has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Aug 12, 2019

⌛ Testing commit 3239c2b with merge ff41962...

bors added a commit that referenced this pull request Aug 12, 2019
Check in CI that the crate builds on targets without libstd

Fixes #109
@bors
Copy link
Contributor

bors commented Aug 12, 2019

💔 Test failed - checks-travis

@Amanieu
Copy link
Member Author

Amanieu commented Aug 13, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Aug 13, 2019

📌 Commit 8ae33c9 has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Aug 13, 2019

⌛ Testing commit 8ae33c9 with merge ddc0542...

bors added a commit that referenced this pull request Aug 13, 2019
Check in CI that the crate builds on targets without libstd

Fixes #109
@bors
Copy link
Contributor

bors commented Aug 13, 2019

💔 Test failed - checks-travis

@Amanieu
Copy link
Member Author

Amanieu commented Aug 13, 2019

@bors retry

@bors
Copy link
Contributor

bors commented Aug 13, 2019

⌛ Testing commit 8ae33c9 with merge 0c1209a...

bors added a commit that referenced this pull request Aug 13, 2019
Check in CI that the crate builds on targets without libstd

Fixes #109
@bors
Copy link
Contributor

bors commented Aug 13, 2019

☀️ Test successful - checks-travis
Approved by: Amanieu
Pushing 0c1209a to master...

@bors bors merged commit 8ae33c9 into rust-lang:master Aug 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hashbrown has a hard time compiling against libcore now
3 participants