(-O1 makes this happen).

This should be handled differently in the future meaning Slurm should figure
out the null part and not rely on *printf to do that correctly.

Bug 4062

Before it would allow all sorts of things like
adding/subtracting/multiplying/etc.  It would cause warnings such as

/home/bart/slurm-tmp5/src/common/layouts_mgr.c: In function ‘_layouts_load_automerge’:
/home/bart/slurm-tmp5/src/common/layouts_mgr.c:363:21: error: ‘*’ in boolean context, suggest ‘&&’ instead [-Werror=int-in-bool-context]
   *lvalue = *lvalue * *rvalue;     \
             ~~~~~~~~^~~~
/home/bart/slurm-tmp5/src/common/layouts_mgr.c:1034:4: note: in expansion of macro ‘_entity_update_kv_helper’
    _entity_update_kv_helper(type_t, operator); \
    ^~~~~~~~~~~~~~~~~~~~~~~~
/home/bart/slurm-tmp5/src/common/layouts_mgr.c:1086:4: note: in expansion of macro ‘_layouts_load_merge’
    _layouts_load_merge(bool, s_p_get_boolean);
[tag] [reply] [−] Private Comment 16

Bug 4062

Setting -Werror tricks the test into failing on the

"error: In the GNU C Library, "major" is defined by <sys/sysmacros.h>.
 For historical compatibility, it is  currently defined by <sys/types.h>
 as well, but we plan to remove this soon. To use "major", include
 <sys/sysmacros.h> directly. If you did not intend to use a system-defined
 macro "major", you should undefine it after including <sys/types.h>."

error. Since the normal Slurm build uses -Werror, this warning on including
both headers will then cause the build itself to fail.

Bug 3982.

This was not needed when ecd5c056 was done.  The other 2 places in that
commit were correct.

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

Do not try to use the installed library, only use the library
as it is currently being built. Otherwise, compiling a newer
version of Slurm with an older version (that uses libslurmfull)
will fail, as any newer symbols that are required will be missing
from the currently-installed version.

Bug 4167.

The pre1 / rc1 are part of the release, not the version.
Switch to %rel to avoid possible namespace conflicts with %patch.

Bug 4246

and display CPU if no TRES' are requested -- except for resveration
reports.

so that there is an un-altered version of the tres list from the
database. tres_list is list of tres that are requested with --tres=str

The memcpy already handled it.

Bug 4246

Bug 4246

when a job spans a reservation that was modified (multiple records for
the same reservation).

Bug 4246

Bug 4246

Down waking nodes right after ResumeTimeout has been reached if they are
not responding. Otherwise we have to wait for ping_nodes() to handle
this work, thus SlurmdTimeout comes into play giving the sensation to
the end user that nodes got stuck in ALLOCATED# and job in CF state
until ping_nodes() decides to mark them DOWN and requeue the job.

Bug 4182