-
Notifications
You must be signed in to change notification settings - Fork 9
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
Bump in Garden: ign-math7 and dependents #574
Comments
@mjcarroll needs to add a dependency to ignition-math7, so I've used
these formulae have identical dependency information to the fortress versions, so the script will need to be run again to actually change the dependency versions |
Running |
Libraries being bumped:
ign-math7
: The main motivation for a bump are are various static initialization issues being fixed onign-math7
that can't be backported toign-math6
.Ignition Math is low-level enough that we shouldn't do this bump lightly. One thing to keep in mind is that a lot of APIs have been updated from v6 to v7 (see gazebosim/gz-math#301), so the bump will come with lots of deprecation warnings for all downstream libraries, plugins, etc. If we'll go through the pain of handling all these deprecation warnings, I think we should consider completing the API updates for v7, so we deal with them all at once, and don't have another round of pain when v8 comes along. Having all the API consistent helps handling the deprecations more easily.
ign-math7
main
branch ongazebodistro
(example)main
branch (example)ign-common5
: DEM support (Geospatial component for heightmap & DEMs gazebosim/gz-common#267)main
branch ongazebodistro
(example)main
branch (example)sdformat13
: Depends onign-math
main
branch ongazebodistro
(example)main
branch (example)ign-physics6
: Depends onign-common
,ign-math
,sdformat
main
branch ongazebodistro
(example)main
branch (example)If #554 and #567 go through, bumping math shouldn't force the bump of anything that wouldn't already be bumped.#554 is in, and #567 was absorbed into this one.Downstream libraries affected:
ign-fuel-tools
ign-gazebo
ign-gui
ign-launch
ign-msgs
ign-rendering
ign-sensors
ign-transport
The text was updated successfully, but these errors were encountered: