From: Eric Sandeen <sandeen@redhat.com>
To: xfs mailing list <xfs@oss.sgi.com>
Subject: [PATCH] xfstests: Check for xlog_assign_lsn in xfs headers
Date: Wed, 03 Jun 2009 13:47:27 -0500 [thread overview]
Message-ID: <4A26C53F.5080105@redhat.com> (raw)
Check for xlog_assign_lsn in xfs headers
Checking for libxfs.h isn't enough; some debian installs
have libxfs.h but no xlog_assign_lsn, and the loggen build still
fails.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Tested-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
---
diff --git a/aclocal.m4 b/aclocal.m4
index ac616d1..8a61e39 100644
--- a/aclocal.m4
+++ b/aclocal.m4
@@ -421,6 +421,13 @@ AC_DEFUN([AC_PACKAGE_WANT_XFS_LIBXFS_H],
AC_SUBST(have_libxfs)
])
+AC_DEFUN([AC_PACKAGE_WANT_XLOG_ASSIGN_LSN],
+ [
+ AC_CHECK_DECL(xlog_assign_lsn,
+ [ have_xlog_assign_lsn=true ], [ have_xlog_assign_lsn=false ],
+ [[#include <xfs/libxfs.h>]])
+ ])
+
AC_DEFUN([AC_PACKAGE_NEED_XFS_XQM_H],
[ AC_CHECK_HEADERS([xfs/xqm.h])
if test "$ac_cv_header_xfs_xqm_h" != "yes"; then
diff --git a/configure.in b/configure.in
index 210a91e..6c2afe7 100644
--- a/configure.in
+++ b/configure.in
@@ -46,6 +46,7 @@ in
AC_MULTILIB($enable_lib64)
AC_PACKAGE_NEED_XFS_XFS_H
AC_PACKAGE_WANT_XFS_LIBXFS_H
+ AC_PACKAGE_WANT_XLOG_ASSIGN_LSN
AC_PACKAGE_NEED_XFS_XQM_H
AC_PACKAGE_NEED_XFSCTL_MACRO
AC_PACKAGE_NEED_XFS_HANDLE_H
diff --git a/include/builddefs.in b/include/builddefs.in
index e8f4b6b..7827ed5 100644
--- a/include/builddefs.in
+++ b/include/builddefs.in
@@ -15,6 +15,7 @@ CFLAGS = @CFLAGS@
CPPFLAGS = @CPPFLAGS@
HAVE_LIBXFS = @have_libxfs@
+HAVE_XLOG_ASSIGN_LSN = @have_xlog_assign_lsn@
LIBXFS = @libxfs@
LIBACL = @libacl@
LIBATTR = @libattr@
diff --git a/m4/package_xfslibs.m4 b/m4/package_xfslibs.m4
index 903e4a7..870882f 100644
--- a/m4/package_xfslibs.m4
+++ b/m4/package_xfslibs.m4
@@ -13,6 +13,14 @@ AC_DEFUN([AC_PACKAGE_WANT_LIBXFS_H],
AC_SUBST(have_libxfs)
])
+AC_DEFUN([AC_PACKAGE_WANT_XLOG_ASSIGN_LSN],
+ [
+ AC_CHECK_DECL(xlog_assign_lsn,
+ [ have_xlog_assign_lsn=true ], [ have_xlog_assign_lsn=false ],
+ [[#include <xfs/libxfs.h>]])
+ AC_SUBST(have_have_xlog_assign_lsn)
+ ])
+
AC_DEFUN([AC_PACKAGE_NEED_XFS_XQM_H],
[ AC_CHECK_HEADERS([xfs/xqm.h])
if test "$ac_cv_header_xfs_xqm_h" != "yes"; then
diff --git a/src/Makefile b/src/Makefile
index 515e16e..7b01754 100644
--- a/src/Makefile
+++ b/src/Makefile
@@ -17,7 +17,7 @@ LINUX_TARGETS = xfsctl bstat t_mtab getdevicesize \
fs_perms testx looptest locktest unwritten_mmap \
bulkstat_unlink_test bulkstat_unlink_test_modified t_dir_offset
-ifeq ($(HAVE_LIBXFS), true)
+ifeq ($(HAVE_XLOG_ASSIGN_LSN), true)
LINUX_TARGETS += loggen
endif
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next reply other threads:[~2009-06-03 18:47 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-03 18:47 Eric Sandeen [this message]
2009-06-03 19:43 ` [PATCH] xfstests: Check for xlog_assign_lsn in xfs headers Christoph Hellwig
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A26C53F.5080105@redhat.com \
--to=sandeen@redhat.com \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.