Re: [PATCH] AC_SYS_LARGEFILE: define _DARWIN_USE_64_BIT_INODE conditionally

[Paul Eggert]

On 01/16/13 06:14, PHO wrote:
> Yes, AC_DEFINE is all I need. In fact I don't really understand why
> AC_SYS_LARGEFILE uses AH_VERBATIM to conditionally define
> _DARWIN_USE_64_BIT_INODE. Just in case a compiler defines it by
> default?

Naah, it was just a brain freeze on my part.  I pushed the following,
which should fix your problem.  I'll follow up on bug-gnulib, as
Gnulib has a similar issue.

>From 57e2988c3ff664c57547c985c4d3e826fc98e208 Mon Sep 17 00:00:00 2001
From: Paul Eggert <address@hidden>
Date: Wed, 16 Jan 2013 09:27:15 -0800
Subject: [PATCH] AC_SYS_LARGEFILE: port better to Mac OS X 10.5.

* lib/autoconf/specific.m4 (AC_SYS_LARGEFILE): Use AC_DEFINE, not
AH_VERBATIM, to define _DARWIN_USE_64_BIT_INODE, to avoid problems
with ino_t size being different for configuration time versus
build/run time.  Problem reported by PHO in
<http://lists.gnu.org/archive/html/bug-autoconf/2013-01/msg00040.html>.
---
 lib/autoconf/specific.m4 | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lib/autoconf/specific.m4 b/lib/autoconf/specific.m4
index bbfa24a..0d173f2 100644
--- a/lib/autoconf/specific.m4
+++ b/lib/autoconf/specific.m4
@@ -168,11 +168,8 @@ if test "$enable_largefile" != no; then
       [_AC_SYS_LARGEFILE_TEST_INCLUDES])
   fi
 
-  AH_VERBATIM([_DARWIN_USE_64_BIT_INODE],
-[/* Enable large inode numbers on Mac OS X 10.5.  */
-#ifndef _DARWIN_USE_64_BIT_INODE
-# define _DARWIN_USE_64_BIT_INODE 1
-#endif])
+  AC_DEFINE([_DARWIN_USE_64_BIT_INODE], [1],
+    [Enable large inode numbers on Mac OS X 10.5.])
 fi
 ])# AC_SYS_LARGEFILE
 
-- 
1.7.11.7