Re: [PATCH 2/3] Ktest: use dodie for critical falures

2018-03-26 Thread Tim Tianyang Chen
Thanks for the review Steve! One quick comment inline: On 03/21/2018 08:16 AM, Steven Rostedt wrote: On Fri, 15 Dec 2017 15:20:10 -0800 Tim Tianyang Chen wrote: Users should get emails when the script dies because of a critical failure. Critical failures are

Re: [PATCH 2/3] Ktest: use dodie for critical falures

2018-03-26 Thread Tim Tianyang Chen
Thanks for the review Steve! One quick comment inline: On 03/21/2018 08:16 AM, Steven Rostedt wrote: On Fri, 15 Dec 2017 15:20:10 -0800 Tim Tianyang Chen wrote: Users should get emails when the script dies because of a critical failure. Critical failures are defined as any errors that could

Re: [PATCH 2/3] Ktest: use dodie for critical falures

2018-03-21 Thread Steven Rostedt
On Fri, 15 Dec 2017 15:20:10 -0800 Tim Tianyang Chen wrote: > Users should get emails when the script dies because of a critical failure. > Critical > failures are defined as any errors that could abnormally terminate the script. > > In order to add email support,

Re: [PATCH 2/3] Ktest: use dodie for critical falures

2018-03-21 Thread Steven Rostedt
On Fri, 15 Dec 2017 15:20:10 -0800 Tim Tianyang Chen wrote: > Users should get emails when the script dies because of a critical failure. > Critical > failures are defined as any errors that could abnormally terminate the script. > > In order to add email support, this patch converts all die()

[PATCH 2/3] Ktest: use dodie for critical falures

2017-12-15 Thread Tim Tianyang Chen
Users should get emails when the script dies because of a critical failure. Critical failures are defined as any errors that could abnormally terminate the script. In order to add email support, this patch converts all die() to dodie() except: * when '-v' is used as an option to get the version

[PATCH 2/3] Ktest: use dodie for critical falures

2017-12-15 Thread Tim Tianyang Chen
Users should get emails when the script dies because of a critical failure. Critical failures are defined as any errors that could abnormally terminate the script. In order to add email support, this patch converts all die() to dodie() except: * when '-v' is used as an option to get the version