Re: [Issue 964] Changed - Temporary file created for $EDITOR is added to repository on import.

On Wed, Nov 13, 2002 at 11:49:55AM -0600, cmpilato@collab.net wrote:
> Colin Watson <cjwatson@flatline.org.uk> writes:
> > On Wed, Nov 13, 2002 at 09:44:10AM -0600, Karl Fogel wrote:
> > > For example, suppose you commit using $EDITOR and then walk out the
> > > door to go on vacation. The commit fails. You come back three weeks
> > > later, see the error message on your screen, and go looking for the
> > > log message file. Whups, sorry, svn put it in /tmp, so now it's gone,
> > > and you have to rewrite it.
> >
> > Maybe that kind of thing could be solved by outputting the attempted log
> > message after a failed commit?
>
> The problem with doing this is that you now have Subversion attempting
> to perform a non-trivial operation (disk file i/o, as opposed to
> something easier like a printf() to the screen) while recovering from
> a failure -- a failure that could have occured for any reason, not the
> least of which might be: "unable to write to disk".

Well, I was actually thinking more of outputting the log message to
stdout/stderr, so that when the committer in Karl's example comes back
to find the error message on the screen, the log message is right after
it for convenient copying-and-pasting.

-- 
Colin Watson                                  [cjwatson@flatline.org.uk]
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org