Skip to content

Commit 9b05b1c

Browse files
committed
Tweak the C++ binding deprecation check
Some of us have platform files that expressly disabled C++ support. While it is true that v5 no longer supports C++ and thus no longer needs us to disable it, there seems no reason to make us create platform files that differentiate based on OMPI version just for that reason. So if someone asks to "disable" the no-longer-existing support, just ignore it. Signed-off-by: Ralph Castain <rhc@pmix.org>
1 parent 2770974 commit 9b05b1c

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

config/ompi_deleted_options.m4

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,20 +37,31 @@ AC_DEFUN([OMPI_CHECK_DELETED_OPTIONS],[
3737

3838
# Open MPI C++ bindings were removed in v5.0
3939
cxx=0
40+
cxxseek=0
41+
cxxex=0
4042
AC_ARG_ENABLE([mpi-cxx],
4143
[AC_HELP_STRING([--enable-mpi-cxx],
4244
[*DELETED* Build the MPI C++ bindings])],
4345
[cxx=1])
4446
AC_ARG_ENABLE([mpi-cxx-seek],
4547
[AC_HELP_STRING([--enable-mpi-cxx-seek],
4648
[*DELETED* Build support for MPI::SEEK])],
47-
[cxx=1])
49+
[cxxseek=1])
4850
AC_ARG_ENABLE([cxx-exceptions],
4951
[AC_HELP_STRING([--enable-cxx-exceptions],
5052
[*DELETED* Build support for C++ exceptions in the MPI C++ bindings])],
51-
[cxx=1])
53+
[cxxex=1])
54+
55+
AS_IF([test "$enable_mpi_cxx" = "no" ],
56+
[cxx=0])
57+
58+
AS_IF([test "$enable_mpi_cxx_seek" = "no" ],
59+
[cxxseek=0])
60+
61+
AS_IF([test "$enable_cxx_exceptions" = "no" ],
62+
[cxxex=0])
5263

53-
AS_IF([test $cxx -eq 1],
64+
AS_IF([test $cxx -eq 1 || test $cxxseek -eq 1 || test $cxxex -eq 1],
5465
[AC_MSG_WARN([The MPI C++ bindings have been removed from Open MPI.])
5566
AC_MSG_WARN([If you need support for the MPI C++ bindings, you])
5667
AC_MSG_WARN([will need to use an older version of Open MPI.])

0 commit comments

Comments
 (0)