cli: Disable html highlighting by default

See code comment for reasoning. This is fixed in v0.20.0 which is not out yet.
radicle-dev · Oct 3, 2023 · 2011f2b · 2011f2b
1 parent 8479666
commit 2011f2b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/radicle-cli/Cargo.toml b/radicle-cli/Cargo.toml
@@ -33,7 +33,9 @@ tree-sitter = { version = "0.20.0" }
 tree-sitter-highlight = { version = "0.20" }
 tree-sitter-rust = { version = "0.20" }
 tree-sitter-typescript = { version = "0.20" }
-tree-sitter-html = { version = "0.19" }
+# N.b. This crate has a C++ token scanner that causes problems when building
+# for the musl target. Hence it is optional for now.
+tree-sitter-html = { version = "0.19", optional = true }
 tree-sitter-css = { version = "0.19" }
 tree-sitter-toml = { version = "0.20" }
 tree-sitter-c = { version = "0.20" }

diff --git a/radicle-cli/src/terminal/highlight.rs b/radicle-cli/src/terminal/highlight.rs
@@ -304,6 +304,7 @@ impl Highlighter {
                 )
                 .expect("Highlighter::config: highlight configuration must be valid")
             })),
+            #[cfg(feature = "tree-sitter-html")]
             "html" => Some(self.configs.entry(language).or_insert_with(|| {
                 ts::HighlightConfiguration::new(
                     tree_sitter_html::language(),