apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Davi Arnaut <d...@haxent.com.br>
Subject Re: [Vote] Release APR 1.2.9/0.9.14 and apr-iconv 1.2.0
Date Wed, 06 Jun 2007 01:44:15 GMT
Bojan Smojver wrote:
> On Wed, 2007-06-06 at 09:38 +1000, Bojan Smojver wrote:
> 
>> testlfs             :  Line 265: Large Files not supported
> 
> Or is this just a misleading message saying "these things are enabled by
> default on this platform"?
> 

Good question. LFS doesn't exist for 64 bit platforms, but because it
supports large files out of the box. This leads to another question,
should APR_HAS_LARGE_FILES be defined on 64-bit systems? It seems
reasonably safe to do so.

How about the following patch?

Index: configure.in
===================================================================
--- configure.in        (revision 544571)
+++ configure.in        (working copy)
@@ -454,7 +454,7 @@
     struct stat64 st;
     off64_t off = 4242;

-    if (sizeof(off64_t) != 8 || sizeof(off_t) != 4)
+    if (sizeof(off64_t) != 8)
        exit(1);
     if ((fd = open("conftest.lfs", O_LARGEFILE|O_CREAT|O_WRONLY)) < 0)
        exit(2);

Mime
View raw message