Fix compilation for ARM/uClibc

Fix compilation for ARM/uClibc


BUG=compile for ARM with uClibc

[digit1, 2012-07-13 08:33:21]

https://chromiumcodereview.appspot.com/10695176/diff/1/src/platform-linux.cc
File src/platform-linux.cc (right):

https://chromiumcodereview.appspot.com/10695176/diff/1/src/platform-linux.cc#...
src/platform-linux.cc:1029: (__UCLIBC__ != 1))
fwiw, I'm working on a similar patch for Android (which will provide ucontext_t
soon).

What about just using this instead:

  #if defined(__GLIBC__) && (__GLIBC__ < 2 || (__GLIBC__ == 2 && __GLIBC_MINOR__
<= 3))

Does UCLibc defines __GLIBC__ ?

[digit1, 2012-07-13 12:36:58]

For the record, my (slightly conflicting) patch is at
https://chromiumcodereview.appspot.com/10690163/

Let me know if you have a suggestion, and I'll modify it.

[remi.duraffort, 2012-07-16 08:43:23]

https://chromiumcodereview.appspot.com/10695176/diff/1/src/platform-linux.cc
File src/platform-linux.cc (right):

https://chromiumcodereview.appspot.com/10695176/diff/1/src/platform-linux.cc#...
src/platform-linux.cc:1029: (__UCLIBC__ != 1))
> What about just using this instead:
> 
>   #if defined(__GLIBC__) && (__GLIBC__ < 2 || (__GLIBC__ == 2 &&
__GLIBC_MINOR__
> <= 3))
> 
> Does UCLibc defines __GLIBC__ ?


uClibc does defines __GLIBC__ for compatibility reasons.
I will update my patch to be a "defined(__UCLIBC__)"

[remi.duraffort, 2012-07-16 09:01:32]

Path updated to only test for the definition of __UCLIBC__.

This fix the compilation on ARM for uClibc.

[Sven Panne, 2012-07-16 09:59:52]

Could you please open a new issue with the correct base URL
https://v8.googlecode.com/svn/branches/bleeding_edge and make lines fit into 80
columns? I can then land the CL for you, it looks OK, but we currently can't
test all ARM-related platforms.

[remi.duraffort, 2012-07-16 11:31:35]

I have created another issue with the correct base-url and wrapped at 80 at:
https://chromiumcodereview.appspot.com/10784012

I compiled v8 with this patch on both armv7_glibc and armv7_uclibc.


Regards.