-
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
Use tinyxml2 for sdformat10 #220
Conversation
Signed-off-by: Michael Carroll <[email protected]>
Used for gazebosim/sdformat#264 |
We also need:
|
# SDFORMAT 10 and above use tinyxml2 | ||
if [[ ${SDFORMAT_MAJOR_VERSION} -ge 10]]; then | ||
SDFORMAT_NO_IGN_DEPENDENCIES="${SDFORMAT_NO_IGN_DEPENDENCIES} \\ | ||
libtinyxml2-dev" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is what it should look like eventually, but until everything is merged, I think it will keep other branches targeting sdf10
working if we install both libtinyxml-dev
and libtinyxml2-dev
for 10+. Once gazebosim/sdformat#264 is merged, the libtinyxml-dev
line can be removed.
how does that sound?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely reasonable, updated.
Signed-off-by: Michael Carroll <[email protected]>
Signed-off-by: Michael Carroll [email protected]