Skip to content
Snippets Groups Projects

BUG: better check for TIXML_USE_STL

Merged BUG: better check for TIXML_USE_STL
tixml-cmakedefine2 into develop
Merged Victor Poughon requested to merge tixml-cmakedefine2 into develop

Second attempt after review from !632 (closed).

Supporting TIXML_USE_STL to false is completely useless because OTB depends on the STL. Because of !487 (merged), this causes a build error in OTB 7.0.0 when using a tinyxml not from the SuperBuild (because it was set to ON in SuperBuild/patches/TINYXML/CMakeLists.txt).

  • Force TIXML_USE_STL to true in otb_tinyxml.h
  • Stop with a fatal error if tinyxml was not built with it

Merge request reports

Merge request pipeline #3049 passed with warnings

Stage: precheck
Stage: prepare
Stage: build
Stage: report
Stage: external

Merge request pipeline passed with warnings for 9f0fcd95

Merged by Victor PoughonVictor Poughon 5 years ago (Nov 19, 2019 3:23pm UTC)

Loading

Pipeline #3066 passed with warnings

Stage: precheck
Stage: prepare
Stage: build
Stage: report
Stage: deploy
Stage: external

Pipeline passed with warnings for 8700c3a5 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply