Skip Menu |

This queue is for tickets about the Net-xFTP CPAN distribution.

Report information
The Basics
Id: 68801
Status: resolved
Priority: 0/
Queue: Net-xFTP
People
Owner: turnerjw784 [...] yahoo.com
Requestors: chris [...] theclonchs.com
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 0.44
Fixed in: (no value)

History Show all quoted text
  Mon Jun 13 16:49:08 2011 chris [...] theclonchs.com - Ticket created
Subject: FTPSSL ls still broken
As noted in bug report #63290, the ls command is not defined in Net::FTPSSL, and nlst must be used. The following diff will take care of it.. @@ -140,7 +140,7 @@ my $path = shift || ''; my $showall = shift || 0; my @dirlist; - @dirlist = $self->{xftp}->ls($path||'.'); + @dirlist = $self->{xftp}->nlst($path||'.'); return unless (defined $dirlist[0]); #ADDED 20070613 TO PREVENT WARNING. shift (@dirlist) if ($dirlist[0] =~ /^total \d/o); #REMOVE TOTAL LINE! my $i = 0;
  Wed Jun 15 10:47:25 2011 turnerjw784 [...] yahoo.com - Correspondence added
Oops - my bad - I addressed bug#63290 with v0.44b1 and never heard back from them & closed the bug assuming done, but never released the beta to CPAN! This exact patch, along w/some other fixes is now incorporated into v0.45 and is now making it's way around CPAN. NOTE - it's NOT on my "official" site (http://home.mesh.net/turnerjw/jim) b/c at the moment I'm having issues w/my ISP. Post back whether 0.45 addresses your issue. Thanks, Jim Turner On Mon Jun 13 16:49:08 2011, cacack wrote: Show quoted text
> As noted in bug report #63290, the ls command is not defined in > Net::FTPSSL, and nlst must be used. The following diff will take care > of it.. > > @@ -140,7 +140,7 @@ > my $path = shift || ''; > my $showall = shift || 0; > my @dirlist; > - @dirlist = $self->{xftp}->ls($path||'.'); > + @dirlist = $self->{xftp}->nlst($path||'.'); > return unless (defined $dirlist[0]); #ADDED 20070613 TO PREVENT > WARNING. > shift (@dirlist) if ($dirlist[0] =~ /^total \d/o); #REMOVE TOTAL LINE! > my $i = 0;
  Wed Jun 15 10:47:26 2011 The RT System itself - Status changed from 'new' to 'open'
  Wed Jun 15 10:47:30 2011 turnerjw784 [...] yahoo.com - Taken
  Wed Jun 15 14:07:06 2011 chris [...] theclonchs.com - Correspondence added
From: chris [...] theclonchs.com
That fixed it! Thanks! On Wed Jun 15 10:47:25 2011, TURNERJW wrote: Show quoted text
> Oops - my bad - I addressed bug#63290 with v0.44b1 and never heard back > from them & closed the bug assuming done, but never released the beta to > CPAN! This exact patch, along w/some other fixes is now incorporated > into v0.45 and is now making it's way around CPAN. NOTE - it's NOT on > my "official" site (http://home.mesh.net/turnerjw/jim) b/c at the moment > I'm having issues w/my ISP. Post back whether 0.45 addresses your issue. > > Thanks, > > Jim Turner > > > > On Mon Jun 13 16:49:08 2011, cacack wrote:
> > As noted in bug report #63290, the ls command is not defined in > > Net::FTPSSL, and nlst must be used. The following diff will take care > > of it.. > > > > @@ -140,7 +140,7 @@ > > my $path = shift || ''; > > my $showall = shift || 0; > > my @dirlist; > > - @dirlist = $self->{xftp}->ls($path||'.'); > > + @dirlist = $self->{xftp}->nlst($path||'.'); > > return unless (defined $dirlist[0]); #ADDED 20070613 TO PREVENT > > WARNING. > > shift (@dirlist) if ($dirlist[0] =~ /^total \d/o); #REMOVE TOTAL
LINE! Show quoted text
> > my $i = 0;
> >
  Wed Jan 04 13:07:12 2012 turnerjw784 [...] yahoo.com - Correspondence added
Marking closed after OP's last reply saying fixed & not hearing anything else for 6mo. Jim Turner
  Wed Jan 04 13:07:13 2012 turnerjw784 [...] yahoo.com - Status changed from 'open' to 'resolved'