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

Re: [PATCH] Python sub-process error checking

From: <kfogel_at_collab.net>
Date: 2003-04-08 04:40:07 CEST

Philip Martin <philip@codematters.co.uk> writes:
> The following patch makes the regression tests raise an error if a
> process started by run_command crashes. This results in better error
> checking on Unix, a command that crashes will be detected
> automatically rather than relying on the crash being detected by the
> absence of some effect. The disadvantage is that if a test is written
> that relies on this new behaviour then it may not be effective on
> Windows, it may pass when it should fail.
> So, is this a good idea?

Because of the concerns you mentioned, I think it's probably not a
good idea. As annoying as it is, we just need to write our tests to
notice the lack of output (or other external symptom) of a crash
program, instead of noticing the crash itself. Otherwise, people
developing on Unix would be tempted to write tests depending on the
behavior, and those tests would be no good on other platforms.

(If this were ever solved portably in Python, then we could switch.)

Not a veto, though. If a majority of the other committers think this
is still a good idea, that's fine.

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Apr 8 05:22:59 2003

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.