search

bkerler / edl

Inofficial Qualcomm Firehose / Sahara / Streaming / Diag Tools :)
GNU General Public License v3.0
1.47k stars 354 forks source link

exit with error #533

Closed bongbui321 closed 2 months ago

bongbui321 commented 2 months ago

Hi, I want to use edl within shell script, and I want it to exit if there were error with the flag -e in shebang.

bongbui321 commented 2 months ago

@bkerler sorry for tagging you, but maybe have a look at this one as well.

I'm trying to use the edl tool in another project like making it less verbose with like a flag -v when using multiple commands, if you are okay with that I'm going to make other PRs later on, want to know your input on that.

bkerler commented 2 months ago

Need to have a look. As for verbose level, if you intent to change that, best use the loglevel and change messages from self.info to self.debug for example

bongbui321 commented 2 months ago

@bkerler. Hi, this PR is just exiting with value 1 if there were failure in any of the command since I want to use the edl tool as a CLI tool. As for the verbose level, I'll open another PR