Do not require PYTHON_VERSION to be defined in .bat files (#4770)

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
eProsima · May 17, 2024 · 0536165 · 0536165
1 parent fbb7877
commit 0536165
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 14 deletions.
diff --git a/tools/fastdds/fastdds.bat b/tools/fastdds/fastdds.bat
@@ -22,12 +22,13 @@ if not %ERRORLEVEL%==0 (
 )
 
 :: Python version in the form "Major.Minor"
-if %PYTHON_VERSION%=="" (
-      echo error retrieving python version. Please, make sure python is installed and accessible.
-      exit /B 65
-   )
+if not "%PYTHON_VERSION%" == "" (
+      :: Use launcher to profit from shebang hints on fastdds.py
+      :: Select the correct python version to source the appropriate paths
+      py -%PYTHON_VERSION% "%dir%\..\tools\fastdds\fastdds.py" %args%
+) else (
+      :: Use launcher to profit from shebang hints on fastdds.py
+      :: Select latest available python version
+      py "%dir%\..\tools\fastdds\fastdds.py" %args%
 )
 
-:: Use launcher to profit from shebang hints on fastdds.py
-:: Select the correct python version to source the appropriate paths
-py -%PYTHON_VERSION% "%dir%\..\tools\fastdds\fastdds.py" %args%
diff --git a/tools/fastdds/ros-discovery.bat b/tools/fastdds/ros-discovery.bat
@@ -22,12 +22,13 @@ if not %ERRORLEVEL%==0 (
 )
 
 :: Python version in the form "Major.Minor"
-if %PYTHON_VERSION%=="" (
-      echo error retrieving python version. Please, make sure python is installed and accessible.
-      exit /B 65
-   )
+if not "%PYTHON_VERSION%" == "" (
+      :: Use launcher to profit from shebang hints on fastdds.py
+      :: Select the correct python version to source the appropriate paths
+      py -%PYTHON_VERSION% "%dir%\..\tools\fastdds\fastdds.py" discovery %args%
+) else (
+	  :: Use launcher to profit from shebang hints on fastdds.py
+      :: Select latest available python version
+      py "%dir%\..\tools\fastdds\fastdds.py" discovery %args%
 )
 
-:: Use launcher to profit from shebang hints on fastdds.py
-:: Select the correct python version to source the appropriate paths
-py -%PYTHON_VERSION% "%dir%\..\tools\fastdds\fastdds.py" discovery %args%