This patch fixes two minor bugs with bjs. Interactive job submission didn't set the NODES environ variable correctly. A check to ensure if the requested nodes are less than or equal to the nodes in a pool was added back.
Signed-off-by: Abhishek Kulkarni <[email protected]> Index: bjs.c =================================================================== --- bjs.c (revision 751) +++ bjs.c (working copy) @@ -581,7 +581,7 @@ sexp_append_atom(sx, tmp); for (l = j->nodes.next; l != &j->nodes; l = l->next) { struct node_alloc_t *n = list_entry(l,struct node_alloc_t,nodes_list); - sprintf(tmp, "%d", bjs_node_idx[n->node]->node); + sprintf(tmp, "%s", bjs_node_idx[n->node]->node->name); sexp_append_atom(sx, tmp); } client_send_sx(j->client, sx); @@ -1149,13 +1149,11 @@ return -1; } } else { - /* FIXME if (nodes > p->nnodes) { bjs_client_error(c, "The maximum number of nodes for pool" " %s is %d.", p->name, p->nnodes); return -1; - } - */ + } } /*--- Check number of seconds requested ---*/
