bugfindutils - Bugs: bug #57291, xargs --verbose adds incorrect...

 
 

bug #57291: xargs --verbose adds incorrect trailing blank

Submitter:  jidanni
Submitted:  Sat 23 Nov 2019 04:10:55 AM UTC
 
Category:  xargs Severity:  3 - Normal
Item Group:  Wrong result Status:  Fixed
Privacy:  Public Assigned to:  berny
Originator Name:  Open/Closed:  Closed
Release:  4.7.0 Fixed Release:  4.8.0
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Mon 02 Nov 2020 11:11:24 PM UTC, comment #5: 
Bernhard Voelker <berny>
Project Administrator
Fri 30 Oct 2020 10:56:38 AM UTC, comment #4: 

Oops - good spot.

Fixed in the generated ChangeLog during 'make dist' with the attached patch.
Unfortunately, the git history can't be changed.

Have a nice day,
Berny

(file #50166)

Bernhard Voelker <berny>
Project Administrator
Fri 30 Oct 2020 09:50:46 AM UTC, comment #3: 

Dear Bernhard,

comment #2:

> Pushed at:
> https://git.sv.gnu.org/cgit/findutils.git/commit/?id=6e55ad98b


Unfortunately this commit references this bug report by a completely wrong URL:

It refers to https://bugs.gnu.org/?57291 (which redirects to and just shows GNU's debbugs start pageā€”and GNU's debbugs doesn't have that many bug reports yet), but correct would have been https://savannah.gnu.org/bugs/?57291

Could you please at least correct the URL in the according changelog entry so that people don't have to search for the correct URL via some search engine like I had to?

Axel Beckert <abe>
Sun 08 Dec 2019 09:57:38 PM UTC, comment #2: 
Bernhard Voelker <berny>
Project Administrator
Mon 25 Nov 2019 09:43:04 PM UTC, comment #1: 

Thanks for the report.
I agree that the additional blank is disturbing.  The attached patch fixes it.

@James: do you agree?

(file #47929)

Bernhard Voelker <berny>
Project Administrator
Sat 23 Nov 2019 04:10:55 AM UTC, original submission:  

$ echo a b c|xargs --verbose -n 1 2>&1|cat -e
echo a $
a$
echo b $
b$
echo c $
c$

Did the user submit 'a '? No. He submitted just 'a', so there should be no trailing blank.

jidanni

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #50166:  0001-maint-fix-bug-URL-in-ChangeLog-entry.patch added by berny (1KiB - text/x-patch - [PATCH] maint: fix bug URL in ChangeLog entry)
file #47929:  0001-xargs-avoid-trailing-blank-in-t-verbose-output.patch added by berny (5KiB - text/x-patch - [PATCH] xargs: avoid trailing blank in -t,--verbose output)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by abe (Posted a comment)
  • -email is unavailable- added by berny (Updated the item)
  • -email is unavailable- added by jidanni (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 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-01-09 berny Open/ClosedOpen Closed
        Fixed ReleaseNone 4.8.0
    2020-10-30 berny Attached File- Added 0001-maint-fix-bug-URL-in-ChangeLog-entry.patch, #50166
    2019-12-08 berny StatusCode Review Fixed
    2019-11-25 berny Attached File- Added 0001-xargs-avoid-trailing-blank-in-t-verbose-output.patch, #47929
        StatusNone Code Review
        Assigned toNone berny

    Back to the top


    Powered by Savane 3.9