[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #61454] [PATCH]?: nroff -V ...: show the (final, real) pipe line

From: G. Branden Robinson
Subject: [bug #61454] [PATCH]?: nroff -V ...: show the (final, real) pipe line
Date: Tue, 9 Nov 2021 20:45:24 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0

Update of bug #61454 (project groff):

                  Status:                    None => Invalid                
             Assigned to:                    None => gbranden               
             Open/Closed:                    Open => Closed                 


Follow-up Comment #1:

Please read the comment immediately above the material you're patching.

# Unfortunately, the shell is a nightmarish hellscape of quoting issues.
# Naïve attempts to solve the problem fail when arguments to nroff
# contain embedded whitespace or shell metacharacters.  The solution
# below works with those, but there is insufficient quoting in -V (dry
# run) mode, such that you can't copy-and-paste the output of 'nroff -V'
# if you pass it a filename like foo"bar (with the embedded quotation
# mark) and expect it to run without further quoting.

There are at least 2 problems with your patch.

1. It ignores the meaning of the -V option and executes the command instead of
writing it to the standard output stream.

$ echo "foobar" | GROFF_TEST_GROFF=./test-groff ./nroff -V -t -ms | cat -s


2. It exhibits the very problem warned about in the comment.

$ GROFF_TEST_GROFF=./test-groff ./nroff -V -t -ms foo\"bar.man
./nroff: 1: eval: Syntax error: Unterminated quoted string

I do not see how this patch constitutes an improvement on the existing code in
any way, and the attached description does not help me to understand how the
existing code is inadequate, if at all.

Please take the time to clearly describe a defect along with, prior to, or
instead of providing a patch.


Reply to this item at:


  Message sent via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]