srun was modified to log
Warning: can't honor --ntasks-per-node set to 8 which doesn't match the requested tasks 23 with the number of requested nodes 3.  Ignoring --ntasks-per-node.
but this tests was parsing that warning, causing a test failure.
Modify the test's parsing logic.

This reverts commit cc242de3
That patch fixed bug 2745, but breaks tests 1.89 and 1.91 on
typical Xeon processors

This also makes use of the -j (join) flag as a normal qsub does.

Fix for uninitialized variable in task binding logic, could leave tasks
    with fewer CPUs than intended.
bug 2766

unsetenv is POSIX1.2001, and should always be available.

To quote our documentation: "... we know of no AIX installations
using Slurm after 2008." It certainly hasn't been tested there
recently and has presumably been broken for several releases.

Part 1: remove all build system linkages. Cleanup of HAVE_AIX ifdef
blocks will follow.

Fix GRES task layout bug that could cause slurmctld to abort with
invalid memory reference.
bug 2766

Coverity 44891.

sizeof(optarg) is incorrect, that's the size of the pointer
not the length of the character string that must be parsed.

Coverity 53128.

Coverity 44992.

bug 2760

The src/common/getopt.h file has been removed from slurm.
Modify the test to use /usr/include/getopt.h instead.

Left behind when -h changed from help to no-header.

Needs to unlock here, not re-lock the lock.

Some tests fail if the partition is configured with Shared=exclusive.
This change disables the same tests as if select/linear is configured.