Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Commit 3c4f09c

Browse files
committed
Merge pull request #1021 from ggouaillardet/topic/v2.x/ucx_configury
configury: UCX should use CPPFLAGS (instead of CFLAGS)
2 parents ea97656 + fa6a704 commit 3c4f09c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

config/ompi_check_ucx.m4

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# -*- shell-script -*-
22
#
33
# Copyright (C) 2015 Mellanox Technologies Ltd. ALL RIGHTS RESERVED.
4-
# Copyright (c) 2015 Research Organization for Information Science
4+
# Copyright (c) 2015-2016 Research Organization for Information Science
55
# and Technology (RIST). All rights reserved.
66
# $COPYRIGHT$
77
#
@@ -59,8 +59,8 @@ AC_DEFUN([OMPI_CHECK_UCX],[
5959

6060
AC_MSG_CHECKING(for UCX version compatibility)
6161
AC_REQUIRE_CPP
62-
old_CFLAGS="$CFLAGS"
63-
CFLAGS="$CFLAGS -I$ompi_check_ucx_dir/include"
62+
old_CPPFLAGS="$CPPFLAGS"
63+
CPPFLAGS="$CPPFLAGS -I$ompi_check_ucx_dir/include"
6464
AC_COMPILE_IFELSE(
6565
[AC_LANG_PROGRAM([[#include <uct/api/version.h>]],
6666
[[
@@ -69,7 +69,7 @@ AC_DEFUN([OMPI_CHECK_UCX],[
6969
[ompi_ucx_version_ok="no"])
7070

7171
AC_MSG_RESULT([$ompi_ucx_version_ok])
72-
CFLAGS=$old_CFLAGS
72+
CPPFLAGS=$old_CPPFLAGS
7373

7474
AS_IF([test "$ompi_ucx_version_ok" = "no"], [ompi_check_ucx_happy="no"])
7575

0 commit comments

Comments
 (0)