Looks ok to me. Reviewed-by: Allison Henderson <allison.henderson@xxxxxxxxxx> On 3/14/19 2:03 PM, Darrick J. Wong wrote:
From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> The only user of fsetxattr and HAVE_FSETXATTR is fsr, which includes sys/xattr.h (from libc). However, the m4 macro to detect fsetxattr support requires attr/xattr.h (from libattr). libattr dropped xattr.h last year, so update the check. Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Reviewed-by: Christoph Hellwig <hch@xxxxxx> --- m4/package_libcdev.m4 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/m4/package_libcdev.m4 b/m4/package_libcdev.m4 index 7ee4acdd..2c0c72ce 100644 --- a/m4/package_libcdev.m4 +++ b/m4/package_libcdev.m4 @@ -223,14 +223,14 @@ AC_DEFUN([AC_HAVE_FLS], ])#-# Check if we have a fsetxattr call (Mac OS X) +# Check if we have a fsetxattr call # AC_DEFUN([AC_HAVE_FSETXATTR], [ AC_CHECK_DECL([fsetxattr], have_fsetxattr=yes, [], [#include <sys/types.h> - #include <attr/xattr.h>] + #include <sys/xattr.h>] ) AC_SUBST(have_fsetxattr) ])
