Fixed the PR title and commit messages. Thanks!

------
Joe Mader
Plexxi
443.703.8088

On Wed, May 9, 2018 at 7:19 PM, IWAMOTO Toshihiro <iwam...@valinux.co.jp>
wrote:

> On Thu, 10 May 2018 03:03:00 +0900,
> Joe Mader wrote:
> >
> > [1  <multipart/alternative (7bit)>]
> > [1.1  <text/plain; UTF-8 (7bit)>]
> > now that this upstream issue has been solved in eventlet 0.22:
> >
> > https://github.com/eventlet/eventlet/issues/401
> >
> > can the requirements (
> > https://github.com/osrg/ryu/blob/master/tools/pip-requires) be updated
> to
> > include eventlet 0.22 (and newer)?
>
> > we have a standalone project (not built atop OpenStack) which uses ryu
> 4.18
> > and eventlet 0.20.1 which currently presents a version conflict.
> >
> > i've submitted the following PR in the hopes that automated tests will
> pass:
> >
> > https://github.com/osrg/ryu/pull/74
> >
> > thanks in advance for your consideration.
>
> The basic idea is to sync the requirement with OpenStack, and it has
> been allowing > 0.21 for a while. (Its tempest tests are still using
> eventlet==0.20.0. though.)
>
> The commit message should say "eventlet >= 0.22" or "eventlet > 0.21",
> but otherwise it looks fine to me.
>
> --
> IWAMOTO Toshihiro
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to