-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Update Code-Style-Language-Versions.rst #4925
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
Conversation
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]>
fujitatomoya
left a comment
There was a problem hiding this 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?
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 |
Signed-off-by: mosfet80 <[email protected]>
sloretz
left a comment
There was a problem hiding this 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.
source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst
Outdated
Show resolved
Hide resolved
…ons.rst Co-authored-by: Shane Loretz <[email protected]> Signed-off-by: mosfet80 <[email protected]>
source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst
Outdated
Show resolved
Hide resolved
Signed-off-by: Shane Loretz <[email protected]>
source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst
Outdated
Show resolved
Hide resolved
Signed-off-by: Shane Loretz <[email protected]>
sloretz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR!
* Update Code-Style-Language-Versions.rst 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]> * Update Code-Style-Language-Versions.rst Signed-off-by: mosfet80 <[email protected]> * Update source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst Co-authored-by: Shane Loretz <[email protected]> Signed-off-by: mosfet80 <[email protected]> * Use RST link syntax Signed-off-by: Shane Loretz <[email protected]> * Fix RST link syntax Signed-off-by: Shane Loretz <[email protected]> --------- Signed-off-by: mosfet80 <[email protected]> Signed-off-by: mosfet80 <[email protected]> Signed-off-by: Shane Loretz <[email protected]> Co-authored-by: Shane Loretz <[email protected]> (cherry picked from commit 709e210)
* Update Code-Style-Language-Versions.rst 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]> * Update Code-Style-Language-Versions.rst Signed-off-by: mosfet80 <[email protected]> * Update source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst Co-authored-by: Shane Loretz <[email protected]> Signed-off-by: mosfet80 <[email protected]> * Use RST link syntax Signed-off-by: Shane Loretz <[email protected]> * Fix RST link syntax Signed-off-by: Shane Loretz <[email protected]> --------- Signed-off-by: mosfet80 <[email protected]> Signed-off-by: mosfet80 <[email protected]> Signed-off-by: Shane Loretz <[email protected]> Co-authored-by: Shane Loretz <[email protected]> (cherry picked from commit 709e210)
* Update Code-Style-Language-Versions.rst 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]> * Update Code-Style-Language-Versions.rst Signed-off-by: mosfet80 <[email protected]> * Update source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst Co-authored-by: Shane Loretz <[email protected]> Signed-off-by: mosfet80 <[email protected]> * Use RST link syntax Signed-off-by: Shane Loretz <[email protected]> * Fix RST link syntax Signed-off-by: Shane Loretz <[email protected]> --------- Signed-off-by: mosfet80 <[email protected]> Signed-off-by: mosfet80 <[email protected]> Signed-off-by: Shane Loretz <[email protected]> Co-authored-by: Shane Loretz <[email protected]> (cherry picked from commit 709e210)
* Update Code-Style-Language-Versions.rst 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 * Update Code-Style-Language-Versions.rst * Update source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst * Use RST link syntax * Fix RST link syntax --------- (cherry picked from commit 709e210) Signed-off-by: mosfet80 <[email protected]> Signed-off-by: mosfet80 <[email protected]> Signed-off-by: Shane Loretz <[email protected]> Co-authored-by: mosfet80 <[email protected]> Co-authored-by: Shane Loretz <[email protected]>
* Update Code-Style-Language-Versions.rst 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 * Update Code-Style-Language-Versions.rst * Update source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst * Use RST link syntax * Fix RST link syntax --------- (cherry picked from commit 709e210) Signed-off-by: mosfet80 <[email protected]> Signed-off-by: mosfet80 <[email protected]> Signed-off-by: Shane Loretz <[email protected]> Co-authored-by: mosfet80 <[email protected]> Co-authored-by: Shane Loretz <[email protected]>
* Update Code-Style-Language-Versions.rst 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 * Update Code-Style-Language-Versions.rst * Update source/The-ROS2-Project/Contributing/Code-Style-Language-Versions.rst * Use RST link syntax * Fix RST link syntax --------- (cherry picked from commit 709e210) Signed-off-by: mosfet80 <[email protected]> Signed-off-by: mosfet80 <[email protected]> Signed-off-by: Shane Loretz <[email protected]> Co-authored-by: mosfet80 <[email protected]> Co-authored-by: Shane Loretz <[email protected]>
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