[Groff] Unintended impact of strip.sed on om.tmac-u?

[G. Branden Robinson]

Hi folks,

I've been inspecting the operation of tmac/strip.sed, and

I noticed that the following substitution:

  s/\([^/]\)doc-/\1/g

has an impact outside what I presume is its intension to shorten mdoc
symbol names.  It also does this:

diff -urN STOCK/om.tmac HACKED2/om.tmac
--- STOCK/om.tmac       2017-11-07 13:00:57.786695535 -0500
+++ HACKED2/om.tmac     2017-11-07 13:13:43.424234377 -0500
@@ -5178,7 +5178,7 @@
 .ds cover-type \\$0
 .substring cover-type 0 2
 .ie '\\*[cover-type]'COV' .ds cover-type cover
-.el .ds cover-type cover
+.el .ds cover-type doc-cover
 .tm1 "[mom]: \\$0 at line \\n[.c] of '\\n[.F]' is no longer valid.
 .tm1 "       Leading of \\*[cover-type] MISC items is now set with 
\\*[replacement], which
 .tm1 "       takes an absolute leading value.  Please update your document.

...which makes the conditional a no-op.

All of the other substitutions in strip.sed are agnostic about symbol
names, and work only at the syntactical level.

Maybe mdoc should get its own sed one-liner that runs separately from
strip.sed?

I think that would be cheaper than expecting all future contributors to
groff to remember not to name any strings or macros doc-whatever if they
want their macro packages stripped.

-- 
Regards,
Branden

signature.asc
Description: PGP signature