]> Savannah Git Hosting - gnulib.git/commitdiff
printf-gnu: New module.
authorBruno Haible <bruno@clisp.org>
Fri, 24 Mar 2023 10:55:38 +0000 (11:55 +0100)
committerBruno Haible <bruno@clisp.org>
Fri, 24 Mar 2023 10:55:38 +0000 (11:55 +0100)
* m4/printf-posix.m4: Renamed from m4/printf-posix-rpl.m4.
(gl_FUNC_PRINTF_IS_POSIX): New macro.
(gl_FUNC_PRINTF_POSIX): Require it. Don't require
gl_FUNC_VFPRINTF_POSIX. Invoke gl_PREREQ_VASNPRINTF_WITH_POSIX_EXTRAS
and gl_REPLACE_VASNPRINTF.
* m4/printf-gnu.m4: New file, based on m4/fprintf-posix.m4.
* modules/printf-posix (Files): Update.
* modules/printf-gnu: New file, based on modules/vasnprintf-gnu.
* doc/posix-functions/printf.texi: Mention the new module.

ChangeLog
doc/posix-functions/printf.texi
m4/printf-gnu.m4 [new file with mode: 0644]
m4/printf-posix-rpl.m4 [deleted file]
m4/printf-posix.m4 [new file with mode: 0644]
modules/printf-gnu [new file with mode: 0644]
modules/printf-posix

index aa826b376a96a8c5307e78ece54902c2df058f4e..cbab39bd9dcb7b02dcc42dce18520169f3f8868c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2023-03-24  Bruno Haible  <bruno@clisp.org>
+
+       printf-gnu: New module.
+       * m4/printf-posix.m4: Renamed from m4/printf-posix-rpl.m4.
+       (gl_FUNC_PRINTF_IS_POSIX): New macro.
+       (gl_FUNC_PRINTF_POSIX): Require it. Don't require
+       gl_FUNC_VFPRINTF_POSIX. Invoke gl_PREREQ_VASNPRINTF_WITH_POSIX_EXTRAS
+       and gl_REPLACE_VASNPRINTF.
+       * m4/printf-gnu.m4: New file, based on m4/fprintf-posix.m4.
+       * modules/printf-posix (Files): Update.
+       * modules/printf-gnu: New file, based on modules/vasnprintf-gnu.
+       * doc/posix-functions/printf.texi: Mention the new module.
+
 2023-03-24  Bruno Haible  <bruno@clisp.org>
 
        vprintf-gnu: Add tests.
