Skip to content

Fix test_ecbuild_override_compiler_flags (fixes #135)#137

Merged
marcosbento merged 1 commit intodevelopfrom
fix-135-test-ecbuild_override_compiler_flags
Mar 26, 2026
Merged

Fix test_ecbuild_override_compiler_flags (fixes #135)#137
marcosbento merged 1 commit intodevelopfrom
fix-135-test-ecbuild_override_compiler_flags

Conversation

@wdeconinck
Copy link
Copy Markdown
Member

Description

This PR fixes the test_ecbuild_override_compiler_flags reported failing in #135 .
The problem was that the file 'compiler-flags.cmake' needed the "${CMAKE_CURRENT_SOURCE_DIR}/" prefixed.

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@marcosbento marcosbento force-pushed the fix-135-test-ecbuild_override_compiler_flags branch from 98473f6 to 8560909 Compare March 26, 2026 14:58
@marcosbento
Copy link
Copy Markdown
Collaborator

This looks OK!

@marcosbento marcosbento merged commit 8560909 into develop Mar 26, 2026
64 of 71 checks passed
@marcosbento marcosbento deleted the fix-135-test-ecbuild_override_compiler_flags branch March 26, 2026 14:59
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