From c73d215cc026c92403e7ea8aa4e68f70c9c72259 Mon Sep 17 00:00:00 2001
From: ed <eddavies95@gmail.com>
Date: Tue, 28 Jul 2020 11:16:11 +0100
Subject: [PATCH] Linux:  Use return code of `which` to determine whether
 executable is available instead of parsing output

---
 modules/juce_gui_basics/native/juce_linux_FileChooser.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/modules/juce_gui_basics/native/juce_linux_FileChooser.cpp b/modules/juce_gui_basics/native/juce_linux_FileChooser.cpp
index 9e8251546c..5642bd3df0 100644
--- a/modules/juce_gui_basics/native/juce_linux_FileChooser.cpp
+++ b/modules/juce_gui_basics/native/juce_linux_FileChooser.cpp
@@ -32,16 +32,13 @@ static bool exeIsAvailable (String executable)
 
     if (child.start ("which " + executable))
     {
-        auto output = child.readAllProcessOutput().trim();
         child.waitForProcessToFinish (60 * 1000);
-
-        return output.isNotEmpty() && ! output.contains ("no " + executable);
+        return (child.getExitCode() == 0);
     }
 
     return false;
 }
 
-
 class FileChooser::Native    : public FileChooser::Pimpl,
                                private Timer
 {