export targets in a addition to include directories / libraries

[dirk-thomas]

Related to ros2/ros2#904.

[codecov]

Codecov Report

Merging #8 into master will increase coverage by 0.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   24.72%   25.40%   +0.67%     
==========================================
  Files          28       26       -2     
  Lines         829      807      -22     
  Branches      188      188              
==========================================
  Hits          205      205              
+ Misses        457      435      -22     
  Partials      167      167              

Flag	Coverage Δ
#unittests	25.40% <ø> (+0.67%)	⬆️

Impacted Files	Coverage Δ
...tics_collector/msg/dummy_message__type_support.cpp
...istics_collector/msg/dummy_message__type_support.c

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3751e8...74a6b46. Read the comment docs.

[dirk-thomas]

Maybe someone with knowledge about the GitHub actions in this repo can clarify what the failure is about?

Also triggering a current build for the target branch (not the one from the latest commit which is 5 days old) could help determining if the problem is already present on the target branch or introduced by this patch.

[thomas-moulard]

LGTM

[dabonnie]

Maybe someone with knowledge about the GitHub actions in this repo can clarify what the failure is about?

Also triggering a current build for the target branch (not the one from the latest commit which is 5 days old) could help determining if the problem is already present on the target branch or introduced by this patch.

The failure with OSX is present in master, so not introduced via this PR. We're tracking the issue here and I suspect it's something missing in https://github.com/ros-tooling/setup-ros, but need more data.

Will move forward with this and merge.