>   if ($perl_lfs64 && !$apr_lfs64) {
>      # Serious Conflict!  Strip -D_FILE_OFFSET_BITS=64 from $perl_cflags.

    if I read your recent APR patch right, because APR uses lfs by default
on platforms that support it this is unlikely unless perl has made a big
mistake or APR has incorrectly detected lfs support?

>   }
>   elsif (!$perl_lfs64 && $apr_lfs64) {
>      # Less serious conflict; this case isn't handled currently.
>      # Don't strip anything from CFLAGS but still do the apr_file_seek
>      # workaround, I think

    compiling APR with --disable-lfs is also an option here, right?

>   }
> 
> Building APR in the manner to give the "less serious conflict" is really
> an unsupported configuration, and didn't work before httpd-2.0.49
> anyway, so it would be OK to just give an error on that extremely
> unlikely situation.
> 
> This avoids needing to ever look at APR_HAS_LARGE_FILES, so should fix
> the issue with APR HEAD, and should work fine for 2.0.x too.

hmm.  in trying to get my head around the issue I still don't see where I
was having the problem that launched the thread.  my perl contains
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64, and APR_HAS_LARGE_FILES was 1
but things still weren't right.  maybe it's because the only apxs option
that comes back for me is -D_LARGEFILE64_SOURCE and -D_FILE_OFFSET_BITS=64
is missing?  does that hint that something in APR's new detection isn't
quite right?

--Geoff

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to