[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: _GNU_SOURCE redundant define
From: |
Ludovic Courtès |
Subject: |
Re: _GNU_SOURCE redundant define |
Date: |
Wed, 20 Aug 2008 19:12:24 +0200 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) |
Hi,
address@hidden (Ludovic Courtès) writes:
> The "current Autoconf way" is to use `AC_USE_SYSTEM_EXTENSIONS' (also
> available in 2.61) in lieu of these macros (see
> http://www.gnu.org/software/autoconf/manual/html_node/Obsolete-Macros.html)
> and to never explicitly define the `_*_SOURCE' CPP macros.
>
> I'll change that and test it ASAP. In the meantime, the easiest way is
> to use 2.61, or to remove the offending macro invocations.
Done (patch attached). Greg: can you confirm that you can now happily
use 2.62?
Thanks,
Ludovic.
>From e210266510fb7dc7362a7ca7fb46d3039436c758 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ludovic=20Court=C3=A8s?= <address@hidden>
Date: Wed, 20 Aug 2008 19:02:41 +0200
Subject: [PATCH] Use `AC_USE_SYSTEM_EXTENSIONS' instead of the obsolete
`AC_AIX', etc.
---
ChangeLog | 8 ++++++++
configure.in | 4 +---
libguile/ChangeLog | 7 +++++++
libguile/eval.c | 2 --
libguile/filesys.c | 1 -
libguile/gc.c | 2 --
libguile/numbers.c | 3 ---
libguile/posix.c | 5 +----
libguile/srfi-14.c | 5 +----
libguile/stime.c | 3 +--
libguile/threads.c | 4 +---
11 files changed, 20 insertions(+), 24 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index e80be78..3d8b7e8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2008-08-20 Ludovic Courtès <address@hidden>
+
+ * configure.in: Use `AC_USE_SYSTEM_EXTENSIONS' instead of the
+ obsolete `AC_AIX', `AC_ISC_POSIX' and `AC_MINIX'. As a side
+ effect, this will define `_POSIX_PTHREAD_SEMANTICS' on Solaris,
+ which provides a standard-conforming version of `readdir_r ()'
+ for instance; it also allows the use of Autoconf 2.62.
+
2008-08-14 Ludovic Courtès <address@hidden>
* configure.in: Remove incorrect and pointless
diff --git a/configure.in b/configure.in
index ee3d527..e1656c8 100644
--- a/configure.in
+++ b/configure.in
@@ -73,9 +73,7 @@ AC_PROG_CC
AC_PROG_CPP
AC_PROG_AWK
-AC_AIX
-AC_ISC_POSIX
-AC_MINIX
+AC_USE_SYSTEM_EXTENSIONS
AC_PROG_CC_C89
diff --git a/libguile/ChangeLog b/libguile/ChangeLog
index bfdb798..1b3f5d3 100644
--- a/libguile/ChangeLog
+++ b/libguile/ChangeLog
@@ -1,3 +1,10 @@
+2008-08-20 Ludovic Courtès <address@hidden>
+
+ * eval.c, filesys.c, gc.c, numbers.c, posix.c, srfi-14.c,
+ stime.c, threads.c: Don't define `_GNU_SOURCE' explicitly as
+ it's now defined in <config.h> thanks to
+ `AC_USE_SYSTEM_EXTENSIONS'.
+
2008-08-19 Han-Wen Nienhuys <address@hidden>
Ludovic Courtès <address@hidden>
diff --git a/libguile/eval.c b/libguile/eval.c
index 021640d..8fa29c8 100644
--- a/libguile/eval.c
+++ b/libguile/eval.c
@@ -18,8 +18,6 @@
-#define _GNU_SOURCE
-
/* This file is read twice in order to produce debugging versions of ceval and
* scm_apply. These functions, deval and scm_dapply, are produced when we
* define the preprocessor macro DEVAL. The file is divided into sections
diff --git a/libguile/filesys.c b/libguile/filesys.c
index c3cc70c..ca50e21 100644
--- a/libguile/filesys.c
+++ b/libguile/filesys.c
@@ -19,7 +19,6 @@
/* See stime.c for comments on why _POSIX_C_SOURCE is not always defined. */
-#define _GNU_SOURCE /* ask glibc for everything */
#define _LARGEFILE64_SOURCE /* ask for stat64 etc */
#ifdef __hpux
#define _POSIX_C_SOURCE 199506L /* for readdir_r */
diff --git a/libguile/gc.c b/libguile/gc.c
index 0b6e8b4..dde740f 100644
--- a/libguile/gc.c
+++ b/libguile/gc.c
@@ -15,8 +15,6 @@
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*/
-#define _GNU_SOURCE
-
/* #define DEBUGINFO */
#if HAVE_CONFIG_H
diff --git a/libguile/numbers.c b/libguile/numbers.c
index fc57bf5..20eb16a 100644
--- a/libguile/numbers.c
+++ b/libguile/numbers.c
@@ -40,9 +40,6 @@
*/
-/* tell glibc (2.3) to give prototype for C99 trunc(), csqrt(), etc */
-#define _GNU_SOURCE
-
#if HAVE_CONFIG_H
# include <config.h>
#endif
diff --git a/libguile/posix.c b/libguile/posix.c
index e803f9f..7d53448 100644
--- a/libguile/posix.c
+++ b/libguile/posix.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001, 2002, 2003, 2004, 2005,
2006, 2007 Free Software Foundation, Inc.
+/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001, 2002, 2003, 2004, 2005,
2006, 2007, 2008 Free Software Foundation, Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -21,9 +21,6 @@
# include <config.h>
#endif
-/* Make GNU/Linux libc declare everything it has. */
-#define _GNU_SOURCE
-
#include <stdio.h>
#include <errno.h>
diff --git a/libguile/srfi-14.c b/libguile/srfi-14.c
index f61db7d..3596fbf 100644
--- a/libguile/srfi-14.c
+++ b/libguile/srfi-14.c
@@ -1,6 +1,6 @@
/* srfi-14.c --- SRFI-14 procedures for Guile
*
- * Copyright (C) 2001, 2004, 2006 Free Software Foundation, Inc.
+ * Copyright (C) 2001, 2004, 2006, 2008 Free Software Foundation, Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -21,9 +21,6 @@
# include <config.h>
#endif
-
-#define _GNU_SOURCE /* Ask for `isblank ()'. */
-
#include <string.h>
#include <ctype.h>
diff --git a/libguile/stime.c b/libguile/stime.c
index 8487b91..fa8b585 100644
--- a/libguile/stime.c
+++ b/libguile/stime.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001, 2003, 2004, 2005, 2006,
2007 Free Software Foundation, Inc.
+/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001, 2003, 2004, 2005, 2006,
2007, 2008 Free Software Foundation, Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -31,7 +31,6 @@
what it takes away, and decide from that whether to use it, instead of
hard coding __hpux. */
-#define _GNU_SOURCE /* ask glibc for everything, in particular strptime */
#ifndef _REENTRANT
# define _REENTRANT /* ask solaris for gmtime_r prototype */
#endif
diff --git a/libguile/threads.c b/libguile/threads.c
index 609fc99..843dfca 100644
--- a/libguile/threads.c
+++ b/libguile/threads.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1995,1996,1997,1998,2000,2001, 2002, 2003, 2004, 2005, 2006
Free Software Foundation, Inc.
+/* Copyright (C) 1995,1996,1997,1998,2000,2001, 2002, 2003, 2004, 2005, 2006,
2008 Free Software Foundation, Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -18,8 +18,6 @@
-#define _GNU_SOURCE
-
#include "libguile/_scm.h"
#if HAVE_UNISTD_H
--
1.5.6.2