[Rpm-maint] [rpm-software-management/rpm] Another attempt at stabilizing posix.fork() and url retrieve tests (PR #3247)

Panu Matilainen notifications at github.com
Tue Aug 20 10:21:32 UTC 2024


Both these tests (currently 12 and 90) are flaky and sometimes fail with the trailing newline at the front, eg test stderr showing:

    +
     warning: posix.fork(): .fork(), .exec(), .wait() and .redirect2null() are deprecated, use rpm.execute() instead
     warning: posix.wait(): .fork(), .exec(), .wait() and .redirect2null() are deprecated, use rpm.execute() instead
     warning: runaway fork() in Lua script
    -

It's all the more bizarre since we're NOT expecting that trailing an empty line after the messages in the first place.

Let's try another trick from autotest: just grab the stderr into a file that we process in the next test. Autoconf manual says "This action is often used when it is desired to use grep to look for a substring in the output, or when the output must be post-processed to normalize error messages into a common form." Which is exactly what we're doing here.

Time will tell, but at least we're now seeing the last newline on both tests as I would expect.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/3247

-- Commit Summary --

  * Another attempt at stabilizing posix.fork() and url retrieve tests

-- File Changes --

    M tests/rpmgeneral.at (11)
    M tests/rpmmacro.at (13)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/3247.patch
https://github.com/rpm-software-management/rpm/pull/3247.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3247
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3247 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20240820/c5dc9592/attachment.html>


More information about the Rpm-maint mailing list