Skip to content

Commit 645f91d

Browse files
committed
fixes env vars in java
1 parent ac16a84 commit 645f91d

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

template/server/messaging.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ async def set_env_vars(self, env_vars: Dict[StrictStr, str]):
150150
elif self.language == "r":
151151
env_commands.append(f'Sys.setenv({k} = "{v}")')
152152
elif self.language == "java":
153-
env_commands.append(f'System.setProperty("{k}", "{v}")')
153+
env_commands.append(f'System.setProperty("{k}", "{v}");')
154154
elif self.language == "bash":
155155
env_commands.append(f"export {k}='{v}'")
156156
else:
@@ -199,7 +199,7 @@ async def reset_env_vars(self, env_vars: Dict[StrictStr, str]):
199199
elif self.language == "r":
200200
remove_commands.append(f"Sys.unsetenv('{key}')")
201201
elif self.language == "java":
202-
remove_commands.append(f'System.clearProperty("{key}")')
202+
remove_commands.append(f'System.clearProperty("{key}");')
203203
elif self.language == "bash":
204204
remove_commands.append(f"unset {key}")
205205
else:
@@ -232,9 +232,10 @@ async def change_current_directory(
232232
)
233233
elif language == "r":
234234
request = self._get_execute_request(message_id, f"setwd('{path}')", True)
235+
# This does not actually change the working directory, but sets the user.dir property
235236
elif language == "java":
236237
request = self._get_execute_request(
237-
message_id, f"System.setProperty('user.dir', '{path}')", True
238+
message_id, f'System.setProperty("user.dir", "{path}");', True
238239
)
239240
else:
240241
return

0 commit comments

Comments
 (0)