Merge #110

110: Bump dependencies r=cuviper a=kpcyrd

This bumps rand and quickcheck to the latest version. We're about to ship num-bigint with this patch applied.

Co-authored-by: kpcyrd <[email protected]>
Co-authored-by: Josh Stone <[email protected]>

Author: 3 people

.travis.yml

@@ -1,10 +1,9 @@
 language: rust
 sudo: false
 rust:
-  - 1.15.0
-  - 1.22.0 # rand
-  - 1.26.0 # has_i128
   - 1.31.0 # 2018!
+  - 1.32.0 # rand
+  - 1.34.0 # quickcheck
   - 1.36.0 # alloc
   - stable
   - beta
@@ -27,7 +26,7 @@ matrix:
       before_script:
         - rustup target add $TARGET
       script:
-        - cargo build --verbose --target $TARGET --no-default-features --features "i128 serde"
+        - cargo build --verbose --target $TARGET --no-default-features --features "serde rand"
     - name: "rustfmt"
       rust: 1.31.0
       before_script:

Cargo.toml

@@ -37,16 +37,17 @@ name = "shootout-pidigits"
 [dependencies.num-integer]
 version = "0.1.42"
 default-features = false
+features = ["i128"]
 
 [dependencies.num-traits]
 version = "0.2.11"
 default-features = false
+features = ["i128"]
 
 [dependencies.rand]
 optional = true
-version = "0.5"
+version = "0.7"
 default-features = false
-features = ["std"]
 
 [dependencies.serde]
 optional = true
@@ -55,17 +56,11 @@ default-features = false
 
 [dependencies.quickcheck]
 optional = true
-version = "0.8"
-default-features = false
-
-[dependencies.quickcheck_macros]
-optional = true
-version = "0.8"
+version = "0.9"
 default-features = false
 
 [features]
 default = ["std"]
-i128 = ["num-integer/i128", "num-traits/i128"]
 std = ["num-integer/std", "num-traits/std"]
 
 [build-dependencies]

README.md

