Hi, I'll be helping out victor too on the changes , and will try to do as much as possible on it. :)
On Tue, Jul 7, 2015 at 11:12 PM, John Dickinson <m...@not.mn> wrote: > Victor, > > 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. > > 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. > > --John > > > > > > > > On Jul 7, 2015, at 3:40 AM, Victor Stinner <vstin...@redhat.com> wrote: > > > > Hi, > > > > 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. > > > > > https://review.openstack.org/#/q/owner:%22Victor+Stinner%22+status:open+project:openstack/swift,n,z > > > > 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. > > Only "Replace dict.iteritems() with dict.items()" got a +2 yet. > > > > 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). > > > > 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? > > > > My last try fix "tox -e py34" in a single patch: > > https://review.openstack.org/#/c/199034/ > > > > Practical issue: it depends on my pyeclib pull request that I sent 3 > months ago... > > > > 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. > > > > What do you think? > > > > Victor > > > > > __________________________________________________________________________ > > OpenStack Development Mailing List (not for usage questions) > > Unsubscribe: > openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > >
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
