[RTCB] Python RTCのcmake policy設定を修正 - Githubissues

OpenRTM / OpenRTP-aist

OpenRTP-aist: RT-Middleware and OMG RTC based component and system development tools implemented by AIST

Other

1 stars 6 forks source link

[RTCB] Python RTCのcmake policy設定を修正 #524

Closed ga-sakamoto closed 7 months ago

ga-sakamoto commented 8 months ago

Identify the Bug

Link to #389

Description of the Change

ご連絡を頂きました内容で，cmake policyの部分を修正させて頂きました．

Verification

[x] Did you succesed the build? Windows上でEclipse2020-06を使用
[x] No warnings for the build? Windows上でEclipse2020-06を使用
[x] Have you passed the unit tests? ユニットテストを修正

n-kawauchi commented 8 months ago

修正して頂いたRTCBでPython RTCを生成しcmakeを実行するとエラーになります。cmake3.23.2で確認。言語だけPythonを指定しただけであとはデフォルトのままコードを生成。

CMake Error at CMakeLists.txt:105 (add_custom_command):
  No TARGET 'ModuleName' has been created in this directory.

「cmake_minimum_required(VERSION 2.8)」ではワーニングとして表示されていたがエラーにはなりませんでした。3.5.1としたことでエラーになったようです。

ImageProcessingのTkCalibGUIはPython RTCで、「cmake_minimum_required(VERSION 3.5.1)」で定義していて、 cmake Policyも定義していないが、ワーニングすら出ずcmakeが通っています。 https://github.com/OpenRTM/ImageProcessing/blob/master/opencv/components/TkCalibGUI/CMakeLists.txt

修正方法を検討してから改めてご連絡します。

n-kawauchi commented 7 months ago

（修正１）一番トップのCMakeLists.txtの106行目付近にある下記コード6行を削除してください。
```
foreach(target ${EXEC_FILES})
add_custom_command( TARGET ${PROJECT_NAME} POST_BUILD
  COMMAND ${CMAKE_COMMAND} -E copy
      ${PROJECT_SOURCE_DIR}/${target} ${PROJECT_BINARY_DIR}
)
endforeach()
```
- 削除理由
- この処理は参考にしたImageProcessingに含まれるTkCalibGUIで定義しているものです
- ImageProcessingの場合、トップから全てのRTCの一括ビルドを行っている関係で、インストール対象のファイルを${PROJECT_BINARY_DIR}へコピーする仕組みを入れています
- 単独Python RTCの場合はこの処理は不要ですし、cpackでmsiやdebを生成する場合でもこの設定は使っていないので削除して問題ありません
（修正２）find_package(PythonInterp 3)行の修正
```
find_package(PythonInterp 3)
↓
find_package(Python 3)
```
- 修正理由
- PythonInterpはcmake3.12から非推奨となり、cmake 3.27から赤字でワーニングが出るようになったためです
- 次期バージョンのOpenRTM2.0.2は、Ubuntu20.04以降がサポート対象で、Ubuntu20.04のデフォルトのcmakeバージョンは3.16.3のため、変更しても問題ありません
- Ubuntu18.04はcmake3.10.2のため変更するとcmakeが通らなくなりますが、OpenRTM2.0.2版ではサポート外のため問題ないとの判断です

ga-sakamoto commented 7 months ago

ご連絡頂きました内容を，再度，修正させて頂きました．