[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] external editor, does this work on Windows...

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2002-05-09 20:42:25 CEST

cmpilato@collab.net writes:

> > * subversion/clients/cmdline/util.c (svn_cl__edit_externally): Check
> > VISUAL before EDITOR. Add system() return value check. Check all
> > function return values. Don't always return success.
> I get errors on undefined macros WIFEXITED, WIFSIGNALED, WTERMSIG,
> WEXITSTATUS. I don't think those things are portable.

I wondered what VC would make of those. Is there a <sys/wait.h> file?

I suppose we have to just print the raw return value, which
unfortunately makes it harder for a user to determine why the command
failed. Can we at least assume that zero is success?

Looking at APR's Win32 implementation of apr_proc_wait for inspiration
I see that it doesn't even set the exitwhy return value. This could
be a problem, since on Unix subversion should really be checking that
before deciding that an exit status of zero indicates success.

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu May 9 20:43:32 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.