Update of bug #38474 (project findutils):

                  Status:                   Fixed => None                   

    _______________________________________________________

Follow-up Comment #11:

I've applied Paul's updated patch.   However, I think I agree with Eric on the
interpretation of the mode argument.

I propose not to reinstante -perm +MODE with different semantics in the
future.

The fact that this went wrong in the first place underlines the fact that more
regression test cases are needed for -perm.

I don't find the argument from compatibility with chmod very convincing since
the mode argument to chmod is understood to describe a modification to the
mode of an existing file.  In the case of -perm, there is no existing file
mode.

Hence a description of how the mode should be interpreted that makes perfect
sense for chmod could still be confusing for find -perm.

This puts me in the uncomfortable position of wondering if mode_change is
really the best basis for find -perm; I'm not sure this is really the intended
use case for that function.   But my first point above probably applies to
gnulib too; if I wanted mode_compile to reject mode strings of the form +MODE
I should probably have contributed a gnulib test case which enforced that.

Anyway I'm marking this bug as not-fixed because better tests are needed (in
findutils, at least).

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?38474>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/


Reply via email to