Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-04-15 Thread Vladimir Volovich
NP == Norbert Preining writes: it seems that appending a PID number is a wrong idea. users should not do this, since it is documented that the name of the fls file will be derived from the base name of the input file. so this scenario is not a race. ok, i see that appending PID was about the

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-04-15 Thread Vladimir Volovich
NP == Norbert Preining writes: As can bee seen both tex/pdftex first use intermediate temporary file (tex.fls / pdftex.fls) and only then rename it to $(jobname).fls So the problem here is that when I try to compile both 1.tex and 2.tex _simultaneously_ (e.g. through make -j2) there

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-04-15 Thread Karl Berry
Hi Vladimir, You are surely right that other filename-related race conditions exist, and that adding the pid to the temp file for fls is merely a palliative, not a fix. However, since that's the case that was reported, and since it was easy to do, it seemed worth doing. That's all. Thanks,

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-03-29 Thread Eddie Kohler
Karl, recorder_name = (string)xmalloc(strlen(kpse_program_name) + strlen(pid_str) + 5); E Karl Berry wrote: I made the change below (this is web2c/lib/openclose.c), let me know if problems. I hope compilers won't give any guff about the cast and sprintf. k --- openclose.c

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-03-29 Thread Karl Berry
I made the change below (this is web2c/lib/openclose.c), let me know if problems. I hope compilers won't give any guff about the cast and sprintf. k --- openclose.c (revision 17609) +++ openclose.c (working copy) @@ -37,12 +37,19 @@ static void recorder_start(void) { -/* Alas, while we

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-03-29 Thread Karl Berry
recorder_name = (string)xmalloc(strlen(kpse_program_name) + strlen(pid_str) + 5); I knew I was forgetting something obvious. Thanks. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-03-28 Thread Karl Berry
but I wonder, why use intermediate file for recorder at all? Why not to open $(jobname).fls in the first place? Don't want to overwrite the file from a previous run, I suppose. I'm not enthused about using mkstemp (and providing a fallback), but perhaps we can add the pid to the temp

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-03-28 Thread Jonathan Kew
On 28 Mar 2010, at 22:44, Karl Berry wrote: but I wonder, why use intermediate file for recorder at all? Why not to open $(jobname).fls in the first place? Don't want to overwrite the file from a previous run, I suppose. I also wonder if there are situations where the jobname is not yet

Bug#575731: [tex-k] tex/pdftex -recorder works incorrectly on parrallel TeX runs (race, breaks make -j)

2010-03-28 Thread Reinhard Kotucha
On 28 March 2010 Karl Berry wrote: but I wonder, why use intermediate file for recorder at all? Why not to open $(jobname).fls in the first place? Don't want to overwrite the file from a previous run, I suppose. Certainly a bad idea. People expect that it provides information