Skip to content

Commit e599504

Browse files
committed
configury: check the existence of perl
perl is required by ompi/mpi/man/make_manpage.pl, that is even used in opal. so simply aborts at configure time if perl is not available Signed-off-by: Gilles Gouaillardet <[email protected]> (cherry picked from commit 5bb3efd)
1 parent 161a201 commit e599504

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

config/opal_config_asm.m4

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ dnl Copyright (c) 2004-2005 The Regents of the University of California.
1111
dnl All rights reserved.
1212
dnl Copyright (c) 2008-2015 Cisco Systems, Inc. All rights reserved.
1313
dnl Copyright (c) 2010 Oracle and/or its affiliates. All rights reserved.
14-
dnl Copyright (c) 2015 Research Organization for Information Science
14+
dnl Copyright (c) 2015-2016 Research Organization for Information Science
1515
dnl and Technology (RIST). All rights reserved.
1616
dnl $COPYRIGHT$
1717
dnl
@@ -1111,8 +1111,6 @@ AC_DEFUN([OPAL_ASM_FIND_FILE], [
11111111
AC_REQUIRE([AC_PROG_FGREP])
11121112
11131113
if test "$opal_cv_asm_arch" != "WINDOWS" && test "$opal_cv_asm_builtin" != "BUILTIN_SYNC" && test "$opal_cv_asm_builtin" != "BUILTIN_OSX" ; then
1114-
AC_CHECK_PROG([PERL], [perl], [perl])
1115-
11161114
# see if we have a pre-built one already
11171115
AC_MSG_CHECKING([for pre-built assembly file])
11181116
opal_cv_asm_file=""

configure.ac

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,10 @@ OPAL_LOAD_PLATFORM
6363
# Start it up
6464
#
6565

66+
AC_CHECK_PROG([PERL],[perl],[perl],[no])
67+
AS_IF([test "X$PERL" = "Xno"],
68+
[AC_MSG_ERROR(["Open MPI requires perl. Aborting"])])
69+
6670
OPAL_CONFIGURE_SETUP
6771
opal_show_title "Configuring project_name_long"
6872

0 commit comments

Comments
 (0)