Skip to content

CMakeLists.txt: Remove CUSTOM_COMPILE_FLAGS option#6

Merged
dougiesquire merged 2 commits into
gtracersfrom
tidy-cmakelists
Aug 6, 2025
Merged

CMakeLists.txt: Remove CUSTOM_COMPILE_FLAGS option#6
dougiesquire merged 2 commits into
gtracersfrom
tidy-cmakelists

Conversation

@dougiesquire
Copy link
Copy Markdown
Collaborator

@dougiesquire dougiesquire commented Aug 5, 2025 •
edited
Loading

This PR:

  • removes the CUSTOM_COMPILE_FLAGS option from the CMake build system. Custom compile flags can be provided via -DCMAKE_Fortran_FLAGS
  • tidies CMakeLists.txt
  • removes unused cmake/MOCSYConfig.cmake.in file

@dougiesquire
CMakeLists.txt: Remove CUSTOM_COMPILE_FLAGS option
38109de 
Fix indending and tidy without functional change

Remove unused cmake/MOCSYConfig.cmake.in file
@dougiesquire dougiesquire self-assigned this Aug 5, 2025
@dougiesquire dougiesquire requested a review from JorgeG94 August 5, 2025 05:33
JorgeG94
JorgeG94 reviewed Aug 5, 2025
View reviewed changes
Comment thread CMakeLists.txt Outdated
JorgeG94
JorgeG94 previously approved these changes Aug 5, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@JorgeG94 JorgeG94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@dougiesquire dougiesquire merged commit b136e1a into gtracers Aug 6, 2025
4 checks passed
@dougiesquire dougiesquire deleted the tidy-cmakelists branch August 6, 2025 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JorgeG94 JorgeG94 JorgeG94 left review comments

Assignees

@dougiesquire dougiesquire

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dougiesquire @JorgeG94