if a previous test fails it doesn't cause the others to fail.

variables.

s_p_options_t struct.

This patch switches to using the Slurm MPI port reservation method for Cray PMI ports.

A function previous believed to be local was being used by the
perl API. It was made static in commit 57345149
This is a partial reversion of that commit.

If a job or step request GRES with a count of zero, do not even
test to see if that GRES type is defined.

job_submit plugin

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

Was failing to print an error message if the debugflag value did
not include a + or -

Conflicts:
	doc/html/gres.shtml
	doc/man/man5/gres.conf.5

The CPU specification enforcement is strict rather than advisory.

based upon unused variable reported by clang

Some new functions in config_info.c are only needed in that
module and not global.

I made some format changes for what I believe is better readability plus replacing spaces in some places with tabs.

The xfree() function does nothing if the input value is NULL, so there is no reason to check first for a non-zero value.

In slurm_write_ctl_conf(), the value tmp_str was xfreed, but not initialized. This was the only significant flaw that I saw in the code.

I also restructured the code in slurm_write_ctl_conf() a bit to eliminate a redundant memory allocation and copy of tmp_str.

Most of the Slurm fields are unsigned, so they should be printed using %u rather than %d.

Add mechanism for administrator to write current Slurm configuration
to a file containing a time stamp.

Mostly dead assignments

warning.

of cpus in the job_resources_t structure so as nodes finish the correct
cpu count is displayed in the user tools.