groff-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[groff] 01/04: Apply contrib ChangeLog discipline.


From: G. Branden Robinson
Subject: [groff] 01/04: Apply contrib ChangeLog discipline.
Date: Mon, 31 May 2021 08:15:57 -0400 (EDT)

gbranden pushed a commit to branch master
in repository groff.

commit ba329a0aaedcf5716ebbb8ac248663e6a2519a73
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
AuthorDate: Mon May 31 01:52:03 2021 +1000

    Apply contrib ChangeLog discipline.
---
 ChangeLog                         | 128 --------------------------------------
 contrib/chem/ChangeLog            |   6 ++
 contrib/glilypond/ChangeLog       |  31 +++++++++
 contrib/gperl/ChangeLog           |  28 +++++++++
 contrib/gpinyin/ChangeLog         |  67 ++++++++++++++------
 contrib/groff_filenames/ChangeLog |   4 ++
 contrib/mm/ChangeLog              | 105 +++++++++++++++++++++++++++++++
 contrib/pdfmark/ChangeLog         |   9 +++
 contrib/rfc1345/ChangeLog         |  20 ++++++
 9 files changed, 250 insertions(+), 148 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index f11aebf..4c295ff 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,109 +1,3 @@
-2021-05-31  G. Branden Robinson <g.branden.robinson@gmail.com>
-
-       * contrib/mm/groff_mm.7.man (Description/Macros) <LT>: Document
-       the "SP" (simplified) letter style more completely.  Salutations
-       and formal closings are omitted.
-
-       Solves the remaining part of bug #60373, "the formal closing is
-       omitted completely".  And, regarding the salutation, #60390.
-
-       Fixes <https://savannah.gnu.org/bugs/?60373> and
-       <https://savannah.gnu.org/bugs/index.php?60390>.
-
-2021-05-30  Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
-
-       * contrib/mm/m.tmac (let@sg_SP): Stop placing a comma after the
-       first argument (an author name set by AU) if the second argument
-       is empty (an author title set by AT).
-
-       Solves another part of bug #60373, "the writer's name at the
-       bottom is followed by a trailing comma".
-
-       Fixes <https://savannah.gnu.org/bugs/?60389>.
-
-2021-05-30  Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
-
-       * contrib/mm/m.tmac (let@print-head): Quote the interpolation of
-       the string `let*lo-SJ`.
-
-       Solves part of bug #60373, "[t]he subject line shows only the
-       first word".
-
-       Fixes <https://savannah.gnu.org/bugs/?60388>.
-
-2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
-
-       Add regression tests for Savannah #60373.
-
-       * contrib/mm/tests/LT_SP_AU_without_AT_works.sh:
-       * contrib/mm/tests/LT_SP_multi-word_LO_SJ_works.sh: Add tests.
-       * contrib/mm/mm.am (mm_TESTS): Run tests.
-
-2021-05-30  Tadziu Hoffman <hoffmann@usm.uni-muenchen.de>
-
-       Fix Savannah #57034.
-
-       * contrib/mm/m.tmac (AT): Count author title declarations.
-       (let*mt-sign): Use correct index when iterating author names.
-       Prevent page break between author names and titles.  Iterate
-       author titles and write them.
-
-       Fixes <https://savannah.gnu.org/bugs/?57034>.
-
-2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
-
-       Add regression test for Savannah #57034.
-
-       * contrib/mm/tests/MT_5_includes_AT_in_SG.sh: Test it.  Thanks
-       to Ken Mandelberg for the reproducer.
-       * contrib/mm/mm.am (mm_TESTS): Add variable storing script name.
-       (TESTS): Append `mm_TESTS` to run it.
-       (EXTRA_DIST):  Append `mm_TESTS` to ship it.
-
-2021-05-30  Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
-
-       * contrib/mm/mm/5.MT (cov@print-title): Fix missing diagnostic.
-       Memorandum type 5 now requires a title to be declared with .TL.
-
-       Fixes <https://savannah.gnu.org/bugs/?59563>.
-
-2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
-
-       [mm]: Cease warning level manipulation.
-
-       Stop mm macro package from manipulating warnings.
-
-       * contrib/mm/m.tmac: Remove code that attempted to enable all
-       warnings if none were given on the command line.  It did this by
-       simply comparing the value of the warning register (\n[.warn])
-       against the default value; but of course, a user could specify
-       -w options that exactly matched the default and the test would
-       not be able to tell, causing puzzling and undesired behavior.
-       Furthermore, the hard-coded default was out of date and did not
-       correspond to recent releases of groff.  If you want all
-       warnings on, use the ".warn" request with no arguments in your
-       mm document or pass "-w w" to groff (see troff(1) or the Texinfo
-       manual for more on warnings).
-
-       See commit 5aa934e7, 20 February 2020.
-
-2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
-
-       * contrib/mm/examples/letter.mm: Revise to be a better example:
-       use macro package more effectively, follow *roff input
-       conventions more carefully, and incorporate more accurate
-       comments.
-
-2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
-
-       * contrib/mm/m.tmac: Update diagnostics.  When aborting,
-       explicitly tell the user we are doing so (see
-       a334cc97abbdfb9c41e28fcd7c187b81a0a3ceac, 25 September 2020).
-       (@mm): Define new string to hold prefix for diagnostic messages.
-       (@warning, @error): Use @mm.  Reorder diagnostic messages to
-       conform with GNU Coding Standards.
-       (OK, PM): Use @warning instead of `tm` request directly.
-
 2021-05-29  G. Branden Robinson <g.branden.robinson@gmail.com>
 
        * tmac/papersize.tmac: Improve diagnostic message; when
