Shouldn't we test

ifndef __ANDROID__

instead?

-Doug

On Tue, May 24, 2011 at 2:39 AM, Guozhi Wei <car...@google.com> wrote:
> Hi
>
> This patch is for google/main.
>
> In order to be compatible with current bionic and sysroot, we need to disable
> getpagesize(). After getpagesize() in bionic is changed and ndk contains that
> change, we can reenable it.
>
> Jing can give more details about it.
>
> This patch has been tested on arm qemu without regression.
>
> thanks
> Carrot
>
> 2011-05-24  Jing Yu  <jin...@google.com>
>
>        * ChangeLog.google-main: New file.
>        * getpagesize.c(getpagesize): Disable it for bionic.
>
>
> Index: ChangeLog.google-main
> ===================================================================
> --- ChangeLog.google-main       (revision 0)
> +++ ChangeLog.google-main       (revision 0)
> @@ -0,0 +1,5 @@
> +Copyright (C) 2011 Free Software Foundation, Inc.
> +
> +Copying and distribution of this file, with or without modification,
> +are permitted in any medium without royalty provided the copyright
> +notice and this notice are preserved.
> Index: getpagesize.c
> ===================================================================
> --- getpagesize.c       (revision 174099)
> +++ getpagesize.c       (working copy)
> @@ -60,11 +60,13 @@ BUGS
>  # endif /* PAGESIZE */
>  #endif /* GNU_OUR_PAGESIZE */
>
> +#if DEFAULT_LIBC != LIBC_BIONIC
>  int
>  getpagesize (void)
>  {
>   return (GNU_OUR_PAGESIZE);
>  }
> +#endif
>
>  #else /* VMS */
>
>
> --
> This patch is available for review at http://codereview.appspot.com/4515131
>

Reply via email to