[RTCB] idl/CMakeLists.txtがLinuxのPython3環境に対応していない

n-kawauchi commented 3 years ago

Describe the bug

独自IDLを持つRTC(C++)の生成時、UbuntuでPython3を利用している場合、IDLコンパイルでエラーが発生する。

原因は、idl/CMakeLists.txt の25行目のpythonコマンド指定。Linuxの場合はpython3とする必要がある。

22     _IDL_OUTPUTS(${_idl} ${CMAKE_CURRENT_BINARY_DIR} ${_idl_srcs_var})
23 
24     add_custom_command(OUTPUT ${${_idl_srcs_var}}
25         COMMAND python ${OPENRTM_DIR}/bin/${_rtm_skelwrapper_command} --include-dir= --skel-suffix=Skel --stu    b-suffix=Stub --idl-file=${_idl}.idl

To Reproduce

ImageProcessing/ImageCalibration RTCを Ubuntu環境で openrtp1.2.2-1 を使って再生成した際にエラーを確認した

$ make
[ 10%] Compiling /home/kawauchi/workspace2/ImageCalibration/idl/CalibrationService.idl
/bin/sh: 1: python: not found
make[2]: *** [idl/CMakeFiles/CalibrationService_TGT.dir/build.make:62: idl/CalibrationServiceSkel.cpp] エラー 127

Environment

OS: Ubnutn 20.04 64bits, Python3.8
OpenRTM-aist Version 1.2.2
OpenRTP-aist Version 1.2.2-1

Additional context 修正方法ですが、Windows環境はPython3であっても使用コマンドはpythonであることから、以下のように指定するのはいかがでしょうか。

 22     _IDL_OUTPUTS(${_idl} ${CMAKE_CURRENT_BINARY_DIR} ${_idl_srcs_var})
 23 
 24     if(WIN32)
 25         set(_python_command "python")
 26     else(WIN32)
 27         set(_python_command "python3")
 28     endif(WIN32)
 29 
 30     add_custom_command(OUTPUT ${${_idl_srcs_var}}
 31         COMMAND ${_python_command} ${OPENRTM_DIR}/bin/${_rtm_skelwrapper_command} --include-dir= --skel-suffi    x=Skel --stub-suffix=Stub --idl-file=${_idl}.idl

n-kawauchi commented 3 years ago

@ga-sakamoto さん、masterブランチへ上記に示した修正を入れて下さい。よろしくお願いします。

ga-sakamoto commented 3 years ago

ご連絡が非常に遅くなってしまい，たいへん申し訳ございません．ご提示頂いた内容で修正したPRを上げさせて頂きました．ご確認のほど，よろしくお願いいたします．

OpenRTM / OpenRTP-aist

[RTCB] idl/CMakeLists.txtがLinuxのPython3環境に対応していない #387