Skip to content

Commit 21f393c

Browse files
committed
cmake/ci lint
1 parent 9ed9d7d commit 21f393c

File tree

3 files changed

+16
-6
lines changed

3 files changed

+16
-6
lines changed

.github/workflows/ci_cmake.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,11 @@ jobs:
2525

2626

2727
macos:
28+
needs: linux
29+
runs-on: macos-latest
2830
env:
2931
FC: gfortran-11
30-
runs-on: macos-latest
32+
3133
steps:
3234
- uses: actions/checkout@v2
3335

@@ -38,9 +40,12 @@ jobs:
3840

3941
windows:
4042
runs-on: windows-latest
43+
env:
44+
CMAKE_GENERATOR: MinGW Makefiles
45+
4146
steps:
4247
- uses: actions/checkout@v2
4348

44-
- run: cmake --preset default -G "MinGW Makefiles"
49+
- run: cmake --preset default
4550
- run: cmake --build build --parallel
4651
- run: ctest --preset default

CMakeLists.txt

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,10 @@ include(CTest)
2929

3030
include (cmake/compilers.cmake)
3131

32+
set(CMAKE_CXX_STANDARD 11)
33+
set(CMAKE_C_STANDARD 99)
34+
3235
add_library(c_lib call_c.c)
33-
target_compile_features(c_lib PRIVATE c_std_99)
3436

3537
add_executable(f_call_c f_call_c.f90 $<TARGET_OBJECTS:c_lib>)
3638
add_test(NAME FortranCallC COMMAND f_call_c)
@@ -39,14 +41,16 @@ add_test(NAME FortranCallC COMMAND f_call_c)
3941
add_library(call_f OBJECT c_call_f.f90)
4042

4143
add_executable(cxx_call_f cxx_call_f.cxx $<TARGET_OBJECTS:call_f>)
42-
target_compile_features(cxx_call_f PRIVATE cxx_std_11)
4344
add_test(NAME CXXCallFortran COMMAND cxx_call_f)
4445

4546
# -- Fortran calling C++
4647
add_library(cxx_lib OBJECT call_cxx.cxx)
47-
target_compile_features(cxx_lib PRIVATE cxx_std_11)
4848

4949
add_executable(f_call_cxx f_call_c.f90 $<TARGET_OBJECTS:cxx_lib>)
5050
# LINKER_LANGUAGE option is necessary for ifort at least
5151
set_target_properties(f_call_cxx PROPERTIES LINKER_LANGUAGE Fortran)
5252
add_test(NAME FortranCallC++ COMMAND f_call_cxx)
53+
54+
set_tests_properties(FortranCallC CXXCallFortran FortranCallC++
55+
PROPERTIES
56+
TIMEOUT 5)

CMakePresets.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,8 @@
3535
},
3636
"execution": {
3737
"noTestsAction": "error",
38-
"stopOnFailure": false}
38+
"stopOnFailure": false
39+
}
3940
}
4041
]
4142
}

0 commit comments

Comments
 (0)