bugGNU troff - Bugs: bug #57813, Test for the macro "AT"...

 
 

bug #57813: Test for the macro "AT" fails

Submitted by:  Bjarni Ingi Gislason <bjarniig>
Submitted on:  Sat 15 Feb 2020 01:57:20 AM UTC  
 
Category:  Macro - man Severity:  3 - Normal
Item Group:  Test Status:  Invalid
Privacy:  Public Assigned to:  G. Branden Robinson <gbranden>
Open/Closed:  Closed Planned Release:  None

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 15 Feb 2020 10:28:28 PM UTC, comment #5: 

> comment #2:
> >   I have not installed a version of my git repository.
> > So the "groff" is from /usr/bin/groff (Debian).
> >
> >   My "test-groff" can be different from yours.
>
> Can you reproduce this failure with the stock `test-groff`?


  The "test-groff" from "master" showed no failure.

  The reason for the failure was a stripped an-old.tmac file,
which I had not updated.

  Thanks for the help.

  Ticket can be closed.

Bjarni Ingi Gislason <bjarniig>
Sat 15 Feb 2020 04:25:51 PM UTC, comment #4: 

If i read the original submission correctly - as usual, Bjarni doesn't really explain properly what he is doing - he uses the groff-1.22.4 from the Debian package for testing.  Of course, the test is expected to fail with that because the 1.22.4 release still contained that bug, and this test aims to make sure that the bug is fixed.

In any case, the remedy suggested by Bjarni is totally bogus.  Of course the .AT macro is not supposed to come after a page transition.  In fact, .AT usually is the second macro in a file, see https://svnweb.freebsd.org/csrg/lib/libc/gen/getgrent.3?annotate=47718 for a typical real-world example.

Please close this ticket as invalid.

Ingo Schwarze <schwarze>
Project Member
Sat 15 Feb 2020 04:11:53 AM UTC, comment #3: 

comment #2:

>   I have not installed a version of my git repository.
> So the "groff" is from /usr/bin/groff (Debian).
>
>   My "test-groff" can be different from yours.


Can you reproduce this failure with the stock `test-groff`?

G. Branden Robinson <gbranden>
Project AdministratorIn charge of this item.
Sat 15 Feb 2020 02:55:23 AM UTC, comment #2: 

  I have not installed a version of my git repository.
So the "groff" is from /usr/bin/groff (Debian).

  My "test-groff" can be different from yours.

(file #48414, file #48415, file #48416)

Bjarni Ingi Gislason <bjarniig>
Sat 15 Feb 2020 02:11:31 AM UTC, comment #1: 

Well, that's odd.  The tests pass for me or I wouldn't have pushed the commit that introduced them.

Can you attach your copy of test-suite.log produced by this test run to this bug?

G. Branden Robinson <gbranden>
Project AdministratorIn charge of this item.
Sat 15 Feb 2020 01:57:20 AM UTC, original submission:  

Subject: Test for the macro "AT" fails

  The first test in
"groff/tmac/tests/an-old_AT_and_UC_footer_saved_and_restored.sh"
fails both with "groff/build/test-groff" and
"/usr/bin/groff (Debian GNU groff version 1.22.4)".

  The .AT and .UC macros must come after a page transition.

  These macros should give a warning
if they are called when no page transition has occurred,
something like:

.if \\n[nl]<=0 \{\
.  tm \\n[.F]:\\n[.c]: The macro \\$0 must be used later
.  tm in the input, e.g., after the text for the first .SH macro.
.\}

  This fact should also be documented in groff_man.7.man.

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 #48414:  test-suite.log added by bjarniig (3KiB - 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 schwarze (Posted a comment)
  • -email is unavailable- added by gbranden (Posted a comment)
  • -email is unavailable- added by bjarniig (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-02-15 schwarze StatusNeed Info => Invalid
        Open/ClosedOpen => Closed
    2020-02-15 bjarniig Attached File- => Added an-old_AT_and_UC_footer_saved_and_restored.sh.log, #48416
    2020-02-15 bjarniig Attached File- => Added test-suite.log, #48414
        Attached File- => Added an-old_AT_and_UC_footer_saved_and_restored.sh.trs, #48415
    2020-02-15 gbranden StatusNone => Need Info
        Assigned toNone => gbranden

    Back to the top


    Powered by Savane 3.5