Skip to content
This repository has been archived by the owner on Feb 3, 2025. It is now read-only.

Added environment variable GAZEBO_SUPPRESS_EOL_WARNING that disables EOL notices. #3406

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

peci1
Copy link
Contributor

@peci1 peci1 commented Nov 19, 2024

🎉 New feature

Closes #3405 (comment)

Summary

Added environment variable GAZEBO_SUPPRESS_EOL_WARNING. If set to 1, the EOL notice added in #3405 will not be shown.

Test it

Launch Gazebo and check for the notices both in console and in menubar.

The notice in About screen has been left untouched intentionally.

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.

@traversaro
Copy link
Collaborator

The vcpkg failure seems unrelated (fyi @talregev).

gazebo/gazebo_shared.cc Outdated Show resolved Hide resolved
@peci1
Copy link
Contributor Author

peci1 commented Nov 19, 2024

Is it possible that test failure https://build.osrfoundation.org/job/gazebo-ci-pr_any-ubuntu_auto-amd64-gpu-nvidia/1256/testReport/(root)/gzTest/SDF/ here was actually caused by #3405? The Ubuntu test was skipped in its latest CI.

I think the checksum might have changed due to the EOL notice. Maybe it isn't a good idea to print the EOL notice in gz sdf output?

@j-rivero
Copy link
Contributor

I think the checksum might have changed due to the EOL notice. Maybe it isn't a good idea to print the EOL notice in gz sdf output?

Building Gazebo I don't see any notice using the gz tool. Also I think that it does not share code changed in #3405 . The gzSDF test has been traditionally quite fragile to changes in sdformat, I vote to omit the failing in this PR and takes care of it in different PRs or issues (if someone wants to do it).

@peci1
Copy link
Contributor Author

peci1 commented Nov 21, 2024

Ok, thanks for the input. I was wondering, because running gz sdf locally, I did not get any of the two hashes reported in the failed test...

@scpeters
Copy link
Member

Is it possible that test failure https://build.osrfoundation.org/job/gazebo-ci-pr_any-ubuntu_auto-amd64-gpu-nvidia/1256/testReport/(root)/gzTest/SDF/ here was actually caused by #3405? The Ubuntu test was skipped in its latest CI.

I think the checksum might have changed due to the EOL notice. Maybe it isn't a good idea to print the EOL notice in gz sdf output?

I see a similar failure in an older test; I think it's unrelated

https://build.osrfoundation.org/job/gazebo-ci-gazebo11-focal-amd64-gpu-nvidia/93/testReport/junit/(root)/gzTest/SDF/

@scpeters scpeters merged commit b360002 into gazebosim:gazebo11 Nov 25, 2024
6 of 10 checks passed
@peci1
Copy link
Contributor Author

peci1 commented Nov 25, 2024

Great, thanks for merging. Are you going to make another release before Gazebo Classic is really EOL, so that this change would get into the binary distribution?

@j-rivero
Copy link
Contributor

Great, thanks for merging. Are you going to make another release before Gazebo Classic is really EOL, so that this change would get into the binary distribution?

Yes. #3408

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants