Skip to content

slurm_ctl_conf_t renamed in 20.11.X#47

Merged
natefoo merged 2 commits intonatefoo:masterfrom
sandynomad:issue-46
Jan 27, 2021
Merged

slurm_ctl_conf_t renamed in 20.11.X#47
natefoo merged 2 commits intonatefoo:masterfrom
sandynomad:issue-46

Conversation

@sandynomad
Copy link
Copy Markdown
Contributor

the slurm 20.02.X structure slurm_ctl_conf_t was renamed to slurm_conf_t to 20.11.X

add a condition check for the Slurm version and then use the correct structure in the variable definition.

wickhagj and others added 2 commits January 4, 2021 12:01
…f_t to 20.11.X

add a condition check for the slurm version and then use the correct structure in the variable definition.
@CristianHuza
Copy link
Copy Markdown

I guess it's always a good idea to check issues/pull requests of a project, I was about to do a pull request for this exact same reason, wasted some precious time for a simple
sed -i 's/slurm_ctl_conf_t/slurm_conf_t/g' test/slurm_ping.c
sed -i 's/slurm_ctl_conf_t/slurm_conf_t/g' slurm_drmaa/drmaa.c

@natefoo
Copy link
Copy Markdown
Owner

natefoo commented Jan 27, 2021

Thanks! I'll get a release out for this shortly.

@natefoo natefoo merged commit 698fcf8 into natefoo:master Jan 27, 2021
This was referenced Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants