Updating ruby script to use standalone executable

[sauk2]

🎉 New feature

Related to gazebosim/gz-tools#7

Summary

Updated the ruby script to use the standalone executable.

I also tried to resolve the issue of building the package because of the FrameSemantics class. You can check the error in ci of your branch.

I updated the CMakeLists and am not sure if the following is the right approach ...

  add_library(gz STATIC gz.cc ../FrameSemantics.cc)

Test it

Test any of the SDF commands

  gz sdf --help

Checklist

• Signed all commits for DCO
• Added tests
• Added example and/or tutorial
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

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

[azeey]

@sauk2 have you had a chance to push this further?

[sauk2]

@sauk2 have you had a chance to push this further?

@azeey Do take a look, I have made some changes to the tests.

[azeey]

Any reason for removing this? The --foce-version argument is handled by gz-tools (see https://github.com/gazebosim/gz-tools/blob/2b228e5b956f1e966053dd860374670573580b41/src/gz.in#L244). We need to make sure it still works with the change to using executables.

[sauk2]

Done. I added it back and the tests should pass now!

[azeey]

Revert?

[sauk2]

The following test failed when --force-version and --versions were kept in.

sdformat/src/gz_TEST.cc

Lines 2071 to 2073 in 181c0ef

	/// \brief Check help message and bash completion script for consistent flags
	TEST(HelpVsCompletionFlags, GZ_UTILS_TEST_DISABLED_ON_WIN32(SDF))
	{

I noticed those tags were removed in bash_completion of gz-transport so I did the same here. Let me know what you think.

[azeey]

hmm, I think we'll need to keep it here. Otherwise, it would not show up in the bash completions. I think it'd be better to modify the HelpVsCompletionFlags test to make it pass in this case.

[azeey]

Other than the --force-versions issue, it LGTM!

[azeey]

hmm, I think we'll need to keep it here. Otherwise, it would not show up in the bash completions. I think it'd be better to modify the HelpVsCompletionFlags test to make it pass in this case.