bugfindutils - Bugs: bug #54509, `-execdir command ;` returns 0...

 
 

bug #54509: `-execdir command ;` returns 0 when it should not

Submitter:  None
Submitted:  Mon 13 Aug 2018 08:23:38 PM UTC
   
 
Category:  find Severity:  3 - Normal
Item Group:  Wrong result Status:  Fixed
Privacy:  Public Assigned to:  berny
Originator Name:  segfault Originator Email:  -email is unavailable-
Open/Closed:  Closed Release:  None
Fixed Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 17 Aug 2018 09:00:39 AM UTC, comment #2: 
Bernhard Voelker <berny>
Group administrator
Tue 14 Aug 2018 08:43:43 PM UTC, comment #1: 

That sentence has recently been added to 'find.1', i.e., after the
release of v4.6.0; references:

  https://bugzilla.redhat.com/show_bug.cgi?id=1325049
  http://lists.gnu.org/archive/html/bug-findutils/2016-04/msg00014.html
  https://git.sv.gnu.org/cgit/findutils.git/commit/?id=ae424b95

The problem is that it only applies to the '+' syntax, not to
the ';' syntax of the -exec/-execdir/-ok/-okdir family.

The attached patch fixes the documentation.

CC-ing Kamil who authored the first version of that commit which
only added that sentence to the description of '-exec ... +';
The misleading part for the -execdir predicate was added by me
in his name in the same commit, sigh.

(file #44782)

Bernhard Voelker <berny>
Group administrator
Mon 13 Aug 2018 08:23:38 PM UTC, original submission:  

This affects find 4.7.0-git on Debian Sid (unstable), but I couldn't select that version in the bug submit form, because it only lists releases up to 4.6.0.

The man page says the following about `-execdir command ;`:

"If any invocation returns a non-zero value as exit status, then find returns a non-zero exitstatus."

This doesn't seem to be correct, for example the following returns 0:


mkdir foo; find foo -execdir false \;


Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #44782:  0001-doc-clarify-again-exit-status-handling-of-exec-comma.patch added by berny (2KiB - text/x-patch - [PATCH] doc: clarify again exit status handling of -exec command {} +)

 

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 berny
  • -email is unavailable- added by None (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
    2018-08-17 berny StatusIn Progress Fixed
        Open/ClosedOpen Closed
    2018-08-14 berny Attached File- Added 0001-doc-clarify-again-exit-status-handling-of-exec-comma.patch, #44782
        StatusNone In Progress
        Assigned toNone berny
        Carbon-Copy- Added kamil dudka <kdudka@redhat.com>

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code