diff options
author | Paul Smith <psmith@gnu.org> | 2011-11-14 00:18:38 +0000 |
---|---|---|
committer | Paul Smith <psmith@gnu.org> | 2011-11-14 00:18:38 +0000 |
commit | 934f51d166848c275828f14ad5cc29fe1c3b578a (patch) | |
tree | b80216784296d9dc9f709d1afdace3b09385ea66 | |
parent | bfc3e1ca7c0c1504c9873ee1baacce73330b037e (diff) | |
download | gunmake-934f51d166848c275828f14ad5cc29fe1c3b578a.tar.gz |
In very obscure situations we may write the free token back to the pipe.
Don't do that. I couldn't come up with a repro case for this!
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | main.c | 3 |
2 files changed, 6 insertions, 2 deletions
@@ -1,8 +1,11 @@ 2011-11-13 Paul Smith <psmith@gnu.org> + * main.c (clean_jobserver): Don't write the free token to the pipe. + Change suggested by Tim Newsome <tnewsome@aristanetworks.com> + * acinclude.m4 (AC_STRUCT_ST_MTIM_NSEC): Add support for Darwin. - Patch provided by Troy Runkel <Troy.Runkel@mathworks.com> * filedef.h (FILE_TIMESTAMP_STAT_MODTIME): Ditto. + Patch provided by Troy Runkel <Troy.Runkel@mathworks.com> 2011-09-18 Paul Smith <psmith@gnu.org> @@ -3116,7 +3116,8 @@ clean_jobserver (int status) "INTERNAL: Exiting with %u jobserver tokens (should be 0)!", jobserver_tokens); else - while (jobserver_tokens--) + /* Don't write back the "free" token */ + while (--jobserver_tokens) { int r; |