@@ -1451,14 +1345,6 @@
 
 2021-02-03  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       [rfc1345]: Add smoke test.
-
-       * contrib/rfc1345/tests/rfc1345-smoke-test.sh: Add test.
-       * contrib/rfc1345/rfc1345.am (rfc1345_TESTS, TESTS, EXTRA_DIST):
-       Run test.
-
-2021-02-03  G. Branden Robinson <g.branden.robinson@gmail.com>
-
        * test-groff.in: Add support for rfc1345.tmac.
 
 2021-02-02  G. Branden Robinson <g.branden.robinson@gmail.com>
@@ -1468,20 +1354,6 @@
        * contrib/rfc1345/rfc1345.am: Do it.
        * Makefile.am: Include contrib/rfc1345/rfc1345.am.
 
-2021-02-02  Dorai Sitaram <ds26gte@yahoo.com>
-
-       contrib/rfc1345: Add new macro file.
-
-       Add new macro file that defines special characters implementing
-       RFC 1345 mnemonics (plus some additions from Vim, which itself
-       uses RFC 1345 for its digraphs).
-
-       * contrib/rfc1345/COPYRIGHT:
-       * contrib/rfc1345/groff_rfc1345.7.man:
-       * contrib/rfc1345/rfc1345.tmac: Add new macro package.
-
-       Fixes <https://savannah.gnu.org/bugs/?59839>.
-
 2021-02-01  G. Branden Robinson <g.branden.robinson@gmail.com>
 
        * src/roff/troff/input.cpp (open_mac_file): Report problem when
diff --git a/contrib/chem/ChangeLog b/contrib/chem/ChangeLog
index 0dbd1ea..7f85775 100644
--- a/contrib/chem/ChangeLog
+++ b/contrib/chem/ChangeLog
@@ -1,3 +1,9 @@
+2020-04-22  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * chem.1.man:
+       * examples/122/README.txt:
+       * examples/README.txt: Delete references to groffer.
+
 2018-02-28  Werner LEMBERG  <wl@gnu.org>
 
        * chem.am (chem, README): Use $(AM_V_GEN) to silence file generation.
diff --git a/contrib/glilypond/ChangeLog b/contrib/glilypond/ChangeLog
index 31bd34f..d9cc70f 100644
--- a/contrib/glilypond/ChangeLog
+++ b/contrib/glilypond/ChangeLog
@@ -1,3 +1,34 @@
+2021-01-06  Colin Watson <cjwatson@debian.org>
+
+       * glilypond.pl: Avoid Perl's unsafe "<>" operator.
+
+       The "<>" operator is implemented using the two-argument form of
+       "open", which interprets magic such as pipe characters, allowing
+       execution of arbitrary commands which is unlikely to be
+       expected.  Perl >= 5.22 has a "<<>>" operator which avoids this,
+       but also forbids the use of "-" to mean the standard input,
+       which is a facility that the affected groff programs document.
+
+       ARGV::readonly would probably also fix this, but I fundamentally
+       dislike the approach of escaping data in preparation for a
+       language facility to unescape it, especially when the required
+       escaping is as non-obvious as it is here.  (For the same reason,
+       I prefer to use subprocess invocation facilities that allow
+       passing the argument list as a list rather than as a string to
+       be interpreted by the shell.)  So I've abandoned this dubious
+       convenience and changed the affected programs to iterate over
+       command-line arguments manually using the three-argument form of
+       open.
+
+       glilypond doesn't need the initial unshift since that's already
+       handled in args.pl.
+
+       Fixes <https://savannah.gnu.org/bugs/?55557>.
+
+2020-04-22  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * glilypond.1.man: Delete references to groffer.
+
 2018-02-28  Werner LEMBERG  <wl@gnu.org>
 
        * glilypond.am (glilypond): Use $(AM_V_GEN) to silence file generation.
