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

Re: [PATCH] Use subprocess.Popen for executing commands in cmdline tests

From: jeremy hinds <jeremy.hinds_at_gmail.com>
Date: Fri, 18 Apr 2008 19:01:36 -0600

I started working on getting a Windows dev environment set up so I can
at least make sure things can be expected to work before asking for
anybody else's time to test it. But I'm not quite there yet. In the
meantime, if someone else happens to get a chance to look at it, that
would be great.

On Fri, Apr 18, 2008 at 6:35 AM, Branko Èibej <brane_at_xbc.nu> wrote:
> Daniel Shahaf wrote:
>
> > jeremy hinds wrote on Wed, 9 Apr 2008 at 22:49 -0600:
> >
> >
> > > 2008/4/8 jeremy hinds <jeremy.hinds_at_gmail.com>:
> > >
> > >
> > > > Thanks for clearing that up. While rewriting the patch to handle
> > > > universal_newlines and shell=False, another question came up.
> > > >
> > > > I made universal_newlines apply on any platform rather than just
> > > > Windows, and that seemed to work fine on everything except
> > > > import_tests.import_eol_style(). The problem is that
> > > > run_and_verify_svn() has text mode hard-coded in it, so the "\r\n"
> > > > sequences were converted to "\n". (Honestly, I don't understand how
> > > > that test worked accurately on Windows before.)
> > > >
> > > > The question is whether it is better to preserve old behavior by
> never
> > > > using universal_newlines on posix, or should I add a "binary_mode"
> > > > parameter to run_and_verify_svn2()?
> > > >
> > > >
> > > >
> > > I decided err on the side of flexibility and add a "binary_mode" param
> > > to run_and_verify_svn2(). So here is my new and improved (and
> > > hopefully good) patch. Again, I've only been able to test this on
> > > Linux.
> > >
> > > [[[
> > > In cmdline tests, use subprocess.Popen for executing commands to allow
> > > support for exit-code checks on both Windows and posix systems. This
> makes
> > > the test suite require Python version >= 2.4.
> > >
> > >
> > >
> >
> > Ping. Has anyone reviewed the binary_mode changes? Could someone verify
> that the patch works on Windows too?
> >
> >
>
> Oh sorry. Looks good to me. I can't test on Windows these days, sorry.
>
> -- Brane
>
>
Received on 2008-04-19 03:02:20 CEST

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.