Justin Erenkrantz <jerenkrantz@apache.org> writes:
> > > + cmd = apr_psprintf (pool, "%s %s", editor, tmpfile_name->data);
> > > +
> > > + system (cmd);
> >
> > What? No checking of the return code?
>
> Yeah, it should. But, note that the return values are going to be
> WEXITSTATUSified.
>
> RETURN VALUE
> The value returned is -1 on error (e.g. fork failed), and
> the return status of the command otherwise. This latter
> return status is in the format specified in wait(2).
> Thus, the exit code of the command will be WEXITSTA
> TUS(status). In case /bin/sh could not be executed, the
> exit status will be that of a command that does exit(127).
>
> I know that CVS detects when no log message is created and pops up
> the "abort, retry, fail" (not exactly) prompt. We may be able to
> also do that when the editor fails to run. (I forget what SVN
> does when there is no log message...) -- justin
SVN should do the same, since it will see that no edits were made
between the time the EDITOR was supposed to have been launched and
time it returned (regardless of the return value).
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Apr 29 20:45:08 2002