fix(ament_cmake_python): respect DESTDIR during compilation

AntoninRousset · AntoninRousset · commit 309c54359232 · 2023-05-19T11:57:54.000+02:00
diff --git a/ament_cmake_python/cmake/ament_python_install_module.cmake b/ament_cmake_python/cmake/ament_python_install_module.cmake
@@ -65,7 +65,7 @@ function(_ament_cmake_python_install_module module_file)
       "execute_process(
         COMMAND
         \"${python_interpreter}\" \"-m\" \"compileall\"
-        \"${CMAKE_INSTALL_PREFIX}/${destination}/${module_file}\"
+        \"\$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}/${destination}/${module_file}\"
       )"
     )
   endif()
diff --git a/ament_cmake_python/cmake/ament_python_install_package.cmake b/ament_cmake_python/cmake/ament_python_install_package.cmake
@@ -192,7 +192,7 @@ setup(
       "execute_process(
         COMMAND
         \"${python_interpreter_config}\" \"-m\" \"compileall\"
-        \"${CMAKE_INSTALL_PREFIX}/${ARG_DESTINATION}/${package_name}\"
+        \"\$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}/${ARG_DESTINATION}/${package_name}\"
       )"
     )
   endif()

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ function(_ament_cmake_python_install_module module_file)`
`65`	`65`	`"execute_process(`
`66`	`66`	`COMMAND`
`67`	`67`	`\"${python_interpreter}\" \"-m\" \"compileall\"`
`68`		`- \"${CMAKE_INSTALL_PREFIX}/${destination}/${module_file}\"`
	`68`	`+ \"\$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}/${destination}/${module_file}\"`
`69`	`69`	`)"`
`70`	`70`	`)`
`71`	`71`	`endif()`
Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,7 @@ setup(`
`192`	`192`	`"execute_process(`
`193`	`193`	`COMMAND`
`194`	`194`	`\"${python_interpreter_config}\" \"-m\" \"compileall\"`
`195`		`- \"${CMAKE_INSTALL_PREFIX}/${ARG_DESTINATION}/${package_name}\"`
	`195`	`+ \"\$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}/${ARG_DESTINATION}/${package_name}\"`
`196`	`196`	`)"`
`197`	`197`	`)`
`198`	`198`	`endif()`