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

Clean windows flags on _rclpy_pybind11 and _rclpy_action #688

Merged
merged 1 commit into from
Mar 1, 2021

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Feb 27, 2021

Fixes #687

The clean_windows_flags function from #681 needs to be applied to these targets.

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz self-assigned this Feb 27, 2021
@sloretz sloretz mentioned this pull request Feb 27, 2021
34 tasks
@sloretz sloretz added the bug Something isn't working label Feb 27, 2021
@sloretz
Copy link
Contributor Author

sloretz commented Feb 27, 2021

CI (build: --packages-up-to rclpy test: --packages-select rclpy)

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

@clalancette
Copy link
Contributor

@ros-pull-request-builder retest this please

@sloretz sloretz merged commit 0328fdc into master Mar 1, 2021
@delete-merged-branch delete-merged-branch bot deleted the clean_windows_flags__add_missing branch March 1, 2021 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

👨‍🌾 Warning in windows packaging jobs "ignoring '/INCREMENTAL' due to '/LTCG' specification"
3 participants