clarify that the snapraid executable needs to be a file

Chronial · Apr 11, 2014 · 716d30b · 716d30b · Sam-Martin · Apr 25, 2015
1 parent 45e0365
commit 716d30b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/snapraid-runner.conf.example b/snapraid-runner.conf.example
@@ -1,6 +1,6 @@
 [snapraid]
-; path to the snapraid executable
-executable =
+; path to the snapraid executable (e.g. /bin/snapraid)
+executable = snapraid
 ; path to the snapraid config to be used
 config = snapraid.conf
 ; abort operation if there are more deletes than this, set to -1 to disable

diff --git a/snapraid-runner.py b/snapraid-runner.py
@@ -217,11 +217,12 @@ def run():
     logging.info("Run started")
     logging.info("=" * 60)
 
-    if not os.path.exists(config["snapraid"]["executable"]):
-        logging.error("Snapraid executable does not exist at " +
-                      config["snapraid"]["executable"])
+    if not os.path.isfile(config["snapraid"]["executable"]):
+        logging.error("The configured snapraid executable \"{}\" does not "
+                      "exist or is not a file".format(
+                          config["snapraid"]["executable"]))
         finish(False)
-    if not os.path.exists(config["snapraid"]["config"]):
+    if not os.path.isfile(config["snapraid"]["config"]):
         logging.error("Snapraid config does not exist at " +
                       config["snapraid"]["config"])
         finish(False)