Here's your webrev with a jtreg compliant test added :
I verified the test behaves properly under jtreg -
before the fix jtreg kills it on time out
after the fix it finishes quickly and successfully
Outside of jtreg it also exits properly although jtreg is the normal way
to run it.
This is all +1 from me but I think someone else should sign off on this
too since I
don't want to self-review the test.
On 1/12/20 11:54 AM, Phil Race wrote:
That needs reworking in location as well as adding jtreg boiler plate and legal
notice. Also there are code changes required to be a well behaved test. I need
to sit in front of a headful Linux system to test those so I will have to
follow up on this (hopefully) tomorrow.
On Jan 11, 2020, at 12:40 PM, Clemens Eisserer <linuxhi...@gmail.com> wrote:
Is there a regression test ?
I don't see a noreg- label.
Sorry I forgot to mention, the regression test is in the "main"
directory of the bugfix:
Thanks and best regards, Clemens