-
Notifications
You must be signed in to change notification settings - Fork 304
Standalone gz sim executable (Jetty) #2849
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
…he gui unversioned component Signed-off-by: Jose Luis Rivero <[email protected]>
@osrf-jenkins run tests |
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
@osrf-jenkins run tests |
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
The gz command tests are failing. Need to fix these before merging.
|
I'm testing this on a Mac laptop. I first built it in a
|
I noticed this homebrew test failure recently and it's also happening in other builds as well, e.g. in #2870. May or may not be related |
I get the same error with all gz packages built from source in a colcon workspace.
Other notes:
|
I am able to run |
Thanks for the test !
I think we already captured this problem in the description.
Oh this is new, to be fixed IMHO without being a blocker for the PR.
Iit works on Linux and Windows as far as I can tell. Seems like we will need more work on Mac. Not a blocker since we are not introducing a regression. |
🎉 New feature
Comes from #2809. Related to gazebosim/gz-tools#7 and #2737.
Summary
This PR introduces a standalone executable for gz sim by following a similar approach to #2724. The new executable is integrated into cmdsim.rb.in, replacing the existing functionality. It is also a step in the direction of decoupling the server from GUI components, allowing us to build gz-sim library without GUI support.
#2809 should give a full context of the work. This PR changes the target branch name to
main
not to include the work in Ionic but in the upcomingJetty
.To port #2809 here I made the following commits:
gz sim
executable #2809 almost cleanlygz-sim-gui
since this new executable in this branch conflicts with the gui component at src/gui in this repository. In thegz-sim9
branch it is known asgz-sim9-gui
but now that we are not using version numbers generates a conflict. I choosedgz-sim-gui-client
but we can use a better name.Include here the list of things broken or for testing:
-v 4
togz-sim-gui-client
says it is not expected.