Skip to content
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

Fix iDynTree 4.3.0 deprecations #588

Merged
merged 2 commits into from
Jan 30, 2023

Conversation

traversaro
Copy link
Collaborator

@traversaro traversaro commented Jan 28, 2023

There were this warnings:

CMake Warning (dev) at cmake/AddBipedalLocomotionLibrary.cmake:73 (target_link_libraries):
  The library that is being linked to, iDynTree::idyntree-modelio-urdf, is
  marked as being deprecated by the owner.  The message provided by the
  developer is:

  Do not use deprecated target iDynTree::idyntree-modelio-urdf, use
  iDynTree::idyntree-modelio instead.

Call Stack (most recent call first):
  src/Estimators/CMakeLists.txt:17 (add_bipedal_locomotion_library)
This warning is for project developers.  Use -Wno-dev to suppress it.

These were introduced in iDynTree 4.3.0, I think it is safe to use the new target.

@traversaro
Copy link
Collaborator Author

My bad, I did not saw #551 , anyhow this one is kind of a rebase of that one over the latest master.

@GiulioRomualdi
Copy link
Member

The failure of the actions are not related to this PR

@GiulioRomualdi GiulioRomualdi merged commit a0b753e into ami-iit:master Jan 30, 2023
@GiulioRomualdi
Copy link
Member

Thank you @traversaro

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

Successfully merging this pull request may close these issues.

3 participants