Skip to content

Conversation

@InvincibleRMC
Copy link
Contributor

@InvincibleRMC InvincibleRMC commented Oct 18, 2025

Bumps cpplint to the current version on Ubuntu 24.04.

Current version info found here.

If rhel gets cpplint could maybe stop vendoring cpplint.py in the future.

Relate issues:
#548
#522
#501
#500

Signed-off-by: Michael Carlstrom <[email protected]>
@christophebedard
Copy link
Member

Pulls: #562
Gist: https://gist.githubusercontent.com/christophebedard/c2878c607000610bb34990690e9fa862/raw/905fb39315abed5400bd906b1397e52746aec6f0/ros2.repos
BUILD args: --packages-above-and-dependencies ament_cpplint
TEST args: --packages-above ament_cpplint
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17314

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@christophebedard
Copy link
Member

Let's see if this breaks any tests downstream. If it does, we'll have to fix them at the same time.

@InvincibleRMC
Copy link
Contributor Author

There seems to be an oversight in cpplint 1.6.1 that it is running C++ lints on C files. To disable this you add // LINT_C_FILE to every .c file. This has already been adjusted upstream to run checks base on the .c file extensions as well.

https://github.com/cpplint/cpplint/blob/b728b3f2b013b8989bd626361ab21ae85f222830/cpplint.py#L1207-L1211

I imagine taking the patch would be less disruptive than adding LINT_C_FILE to every c file in ROS.

Signed-off-by: Michael Carlstrom <[email protected]>
Signed-off-by: Michael Carlstrom <[email protected]>
@christophebedard
Copy link
Member

Pulls: #562
Gist: https://gist.githubusercontent.com/christophebedard/181b266114b97d4e6455eac316112568/raw/905fb39315abed5400bd906b1397e52746aec6f0/ros2.repos
BUILD args: --packages-above-and-dependencies ament_cpplint
TEST args: --packages-above ament_cpplint
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17326

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Signed-off-by: Michael Carlstrom <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants