[PATCH] fixes for android in libraries/unix
Nathan Hüsken
nathan.huesken at posteo.de
Tue Jan 29 23:34:30 CET 2013
On 01/29/2013 11:34 AM, Simon Marlow wrote:
> On 25/01/13 08:55, Nathan Hüsken wrote:
>> These patches are for libraries/unix.
>> They may need some discussing, because I am not sure if this is the
>> right thing.
>>
>> 1. On android the passwd struct does not define pw_gecos. The first
>> patch therefor sets it to the empty string.
>>
>> 2. The android ndk does not define _POSIX_VDISABLE, but it defines
>> _PC_VDISABLE. The second patch defines _POSIX_VDISABLE to _PC_VIDISABLE,
>> which is (as I undestand it) what the POSIX programmer guide suggests.
>>
>> 3. The android ndk does not define telldir and seekdir. The third patch
>> removes them when compiling for android.
>
> Thanks for the patch. A better way to do these things is to add
> detection logic to the configure script, rather than platform-specific
> #ifdefs.
>
> See:
>
> http://hackage.haskell.org/trac/ghc/wiki/Commentary/Rts/Conventions#Otherportabilityconventions
>
>
> (the point about "avoid conditional code")
>
> There should be plenty of examples in the configure script that you can
> copy and modify.
>
Appended are update patches for 1 und 3.
Regards,
Nathan
More information about the ghc-devs
mailing list