From 23f70677aa29b0738debecda9ddd89736941298b Mon Sep 17 00:00:00 2001
From: Howard Butler <howard@hobu.co>
Date: Wed, 28 Aug 2019 16:13:59 -0500
Subject: [PATCH] log portable_popen

---
 pdal/util/Utils.cpp                      | 3 +++
 plugins/python/test/PythonFilterTest.cpp | 1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/pdal/util/Utils.cpp b/pdal/util/Utils.cpp
index 89ef183054..121e0dfcea 100644
--- a/pdal/util/Utils.cpp
+++ b/pdal/util/Utils.cpp
@@ -384,7 +384,10 @@ int Utils::run_shell_command(const std::string& cmd, std::string& output)
     FILE* fp = portable_popen(cmd.c_str(), "r");
 
     if (fp == NULL)
+    {
+        std::cerr << "portable_popen failed for '" << cmd.c_str() << "'"<< std::endl;
         return 1;
+    }
 
     while (!feof(fp))
     {
diff --git a/plugins/python/test/PythonFilterTest.cpp b/plugins/python/test/PythonFilterTest.cpp
index d9ad690093..de05e378d3 100644
--- a/plugins/python/test/PythonFilterTest.cpp
+++ b/plugins/python/test/PythonFilterTest.cpp
@@ -1077,7 +1077,6 @@ static void run_pipeline(std::string const& pipeline)
     if (stat)
     {
         std::cerr << "error message in run_pipeline: " << output << std::endl;
-        std::cout << "error message in run_pipeline: " << output << std::endl;
     }
 }