[PATCH] Fix make closures.c use dlmmap,dlmunmap on cygwin.
Dave Korn
dave.korn.cygwin@googlemail.com
Wed Jul 8 14:10:00 GMT 2009
Andrew Haley wrote:
> Dave Korn wrote:
>> [ Re: http://gcc.gnu.org/ml/gcc-patches/2009-06/threads.html#02228 ]
>>
>> Hey all,
>>
>> Take 2 of this patch. Works in the simplest way possible, by saying that
>> cygwin is an exception to the rule about win32/64 platforms not using the
>> local dlmmap/dlmunmap implementations.
>>
>> Sticklers for formatting will note that two of the #endif lines reach 81
>> characters, because of the comment repeating the #if condition. I didn't
>> think it was worth wrapping the lines for such a tiny excess length in
>> something this trivial, but I'll do so before committing if that's what the
>> maintainers decide.
>>
>> libffi/ChangeLog:
>>
>> * closures.c (mmap, munmap): Don't define replacement macros pointing
>> to dl* versions on windows platforms.
>>
>> Bootstrapping now. Assuming it completes and libffi tests run alright, OK?
>
> Sure: obvious/trivial, fixes bootstrap.
>
> Commit upstream and gcc trees, please.
>
> Andrew.
Doesn't bootstrap directly. Missing reference to `_is_selinux_enabled'.
Respin on the way. (Will probably just "#define is_selinux_enabled 0" for
cygwin.)
cheers,
DaveK
More information about the Java-patches
mailing list