Skip to content

Commit 24b5b83

Browse files
authored
Merge pull request #3334 from rhc54/cmr20x/backward
Backport disabling of pmix support for pmi-1 and pmi-2 since these ca…
2 parents c56ff64 + 0038bf2 commit 24b5b83

File tree

5 files changed

+37
-10
lines changed

5 files changed

+37
-10
lines changed

opal/mca/pmix/pmix112/configure.m4

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
# Copyright (c) 2011-2013 Los Alamos National Security, LLC.
1414
# All rights reserved.
1515
# Copyright (c) 2010-2016 Cisco Systems, Inc. All rights reserved.
16-
# Copyright (c) 2013-2015 Intel, Inc. All rights reserved.
16+
# Copyright (c) 2013-2017 Intel, Inc. All rights reserved.
1717
# Copyright (c) 2015-2016 Research Organization for Information Science
1818
# and Technology (RIST). All rights reserved.
1919
# $COPYRIGHT$
@@ -38,7 +38,7 @@ AC_DEFUN([MCA_opal_pmix_pmix112_CONFIG],[
3838
opal_pmix_pmix112_save_LDFLAGS=$LDFLAGS
3939
opal_pmix_pmix112_save_LIBS=$LIBS
4040

41-
opal_pmix_pmix112_args="--enable-embedded-mode --with-pmix-symbol-prefix=opal_pmix_pmix112_ --with-libevent-header=\\\"opal/mca/event/$opal_event_base_include\\\" --with-hwloc-header=\\\"$opal_hwloc_base_include\\\""
41+
opal_pmix_pmix112_args="--enable-embedded-mode --disable-pmix-backward-compatibility --with-pmix-symbol-prefix=opal_pmix_pmix112_ --with-libevent-header=\\\"opal/mca/event/$opal_event_base_include\\\" --with-hwloc-header=\\\"$opal_hwloc_base_include\\\""
4242
AS_IF([test "$enable_debug" = "yes"],
4343
[opal_pmix_pmix112_args="--enable-debug $opal_pmix_pmix112_args"
4444
CFLAGS="$OPAL_CFLAGS_BEFORE_PICKY $OPAL_VISIBILITY_CFLAGS -g"],

opal/mca/pmix/pmix112/pmix/config/pmix.m4

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ dnl reserved.
1818
dnl Copyright (c) 2009-2011 Oak Ridge National Labs. All rights reserved.
1919
dnl Copyright (c) 2011-2013 NVIDIA Corporation. All rights reserved.
2020
dnl Copyright (c) 2015-2016 Research Organization for Information Science
21-
dnl Copyright (c) 2013-2016 Intel, Inc. All rights reserved
21+
dnl Copyright (c) 2013-2017 Intel, Inc. All rights reserved.
2222
dnl Copyright (c) 2016 Mellanox Technologies, Inc.
2323
dnl All rights reserved.
2424
dnl Copyright (c) 2016 IBM Corporation. All rights reserved.
@@ -795,6 +795,21 @@ fi
795795
AC_DEFINE_UNQUOTED([PMIX_ENABLE_TIMING], [$WANT_TIMING],
796796
[Whether we want developer-level timing support or not])
797797

798+
#
799+
# Install backward compatibility support for PMI-1 and PMI-2
800+
#
801+
AC_MSG_CHECKING([if want backward compatibility for PMI-1 and PMI-2])
802+
AC_ARG_ENABLE(pmix-backward-compatibility,
803+
AC_HELP_STRING([--enable-pmix-backward-compatibility],
804+
[enable PMIx support for PMI-1 and PMI-2 (default: enabled)]))
805+
if test "$enable_pmix_backward_compatibility" = "no"; then
806+
AC_MSG_RESULT([no])
807+
WANT_PMIX_BACKWARD=0
808+
else
809+
AC_MSG_RESULT([yes])
810+
WANT_PMIX_BACKWARD=1
811+
fi
812+
798813
])dnl
799814

800815
# Specify the symbol prefix
@@ -810,7 +825,7 @@ AC_DEFUN([PMIX_DO_AM_CONDITIONALS],[
810825
AM_CONDITIONAL([PMIX_COMPILE_TIMING], [test "$WANT_TIMING" = "1"])
811826
AM_CONDITIONAL([PMIX_WANT_MUNGE], [test "$pmix_munge_support" = "1"])
812827
AM_CONDITIONAL([PMIX_WANT_SASL], [test "$pmix_sasl_support" = "1"])
828+
AM_CONDITIONAL([WANT_PMIX_BACKWARD], [test "$WANT_PMIX_BACKWARD" = "1"])
813829
])
814830
pmix_did_am_conditionals=yes
815831
])dnl
816-

