Fixed handling of env vars in Slurm. The existing template made some

assumptions about how the process worked that were incorrect. Specifically,
you cannot issue multiple `#SBATCH --export` directives and expect them
to add up. Only one should be used.

Author: hategan

src/psij/executors/batch/slurm/slurm.mustache

@@ -8,13 +8,6 @@
 #SBATCH --chdir="{{.}}"
 {{/job.spec.directory}}
 
-{{#job.spec.inherit_environment}}
-#SBATCH --export=ALL
-{{/job.spec.inherit_environment}}
-{{^job.spec.inherit_environment}}
-#SBATCH --export=NONE
-{{/job.spec.inherit_environment}}
-
 {{#job.spec.resources}}
     {{#exclusive_node_use}}
 #SBATCH --exclusive
@@ -80,9 +73,17 @@ export PSIJ_NODEFILE
 
 
 {{#env}}
-#SBATCH --export={{name}}={{value}}
+export {{name}}={{value}}
 {{/env}}
 
+{{#job.spec.inherit_environment}}
+#SBATCH --export=ALL{{#env}},{{name}}{{/env}}
+{{/job.spec.inherit_environment}}
+{{^job.spec.inherit_environment}}
+#SBATCH --export={{#env}}{{name}},{{/env}}
+{{/job.spec.inherit_environment}}
+
+
 {{!redirect output here instead of through #SBATCH directive since SLURM_JOB_ID is not available
 when the directives are evaluated; the reason for using the job id in the first place being the
 same as for the exit code file.}}