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

Bump CMake version to avoid CMP0048 warning #1869

Merged
merged 1 commit into from
Feb 18, 2020

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Jan 25, 2020

This bumps the minimum CMake version to 3.0.2, which is the minimum supported by ROS Kinetic and new enough to default to the NEW behavior of CMP0048. This avoids a CMake warning when building this package in Debian Buster and Ubuntu Focal.

Same as ros/catkin#1052

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz self-assigned this Jan 25, 2020
@dirk-thomas
Copy link
Member

Waiting for Noetic PR to be passing...

@dirk-thomas
Copy link
Member

@ros-pull-request-builder retest this please

@dirk-thomas
Copy link
Member

dirk-thomas commented Feb 18, 2020

While the Noetic CI is still red the patch addresses the CMake warnings compared to the state on the target branch: http://build.ros.org/view/Ndev/job/Ndev__ros_comm__ubuntu_focal_amd64/20/ (same number of compiler warnings).

@dirk-thomas dirk-thomas merged commit a2cc2a7 into ros:melodic-devel Feb 18, 2020
stonier added a commit to splintered-reality/py_trees that referenced this pull request Aug 3, 2020
This is the officially supported version and eliminates
a CMP0048 warning on focal. See also
    ros/ros_comm#1869
stonier added a commit to splintered-reality/py_trees_ros that referenced this pull request Aug 3, 2020
This is the officially supported version and eliminates
    a CMP0048 warning on focal. See also
        ros/ros_comm#1869
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants