Skip to content

Commit 9894f9b

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

File tree

3 files changed

+38
-14
lines changed

3 files changed

+38
-14
lines changed

config/ompi_configure_options.m4

Lines changed: 12 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,16 @@ 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 component system and
263+
its specially modified version of ROMIO])])
264+
265+
AS_IF([test "$enable_mpi_io" != "no"],
266+
[define_mpi_io=1],
267+
[define_mpi_io=0])
268+
258269
])dnl
259270

ompi/mca/common/configure.m4

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# -*- shell-script -*-
2+
#
3+
# Copyright (c) 2006-2007 Los Alamos National Security, LLC.
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.
8+
# $COPYRIGHT$
9+
#
10+
# Additional copyrights may follow
11+
#
12+
# $HEADER$
13+
#
14+
15+
# MCA_ompi_common_CONFIG(project_name, framework_name)
16+
# -------------------------------------------
17+
AC_DEFUN([MCA_ompi_common_CONFIG],
18+
[
19+
MCA_CONFIGURE_FRAMEWORK([$1], [$2], [$define_mpi_io])
20+
])

ompi/mca/io/configure.m4

Lines changed: 6 additions & 13 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,11 @@
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])])
23-
2419
OMPI_MPIF_IO_CONSTANTS_INCLUDE=
2520
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\""])
3124
AC_SUBST(OMPI_MPIF_IO_CONSTANTS_INCLUDE)
3225
AC_SUBST(OMPI_MPIF_IO_HANDLES_INCLUDE)
3326

0 commit comments

Comments
 (0)