Skip to content

Commit

Permalink
fix: When using extended LLVM check if the include folder is present
Browse files Browse the repository at this point in the history
  • Loading branch information
SergioGasquez committed Dec 5, 2023
1 parent 4cd89f3 commit 025792b
Showing 1 changed file with 12 additions and 3 deletions.
15 changes: 12 additions & 3 deletions src/toolchain/llvm.rs
Original file line number Diff line number Diff line change
Expand Up @@ -187,10 +187,19 @@ impl Installable for Llvm {
let mut exports: Vec<String> = Vec::new();

#[cfg(unix)]
let is_installed = Path::new(&self.path).exists();
let install_path = if self.extended {
Path::new(&self.path).join("esp-clang").join("include")
} else {
Path::new(&self.path).to_path_buf()
};
#[cfg(windows)]
let is_installed = self.path.join(&self.version).exists();
if is_installed {
let install_path = if self.extended {
self.path.join(&self.version).join("include")
} else {
self.path.join(&self.version)
};

if install_path.exists() {
warn!(
"Previous installation of LLVM exists in: '{}'. Reusing this installation",
self.path.to_str().unwrap()
Expand Down

0 comments on commit 025792b

Please sign in to comment.