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

Modern cmake targets #11

Merged
merged 12 commits into from
Jan 22, 2019
Merged

Modern cmake targets #11

merged 12 commits into from
Jan 22, 2019

Conversation

lettleli
Copy link
Contributor

Split the CMakeLists.txt file into one main CMakeLists.txt and additional CMakeLists.txt files for the testing and library subdirectories. Use "modern" CMake with only targets and target properties, which are also exported into the build and install tree.

Keep the output and build behaviour of the old CMake build process.

Add testing possibility.
Automatically check if different Fortran kinds are supported by the used compiler.

@codecov-io
Copy link

Codecov Report

Merging #11 into master will increase coverage by 4.3%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #11     +/-   ##
========================================
+ Coverage    92.6%   96.9%   +4.3%     
========================================
  Files         113     113             
  Lines         906     906             
  Branches       68       0     -68     
========================================
+ Hits          839     878     +39     
  Misses         28      28             
+ Partials       39       0     -39
Impacted Files Coverage Δ
src/lib/penf.F90 97.67% <0%> (+5.81%) ⬆️
src/lib/penf_stringify.F90 92.02% <0%> (+10.42%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c35165...2b0f9b1. Read the comment docs.

@szaghi szaghi merged commit 6e1b50e into szaghi:master Jan 22, 2019
@lettleli lettleli deleted the modern_cmake_targets branch October 2, 2019 11:38
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.

3 participants