search

ros-infrastructure / catkin_pkg

Standalone Python library for the catkin build system.
https://github.com/ros/catkin
Other
47 stars 91 forks source link

Fix generate_changelog when multiple tags/branches point to the same commit #348

Closed orensbruli closed 1 year ago

orensbruli commented 2 years ago

Signed-off-by: Esteban Martinena orensbruli@gmail.com

nuclearsandwich commented 1 year ago

Do you have an example repository which exhibits this issue for testing? I think that a better experience here is necessary but I'm not sure if it makes sense to pipe the STDERR and then completely ignore it. If ignoring it is correct then it would make more sense to send it to subprocess.DEVNULL but if we have to handle that warning somehow we will need to expand this PR to at least check for it and echo it back.