Skip to content

Commit 21e0d4c

Browse files
bors[bot]fishrockz
andauthored
Merge #116
116: Add check for underlying tool and install hint r=Emilgardis a=fishrockz This addresses #113 or at least the eventual issue of not realising that i had not installed everything needed. I am not sure this is ideal but seems to work ok. I'm happy to adapt etc I only know the code base enough to test this out locally and have something helpful to me. Co-authored-by: William Salmon <[email protected]>
2 parents be3b709 + 1f467c6 commit 21e0d4c

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/tool.rs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,11 @@ impl Tool {
6262
Ok(p) => p,
6363
};
6464

65+
if !path.exists() {
66+
eprintln!("Could not find tool: {}\nat: {}\nConsider `rustup component add llvm-tools-preview`", self.name(), path.to_string_lossy());
67+
process::exit(102)
68+
};
69+
6570
// Note: The first argument is the name of the binary (e.g. `rust-nm`)
6671
let args = env::args().skip(1);
6772

0 commit comments

Comments
 (0)