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

) Signed-off-by: Mario Dominguez <[email protected]> (cherry picked from commit 39e8957) Co-authored-by: Mario Domínguez López <[email protected]>
eProsima · May 24, 2024 · db64268 · db64268
1 parent db3d420
commit db64268
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%