From 4877e9ee5ca551e1a68749b6c5efa3eadc2f1a67 Mon Sep 17 00:00:00 2001 From: Robin Krahl Date: Tue, 1 Jan 2019 22:17:16 +0000 Subject: Manually set the libnitrokey version MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As the nitrokey-sys crate’s version is no longer directly mapped to the libnitrokey version, this patch updates the build script. Instead of assuming that the nitrokey-sys version is also the libnitrokey version, we specify the libnitrokey version manually. --- build.rs | 73 +++++++++++++++++++++++----------------------------------------- 1 file changed, 26 insertions(+), 47 deletions(-) diff --git a/build.rs b/build.rs index c52bf34..af64a69 100644 --- a/build.rs +++ b/build.rs @@ -1,57 +1,38 @@ -extern crate cc; - use std::env; +use std::fs; use std::io; use std::io::{Read, Write}; -use std::fs; use std::path; +use std::string; -struct Version { - major: String, - minor: String, - git: String, -} +use cc; -fn stringify(err: env::VarError) -> String { - format!("{}", err) +#[derive(Clone, Copy, Debug, PartialEq)] +struct Version { + major: u32, + minor: u32, + patch: Option, } -fn extract_git_version(pre: &str) -> Result { - // If a pre-release version is set, it is expected to have the format - // pre.v...g, where and are the last major and minor version, - // is the number of commits since this version and is the hash of the last commit. - let parts: Vec<&str> = pre.split('.').collect(); - if parts.len() != 5 { - return Err(format!("'{}' is not a valid pre-release version", pre)); +impl string::ToString for Version { + fn to_string(&self) -> String { + match self.patch { + Some(patch) => format!("v{}.{}.{}", self.major, self.minor, patch), + None => format!("v{}.{}", self.major, self.minor), + } } - Ok(format!("{}.{}-{}-{}", parts[1], parts[2], parts[3], parts[4])) } -fn get_version() -> Result { - let major = env::var("CARGO_PKG_VERSION_MAJOR").map_err(stringify)?; - let minor = env::var("CARGO_PKG_VERSION_MINOR").map_err(stringify)?; - let patch = env::var("CARGO_PKG_VERSION_PATCH").map_err(stringify)?; - let pre = env::var("CARGO_PKG_VERSION_PRE").map_err(stringify)?; - - let git = match pre.is_empty() { - true => match patch.is_empty() { - true => format!("v{}.{}", major, minor), - false => format!("v{}.{}.{}", major, minor, patch), - }, - false => extract_git_version(&pre)?, - }; - - Ok(Version { - major, - minor, - git, - }) -} +const LIBNITROKEY_VERSION: Version = Version { + major: 3, + minor: 4, + patch: Some(1), +}; fn prepare_version_source( - version: &Version, + version: Version, out_path: &path::Path, - library_path: &path::Path + library_path: &path::Path, ) -> io::Result { let out = out_path.join("version.cc"); let template = library_path.join("version.cc.in"); @@ -62,9 +43,9 @@ fn prepare_version_source( drop(file); let data = data - .replace("@PROJECT_VERSION_MAJOR@", &version.major) - .replace("@PROJECT_VERSION_MINOR@", &version.minor) - .replace("@PROJECT_VERSION_GIT@", &version.git); + .replace("@PROJECT_VERSION_MAJOR@", &version.major.to_string()) + .replace("@PROJECT_VERSION_MINOR@", &version.minor.to_string()) + .replace("@PROJECT_VERSION_GIT@", &version.to_string()); let mut file = fs::File::create(&out)?; file.write_all(data.as_bytes())?; @@ -76,8 +57,6 @@ fn main() { let out_dir = env::var("OUT_DIR").expect("Environment variable OUT_DIR is not set"); let out_path = path::PathBuf::from(out_dir); - let version = get_version().expect("Could not extract library version"); - let sources = [ "DeviceCommunicationExceptions.cpp", "NK_C_API.cc", @@ -87,10 +66,10 @@ fn main() { "log.cc", "misc.cc", ]; - let library_dir = format!("libnitrokey-{}", version.git); + let library_dir = format!("libnitrokey-{}", LIBNITROKEY_VERSION.to_string()); let library_path = path::Path::new(&library_dir); - let version_source = prepare_version_source(&version, &out_path, &library_path) + let version_source = prepare_version_source(LIBNITROKEY_VERSION, &out_path, &library_path) .expect("Could not prepare the version source file"); cc::Build::new() -- cgit v1.2.1