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

2010-04-13 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-rc-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"  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() { ... }
> 
> instead of
> 
> while() { ... }
> 
> 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-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org