Bug#881287: python-daiquiri: Add python2 support

2017-11-20 Thread Corey Bryant
On Mon, Nov 20, 2017 at 2:49 AM, Chris Lamb wrote: > Chris Lamb wrote: > > > Alas your patch doesn't seem to work properly here: > > [..] > > Gentle ping on this? :) > > > Sorry, I don't know how I missed your previous message. If you don't mind I don't think we should add the py2 support. I'm fi

Bug#881287: python-daiquiri: Add python2 support

2017-11-19 Thread Chris Lamb
Chris Lamb wrote: > Alas your patch doesn't seem to work properly here: [..] Gentle ping on this? :) Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org / chris-lamb.co.uk `-

Bug#881287: python-daiquiri: Add python2 support

2017-11-09 Thread Chris Lamb
Hi Corey, > Good point. I think we just need to get moving on our end. We currently > only ship py2 for core openstack packages because upstream hasn't > completely moved to py3 Happy to apply in that case; I was just checking it wasn't because "there is no Python 2 package so we must make one...

Bug#881287: python-daiquiri: Add python2 support

2017-11-09 Thread Corey Bryant
On Thu, Nov 9, 2017 at 3:24 PM, Chris Lamb wrote: > Hi Corey, > > Hi Chris, > > In Ubuntu, the attached patch was applied to achieve the following: > > Thanks for the patch. However, is Python 2 support explicitly required by > a user or reverse-dependency? > > Good point. I think we just need

Bug#881287: python-daiquiri: Add python2 support

2017-11-09 Thread Chris Lamb
Hi Corey, > In Ubuntu, the attached patch was applied to achieve the following: Thanks for the patch. However, is Python 2 support explicitly required by a user or reverse-dependency? I would like to avoid introducing further Python 2 packages into the world - just another we will have to remove

Bug#881287: python-daiquiri: Add python2 support

2017-11-09 Thread Corey Bryant
Package: python-daiquiri Version: 1.3.0-1 Severity: normal Tags: patch User: ubuntu-de...@lists.ubuntu.com Usertags: origin-ubuntu bionic ubuntu-patch Dear Maintainer, In Ubuntu, the attached patch was applied to achieve the following: * d/control, d/rules, d/tests/*: Add python 2 support. T