[bug #52444] Half the man pages save and restore compatibility mode like macro packages

[G. Branden Robinson]

Update of bug #52444 (project groff):

                  Status:                    None => Need Info              
             Assigned to:                    None => gbranden               

    _______________________________________________________

Follow-up Comment #3:

Okay, Werner's hint to cast git blame led me to the answer.

https://savannah.gnu.org/bugs/index.php?44708

I don't understand how or why the original bug submitter had compatibility
mode on by default, but that may not matter.

If I "git log 8ac03ed2a5e1672754749df87c1742e7d161e32c" I see a major effort
by Werner to make all the man pages work in compatibility mode.  Rather than
rewrite them all in a portable subset, this trick was done.

A quick check reveals that a couple of pages have problems.  One I introduced
on 2 November, and I'll commit a fix for it after I've done some tests.

The other one was around even back then.  What's the best way to fix it?


$ for M in $(find -name "*man"); do ./test-groff -C -t -p -mandoc $M
>/dev/null; done
./src/preproc/soelim/soelim.1.man:2: warning: macro 'ms' not defined
$ git blame ./src/preproc/soelim/soelim.1.man |head
d12c7ba3c src/preproc/soelim/soelim.man   (Werner LEMBERG      2004-08-07
04:52:54 +0000   1) '\" p
c1886fcaa src/preproc/soelim/soelim.man   (Bernd Warken        2014-09-18
01:41:01 +0200   2) .mso pic.tmac
c1886fcaa src/preproc/soelim/soelim.man   (Bernd Warken        2014-09-18
01:41:01 +0200   3) .TH @address@hidden @MAN1EXT@ "@MDATE@" "Groff Version
@VERSION@"
c1886fcaa src/preproc/soelim/soelim.man   (Bernd Warken        2014-09-18
01:41:01 +0200   4) .SH NAME
c1886fcaa src/preproc/soelim/soelim.man   (Bernd Warken        2014-09-18
01:41:01 +0200   5) @address@hidden \- interpret .so requests in groff input
c1886fcaa src/preproc/soelim/soelim.man   (Bernd Warken        2014-09-18
01:41:01 +0200   6) .
c1886fcaa src/preproc/soelim/soelim.man   (Bernd Warken        2014-09-18
01:41:01 +0200   7) .
8ac03ed2a src/preproc/soelim/soelim.man   (Werner Lemberg      2015-04-03
16:55:03 +0200   8) .do nr groff_C \n[.C]
8ac03ed2a src/preproc/soelim/soelim.man   (Werner Lemberg      2015-04-03
16:55:03 +0200   9) .cp 0
8ac03ed2a src/preproc/soelim/soelim.man   (Werner Lemberg      2015-04-03
16:55:03 +0200  10) .


And here's my patch for the other problem:


diff --git a/contrib/gperl/gperl.1.man b/contrib/gperl/gperl.1.man
index f6f341a2..818e4cd1 100644
--- a/contrib/gperl/gperl.1.man
+++ b/contrib/gperl/gperl.1.man
@@ -267,7 +267,7 @@ generates a
 storage line
 .RS
 .EX
-\fB.ds \fIvar_name content
+.BI .ds " var_name content"
 .EE
 .RE
 .
@@ -276,7 +276,7 @@ In
 the following groff command is generated
 .RS
 .EX
-\fB.nr \fIvar_name content
+.BI .nr " var_name content"
 .EE
 .RE
 .







    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?52444>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/