Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HAVE_COMM_F2C not set properly #20

Open
jmrosinski opened this issue May 29, 2018 · 1 comment
Open

HAVE_COMM_F2C not set properly #20

jmrosinski opened this issue May 29, 2018 · 1 comment

Comments

@jmrosinski
Copy link
Owner

It appears that HAVE_COMM_F2C is not set properly. On my laptop the MPI function MPI_Comm_f2c exists but HAVE_COMM_F2C is undefined. I'm guessing something like the following needs to be added to configure.ac (based solely on pattern matching what is in configure.ac currently)

Does the MPI library support MPI_Comm_f2c?

AC_MSG_CHECKING([whether MPI_Comm_f2c is available])
AC_CHECK_LIB([mpi], [MPI_Comm_f2c])
AM_CONDITIONAL([HAVE_COMM_F2C], [test "x$ac_cv_lib_mpi_MPI_Comm_f2c" = xyes])
if test "x$ac_cv_lib_mpi_MPI_Comm_f2c" = xyes; then
AC_DEFINE([HAVE_COMM_F2C], [1], [MPI_Comm_f2c is present])

@edhartnett
Copy link
Collaborator

I think we need an AC_DEFINE but not an AM_CONDITIONAL.

Here's what I just added, do a git pull to get it:

# Do we have function MPI_Comm_f2c?
AC_CHECK_FUNC([MPI_Comm_f2c], [have_mpi_comm_f2c=yes])
if test "x$have_mpi_comm_f2c" = xyes; then
   AC_DEFINE([HAVE_COMM_F2C], [1], [MPI_Comm_f2c is present])
fi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants