This enables node power save logic to be used in conjunction with
  node reboot (e.g. "sbatch --reboot ...")

if data is null, we return early from the function 8 lines
earlier. coverity 44794.

Coverity doesn't like the logically-dead block now that
SLURM_MIN_PROTOCOL_VERSION has been moved forward to 14.11.

no way for this check to fail, if null we wouldn't be in the loop
body to begin with. coverity issue 44804

remove macros and replace with direct do {} while (0) blocks
as is done throughout the rest of the codebase

This commit addresses some synchronization issues with respect to
  nodes powering up or rebooting.

Add slurmd "-b" option to report node rebooted at daemon start time. Used
    for testing purposes.

Remove obsolete packaging format. Project is dead, website no
longer resolves, and if the package depends on slurm-switch-elan
that's a good sign it hasn't worked for a long time.

Remove obsolete SVN $Id$ tags from top level and src/{api,common}.

reduce reliance on fixed-sized buffers for output, helps reduce
warnings from coverity et al.

split up key/value pairs in preparation for JSON output work.
xstrfmtcat exists and is cleaner than snprintf followed by xstrcat.

use a consistent line ending rather than repeat conditional block.

output format should be unchanged, and has been tested to match
on common cases and passes all relevant regression tests.

Warning flag -Wempty-body in clang-3.7 now errors on this, split
macro definition up over multiple lines to silence.

Both the original logic and modified logic failed to lock the job
data structure prior to decrementing "prolog_running" counter.

in the forward message logic.

Previously under some conditions that boot completion was ignored
and the job kept pending.

silence clang warning about unsupported flag. if we want
performance we have other ways to chase after it, the reason
for this specific flag has long since passed. if you're still
using gcc 3.4 there are plenty of other issues in modern slurm.

run autogen.sh to account for removal of auxdir/x_ac_cflags.m4

Superfluous xstrdup call to passed argument causes
a small memory leak in sreport, caught by Coverity.

Update some node feature logic to support KNL nodes.

If a user submits a job with duplcate partition names (e.g.
  "sbatch --partition=debug,debug,debug", slurm was creating
  multiple (duplicate) records in the job's partition list.
  This change prevents duplicate entries in that list.

Bug 2364

Commit fa331e30 fixes this.  The logic was bad to begin with...

uint32_t new_cpus = detail_ptr->num_tasks
	/ detail_ptr->cpus_per_task;

The / should had been * this whole time.  This was the reason we found
this in the first place.

bug 2369

# Conflicts:
#	src/slurmctld/reservation.c