search

fkanehiro / choreonoid_ros_pkg

0 stars 10 forks source link

依存パッケージ・ヘッダの修正 #96

Closed ssr-yuki closed 4 years ago

ssr-yuki commented 4 years ago

95 の問題に対応しました．

また，choreonoid_pluginsとchoreonoidヘッダの依存関係について修正を行い，一部実装を書き換えています．

各パッケージのpackage.xmlについても，最新フォーマットへの更新を行っています．

choreonoidバージョン1.7，開発版でビルド確認を行い，問題ありませんでした．

ssr-yuki commented 4 years ago

ビルド方法の変更が伴います． 問題があれば仰ってください．

fkanehiro commented 4 years ago

ビルド方法が変わるとのことですが、https://github.com/fkanehiro/choreonoid_ros_pkg/blob/master/README.md や https://github.com/fkanehiro/choreonoid_ros_pkg_doc の変更は不要でしょうか？

ssr-yuki commented 4 years ago

CmakeLists.txt 内での変更で，ビルドコマンド等に変更はないため，大丈夫かと思います．

ただ，現在chorenoid（最新版，v1.7のいずれも）が単体でのcatkin buildに対応している現状，choreonoid_rosパッケージ自体を削除するべきかもしれません．（参考：開発版マニュアル） この提案が受け入れられる場合，本PRに追加を行い，全て最新の望ましい依存関係に修復します．

fkanehiro commented 4 years ago

そうすると「ビルド方法の変更が伴います」とはどういう意味でしたでしょうか？

ssr-yuki commented 4 years ago

理由を存じ上げないのですが，変更箇所に関連してMac PCでのビルドに関する項目があり，ドキュメント上サポート外と考え，これを削除しました． 具体的には APPLE 変数の有無による条件分岐ビルドを削除しています．