issue 2561, send all [bat error]s to stderr in default_error_handler

CHANGELOG.md

@@ -3,7 +3,7 @@
 ## Features
 
 ## Bugfixes
-
+- Send all bat error messages to stderr, see #2827 (@deboard)
 - Fix `NO_COLOR` support, see #2767 (@acuteenvy)
 
 ## Other

src/error.rs

@@ -67,7 +67,15 @@ pub fn default_error_handler(error: &Error, output: &mut dyn Write) {
             .ok();
         }
         _ => {
-            writeln!(output, "{}: {}", Red.paint("[bat error]"), error).ok();
+            // default - always write [bat error] to stderr
+            let stderr = std::io::stderr();
+            writeln!(
+                &mut stderr.lock(),
+                "{}: {}",
+                Red.paint("[bat error]"),
+                error
+            )
+            .ok();
         }
     };
 }

tests/scripts/stderr1.sh

@@ -0,0 +1,14 @@
+#!/bin/bash
+
+## test for issue 2561
+
+OUTPUT=$(mktemp)
+BAT=bat
+code=$($BAT /tmp 2> $OUTPUT; cat $OUTPUT | grep error; echo $?)
+
+if [[ $code == 1 ]]; then
+    echo "stderr test fsil"
+    exit 1
+fi
+
+exit 0