diff --git a/contrib/gperl/ChangeLog b/contrib/gperl/ChangeLog
index c47fc1f..3e8de55 100644
--- a/contrib/gperl/ChangeLog
+++ b/contrib/gperl/ChangeLog
@@ -1,3 +1,31 @@
+2021-01-06  Colin Watson <cjwatson@debian.org>
+
+       * gperl.pl: Avoid Perl's unsafe "<>" operator.
+
+       The "<>" operator is implemented using the two-argument form of
+       "open", which interprets magic such as pipe characters, allowing
+       execution of arbitrary commands which is unlikely to be
+       expected.  Perl >= 5.22 has a "<<>>" operator which avoids this,
+       but also forbids the use of "-" to mean the standard input,
+       which is a facility that the affected groff programs document.
+
+       ARGV::readonly would probably also fix this, but I fundamentally
+       dislike the approach of escaping data in preparation for a
+       language facility to unescape it, especially when the required
+       escaping is as non-obvious as it is here.  (For the same reason,
+       I prefer to use subprocess invocation facilities that allow
+       passing the argument list as a list rather than as a string to
+       be interpreted by the shell.)  So I've abandoned this dubious
+       convenience and changed the affected programs to iterate over
+       command-line arguments manually using the three-argument form of
+       open.
+
+       Fixes <https://savannah.gnu.org/bugs/?55557>.
+
+2020-04-22  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * gperl.1.man: Delete references to groffer.
+
 2018-02-28  Werner LEMBERG  <wl@gnu.org>
 
        * gperl.am (gperl): Use $(AM_V_GEN) to silence file generation.
diff --git a/contrib/gpinyin/ChangeLog b/contrib/gpinyin/ChangeLog
index 87d2699..cdccfa6 100644
--- a/contrib/gpinyin/ChangeLog
+++ b/contrib/gpinyin/ChangeLog
@@ -1,26 +1,26 @@
 2021-05-11  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       * contrib/gpinyin/gpinyin.1.man: Render the tone mark table only
-       if the output device might be capable.
+       * gpinyin.1.man: Render the tone mark table only if the output
+       device might be capable.
 
        Fixes <https://savannah.gnu.org/bugs/index.php?55215>.
 
 2021-05-10  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       * contrib/gpinyin/gpinyin.1.man: Work around inability of grops
-       and gropdf to construct some Unicode composite characters.  Use
-       groff composite special characters for "a" with acute and grave
+       * gpinyin.1.man: Work around inability of grops and gropdf to
+       construct some Unicode composite characters.  Use groff
+       composite special characters for "a" with acute and grave
        accents, and define strings for "a" with macron (overline) and
        with háček/caron accents.  Use different string definitions for
        nroff and troff modes so we don't regress UTF-8 terminal output.
 
 2021-05-10  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       * contrib/gpinyin/subs.pl (vowel_t): Fix incorrect rendering of
-       base glyph 'U', which was being forced to lowercase when a
-       dieresis and tone mark were both being applied.  This caused the
-       tone mark to overlap the dieresis, so decrease the font size of
-       the base glyph even more to compensate.  The result is ugly but
+       * subs.pl (vowel_t): Fix incorrect rendering of base glyph 'U',
+       which was being forced to lowercase when a dieresis and tone
+       mark were both being applied.  This caused the tone mark to
+       overlap the dieresis, so decrease the font size of the base
+       glyph even more to compensate.  The result is ugly but
        comprehensible.
 
        See <https://savannah.gnu.org/bugs/?57524>, partially mitigated
@@ -28,26 +28,25 @@
 
 2021-05-09  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       * contrib/gpinyin/subs.pl (%tones1_Unicode): Fix copy and paste
-       error.  Emit U+01D5 (Latin capital letter u with dieresis and
-       macron) for Ü with tone 1, instead of U+016A (Latin capital
-       letter u with macron).
+       * subs.pl (%tones1_Unicode): Fix copy and paste error.  Emit
+       U+01D5 (Latin capital letter u with dieresis and macron) for Ü
+       with tone 1, instead of U+016A (Latin capital letter u with
+       macron).
 
        Fixes <https://savannah.gnu.org/bugs/?60562>.
 
 2021-05-09  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       * contrib/gpinyin/subs.pl (handle_word): Emit \[cq] instead of
