Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • otb otb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 207
    • Issues 207
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Main Repositories
  • otbotb
  • Merge requests
  • !517

Fix Superbuild default OTB_DATA_ROOT

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Cédric Traizet requested to merge superbuild_default_dataroot into develop Jun 17, 2019
  • Overview 7
  • Commits 2
  • Pipelines 2
  • Changes 2

Summary

Change the default OTB_DATA_ROOT to ${OTB-SuperBuild_SOURCE_DIR}/../Data in Superbuild.

Rationale

There is a OTB_DATA_ROOT cmake variable in the SuperBuild project. Currently, this variable is initialized with :

find_path(OTB_DATA_ROOT README-OTB-Data
  PATHS
  $ENV{OTB_DATA_ROOT}
  ${OTB-SuperBuild_SOURCE_DIR}/../../OTB-Data)

But the default location for the data folder in OTB is now OTB_SRC/Data (since the migration to git lfs), so we should modify the default accordingly, as it is done in the OTB project.

This MR also removes the $ENV{OTB_DATA_ROOT} from PATHS/HINTS option of find_path in otb and Superbuild. The right way to set a custom data root is to add a cmake option -DOTB_DATA_ROOT when calling cmake.

Additional notes

This MR will be tested as superbuild branch on the dashboard. It is probably the cause of most of the failing tests on windows platforms.

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 👍 votes from core developers, no 👎 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
  • Optionally, run git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i on latest changes and commit
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: superbuild_default_dataroot