Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Please make tests build conditional on the cmake variable BUILD_TESTING #44

Closed
yurivict opened this issue May 7, 2022 · 2 comments · Fixed by #45
Closed

Please make tests build conditional on the cmake variable BUILD_TESTING #44

yurivict opened this issue May 7, 2022 · 2 comments · Fixed by #45

Comments

@yurivict
Copy link

yurivict commented May 7, 2022

BUILD_TESTING is defined by the cmake documentation for this purpose.

There is no need to build tests in the course of the package build. They should be built only when needed.
@yurivict yurivict changed the title Please make tests build conditional on a cmake variable BUILD_TESTING Please make tests build conditional on the cmake variable BUILD_TESTING May 7, 2022
@bhourahine bhourahine mentioned this issue May 8, 2022
Closed
@awvwgk
Copy link
Member

awvwgk commented May 8, 2022

For details on BUILD_TESTING usage see fortran-lang/stdlib#629, as there are some caveats to get this to work due to requiring an extra include.

@bhourahine bhourahine mentioned this issue May 8, 2022
Merged
@awvwgk
Copy link
Member

awvwgk commented May 8, 2022

There is an option called BUILD_EXPORTED_TARGETS_ONLY to skip the testing already available.

@awvwgk awvwgk closed this as completed May 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Build testing bhourahine/mpifx
2 participants
@yurivict @awvwgk