Skip to content

Commit 4961f69

Browse files
authored
Merge pull request #424 from ExaWorks/fix_env_var_escaping
There is no need to escape values here since the script generator alr…
2 parents 45c7f1c + 0d10772 commit 4961f69

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/psij/executors/batch/batch_scheduler_executor.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
from threading import Thread, RLock
1010
from typing import Optional, List, Dict, Collection, cast, Union, IO
1111

12-
from .escape_functions import bash_escape
1312
from psij.launchers.script_based_launcher import ScriptBasedLauncher
1413

1514
from psij import JobExecutor, JobExecutorConfig, Launcher, Job, SubmitException, \
@@ -68,7 +67,7 @@ def _env_to_mustache(job: Job) -> List[Dict[str, str]]:
6867

6968
r = []
7069
for k, v in job.spec.environment.items():
71-
r.append({'name': k, 'value': bash_escape(v)})
70+
r.append({'name': k, 'value': v})
7271
return r
7372

7473

0 commit comments

Comments
 (0)