Follow-up Comment #6, bug #52137 (project findutils):

> I squashed the commits into one, and changed the following:
> - use error() rather than fprintf(),

I tried that first but it broke the testsuite because error() always outputs
argv[0] and argv[0] will include the path when run from testsuite. You have
fixed that in config/unix.exp. ;-)

[...]
> - change the test cases to always only have 2 conflicting options,

I am not too happy with that, I can understand *adding* two-option examples
but not replacing the three-case ones.

Your patch introduces an unintended behavior change in respect to "-I REPL
-n1":

xargs release:
(sid)ametzler@argenau:/tmp/FIND/findutils-4.6.0+git+20170828/xargs$ echo a b c
| xargs -I x -n1  echo x z
a b c z

with patch
(sid)ametzler@argenau:/tmp/FIND/findutils-4.6.0+git+20170828/xargs$ echo a b c
| ./xargs -I x -n1  echo x z
x z a b c

cu Andreas

    _______________________________________________________

Reply to this item at:

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

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


Reply via email to