bugGNU roff - Bugs: bug #63231, groff-man-pages.utf8.txt: grotty:...

 
 

bug #63231: groff-man-pages.utf8.txt: grotty: output above first line discarded

Submitter:  Bjarni Ingi Gislason <bjarniig>
Submitted:  Mon 17 Oct 2022 10:15:55 PM UTC
   
 
Category:  Driver grotty Severity:  3 - Normal
Item Group:  Warning/Suspicious behaviour Status:  Duplicate
Privacy:  Public Assigned to:  gbranden
Open/Closed:  Closed Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 02 Nov 2022 11:09:04 AM UTC, comment #9: 

Confirmed as a duplicate of bug #62471.

I'm working around it as follows.


diff --git a/tmac/groff_mdoc.7.man b/tmac/groff_mdoc.7.man
index 9547e303c..c6ddfe02d 100644
--- a/tmac/groff_mdoc.7.man
+++ b/tmac/groff_mdoc.7.man
@@ -781,6 +781,8 @@ nor one of the above described keywords,
 the third parameter is used verbatim as the volume name.
 .
 .
+.br
+.ne 10v \" Keep explanatory paragraph with the following table.
 .Pp
 The effects of varying
 .Ql .Dt


G. Branden Robinson <gbranden>
Group administrator
Wed 02 Nov 2022 10:52:37 AM UTC, comment #8: 

Bjarni, it'd be useful if you (A) supplied a unified diff of changes to groff source code and (B) identified the version you're patching.

Fortunately, I was able to reproduce the problem you describe by applying the following patch to what is now master on Savannah.


diff --git a/tmac/groff_mdoc.7.man b/tmac/groff_mdoc.7.man
index 0ac5cd9bd..250fa9976 100644
--- a/tmac/groff_mdoc.7.man
+++ b/tmac/groff_mdoc.7.man
@@ -794,6 +794,11 @@ prevents the digit\~2 from being treated as a manual section number.
 .
 .Pp
 .\" XXX: Figure out how to indent this.
+.\" begin bjarnig
+.rs
+Line before page 217.
+.bp
+.\" end bjarnig
 .TS
 tab(@);
 Lf(CR) | L C R.


...and with that said I'm pretty sure I know what the problem is; since the table has a vertical rule in it, I suspect it is bug #62471.

But I'll check.

G. Branden Robinson <gbranden>
Group administrator
Tue 25 Oct 2022 01:56:17 AM UTC, comment #7: 

  A simple creation of the bug; add the extra code to
"tmac/groff_mdoc.7.man".

.Pp
.\" XXX: Figure out how to indent this.
.\" Add following
.rs
Send this line before page 217.
.bp\" 217
.\" End addition
.TS


Bjarni Ingi Gislason <bjarniig>
Sun 23 Oct 2022 11:46:38 PM UTC, comment #6: 

  Look at the table above the text "Local,".  You can trigger the error
by moving the table to the right place (on the page) in the series of
man pages.  Use the intermediate output (option -Z) to find the nearest
page brake to that table.

  An example:

.pl 6v
.TS
tab(@);
L | L | L | L .
One@two@three@four
five@six@seven@eight
.TE
.
.bp
.pl 10v
.if n \{\
.  if (\n[.d]u < \n[.v]u) .sp 1
.\}
.
.TS
tab(@);
L | L | L | L .
One@two@three@four
five@six@seven@eight
.TE


Bjarni Ingi Gislason <bjarniig>
Thu 20 Oct 2022 09:54:25 PM UTC, comment #5: 

Thanks for the attachment.  While I can reproduce the diagnostic message from that artifact, it doesn't persuade me that you haven't changed something in your private groff fork so as to cause the problem.  That file does not rule out problems in your versions of troff or the macro packages.

When I re-order the production of the collected man page documents as follows, the problem does not appear.


$ git diff doc
diff --git a/doc/doc.am b/doc/doc.am
index a5eae91fe..f07990e29 100644
--- a/doc/doc.am
+++ b/doc/doc.am
@@ -175,6 +175,7 @@ EXTRA_DIST += \
 # localization at all to format it would be gauche).
 GROFF_MAN_PAGES1 = \
   src/utils/addftinfo/addftinfo.1 \
+  tmac/groff_mdoc.7 \
   src/utils/afmtodit/afmtodit.1 \
   contrib/chem/chem.1 \
   src/preproc/eqn/eqn.1 \
@@ -224,7 +225,6 @@ GROFF_MAN_PAGES1 = \
   contrib/hdtbl/groff_hdtbl.7 \
   tmac/groff_man.7 \
   tmac/groff_man_style.7 \
-  tmac/groff_mdoc.7 \
   tmac/groff_me.7 \
   contrib/mm/groff_mm.7


$ (cd build && make -j)
make  all-am
make[1]: Entering directory '/home/branden/src/GIT/groff/build'
  CXX      src/devices/grolj4/lj4.o
  GROFF    doc/groff-man-pages.pdf
  GROFF    doc/groff-man-pages.utf8.txt
  CXXLD    grolj4
troff: backtrace: file 'man/groff_char.7':1011
troff:man/groff_char.7:1011: warning: special character '.j' not defined
troff: backtrace: file 'man/groff_char.7':1431
troff:man/groff_char.7:1431: warning: special character 'vA' not defined
troff: backtrace: file 'man/groff_char.7':1560
troff:man/groff_char.7:1560: warning: special character 'bs' not defined
troff: backtrace: file 'man/groff_char.7':1731
troff:man/groff_char.7:1731: warning: special character '-+' not defined
troff: backtrace: file 'man/groff_char.7':1780
troff:man/groff_char.7:1780: warning: special character 'coproduct' not defined
troff: backtrace: file 'man/groff_char.7':1875
troff:man/groff_char.7:1875: warning: special character '+e' not defined
contrib/mm/groff_mm.7:962: warning: table row will not fit on page 310
make[1]: Leaving directory '/home/branden/src/GIT/groff/build'


I recommend you check your local changes in the tmac directory.

G. Branden Robinson <gbranden>
Group administrator
Thu 20 Oct 2022 04:53:39 PM UTC, comment #4: 

  The attachment contains (the bz compressed) part of the intermediate
output from "groff-man-pages-utf8.txt" that contains "groff_mdoc.7" and
the necessary start and end commands and shows the error when executed
with "grotty" in git HEAD.

  GNU grotty (groff) version 1.22.4 (Debian) shows it also.



(file #53886)

Bjarni Ingi Gislason <bjarniig>
Thu 20 Oct 2022 12:34:37 AM UTC, comment #3: 


comment #2:

>   This warning popped up on 16th October at 2021 UTC but not at
> 00.47 UTC.


I don't know how that information is supposed to help me troubleshoot your problem.  How did your working copy change between those two time stamps?
 

>   It depends on the place of the "groff_mdoc.7" file in the input files.


Why is your build constructing the collected man pages in a different order than the groff master branch?

https://git.savannah.gnu.org/cgit/groff.git/tree/doc/doc.am?id=c2e894cf5b42cfdaae48f03476fee24a6103f9c9#n242

https://git.savannah.gnu.org/cgit/groff.git/tree/doc/doc.am?id=c2e894cf5b42cfdaae48f03476fee24a6103f9c9#n259

>   Changing place with the next file, and it does not appear.


I'll see if I can reproduce it with a minor permutation.
 

>   The reported line number is way out of being sensible.
>
>   The input files contain 65,834 lines.



grotty:<standard input>:(tmac/groff_mdoc.7):352605: error: output above
first line discarded


Pause for a moment and consider what the input to the grotty program is.

G. Branden Robinson <gbranden>
Group administrator
Wed 19 Oct 2022 10:52:13 PM UTC, comment #2: 

  This warning popped up on 16th October at 2021 UTC but not at
00.47 UTC.

  It depends on the place of the "groff_mdoc.7" file in the input files.

  Changing place with the next file, and it does not appear.

  The reported line number is way out of being sensible.

  The input files contain 65,834 lines.

Bjarni Ingi Gislason <bjarniig>
Tue 18 Oct 2022 02:24:08 PM UTC, comment #1: 

I can't reproduce this.  Try with an unmodified checkout of groff Git HEAD.


$ (cd build && rm -f doc/groff-man-pages.utf8.txt && make doc/groff-man-pages.utf8.txt)
  GROFF    doc/groff-man-pages.utf8.txt


G. Branden Robinson <gbranden>
Group administrator
Mon 17 Oct 2022 10:15:55 PM UTC, original submission:  

Subject: groff-man-pages.utf8.txt: grotty: output above first line discarded

  GROFF    doc/ms.pdf
  GROFF    doc/pic.pdf
  GROFF    doc/groff-man-pages.utf8.txt
grotty:<standard input>:(tmac/groff_mdoc.7):352605: error: output above
first line discarded

Bjarni Ingi Gislason <bjarniig>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #53886:  groff_mdoc.7.Z.bz added by bjarniig (29KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Posted a comment)
  • -email is unavailable- added by bjarniig (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-11-02 gbranden StatusConfirmed Duplicate
        Open/ClosedOpen Closed
    2022-11-02 gbranden StatusUnreproducible Confirmed
    2022-10-20 bjarniig Attached File- Added groff_mdoc.7.Z.bz, #53886
    2022-10-18 gbranden StatusNone Unreproducible
        Assigned toNone gbranden

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code