Skip to content
This repository was archived by the owner on Aug 3, 2020. It is now read-only.

[urdf] address gcc6 build error and tune#157

Merged
wjwwood merged 1 commit into
ros:kinetic-develfrom
bulwahn:kinetic-devel
Sep 28, 2016
Merged

[urdf] address gcc6 build error and tune#157
wjwwood merged 1 commit into
ros:kinetic-develfrom
bulwahn:kinetic-devel

Conversation

@bulwahn
Copy link
Copy Markdown
Contributor

@bulwahn bulwahn commented Sep 21, 2016

With gcc6, compiling fails with stdlib.h: No such file or directory, as including '-isystem /usr/include' breaks with gcc6, cf., https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70129.

This commit addresses this issue for this package in the same way it was addressed in various other ROS packages. A list of related commits and pull requests is at ros/rosdistro#12783.

@wjwwood
Copy link
Copy Markdown
Member

wjwwood commented Sep 21, 2016

It looks like the pr failure is from an existing issue on xenial? Probably a ripple from the latest catkin release.

@wjwwood
Copy link
Copy Markdown
Member

wjwwood commented Sep 21, 2016

This will need to be rebased after we fix that in: #155

With gcc6, compiling fails with `stdlib.h: No such file or directory`,
as including '-isystem /usr/include' breaks with gcc6, cf.,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70129.

This commit addresses this issue for this package in the same way
it was addressed in various other ROS packages. A list of related
commits and pull requests is at:

  ros/rosdistro#12783

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@oss.bmw-carit.de>
@bulwahn
Copy link
Copy Markdown
Contributor Author

bulwahn commented Sep 23, 2016

This patch has been rebased after #155.

@bulwahn
Copy link
Copy Markdown
Contributor Author

bulwahn commented Sep 26, 2016

Please merge if there are not any further concerns. Thanks.

@wjwwood
Copy link
Copy Markdown
Member

wjwwood commented Sep 28, 2016

Thanks.

@wjwwood wjwwood merged commit ce0fcc8 into ros:kinetic-devel Sep 28, 2016
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