----- "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-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to