--- Begin Message ---
|
Subject: |
Test "colors" fails on Debian/ppc64el |
|
Date: |
Mon, 18 Feb 2019 01:57:09 +0100 (CET) |
|
User-agent: |
Alpine 2.20 (DEB 67 2015-01-07) |
Hello.
I've just uploaded diffutils 3.7 for Debian unstable (hopefully to be
part of Debian 10, since it contains only small fixes).
But there is a problem: The build in the ppc64el autobuilder failed:
https://buildd.debian.org/status/logs.php?pkg=diffutils&ver=1%3A3.7-1&arch=ppc64el
In tests/colors there was a race condition which I tried to fix
by adding a "sleep 1", like this:
mkfifo fifo
printf '%1000000s-a' > a
printf '%1000000s-b' > b
head -c 10 < fifo > /dev/null &
sleep 1
diff --color=always ---presume-output-tty a b > fifo
test $? = 141 || fail=1
It is possible that the "head" command takes (sometimes) more than one second to
complete on ppc64el?
Are there any other places in the test where a race condition could be
happening?
Thanks.
--- End Message ---
--- Begin Message ---
|
Subject: |
Re: [bug-diffutils] bug#36488: diffutils 3.7 make check failure ppc64le opensuse on colors test |
|
Date: |
Sun, 29 Aug 2021 00:22:27 -0700 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 8/28/21 8:40 AM, Thiago Jung Bauermann via bug-diffutils via All
diffutils discussion. wrote:
I believe this is the same problem reported in bug 34519.
The Debian build also fails with "diff: standard output: Broken pipe".
Thanks for tracking that down. Frédéric's analysis in
<https://bugs.debian.org/922552#19> was helpful.
I found some time to look into this bug, and installed into
Savannah.gnu.org diffutils the attached patch, which I hope fixes the
bug although I don't have the relevant platform to test it. Please give
it a try.
Once this patch is part of a release, Debian shouldn't need any patches
for diffutils.
For now I am closing the diffutils bug report
<https://bugs.gnu.org/36488>; if I was too optimistic and the patch
doesn't fix things we can always reopen it.
0001-diff-cleanup-signal-handling-just-before-exit.patch
Description: Text Data
--- End Message ---