changeset 9782:1c4f650f9b07

Document a pitfall.
author Bruno Haible <bruno@clisp.org>
date Tue, 11 Mar 2008 05:28:42 +0100
parents 118f0bb3f732
children 49cbb41f6a9b
files ChangeLog m4/absolute-header.m4 m4/include_next.m4
diffstat 3 files changed, 16 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Tue Mar 11 05:25:50 2008 +0100
+++ b/ChangeLog	Tue Mar 11 05:28:42 2008 +0100
@@ -1,3 +1,9 @@
+2008-03-10  Bruno Haible  <bruno@clisp.org>
+
+	* m4/include_next.m4 (gl_CHECK_NEXT_HEADERS): Document a restriction.
+	* m4/absolute-header.m4 (gl_ABSOLUTE_HEADER): Likewise.
+	Reported by Peter O'Gorman <bug-gnulib@mlists.thewrittenword.com>.
+
 2008-03-10  Bruno Haible  <bruno@clisp.org>
 
 	* modules/stdlib (Depends-on): Add include_next, remove
--- a/m4/absolute-header.m4	Tue Mar 11 05:25:50 2008 +0100
+++ b/m4/absolute-header.m4	Tue Mar 11 05:28:42 2008 +0100
@@ -1,5 +1,5 @@
-# absolute-header.m4 serial 7
-dnl Copyright (C) 2006, 2007 Free Software Foundation, Inc.
+# absolute-header.m4 serial 8
+dnl Copyright (C) 2006-2008 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
 dnl with or without modifications, as long as this notice is preserved.
@@ -16,6 +16,9 @@
 # The three "///" are to pacify Sun C 5.8, which otherwise would say
 # "warning: #include of /usr/include/... may be non-portable".
 # Use `""', not `<>', so that the /// cannot be confused with a C99 comment.
+# Note: This macro assumes that the header file is not empty after
+# preprocessing, i.e. it does not only define preprocessor macros but also
+# provides some type/enum definitions or function/variable declarations.
 AC_DEFUN([gl_ABSOLUTE_HEADER],
 [AC_LANG_PREPROC_REQUIRE()dnl
 AC_FOREACH([gl_HEADER_NAME], [$1],
--- a/m4/include_next.m4	Tue Mar 11 05:25:50 2008 +0100
+++ b/m4/include_next.m4	Tue Mar 11 05:28:42 2008 +0100
@@ -1,5 +1,5 @@
-# include_next.m4 serial 4
-dnl Copyright (C) 2006, 2007 Free Software Foundation, Inc.
+# include_next.m4 serial 5
+dnl Copyright (C) 2006-2008 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
 dnl with or without modifications, as long as this notice is preserved.
@@ -63,6 +63,9 @@
 # The three "///" are to pacify Sun C 5.8, which otherwise would say
 # "warning: #include of /usr/include/... may be non-portable".
 # Use `""', not `<>', so that the /// cannot be confused with a C99 comment.
+# Note: This macro assumes that the header file is not empty after
+# preprocessing, i.e. it does not only define preprocessor macros but also
+# provides some type/enum definitions or function/variable declarations.
 AC_DEFUN([gl_CHECK_NEXT_HEADERS],
 [
   AC_REQUIRE([gl_INCLUDE_NEXT])