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

[fix tests] link against pthread #756

Conversation

moriarty
Copy link
Contributor

Sorry @mikeferguson this should fix why the CI timed out after merging #752.

adds -lpthread in front of all calls to catkin_add_gtest

With this the tests all pass locally, without they can't find symbols.

I've also put this commit onto my branch for #755

adds -lpthread in front of all calls to catkin_add_gtest

With this the tests all pass locally, without they can't find symbols.
Should fix why the CI timed out after merging ros-planning#752.
@moriarty
Copy link
Contributor Author

I see that the restart of the timed out tests passed...
Also, I ran the the tests on the robot and they passed.

So I don't know why on my desktop I needed this extra flag... Other than my desktop being slightly ahead of the robot with a newer gcc

[build] Summary: All 15 packages succeeded!                                                                                                                                                                
[build]   Ignored:   1 packages were skipped or are blacklisted.                                                                                                                                           
[build]   Warnings:  1 packages succeeded with warnings.                                                                                                                                                   
[build]   Abandoned: None.                                                                                                                                                                                 
[build]   Failed:    None.                                                                                                                                                                                 
[build] Runtime: 4 minutes and 15.7 seconds total.                                                                                                                                                         
amoriarty at SW44 in ~/melodic-unique
$ uname -a
Linux SW44 4.15.0-28-generic #30-Ubuntu SMP Thu Jul 12 17:35:57 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
amoriarty at SW44 in ~/melodic-unique
$ gcc --version
gcc (Ubuntu 8-20180414-1ubuntu2) 8.0.1 20180414 (experimental) [trunk revision 259383]
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

@mikeferguson
Copy link
Contributor

I don't think we need this -- CI is working on both your PR and the merged version now (And not working on this PR). Going to close -- please reopen if you find out what is actually going on with your setup. Thanks.

@mfinean
Copy link

mfinean commented Oct 31, 2019

Also encountered this problem and the above fix worked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants