Skip to content

Commit 5bb3efd

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]>
1 parent f91f8ce commit 5bb3efd

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 (c) 2014-2016 Los Alamos National Security, LLC. All rights
1717
dnl reserved.
@@ -1235,8 +1235,6 @@ AC_DEFUN([OPAL_ASM_FIND_FILE], [
12351235
AC_REQUIRE([AC_PROG_FGREP])
12361236
12371237
if test "$opal_cv_asm_arch" != "WINDOWS" && test "$opal_cv_asm_builtin" != "BUILTIN_SYNC" && test "$opal_cv_asm_builtin" != "BUILTIN_OSX" ; then
1238-
AC_CHECK_PROG([PERL], [perl], [perl])
1239-
12401238
# see if we have a pre-built one already
12411239
AC_MSG_CHECKING([for pre-built assembly file])
12421240
opal_cv_asm_file=""

configure.ac

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

68+
AC_CHECK_PROG([PERL],[perl],[perl],[no])
69+
AS_IF([test "X$PERL" = "Xno"],
70+
[AC_MSG_ERROR(["Open MPI requires perl. Aborting"])])
71+
6872
OPAL_CONFIGURE_SETUP
6973
opal_show_title "Configuring project_name_long"
7074

0 commit comments

Comments
 (0)