Old functions were completely wrong, but not used on most systems

Old logic uses some vestigial power management logic that no longer
applies. It creates a job allocation, but the job step blocks
indefinitely.

Once the first complete job is found, there is no need to check for
its dependencies as they will not stop another job from starting

Added for MR+ operation

Conflicts:
	NEWS

Conflicts:
	META

since left the system.

to compile correctly on newer compilers

Signed-off-by: Danny Auble <da@schedmd.com>

If job involved in dependency completes and is purged the logic
used to test for circular dependencies can use the invalid pointer
and generate an invalid memory reference before the pointer is
cleared from the dependency list data structure.

Without this change, allocations less than a whole node can result
in an incorrect task binding for power7 processors

Without this change, the testing only happens with task/affinity

This bug was introduced in 2.5.2 for the case where a GPU count was
configured, but without device files. Unfortunately it left the
CUDA_VISIBLE_DEVICES environment variable unset if GPUs were configured,
but the job did not request any of them.

Conflicts:
	NEWS

The acct_gather_energy_ipmi.so and acct_gather_energy_rapl.so
files are conditionally built

rpmbuild is generating an error otherwise:
error: line 99: Illegal char '-' in: Release: 0-pre1