index 8a3971aa30466d48a94bf724105d49a48ab72a46..382873e9e0261f65612bb97b15a2f039916e3571 100644 (file)
@@ -4,9 +4,9 @@
 
 POSIX specification:@* @url{https://pubs.opengroup.org/onlinepubs/9699919799/functions/printf.html}
 
-Gnulib module: printf-posix or stdio, nonblocking, sigpipe
+Gnulib module: printf-posix or printf-gnu or stdio, nonblocking, sigpipe
 
-Portability problems fixed by Gnulib module @code{printf-posix}:
+Portability problems fixed by either Gnulib module @code{printf-posix} or @code{printf-gnu}:
 @itemize
 @item
 This function does not support size specifiers as in C99 (@code{hh}, @code{ll},
@@ -77,7 +77,14 @@ This function can crash in out-of-memory conditions on some platforms:
 FreeBSD 13.0, NetBSD 5.0.
 @end itemize
 
-Portability problems fixed by Gnulib module @code{stdio} or @code{printf-posix}, together with module @code{nonblocking}:
+Portability problems fixed by Gnulib module @code{printf-gnu}:
+@itemize
+@item
+This function does not support the @samp{B} directive on some platforms:
+glibc 2.34, FreeBSD 13.1, NetBSD 9.0, OpenBSD 7.2, macOS 12.5, AIX 7.2, Solaris 11.4, and others.
+@end itemize
+
+Portability problems fixed by Gnulib module @code{stdio} or @code{printf-posix} or @code{printf-gnu}, together with module @code{nonblocking}:
 @itemize
 @item
 When writing to a non-blocking pipe whose buffer is full, this function fails
@@ -86,7 +93,7 @@ platforms:
 mingw, MSVC 14.
 @end itemize
 
-Portability problems fixed by Gnulib module @code{stdio} or @code{printf-posix}, together with module @code{sigpipe}:
+Portability problems fixed by Gnulib module @code{stdio} or @code{printf-posix} or @code{printf-gnu}, together with module @code{sigpipe}:
 @itemize
 @item
 When writing to a pipe with no readers, this function fails, instead of
diff --git a/m4/printf-gnu.m4 b/m4/printf-gnu.m4
new file mode 100644 (file)
index 0000000..4d8f3c7
--- /dev/null
@@ -0,0 +1,25 @@
+# printf-gnu.m4 serial 1
+dnl Copyright (C) 2023 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.
+
+AC_DEFUN([gl_FUNC_PRINTF_GNU],
+[
+  AC_REQUIRE([gl_FUNC_PRINTF_IS_POSIX])
+  AC_REQUIRE([gl_PRINTF_DIRECTIVE_UPPERCASE_B])
+  gl_cv_func_printf_gnu=no
+  if test $gl_cv_func_printf_posix = yes; then
+    case "$gl_cv_func_printf_directive_uppercase_b" in
+      *yes)
+        # printf is already POSIX compliant and GNU compatible.
+        gl_cv_func_printf_gnu=yes
+        ;;
+    esac
+  fi
+  if test $gl_cv_func_printf_gnu = no; then
+    gl_PREREQ_VASNPRINTF_WITH_GNU_EXTRAS
+    gl_REPLACE_VASNPRINTF
+    gl_REPLACE_PRINTF
+  fi
+])
diff --git a/m4/printf-posix-rpl.m4 b/m4/printf-posix-rpl.m4
deleted file mode 100644 (file)
index 36156d4..0000000
+++ /dev/null
@@ -1,26 +0,0 @@
-# printf-posix-rpl.m4 serial 4
-dnl Copyright (C) 2007-2023 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.
-
-AC_DEFUN([gl_FUNC_PRINTF_POSIX],
-[
-  AC_REQUIRE([gl_FUNC_VFPRINTF_POSIX])
-  if test $gl_cv_func_vfprintf_posix = no; then
-    gl_REPLACE_PRINTF
-  fi
-])
-
-AC_DEFUN([gl_REPLACE_PRINTF],
-[
-  AC_REQUIRE([gl_STDIO_H_DEFAULTS])
-  AC_REQUIRE([gl_ASM_SYMBOL_PREFIX])
-  AC_LIBOBJ([printf])
-  REPLACE_PRINTF=1
-  AC_DEFINE([REPLACE_PRINTF_POSIX], [1],
-    [Define if printf is overridden by a POSIX compliant gnulib implementation.])
-  gl_PREREQ_PRINTF
-])
-
-AC_DEFUN([gl_PREREQ_PRINTF], [:])
diff --git a/m4/printf-posix.m4 b/m4/printf-posix.m4
new file mode 100644 (file)
index 0000000..9aebf40
--- /dev/null
@@ -0,0 +1,36 @@
+# printf-posix.m4 serial 5
+dnl Copyright (C) 2007-2023 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.
+
+AC_DEFUN([gl_FUNC_PRINTF_POSIX],
+[
+  AC_REQUIRE([gl_FUNC_PRINTF_IS_POSIX])
+  if test $gl_cv_func_printf_posix = no; then
+    gl_PREREQ_VASNPRINTF_WITH_POSIX_EXTRAS
+    gl_REPLACE_VASNPRINTF
+    gl_REPLACE_PRINTF
+  fi
+])
+
+dnl Test whether printf is POSIX compliant.
+dnl Result is gl_cv_func_printf_posix.
+AC_DEFUN([gl_FUNC_PRINTF_IS_POSIX],
+[
+  AC_REQUIRE([gl_FUNC_VFPRINTF_IS_POSIX])
+  gl_cv_func_printf_posix="$gl_cv_func_vfprintf_posix"
+])
+
+AC_DEFUN([gl_REPLACE_PRINTF],
+[
+  AC_REQUIRE([gl_STDIO_H_DEFAULTS])
+  AC_REQUIRE([gl_ASM_SYMBOL_PREFIX])
+  AC_LIBOBJ([printf])
+  REPLACE_PRINTF=1
+  AC_DEFINE([REPLACE_PRINTF_POSIX], [1],
+    [Define if printf is overridden by a POSIX compliant gnulib implementation.])
+  gl_PREREQ_PRINTF
+])
+
+AC_DEFUN([gl_PREREQ_PRINTF], [:])
diff --git a/modules/printf-gnu b/modules/printf-gnu
new file mode 100644 (file)
index 0000000..f42a18f
--- /dev/null
@@ -0,0 +1,29 @@
+Description:
+POSIX and GNU compatible printf() function: print formatted output to
+standard output
+
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
+Files:
+m4/printf-gnu.m4
+
+Depends-on:
+printf-posix
+
+configure.ac:
+gl_FUNC_PRINTF_GNU
+
+Makefile.am:
+
+Include:
+<stdio.h>
+
+License:
+LGPL
+
+Maintainer:
+all
index 51e2c1ac2a64ce56d270dfcc839d22cef9efab91..8ad5c7581019b59728f7d3cb9c7baaeafd555317 100644 (file)
@@ -9,7 +9,7 @@ that module occurs among the main modules in lib/.
 
 Files:
 lib/printf.c
-m4/printf-posix-rpl.m4
+m4/printf-posix.m4
 m4/printf.m4
 m4/asm-underscore.m4