The safe_read() and safe_write() macros are the better option, and
these were only used in one location each. (Converted in previous
patch.)

xmalloc cannot return NULL here - on error it would fatal() already.

Fix function declaration and code to match style guide.

Change the debug3 message to print raw timestamps, rather than
convert, same as the other debug3 messages. This also then removes
the last calls to timestr(), so remove that static function as well.

strcpy would not have worked for the described purpose anyways.

I was going to rework these to remove the strcpy() calls, but
found that each of these have equivalent debug3() statements
just without the timestamps rendered as time strings.

Given that these are debug3(), and that you'd need to manually
inject a DEBUG_TIME definition somewhere to expose them, I don't
see the value of keeping them around. Especially as if you're digging
this far in to the source, it seems easier to troubleshoot off the raw
timestamp anyways IMNSHO.

Replace strcpy() with strlcpy().

Replace strcpy() with strlcpy(), and refactor the code. Fix the
function prototype to match the style guide while here.

Replace strlen(), xmalloc(), strcpy() sequence with xstrdup().
Add const annotation and fix function declaration style while here.

Cleanup other code around here as well, and replace msg_type
with msg->msg_type now that select/serial cannot swap that out.

Cleanup other src/slurmd locations as well.

Without select/serial, replace_batch_job() collapses to a call to
slurm_send_rc_msg(msg, SLURM_SUCCESS) and then returns false.

So just do that instead, and delete this entire function.

Bug 5713.

This is not recommended for use, and will be removed in 19.05.

Update slurm.spec and slurm.spec-legacy as well.

Update slurm.spec and slurm.spec-legacy as well.

Update slurm.spec and slurm.spec-legacy as well.

Alongside previous commit, fixes errors like:

error: acct_policy_handle_accrue_time: QOS normal accrue_cnt underflow

Bug 5863.

Would have been incorrectly set to 0 if begin_time hadn't been
established yet.

Bug 5863.

Pull out BlueGene specific pieces of etc/init.d that are no longer
relevant, and run autogen.sh to pick up the build system changes.

Bug 5880.

Correct spelling in symbol name

Pull out BlueGene specific pieces of etc/init.d that are no longer
relevant, and run autogen.sh to pick up the build system changes.

Bug 5880.

ESLURM_UNSUPPOERTED_GRES ==
Requested GRES option unsupported by configured SelectType plugin