Update logic for detecting if the output stream is connected to a ter…

…minal remkop#2083
cushon · Aug 11, 2023 · 3212b4a · 3212b4a
1 parent 71553ae
commit 3212b4a
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/src/main/java/picocli/CommandLine.java b/src/main/java/picocli/CommandLine.java
@@ -17840,7 +17840,18 @@ static final boolean forceEnabled() { return System.getenv("CLICOLOR_FORCE") !=
                     && !"0".equals(System.getenv("CLICOLOR_FORCE"));}
             /** http://stackoverflow.com/questions/1403772/how-can-i-check-if-a-java-programs-input-output-streams-are-connected-to-a-term */
             static boolean calcTTY() {
-                try { return System.class.getDeclaredMethod("console").invoke(null) != null; }
+                try {
+                    Object console = System.class.getDeclaredMethod("console").invoke(null);
+                    if (console == null) {
+                        return false;
+                    }
+                    try {
+                        Method isTerminal = Console.class.getDeclaredMethod("isTerminal");
+                        return (boolean) isTerminal.invoke(console);
+                    } catch (NoSuchMethodException e) {
+                        return true;
+                    }
+                }
                 catch (Throwable reflectionFailed) { return true; }
             }
             /** Cygwin and MSYS use pseudo-tty and console is always null... */