Move include to allow library re-use

[timn]

We would like to be able to re-use the alvar library in other nodes or software. In particular, we strive to use it as a fallback on Ubuntu for the RoboCup Logistics League Simulation. However, this is currently prevented because the file Camera.h contains the following:

include <tf/transform_broadcaster.h>

This pulls in a lot of the tf2 infrastructure. This in turn is currently incompatible on our side. The include is actually not used in the library defined by the headers. Rather the only reference is within the nodes. Therefore, if the line was moved out of the header and into the individual nodes, we could use the alvar library and the nodes would work without any difference.

It would be great if this were accepted and could be released as an indigo update. I can provide a patch if desired.

[130s]

+1 for the pull request ;)

[timn]

Hi Scott. Could you release 0.5.2 to jade so we have the fix there as well? It's still 0.5.1 and causing problems for us. Thanks!

[sniekum]

Due to time constraints, I don’t handle the releases (or maintenance) anymore - there are a few maintainers I’ve asked to help out. Would someone be willing to do this? Isaac maybe?

—Scott

On Feb 1, 2016, at 2:47 AM, Tim Niemueller [email protected] wrote:

Hi Scott. Could you release 0.5.2 to jade so we have the fix there as well? It's still 0.5.1 and causing problems for us. Thanks!

—
Reply to this email directly or view it on GitHub #66 (comment).

[130s]

Sure.

[sniekum]

Thanks!

On Feb 1, 2016, at 12:29 PM, Isaac I.Y. Saito [email protected] wrote:

Sure.

—
Reply to this email directly or view it on GitHub #66 (comment).

[130s]

Done
I ros/rosdistro#10393
J ros/rosdistro#10394