jsk-ros-pkg / jsk_aerial_robot

The platfrom for aerial robot (e.g. general multirotor, hydrus, di, dragon, etc)
http://www.jsk.t.u-tokyo.ac.jp/index-j.html
34 stars 34 forks source link

READMEのインストール手順の修正について #439

Closed chibi314 closed 3 years ago

chibi314 commented 3 years ago
  1. tongtybj/aerial_robotをJSKAerialRobot/aerial_robotになおす
  2. .rosinstallの中のURLを修正 をしないといけないですが、 2について、一部はJSKAerialRobotに移っていますが一部は移っていないという状態だと思います。 まだ移行作業中でしょうか?だとしたらそれが終わったらPRを出します。

@tongtybj

tongtybj commented 3 years ago

.rosinstallの中のURLを修正

は #434 で対応。 ただ、slackでも言ったよう。

最近travisでのビルド(主にrostest周り)がことごとくfailedしているので、片手間で直しているので、少し遅くなりました。
travisで通らないPRは原則絶対にマージしないようにしているので、他のPRも溜まっています。
頑張って今日中にtravisが正常稼働するように修正作業を完成させます。

あと、JSKAerialRobotに移すべきものは全部移したつもりで、#434 で移していないのはmocap_optitrack と realsense_rosで、いずれも、それぞれの本家からforkしたもので、JSKAerialRobotに移すべきではないと考えています。

chibi314 commented 3 years ago

了解です。

434でついでに1のURL修正についても行っていただけると助かります(1行のみ修正すれば大丈夫だと思います)。

tongtybj commented 3 years ago

434でついでに1のURL修正についても行っていただけると助かります(1行のみ修正すれば大丈夫だと思います)。

あれ、どこかミスってた? どの行か教えてくれる?

chibi314 commented 3 years ago
cd <catkin_ws>
wstool init src
wstool set -u -t src aerial_robot http://github.com/tongtybj/aerial_robot --git
wstool merge -t src src/aerial_robot/aerial_robot_${ROS_DISTRO}.rosinstall
wstool update -t src
rosdep install -y -r --from-paths src --ignore-src --rosdistro $ROS_DISTRO
catkin build

これの3行目です。 (#434では直されていないように見えるのですが他のコミットで直してたらすいません。)

tongtybj commented 3 years ago

教えてくれてありがとう!

tongtybj commented 3 years ago

solved by #434 .