README out of date

[mrcnski]

The README makes it seem that --until-error does not require an argument:

Or until it fails with --until-error (which also accepts an optional error code):

But when I try it without an argument, I get this:

error: The argument '--until-error <until_error>' requires a value but none was supplied

USAGE:
    loop <input> --count-by <count_by> --every <every> --offset <offset> --until-error <until_error>

So I think that maybe --until-fail is what I want, based on this:

Or, if you have a list of files but need to find the one which causes your program to fail:
$ ls | loop './my_program $ITEM' --until-fail;

But I get this:

error: Found argument '--until-fail' which wasn't expected, or isn't valid in this context

And I still don't know how to run a command until it fails. Seems like it is currently impossible.

[Miserlou]

Yep, that seems like a bug, probably introduced with the refactor. Probably maybe still works with --until-fail 1?

[mrcnski]

Ah, nope:

error: Found argument '--until-fail' which wasn't expected, or isn't valid in this context
	Did you mean --until-time?

Thanks for replying, cool tool 👍

[art-of-dom]

So in the current master, I see --until-fail is works as expected. --until-error still has the same message:

 error: The argument '--until-error <until_error>' requires a value but none was supplied

I'm going to start looking into it unless there are objections.

[Miserlou]

Sounds good! I think it should error out for any non zero, unless they supply an error code, in which case only exit on that

…

On Sun, Mar 24, 2019 at 9:32 PM art-of-dom ***@***.***> wrote: So in the current master, I see --until-fail is works as expected. --until-error still has the same message: error: The argument '--until-error <until_error>' requires a value but none was supplied I'm going to start looking into it unless there are objections. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#31 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAIi04hbcuIzL0cuypSDirto9yv5lAhBks5vaCcfgaJpZM4WTKB2> .