Skip to content

Conversation

@n-kawauchi
Copy link

close #1199

Identify the Bug

Link to #1199

Description of the Change

  • (1)Policy CMP0144への対応
    • find_packageで指定しているパッケージ名が全て大文字になっていないという指摘
    • 指摘されている部分は、find_package(OpenSSL REQUIRED) だが、現時点でFindOpenSSL.cmakeが大文字のパッケージ名OPENSSLに対応していないため、cmake_policyでこのワーニングを抑制した
      if(POLICY CMP0144)
        cmake_policy(SET CMP0144 NEW)
      endif()
      
  • (2)Policy CMP0177への対応
    • install()のDESTINATION の文字列が正規化前後で変わる可能性(例: 末尾の /、//、./、../ が混入)があるとワーニングが出る
    • ワーニングが出ない書き方に変更した
      install(PROGRAMS ${OMNIORB_DLLS} DESTINATION ${ORB_INSTALL_DIR}/bin/${ARCH_NAME} COMPONENT corbalib)
            ↓
      cmake_path(SET _dest NORMALIZE "${ORB_INSTALL_DIR}/bin/${ARCH_NAME}")
      install(PROGRAMS ${OMNIORB_DLLS} DESTINATION "${_dest}" COMPONENT corbalib)
      
  • (3)「Compatibility with CMake < 3.10 will be removed from a future version of CMake.」への対応
    • cmake_minimum_required(VERSION 3.5.1) の部分を指摘されている
    • RTCBのテンプレートと同様の下記定義に変更した
      cmake_minimum_required(VERSION 3.16)
      

Verification

  • Windows環境でワーニングが出ないことを確認
  • Windows環境でソースからインストールしたディレクトリで、install文で指定しているファイルの存在を確認
  • Did you succeed the build?
  • No warnings for the build?
  • Have you passed the unit tests?

@n-kawauchi n-kawauchi requested a review from n-ando October 10, 2025 03:03
@n-kawauchi n-kawauchi self-assigned this Oct 10, 2025
@n-ando n-ando merged commit fe6cc11 into OpenRTM:master Oct 14, 2025
3 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CMakeのワーニング対応

2 participants