Fixes permission issues with userns on Docker host

Fixes #90

Author: lawliet89

Cargo.lock

@@ -11,6 +11,8 @@ version = "0.1.11-dev"
 
 [dependencies]
 error-chain = "0.7.1"
+lazy_static = "0.2"
 libc = "0.2.18"
 rustc_version = "0.1.7"
+semver = "0.6.0"
 toml = "0.2.1"

Cargo.toml

@@ -3,17 +3,54 @@ use std::path::PathBuf;
 use std::process::{Command, ExitStatus};
 use std::{env, fs};
 
+use semver::{Version, VersionReq};
+
 use {Target, Toml};
 use cargo::Root;
 use errors::*;
 use extensions::CommandExt;
 use id;
 use rustc;
 
+lazy_static! {
+    /// Retrieve the Docker Daemon version.
+    ///
+    /// # Panics
+    /// Panics if the version cannot be retrieved or parsed
+    static ref DOCKER_VERSION: Version = {
+        let version_string = Command::new("docker")
+                                .arg("version")
+                                .arg("--format={{.Server.APIVersion}}")
+                                .run_and_get_stdout(false)
+                                .expect("Unable to obtain Docker version");
+        // API versions don't have "patch" version
+        Version::parse(&format!("{}.0", version_string.trim()))
+            .expect("Cannot parse Docker engine version")
+    };
+
+    /// Version requirements for user namespace.
+    ///
+    /// # Panics
+    /// Panics if the parsing fails
+    static ref USERNS_REQUIREMENT: VersionReq = {
+        VersionReq::parse(">= 1.24")
+            .expect("Unable to parse version requirements")
+    };
+}
+
+/// Add the `userns` flag, if needed
+pub fn docker_command(subcommand: &str) -> Command {
+    let mut docker = Command::new("docker");
+    docker.arg(subcommand);
+    if USERNS_REQUIREMENT.matches(&DOCKER_VERSION) {
+        docker.args(&["--userns", "host"]);
+    }
+    docker
+}
+
 /// Register QEMU interpreters
 pub fn register(verbose: bool) -> Result<()> {
-    Command::new("docker")
-        .arg("run")
+    docker_command("run")
         .arg("--privileged")
         .arg("--rm")
         .arg("-it")
@@ -64,10 +101,9 @@ pub fn run(target: &Target,
         .run(verbose)
         .chain_err(|| "couldn't generate Cargo.lock")?;
 
-    let mut docker = Command::new("docker");
+    let mut docker = docker_command("run");
 
     docker
-        .arg("run")
         .arg("--rm")
         .args(&["--user", &format!("{}:{}", id::user(), id::group())])
         .args(&["-e", "CARGO_HOME=/cargo"])

src/docker.rs

@@ -1,7 +1,10 @@
 #[macro_use]
 extern crate error_chain;
+#[macro_use]
+extern crate lazy_static;
 extern crate libc;
 extern crate rustc_version;
+extern crate semver;
 extern crate toml;
 
 mod cargo;