[mpich-discuss] ROMIO configure looks for lstat in the wrong header

[William Throwe]

Hello,

I hope this is the correct place to report a minor bug.  If not, please
point me in the right direction.

The ROMIO configure script checks for a declaration of lstat in
unistd.h, but, at least on the Linux machines I checked, lstat is in
sys/stat.h.  (The detection failure led to a linker error when building
ROMIO as part of OpenMPI on one of my admittedly strangely configured
machines, somehow.)  It appears from the man page that either location
is possible, so I suggest checking both.  Suggested patch below.

Thanks,
William Throwe

diff -Naur a/src/mpi/romio/configure.ac b/src/mpi/romio/configure.ac
--- a/src/mpi/romio/configure.ac	2015-09-14 18:33:06.000000000 -0400
+++ b/src/mpi/romio/configure.ac	2015-09-14 18:36:06.000000000 -0400
@@ -1562,7 +1562,8 @@
 AC_CHECK_FUNCS(lstat)
 if test "$ac_cv_func_lstat" = "yes" ; then
     # Do we need to declare lstat?
-    PAC_FUNC_NEEDS_DECL([#include <unistd.h>],lstat)
+    PAC_FUNC_NEEDS_DECL([#include <unistd.h>
+                         #include <sys/stat.h>],lstat)
 fi
 AC_CHECK_FUNCS(readlink)
 if test "$ac_cv_func_readlink" = "yes" ; then
_______________________________________________
discuss mailing list     discuss at mpich.org
To manage subscription options or unsubscribe:
https://lists.mpich.org/mailman/listinfo/discuss