From 93e6eac15a1302aa790fec2778cdfee3eefd94e3 Mon Sep 17 00:00:00 2001 From: Michael Davis Date: Sat, 28 May 2022 21:06:31 -0500 Subject: [PATCH] mention the C++ compiler in the grammar build failure message (#2597) Earlier in the builder we enable C++ (`.cpp(true)`) but only mention the C compiler in the build failure message. Some grammars that have C++ external scanners can provoke build failures in this step if a C++ compiler isn't installed, so mentioning it in the error message should help out debugging. --- helix-loader/src/grammar.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/helix-loader/src/grammar.rs b/helix-loader/src/grammar.rs index df88f24b..99791f26 100644 --- a/helix-loader/src/grammar.rs +++ b/helix-loader/src/grammar.rs @@ -355,7 +355,9 @@ fn build_tree_sitter_library(src_path: &Path, grammar: GrammarConfiguration) -> } } - let output = command.output().context("Failed to execute C compiler")?; + let output = command + .output() + .context("Failed to execute C/C++ compiler")?; if !output.status.success() { return Err(anyhow!( "Parser compilation failed.\nStdout: {}\nStderr: {}",