<div dir="ltr">So you uploaded package without run of autopkgtest, which can check if package works and without even trying to install that package? So just blindly upgrade from upstream and upload? Huh.<br><div class="gmail_extra"><br><div class="gmail_quote">2017-11-12 9:50 GMT+01:00 Thomas Goirand <span dir="ltr"><<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11/12/2017 08:51 AM, Chris Lamb wrote:<br>
> Hi,<br>
><br>
>> python-eventlet fails to install on amd64: SyntaxError: invalid syntax<br>
><br>
>  Â Sat 11 18:37 < zigo> In Python 2, with what I just uploaded for python-eventlet, I have these issues:<br>
>  Â Sat 11 18:37 < zigo> Setting up python-eventlet (0.20.0-1) ...<br>
>  Â Sat 11 18:37 < zigo>  Â File "/usr/lib/python2.7/dist-<wbr>packages/eventlet/green/http/<wbr>client.py", line 195<br>
>  Â Sat 11 18:37 < zigo>  Â  Â _is_legal_header_name = re.compile(rb'[^:\s][^:\r\n]*\<wbr>Z').match<br>
>  Â Sat 11 18:37 < zigo>  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  ^<br>
>  Â Sat 11 18:37 < zigo> SyntaxError: invalid syntax<br>
>  Â Sat 11 18:37 < zigo>  Â File "/usr/lib/python2.7/dist-<wbr>packages/eventlet/green/http/<wbr>cookiejar.py", line 1269<br>
>  Â Sat 11 18:37 < zigo>  Â  Â yield from deepvalues(obj)<br>
>  Â Sat 11 18:37 < zigo>  Â  Â  Â  Â  Â  Â  ^<br>
>  Â Sat 11 18:37 < zigo> SyntaxError: invalid syntax<br>
>  Â Sat 11 18:37 < zigo> Does anyone know how to patch this?<br>
>  Â Sat 11 18:38 < zigo> It works perfectly in py3...<br>
>  Â Sat 11 18:40 < zigo> Looks like doing b'' for the string of the first issue is fixing the issue for py2, and works in py3.<br>
>  Â Sat 11 18:40 < zigo> I'm not sure for the yield though.<br>
>  Â Sat 11 18:44 < wRAR> of course you can't "fix" yield from for Python < 3.5<br>
>  Â Sat 11 18:45 < zigo> Hum... I fix I got it ! :)<br>
>  Â Sat 11 18:46 < wRAR> <a href="https://github.com/eventlet/eventlet/blob/master/eventlet/green/http/__init__.py#L56" rel="noreferrer" target="_blank">https://github.com/eventlet/<wbr>eventlet/blob/master/eventlet/<wbr>green/http/__init__.py#L56</a><br>
>  Â Sat 11 18:46 < wRAR> you don't need to fix the *code*<br>
>  Â Sat 11 18:47 < zigo> wRAR: I actually do...<br>
>  Â Sat 11 18:48 < wRAR> ok<br>
>  Â Sat 11 18:48 < wRAR> and for the first line do you mean you removed the r prefix?<br>
>  Â Sat 11 18:48 < zigo> wRAR: Yes. Is this wrong?<br>
>  Â Sat 11 18:48 < wRAR> do you know what does this prefix mean?<br>
>  Â Sat 11 18:49 < zigo> wRAR: I know what b means, not sure what rb does.<br>
>  Â Sat 11 18:49 < wRAR> I see<br>
>  Â Sat 11 18:49 < zigo> Which is why I'm asking in the channel.<br>
>  Â Sat 11 18:49 < wRAR> <a href="https://docs.python.org/3/library/stdtypes.html#text-sequence-type-str" rel="noreferrer" target="_blank">https://docs.python.org/3/<wbr>library/stdtypes.html#text-<wbr>sequence-type-str</a><br>
>  Â Sat 11 18:50 < wRAR> I guess you never wrote regulare expression code in Python?<br>
>  Â Sat 11 18:51 < zigo> Right.<br>
>  Â Sat 11 18:52 < zigo> Oh...<br>
>  Â Sat 11 18:52 < zigo> Indeed, removing r is a bad idea... :P<br>
><br>
> ie. the maintainer is at least aware of the issue.<br>
><br>
> Zigo, I just went to provide a patch of my own but I cannot find<br>
> the latest packaging corresponding to this version in the Vcs-Git<br>
> repository.<br>
><br>
> Regards,<br>
<br>
That's nice of you Chris. We're currently not sure where to host the VCS<br>
of this package, and I'm not sure it's going to stay there. Anyway, I've<br>
currently pushed it to:<br>
<br>
/git/openstack/python/python-<wbr>eventlet.git<br>
<br>
I've produced 2 patches already for it:<br>
fix-string-using-rb-in-<wbr>python2.patch<br>
fix-yield-from-in-py2.patch<br>
<br>
though I'm not sure they are good enough, and they are certainly not<br>
enough (ie: after applying, there's still Py 2.7 compile problems).<br>
<br>
Thanks a lot for any help you may provide, Eventlet is a key package for<br>
OpenStack, and we need it for finishing this release. I tried upgrading<br>
to 0.20.0, because there's a problem with building Neutron, which seems<br>
to be related to this (ie: unit test fails to start with the older<br>
version 0.19.0).<br>
<br>
Cheers,<br>
<br>
Thomas Goirand (zigo)<br>
<br>
______________________________<wbr>_________________<br>
Openstack-devel mailing list<br>
<a href="mailto:Openstack-devel@lists.alioth.debian.org">Openstack-devel@lists.alioth.<wbr>debian.org</a><br>
<a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/openstack-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.<wbr>org/cgi-bin/mailman/listinfo/<wbr>openstack-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>S pozdravem/Best regards<br> OndÅ™ej Nový<br> <br>Email: <a href="mailto:novy@ondrej.org" target="_blank">novy@ondrej.org</a></div><div>PGP: 3D98 3C52 EB85 980C 46A5 Â 6090 3573 1255 9D1E 064B<br>Jabber: <a href="mailto:onovy@njs.netlab.cz" target="_blank">onovy@njs.netlab.cz</a><br>ICQ: 115-674-713<br>Facebook: <a href="http://www.facebook.com/onovy" target="_blank">http://www.facebook.com/onovy</a><br>Tel/Cell: +420 777 963 207<br><div>Datová schránka: aypqr6c</div></div></div></div></div></div></div></div></div></div>
</div></div>