Skip to content

adding a delimiter option for --slurm_option

Lucas Frérot requested to merge prs513rosewood/blackdynamite:lucas into master

Salut @ganciaux ,

I've made some changes to how --slurm_option takes its arguments: instead of a comma-separated list of options to forward to slurm, it takes a single string and splits it with the value of --option_delimiter. This allows to do --slurm_option='--exclude=ln[01,02,03]', which did not work properly before. The default delimiter is a comma for backward compatibility.

I thought this could be generalized to other coatings, let me know what you think.

Merge request reports