Skip to content

Commit c3f4b7b

Browse files
committed
configury: fix --disable-mpi-io
- move the mpi-io configury option into config/ompi_configure_options.m4 - add ompi/mca/common/ompio/configure.m4 so this component is not built when Open MPI is configure'd with --disable-mpi-io Fixes #2009
1 parent af7fd57 commit c3f4b7b

File tree

3 files changed

+43
-16
lines changed

3 files changed

+43
-16
lines changed

config/ompi_configure_options.m4

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ dnl Copyright (c) 2009 Los Alamos National Security, LLC. All rights
1717
dnl reserved.
1818
dnl Copyright (c) 2009 Oak Ridge National Labs. All rights reserved.
1919
dnl Copyright (c) 2013 Intel, Inc. All rights reserved.
20-
dnl Copyright (c) 2015 Research Organization for Information Science
20+
dnl Copyright (c) 2015-2016 Research Organization for Information Science
2121
dnl and Technology (RIST). All rights reserved.
2222
dnl
2323
dnl $COPYRIGHT$
@@ -255,5 +255,13 @@ AC_DEFINE_UNQUOTED([OMPI_BUILD_FORTRAN_F08_SUBARRAYS],
255255
[$OMPI_BUILD_FORTRAN_F08_SUBARRAYS],
256256
[Whether we built the 'use mpi_f08' prototype subarray-based implementation or not (i.e., whether to build the use-mpi-f08-desc prototype or the regular use-mpi-f08 implementation)])
257257

258+
AC_ARG_ENABLE([mpi-io],
259+
[AC_HELP_STRING([--disable-mpi-io],
260+
[Disable built-in support for MPI-2 I/O, likely because
261+
an externally-provided MPI I/O package will be used.
262+
Default is to use the internal framework system that uses
263+
the ompio component and a specially modified version of ROMIO
264+
that fits inside the romio314 component])])
265+
258266
])dnl
259267

ompi/mca/common/ompio/configure.m4

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# -*- shell-script -*-
2+
#
3+
# Copyright (c) 2016 Research Organization for Information Science
4+
# and Technology (RIST). All rights reserved.
5+
# $COPYRIGHT$
6+
#
7+
# Additional copyrights may follow
8+
#
9+
# $HEADER$
10+
#
11+
12+
# MCA_ompi_common_ompio_CONFIG([action-if-can-compile],
13+
# [action-if-cant-compile])
14+
# ------------------------------------------------
15+
AC_DEFUN([MCA_ompi_common_ompio_CONFIG],[
16+
AC_CONFIG_FILES([ompi/mca/common/ompio/Makefile])
17+
18+
AS_IF([test "$enable_mpi_io" != "no"],
19+
[$1],
20+
[$2])
21+
])dnl

ompi/mca/io/configure.m4

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
# -*- shell-script -*-
22
#
33
# Copyright (c) 2006-2007 Los Alamos National Security, LLC.
4-
# All rights reserved.
5-
# Copyright (c) 2012 Cisco Systems, Inc. All rights reserved.
4+
# All rights reserved.
5+
# Copyright (c) 2012 Cisco Systems, Inc. All rights reserved.
6+
# Copyright (c) 2016 Research Organization for Information Science
7+
# and Technology (RIST). All rights reserved.
68
# $COPYRIGHT$
79
#
810
# Additional copyrights may follow
@@ -14,20 +16,15 @@
1416
# -------------------------------------------
1517
AC_DEFUN([MCA_ompi_io_CONFIG],
1618
[
17-
AC_ARG_ENABLE([mpi-io],
18-
[AC_HELP_STRING([--disable-mpi-io],
19-
[Disable built-in support for MPI-2 I/O, likely because
20-
an externally-provided MPI I/O package will be used.
21-
Default is to use the internal component system and
22-
its specially modified version of ROMIO])])
19+
OPAL_VAR_SCOPE_PUSH([define_mpi_io])
2320

24-
OMPI_MPIF_IO_CONSTANTS_INCLUDE=
25-
OMPI_MPIF_IO_HANDLES_INCLUDE=
2621
AS_IF([test "$enable_mpi_io" != "no"],
27-
[define_mpi_io=1
28-
OMPI_MPIF_IO_CONSTANTS_INCLUDE="include \"mpif-io-constants.h\""
29-
OMPI_MPIF_IO_HANDLES_INCLUDE="include \"mpif-io-handles.h\""],
30-
[define_mpi_io=0])
22+
[OMPI_MPIF_IO_CONSTANTS_INCLUDE="include \"mpif-io-constants.h\""
23+
OMPI_MPIF_IO_HANDLES_INCLUDE="include \"mpif-io-handles.h\""
24+
define_mpi_io=1],
25+
[OMPI_MPIF_IO_CONSTANTS_INCLUDE=
26+
OMPI_MPIF_IO_HANDLES_INCLUDE=
27+
define_mpi_io=0])
3128
AC_SUBST(OMPI_MPIF_IO_CONSTANTS_INCLUDE)
3229
AC_SUBST(OMPI_MPIF_IO_HANDLES_INCLUDE)
3330

@@ -37,6 +34,7 @@ AC_DEFUN([MCA_ompi_io_CONFIG],
3734
AC_SUBST(OMPI_PROVIDE_MPI_FILE_INTERFACE)
3835
AC_DEFINE_UNQUOTED([OMPI_PROVIDE_MPI_FILE_INTERFACE], [$define_mpi_io],
3936
[Whether OMPI should provide MPI File interface])
40-
AM_CONDITIONAL([OMPI_PROVIDE_MPI_FILE_INTERFACE], [test "$define_mpi_io" = "1"])
37+
AM_CONDITIONAL([OMPI_PROVIDE_MPI_FILE_INTERFACE], [test "$enable_mpi_io" != "no"])
4138

39+
OPAL_VAR_SCOPE_POP
4240
])

0 commit comments

Comments
 (0)