Improved error logging (#284)

src/cli/cli_handler.cc

@@ -40,16 +40,7 @@ void cli_handler::setup_logging()
 {
     spdlog::drop("clanguml-logger");
 
-    if (!progress) {
-        spdlog::register_logger(logger_);
-    }
-    else {
-        // Setup null logger for clean progress indicators
-        std::vector<spdlog::sink_ptr> sinks;
-        logger_ = std::make_shared<spdlog::logger>(
-            "clanguml-logger", begin(sinks), end(sinks));
-        spdlog::register_logger(logger_);
-    }
+    spdlog::register_logger(logger_);
 
     logger_->set_pattern("[%^%l%^] [tid %t] %v");
 
@@ -196,6 +187,16 @@ cli_flow_t cli_handler::handle_options(int argc, const char **argv)
 
     config.inherit();
 
+    if (progress) {
+        spdlog::drop("clanguml-logger");
+
+        // Setup null logger for clean progress indicators
+        std::vector<spdlog::sink_ptr> sinks;
+        logger_ = std::make_shared<spdlog::logger>(
+            "clanguml-logger", begin(sinks), end(sinks));
+        spdlog::register_logger(logger_);
+    }
+
     return res;
 }
 

src/util/util.cc

@@ -35,7 +35,7 @@ public:
         : result_{result}
         ,
 #if defined(__linux) || defined(__unix) || defined(__APPLE__)
-        pipe_{popen(command.c_str(), "r")}
+        pipe_{popen(fmt::format("{} 2>&1", command).c_str(), "r")}
 #elif defined(_WIN32)
         pipe_{_popen(command.c_str(), "r")}
 #endif