Skip Menu |

This queue is for tickets about the clearpress CPAN distribution.

Report information
The Basics
Id: 47838
Status: resolved
Priority: 0/
Queue: clearpress
People
Owner: Nobody in particular
Requestors: JCLEMENTS [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 334
Fixed in: (no value)

History Show all quoted text
  Mon Jul 13 12:47:53 2009 JCLEMENTS [...] cpan.org - Ticket created
Subject: Missing Dependency
It looks as if having Net::LDAP is required to pass the tests and so should be a dependency not optional. t/00-distribution.t ........... 2/65 # Failed test 'use ClearPress::authenticator::ldap;' # at /usr/local/share/perl/5.10.0/Test/Distribution.pm line 177. # Tried to use 'ClearPress::authenticator::ldap'. # Error: Can't locate Net/LDAP.pm in @INC (@INC contains: /root/.cpan/build/ClearPress-334-NtN2xf/blib/lib /root/.cpan/build/ClearPress-334-NtN2xf/blib/arch /etc/perl /usr/local/lib/perl/5.10.0 /usr/local/share/perl/5.10.0 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.10 /usr/share/perl/5.10 /usr/local/lib/site_perl .) at /root/.cpan/build/ClearPress-334-NtN2xf/blib/lib/ClearPress/authenticator/ldap.pm line 12.
  Mon Jan 18 09:50:23 2010 rmp [...] psyphi.net - Correspondence added
Fixed in r350. Was Test::Distribution trying to compile everything even though the ldap dep wasn't explicitly tested. Couldn't figure out how to disabled a specific compilation with T::D so I've dynamically-required Net::LDAP when needed at runtime.
  Mon Jan 18 09:50:25 2010 The RT System itself - Status changed from 'new' to 'open'
  Mon Jan 18 09:50:25 2010 rmp [...] psyphi.net - Status changed from 'open' to 'resolved'