<br><br><div class="gmail_quote">On Sat, Apr 14, 2012 at 10:21, Thomas Kluyver <span dir="ltr">&lt;<a href="mailto:takowl@gmail.com">takowl@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On 14 April 2012 10:17, Fernando Perez &lt;<a href="mailto:fperez.net@gmail.com">fperez.net@gmail.com</a>&gt; wrote:<br>
&gt; <a href="https://github.com/ipython/ipython/commit/74c10b3c03e377612b54eac8eb19fe0c95930683" target="_blank">https://github.com/ipython/ipython/commit/74c10b3c03e377612b54eac8eb19fe0c95930683</a><br>
&gt;<br>
&gt; but I&#39;d like in particular Thomas to have a look in case there&#39;s<br>
&gt; anything specific to our py3 machinery I should/could have done<br>
&gt; differently.  I think it&#39;s OK, but since I don&#39;t use py3 regularly, a<br>
&gt; bit of review would be good (and  in any case, for these kinds of<br>
&gt; unreviewed emergency commits, post-hoc review is always a good<br>
&gt; practice).<br>
<br>
</div>We&#39;re interpolating a bytes string into a regular string, which<br>
doesn&#39;t quite work as expected:<br>
<br>
commit = &quot;b&#39;816e3fa&#39;&quot;<br>
<br>
We&#39;ll need to explicitly decode it first.<br>
<br>
Apart from that, since the file should always be pure ASCII, it should<br>
be possible to use the built in open() function, rather than<br>
io.open(), and avoid having to do map(unicode, ...).<br>
<br>
I&#39;ll prepare a small PR to add to our stack ;-)<br></blockquote><div><br></div><div>Yes, I imagine just using builtin open and str literals is the right way to go. No need to deal with unicode for two lines of ascii text.  I just got carried away with our recently discovery that we should use io.open everywhere (clearly that excludes setup).</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
Thomas<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
IPython-dev mailing list<br>
<a href="mailto:IPython-dev@scipy.org">IPython-dev@scipy.org</a><br>
<a href="http://mail.scipy.org/mailman/listinfo/ipython-dev" target="_blank">http://mail.scipy.org/mailman/listinfo/ipython-dev</a><br>
</div></div></blockquote></div><br>