Hello,

On Sat, May 16, 2009 at 6:01 PM, Y Giridhar Appaji Nag <app...@debian.org>wrote:

> Hi Cody,
>
> Sorry this one got dropped off the radar, I thought I replied already.
>
> On 08/11/10 03:40 -0500, Cody A.W. Somerville said ...
> > I've filed a bug to get this patch included in Ubuntu and figured it
> would be
> > of interest to Debian as well. You can find the bug report and patch at
> > https://bugs.launchpad.net/ubuntu/+source/dput/+bug/295613
>
> I am afraid I am a bit reluctant to accept this patch in its current form.
>  I
> would've preferred that the sftp support were implemented directly using
> the
> paramiko python modules rather than using bzr.  I wouldn't like to force
> dput
> users who want sftp support for install brz.


I can certainly understand your reluctance. Initially, I had wanted to
directly implement it with paramiko as well but quickly realized it would be
a much bigger task than I originally anticipated. Using bzrlib's transport
module provides some very important features such as automatically
respecting one's openssh configuration, automatically using available keys,
automatically dropping back to openssh if paramiko is not available, etc. By
using bzrlib, we get this functionaltiy for free and a group of paid
developers to maintain it.

It would be great if you could accept this patch. You have my promise that I
won't be offended if someone comes along and decides to rewrite it to not
require bzrlib. If you don't accept it, that won't offend me either.


> As for the second bit about "ppa:",
> https://help.launchpad.net/Packaging/PPA
> doesn't seem to advocate that sort of usage.  Has that been an often used
> feature of dput in Ubuntu after it has been implemented?  (The current
> dput.cf
> in Ubuntu continues to have incoming = ~%(ppa)s/ubuntu despite the above
> help
> doc indicating that it should be ~<your_launchpad_id>/<ppa_name>/ubuntu/
> which
> means that the feature is probably not being used).


This feature is indeed being used (at least by folks who have upgraded to
the recently released Ubuntu 9.04 / Jaunty). Infact, I was asked to write up
a blog post article which was published by the Launchpad team in the
Launchpad blog (
http://blog.launchpad.net/ppa/simplifying-dputcf-for-multiple-ppas).

Thanks for pointing out that documentation though. I'll have someone update
that.


> Cheers,
>
> Giridhar
>

Cheers,

-- 
Cody A.W. Somerville
Software Systems Release Engineer
Foundations Team
Custom Engineering Solutions Group
Canonical OEM Services
Phone: +1-781-850-2087
Cell: +1-506-471-8402
Email: cody.somervi...@canonical.com

Reply via email to