gbranden pushed a commit to branch master
in repository groff.

commit 95559f02feac3c742c2e5581a2454e29a519128b
Author: G. Branden Robinson <[email protected]>
AuthorDate: Sun Mar 15 10:34:41 2026 -0500

    Deprecate --enable-groff-allocator config option.
    
    * m4/groff.m4 (GROFF_USE_GROFF_ALLOCATOR): Update help string to mark
      `--enable-groff-allocator` configuration option as deprecated.
    
      (GROFF_ALLOCATOR_NOTICE): New macro warns user of option's planned
      withdrawal if they have selected it.
    
    * configure.ac: Interpolate `GROFF_ALLOCATOR_NOTICE` macro.
    
    * NEWS: Document this.
    
    Fixes <https://savannah.gnu.org/bugs/?67936>.
---
 ChangeLog    | 13 +++++++++++++
 NEWS         | 10 ++++++++++
 configure.ac |  1 +
 m4/groff.m4  | 17 ++++++++++++++++-
 4 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 030b2211f..5d0d8d21b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2026-03-15  G. Branden Robinson <[email protected]>
+
+       * m4/groff.m4 (GROFF_USE_GROFF_ALLOCATOR): Update help string to
+       mark `--enable-groff-allocator` configuration option as
+       deprecated.
+       (GROFF_ALLOCATOR_NOTICE): New macro warns user of option's
+       planned withdrawal if they have selected it.
+       * configure.ac: Interpolate `GROFF_ALLOCATOR_NOTICE` macro.
+
+       * NEWS: Document this.
+
+       Fixes <https://savannah.gnu.org/bugs/?67936>.
+
 2026-03-15  G. Branden Robinson <[email protected]>
 
        * configure.ac: Stop reporting use of libgroff's custom memory
diff --git a/NEWS b/NEWS
index b4790c877..4a29775d7 100644
--- a/NEWS
+++ b/NEWS
@@ -26,6 +26,16 @@ troff
    input that is non-idiomatic or likely to produce an unexpected result
    (or none at all), but is not invalid.
 
+Miscellaneous
+-------------
+
+*  The 'configure' options '--{en,dis}able-groff-allocator' introduced
+   in groff 1.23.0 are now deprecated.  `--disable-groff-allocator` has
+   been implicit since that release, and we've received no reports of a
+   C++ runtime unable to handle groff's demands on its heap/"free store"
+   allocator.  We expect to withdraw these configuration options
+   completely in groff 1.26.
+
 
 VERSION 1.24.1
 ==============
diff --git a/configure.ac b/configure.ac
index fda719be6..e90ce446f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -249,3 +249,4 @@ GROFF_URW_FONTS_NOTICE
 GROFF_UCHARDET_NOTICE
 GROFF_GROHTML_PROGRAM_NOTICE
 GROFF_MAKEINFO_PROGRAM_NOTICE
+GROFF_ALLOCATOR_NOTICE
diff --git a/m4/groff.m4 b/m4/groff.m4
index d5d1b37e7..e0b81a9ef 100644
--- a/m4/groff.m4
+++ b/m4/groff.m4
@@ -1834,11 +1834,26 @@ AC_DEFUN([GROFF_UCHARDET_NOTICE], [
   fi
 ])
 
+dnl TODO: Drop this macro in groff 1.26.
+AC_DEFUN([GROFF_ALLOCATOR_NOTICE], [
+  if test "$groff_use_own_allocator" = yes
+  then
+    AC_MSG_NOTICE([Use of groff's memory allocator is deprecated.
+
+  We expect to drop support for groff's replacement allocator for C++'s
+  'new' and 'delete' operators completely in groff 1.26.  Please contact
+  the groff at gnu dot org mailing list if your C++ language runtime
+  does not reliably allocate or free memory from the heap or free store.
+  ])
+  fi
+])
+
 
+dnl TODO: Drop this macro in groff 1.26.
 AC_DEFUN([GROFF_USE_GROFF_ALLOCATOR], [
   AC_ARG_ENABLE([groff-allocator],
     [AS_HELP_STRING([--enable-groff-allocator], [enable libgroff's \
-allocator for C++ new/delete])],
+allocator for C++ new/delete (DEPRECATED)])],
     [test "$enableval" = yes && groff_use_own_allocator=yes],
     [groff_use_own_allocator=no])
 ])

_______________________________________________
groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit

Reply via email to