Skip to content

Commit 9938797

Browse files
committed
Don't accept UCX < 1.9
Don't just print a warning if the UCX version is too old, bail out and demand a newer version (at least 1.9, now more than 4 years old). Fixes #13017. Signed-off-by: George Bosilca <[email protected]>
1 parent 0bccfcd commit 9938797

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

config/ompi_check_ucx.m4

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,14 +75,16 @@ AC_DEFUN([OMPI_CHECK_UCX],[
7575
AS_IF([test "${ompi_check_ucx_cv_have_version_1_8}" = "yes"],
7676
[AC_MSG_WARN([UCX support skipped because version 1.8.x was found, which has a known catastrophic issue.])
7777
ompi_check_ucx_happy=no])])
78-
AC_PREPROC_IFELSE([AC_LANG_PROGRAM([[
78+
AC_PREPROC_IFELSE([AC_LANG_PROGRAM([[
7979
#include <ucp/api/ucp_version.h>
8080
]], [[
8181
#if (UCP_API_MAJOR < 1) || ((UCP_API_MAJOR == 1) && (UCP_API_MINOR < 9))
8282
#error "Version too low"
8383
#endif
8484
]])],
85-
[], [AC_MSG_WARN([UCX version is too old, please upgrade to 1.9 or higher.])])
85+
[],
86+
[AC_MSG_WARN([UCX version is too old, please upgrade to 1.9 or higher.])
87+
ompi_check_ucx_happy=no])
8688

8789
AS_IF([test "$ompi_check_ucx_happy" = yes],
8890
[AC_CHECK_DECLS([ucp_tag_send_nbr],

0 commit comments

Comments
 (0)