Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

exit status #11

Open
dicktyr opened this issue Feb 10, 2015 · 2 comments
Open

exit status #11

dicktyr opened this issue Feb 10, 2015 · 2 comments

Comments

@dicktyr
Copy link

dicktyr commented Feb 10, 2015

it would be useful if cropgui returned a non-zero exit status in cases where it terminates without doing any cropping

@za3k
Copy link

za3k commented Feb 11, 2017

What's the use-case?

I would find some kind of indicator of whether it cropped the file useful, but I don't think exit status with no special flags passed is appropriate--that should be for errors. Also, cropgui can accept multiple command-line arguments and crop each, if you're not aware.
Maybe outputting a list of cropped files to stdout would be nice, because then I wouldn't have to reverse-engineer the naming convention in my shell scripts.

@dicktyr
Copy link
Author

dicktyr commented Feb 11, 2017

What's the use-case?

more efficient scripting

I don't think exit status with no special flags passed is appropriate--that should be for errors

fair enough
but grep and others use exit status as I suggest
(i.e. not strictly for errors but also for command failure)

% echo abc | grep x
% echo $?
1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants