-
Notifications
You must be signed in to change notification settings - Fork 492
Added environment variable GAZEBO_SUPPRESS_EOL_WARNING that disables EOL notices. #3406
Conversation
The vcpkg failure seems unrelated (fyi @talregev). |
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 |
Signed-off-by: Martin Pecka <[email protected]>
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). |
Ok, thanks for the input. I was wondering, because running |
I see a similar failure in an older test; I think it's unrelated |
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 |
🎉 New feature
Closes #3405 (comment)
Summary
Added environment variable
GAZEBO_SUPPRESS_EOL_WARNING
. If set to1
, 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
codecheck
passed (See contributing)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.