On 02/28/2011 01:46 AM, Philip Martin wrote:
> Barry Scott<barry_at_barrys-emacs.org> writes:
>
>> On 10 Jan 2011, at 03:59, Daniel Shahaf wrote:
>>
>>> Are the typedefs for SVN_FILESIZE_T_FMT and APR_OFF_T_FMT correct?
>>
>> No APR_OFF_T_FMT is wrong. Its "%lld" but needs to be APR_INT64_T_FMT so
>> that the apr_xxxprintf code will process it. It turns out that %lld is only half handled
>> and that leads to the crash.
>>
>> I applied this patch to get things working for me on Mac OS X 10.6 using the 10.6 SDK.
>>
>> --- ./apr/configure~ 2011-02-27 11:37:40.000000000 +0000
>> +++ ./apr/configure 2011-02-27 11:37:40.000000000 +0000
>
> To get this into APR you need to patch configure.in, not configure as
> that is a generated file.
>
>> @@ -29986,7 +29986,7 @@
>> *apple-darwin10.*)
>> # off_t is a long long, but long == long long
>> if test "$ac_cv_sizeof_long" = "$ac_cv_sizeof_long_long"; then
>> - off_t_fmt='#define APR_OFF_T_FMT "lld"'
>> + off_t_fmt='#define APR_OFF_T_FMT APR_INT64_T_FMT'
>> fi
>> ;;
>> esac
>>
>> The APR folks will need to fix their configure for Mac.
>
> Looking at the log:
>
> http://svn.apache.org/viewvc/apr/apr/trunk/configure.in?view=log
>
> I think r1050391 makes the change you are advocating, but it was
> reverted in r1051230.
Fyi, MacPorts applies the following patch to APR:
http://svn.macports.org/repository/macports/trunk/dports/devel/apr/files/patch-configure.diff
Blair
Received on 2011-02-28 20:24:47 CET