[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] AC_SYS_LARGEFILE: define _DARWIN_USE_64_BIT_INODE conditiona
From: |
Paul Eggert |
Subject: |
Re: [PATCH] AC_SYS_LARGEFILE: define _DARWIN_USE_64_BIT_INODE conditionally |
Date: |
Wed, 16 Jan 2013 09:29:09 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
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