-       \[aq] when interpolating an apostrophe before a vowel.
+       * subs.pl (handle_word): Emit \[cq] instead of \[aq] when
+       interpolating an apostrophe before a vowel.
 
        Fixes <https://savannah.gnu.org/bugs/?60561>.
 
 2021-05-09  G. Branden Robinson <g.branden.robinson@gmail.com>
 
-       * contrib/gpinyin/subs.pl (%tones_glyphs, %tones4_glyphs): Fix
-       hash keys to use the groff dotless i special character into
-       which a lowercase "i" has already been transformed instead of
-       'i' itself.
+       * subs.pl (%tones_glyphs, %tones4_glyphs): Fix hash keys to use
+       the groff dotless i special character into which a lowercase "i"
+       has already been transformed instead of 'i' itself.
        (vowel_n, vowel_t): Rename variable so that tone-transformed
        vowel is stored separately.  Add "or warn" to test the result
        and cheaply assert that we got a string back from our hash
@@ -55,6 +54,34 @@
 
        Fixes <https://savannah.gnu.org/bugs/?60560>.
 
+2021-01-06  Colin Watson <cjwatson@debian.org>
+
+       * gpinyin.pl: Avoid Perl's unsafe "<>" operator.
+
+       The "<>" operator is implemented using the two-argument form of
+       "open", which interprets magic such as pipe characters, allowing
+       execution of arbitrary commands which is unlikely to be
+       expected.  Perl >= 5.22 has a "<<>>" operator which avoids this,
+       but also forbids the use of "-" to mean the standard input,
+       which is a facility that the affected groff programs document.
+
+       ARGV::readonly would probably also fix this, but I fundamentally
+       dislike the approach of escaping data in preparation for a
+       language facility to unescape it, especially when the required
+       escaping is as non-obvious as it is here.  (For the same reason,
+       I prefer to use subprocess invocation facilities that allow
+       passing the argument list as a list rather than as a string to
+       be interpreted by the shell.)  So I've abandoned this dubious
+       convenience and changed the affected programs to iterate over
+       command-line arguments manually using the three-argument form of
+       open.
+
+       Fixes <https://savannah.gnu.org/bugs/?55557>.
+
+2020-04-22  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * gpinyin.1.man: Delete references to groffer.
+
 2018-02-28  Werner LEMBERG  <wl@gnu.org>
 
        * gpinyin.am (gpinyin): Use $(AM_V_GEN) to silence file generation.
diff --git a/contrib/groff_filenames/ChangeLog 
b/contrib/groff_filenames/ChangeLog
index ae4ad6a..9cc36d9 100644
--- a/contrib/groff_filenames/ChangeLog
+++ b/contrib/groff_filenames/ChangeLog
@@ -1,3 +1,7 @@
+2020-04-22  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * groff_filenames.5.man: Delete references to groffer.
+
 2015-08-22  Bernd Warken  <groff-bernd.warken-72@web.de>
 
        * groff_filenames.5.man: Use section 5 within this man-page.
