[bug #61734] [man] localize like other packages

[Dave]

Follow-up Comment #7, bug #61734 (project groff):

I believe you that this is fixed for real in -man, but the -me change does not
seem entirely valid.  The new code added:

.\" Set package default hyphenation mode, but override it with groff's
.\" localized value if available.
.hy 6
.do hy \\n[\\*[locale]*hyphenation-mode-trap]

says in the comment that it conditionally uses groff's localization if
available, but in the code unconditionally overwrites the ".hy 6" with
whatever is in the localization register--which, if it doesn't exist, is
interpolated as 0, almost certainly not what is intended.

The straightforward solution for groff is to sequester the second .hy call
behind an ".if r" on the register.

This breaks the package's compatibility with other *roffs, which probably lack
the r conditional (even Heirloom, which has numerous groff extensions, lacks
it), torpedoing one of the stated reasons for the dual .hy setting.  There may
be some clever way I'm not thinking of to preserve that compatibility, but at
least the groff behavior should be correct in the case where no localization
register is set.

This maybe should be a new bug report, since the current one is primarily
concerned with -man, but I thought I'd float it here first since this is where
the commit was registered.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?61734>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/