Bump version

Bump mongoc driver to 1.22.0

CHANGELOG.md

@@ -1,3 +1,25 @@
+# 0.14.2
+* Disable zstd if package is not found
+
+# 0.14.1
+* Disable snappy if package is not found
+
+# 0.14.0
+* Bump mongoc to 1.22.0
+
+# 0.13.6
+* Memory leak fix in Bscon::new by (by austinjones)
+
+# 0.13.5
+* Hard version lock on sys dependency
+
+# 0.13.4
+* Tweak linking options, use snappy if available
+
+# 0.13.3
+* Upgrade libmongoc to 1.17.4
+* Another static linking fix
+
 # 0.13.2
 * Fix probe of libbson so static linking actually works
 

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name          = "mongo_driver"
-version       = "0.13.2"
+version       = "0.14.2"
 authors       = ["Thijs Cadier <thijs@appsignal.com>"]
 description   = "Mongo Rust driver built on top of the Mongo C driver"
 readme        = "README.md"
@@ -12,7 +12,6 @@ license       = "MIT/Apache-2.0"
 edition       = "2018"
 
 [badges]
-travis-ci = { repository = "thijsc/mongo-rust-driver" }
 is-it-maintained-issue-resolution = { repository = "thijsc/mongo-rust-driver" }
 is-it-maintained-open-issues = { repository = "thijsc/mongo-rust-driver" }
 
@@ -25,7 +24,7 @@ serde_derive = "1.0"
 
 [dependencies.mongoc-sys]
 path    = "mongoc-sys"
-version = "1.17.5"
+version = "=1.22.0-2"
 
 [dev-dependencies]
 chrono = "^0.4"

README.md

@@ -1,6 +1,5 @@
 # Mongo Rust Driver
 
-[![Build Status](https://travis-ci.org/thijsc/mongo-rust-driver.svg)](https://travis-ci.org/thijsc/mongo-rust-driver)
 [![Crate](http://meritbadge.herokuapp.com/mongo_driver)](https://crates.io/crates/mongo_driver)
 
 Mongo Rust driver built on top of the [Mongo C driver](https://github.com/mongodb/mongo-c-driver).

mongoc-sys/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name        = "mongoc-sys"
-version     = "1.17.5"
+version     = "1.22.0-2"
 description = "Sys package with installer and bindings for mongoc"
 authors     = ["Thijs Cadier <thijs@appsignal.com>"]
 build       = "build.rs"

mongoc-sys/build.rs

@@ -10,83 +10,94 @@ fn main() {
         .next()
         .expect("Crate version is not valid");
 
-    pkg_config::Config::new().probe("zlib").expect("Cannot find zlib");
+    let out_dir_var = env::var("OUT_DIR").expect("No out dir");
+    let out_dir = Path::new(&out_dir_var);
+    let driver_src_path = out_dir.join(format!("mongo-c-driver-{}", mongoc_version));
 
-    if pkg_config::Config::new()
-        .atleast_version(mongoc_version)
-        .statik(true)
-        .probe("libmongoc-static-1.0")
-        .is_err()
-        {
-            let out_dir_var = env::var("OUT_DIR").expect("No out dir");
-            let out_dir = Path::new(&out_dir_var);
-            let driver_src_path = out_dir.join(format!("mongo-c-driver-{}", mongoc_version));
+    let libmongoc_path = out_dir.join("usr/local/lib/libmongoc-static-1.0.a");
+    if !libmongoc_path.exists() {
+        // Download and extract driver archive
+        let url = format!(
+            "https://github.com/mongodb/mongo-c-driver/releases/download/{}/mongo-c-driver-{}.tar.gz",
+            mongoc_version,
+            mongoc_version
+        );
+        assert!(
+            Command::new("curl").arg("-O") // Save to disk
+                .current_dir(out_dir)
+                .arg("-L") // Follow redirects
+                .arg(url)
+                .status()
+                .expect("Could not run curl")
+                .success()
+        );
 
-            let libmongoc_path = out_dir.join("usr/local/lib/libmongoc-static-1.0.a");
-            if !libmongoc_path.exists() {
-                // Download and extract driver archive
-                let url = format!(
-                    "https://github.com/mongodb/mongo-c-driver/releases/download/{}/mongo-c-driver-{}.tar.gz",
-                    mongoc_version,
-                    mongoc_version
-                );
-                assert!(
-                    Command::new("curl").arg("-O") // Save to disk
-                        .current_dir(out_dir)
-                        .arg("-L") // Follow redirects
-                        .arg(url)
-                        .status()
-                        .expect("Could not run curl")
-                        .success()
-                );
+        let archive_name = format!("mongo-c-driver-{}.tar.gz", mongoc_version);
+        assert!(
+            Command::new("tar")
+                .current_dir(out_dir)
+                .arg("xzf")
+                .arg(&archive_name)
+                .status()
+                .expect("Could not run tar")
+                .success()
+        );
 
-                let archive_name = format!("mongo-c-driver-{}.tar.gz", mongoc_version);
-                assert!(
-                    Command::new("tar")
-                        .current_dir(out_dir)
-                        .arg("xzf")
-                        .arg(&archive_name)
-                        .status()
-                        .expect("Could not run tar")
-                        .success()
-                );
+        // Set up cmake command
+        let mut cmake = Command::new("cmake");
+        cmake.current_dir(&driver_src_path);
 
-                // Set up cmake command
-                let mut cmake = Command::new("cmake");
-                cmake.current_dir(&driver_src_path);
+        let pkg = pkg_config::Config::new();
+        pkg.probe("zlib").expect("Cannot find zlib");
+        #[cfg(target_os = "linux")] pkg.probe("icu-i18n").expect("Cannot find icu");
+        match pkg.probe("snappy") {
+            Ok(_) => {
+                cmake.arg("-DENABLE_SNAPPY=ON");
+            },
+            Err(e) => {
+                println!("Snappy not found: {}", e);
+                cmake.arg("-DENABLE_SNAPPY=OFF");
+            }
+        }
+        match pkg.probe("zstd") {
+            Ok(_) => {
+                cmake.arg("-DENABLE_ZSTD=ON");
+            },
+            Err(e) => {
+                println!("Zstd not found: {}", e);
+                cmake.arg("-DENABLE_ZSTD=OFF");
+            }
+        }
 
-                cmake.arg("-DENABLE_AUTOMATIC_INIT_AND_CLEANUP=OFF");
-                cmake.arg("-DENABLE_SSL=OPENSSL");
-                cmake.arg("-DENABLE_SASL=OFF");
-                cmake.arg("-DENABLE_STATIC=ON");
-                cmake.arg("-DENABLE_BSON=ON");
-                cmake.arg("-DENABLE_ENABLE_EXAMPLES=OFF");
-                cmake.arg("-DENABLE_TESTS=OFF");
-                cmake.arg("-DENABLE_SHM_COUNTERS=OFF");
-                cmake.arg("-DWITH_PIC=ON");
-                cmake.arg("-DWITH_SNAPPY=OFF");
+        cmake.arg("-DENABLE_AUTOMATIC_INIT_AND_CLEANUP=OFF");
+        cmake.arg("-DENABLE_SSL=OPENSSL");
+        cmake.arg("-DENABLE_SASL=OFF");
+        cmake.arg("-DENABLE_STATIC=ON");
+        cmake.arg("-DENABLE_BSON=ON");
+        cmake.arg("-DENABLE_ENABLE_EXAMPLES=OFF");
+        cmake.arg("-DENABLE_TESTS=OFF");
+        cmake.arg("-DENABLE_SHM_COUNTERS=OFF");
+        cmake.arg("-DWITH_PIC=ON");
 
-                // Run in current dir
-                cmake.arg(".");
+        // Run in current dir
+        cmake.arg(".");
 
-                // Run cmake command
-                assert!(cmake.status().expect("Could not run cmake").success());
+        // Run cmake command
+        assert!(cmake.status().expect("Could not run cmake").success());
 
-                // Set up make install command
-                let mut make = Command::new("make");
-                make.current_dir(&driver_src_path);
-                make.arg(format!("DESTDIR={}", out_dir.to_string_lossy()));
-                make.arg("install");
+        // Set up make install command
+        let mut make = Command::new("make");
+        make.current_dir(&driver_src_path);
+        make.arg(format!("DESTDIR={}", out_dir.to_string_lossy()));
+        make.arg("install");
 
-                // Run make command
-                assert!(make.status().expect("Could not run make install").success());
-            }
+        // Run make command
+        assert!(make.status().expect("Could not run make install").success());
+    }
 
-            // Output to Cargo
-            println!("cargo:rustc-link-search=native={}/usr/local/lib", &out_dir.to_string_lossy());
-            println!("cargo:rustc-link-lib=static=bson-static-1.0");
-            println!("cargo:rustc-link-lib=static=mongoc-static-1.0");
-            println!("cargo:rustc-link-lib=resolv");
-            #[cfg(target_os = "linux")] println!("cargo:rustc-link-lib=icuuc");
-        }
+    // Output to Cargo
+    println!("cargo:rustc-link-search=native={}/usr/local/lib", &out_dir.to_string_lossy());
+    println!("cargo:rustc-link-lib=static=bson-static-1.0");
+    println!("cargo:rustc-link-lib=static=mongoc-static-1.0");
+    println!("cargo:rustc-link-lib=resolv");
 }

src/bsonc.rs

@@ -16,7 +16,12 @@ pub struct Bsonc {
 
 impl Bsonc {
     pub fn new() -> Bsonc {
-        Bsonc::from_ptr(unsafe { bindings::bson_new() })
+        let inner: *const bindings::bson_t = unsafe { bindings::bson_new() };
+        assert!(!inner.is_null());
+        Bsonc {
+            inner: inner as *mut bindings::bson_t,
+            destroy_inner_on_drop: true,
+        }
     }
 
     /// Create a bsonc from a raw pointer. Does not run cleanup
@@ -111,6 +116,13 @@ impl Drop for Bsonc {
 
 #[cfg(test)]
 mod tests {
+    #[test]
+    fn test_bsonc_new() {
+        for _ in 0..100 {
+            let _ = super::Bsonc::new();
+        }
+    }
+
     #[test]
     fn test_bsonc_from_and_as_document() {
         let document = doc! { "key": "value" };

src/database.rs

@@ -45,8 +45,7 @@ pub struct Database<'a> {
 }
 
 impl<'a> Database<'a> {
-    #[doc(ignore)]
-    pub fn new(
+    pub(crate) fn new(
         created_by: CreatedBy<'a>,
         inner: *mut bindings::mongoc_database_t
     ) -> Database<'a> {