Merge pull request #20 from zkhan12/master

change logic towards writing to log file when replace is no

Author: jdkent

accel_code/run.py

@@ -95,13 +95,14 @@ def main():
 
     # copy the file to the new project specific location
     try:
-        utils.make_directory(opts.old_file_path, new_file_path, opts.replace)
+        file_copied = utils.make_directory(opts.old_file_path, new_file_path, opts.replace)
     except Exception as e:
         msg = "could not copy file: {} ({})"
         logger.exception(msg.format(lab_id, str(date)))
         raise(e)
-
-    logger.info("{of} -> {nf}".format(of=opts.old_file_path, nf=new_file_path))
+    
+    if file_copied:
+        logger.info("{of} -> {nf}".format(of=opts.old_file_path, nf=new_file_path))
 
     return
 

accel_code/utils.py

@@ -159,10 +159,14 @@ def get_test_data_path(project):
 def make_directory(old_path, new_path, replace):
     if os.path.exists(new_path):
         if replace == 'no':
-            raise ValueError('replace option was not specified and output file exists', new_path)
+            print('replace option was set to no and output file exists', new_path)
+            return False
+
         if replace == 'yes':
             os.remove(new_path)
             copyfile(old_path, new_path)
+            return True
     else:
         os.makedirs(os.path.dirname(new_path), exist_ok=True)
         copyfile(old_path, new_path)
+        return True