diff --git a/contrib/mm/ChangeLog b/contrib/mm/ChangeLog
index d135891..423f975 100644
--- a/contrib/mm/ChangeLog
+++ b/contrib/mm/ChangeLog
@@ -1,3 +1,108 @@
+2021-05-31  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * groff_mm.7.man (Description/Macros) <LT>: Document the "SP"
+       {simplified} letter style more completely.  Salutations and
+       formal closings are omitted.
+
+       Solves the remaining part of bug #60373, "the formal closing is
+       omitted completely".  And, regarding the salutation, #60390.
+
+       Fixes <https://savannah.gnu.org/bugs/?60373> and
+       <https://savannah.gnu.org/bugs/index.php?60390>.
+
+2021-05-30  Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
+
+       * m.tmac (let@sg_SP): Stop placing a comma after the first
+       argument (an author name set by AU) if the second argument is
+       empty (an author title set by AT).
+
+       Solves another part of bug #60373, "the writer's name at the
+       bottom is followed by a trailing comma".
+
+       Fixes <https://savannah.gnu.org/bugs/?60389>.
+
+2021-05-30  Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
+
+       * m.tmac (let@print-head): Quote the interpolation of the string
+       `let*lo-SJ`.
+
+       Solves part of bug #60373, "[t]he subject line shows only the
+       first word".
+
+       Fixes <https://savannah.gnu.org/bugs/?60388>.
+
+2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       Add regression tests for Savannah #60373.
+
+       * tests/LT_SP_AU_without_AT_works.sh:
+       * tests/LT_SP_multi-word_LO_SJ_works.sh: Add tests.
+       * mm.am (mm_TESTS): Run tests.
+
+2021-05-30  Tadziu Hoffman <hoffmann@usm.uni-muenchen.de>
+
+       Fix Savannah #57034.
+
+       * m.tmac (AT): Count author title declarations.
+       (let*mt-sign): Use correct index when iterating author names.
+       Prevent page break between author names and titles.  Iterate
+       author titles and write them.
+
+       Fixes <https://savannah.gnu.org/bugs/?57034>.
+
+2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       Add regression test for Savannah #57034.
+
+       * tests/MT_5_includes_AT_in_SG.sh: Test it.  Thanks to Ken
+       Mandelberg for the reproducer.
+       * mm.am (mm_TESTS): Add variable storing script name.
+       (TESTS): Append `mm_TESTS` to run it.
+       (EXTRA_DIST):  Append `mm_TESTS` to ship it.
+
+2021-05-30  Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
+
+       * mm/5.MT (cov@print-title): Fix missing diagnostic.  Memorandum
+       type 5 now requires a title to be declared with .TL.
+
+       Fixes <https://savannah.gnu.org/bugs/?59563>.
+
+2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       [mm]: Cease warning level manipulation.
+
+       Stop mm macro package from manipulating warnings.
+
+       * m.tmac: Remove code that attempted to enable all warnings if
+       none were given on the command line.  It did this by simply
+       comparing the value of the warning register (\n[.warn]) against
+       the default value; but of course, a user could specify -w
+       options that exactly matched the default and the test would not
+       be able to tell, causing puzzling and undesired behavior.
+       Furthermore, the hard-coded default was out of date and did not
+       correspond to recent releases of groff.  If you want all
+       warnings on, use the ".warn" request with no arguments in your
+       mm document or pass "-w w" to groff (see troff(1) or the Texinfo
+       manual for more on warnings).
+
+       See commit 5aa934e7, 20 February 2020.
+
+2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * examples/letter.mm: Revise to be a better example: use macro
+       package more effectively, follow *roff input conventions more
+       carefully, and incorporate more accurate comments.
+
+2021-05-30  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * m.tmac: Update diagnostics.  When aborting, explicitly tell
+       the user we are doing so (see
+       a334cc97abbdfb9c41e28fcd7c187b81a0a3ceac, 25 September 2020).
+       (@mm): Define new string to hold prefix for diagnostic messages.
+       (@warning, @error): Use @mm.  Reorder diagnostic messages to
+       conform with GNU Coding Standards.
+       (OK, PM): Use @warning instead of `tm` request directly.
+
 2019-09-10  G. Branden Robinson <g.branden.robinson@gmail.com>
 
        * Makefile.sim: Delete sed transformation of @G@.  It is no
diff --git a/contrib/pdfmark/ChangeLog b/contrib/pdfmark/ChangeLog
index f813fc3..83487cb 100644
--- a/contrib/pdfmark/ChangeLog
+++ b/contrib/pdfmark/ChangeLog
@@ -1,3 +1,12 @@
+2020-12-25  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * pdfmark.am (PDFROFF): Call pdfroff without
+       `--keep-temporary-files` option.  Temporary directories are
+       created with mktemp(1) and files with an embedded process
+       identifier, which frustrates reproducible builds.
+
+       See <https://savannah.gnu.org/bugs/?57218>.
+
 2018-02-28  Werner LEMBERG  <wl@gnu.org>
 
        * pdfmark.am (pdfmark.pdf): Use $(GROFF_V).
diff --git a/contrib/rfc1345/ChangeLog b/contrib/rfc1345/ChangeLog
new file mode 100644
index 0000000..69a490d
--- /dev/null
+++ b/contrib/rfc1345/ChangeLog
@@ -0,0 +1,20 @@
+2021-02-03  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       [rfc1345]: Add smoke test.
+
+       * tests/rfc1345-smoke-test.sh: Add test.
+       * rfc1345.am (rfc1345_TESTS, TESTS, EXTRA_DIST): Run test.
+
+2021-02-02  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       Integrate rfc1345.tmac into build system.
+
+       * rfc1345.am: Do it.
+
+2021-02-02  Dorai Sitaram <ds26gte@yahoo.com>
+
+       * COPYRIGHT:
+       * groff_rfc1345.7.man:
+       * rfc1345.tmac: Add new macro package.
+
+       Fixes <https://savannah.gnu.org/bugs/?59839>.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]