Skip Menu |

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

Report information
The Basics
Id: 21614
Status: resolved
Worked: 30 min
Priority: 0/
Queue: Net-SFTP-Foreign
People
Owner: Nobody in particular
Requestors: andre [...] tomt.net
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in:
  • 0.90_01
  • 0.90_02
  • 0.90_03
  • 0.90_04
  • 0.90_05
  • 0.90_06
  • 0.90_07
  • 0.90_08
  • 0.90_09
Fixed in: (no value)

History Show all quoted text
  Fri Sep 22 11:11:36 2006 andre [...] tomt.net - Ticket created
Subject: Unportable use of chmod in get method
Quite a few platforms are unable to chmod filehandles, as done in Foreign.pm, line 1184, version 0.90_12 Changing it from: unless (chmod $perm & $numask, $fh) { To: unless (chmod $perm & $numask, $local) { Makes it work on Solaris 10, for example. Probably others too. Non-0.90 versions are in the clear, as it doesn't chmod $fh, but $local like the proposed fix.
  Fri Sep 22 13:48:14 2006 salva [...] cpan.org - Correspondence added 30 min
On Fri Sep 22 11:11:36 2006, atomt wrote: Show quoted text
> Quite a few platforms are unable to chmod filehandles, as done in > Foreign.pm, line 1184, version 0.90_12 > > Changing it from: > unless (chmod $perm & $numask, $fh) { > > To: > unless (chmod $perm & $numask, $local) { > > Makes it work on Solaris 10, for example. Probably others too. > > Non-0.90 versions are in the clear, as it doesn't chmod $fh, but $local > like the proposed fix.
Thank you for the report. I have uploaded a new version of the module to CPAN.
  Fri Sep 22 13:48:15 2006 The RT System itself - Status changed from 'new' to 'open'
  Fri Sep 22 13:48:16 2006 salva [...] cpan.org - Status changed from 'open' to 'resolved'