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

rand: Remove serde1 feature #830

Merged
merged 2 commits into from Jun 25, 2019
Merged

rand: Remove serde1 feature #830

merged 2 commits into from Jun 25, 2019

Commits on Jun 24, 2019

  1. rand: Remove serde1 feature

    This feature does not do anything in the `rand` crate and was forwarded
    to `rand_core`, where it was used for `BlockRng` and `BlockRng64`.
    However, crates using this feature would use `rand_core` directly
    instead, making the feature pointless for `rand`.
    
    It is not expected to use this feature for any of the RNGs in `rand`.
    
    This hopefully makes it possible to build `rand` on docs.rs again.
    (See rust-lang/docs.rs#369.)
    
    Now the serde1 feature does not do anything, but it is still available
    for backwards compatibility.
    vks committed Jun 24, 2019
    Copy the full SHA
    84b0ddf View commit details
    Browse the repository at this point in the history
  2. Change miri and cross platform checks for serde1 feature

    - `rand_xoshiro` should be built with the feature.
    -  For `rand`, it does not do anything. Instead, test it for
       `rand_core`.
    vks committed Jun 24, 2019
    Copy the full SHA
    5b530fb View commit details
    Browse the repository at this point in the history