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 to uncrustify 0.72 #27

Merged
merged 2 commits into from
Dec 17, 2021
Merged

Conversation

clalancette
Copy link
Contributor

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

This updates our vendored version of uncrustify to 0.72. The most compelling reason to do this is to make it match the version that is going to ship with Ubuntu Jammy. Along with this change, we are also going to need updates to a number of downstream repositories. I'll link each of the changes we need to this pull request, and run CI on the whole lot here. I'll pull it out of draft once I have CI happy.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette
Copy link
Contributor Author

First try at CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor Author

Here's another CI (Linux aarch64 only) with more fixes:

  • Linux-aarch64 Build Status

@clalancette
Copy link
Contributor Author

Full CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor Author

macOS failures look like a flake, since it is failing in xmllint (which we didn't change here).

The Windows ones are a problem, since they are failing with "Could not find 'uncrustify' executable". Need to look into the installation on Windows here.

This may fix the Windows problem.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette
Copy link
Contributor Author

clalancette commented Dec 15, 2021

Here's another try at CI, now that I've done some changes here:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette clalancette marked this pull request as ready for review December 16, 2021 13:17
@clalancette
Copy link
Contributor Author

The failing tests on Windows are weird. A few of them show up in the latest Windows nightlies (like https://ci.ros2.org/view/nightly/job/nightly_win_rel/2150/#showFailuresLink , https://ci.ros2.org/view/nightly/job/nightly_win_rel/2149/, and https://ci.ros2.org/view/nightly/job/nightly_win_rel/2148/#showFailuresLink), but not all of them. I just can't see how this PR would cause those additional errors, though.

So I'm marking this as ready to review. @audrow can you take a look?

Copy link
Contributor

@audrow audrow left a comment

Choose a reason for hiding this comment

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

Looks good to me. I'm also not sure what to make of the windows failures. It's also not clear to me how they relate to this change.

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