Bug#638190: pu: package shelldap/0.2-1+squeeze1

["Adam D. Barratt" <adam@adam-barratt.org.uk>]

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