[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #57873] grog mangles explicitly given groff options
From: |
G. Branden Robinson |
Subject: |
[bug #57873] grog mangles explicitly given groff options |
Date: |
Tue, 29 Jun 2021 00:27:39 -0400 (EDT) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0 |
Update of bug #57873 (project groff):
Status: In Progress => Fixed
Open/Closed: Open => Closed
Planned Release: None => 1.23.0
_______________________________________________________
Follow-up Comment #4:
commit 6145868098b56a77b5e585f0ee3ec04ab73c5e66
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date: Tue Jun 29 14:11:14 2021 +1000
[grog]: Refactor command-line argument handling.
* src/utils/grog/grog.pl:
- Drop scalars `groff_opts`, `device`, and `with_warnings`.
- Move scalar `pdf_with_ligatures`...
(process_arguments): ...here. Recognize all groff options that take
an optionally whitespace-separated option argument and apply the
existing "delayed option" handling to them. Push any groff option
except `-m` as-is onto the constructed groff command's argument list
(`-m` handling is unchanged). Match long option names exactly, not
sloppily. Stop recognizing `--with-warnings` option. Emit error
diagnostic if unrecognized long option encountered.
(infer_device): Delete subroutine and its top-level call site.
(help): Undocument `--warnings`.
* src/utils/grog/grog.1.man:
(Synopsis): Use font style macros instead of .OP. Undocument
`--warnings`.
(Options): Note that the groff arguments produced by the grog
`--ligatures` option are supported only by the `pdf` device.
Undocument `--warnings`. Undocument groff-incompatible restriction on
whitespace before option arguments; grog is compatible now. Simplify
discussion in light of simplified grog logic.
(Details): Undocument scenario where grog infers multiple
main/major/full-service macro packages; this no longer happens.
(Examples): Update to no longer illustrate includion of `-T ps`
option; this no longer happens by default.
* NEWS: Document removal of grog's `--warnings` option.
Fixes <https://savannah.gnu.org/bugs/?57873>; groff options are
no longer "mangled".
commit ee76f84c0ebb855b6a0bb8f6d6ed49520619b32d
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date: Tue Jun 29 13:44:44 2021 +1000
Add regression test for Savannah #57873.
* src/utils/grog/tests/preserve-groff-options.sh: Test it.
* src/utils/grog/grog.am (grog_TESTS): Run test.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?57873>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/