KPMP-5665: make sure it's a file

KPMP · Dec 17, 2024 · 11310fa · 11310fa
1 parent 1571425
commit 11310fa
Showing 1 changed file with 15 additions and 12 deletions.
diff --git a/scripts/packageChecker/packageChecker.py b/scripts/packageChecker/packageChecker.py
@@ -140,18 +140,21 @@ def find_empty_packages(self, move_derived=False):
     def move_file_to_derived(self, package_directory, file_name):
         derived_dir = package_directory + "/derived"
         file_path = package_directory + "/" + file_name
-        print(f"Moving file '{file_path}' to '{derived_dir}'.")
-
-        # if not os.path.exists(derived_dir):
-        #     os.makedirs(derived_dir)
-        #
-        # try:
-        #     shutil.move(file_path, derived_dir)
-        #     print(f"File '{file_path}' moved to '{derived_dir}' successfully.")
-        # except FileNotFoundError:
-        #     print(f"Error: File '{source_file}' not found.")
-        # except Exception as e:
-        #     print(f"An error occurred: {e}")
+        if os.path.isfile(file_path):
+            print(f"Moving file '{file_path}' to '{derived_dir}'.")
+            # if not os.path.exists(derived_dir):
+            #     os.makedirs(derived_dir)
+            #
+            # try:
+            #     shutil.move(file_path, derived_dir)
+            #     print(f"File '{file_path}' moved to '{derived_dir}' successfully.")
+            # except FileNotFoundError:
+            #     print(f"Error: File '{source_file}' not found.")
+            # except Exception as e:
+            #     print(f"An error occurred: {e}")
+        else:
+            print(f"'{file_path}' is not a file. Skipping.")
+
 
 
 if __name__ == "__main__":