opal/mca/pmix/pmix112/pmix/include/Makefile.am

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#
2-
# Copyright 2015-2016 Intel, Inc. All rights reserved
2+
# Copyright (c) 2015-2017 Intel, Inc. All rights reserved.
33
# Copyright (c) 2016 IBM Corporation. All rights reserved.
44
#
55
# $COPYRIGHT$
@@ -15,9 +15,13 @@
1515
if ! PMIX_EMBEDDED_MODE
1616
include_HEADERS = \
1717
include/pmix.h \
18-
include/pmix_server.h \
18+
include/pmix_server.h
19+
20+
if WANT_PMIX_BACKWARD
21+
include_HEADERS += \
1922
include/pmi.h \
2023
include/pmi2.h
24+
endif
2125

2226
include_pmixdir = $(includedir)/pmix
2327
include_pmix_HEADERS = \

opal/mca/pmix/pmix112/pmix/src/client/Makefile.am

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#
2-
# Copyright (c) 2014-2015 Intel, Inc. All rights reserved.
2+
# Copyright (c) 2014-2017 Intel, Inc. All rights reserved.
33
# Copyright (c) 2014 Artem Y. Polyakov <[email protected]>.
44
# All rights reserved.
55
# $COPYRIGHT$
@@ -20,7 +20,7 @@ sources += \
2020
src/client/pmix_client_spawn.c \
2121
src/client/pmix_client_connect.c
2222

23-
if !PMIX_EMBEDDED_MODE
23+
if WANT_PMIX_BACKWARD
2424
sources += \
2525
src/client/pmi1.c \
2626
src/client/pmi2.c

opal/mca/pmix/pmix112/pmix/test/Makefile.am

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
# All rights reserved.
1212
# Copyright (c) 2006-2010 Cisco Systems, Inc. All rights reserved.
1313
# Copyright (c) 2012-2013 Los Alamos National Security, Inc. All rights reserved.
14-
# Copyright (c) 2013-2016 Intel, Inc. All rights reserved
14+
# Copyright (c) 2013-2017 Intel, Inc. All rights reserved.
1515
# $COPYRIGHT$
1616
#
1717
# Additional copyrights may follow
@@ -29,8 +29,14 @@ headers = test_common.h cli_stages.h server_callbacks.h utils.h test_fence.h tes
2929

3030
AM_CPPFLAGS = -I$(top_builddir)/src -I$(top_builddir)/src/include -I$(top_builddir)/src/api
3131

32+
noinst_PROGRAMS =
33+
3234
noinst_SCRIPTS = pmix_client_otheruser.sh
33-
noinst_PROGRAMS = pmi_client pmi2_client
35+
36+
if WANT_PMIX_BACKWARD
37+
noinst_PROGRAMS += pmi_client pmi2_client
38+
endif
39+
3440
if !WANT_HIDDEN
3541
noinst_PROGRAMS += pmix_test pmix_client pmix_regex
3642
endif
@@ -41,6 +47,7 @@ pmix_test_LDFLAGS = $(PMIX_PKG_CONFIG_LDFLAGS)
4147
pmix_test_LDADD = \
4248
$(top_builddir)/libpmix.la
4349

50+
if WANT_PMIX_BACKWARD
4451
pmi_client_SOURCES = $(headers) \
4552
pmi_client.c
4653
pmi_client_LDFLAGS = $(PMIX_PKG_CONFIG_LDFLAGS)
@@ -52,6 +59,7 @@ pmi2_client_SOURCES = $(headers) \
5259
pmi2_client_LDFLAGS = $(PMIX_PKG_CONFIG_LDFLAGS)
5360
pmi2_client_LDADD = \
5461
$(top_builddir)/libpmix.la
62+
endif
5563

5664
pmix_client_SOURCES = $(headers) \
5765
pmix_client.c test_fence.c test_common.c test_publish.c test_spawn.c test_cd.c test_resolve_peers.c test_error.c

0 commit comments

Comments
 (0)