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

[kinetic] Added rosbag::Bag::isOpen #1789

Merged
merged 1 commit into from Feb 10, 2020

Conversation

artivis
Copy link
Contributor

@artivis artivis commented Aug 9, 2019

Backport #1190 to kinetic-devel.

Fix #1448.

@artivis artivis changed the title Backport #1190 Backport #1190 to kinetic Aug 9, 2019
@artivis
Copy link
Contributor Author

artivis commented Aug 9, 2019

Just found out that #1190 was marked as 'not backported' in #1205.
While it is a 'new feature' it is also useful and ridiculously small enough not to be considered a maintenance burden. Up to the maintainers.

@dirk-thomas dirk-thomas changed the title Backport #1190 to kinetic [kinetic] Backport #1190 Feb 5, 2020
@dirk-thomas dirk-thomas changed the title [kinetic] Backport #1190 [kinetic] Added rosbag::Bag::isOpen Feb 10, 2020
@dirk-thomas
Copy link
Member

Accepting the backport since it has worked fine in Melodic for a long time and is API/ABI compatible.

@dirk-thomas dirk-thomas merged commit 19db93a into ros:kinetic-devel Feb 10, 2020
@artivis artivis deleted the kinetic-devel branch February 11, 2020 13:11
@DLu DLu mentioned this pull request Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants