try go fix cargo fmt invocation

Author: matthiaskrgr

clippy_dev/Cargo.toml

@@ -11,3 +11,4 @@ regex = "1"
 lazy_static = "1.0"
 shell-escape = "0.1"
 walkdir = "2"
+home = "0.5.0"

clippy_dev/src/fmt.rs

@@ -1,3 +1,4 @@
+use home;
 use shell_escape::escape;
 use std::ffi::OsStr;
 use std::io;
@@ -31,6 +32,26 @@ struct FmtContext {
     verbose: bool,
 }
 
+fn cargo_fmt(context: &FmtContext, path: &Path) -> Result<bool, CliError> {
+    let mut args = vec!["+nightly", "fmt", "--all"];
+    if context.check {
+        args.push("--");
+        args.push("--check");
+    }
+    // in case something aliases cargo=cargo --foo, the order of args will be messed up
+    // because we will end up with cargo --foo +nightly fmt but the "+nightly" needs to come first
+    // get the absolute path to the cargo binary inside $CARGO_HOME
+    let cargo_home = home::cargo_home().expect("failed to get cargo home path!");
+    let cargo_path = cargo_home.join("bin").join("cargo");
+    assert!(
+        cargo_path.is_file(),
+        format!("ERROR: '{}' is not a path to the cargo binary!", cargo_path.display())
+    );
+    let success = exec(context, cargo_path, path, &args)?;
+
+    Ok(success)
+}
+
 pub fn run(check: bool, verbose: bool) {
     fn try_run(context: &FmtContext) -> Result<bool, CliError> {
         let mut success = true;
@@ -134,17 +155,6 @@ fn exec(
     Ok(success)
 }
 
-fn cargo_fmt(context: &FmtContext, path: &Path) -> Result<bool, CliError> {
-    let mut args = vec!["+nightly", "fmt", "--all"];
-    if context.check {
-        args.push("--");
-        args.push("--check");
-    }
-    let success = exec(context, "cargo", path, &args)?;
-
-    Ok(success)
-}
-
 fn rustfmt_test(context: &FmtContext) -> Result<(), CliError> {
     let program = "rustfmt";
     let dir = std::env::current_dir()?;