From 745e57d7cddab4edce7cb43f4c1258ba25615099 Mon Sep 17 00:00:00 2001 From: Alex Bilger Date: Mon, 2 Mar 2026 12:18:54 +0100 Subject: [PATCH] improve error message and clean the paths --- tools/RegressionSceneList.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/RegressionSceneList.py b/tools/RegressionSceneList.py index f2fa7b4..9a355d1 100644 --- a/tools/RegressionSceneList.py +++ b/tools/RegressionSceneList.py @@ -56,7 +56,7 @@ def process_file(self): if ("REGRESSION_DIR" in os.environ): self.ref_dir_path = values[0].replace("$REGRESSION_DIR", os.environ["REGRESSION_DIR"]) else: - helper.writeError(f"While processing $REGRESSION_DIR: Environment variable REGRESSION_DIR is not set.") + helper.writeError(f"The environment variable $REGRESSION_DIR is required in {self.file_path} but not set. Please set this variable to the root directory of your regression tests to proceed.") return else: # direct absolute or relative path self.ref_dir_path = os.path.join(self.file_dir, values[0]) @@ -81,8 +81,8 @@ def process_file(self): helper.writeLog(f'Filtered out {self.filter}: {values[0]}') continue - full_file_path = os.path.join(self.file_dir, values[0]) - full_ref_file_path = os.path.join(self.ref_dir_path, values[0]) + full_file_path = os.path.normpath(os.path.join(self.file_dir, values[0])) + full_ref_file_path = os.path.normpath(os.path.join(self.ref_dir_path, values[0])) meca_in_mapping = False if values[3] == '1': # converting string to Bool always gives True