Skip to content

Conversation

@Nobu19800
Copy link

@Nobu19800 Nobu19800 commented Jan 11, 2022

※2.0ブランチを作成するまではマージは保留にする。

Identify the Bug

Description of the Change

以下と同じ。

Verification

  • Did you succeed the build?
  • No warnings for the build?
  • Have you passed the unit tests?

@Nobu19800 Nobu19800 added the enhancement New feature or request label Jan 11, 2022
@Nobu19800 Nobu19800 requested a review from n-ando January 11, 2022 11:31
@Nobu19800 Nobu19800 self-assigned this Jan 11, 2022
@Nobu19800 Nobu19800 marked this pull request as draft January 19, 2022 05:45
n-ando
n-ando previously approved these changes Jan 25, 2022
@n-kawauchi n-kawauchi marked this pull request as ready for review September 30, 2025 04:26
@n-kawauchi
Copy link

下記修正を追加しました。

  • rtc.http.conf、rtc.https.conf、rtc.ws.conf、rtc.wss.conf で定義されている manager.modules.load_path や orba.http.certificate_authority_file のパスは、ソース定義では相対パスになっているが、debパッケージでインストールした場合は絶対パスにする
    • C++の場合はcmake時のオプションでソースからインストールする場合とdebパッケージでインストールする場合の切り替えが可能だがPythonソースの場合はこの仕組みが使えない
  • Pythonの場合は、debパッケージ向けconfファイルを新たなディレクトリ OpenRTM-aist-Python/packages/deb/conf で管理することにしたためこれに関する修正を行った
    • SSLで使用する証明書等はC++版でインストールされるものを使用することになったため、下記パスで定義している
      corba.http.certificate_authority_file:/usr/share/openrtm-2.1/ssl/root.crt
      corba.http.key_file:/usr/share/openrtm-2.1/ssl/server.pem
      
  • また、debパッケージビルド時のwhlファイル名小文字表記へは対応済みだったが、debパッケージに含めたいdist-infoディレクトリとtar.gzファイルが区別が出来ていなかった不具合に対応した

n-kawauchi
n-kawauchi previously approved these changes Oct 2, 2025
Copy link

@n-kawauchi n-kawauchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ubuntu24.04環境で、HTTP, HTTPS, WS, WSS, SLL 各動作OKを確認しました。
確認の詳細は下記Redmineに記載してあります。
https://openrtm.org/redmine/projects/openrtm-aist-cxx/wiki/ネームサーバー、マスターマネージャにIIOP以外でアクセスできるようにする手順#Python

@n-ando n-ando merged commit 9365481 into OpenRTM:master Oct 6, 2025
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants