Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 <louise@openrobotics.org>
@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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants