-
Notifications
You must be signed in to change notification settings - Fork 212
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
[osx][rviz_rendering] using ::isless undefined #440
Comments
I've seen this a fair bit in osx on high sierra (10.13), but I think you're using mojave. On high sierra, it seemed to happen when using a newer macOS SDK than the host system. We change some environment variables to work around it: but I'm not sure if this is the same problem that you are having |
Looking into it mit
I am on Mojave 10.14.6 and thus it should be okay with respect to version SDK. However, I managed to get it built with disabling the tests for this repo.
I hope this limits the search space. |
Hello, I was referred here from a failure to build nav2_rviz_plugins - see ros-navigation/navigation2#1043 Can anyone kindly suggest how to modify the rviz_ogre_vendor package to not include the MacOS SDK root as a system include? Or if I'm barking up the wrong tree, I can file a separate issue |
some relevant sounding discussion over here: https://gitlab.kitware.com/cmake/cmake/issues/19180 |
Can you give the latest RViz |
@Karsten1987 Do you have a .repos file handy? I tried building stock Dashing ros.repos + grabbing latest ros2 branch of Rviz, and rviz_common failed to build due to missing tf2/buffer_core_interface.h, would prefer to not guess on dependencies. |
@oneattosecond Try using the latest repos file: https://github.com/ros2/ros2/blob/master/ros2.repos |
dang, I'm getting a bunch of link errors in nav2_rviz_plugins... at least it gets farther??? --- stderr: nav2_rviz_plugins |
@Karsten1987 did this resolve the problem for you? I'm building on master and have the same issue. |
It did re-appear on my machine on a fresh build. My workaround to build it is by disabling the tests:
I know that's not a real solution, but at least it makes RViz compile and workable. |
@mjcarroll did you say you investigated this and thought it was a test that wasn't being built on CI due to the lack of a display? |
That was the theory. I was waiting to test it before commenting and got mired in qt installation grief. Having worked through it, rviz_rendering appears to build albeit with some stderr. I get the same results on our Mojave host lore when building from a GUI terminal (accessed via VNC) as the buildfarm yields when building in a "headless" terminal during the CI job. Example from https://ci.ros2.org/job/nightly_osx_release/1429/
Host details:
|
are you building with -Werror? https://gitlab.kitware.com/cmake/cmake/issues/19845 |
I have this problems when trying to build RViz on a overlay workspace on my OSX machine:
I googled quite a bit around and the closest I've come to this issue that it's somewhat related to whether one includes
math.h
beforecmath
but I can't really make sense out of it.Andersbakken/rtags#937
My OSX:
Any ideas? I am pretty much stuck with it.
The text was updated successfully, but these errors were encountered: