[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: replace autom4te output file atomically
From: |
Ralf Wildenhues |
Subject: |
Re: replace autom4te output file atomically |
Date: |
Tue, 19 Aug 2008 08:03:25 +0200 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
Hi Ben,
* Ben Pfaff wrote on Tue, Aug 19, 2008 at 07:15:36AM CEST:
> >
> > * bin/autom4te.in (handle_output): Atomically replace the output
> > file, in most circumstances, instead of overwriting it, to avoid
> > leaving a partially written output file.
>
> I sent this revised patch to address@hidden about a
> week ago, but I've seen no comments on this version. Should I
> re-send it?
No need. I'm stumbling over the testsuite failure though
(and wondering slightly if this can in any way be a destabilizing
change).
Cheers,
Ralf
# -*- compilation -*-
6. tools.at:217: testing ...
../../autoconf/tests/tools.at:226: autom4te -o file file.m4
../../autoconf/tests/tools.at:228: cat file
--- - 2008-08-19 07:54:57.355633336 +0200
+++ /tmp/autoconf/build/tests/testsuite.dir/at-groups/6/stdou
@@ -1,2 +1,3 @@
-right
+m4_syscmd([kill $PPID])
+wrong
6. tools.at:217: 6. autom4te atomic replacement of output
(tools.at:217): FAILED
- replace autom4te output file atomically, Ben Pfaff, 2008/08/12
- Re: replace autom4te output file atomically, Ralf Wildenhues, 2008/08/12
- Re: replace autom4te output file atomically, Ben Pfaff, 2008/08/13
- Re: replace autom4te output file atomically, Ben Pfaff, 2008/08/19
- Re: replace autom4te output file atomically,
Ralf Wildenhues <=
- Re: replace autom4te output file atomically, Ben Pfaff, 2008/08/19
- Re: replace autom4te output file atomically, Ralf Wildenhues, 2008/08/20
- Re: replace autom4te output file atomically, Eric Blake, 2008/08/20
- Re: replace autom4te output file atomically, Ben Pfaff, 2008/08/21
- Re: replace autom4te output file atomically, Ralf Wildenhues, 2008/08/21
- Re: replace autom4te output file atomically, Eric Blake, 2008/08/19