Re: install/56303: On a fresh installation, /tmp on sparc64 is not sticky

[Andreas Gustafsson <gson%gson.org@localhost>]

The following reply was made to PR install/56303; it has been noted by GNATS.

From: Andreas Gustafsson <gson%gson.org@localhost>
To: martin%netbsd.org@localhost
Cc: gnats-bugs%netbsd.org@localhost,
    roland.illig%gmx.de@localhost,
    rvp%SDF.ORG@localhost
Subject: Re: install/56303: On a fresh installation, /tmp on sparc64 is not
 sticky
Date: Mon, 26 Jul 2021 13:20:14 +0300

 Martin Husemann wrote:
 >  I wonder if this is the same underlying cause (my guess: a tar(1) bug)
 >  that causes PR install/56326, but I have no idea how to catch it
 >  (besides sprinkling some kind of logging into the tar binary).
 
 Have you looked at rvp's test script and patch?  I couldn't reproduce
 the problem using the script on -9/amd64, and I don't fully understand
 the patch, but this line in progress.c which is removed by the patch
 certainly looks wrong because it's testing errno when no error has
 occurred:
 
   if (deadpid != -1 && errno == EINTR)
 
 I wonder if changing it to this would be sufficient:
 
   if (deadpid == -1 && errno == EINTR)
 
 -- 
 Andreas Gustafsson, gson%gson.org@localhost