You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by "William A. Rowe, Jr." <wr...@rowe-clan.net> on 2006/07/14 18:46:03 UTC

Re: svn commit: r421800 - /apr/apr/trunk/build/apr_hints.m4

jerenkrantz@apache.org wrote:
> Author: jerenkrantz
> Date: Thu Jul 13 21:20:33 2006
> New Revision: 421800
> 
> URL: http://svn.apache.org/viewvc?rev=421800&view=rev
> Log:
> Enable LFS on mingw.
> 
> (Tweaked based on a suggestion from Joe Orton.)
> 
> Originally titled: [patch 07/17] Win32 LFS
> --- apr/apr/trunk/build/apr_hints.m4 (original)
> +++ apr/apr/trunk/build/apr_hints.m4 Thu Jul 13 21:20:33 2006
> @@ -427,6 +427,7 @@
>  	APR_SETIFNULL(apr_lock_method, [win32])
>  	APR_SETIFNULL(have_unicode_fs, [1])
>  	APR_SETIFNULL(have_proc_invoked, [1])
> +	APR_SETIFNULL(apr_cv_use_lfs64, [yes])

Ewww.  Not apr_ decorated.  This should be private/internals.

Bill

Re: svn commit: r421800 - /apr/apr/trunk/build/apr_hints.m4

Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
On 7/14/06, William A. Rowe, Jr. <wr...@rowe-clan.net> wrote:
> > +     APR_SETIFNULL(apr_cv_use_lfs64, [yes])
>
> Ewww.  Not apr_ decorated.  This should be private/internals.

Hmm?

This is the value that our autoconf checks explicitly looks for.  Feel
free to change the tests, but as written, this is the right variable
to set.  -- justin

Re: svn commit: r421800 - /apr/apr/trunk/build/apr_hints.m4

Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
On 7/14/06, William A. Rowe, Jr. <wr...@rowe-clan.net> wrote:
> > +     APR_SETIFNULL(apr_cv_use_lfs64, [yes])
>
> Ewww.  Not apr_ decorated.  This should be private/internals.

Hmm?

This is the value that our autoconf checks explicitly looks for.  Feel
free to change the tests, but as written, this is the right variable
to set.  -- justin