Skip Menu |

This queue is for tickets about the perl-ldap CPAN distribution.

Report information
The Basics
Id: 44110
Status: resolved
Priority: 0/
Queue: perl-ldap

People
Owner: Nobody in particular
Requestors: phil.lembo [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 0.34
  • 0.35
  • 0.36
  • 0.37
  • 0.38
  • 0.39
Fixed in: (no value)



Subject: Net::LDAP::LDIF
Can't seem to get current_lines method to work latest versions of LDIF.pm (0.17, 0.18) that come in Net::LDAP 0.34 - 0.39. Tested with this simple code: #!/usr/bin/perl use Net::LDAP; use Net::LDAP::Entry; use Net::LDAP::LDIF; my $HOME = $ENV{'HOME'}; my $infile = "$HOME/test.ldif" or die $!; my $ldif = Net::LDAP::LDIF->new($infile,'r') or die $!; while (not $ldif->eof() ) { my $entry = $ldif->read_entry(); print $ldif->current_lines(), "\n"; print "\n"; } $ldif->done; __END__; Data was standard compliant LDIF v1 (no comments) generated using ldapsearch (OpenLDAP 2.4.11). After building LDIF.pm 0.16 from 0.33 distribution and copying over latest 0.18 that came with 0.39, was able to see this code work. I am running Ubuntu Linux 8.10 x86_64. Net::LDAP was installed from cpan, not distro packages. Perl version is 5.10.0 from Ubuntu 8.10 base install, with updates.
Subject: Re: [rt.cpan.org #44110] Net::LDAP::LDIF
Date: Fri, 13 Mar 2009 18:10:20 -0500
To: bug-perl-ldap [...] rt.cpan.org
From: Graham Barr <gbarr [...] pobox.com>
Argh, this was introduced nearly 4 years ago in this commit http://git.goingon.net/?p=perl-ldap.git;a=commitdiff;h=cdeb93c0a2b4e7414455ddd7ffa7686585dfdca3 next_lines is also broken. I think next_lines was added just because the old code happened to have it available. Adding $self->{_current_lines} = $entry; just before the un-wrapping of the lines should restore things Graham.
Subject: Re: [rt.cpan.org #44110] Net::LDAP::LDIF
Date: Sat, 14 Mar 2009 10:15:03 -0400
To: bug-perl-ldap [...] rt.cpan.org
From: Phil Lembo <phil.lembo [...] gmail.com>
Thanks Graham. On Fri, Mar 13, 2009 at 7:10 PM, Graham Barr via RT < bug-perl-ldap@rt.cpan.org> wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=44110 > > > Argh, > > this was introduced nearly 4 years ago in this commit > > > http://git.goingon.net/?p=perl-ldap.git;a=commitdiff;h=cdeb93c0a2b4e7414455ddd7ffa7686585dfdca3 > > next_lines is also broken. I think next_lines was added just because > the old code happened to have it available. > > Adding $self->{_current_lines} = $entry; just before the un-wrapping > of the lines should restore things > > Graham. > > >
-- Phil Lembo e-mail: phil.lembo@gmail.com
Subject: Re: [rt.cpan.org #44110] Net::LDAP::LDIF
Date: Sat, 14 Mar 2009 10:46:49 -0500
To: bug-perl-ldap [...] rt.cpan.org
From: Graham Barr <gbarr [...] pobox.com>
Did you try the suggestion ? Did it work. Sorry I have not have time to test it myself, but if it works for you I will make the change in the next release. Graham.
Subject: Re: [rt.cpan.org #44110] Net::LDAP::LDIF
Date: Sun, 15 Mar 2009 08:53:20 -0400
To: bug-perl-ldap [...] rt.cpan.org
From: Phil Lembo <phil.lembo [...] gmail.com>
Yes. It worked perfectly. I opened up a fresh archive of 0.39, added the lines recommended to LDIF.pm in between 128 and 129, recompiled and then installed. Everything functions as required now.