Skip to content

Commit

Permalink
Merge branch 'framework/no_FCINCLUDES_check' into release-v8.2.0 (PR #…
Browse files Browse the repository at this point in the history
…1197)

This merge removes FCINCLUDES from the set of build variables that are checked
for build compatibility.

With commit c740150, the FCINCLUDES variable is modified with core-specific
options (for the atmosphere core specifically) and so including FCINCLUDES in
the set of build options that are checked for compatibility in the top-level
Makefile leads to a failed check, forcing a clean between the compilation of the
atmosphere core and other cores.

The FCINCLUDES variable contains very similar information to the CPPINCLUDES
variable, which is included in compatibility checks, and in order to enable the
atmosphere core to be compiled after compilation of another core, this merge
removes FCINCLUDES from the variables that are checked for build compatibility.

* framework/no_FCINCLUDES_check:
  Don't include FCINCLUDES in build options checked for build compatibility
  • Loading branch information
mgduda committed Jun 27, 2024
2 parents 760f9de + 14dda2b commit 82681e9
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -1042,7 +1042,6 @@ rebuild_check:
CPPFLAGS=$(CPPFLAGS)\n$\
LIBS=$(LIBS)\n$\
CPPINCLUDES=$(CPPINCLUDES)\n$\
FCINCLUDES=$(FCINCLUDES)\n$\
OPENMP=$(OPENMP)\n$\
OPENMP_OFFLOAD=$(OPENMP_OFFLOAD)\n$\
OPENACC=$(OPENACC)\n$\
Expand Down

0 comments on commit 82681e9

Please # to comment.