Is a relative path prefix needed for Command?

[calebwherry]

Does just using the exe name work instead of having to join with the full path?

Originally posted by @calebwherry in #1 (comment)

[robertdfrench]

In e187ac1 you showed that we can just write Path::new("./target/debug/whatever") and it will still work on Windows. But to answer this question completely, I removed the leading ./ and all the tests still passed:

     #[test]
     fn query_version_info() {
-        let path = Path::new("./target/debug/fake_git");
+        let path = Path::new("target/debug/fake_git");
         let fake_git = Git::with_path(path);
         let version = fake_git.version().unwrap();
         assert!(version.starts_with("fake_git version 1"));

So you are right; we do not need the relative prefix here. I am inclined to say we leave it anyhow, since it's a little more explicit and Windows seems to tolerate it. But if you want to drop it, I'm not opposed.

[calebwherry]

I'm fine with leaving it, I think it makes it more clear on the intention of running from the PWD.