bugfindutils - Bugs: bug #61774, Docs for find -delete out of sync

 
 

bug #61774: Docs for find -delete out of sync

Submitted by:  Andreas Metzler <ametzler>
Submitted on:  Sat 08 Jan 2022 05:38:08 PM UTC
 
Category:  documentation Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  berny
Originator Name:  Open/Closed:  Closed
Release:  4.8.0 Fixed Release:  4.9.0
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Tue 01 Feb 2022 06:07:43 PM UTC, comment #3: 

Hi Andreas,

thanks for the review.
I pushed it in your name [1] - well, the "Co-authored-by:" reflects
the collaboration, and changing it around would just be a (little)
risk to mess it up - and there's laziness. ;-)

[1] https://git.sv.gnu.org/cgit/findutils.git/commit/?id=58747d77

Bernhard Voelker <berny>
Project Administrator
Tue 01 Feb 2022 05:50:32 PM UTC, comment #2: 

comment #1:

> Thanks for the report and the suggested patch.

[...]

> I tried further to improve and unify the -delete description
> in the attached git patch.


Thank you! Docs are nicely in sync.

> Okay to push in your name?


I think you did more (and better) work on this. (From: Berny [...] perhaps "suggested by ametzler") would reflect reality better. Feel free to push under your or my name.
cu Andreas

Andreas Metzler <ametzler>
Sun 30 Jan 2022 11:57:36 PM UTC, comment #1: 

Thanks for the report and the suggested patch.

The formatting for the Texinfo changes were not ideal
(e.g. it is better to use a "@quotation" environment for
a warning block.

Also "Because -delete implies @samp{-delete}, ..." was obviously wrong.

I tried further to improve and unify the -delete description
in the attached git patch.
Okay to push in your name?

Thanks & have a nice day,
Berny

(file #52778)

Bernhard Voelker <berny>
Project Administrator
Sat 08 Jan 2022 05:38:08 PM UTC, original submission:  

Hello,

this was originally reported as https://bugs.debian.org/1003339 by Chris Davies. He noted that the manpage says "Delete files;" but info has "Delete files or directories".

Looking at both I found there are multiple differences with no good reason (manpage is not more terse). I have tried to come up with a merged unified version. - Patch attached.

cu Andreas

Andreas Metzler <ametzler>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #52778:  0001-doc-improve-description-of-find-delete.patch added by berny (6KiB - text/x-patch - [PATCH] doc: improve description of 'find -delete')
file #52630:  sync-delet-docs.diff added by ametzler (3KiB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by berny (Updated the item)
  • -email is unavailable- added by ametzler (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
    2022-02-01 berny Open/ClosedOpen => Closed
    2022-02-01 berny StatusCode Review => Fixed
    2022-01-30 berny Attached File- => Added 0001-doc-improve-description-of-find-delete.patch, #52778
        StatusNone => Code Review
        Assigned toNone => berny
        Fixed ReleaseNone => 4.9.0
    2022-01-08 ametzler Attached File- => Added sync-delet-docs.diff, #52630

    Back to the top


    Powered by Savane 3.9