Project

General

Profile

整備 #4132

インストールされているOpenRTM-aistのバージョン番号取得処理を見直す

Added by n.kawauchi over 4 years ago. Updated about 4 years ago.

Status:
終了
Priority:
通常
Assignee:
Start date:
07/04/2017
Due date:
% Done:

100%

Estimated time:

Description

  • プロジェクト直下のCMakeLists.txtの修正(C++, Python)
  • 取得したバージョン番号は、インストーラパッケージのインストール先パスの設定で使用される
  • Windows用の取得処理は定義済みだがLinux用は未定義だったので整備する
  • バージョン番号はOpenRTMConfig.cmakeから取得するように変更する
  • 修正内容は次の通り
    if(WIN32)
        string(REPLACE "\\" "/" OPENRTM_DIR "$ENV{RTM_ROOT}")
        string(LENGTH ${OPENRTM_DIR} RTM_ROOT_LEN)
        string(REPLACE "\\" "/" RTM_BASE "$ENV{RTM_BASE}")
        string(LENGTH ${RTM_BASE} RTM_BASE_LEN)
        string(SUBSTRING ${OPENRTM_DIR} ${RTM_BASE_LEN} ${RTM_ROOT_LEN} RTM_VER)
        string(REGEX REPLACE "/$" "" RTM_VER "${RTM_VER}")
        set(RTM_SHORT_VER ${RTM_VER})
        string(REPLACE "." "" RTM_SHORT_VER "${RTM_SHORT_VER}")
    endif()
         ↓
    find_package(OpenRTM)
    set(RTM_VER ${OPENRTM_VERSION})
    set(RTM_SHORT_VER ${OPENRTM_VERSION_MAJOR}${OPENRTM_VERSION_MINOR}${OPENRTM_VERSION_PATCH})
    -----
    set(OPENRTM_SHARE_PREFIX "share/openrtm-${PROJECT_VERSION_MAJOR}.${PROJECT_VERSION_MINOR}")
         ↓
    set(OPENRTM_SHARE_PREFIX "share/openrtm-${OPENRTM_VERSION_MAJOR}.${OPENRTM_VERSION_MINOR}")
    

Associated revisions

Revision 744 (diff)
Added by ga over 4 years ago

Modified CMakeLists.txt #4132

Revision 745 (diff)
Added by ga over 4 years ago

Modified CMakeLists.txt #4132

Revision 756 (diff)
Added by ga about 4 years ago

Modified CMakeLists.txt #4132

History

#1 Updated by ga over 4 years ago

  • Status changed from 新規 to 解決
  • % Done changed from 0 to 100

修正させて頂きました r744,745

#2 Updated by n.kawauchi over 4 years ago

  • Status changed from 解決 to フィードバック
  • % Done changed from 100 to 90

ありがとうございました。
修正して頂きましたが、Linux環境でのPython RTCはOpenRTMConfig.cmakeを見つけられないことが分かりました。

原因は今回の修正ではなく、projectの第2引数で指定している「NONE」でした。C++の場合はこの指定はありません。
Pythonの場合もNONEを外して下さい。

このことに関する動作確認の詳細は、CMakeのprojectの第2引数についての調査 をご覧ください。
よろしくお願いいたします。

#3 Updated by ga about 4 years ago

  • Status changed from フィードバック to 解決
  • % Done changed from 90 to 100

修正させて頂きました r756

#4 Updated by n.kawauchi about 4 years ago

確認しました。ありがとうございました。

#5 Updated by n-ando about 4 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF