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

Windows: Add in registry key that is expected for ROS 2 Chocolatey. #205

Merged

Conversation

clalancette
Copy link
Contributor

Description

Windows: Add in registry key that is expected for ROS 2 Chocolatey.

The ROS 2 NUGET Chocolatey packages expect a registry entry of
HKCU\SOFTWARE\Kitware\CMake to exist; if it doesn't, they don't
properly register themselves with CMake and thus downstream software
can't properly find them. The Windows image that is currently available
to GitHub actions (https://github.com/actions/virtual-environments/blob/win19/20200608.1/images/win/Windows2019-Readme.md)
doesn't seem to have this key, so add it by hand here.

  • Does this PR check in generated JS code (npm run build)?

The comment has more information about why.

Signed-off-by: Chris Lalancette <[email protected]>
@clalancette
Copy link
Contributor Author

Can someone merge this one as well? Thanks!

@thomas-moulard thomas-moulard merged commit b1c8132 into ros-tooling:master Jun 26, 2020
@clalancette
Copy link
Contributor Author

Thank you!

@clalancette clalancette deleted the clalancette/add-cmake-registry branch June 26, 2020 15:37
@thomas-moulard
Copy link
Member

I also released 0.0.25 so you have a release with the fix.

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.

4 participants