<div dir="ltr"><div>Hi,<br><br></div>   I'll be helping out victor too on the changes , and will try to do as much as possible on it. :)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 7, 2015 at 11:12 PM, John Dickinson <span dir="ltr"><<a href="mailto:me@not.mn" target="_blank">me@not.mn</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Victor,<br>
<br>
Thank you for working on this. Please don't give up on this work! I'll be asking others to look at these patches, and I'll try to look, myself, soon.<br>
<br>
For the dependencies (pyeclib), we'll get those as we can. The first priority is getting the Swift code, which we control, squared away. Then as the dependencies are able to support py3, we'll have the gate passing and can set it to voting.<br>
<br>
--John<br>
<br>
<br>
<br>
<br>
<br>
<br>
> On Jul 7, 2015, at 3:40 AM, Victor Stinner <<a href="mailto:vstinner@redhat.com">vstinner@redhat.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> I have 9 pending patches to fix Python 3 issues in Swift, but they didn't get much attention yet. Most of these patches replace a pattern with a new pattern to add Python 3 support in addition to Python 2 support.<br>
><br>
> <a href="https://review.openstack.org/#/q/owner:%22Victor+Stinner%22+status:open+project:openstack/swift,n,z" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/owner:%22Victor+Stinner%22+status:open+project:openstack/swift,n,z</a><br>
><br>
> The problem is that these patches are long, and so it's common to get conflicts. It takes me a lot of time just to rebase these patches.<br>
> Only "Replace dict.iteritems() with dict.items()" got a +2 yet.<br>
><br>
> I hesitate to simply give up on porting Swift to Python 3, to focus on other projects which are faster to review my Python 3 patches (ceilometer, cinder, glance, keystone, nova).<br>
><br>
> Maybe I took the wrong strategy for Swift. Instead of replacing a pattern in the whole Swift project, I should maybe try to port tests one by one to have shorter patches?<br>
><br>
> My last try fix "tox -e py34" in a single patch:<br>
> <a href="https://review.openstack.org/#/c/199034/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/199034/</a><br>
><br>
> Practical issue: it depends on my pyeclib pull request that I sent 3 months ago...<br>
><br>
> If this Swift "Fix tox -e py34" patch is merged, my pyeclib pull request is merged, and a new version of pyeclib including my fix is released, it will become possible to make the gate-swift-python34 voting to avoid Python 3 regressions. It should be nice milestone to start with shorter patches.<br>
><br>
> What do you think?<br>
><br>
> Victor<br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>