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.

Sign up for GitHub

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

Jump to bottom

Getting an error that says there may be race condition issues with TF2 #9

Closed
ablasdel opened this issue Jun 27, 2013 · 2 comments
Closed

Getting an error that says there may be race condition issues with TF2 #9

ablasdel opened this issue Jun 27, 2013 · 2 comments
Labels
bug

Comments

@ablasdel
Copy link
Contributor

[ERROR] You've passed in true for auto_start for the C++ action server at [/tf2_buffer_server]. You should always pass in false to avoid race conditions.

Is this a bug? Is there a reason why true is used instead of false here?
@jonbinney
Copy link
Contributor

This is using the version in groovy debs. To reproduce:

rosrun tf2_ros buffer_server

@tfoote
Copy link
Member

tfoote commented Jun 27, 2013

I believe that it's a race condition which won't hurt this program so we didn't bother to spin the thread separately. It should be cleaned up not to warn.

@ablasdel ablasdel mentioned this issue Jun 28, 2013
Merged
dhood added a commit to dhood/geometry2 that referenced this issue May 30, 2017
@dhood
tf2_geometry_msgs needs to ament export dependencies (ros#9)
e2392e0 
* Use ament_auto_package to export dependencies

* Ensure required packages are found

* Remove obsolete install call

* Revert "Ensure required packages are found"

This reverts commit 9810df8.

* Alternate method for ensuring dependencies found

* Copy-paste correctly

* Use new auto REQUIRED option
@wasowski wasowski mentioned this issue Feb 20, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tfoote @jonbinney @ablasdel