pschillinger/rqt_launchtree

Release 0.1.5 (or higher)

Closed this issue · 8 comments

130s commented

Thanks!

Also please consider making it into ROS Kinetic.

130s commented

There's a discussion right in time for your concern. In a nutshell, I recommend to increment the minor version (in this case to make 0.2.x for Kinetic).

130s commented

I guess that's also a valid concern.

"1.4.0" and "1.4.0+kinetic"

I'm personally not sure if bloom supports it but I think it's worthwhile giving a shot.

130s commented

While I'm curious to resolve #10 (or confirm that #10 isn't actually an issue), I'm personally craving for #5 to be available via DEB.

For the version number, I think you can try making a release with 1.4.0+kinetic. bloom is rigorous enough to be bug-free so it'll tell if your format is not allowed so that you wouldn't mess up rosdistro. It's also possible, however, that bloom would pass but you would still mess, but then you just open a fix PR (ROS buildfarm maintainers are very forgiving :) ).

Only thing I should highlight is that you never want to use the same version for different package content (regardless the difference is feature/config/bugfix/whitespace/etc.) Using the same version number for different code, there won't be any way to tell from each other, which could make debugging unnecessarily harder.

Trying to release 0.1.5 right now (indigo, jade) and will continue with attempting to release 0.1.5+kinetic afterwards. Internet is really slow, but I hope it will be sufficient. Otherwise I will try to finish it by tomorrow.

Unfortunately, no luck with the +kinetic option:
catkin_pkg.package.InvalidPackage: Invalid package manifest "././package.xml": Package version "0.1.4+kinetic" does not follow version conventions

I will release it as 0.2.0 for kinetic then (same features/fixes as 0.1.5 for indigo/jade).

rqt_launchtree 0.2.0 will be in the next update of kinetic as soon as the PR is merged.