Fix an ordering problem that crept in due to the change in MCA param system. One MCA param would set a value, and then we did a hard reset of that value before testing another MCA param, thus removing a critical value for proper operation of the first param.

cmr:v1.7.3:reviewer=brbarret

This commit was SVN r28788.

orte/runtime/orte_mca_params.c

@@ -545,6 +545,14 @@ int orte_register_params(void)
                                  OPAL_INFO_LVL_9, MCA_BASE_VAR_SCOPE_READONLY,
                                  &orte_allocation_required);
 
+    /* whether or not to map stddiag to stderr */
+    orte_map_stddiag_to_stderr = false;
+    (void) mca_base_var_register ("orte", "orte", NULL, "map_stddiag_to_stderr",
+                                  "Map output from opal_output to stderr of the local process [default: no]",
+                                  MCA_BASE_VAR_TYPE_BOOL, NULL, 0, 0,
+                                  OPAL_INFO_LVL_9, MCA_BASE_VAR_SCOPE_READONLY,
+                                  &orte_map_stddiag_to_stderr);
+
    /* generate new terminal windows to display output from specified ranks */
    orte_xterm = NULL;
    (void) mca_base_var_register ("orte", "orte", NULL, "xterm",
@@ -565,14 +573,6 @@ int orte_register_params(void)
         orte_map_stddiag_to_stderr = true;
     }
 
-    /* whether or not to map stddiag to stderr */
-    orte_map_stddiag_to_stderr = false;
-    (void) mca_base_var_register ("orte", "orte", NULL, "map_stddiag_to_stderr",
-                                  "Map output from opal_output to stderr of the local process [default: no]",
-                                  MCA_BASE_VAR_TYPE_BOOL, NULL, 0, 0,
-                                  OPAL_INFO_LVL_9, MCA_BASE_VAR_SCOPE_READONLY,
-                                  &orte_map_stddiag_to_stderr);
-
     /* whether or not to forward SIGTSTP and SIGCONT signals */
     orte_forward_job_control = false;
     (void) mca_base_var_register ("orte", "orte", NULL, "forward_job_control",