Skip to content

ビルドスクリプトを修正#593

Open
ga-sakamoto wants to merge 4 commits intoOpenRTM:masterfrom
ga-sakamoto:Build_Script
Open

ビルドスクリプトを修正#593
ga-sakamoto wants to merge 4 commits intoOpenRTM:masterfrom
ga-sakamoto:Build_Script

Conversation

@ga-sakamoto
Copy link
Copy Markdown

Description of the Change

メールにてご連絡を頂きました内容を元に,再度,ビルドスクリプトを修正してみました.

Verification

  • Did you succesed the build? Windows上でEclipse2020-06を使用
  • No warnings for the build? Windows上でEclipse2020-06を使用
  • Have you passed the unit tests? サンプルコードの内容を基にユニットテストも作成

n-ando
n-ando previously approved these changes Apr 1, 2026
@n-kawauchi
Copy link
Copy Markdown

@ga-sakamoto

本PRを最新masterへマージしてビルド動作を確認したところ、前回出たエラーは解消されていますが別のところでエラーになります。

Copying created jar file into openrtp-2.1.0.v20260401...
- Starting ant build of project: org.iso.iso22166.part202.profile
target : buildAll libs:-lib ../lib -lib ../lib/a4e-2137 -lib ../lib/a4e-2137/libs -lib /home/vagrant/eclipse/plugins
Buildfile: build.xml does not exist!
Build failed
[ERROR] build failed:  org.iso.iso22166.part202.profile
Aborting...
[ERROR] build plugins failed. Aborting...

@ga-sakamoto
Copy link
Copy Markdown
Author

build.xmlは作成していなかったので,追加させて頂きました.
また,再度確認した所,色々とゴミが残ってしまっていたのと,ビルドスクリプトに余計なプロジェクトを追加してしまっていたので,こちらも修正させて頂きました.

@n-kawauchi
Copy link
Copy Markdown

@ga-sakamoto

修正して頂いてだいぶビルドは進んだようですが、それでもエラーになります。

Copying created jar file into openrtp-2.1.0.v20260401...
Checking plugin jar files...
[OK]    jp.go.aist.rtm.toolscommon.profiles_2.1.0.v20260401.jar found
[OK]    jp.go.aist.rtm.toolscommon.profiles.nl1_2.1.0.v20260401.jar found
[OK]    jp.go.aist.rtm.toolscommon_2.1.0.v20260401.jar found
[OK]    jp.go.aist.rtm.toolscommon.nl1_2.1.0.v20260401.jar found
[ERROR] org.iso.iso22166.part202.profile_2.1.0.v20260401.jar not found
Build failed? Some jar files are not found. Aborting...
[ERROR] build plugins failed. Aborting...

@ga-sakamoto
Copy link
Copy Markdown
Author

申し訳ございません.
こちらでも色々と確認してみたのですが,原因が良くわからない状況です.

エラーメッセージ的には,org.iso.iso22166.part202.profileのJARファイルが作成できていないようなのですが,ご連絡を頂きましたエラーメッセージより前に,何かエラー,警告は出ておりませんでしょうか?

ビルド関係のスクリプトは,こちらで作成したモノではないのと,そのまま実行できる環境がこちらにないので,ちょっと原因が掴めておりません.
(build.xml自体は,jp.go.aist.rtm.toolscommon.profilesの内容を参考に修正したものをコミットさせて頂きました)
申し訳ございません.

@n-kawauchi
Copy link
Copy Markdown

@ga-sakamoto

  • 現在確認しているソースのコミット履歴は下記となっています
$ git log --oneline -n 5
8cee3e2d (HEAD -> master, Build_Script) #592 Added build.xml
4d3ecc8a #592 Modified build scripts.
e117b408 (origin/master, origin/HEAD) Merge pull request #592 from ga-sakamoto/iso22166-202-profile
3f717198 #592 Modified build scripts.
d5311d47 #592 Deleted unnecessary Japanese comments.
  • このソースでのビルドログをファイルに落としたので添付します
    0401-src-build.log.txt
  • 前回、PR592のソースをmasterへマージせずに確認した時の環境が残っていましたのでログを確認してみました。
  • 3/14(土)に更新していただいたようで、私の方ではソースビルドが通って全部入りパッケージを生成できるところまで確認していました。
    eclipse416-openrtp202v20260314-ja-win32-x86_64.zip <==★バージョンが2.0.2で、3/14にビルドしたと分かります
  • このWindows用パッケージを展開してC++ RTCを生成し、Ubuntu16.04と18.04用のDockerfileが生成されることは、3/16(月)に確認しています
  • このソースビルドが通った時の更新ログを見ると、かなり古いです
