Skip to content

Use chocolatey_package resource and pin cppcheck version.#8

Merged
nuclearsandwich merged 2 commits intomasterfrom
use-chocolatey-package-resource
Jul 29, 2020
Merged

Use chocolatey_package resource and pin cppcheck version.#8
nuclearsandwich merged 2 commits intomasterfrom
use-chocolatey-package-resource

Conversation

@nuclearsandwich
Copy link
Copy Markdown
Contributor

The latest round of testing has issues with cppcheck 2.0 which are still being worked on. We need to propagate the pin but I also saw and forgot why this was being done with an execute resource rather than a chocolatey_package resource.

So this PR includes two changes.

  • Use chocolatey_package resource to install chocolatey packages.
  • Install cppcheck at version 1.90.

The latest round of testing has issues with cppcheck 2.0 which are still
being worked on. We need to propagate the pin but I also saw and forgot
why this was being done with an execute resource rather than a
chocolatey_package resource.
@nuclearsandwich nuclearsandwich requested a review from brawner July 28, 2020 21:37
@nuclearsandwich nuclearsandwich self-assigned this Jul 28, 2020
@nuclearsandwich
Copy link
Copy Markdown
Contributor Author

When running a test build of this PR it fails later on with a cffi installation but the chocolatey packages all seem to install correctly.

https://ci.ros2.org/job/test_ci_windows/185/ is running a non debug build to show us a successful build.

Copy link
Copy Markdown
Collaborator

@brawner brawner left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. I'm still racking my head. I'll try to do some digging myself for the history.

@brawner
Copy link
Copy Markdown
Collaborator

brawner commented Jul 28, 2020

I can't find the explanation, but this original switch to using the command-line version happened here in #3

76e4f3d

@nuclearsandwich
Copy link
Copy Markdown
Contributor Author

This branch was out of sync with master and thus having build issues. I've updated the branch to include master

If this build passes I think we'll be good to merge this change. Build Status

@nuclearsandwich
Copy link
Copy Markdown
Contributor Author

If this build passes I think we'll be good to merge this change.

The cookbook converged and the build had only failures that are also in the most recent nightly so I think this gets us where we need to be.

@nuclearsandwich nuclearsandwich merged commit 307961a into master Jul 29, 2020
@nuclearsandwich nuclearsandwich deleted the use-chocolatey-package-resource branch July 29, 2020 21:54
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