Both parameters will be supported for now.

Fix for sview started on a non-bluegene system to pick colors correctly when talking to a real bluegene system.

    Use PREFIX instead to avoid build errors from multiple installation
    specifications.

    reservation may use the licenses associated with it plus any compute nodes.
    Otherwise the job is limited to the compute nodes associated with the
    reservation.

    a full midplane on BlueGene computers. Treat cnodes as liceses which can
    be reserved and are consumed by jobs.

    Perl can not build if both are set.

This removes a redundant test for NODE_RESUME if the
old state was NODE_STATE_UNKNOWN, and an unreached
break.

Just a suggestion, also updates comment text.

    step names, and account names with an underscore to insure proper parsing.

Since "aprun" is used on Cray instead of srun, the --no-shell option does not
make any difference: with or without this option, the ALPS reservation is made, 
and since it is confirmed using the SID of the current shell, aprun will run 
even if the BASIL_RESERVATION_ID is not set.

NB: the patch aborts with an error message. If deciding to turn this into a
    warning, and continue processing, opt.no_shell should be disabled, since
    otherwise interactive mode (and thus job control) is disabled.

return_hostlist is not populated in validate_nodes_via_front_end,
hence never printed out.

This 
 * removes outdated and no longer applicable comments regarding
   consecutive node numbering (dating from an earlier revision);
 * fixes a typo and clarifies condition on XT/SeaStar systems.

This fixes an inconsistency: time_t is not necessarily u32, use a separate
routine to parse the absolute value and use proper time_t type.

Also tidied up code where possible.

This reduces the amount of error text printed on failure of do_basil_release():
 * parameter failures are caught by the existing calls to error(),
 * internal (ALPS) errors are printed by basil_release(),
 * there is no need to return additional error information via errno,
 * functions calling select_g_job_fini() just interpret the error, but no
   further action is taken, hence it is not necessary to indicate failure
   more than once.

The following shows how setting SLURM_ERROR/errno produces unnecessarily long error text:

 [2011-02-09T18:19:51] debug2: Processing RPC: REQUEST_CANCEL_JOB_STEP uid=21215
 [2011-02-09T18:19:51] error: PERMANENT ALPS BACKEND error: ALPS error: apsched: No entry for resId 286
 [2011-02-09T18:19:51] error: releasing ALPS resId 286 for JobId 2940 FAILED with -5
 [2011-02-09T18:19:51] error: select_g_job_fini(2940): No error

With the patch, only						       
 [2011-02-09T18:19:51] error: PERMANENT ALPS BACKEND error: ALPS error: apsched: No entry for resId 286
would be printed, which is sufficient to diagnose the problem (resId 286 had been
terminated by ALPS internally, after not receiving a confirmation quickly enough).