$ cd ~/PR/0314-592/OpenRTP-aist
$ git branch
* iso22166-202-profile
  master
$ git log --oneline -n 5
12d53c7c (HEAD -> iso22166-202-profile) Updated RCP plugin version to 2.0.2.
44443299 Merge pull request #526 from ga-sakamoto/NameService_517
8b79ae11 Merge pull request #527 from ga-sakamoto/Java_CMake_309
caca3f57 Merge pull request #522 from ga-sakamoto/ConnectionChain_519
8a22eaf8 Modified CMakeLists.txt for Java RTC. #309
  • 改めて坂本さんのサイトからこのブランチをcloneしてみると、コミットログが全く違いますね
  • 私が確認した時のログは下記の「<--★ここ」です
  • OpenRTPの最新masterソースを使って坂本さんの環境でRTCを作成できるのでしょうか?
$ git clone https://github.com/ga-sakamoto/OpenRTP-aist 
$ cd OpenRTP-aist
$ git checkout iso22166-202-profile
$ git log --oneline
3f717198 (HEAD -> iso22166-202-profile, origin/iso22166-202-profile) #592 Modified build scripts.
d5311d47 #592 Deleted unnecessary Japanese comments.
7ab434ed Modified Help message
3dff10e6 Modified Hint Sections.
a1a507b1 Implement Dockerfile generation.
a06cd658 Add a settings screen for 'container creation'.
3ef42d9b Add a settings screen for 'container creation'.
e0006e59 Modified RTC-ISO mapping.
6450670e Modified ISO profile inport/export.
d8c2ece7 Profile convert
4ed933c6 Merge pull request #570 from ga-sakamoto/Proseccing_566_2
2c507b95 #566 After generating the code for the Processing RTC, the perspective switching is no longer checked.
b1a77c8b Merge pull request #568 from n-kawauchi/update_build_script
617ac357 Added Processing language setting to the build scripts.
...skipping...
12d53c7c Updated RCP plugin version to 2.0.2. <--★ここ
44443299 Merge pull request #526 from ga-sakamoto/NameService_517
8b79ae11 Merge pull request #527 from ga-sakamoto/Java_CMake_309
caca3f57 Merge pull request #522 from ga-sakamoto/ConnectionChain_519
8a22eaf8 Modified CMakeLists.txt for Java RTC. #309

@ga-sakamoto
Copy link
Copy Markdown
Author

ログ情報の取得,ご送付ありがとうございました.

内容を拝見させて頂いたのですが・・・最後以外は,特にエラーは出ていないようですね・・・
また,最後のエラーも
[ERROR] org.iso.iso22166.part202.profile_2.1.0.v20260401.jar not found
というメッセージなのですが,846行目辺りで,

jar:
     [copy] Copying 1 file to /home/vagrant/PR/593/OpenRTP-aist/org.iso.iso22166.part202.profile/bin
    [mkdir] Created dir: /home/vagrant/PR/593/OpenRTP-aist/org.iso.iso22166.part202.profile/jar
      [jar] Building jar: /home/vagrant/PR/593/OpenRTP-aist/org.iso.iso22166.part202.profile/jar/org.iso.iso22166.part202.profile_2.1.0.v20260401.jar

BUILD SUCCESSFUL

となっており,jarファイルの生成には成功しているようでした.

ただ,再度,jp.go.aist.rtm.toolscommon.profilesの内容を確認し,幾つか足りていなかったファイルや設定があったので,そちらを反映させたソースをコミットさせて頂きました.

本当に何度も申し訳ございませんが,こちらでお試し頂けませんでしょうか?

