[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