Disable test if underlying select/linear use

Modify qsub test to explicitly create and destroy the error files
  to avoid leaving around a bunch of error files (even if they are
  normally empty).

Provide better insight into the failure

Turns out this never worked, ever.  What used to happen is if the protocol_version that was
read in didn't match the rpc_version given to unpack things was just 0.  What this does
now is set the rpc_version to what was stored making it all good.

Most of these tests could be modified to work with ALPS, but since
  ALPS support is going way soon, just disable the tests as appropriate.

correctly.

Revert logic originally added for bug 3166.
  Revisit as time permits.
bug 3166

job_queue_rec wasn't being free'd before continues and breaks.

Add new-lines in some log messages to better reflect the actual I/O
  to apbasil.

unbuffered stdout.  After that commit 4ce0be75 was added which appeared to bandaid the
fallout making the only place using the unbuffered option was introduced in the former commit and
only really have unbuffered output on a PTY.  The latter commit would potentially send back
unfinished output without a newline causing jumbled output when expecting buffered output thus
resulting in potentially jumbled/unbuffered output.

I was able to reproduce this regularly with test1.31.

In any case making it so the code is clearer to read is a win.

This reverts commit a6148002.

Leaving RPC changes in place. Will add back into 17.11.
Commit: 4874f988

Interpet all format options in output/error file to log prolog errors. Prior
    logic only supported "%j" (job ID) option.
bug 3354

We will need to reference these functions from slurmd/slurmd/req.c

Bug 3431

Commit c08f8922 introduced a bug
  that results in truncated job stdout/stderr.
bug 3166

This reverts commit 43f4a07a.

This reverts commit 0003cec5.

This reverts commit d0d64158.

This reverts commit 265d60e8.

This reverts commit 69af0447.