On Fri, 2011-09-09 at 12:45 +0200, Salvatore Bonaccorso wrote: > On Tue, Sep 06, 2011 at 09:56:44PM +0100, Adam D. Barratt wrote: > > On Wed, 2011-08-17 at 16:46 +0200, Salvatore Bonaccorso wrote: > > ++ eval 'use IO::Socket::SSL'; > > ++ die qq{IO::Socket::SSL not installed, but is required for > > SSL or TLS connections. [...] > I agree, changing it to > > eval { require IO::Socket::SSL; }; > die qq{...} if $@; > > is probably more clean. I can change this in the proposed debdiff and > also suggest it to upstream. Using
I'm still arguing with myself a little here. Theoretically we should use the fix from unstable, but the suggested improvement seems obviously correct (and not just because I suggested it *g*). > > > Is it possible that I can upload an updated package with 1) upstreams > > > fix on the error message, and 2) furthermore add of Recommends on > > > > > > > > > libio-socket-ssl-perl? > > > > Hmmm. We generally don't change dependencies in stable updates; I'm > > debating the extent to which recommends should be treated in the same > > way. [...] > I would like to have it more easely for users of shelldap, and I had > already two bugreports about it. So if at least if the error message > could be clearer it would be an improvement. I think I'd prefer that option, to be honest. Regards, Adam -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org