12d53c7c Updated RCP plugin version to 2.0.2. <--★ここ
こちらはだいぶ古いですね・・・日付を確認した所,2024/1/22にコミットされていました.

このWindows用パッケージを展開してC++ RTCを生成し、Ubuntu16.04と18.04用のDockerfileが生成されることは、3/16(月)に確認しています

申し訳ございません.こちらは完全に説明が不足しておりました.

上記のお話は,恐らく,対象プロジェクトの「scripts」内に「ubuntu_1604」「ubuntu_1804」のディレクトリができていて,その中にDockerfileが存在している,というお話かと思います.
こちらは結構前からある機能となります.
(なので,ubuntu_1604も対象となっている)

今回の修正では,「scripts」ディレクトリの直下に
ModuleName222__Ubuntu-20.04__ROS1-noetic__cpp_Min.Dockerfile
のようなファイル名のDockerfileが生成されるようになります.
(実際のファイル名は,「コンテナ生成」タブで入力した内容に依存します)

このため,3月中旬にご確認頂いた時点で,上手くビルドできていなかったのではないかと思います.
(対象コミットを考えますと,恐らく「コンテナ生成」タブも表示されないのではないかと思います
「コンテナ生成」タブは以下のような画面です)
image

OpenRTPの最新masterソースを使って坂本さんの環境でRTCを作成できるのでしょうか?

はい,こちらは確認しております.

@n-kawauchi
Copy link
Copy Markdown

修正ソースをmasterへマージしてソースビルドしましたが、前回とエラー内容は変わりませんでした。
ビルドログで下記以外にエラーは出ていないのも前回と同じです。

Copying created jar file into openrtp-2.1.0.v20260402...
Checking plugin jar files...
[OK]    jp.go.aist.rtm.toolscommon.profiles_2.1.0.v20260402.jar found
[OK]    jp.go.aist.rtm.toolscommon.profiles.nl1_2.1.0.v20260402.jar found
[OK]    jp.go.aist.rtm.toolscommon_2.1.0.v20260402.jar found
[OK]    jp.go.aist.rtm.toolscommon.nl1_2.1.0.v20260402.jar found
[ERROR] org.iso.iso22166.part202.profile_2.1.0.v20260402.jar not found
Build failed? Some jar files are not found. Aborting...
[ERROR] build plugins failed. Aborting...

コミット履歴は下記です。

$ git log --oneline -n 5
7beb551c (HEAD -> master, Build_Script) #592 Modified build.properties.
8cee3e2d #592 Added build.xml
4d3ecc8a #592 Modified build scripts.
e117b408 (origin/master, origin/HEAD) Merge pull request #592 from ga-sakamoto/iso22166-202-profile
3f717198 #592 Modified build scripts.

@ga-sakamoto
Copy link
Copy Markdown
Author

早速のご確認,ご連絡ありがとうございます.
昨日お送り頂きましたログの内容などを元に,再度,確認・検討してみます.
少しお時間を頂ければ幸いです.
ご面倒,ご迷惑をお掛けしてしまっており,たいへん申し訳ございません.

@ga-sakamoto
Copy link
Copy Markdown
Author

ga-sakamoto commented Apr 5, 2026

ログファイルを再度確認した所,jp.go.aist.rtm.toolscommon内でエラーが発生しているようでしたので,こちらのビルドスクリプトを修正してみました.
本当に何度も申し訳ございませんが,こちらでお試し頂けませんでしょうか?

また,ビルドに失敗した場合には,ログファイルを頂けませんでしょうか?

@n-kawauchi
Copy link
Copy Markdown

やはりビルドに失敗します。使用したソースの修正コミットは下記となっています。

$ git log --oneline -n 5
0f948f37 (HEAD -> master, Build_Script) #592 Modified build.xml in jp.go.aist.rtm.toolscommon.
7beb551c #592 Modified build.properties.
8cee3e2d #592 Added build.xml
4d3ecc8a #592 Modified build scripts.
e117b408 (origin/master, origin/HEAD) Merge pull request #592 from ga-sakamoto/iso22166-202-profile

ビルドログを添付します。
PR593.log.txt

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.

3 participants