NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

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



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

From: RVP <rvp%SDF.ORG@localhost>
To: Andreas Gustafsson <gson%gson.org@localhost>
Cc: gnats-bugs%netbsd.org@localhost, martin%netbsd.org@localhost, roland.illig%gmx.de@localhost,
        Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>, vezhlys%gmail.com@localhost
Subject: Re: install/56303: On a fresh installation, /tmp on sparc64 is not
 sticky
Date: Tue, 17 Aug 2021 22:54:28 +0000 (UTC)

 On Fri, 13 Aug 2021, Andreas Gustafsson wrote:
 
 > You wrote:
 >> As I noted before, the current progress sometimes loses the status
 >> of the gzip command, so, I've modified, and simplified, progress.c
 >> slightly so that it returns a proper exit code every time.
 >>
 >> Please try this new version; then we can see what causes tar to
 >> abort mid-stream.
 >
 > I believe my minimal change of progress.c 1.24 was sufficient to fix
 > the problem of the present PR.  Your suggested changes hint at other
 > bugs in progress(1) that also ought to be fixed, but a complete
 > replacement of the entire source file is not the appropriate way to
 > fix them.  Please file a separate PR for each remaining progress(1)
 > bug, with a patch if possible, but more importantly, with a
 > description of the problem.
 >
 
 Sorry for the late reply, but I am a bit bogged down at the moment.
 
 Re: progress.c 1.24: This test which you modified is not needed:
 
  	if (deadpid == -1 && errno == EINTR)
  		continue;
 
 By ths point all read/writes have finished, and `outpipe[1]' has
 been closed. So you can stop the progress bar code right after the
 `while (1)' loop and no interrupts can occur. This is what my
 initial patch did. The subsequent patch is more of the same: dotting
 the Is and crossing the Ts--very straight-forward stuff, but I
 wanted to ensure that progress(1) exited with a proper error code
 everytime.
 
 Hope that helps. Feel free to use any or all of the changes I made.
 
 -RVP
 


Home | Main Index | Thread Index | Old Index