Hi, Yes they are. I just send recipe upgrade to OE-Core, the patches are already in git. Regards, Anibal

Hi Alex, Do you have a repo/branch for this patch?. I'm having issues applying... ... alimon@blackbox:~/upstream/ptest-runner2$ git am -3 ~/Downloads/\[ptest-runner\]\[PATCH\ 3_3\]\ utils.c_\ add\ sys

I was OOO for futher you can find me with nickname alimon at #yocto at Libera. Applied, Thanks, Anibal

Hi, I tried to apply without success, looks like the patch is corrupted, can you send it again?, or share a git repo/branch. Regards, Anibal

Indeed this change doesn't has a direct impact but is fine to me. Applied, Thanks!

Applied, Thanks!,

Hi, In a normal execution this memleak doesn't have a bad effect in practical terms because at end of the program the memory will be free'd. I'm fine to pick this patch. Regards, Anibal

Applied, Thanks!, Anibal.

Hi all, Just replied in other email about this situation, so yes for me is better at least return 1 when any test fails to make it backwards compatible and knows when some test fails. Regards, Anibal

Since we are using threads to read from child, no complex logic is needed for handle timeouts by std{out,err} in the child using alarm(2). [YOCTO #14220] Signed-off-by: Aníbal Limón <anibal.limon@lina

Ack, I changed the behavior of timeout so that's the reason I'm seeing this error. I don't see any reason to not support this may be useful in the future. Regards, Anibal

Yeah, thanks for remind me the old behavior, some years and I forget about it, I will fix it and put certain comments about that. I pushed to dev branch, http://git.yoctoproject.org/cgit/cgit.cgi/ptes

Signed-off-by: Aníbal Limón <anibal.limon@...> --- conf/layer.conf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/conf/layer.conf b/conf/layer.conf index 178ce1b..ff009ef 100

The ptest-runner has a default timeout of 300 secs and can be override usint -t option in the runner. There is a need to specify timeout by ptest because not all ptests takes the sametime also are aff

Signed-off-by: Aníbal Limón <anibal.limon@...> --- tests/ptest_list.c | 10 +++++----- tests/utils.c | 14 +++++++------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/pte

Since we are using threads to read from child, no complex logic is needed for handle timeouts use alarm(2). [YOCTO #14220] Signed-off-by: Aníbal Limón <anibal.limon@...> --- utils.c | 60 ++++++

In order to handle large output add a thread for read from childs using a pipe and remove non-blocking option. Modify bash unittest to output large data and cover this scenario. [YOCTO #14220] Signed-

Modify testcase to validate the actual exit status. [YOCTO #14217] Signed-off-by: Aníbal Limón <anibal.limon@...> --- tests/data/fail/ptest/run-ptest | 3 +++ tests/utils.c | 2 +- utils.c | 3 ++

Fixes, $ ./ptest-runner -d ./tests/data bash Signed-off-by: Aníbal Limón <anibal.limon@...> --- utils.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/utils.c b/ut

Hi Randy, I pushed the rev, my mistake, sorry for the trouble. Regards, Anibal