Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Package Layoutのルールに従ってヘッダーファイルをinstallする #36

Open
ShotaAk opened this issue Dec 10, 2019 · 3 comments

Comments

@ShotaAk
Copy link
Contributor

ShotaAk commented Dec 10, 2019

https://index.ros.org/doc/ros2/Contributing/Developer-Guide/#package-layout によると

src: contains all C and C++ code
    Also contains C/C++ headers which are not installed

include: contains all C and C++ headers which are installed
    <package name>: for all C and C++ installed headers they should be folder namespaced by the 
                                  package name

とある。

includeディレクトリにあるヘッダーファイルをinstallしてないものがいくつかあるので修正する。

参考:https://github.com/ros2/rosbag2/blob/master/rosbag2/CMakeLists.txt

やり方:

install(
  DIRECTORY include/
  DESTINATION include)

結果(例):
$ ls ~ros2_ws/install/hoge/include/hoge
huga.hpp

@ShotaAk
Copy link
Contributor Author

ShotaAk commented Dec 10, 2019

コンフリクトを避けるため、PRを消化し、パッケージが揃ってきてから取り掛かること

@spiralray
Copy link
Contributor

protoファイルについてはパッケージ直下のprotoディレクトリに配置し、
CMakeLists.txtにて以下のようにインストールするのはいかがでしょうか。

install(
  DIRECTORY proto/
  DESTINATION proto)

@spiralray
Copy link
Contributor

protoについて、pythonパッケージの関係もありますので #43 で議論したいです。
よろしくお願い致します

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants