Skip to content

Commit 1286709

Browse files
committed
Require flex only when keyval_lex.c is not provided
We currently require flex whenever we are in a Git clone, but that really isn't the requirement. We need flex whenever the flex output files are not present - otherwise, you can build just fine. So open things up a bit by tying the flex requirement to the actual one (i.e., that the flex output file exist). Signed-off-by: Ralph Castain <[email protected]>
1 parent af5dd49 commit 1286709

File tree

2 files changed

+19
-20
lines changed

2 files changed

+19
-20
lines changed

config/pmix.m4

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1026,25 +1026,6 @@ AC_DEFUN([PMIX_DEFINE_ARGS],[
10261026
AC_DEFINE_UNQUOTED(PMIX_ENABLE_DLOPEN_SUPPORT, $PMIX_ENABLE_DLOPEN_SUPPORT,
10271027
[Whether we want to enable dlopen support])
10281028

1029-
#
1030-
# Is this a developer copy?
1031-
#
1032-
1033-
if test -e $PMIX_TOP_SRCDIR/.git; then
1034-
PMIX_DEVEL=1
1035-
# check for Flex
1036-
AC_PROG_LEX(yywrap)
1037-
if test "x$LEX" != xflex; then
1038-
AC_MSG_WARN([PMIx requires Flex to build from non-tarball sources,])
1039-
AC_MSG_WARN([but Flex was not found. Please install Flex into])
1040-
AC_MSG_WARN([your path and try again])
1041-
AC_MSG_ERROR([Cannot continue])
1042-
fi
1043-
else
1044-
PMIX_DEVEL=0
1045-
fi
1046-
1047-
10481029
#
10491030
# Developer picky compiler options
10501031
#

configure.ac

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,25 @@ PMIX_SETUP_WRAPPER_INIT
209209
# This did not exist pre AM 1.11.x (where x is somewhere >0 and <3),
210210
# but it is necessary in AM 1.12.x.
211211
m4_ifdef([AM_PROG_AR], [AM_PROG_AR])
212-
AC_PROG_LEX(yywrap)
212+
213+
#
214+
# Is this a developer copy?
215+
#
216+
217+
if test -e $PMIX_TOP_SRCDIR/.git; then
218+
PMIX_DEVEL=1
219+
else
220+
PMIX_DEVEL=0
221+
fi
222+
# check for Flex
223+
AC_PROG_LEX(noyywrap)
224+
if test "x$LEX" != xflex && test ! -e $PMIX_TOP_SRCDIR/util/keyval/keyval_lex.c; then
225+
AC_MSG_WARN([PMIx requires Flex to build from sources that were not])
226+
AC_MSG_WARN([fully pre-processed (e.g., an official release tarball),])
227+
AC_MSG_WARN([but Flex was not found. Please install Flex into])
228+
AC_MSG_WARN([your path and try again])
229+
AC_MSG_ERROR([Cannot continue])
230+
fi
213231

214232
############################################################################
215233
# Configuration options

0 commit comments

Comments
 (0)