Bug#577340: [lemonldap-ng-dev] Re: Bug in Net-LDAP causing FTBFS in lemonldap-ng [was: Processed: reassign]

2010-04-14 Thread gregor herrmann 
On Wed, 14 Apr 2010 07:44:23 +0200, Xavier wrote:

 Lemonldap::NG use this feature for a long time, and it has been
 tested many times on the CPAN
 (http://www.cpantesters.org/distro/L/Lemonldap-NG-Portal.html), and
 today only 1 Debian amd64 test has failed. I don't understand
 why...

The last tests there seem to be from 4th March.
perl-ldap-0.4001 (and also -0.40), where Net::LDAP::Constant changed,
is younger than that.

So we have a change in libnet-ldap-perl that causes a FTBFS in
Lemonldap::NG; if there's a bug in perl-ldap or if Lemonldap::NG uses
it wrong is the good question; and of course how to fix it together :)

Cheers,
gregor

-- 
 .''`.   http://info.comodo.priv.at/ -- GPG Key IDs: 0x8649AA06, 0x00F3CFE4
 : :' :  Debian GNU/Linux user, admin,  developer - http://www.debian.org/
 `. `'   Member of VIBE!AT  SPI, fellow of Free Software Foundation Europe
   `-You're dead, Jim.  -- McCoy, Amok Time, stardate 3372.7 



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#577340: [lemonldap-ng-dev] Re: Bug in Net-LDAP causing FTBFS in lemonldap-ng [was: Processed: reassign]

2010-04-13 Thread Xavier 
- Damyan Ivanov d...@debian.org wrote :
 ...
 A bit of explaination would have been nice :/

Hi,

sorry, it was the first time I use reassign

 After some bouncing of ideas on IRC, it appears the bug is really in 
 Net-LDAP, but only lemonldap-ng's test suite manages to trigger it.
 
 The read-only value in question is $_, which is aliased to a constant.
 
 Thanks to the world-dominating search engine, we have found this 
 educational link[1]
 
 [1] http://www.perlmonks.org/?node_id=570712#default_unlocalized
 
 In essence, this doesn't work:
 
 perl -we'for(1,2) { require Net::LDAP }'
 
 How lemonldap-ng's test suite manages to alias $_ to a constant is not
 yet known, but the details are not important. The above code sould 
 work even if it looks weird.

Lemonldap::NG loads 2 modules (following the configuration): authentication 
and userDB in a loop where $_ points to one of those words (as a constant). 
In the test which failed, both inherits from Lemonldap::NG::Portal::_LDAP which 
contains use Net::LDAP.

 My proposal is to patch Constants.pm to use
 
 local $_;
 while(DATA) { ... }
 
 instead of
 
 while(DATA) { ... }
 
 Any objections?

Lemonldap::NG use this feature for a long time, and it has been tested many 
times on the CPAN 
(http://www.cpantesters.org/distro/L/Lemonldap-NG-Portal.html), and today only 
1 Debian amd64 test has failed. I don't understand why...

Best regards,
Xavier



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org