Patchwork [v2] configure: include stddef.h for NULL

login
register
mail settings
Submitter Scott Wood
Date Oct. 5, 2010, 7:28 p.m.
Message ID <20101005192817.GA31707@udp111988uds.am.freescale.net>
Download mbox | patch
Permalink /patch/66853/
State New
Headers show

Comments

Scott Wood - Oct. 5, 2010, 7:28 p.m.
This fixes an observed failure to detect madvise() on Linux.

To avoid similar issues, all other tests that use NULL but don't already
have stddef.h (or another header that is defined to provide NULL,
such as stdio.h, unistd.h, or time.h) are also fixed.

Signed-off-by: Scott Wood <scottwood@freescale.com>
---
This version doesn't add stddef.h where another header is defined
to provide NULL.

 configure |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
Blue Swirl - Oct. 9, 2010, 9:42 a.m.
Thanks, applied.

On Tue, Oct 5, 2010 at 7:28 PM, Scott Wood <scottwood@freescale.com> wrote:
> This fixes an observed failure to detect madvise() on Linux.
>
> To avoid similar issues, all other tests that use NULL but don't already
> have stddef.h (or another header that is defined to provide NULL,
> such as stdio.h, unistd.h, or time.h) are also fixed.
>
> Signed-off-by: Scott Wood <scottwood@freescale.com>
> ---
> This version doesn't add stddef.h where another header is defined
> to provide NULL.
>
>  configure |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/configure b/configure
> index e0d34fd..0eaf2cf 100755
> --- a/configure
> +++ b/configure
> @@ -1319,6 +1319,7 @@ if test "$vnc_png" != "no" ; then
>  cat > $TMPC <<EOF
>  //#include <stdio.h>
>  #include <png.h>
> +#include <stddef.h>
>  int main(void) {
>     png_structp png_ptr;
>     png_ptr = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
> @@ -1503,6 +1504,7 @@ if test "$brlapi" != "no" ; then
>   brlapi_libs="-lbrlapi"
>   cat > $TMPC << EOF
>  #include <brlapi.h>
> +#include <stddef.h>
>  int main( void ) { return brlapi__openConnection (NULL, NULL, NULL); }
>  EOF
>   if compile_prog "" "$brlapi_libs" ; then
> @@ -1737,6 +1739,7 @@ if test "$linux_aio" != "no" ; then
>   cat > $TMPC <<EOF
>  #include <libaio.h>
>  #include <sys/eventfd.h>
> +#include <stddef.h>
>  int main(void) { io_setup(0, NULL); io_set_eventfd(NULL, 0); eventfd(0, 0); return 0; }
>  EOF
>   if compile_prog "" "-laio" ; then
> @@ -2094,6 +2097,7 @@ madvise=no
>  cat > $TMPC << EOF
>  #include <sys/types.h>
>  #include <sys/mman.h>
> +#include <stddef.h>
>  int main(void) { return madvise(NULL, 0, MADV_DONTNEED); }
>  EOF
>  if compile_prog "" "" ; then
> @@ -2106,6 +2110,7 @@ fi
>  posix_madvise=no
>  cat > $TMPC << EOF
>  #include <sys/mman.h>
> +#include <stddef.h>
>  int main(void) { return posix_madvise(NULL, 0, POSIX_MADV_DONTNEED); }
>  EOF
>  if compile_prog "" "" ; then
> --
> 1.7.0.4
>
>
>

Patch

diff --git a/configure b/configure
index e0d34fd..0eaf2cf 100755
--- a/configure
+++ b/configure
@@ -1319,6 +1319,7 @@  if test "$vnc_png" != "no" ; then
 cat > $TMPC <<EOF
 //#include <stdio.h>
 #include <png.h>
+#include <stddef.h>
 int main(void) {
     png_structp png_ptr;
     png_ptr = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
@@ -1503,6 +1504,7 @@  if test "$brlapi" != "no" ; then
   brlapi_libs="-lbrlapi"
   cat > $TMPC << EOF
 #include <brlapi.h>
+#include <stddef.h>
 int main( void ) { return brlapi__openConnection (NULL, NULL, NULL); }
 EOF
   if compile_prog "" "$brlapi_libs" ; then
@@ -1737,6 +1739,7 @@  if test "$linux_aio" != "no" ; then
   cat > $TMPC <<EOF
 #include <libaio.h>
 #include <sys/eventfd.h>
+#include <stddef.h>
 int main(void) { io_setup(0, NULL); io_set_eventfd(NULL, 0); eventfd(0, 0); return 0; }
 EOF
   if compile_prog "" "-laio" ; then
@@ -2094,6 +2097,7 @@  madvise=no
 cat > $TMPC << EOF
 #include <sys/types.h>
 #include <sys/mman.h>
+#include <stddef.h>
 int main(void) { return madvise(NULL, 0, MADV_DONTNEED); }
 EOF
 if compile_prog "" "" ; then
@@ -2106,6 +2110,7 @@  fi
 posix_madvise=no
 cat > $TMPC << EOF
 #include <sys/mman.h>
+#include <stddef.h>
 int main(void) { return posix_madvise(NULL, 0, POSIX_MADV_DONTNEED); }
 EOF
 if compile_prog "" "" ; then