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

Bump to SDFormat 10 for Dome #85

Merged
merged 1 commit into from
Jul 27, 2020
Merged

Bump to SDFormat 10 for Dome #85

merged 1 commit into from
Jul 27, 2020

Conversation

chapulina
Copy link
Contributor

Several CI PRs will need to be merged before this goes in.

Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina requested a review from mxgrey as a code owner July 18, 2020 01:40
@chapulina chapulina requested a review from j-rivero July 18, 2020 01:40
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Jul 18, 2020
@chapulina
Copy link
Contributor Author

chapulina commented Jul 22, 2020

Windows CI is failing to find DART:

-- Could NOT find DART (missing: DART_DIR)
-- Looking for DART - not found

-- -------------------------------------------

-- BUILD WARNINGS
-- 	Cannot build component [dartsim] - Missing: DART (Components: collision-ode, utils, utils-urdf, CONFIG)
-- END BUILD WARNINGS

Ticketed #87

@codecov
Copy link

codecov bot commented Jul 22, 2020

Codecov Report

Merging #85 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   82.46%   82.46%           
=======================================
  Files         106      106           
  Lines        3587     3587           
=======================================
  Hits         2958     2958           
  Misses        629      629           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5056ae8...b64de0f. Read the comment docs.

@chapulina
Copy link
Contributor Author

The Windows build is failing to find tinyxml2:

  Target "UNIT_tpelib_Link_TEST" links to target "tinyxml2::tinyxml2" but the
  target was not found.  Perhaps a find_package() call is missing for an
  IMPORTED target, or an ALIAS target is missing?

I'm not sure if something needs to be changed on ign-physics, on CI infrastructure, or on SDFormat.

Here's the SDFormat PR that bumped it to use tinyxml2 for reference: gazebosim/sdformat#264

@scpeters scpeters closed this Jul 23, 2020
@scpeters scpeters reopened this Jul 23, 2020
@scpeters
Copy link
Member

The Windows build is failing to find tinyxml2:

  Target "UNIT_tpelib_Link_TEST" links to target "tinyxml2::tinyxml2" but the
  target was not found.  Perhaps a find_package() call is missing for an
  IMPORTED target, or an ALIAS target is missing?

I'm not sure if something needs to be changed on ign-physics, on CI infrastructure, or on SDFormat.

Here's the SDFormat PR that bumped it to use tinyxml2 for reference: osrf/sdformat#264

@mjcarroll do we need to update the sdformat10 cmake config file to include something about tinyxml2? does tinyxml2 need to be linked as PUBLIC? I think it could be PRIVATE instead?

@scpeters
Copy link
Member

I created gazebo-tooling/release-tools#247 to help with this, but I don't think it actually fixes the problem

@chapulina
Copy link
Contributor Author

I think it could be PRIVATE instead?

+1, it's my understanding that tinyxml2 isn't exposed on libSDFormat's public API

@chapulina
Copy link
Contributor Author

I think it could be PRIVATE instead?

That worked. See gazebosim/sdformat#321 and Build Status

@chapulina
Copy link
Contributor Author

@osrf-jenkins run tests please

1 similar comment
@chapulina
Copy link
Contributor Author

@osrf-jenkins run tests please

@chapulina chapulina merged commit 061965f into master Jul 27, 2020
@chapulina chapulina deleted the chapulina/sdf10 branch July 27, 2020 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants