Skip to content

CMake and CI refinements - #44

Open
scivision wants to merge 4 commits into
likle:masterfrom
scivision:cmake-refine
Open

scivision wants to merge 4 commits into
likle:masterfrom
scivision:cmake-refine

Conversation

@scivision

@scivision scivision commented Feb 4, 2024

Copy link
Copy Markdown
  • CI: remove unused variables
  • CMake >= 3.14 was actually required with the project before this PR
  • use add_compile_options() and add_link_options() rather than edit CMake variables (this is canonical way)
  • simplify and generalize compiler options
  • add labels to tests to allow selection of subsets of tests

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.

1 participant