remove unneeded variables
Merge remote-tracking branch 'origin/slurm-2.5'

Conflicts:
	META
news for next tag
Merge remote-tracking branch 'origin/slurm-2.5'

Conflicts:
	NEWS
news for next tag
Fix for bad index
remove duplicate code
Fix minor threading issue (shouldn't ever really matter)
Gres fix for requeued jobs.
Gres accounting - Fix regression in 2.5.5 for keeping track of gres
requested and allocated.
Tag for 2.5.6
Merge remote-tracking branch 'origin/slurm-2.5'

Conflicts:
	META
News update for next version
Merge remote-tracking branch 'origin/slurm-2.5'

Conflicts:
	NEWS
No where else in the code is the return code checked here.  With the advent
of the ThreadID DebugFlag this happens to make it so no jobs can run.

Currently we do not see any negative effects of not checking the return
code.  But this needs to probably be more thought out.

Possibly revert 27564d627bae75cceb70ff72466bb2901779ec0d
Document the procedure to add files to Slurm
re-enable log file use when daemons in the foreground

Partial revert of commit 3a6bd33641fe3dcb7d9b759342cf47d5a773d0b
Log out of memory to both the log file and stderr if daemon in foreground

Previously nothing would be printed to stderr of foreground program on OOM.

Eliminate the need for log_fp() function
Fix for linking to the select/cray plugin to not give warning about
undefined variable.
Merge remote-tracking branch 'origin/slurm-2.5'