diff mbox

[3/4] configure.ac: Look for attr/xattr.h, not libattr.so

Message ID 20140708164734.15173.23167.stgit@seurat.1015granger.net
State Accepted
Headers show

Commit Message

Chuck Lever July 8, 2014, 4:47 p.m. UTC 
Nothing in fedfs-utils uses libattr.so. The "*xattr" API lives in
glibc. The check for libattr.so in configure.ac adds an extraneous
package build dependency.

configure.ac should look for the presence of the attr/xattr.h header
instead, as that is what the fedfs-utils build requires.

Fixes: f5c16606 (configure.ac: check for presence of libattr)
Reported-by: Cristian Rodríguez <crrodriguez@opensuse.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
---
 configure.ac |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)
diff mbox

Patch

diff --git a/configure.ac b/configure.ac
index c70918852ce5..8b8d46323880 100644
--- a/configure.ac
+++ b/configure.ac
@@ -147,14 +147,9 @@  AC_CHECK_LIB([ssl], [SSL_CTX_new],
 		 AC_DEFINE([HAVE_LIBSSL], [1],
 			   [Define if you have libssl])],
 		[AC_MSG_ERROR([libssl not found.])])
-AC_CHECK_LIB([attr], [fgetxattr],
-		[AC_SUBST([LIBATTR], ["-lattr"])
-		 AC_DEFINE([HAVE_LIBATTR], [1],
-			   [Define if you have libattr])],
-		[AC_MSG_ERROR([libattr not found.])])
 
 # Checks for header files.
-AC_CHECK_HEADERS([fcntl.h langinfo.h locale.h memory.h netdb.h netinet/in.h stdint.h stdlib.h string.h sys/socket.h syslog.h termios.h unistd.h wchar.h])
+AC_CHECK_HEADERS([fcntl.h langinfo.h locale.h memory.h netdb.h netinet/in.h stdint.h stdlib.h string.h sys/socket.h syslog.h termios.h unistd.h wchar.h attr/xattr.h])
 AC_HEADER_STDBOOL
 
 # Checks for typedefs, structures, and compiler characteristics.