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.

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

Set Release as default build type #328

Merged
merged 3 commits into from
Mar 30, 2021

Conversation

olemke
Copy link
Member

@olemke olemke commented Mar 25, 2021

Set default build type and make the options visible in cmake-gui.
Based on https://blog.kitware.com/cmake-and-the-default-build-type/

Enable ccache by default if available.

Set default build type and make the options visible in cmake-gui.
Based on https://blog.kitware.com/cmake-and-the-default-build-type/
CMakeLists.txt Outdated Show resolved Hide resolved
@riclarsson
Copy link
Contributor

As an additional comment: These days when I start a new CPP project, I tends to use: https://github.com/lefticus/cpp_starter_project as base for CMake (stripping it of most things I don't need). I like its way of dealing with some of the default options (like ccache being activated by default if available).

@olemke
Copy link
Member Author

olemke commented Mar 29, 2021

Yes, it's a good template, I looked at it after he mentioned it in one of the C++ weekly episodes. I'm planning on cleaning up our CMakelists files to adapt them more to modern Cmake standards. Especially not setting global compile options but using the proper target compile options with PRIVATE/INTERFACE etc instead.
For ccache, I'd be fine with enabling it by default if available.

@olemke olemke merged commit 090e508 into atmtools:master Mar 30, 2021
@olemke olemke deleted the set-default-build-type-release branch March 30, 2021 12:24
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.

2 participants