[gridengine users] JSV having problem with comma in CMDARGS

[Ron Chen]

Hi Dave,

I would like to see if we can do some inter-fork collaboration here on the list.

Can you review the fix for this issue? If you think it is OK, then I will check it into the Open Grid Scheduler project, and you are free to check it into the Son of Grid Engine project.

Note that I developed this fix independently, and Oracle has fixed it in a more compact way (they have better reg exp Perl programmers!), but Oracle's code is not licensed under SISSL (but my fix is under SISSL), so it is better not to look at their fix before fixing bugs or else we won't get clean room implementations of the code.

Index: dist/util/resources/jsv/JSV.pm
===================================================================
--- dist/util/resources/jsv/JSV.pm	(revision 13)
+++ dist/util/resources/jsv/JSV.pm	(working copy)
@@ -162,7 +162,7 @@
          $param{$key} = { %h };
       } else {
          # not list command
-         if ($key eq 'A' || $val !~ /,/) {
+         if ($key =~ /CMDARG\d+/ || $key eq 'CMDNAME' || $key eq 'A' || $key eq 'N' || $val !~ /,/) {
             # the -A option can have any content, also a comma
             $param{$key} = $val;
          } else {

 -Ron


--- On Thu, 3/17/11, Rayson Ho <rayrayson at gmail.com> wrote:
> Thanks for the fix.
> 
> It's a problem with the Perl code, and I can reproduce the
> bug outside
> of SGE. So it should be a small fix and after further
> testing &
> checking we will create a patch and share the results with
> other forks
> (so not just Open Grid Scheduler will benefit from this bug
> report).
> 
> Rayson
> 
> 
> >
> > Thanks,
> > Brad
> >
> > _______________________________________________
> > users mailing list
> > users at gridengine.org
> > https://gridengine.org/mailman/listinfo/users
> >
> 
> _______________________________________________
> users mailing list
> users at gridengine.org
> https://gridengine.org/mailman/listinfo/users
>