Re: [PATCH] kunit: tool: fix extra trailing \n in parsed test output

2020-10-30 Thread Daniel Latypov
On Fri, Oct 30, 2020 at 12:22 AM David Gow wrote: > > On Fri, Oct 30, 2020 at 1:41 PM Daniel Latypov wrote: > > > > On Thu, Oct 29, 2020 at 7:34 PM David Gow wrote: > > > > > > On Wed, Oct 21, 2020 at 7:32 AM Daniel Latypov > > > wrote: > > > > > > > > For simplcity, strip all trailing whitesp

Re: [PATCH] kunit: tool: fix extra trailing \n in parsed test output

2020-10-30 Thread David Gow
On Fri, Oct 30, 2020 at 1:41 PM Daniel Latypov wrote: > > On Thu, Oct 29, 2020 at 7:34 PM David Gow wrote: > > > > On Wed, Oct 21, 2020 at 7:32 AM Daniel Latypov wrote: > > > > > > For simplcity, strip all trailing whitespace from parsed output. > > > I imagine no one is printing out meaningful

Re: [PATCH] kunit: tool: fix extra trailing \n in parsed test output

2020-10-29 Thread Daniel Latypov
On Thu, Oct 29, 2020 at 7:34 PM David Gow wrote: > > On Wed, Oct 21, 2020 at 7:32 AM Daniel Latypov wrote: > > > > For simplcity, strip all trailing whitespace from parsed output. > > I imagine no one is printing out meaningful trailing whitespace via > > KUNIT_FAIL() or similar, and that if they

Re: [PATCH] kunit: tool: fix extra trailing \n in parsed test output

2020-10-29 Thread David Gow
On Wed, Oct 21, 2020 at 7:32 AM Daniel Latypov wrote: > > For simplcity, strip all trailing whitespace from parsed output. > I imagine no one is printing out meaningful trailing whitespace via > KUNIT_FAIL() or similar, and that if they are, they really shouldn't. > > At some point, the lines from

[PATCH] kunit: tool: fix extra trailing \n in parsed test output

2020-10-20 Thread Daniel Latypov
For simplcity, strip all trailing whitespace from parsed output. I imagine no one is printing out meaningful trailing whitespace via KUNIT_FAIL() or similar, and that if they are, they really shouldn't. At some point, the lines from `isolate_kunit_output()` started having trailing \n, which result