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

Fix wasm-bindgen feature #559

Merged
merged 1 commit into from
Jul 18, 2018
Merged

Conversation

pitdicker
Copy link
Contributor

Fixes failing build on the CI because of rust-lang/rust#52081.

.travis.yml Outdated
@@ -53,7 +53,8 @@ matrix:
#- cargo --list | egrep "\binstall-update$" -q || cargo install cargo-update
#- cargo install-update -i cargo-web
# Cargo has errors with sub-commands so ignore updating for now:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dhardy Can you remember what the problem was here (so we can track it)?

@pitdicker
Copy link
Contributor Author

pitdicker commented Jul 18, 2018

Fixed the other CI failure by pushing one commit with cargo install --force cargo-web in .travis.yml, and removing it later.

The third failure was solved by koute/stdweb#252. Finally green again 🎉.

@pitdicker pitdicker merged commit 0159528 into rust-random:master Jul 18, 2018
@dhardy
Copy link
Member

dhardy commented Jul 18, 2018

Thanks 👍

@pitdicker
Copy link
Contributor Author

No luck, it didn't work because the Travis tests on master use a different cache. I have been bad and dropped the cache of the master branch.

@pitdicker pitdicker deleted the fix_wasm_bindgen branch July 18, 2018 18:57
@dhardy
Copy link
Member

dhardy commented Jul 23, 2018

Clearing the cache is probably a good solution.

AppVeyor tests are now failing:

error[E0432]: unresolved import `core::simd`
   --> src\distributions\uniform.rs:127:11
    |
127 | use core::simd::*;

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.

2 participants