Skip Menu |
 

This queue is for tickets about the LSF CPAN distribution.

Report information
The Basics
Id: 77141
Status: new
Priority: 0/
Queue: LSF

People
Owner: Nobody in particular
Requestors: MOLECULES [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in:
  • 0.3
  • 0.4
  • 0.5
  • 0.6
  • 0.7
  • 0.9
Fixed in: (no value)



Subject: LSF version not recognized for some systems (patch attached)
Download (untitled) / with headers
text/plain 784b
------------------------------- First of all, thank you so much for creating this module. I look forward to automating many of my job submissions. ------------------------------- On line 17 of LSF.pm (0.9), the LSF version is checked using the following regex to check the output of the command 'lsid -V': /^LSF ([^,]+)/m However, the first line of this output for our system is the following: Platform LSF 7.0.6.134609, Sep 04 2009 Simply removing the beginning of string anchor fixes it for our system: /LSF ([^,]+)/m Since 'lsid' is used starting in version 0.3, this should affect all versions 0.3 through 0.9. Attached is a patch that would remove the beginning of string anchor, plus increment the version from 0.9 to 0.9001. Thanks, Christopher Bottoms
Subject: lsf.patch
Download lsf.patch
text/x-diff 566b
diff -Naur LSF-0.9/LSF.pm LSF-0.9001/LSF.pm --- LSF-0.9/LSF.pm 2002-10-25 11:06:16.000000000 -0500 +++ LSF-0.9001/LSF.pm 2012-05-11 10:46:49.000000000 -0500 @@ -1,6 +1,6 @@ package LSF; -$VERSION = 0.9; +$VERSION = 0.9001; use strict; use warnings; @@ -14,7 +14,7 @@ if($@){ die "Cannot find LSF executables. \$PATH is $ENV{PATH}\n"; }else{ - die "Cannot determine LSF version\n" unless $err =~ /^LSF ([^,]+)/m; + die "Cannot determine LSF version\n" unless $err =~ /LSF ([^,]+)/m; $__PACKAGE__::LSF = $1; } }


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.