diff mbox series

[3/5] m4: fix statx override selection if /usr/include doesn't define it

Message ID 173689081926.3476119.12874616172883806766.stgit@frogsfrogsfrogs (mailing list archive)
State New
Headers show
Series [1/5] xfs_db: improve error message when unknown btree type given to btheight | expand

Commit Message

Darrick J. Wong Jan. 14, 2025, 9:41 p.m. UTC 
From: Darrick J. Wong <djwong@kernel.org>

If the system headers (aka the ones in /usr/include) do not define
struct statx at all, we need to use our internal override.  The m4 code
doesn't handle this admittedly corner case, but let's fix it for anyone
trying to build new xfsprogs on a decade-old distribution.

Fixes: 409477af604f46 ("xfs_io: add support for atomic write statx fields")
Signed-off-by: "Darrick J. Wong" <djwong@kernel.org>
---
 m4/package_libcdev.m4 |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Christoph Hellwig Jan. 15, 2025, 5:20 a.m. UTC | #1 
Looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>
diff mbox series

Patch

diff --git a/m4/package_libcdev.m4 b/m4/package_libcdev.m4
index 6db1177350b643..4ef7e8f67a3ba6 100644
--- a/m4/package_libcdev.m4
+++ b/m4/package_libcdev.m4
@@ -112,7 +112,7 @@  AC_DEFUN([AC_NEED_INTERNAL_STATX],
           need_internal_statx=yes,
           [#include <linux/stat.h>]
         )
-      ],,
+      ],need_internal_statx=yes,
       [#include <linux/stat.h>]
     )
     AC_SUBST(need_internal_statx)