Re: [PATCH 0/2] selftests: Replace "Bail out" with "Error" in ksft_exit_fail_msg()

From: Shuah Khan
Date: Fri Apr 05 2024 - 17:08:03 EST


On 4/5/24 14:38, Muhammad Usama Anjum wrote:
Hi Tim,

On 4/6/24 12:36 AM, Bird, Tim wrote:
Sorry I didn't catch this on the original submission.

-----Original Message-----
From: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>

On 4/5/24 07: 17, Muhammad Usama Anjum wrote: > "Bail out! " is not descriptive. It rather should be: "Failed: " and > then this added prefix
doesn't need to be added everywhere. Usually in > the logs, we are searching for "Failed"
ZjQcmQRYFpfptBannerStart
Caution : This email originated from outside of Sony.
Do not click links or open any attachments unless you recognize the sender and know the content is safe. Please report phishing if unsure.

ZjQcmQRYFpfptBannerEnd
On 4/5/24 07:17, Muhammad Usama Anjum wrote:
"Bail out! " is not descriptive. It rather should be: "Failed: " and
then this added prefix doesn't need to be added everywhere. Usually in
the logs, we are searching for "Failed" or "Error" instead of "Bail
out" so it must be replace.

Bail out! is the wording in the original TAP spec. We should not change
it unless we plan to abandon compatibility with that spec. (which I
would advise against).

See https://testanything.org/tap-specification.html
I didn't know that exact words are coming from TAP. Thank you for catching
it. We don't intend to move away from the spec.


The reason "Bail out!" is preferred (IMO) is that it is less likely to be emitted
in other test output, and is more 'grepable'.
Makes sense.


This would get a NAK from me.
Let's drop this series.


Thank you. Dropped now.

thanks,
-- Shuah