Skip to content

Commit f4a47a5

Browse files
committed
fortran: remove useless CPPFLAGS assignment
These -D's are for C compilation, not Fortran compilation. Remove this useless statement. Signed-off-by: Jeff Squyres <[email protected]>
1 parent d2b68e6 commit f4a47a5

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

ompi/mpi/fortran/use-mpi-ignore-tkr/Makefile.am

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# -*- makefile -*-
22
#
3-
# Copyright (c) 2006-2015 Cisco Systems, Inc. All rights reserved.
3+
# Copyright (c) 2006-2019 Cisco Systems, Inc. All rights reserved.
44
# Copyright (c) 2015-2018 Research Organization for Information Science
55
# and Technology (RIST). All rights reserved.
66
# Copyright (c) 2016 IBM Corporation. All rights reserved.
@@ -19,8 +19,6 @@ include $(top_srcdir)/Makefile.ompi-rules
1919
# mpi" MPI bindings.
2020
if OMPI_BUILD_FORTRAN_USEMPI_IGNORE_TKR_BINDINGS
2121

22-
AM_CPPFLAGS = -DOMPI_PROFILE_LAYER=0 -DOMPI_COMPILING_FORTRAN_WRAPPERS=1
23-
2422
AM_FCFLAGS = -I$(top_builddir)/ompi/include -I$(top_srcdir)/ompi/include \
2523
-I$(top_builddir) -I$(top_srcdir) $(FCFLAGS_f90)
2624

ompi/mpiext/pcollreq/use-mpi-f08/Makefile.am

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#
22
# Copyright (c) 2011-2012 Cisco Systems, Inc. All rights reserved.
3-
# Copyright (c) 2017-2018 Research Organization for Information Science
4-
# and Technology (RIST). All rights reserved.
3+
# Copyright (c) 2017-2019 Research Organization for Information Science
4+
# and Technology (RIST). All rights reserved.
55
# Copyright (c) 2018 FUJITSU LIMITED. All rights reserved.
66
# $COPYRIGHT$
77
#
@@ -13,10 +13,17 @@
1313
# This file builds the use_mpi_f08-based bindings for MPI extensions. It
1414
# is optional in MPI extensions.
1515

16+
# Note that Automake's Fortran-buidling rules uses CPPFLAGS and
17+
# AM_CPPFLAGS. This can cause weirdness (e.g.,
18+
# https://github.com/open-mpi/ompi/issues/7253). Let's just zero
19+
# those out and rely on AM_FCFLAGS.
20+
CPPFLAGS =
21+
AM_CPPFLAGS =
22+
1623
# We must set these #defines and include paths so that the inner OMPI
1724
# MPI prototype header files do the Right Thing.
1825
AM_FCFLAGS = $(OMPI_FC_MODULE_FLAG)$(top_builddir)/ompi/mpi/fortran/use-mpi-f08/mod \
19-
-I$(top_srcdir) $(FCFLAGS_f90)
26+
-I$(top_builddir) -I$(top_srcdir) $(FCFLAGS_f90)
2027

2128
# Note that the mpi_f08-based bindings are optional -- they can only
2229
# be built if OMPI is also building the Fortran-based bindings. So we

0 commit comments

Comments
 (0)