Skip to content

[CMake] Added optional parameter REQUIREMENTS to AddTest().#1610

Merged
endJunction merged 1 commit intoufz:masterfrom
bilke:conditional-ctests
Dec 13, 2016
Merged

[CMake] Added optional parameter REQUIREMENTS to AddTest().#1610
endJunction merged 1 commit intoufz:masterfrom
bilke:conditional-ctests

Conversation

@bilke
Copy link
Copy Markdown
Member

@bilke bilke commented Dec 12, 2016

Handles optional simple boolean expression which has to be true to enable the test, e.g.

AddTest(
  ..
  REQUIREMENTS OGS_USE_PETSC AND (OGS_USE_EIGEN OR OGS_USE_LIS)
)

Handles optional simple boolean expression which has to be true to
enable the test, e.g.

AddTest(
  ..
  REQUIREMENTS OGS_USE_PETSC AND (OGS_USE_EIGEN OR OGS_USE_LIS)
)
Copy link
Copy Markdown
Member

@endJunction endJunction left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 🎆

@chleh
Copy link
Copy Markdown
Collaborator

chleh commented Dec 13, 2016

👍 nice.
What will be the first test that needs such a distinction?

Copy link
Copy Markdown
Member

@wenqing wenqing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Copy Markdown
Member

@TomFischer TomFischer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks very useful.

@endJunction
Copy link
Copy Markdown
Member

First test case: The current SD/LIE and HM/LIE benchmarks run only with Eigen; there is no LIS solver good enough for this problems.

@endJunction endJunction merged commit 6996d28 into ufz:master Dec 13, 2016
@bilke bilke deleted the conditional-ctests branch December 13, 2016 15:49
@ogsbot
Copy link
Copy Markdown
Member

ogsbot commented Jun 19, 2020

OpenGeoSys development has been moved to GitLab.

See this pull request on GitLab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants