Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add package.xml / make it buildable with catkin/colcon #4

Open
SteveMacenski opened this issue Mar 5, 2021 · 0 comments
Open

Add package.xml / make it buildable with catkin/colcon #4

SteveMacenski opened this issue Mar 5, 2021 · 0 comments

Comments

@SteveMacenski
Copy link

SteveMacenski commented Mar 5, 2021

It would be great if this was a package that could itself be built in the catkin / colcon workspaces so that it doesn't have to be installed separately from the normal ROS toolchain (unless they like). Then it could also be released as binaries to apt so that ROS users can just install it.

This will actually be necessary in order to use the ROS build farm to package your ROS1/ROS2 drivers since they will require this librover to build. So either you need to release to an apt server yourself or leverage the ROS infra to do so if you want your robot drivers to be apt install ...-able. Package.xml files not make it required to use ament or catkin in the cmake files, you can set the built tool to cmake itself and it should all work out. But you do need the package.xml file for the build farm to even be actionable and for the build systems to let us build this in a ROS1/2 workspace.

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

No branches or pull requests

1 participant