Skip to content
Snippets Groups Projects

Set OTB_USE_DEPRECATED to ON by default

Closed Victor Poughon requested to merge deprecated_on_default into develop

Summary

  • Make OTB_USE_DEPRECATED default value ON
  • Un-mark as advanced

Rationale

I think OTB_USE_DEPRECATED should be ON by default. Being off by default means that deprecation == removal in the default build configuration. What's more it's an 'advanced' CMake variable, so well hidden from users (it's also not documented anywhere). This is not suitable for semantic versionning and it causes some dashboard problems when testing deprecated features. See the current test fail about apTvPrConvertSensorToGeoPoint.

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.


Check before merging:

  • All discussions are resolved
  • At least 2 :thumbsup: votes from core developers, no :thumbsdown: vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement

Merge request reports

Closed by Victor PoughonVictor Poughon 6 years ago (Dec 13, 2018 2:48pm UTC)

Loading

Activity

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