<div dir="ltr">Do we have an eventlet expert available to figure out if this is going to bite us?<div><br></div><div>Doug<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Sergey Shepelev</b> <span dir="ltr"><<a href="mailto:temotor@gmail.com">temotor@gmail.com</a>></span><br>
Date: Tue, Jun 4, 2013 at 1:26 PM<br>Subject: Eventlet 0.13 is going to have minor incompatible change in subprocess API<br>To: <a href="mailto:comp-lang-python-announce@moderators.isc.org">comp-lang-python-announce@moderators.isc.org</a><br>
<br><br>Hello.<br>
<br>
I'm trying to reach out for as many Eventlet users as possible. This message contains specific actions to be taken by project owners to ensure compatibility with future versions of eventlet.<br>
<br>
1. We're going to have a minor backward incompatible change. It will *only* affect you if you use Eventlet specific `check_interval` positional argument, e.g. subprocess.Popen(...).wait(0.1). Which is very unlikely, but I have to make sure. Checking will take less than minute best case.<br>

<br>
2. Please run the following search against your code base:<br>
<br>
grep -lr -E 'import.+subprocess' . |xargs grep -nE '\.wait\([^)]' |fgrep -v 'check_interval='<br>
<br>
3. If you have any results on step 3, please check that either you imported subprocess from eventlet.green or used monkey_patch and matched lines correspond to eventlet.green.subprocess.Popen objects<br>
<br>
4. If you have any results on step 4, that is, you pass check_interval as first positional argument to .wait() method of Popen object, please change it to keyword argument `check_interval=...`<br>
<br>
5. Please, spread this message to your coworkers, friends or other people who uses Eventlet in their projects.<br>
<br>
Thank you very much.<br>
<br>
<br>
If you are curious what's going on, here's the full story:<br>
Python 3.3 introduced `timeout` kwarg to lots of methods of subprocess module. The RHEL guys backported it to their Python 2.6 package. At some point, the code they used started to use subprocess from Eventlet which does not have the `timeout` kwarg.<br>

<a href="https://bitbucket.org/eventlet/eventlet/issue/89" target="_blank">https://bitbucket.org/eventlet/eventlet/issue/89</a><br>
<a href="https://bitbucket.org/eventlet/eventlet/pull-request/30" target="_blank">https://bitbucket.org/eventlet/eventlet/pull-request/30</a><br>
Now in Eventlet, I am going to introduce the `timeout` argument and to make it forward compatible with Python3, it's going to get first place, like in stdlib.<br>
To interested people, you may leave a comment in this discussion or in this Github thread: <a href="https://github.com/eventlet/eventlet/pull/34" target="_blank">https://github.com/eventlet/eventlet/pull/34</a><br>
<span class="HOEnZb"><font color="#888888">--<br>
<a href="http://mail.python.org/mailman/listinfo/python-announce-list" target="_blank">http://mail.python.org/mailman/listinfo/python-announce-list</a><br>
<br>
        Support the Python Software Foundation:<br>
        <a href="http://www.python.org/psf/donations/" target="_blank">http://www.python.org/psf/donations/</a><br>
</font></span></div><br></div></div>