Navigation Menu

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

Export Qt5 dependencies properly #687

Merged
merged 2 commits into from May 13, 2021
Merged

Export Qt5 dependencies properly #687

merged 2 commits into from May 13, 2021

Conversation

hidmic
Copy link
Contributor

@hidmic hidmic commented May 12, 2021

This patch modifies rviz_rendering and rviz_default_plugins packages to export Qt5's Widgets component as a dependency.

CI up to rviz_default_plugins and rviz_rendering_tests:

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

@hidmic hidmic force-pushed the hidmic/export-qt5-dependency branch from ec1dfe5 to ea3fea8 Compare May 12, 2021 14:59
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@hidmic hidmic merged commit f51878a into ros2 May 13, 2021
@delete-merged-branch delete-merged-branch bot deleted the hidmic/export-qt5-dependency branch May 13, 2021 20:55
hidmic added a commit that referenced this pull request Oct 29, 2021
* Export Qt5 as rviz_default_plugins dependency properly
* Export Qt5 as rviz_rendering dependency properly

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
hidmic added a commit that referenced this pull request Nov 8, 2021
* Export Qt5 as rviz_default_plugins dependency properly
* Export Qt5 as rviz_rendering dependency properly

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@jacobperron jacobperron added this to Proposed in Foxy Patch Release 7 via automation Nov 8, 2021
@jacobperron jacobperron moved this from Proposed to Needs backport in Foxy Patch Release 7 Nov 8, 2021
jacobperron pushed a commit that referenced this pull request Jan 31, 2022
* Export Qt5 as rviz_default_plugins dependency properly
* Export Qt5 as rviz_rendering dependency properly

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@jacobperron jacobperron removed this from Needs backport in Foxy Patch Release 7 Jan 31, 2022
jacobperron added a commit that referenced this pull request Feb 3, 2022
* Export Qt5 dependencies properly (#687)

* Export Qt5 as rviz_default_plugins dependency properly
* Export Qt5 as rviz_rendering dependency properly

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Lint

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

Co-authored-by: Michel Hidalgo <michel@ekumenlabs.com>
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