Skip to content

Commit 338dd78

Browse files
authored
Merge pull request #7491 from rhc54/topic/tweak
Tweak the C++ binding deprecation check
2 parents 2770974 + 4fe9ae3 commit 338dd78

File tree

2 files changed

+22
-4
lines changed

2 files changed

+22
-4
lines changed

config/ompi_deleted_options.m4

Lines changed: 20 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,12 @@ AC_DEFUN([OMPI_CHECK_DELETED_OPTIONS],[
2323
[*DELETED* Look for libpmi or libpmi2 in the given directory DIR, DIR/lib or DIR/lib64])],
2424
[with_pmi_libdir_given=yes])
2525

26+
AS_IF([test "$with_pmi" = "no"],
27+
[with_pmi_given=no])
28+
29+
AS_IF([test "$with_pmi_libdir" = "no"],
30+
[with_pmi_libdir_given=no])
31+
2632
if test "$with_pmi_given" = "yes" || test "$with_pmi_libdir_given" = "yes"; then
2733
AC_MSG_WARN([Open MPI no longer supports PMI-1 or PMI-2 libraries.])
2834
AC_MSG_WARN([PMIx is now required. Either the internal version or an])
@@ -37,20 +43,31 @@ AC_DEFUN([OMPI_CHECK_DELETED_OPTIONS],[
3743

3844
# Open MPI C++ bindings were removed in v5.0
3945
cxx=0
46+
cxxseek=0
47+
cxxex=0
4048
AC_ARG_ENABLE([mpi-cxx],
4149
[AC_HELP_STRING([--enable-mpi-cxx],
4250
[*DELETED* Build the MPI C++ bindings])],
4351
[cxx=1])
4452
AC_ARG_ENABLE([mpi-cxx-seek],
4553
[AC_HELP_STRING([--enable-mpi-cxx-seek],
4654
[*DELETED* Build support for MPI::SEEK])],
47-
[cxx=1])
55+
[cxxseek=1])
4856
AC_ARG_ENABLE([cxx-exceptions],
4957
[AC_HELP_STRING([--enable-cxx-exceptions],
5058
[*DELETED* Build support for C++ exceptions in the MPI C++ bindings])],
51-
[cxx=1])
59+
[cxxex=1])
60+
61+
AS_IF([test "$enable_mpi_cxx" = "no" ],
62+
[cxx=0])
63+
64+
AS_IF([test "$enable_mpi_cxx_seek" = "no" ],
65+
[cxxseek=0])
66+
67+
AS_IF([test "$enable_cxx_exceptions" = "no" ],
68+
[cxxex=0])
5269

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

ompi/mca/pml/base/pml_base_frame.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
* Copyright (c) 2015 Research Organization for Information Science
1717
* and Technology (RIST). All rights reserved.
1818
* Copyright (c) 2018 IBM Corporation. All rights reserved.
19+
* Copyright (c) 2020 Intel, Inc. All rights reserved.
1920
* $COPYRIGHT$
2021
*
2122
* Additional copyrights may follow
@@ -32,6 +33,7 @@
3233
#include <unistd.h>
3334
#endif /* HAVE_UNIST_H */
3435
#include "ompi/mca/mca.h"
36+
#include "opal/util/argv.h"
3537
#include "opal/util/output.h"
3638
#include "opal/mca/base/base.h"
3739

@@ -214,7 +216,6 @@ static int mca_pml_base_open(mca_base_open_flag_t flags)
214216
if( (NULL == default_pml || NULL == default_pml[0] ||
215217
0 == strlen(default_pml[0])) || (default_pml[0][0] == '^') ) {
216218
opal_pointer_array_add(&mca_pml_base_pml, strdup("ob1"));
217-
opal_pointer_array_add(&mca_pml_base_pml, strdup("yalla"));
218219
opal_pointer_array_add(&mca_pml_base_pml, strdup("ucx"));
219220
opal_pointer_array_add(&mca_pml_base_pml, strdup("cm"));
220221
} else {

0 commit comments

Comments
 (0)