| Subject: | LSF version not recognized for some systems (patch attached) |
-------------------------------
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 |
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;
}
}