[on #249 of start-jsk/rtmros_gazebo]Add melodic test

[Naoki-Hiraoka]

This PR includes #567, #570.

[Naoki-Hiraoka]

• nextage_tutorials depends on rtmros_nextage and rtmros_hironx, but it is not released in melodic
• hrpsys_gazebo_general needs hrpsys/src, but hrpsys/src is not installed with released hrpsys on melodic. (It is not problem in travis because travis uses ros-testing. But it is problem for normal users)

[Naoki-Hiraoka]

travis passed

[pazeshun]

Awesome!
@k-okada Could you review this?

[pazeshun]

@k-okada
@Naoki-Hiraoka wants to keep hrpsys, rtmros_common, and rtmros_hironx until they are relased to http://packages.ros.org/ros/ubuntu (not ros-testing) in .travis.rosinstall:
#571 (comment)
because we recommend users without rtmros_hrp2 to use .travis.rosinstall:
https://github.com/start-jsk/rtmros_tutorials#install
https://github.com/start-jsk/rtmros_common#install
This is because we don't want to manage many rosintall files.

[k-okada]

please confirm that we use ros-testing for jsk_travis and rtmros_hironx is already released into melodic (https://github.com/ros/rosdistro/search?q=rtmros&type=Commits)

++ sudo apt-get update -q
Get:1 http://packages.ros.org/ros-testing/ubuntu bionic InRelease [4669 B]
Get:2 http://packages.ros.org/ros-testing/ubuntu bionic/main amd64 Packages [670 kB]
Hit:3 http://security.ubuntu.com/ubuntu bionic-security InRelease
Hit:4 http://archive.ubuntu.com/ubuntu bionic InRelease
Hit:5 http://archive.ubuntu.com/ubuntu bionic-updates InRelease
Hit:6 http://archive.ubuntu.com/ubuntu bionic-backports InRelease
Fetched 675 kB in 1s (500 kB/s)

[pazeshun]

すみません、言いたかったのはjsk_travisの話ではなく、普通のユーザーはros-testing使ってなくて、そのユーザーがrtmros_tutorialsをビルド・インストールしようとした場合に、.travis.rosinstallをworkspaceの構築に使ってもらうような形にREADMEを書いているので、ros-testingでしかリリースされていないパッケージの存在を前提として.travis.rosinstallを変えてしまうと、一時的にREADMEに従うとうまくビルドできないことになるな、ということでした。
ただ、あくまで一時的なものであるということと、rtmros_hrp2を使う人たちはそっちのrosinstallを使うので、問題ある人たちは限られるから大丈夫、ということにはなるかもしれません。

これが嫌なので新しくrosinstallを作る、ってなると、ほぼ同じファイルを二つ管理しないといけなくなって管理漏れしそうなので、統一してtravisの方で使っておく、というのはなるべく維持しておきたいです。

[k-okada]

なるほど．それが気になるときは - ROS_DISTRO=melodic USE_DEB=false ..... ROS_REPOSITORY_PATH="http://packages.ros.org/ros/ubuntu" をついかしておきましょう．まぁいいや，であれば，このままでいいと思います．

…

-- ◉ Kei Okada 2020年5月15日(金) 22:11 Shun Hasegawa <[email protected]>:

すみません、言いたかったのはjsk_travisの話ではなく、普通のユーザーはros-testing使ってなくて、そのユーザーがrtmros_tutorialsをビルド・インストールしようとした場合に、.travis.rosinstallをworkspaceの構築に使ってもらうような形にREADMEを書いているので、ros-testingでしかリリースされていないパッケージの存在を前提として.travis.rosinstallを変えてしまうと、一時的にREADMEに従うとうまくビルドできないことになるな、ということでした。 ただ、あくまで一時的なものであるということと、rtmros_hrp2を使う人たちはそっちのrosinstallを使うので、問題ある人たちは限られるから大丈夫、ということにはなるかもしれません。 これが嫌なので新しくrosinstallを作る、ってなると、ほぼ同じファイルを二つ管理しないといけなくなって管理漏れしそうなので、統一してtravisの方で使っておく、というのは維持しておきたいです。 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[pazeshun]

なるほど．それが気になるときは - ROS_DISTRO=melodic USE_DEB=false ..... ROS_REPOSITORY_PATH="http://packages.ros.org/ros/ubuntu" をついかしておきましょう．

おっしゃるとおりです、すみませんでした。

まぁいいや，であれば，このままでいいと思います．

これは、

1. この状態のままhttp://packages.ros.org/ros/ubuntu にreleaseされるまでmergeを待つ
2. @Naoki-Hiraoka のcommitまで戻してROS_REPOSITORY_PATH="http://packages.ros.org/ros/ubuntu"を入れる
3. この状態のままmergeして、ソースで入れたい人向けに別の方法を考える(？)

のどちらを意図されたものでしょうか。

[k-okada]

3です．皆がコレに気がつく頃にはros/ubuntu にリリースされるので

[pazeshun]

ありがとうございます