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

Use specific msvc cmake generator. #1

Merged
merged 1 commit into from Jul 23, 2020
Merged

Conversation

hidmic
Copy link
Contributor

@hidmic hidmic commented Jul 22, 2020

Precisely what the title says.

@hidmic hidmic requested a review from Blast545 July 22, 2020 17:21
Copy link
Contributor

@Blast545 Blast545 left a comment

Choose a reason for hiding this comment

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

LGTM

@Blast545
Copy link
Contributor

Blast545 commented Jul 22, 2020

CI:

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

@Blast545
Copy link
Contributor

CI result for MacOS is throwing this warning: Policy CMP0042 is not set: MACOSX_RPATH is enabled by default. Run "cmake. Have any idea how to solve this?

I won't block this PR for that warning, can be fixed with Windows CI error pending to be resolved.

@hidmic
Copy link
Contributor Author

hidmic commented Jul 22, 2020

Let's give it a shot now.

Copy link
Contributor

@Blast545 Blast545 left a comment

Choose a reason for hiding this comment

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

3/4 green, LGTM

@hidmic
Copy link
Contributor Author

hidmic commented Jul 22, 2020

Argh, this is giving me a hard time.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@hidmic hidmic force-pushed the hidmic/use-msvc-generator branch from 2ca69a3 to 3dac1fd Compare July 23, 2020 00:12
@Blast545
Copy link
Contributor

All tier 1 CI Green, feel free to merge! This will unblock ros2/ros2#985

@hidmic hidmic merged commit 7687045 into master Jul 23, 2020
@delete-merged-branch delete-merged-branch bot deleted the hidmic/use-msvc-generator branch July 23, 2020 13:50
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.

None yet

2 participants