explicitly expect a directory for logging

Verify the logging parameter is explicitly a directory.  Treat the path
as a directory for creating the logging files.

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I84d6be79780df3f52deeb1ff1b7a50de6175c055

2 files changed, 3 insertions, 2 deletions

diff --git a/main.cpp b/main.cpp
index 3726f50..8c99154 100644
--- a/main.cpp
+++ b/main.cpp
@@ -72,7 +72,8 @@ int main(int argc, char* argv[])
                    "Optional parameter to specify configuration at run-time")
         ->check(CLI::ExistingFile);
     app.add_option("-l,--log", loggingPath,
-                   "Optional parameter to specify logging path");
+                   "Optional parameter to specify logging folder")
+        ->check(CLI::ExistingDirectory);
     app.add_flag("-t,--tuning", tuningEnabled, "Enable or disable tuning");
 
     loggingEnabled = (!loggingPath.empty());
diff --git a/pid/zone.hpp b/pid/zone.hpp
index fcadac0..53139ec 100644
--- a/pid/zone.hpp
+++ b/pid/zone.hpp
@@ -54,7 +54,7 @@ class PIDZone : public ZoneInterface, public ModeObject
     {
         if (loggingEnabled)
         {
-            _log.open(loggingPath + std::to_string(zone));
+            _log.open(loggingPath + "/zone_" + std::to_string(zone) + ".log");
         }
     }