Skip to content

Commit

Permalink
Merge pull request #209 from jedwards4b/fix_gnu_debug_bld
Browse files Browse the repository at this point in the history
the build in github was not consistant with that outside github
  • Loading branch information
jedwards4b authored Jan 17, 2023
2 parents f027aa6 + ea9e05b commit 11e8c64
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/extbuild.yml
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,6 @@ jobs:
export SRC_ROOT=
mkdir build-cdeps
pushd build-cdeps
cmake -Wno-dev -DCMAKE_BUILD_TYPE=DEBUG -DCMAKE_Fortran_FLAGS="-DCPRGNU -g -Wall -ffree-form -ffree-line-length-none -fallow-argument-mismatch -Wno-unused-dummy-argument " -DWERROR=ON ../
cmake -Wno-dev -DCMAKE_BUILD_TYPE=DEBUG -DCMAKE_Fortran_FLAGS="-DCPRGNU -g -Wall -ffree-form -ffree-line-length-none -fallow-argument-mismatch " -DWERROR=ON ../
make VERBOSE=1
popd
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ foreach(COMP datm dice dlnd docn drof dwav)
$<INSTALL_INTERFACE:mod>)

if(WERROR)
target_compile_options(${COMP} PRIVATE -Werror)
target_compile_options(${COMP} PRIVATE -Werror --warn-no-unused-dummy-argument)
endif()
install(TARGETS ${COMP}
EXPORT ${COMP}
Expand All @@ -118,7 +118,7 @@ foreach(DEPS streams dshr cdeps_share FoX_dom FoX_wxml FoX_sax FoX_common FoX_ut
string(SUBSTRING ${DEPS} 0 3 fox)
if(WERROR AND NOT ${fox} STREQUAL "FoX")
message("Adding Werror flag to ${DEPS} (fox = ${fox})")
target_compile_options(${DEPS} PRIVATE -Werror)
target_compile_options(${DEPS} PRIVATE -Werror --warn-no-unused-dummy-argument)
endif()

install(TARGETS ${DEPS}
Expand Down

0 comments on commit 11e8c64

Please sign in to comment.