Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • O ogs
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 72
    • Issues 72
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 27
    • Merge requests 27
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ogs
  • ogs
  • Merge requests
  • !3983

[cmake] Refactored (and fixed) processes setup logic.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Lars Bilke requested to merge bilke/ogs:cmake-processes into master Feb 07, 2022
  • Overview 14
  • Commits 5
  • Pipelines 6
  • Changes 8
  • Fixed OGS_BUILD_PROCESSES with multiple values.
  • OGS_BUILD_PROCESSES is now a temporary variable (not cached between CMake runs) and should be set via cli only.
  • OGS_BUILD_PROCESSES overrides individual OGS_BUILD_PROCESS_X-variables:
    • -DOGS_BUILD_PROCESSES=ON --> all OGS_BUILD_PROCESS_X set to ON.
    • -DOGS_BUILD_PROCESSES=OFF --> all OGS_BUILD_PROCESS_X set to OFF.
    • e.g. -DOGS_BUILD_PROCESSES="HT;TH2M" --> OGS_BUILD_PROCESS_HT and OGS_BUILD_PROCESS_TH2M set to ON, all others set to OFF.

I would like to keep the previous implementation in this MR in the git history.

Before this change:

One could use the CMake variables OGS_BUILD_PROCESSES or OGS_BUILD_PROCESS_XYZ but behavior was not consistent when switching back and forth and using both methods. Also building multiple processes via OGS_BUILD_PROCESSES was broken.

Now changed:

  • Removed OGS_BUILD_PROCESS_XYZ options.
  • Use OGS_BUILD_PROCESSES with the following values:
    • ON -> All processes are built.
    • OFF -> Disable all processes.
    • Select a single process by name, e.g. TH2M.
    • Give a ;-separated list of processes, e.g. "THM;TH2M"
  • OGS_BUILD_PROCESSES can be set via ccmake or cmake-gui with a convenient dropdown, although setting multiple processes can only be done via the cli!
  1. Feature description was added to the changelog
  2. Tests covering your feature were added?
  3. Any new feature or behavior change was documented?
Edited Feb 08, 2022 by Lars Bilke
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: cmake-processes