search

OpenRTM / OpenRTM-aist

OpenRTM-aist: RT-Middleware and OMG RTC implementation in C++ implemented by AIST
https://openrtm.org/
Other
19 stars 12 forks source link

2.0のUbuntu18.04用debパッケージ作成設定を整える #777

Closed n-kawauchi closed 4 years ago

n-kawauchi commented 4 years ago

Is your feature request related to a problem? Please describe.

r-kurose commented 4 years ago

メモ： hrtm は不要 サンプルコンポーネントの静的ライブラリ(.a)は不要 サンプルコンポーネントの動的ライブラリ(.so)へのシンボリックは不要

n-kawauchi commented 4 years ago

了解しました。PRソースを修正します。 hrtmが不要とは、libhrtm.a、libhrtm.so と、htrmのヘッダーファイルが不要との理解でよろしいですか？

r-kurose commented 4 years ago

そうです。 そもそも hrtm ディレクトリにある CMakeLists.txt に書かれていると思われる install() を消して終えば良いように思います

n-kawauchi commented 4 years ago

なるほど。 CMakeLists.txtでの対応は思いつきませんでした。 既にrulesでの対応を追加修正済みで、これにてdebパッケージに含まれないようにしました。

r-kurose commented 4 years ago

スクリプトの記述量は少なければ少ないほど良いので、 もし「削除する」行が増えたのだとしたら改善を考えた方が良いかもしれません。 hrtm という文字列がパッケージ作成中に出てこない方が良いでしょう。 今後もインストール先の変更やファイルの増減などが発生した場合、 CMakeLists.txt を整える方を先に検討してみてください。

n-kawauchi commented 4 years ago

ありがとうございます。了解しました。