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

Update Code-Style-Language-Versions.rst #4925

Open
wants to merge 2 commits into
base: rolling
Choose a base branch
from

Conversation

mosfet80
Copy link

@mosfet80 mosfet80 commented Jan 7, 2025

Compatibility with versions of CMake older than 3.10 is now deprecated and will be removed from a future version. Oldest supported Ubuntu 20.04 use cmake 3.15

https://cmake.org/cmake/help/v3.31/release/3.31.html

Compatibility with versions of CMake older than 3.10 is now deprecated and will be removed from a future version.
Oldest supported Ubuntu 20.04 use cmake 3.15

https://cmake.org/cmake/help/v3.31/release/3.31.html

Signed-off-by: mosfet80 <[email protected]>
Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CMake versions are different for each platform, we would want to remove the version here and link https://www.ros.org/reps/rep-2000.html#jazzy-jalisco-may-2024-may-2029 instead?

@mosfet80
Copy link
Author

mosfet80 commented Jan 7, 2025

CMake versions are different for each platform, we would want to remove the version here and link https://www.ros.org/reps/rep-2000.html#jazzy-jalisco-may-2024-may-2029 instead?

looking at the tables in the link, the oldest cmake version used among the supported ros versions is 3.16.5. this corresponds to the use of ros humble on openembedded

Copy link
Contributor

@sloretz sloretz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 to including a link to REP 2000, and also +1 to including the minimum version of any supported ROS distro so the reader doesn't have to look through it.

@@ -353,7 +353,7 @@ CMake
Version
^^^^^^^

We will target CMake 3.8.
We will target CMake 3.16.5
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
We will target CMake 3.16.5
Read [REP 2000](https://www.ros.org/reps/rep-2000.html) to determine the minimum CMake version you should support.
Currently the minimum version of any supported ROS distro is **3.14.4** (ROS Humble on macOS).

@sloretz sloretz added the more-information-needed Further information is required label Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more-information-needed Further information is required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants