Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

add missing mutex lock for publisher links#1090

Merged
dirk-thomas merged 1 commit intolunar-develfrom
lock_mutex_publisher_links
Jul 7, 2017
Merged

add missing mutex lock for publisher links#1090
dirk-thomas merged 1 commit intolunar-develfrom
lock_mutex_publisher_links

Conversation

@dirk-thomas
Copy link
Copy Markdown
Member

Fixes #1079.

@alecive
Copy link
Copy Markdown

alecive commented Jul 7, 2017

Tested the same patch on indigo, and the #1079 does not show up any more.

@dirk-thomas
Copy link
Copy Markdown
Member Author

Thank you for trying the patch!

@dirk-thomas dirk-thomas merged commit 3706767 into lunar-devel Jul 7, 2017
@alecive
Copy link
Copy Markdown

alecive commented Jul 10, 2017

No problem! So is this patch going to reach indigo-level in some way or not?

@dirk-thomas dirk-thomas deleted the lock_mutex_publisher_links branch July 10, 2017 14:46
@dirk-thomas
Copy link
Copy Markdown
Member Author

Yes, in regular intervals all commits on the latest devel branch get reviewed and considered for backport. Usually only bug fixes are considered. This patch certainly qualifies for that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants