bugGNU roff - Bugs: bug #54461, hdtbl.am: generate examples with...

 
 

bug #54461: hdtbl.am: generate examples with warnings and backtraces enabled

Submitter:  Bjarni Ingi Gislason <bjarniig>
Submitted:  Tue 07 Aug 2018 10:06:57 PM UTC
   
 
Category:  Macro - others/general Severity:  1 - Wish
Item Group:  Build/Installation Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 28 Jul 2023 08:53:44 PM UTC, comment #4: 

Also worth noting: the patch wouldn't even apply anymore as-is: the variable it changes was renamed in commit e5a55040, then went away entirely in commit a5af2f4d.  The flags would now have to be added to the $(GROFFBIN) line in the same file.

Dave <barx>
Group Member
Sat 22 Jul 2023 07:35:07 PM UTC, comment #3: 

The patch proposed here could be a useful diagnostic tool for someone who wants to take on maintenance of hdtbl.  But no one seems to think it should be applied to git, so I'm removing the "[PATCH]" annotation from the Summary.  In other words, while the bug itself isn't being rejected, the patch is.

Dave <barx>
Group Member
Sun 04 Nov 2018 06:21:55 PM UTC, comment #2: 

Yes, I object.

If the component is totally unmaintained then we should drop it from the groff distribution.

Keeping it in the distribution implies that we intend to maintain it.

It's a biconditional, in my view.

G. Branden Robinson <gbranden>
Group administrator
Thu 09 Aug 2018 09:45:05 PM UTC, comment #1: 

I think this should be rejected.

Making the build of a totally unmaintained component more verbose is detrimental and distracting.  If somebody wants to pick up maintenance of these macros and also takes care of the resulting noise in the build log, all the better, but not as long as nobody is working on this stuff.

Any objections to closing this ticket as "Invalid"?

Ingo Schwarze <schwarze>
Group Member
Tue 07 Aug 2018 10:06:57 PM UTC, original submission:  


From 54fe68a3c2f70626ffe8e3f9e093d1cebe6d32f9 Mon Sep 17 00:00:00 2001
From: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
Date: Tue, 7 Aug 2018 21:23:24 +0000
Subject: [PATCH] hdtbl.am: Allow "groff" to use its diagnostic power

  Add the options "-b -ww" to the variable "HDTBL_PFLAG" to activate
diagnostics to show the place where errors occur.

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
---
 contrib/hdtbl/hdtbl.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/hdtbl/hdtbl.am b/contrib/hdtbl/hdtbl.am
index 1b80a9a9..90f338cc 100644
--- a/contrib/hdtbl/hdtbl.am
+++ b/contrib/hdtbl/hdtbl.am
@@ -26,7 +26,7 @@ man7_MANS += contrib/hdtbl/groff_hdtbl.7

 # Groff command used to generate .ps files
 HDTBL_TFLAG = -M$(hdtbl_srcdir) -M$(hdtbl_builddir)
-HDTLB_PFLAG=-t -p -e
+HDTLB_PFLAG=-t -p -e -b -ww
 HDTBLGROFF = \
   GROFF_COMMAND_PREFIX= \
   GROFF_BIN_PATH="$(GROFF_BIN_PATH)" \
--
2.18.0


Bjarni Ingi Gislason <bjarniig>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by barx (Posted a comment)
  • -email is unavailable- added by gbranden (Posted a comment)
  • -email is unavailable- added by schwarze (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
    2023-08-10 gbranden Summaryhdtbl.am: Allow &quot;groff&quot; to use its diagnostic power hdtbl.am: generate examples with warnings and backtraces enabled
    2023-08-10 gbranden StatusPostponed None
    2023-07-22 barx Summary[PATCH] hdtbl.am: Allow &quot;groff&quot; to use its diagnostic power hdtbl.am: Allow "groff" to use its diagnostic power
    2018-08-13 schwarze StatusNone Postponed
    2018-08-09 schwarze CategoryNone Macro - others/general
        Severity3 - Normal 1 - Wish

    Back to the top

    Powered by Savane 3.12.
    Corresponding source code