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

Update cmd/CMakeLists to conform with all other gz libraries (#478) #480

Merged
merged 2 commits into from
Sep 1, 2022

Conversation

azeey
Copy link
Contributor

@azeey azeey commented Aug 31, 2022

➡️ Forward port

Cherry-pick #478 from ign-gui3 since merging forward would introduce new features during feature freeze. This is needed to make gz gui work properly. It could come in a minor release since gz gui is not a commonly used tool.

Note to maintainers: Remember to use Rebase-and-merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@github-actions github-actions bot added the 🌱 garden Ignition Garden label Aug 31, 2022
@jennuine
Copy link
Contributor

These tests are failing:

CmdLine.list
gzTest.GuiHelpVsCompletionFlags

…im#478)

This fixes the library path obtained by cmdgui.rb such that it points to
the .so file contained in the non-dev debian pack

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
@codecov
Copy link

codecov bot commented Aug 31, 2022

Codecov Report

Merging #480 (131eeb4) into gz-gui7 (b5f354a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           gz-gui7     #480   +/-   ##
========================================
  Coverage    67.82%   67.82%           
========================================
  Files           44       44           
  Lines         4826     4826           
========================================
  Hits          3273     3273           
  Misses        1553     1553           
Impacted Files Coverage Δ
src/plugins/camera_tracking/CameraTracking.cc 68.45% <ø> (ø)
src/plugins/grid_config/GridConfig.cc 22.35% <ø> (ø)
src/plugins/grid_config/GridConfig.hh 100.00% <ø> (ø)
src/plugins/image_display/ImageDisplay.cc 31.70% <ø> (ø)
...interactive_view_control/InteractiveViewControl.cc 17.88% <ø> (ø)
src/plugins/key_publisher/KeyPublisher.cc 56.52% <ø> (ø)
src/plugins/marker_manager/MarkerManager.cc 59.41% <ø> (ø)
src/plugins/minimal_scene/MinimalScene.cc 65.61% <ø> (ø)
src/plugins/publisher/Publisher.cc 90.78% <ø> (ø)
src/plugins/screenshot/Screenshot.cc 30.12% <ø> (ø)
... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@chapulina chapulina added the bug Something isn't working label Aug 31, 2022
src/cmd/CMakeLists.txt Outdated Show resolved Hide resolved
Signed-off-by: Louise Poubel <louise@openrobotics.org>

Co-authored-by: Jenn Nguyen <jenn@openrobotics.org>
@chapulina chapulina merged commit 7bf2018 into gazebosim:gz-gui7 Sep 1, 2022
@azeey azeey deleted the cherry_478 branch September 2, 2022 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🌱 garden Ignition Garden
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants