On 2012-12-11 18:24, Thomas Kluyver wrote:
> On 11 December 2012 17:07, Laurent Gautier <lgaut...@gmail.com 
> <mailto:lgaut...@gmail.com>> wrote:
>
>     I was bugfixing 2.3.0, and since at it I implemented the suggested
>     changes to the default (dev) branch. That's completely untested;
>     someone should try.
>
>
> Just looking at the code, I doubt it will work:
>
> rp = (p.stdin, p.stdout, p.stderr)
> ...
> rversion = rp.readline()
>
> Is going to throw "AttributeError: tuple object has no attribute 'readline'", 
> surely?

Definitely. Won't work. Good catch.

>   But then, I'm not
>
>
> sure how it ever worked on Windows with popen3 anyway.

I missed a "[2]" in the rush. I just push a change that "might" fix it.

Thanks,

L.


> Thomas
>
>
>
> ------------------------------------------------------------------------------
> LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
> Remotely access PCs and mobile devices and provide instant support
> Improve your efficiency, and focus on delivering more value-add services
> Discover what IT Professionals Know. Rescue delivers
> http://p.sf.net/sfu/logmein_12329d2d
>
>
> _______________________________________________
> rpy-list mailing list
> rpy-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/rpy-list


------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
rpy-list mailing list
rpy-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/rpy-list

Reply via email to