@@ -2,7 +2,7 @@
 
 [![crate](https://img.shields.io/crates/v/num-bigint.svg)](https://crates.io/crates/num-bigint)
 [![documentation](https://docs.rs/num-bigint/badge.svg)](https://docs.rs/num-bigint)
-![minimum rustc 1.15](https://img.shields.io/badge/rustc-1.15+-red.svg)
+![minimum rustc 1.31](https://img.shields.io/badge/rustc-1.31+-red.svg)
 [![Travis status](https://travis-ci.org/rust-num/num-bigint.svg?branch=master)](https://travis-ci.org/rust-num/num-bigint)
 
 Big integer types for Rust, `BigInt` and `BigUint`.
@@ -13,7 +13,7 @@ Add this to your `Cargo.toml`:
 
 ```toml
 [dependencies]
-num-bigint = "0.2"
+num-bigint = "0.3"
 ```
 
 and this to your crate root:
@@ -29,30 +29,26 @@ The `std` crate feature is enabled by default, and is mandatory before Rust
 `default-features = false`, you must manually enable the `std` feature yourself
 if your compiler is not new enough.
 
-Implementations for `i128` and `u128` are only available with Rust 1.26 and
-later.  The build script automatically detects this, but you can make it
-mandatory by enabling the `i128` crate feature.
-
 ### Random Generation
 
 `num-bigint` supports the generation of random big integers when the `rand`
 feature is enabled. To enable it include rand as
 
 ```toml
-rand = "0.5"
-num-bigint = { version = "0.2", features = ["rand"] }
+rand = "0.7"
+num-bigint = { version = "0.3", features = ["rand"] }
 ```
 
 Note that you must use the version of `rand` that `num-bigint` is compatible
-with: `0.5`.
+with: `0.7`.
 
 ## Releases
 
 Release notes are available in [RELEASES.md](RELEASES.md).
 
 ## Compatibility
 
-The `num-bigint` crate is tested for rustc 1.15 and greater.
+The `num-bigint` crate is tested for rustc 1.31 and greater.
 
 ## Alternatives
 
@@ -62,7 +58,7 @@ table offers a brief comparison to a few alternatives.
 
 | Crate            | License        | Min rustc | Implementation |
 | :--------------- | :------------- | :-------- | :------------- |
-| **`num-bigint`** | MIT/Apache-2.0 | 1.15      | pure rust |
+| **`num-bigint`** | MIT/Apache-2.0 | 1.31      | pure rust |
 | [`ramp`]         | Apache-2.0     | nightly   | rust and inline assembly |
 | [`rug`]          | LGPL-3.0+      | 1.31      | bundles [GMP] via [`gmp-mpfr-sys`] |
 | [`rust-gmp`]     | MIT            | stable?   | links to [GMP] |

benches/bigint.rs

@@ -9,7 +9,8 @@ extern crate test;
 
 use num_bigint::{BigInt, BigUint, RandBigInt};
 use num_traits::{FromPrimitive, Num, One, Pow, Zero};
-use rand::{SeedableRng, StdRng};
+use rand::rngs::StdRng;
+use rand::SeedableRng;
 use std::mem::replace;
 use test::Bencher;
 

benches/gcd.rs

@@ -10,7 +10,8 @@ extern crate test;
 use num_bigint::{BigUint, RandBigInt};
 use num_integer::Integer;
 use num_traits::Zero;
-use rand::{SeedableRng, StdRng};
+use rand::rngs::StdRng;
+use rand::SeedableRng;
 use test::Bencher;
 
 fn get_rng() -> StdRng {

benches/roots.rs

@@ -8,7 +8,8 @@ extern crate test;
 
 use num_bigint::{BigUint, RandBigInt};
 use num_traits::Pow;
-use rand::{SeedableRng, StdRng};
+use rand::rngs::StdRng;
+use rand::SeedableRng;
 use test::Bencher;
 
 // The `big64` cases demonstrate the speed of cases where the value

build.rs

@@ -7,17 +7,9 @@ use std::io::Write;
 use std::path::Path;
 
 fn main() {
-    let ac = autocfg::new();
-
-    if ac.probe_type("i128") {
-        autocfg::emit("has_i128");
-
-        let pointer_width = env::var("CARGO_CFG_TARGET_POINTER_WIDTH");
-        if pointer_width.as_ref().map(String::as_str) == Ok("64") {
-            autocfg::emit("u64_digit");
-        }
-    } else if env::var_os("CARGO_FEATURE_I128").is_some() {
-        panic!("i128 support was not detected!");
+    let pointer_width = env::var("CARGO_CFG_TARGET_POINTER_WIDTH");
+    if pointer_width.as_ref().map(String::as_str) == Ok("64") {
+        autocfg::emit("u64_digit");
     }
 
     autocfg::rerun_path("build.rs");

ci/big_quickcheck/Cargo.toml

@@ -0,0 +1,14 @@
+[package]
+name = "big_quickcheck"
+version = "0.1.0"
+authors = ["Josh Stone <[email protected]>"]
+
+[dependencies]
+num-integer = "0.1.42"
+num-traits = "0.2.11"
+quickcheck = "0.9"
+quickcheck_macros = "0.9"
+
+[dependencies.num-bigint]
+features = ["quickcheck"]
+path = "../.."

tests/quickcheck.rs renamed to ci/big_quickcheck/src/lib.rs

@@ -1,5 +1,10 @@
-#![cfg(feature = "quickcheck")]
-#![cfg(feature = "quickcheck_macros")]
+//! Quickcheck of `BigUint` and `BigInt`
+//!
+//! This test is in a completely separate crate so we can use `quickcheck_macros` only when
+//! `quickcheck` is active. The main crate can't have optional dev-dependencies, and it's
+//! better not to expose it as a "feature" optional dependency.
+
+#![cfg(test)]
 
 extern crate num_bigint;
 extern crate num_integer;

ci/big_rand/Cargo.toml

@@ -0,0 +1,18 @@
+[package]
+name = "big_rand"
+version = "0.1.0"
+authors = ["Josh Stone <[email protected]>"]
+
+[dependencies]
+num-traits = "0.2.11"
+rand_chacha = "0.2"
+rand_isaac = "0.2"
+rand_xorshift = "0.2"
+
+[dependencies.num-bigint]
+features = ["rand"]
+path = "../.."
+
+[dependencies.rand]
+features = ["small_rng"]
+version = "0.7"