[GE issues] [Issue 3239] -terse option in sge_request files is ignored

rayson rayrayson at gmail.com
Sat Feb 13 02:04:24 GMT 2010


http://gridengine.sunsource.net/issues/show_bug.cgi?id=3239






------- Additional comments from rayson at sunsource.net Fri Feb 12 18:04:23 -0800 2010 -------
The fix is to check all the option lists, not just the options from the command line:

hccgalaxy:/tmp/s/source> cvs diff -C4 clients/qsub/qsub.c
Index: clients/qsub/qsub.c
===================================================================
RCS file: /cvs/gridengine/source/clients/qsub/qsub.c,v
retrieving revision 1.71
diff -C 4 -r1.71 qsub.c
*** clients/qsub/qsub.c 23 Jul 2009 10:06:23 -0000      1.71
--- clients/qsub/qsub.c 13 Feb 2010 02:03:05 -0000
***************
*** 168,182 ****
        SGE_EXIT((void**)&ctx, 0);
     }

     /*
-     * Check if -terse is requested
-     */
-    if (opt_list_has_X(opts_cmdline, "-terse")) {
-       has_terse = true;
-    }
-
-    /*
      * We will only read commandline options from scripfile if the script
      * itself should not be handled as binary
      */
     if (opt_list_is_X_true(opts_cmdline, "-b") ||
--- 168,175 ----
***************
*** 200,207 ****
--- 193,207 ----
      */
     opt_list_merge_command_lines(&opts_all, &opts_defaults,
                                  &opts_scriptfile, &opts_cmdline);

+    /*
+     * Check if -terse is requested
+     */
+    if (opt_list_has_X(opts_all, "-terse")) {
+       has_terse = true;
+    }
+
     /* If "-sync y" is set, wait for the job to end. */
     /* Remove all -sync switches since cull_parse_job_parameter()
      * doesn't know what to do with them. */
     while ((ep = lGetElemStr(opts_all, SPA_switch, "-sync"))) {

------------------------------------------------------
http://gridengine.sunsource.net/ds/viewMessage.do?dsForumId=36&dsMessageId=244500

To unsubscribe from this discussion, e-mail: [issues-unsubscribe at gridengine.sunsource.net].



More information about the gridengine-users mailing list