Skip to content

Commit 4ce10d5

Browse files
committed
[OMCSessionZMQ] cleanup self._set_omc_command()
1 parent eeccdfa commit 4ce10d5

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

OMPython/OMCSession.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -310,9 +310,9 @@ def __init__(self, timeout=10.00,
310310
self._port_file = ((pathlib.Path("/tmp") if docker else self._temp_dir) / port_file).as_posix()
311311
self._interactivePort = port
312312
# set omc executable path and args
313-
self._set_omc_command(omc_path_and_args_list=["--interactive=zmq",
314-
"--locale=C",
315-
f"-z={self._random_string}"])
313+
self._omc_command = self._set_omc_command(omc_path_and_args_list=["--interactive=zmq",
314+
"--locale=C",
315+
f"-z={self._random_string}"])
316316
# start up omc executable, which is waiting for the ZMQ connection
317317
self._omc_process = self._start_omc_process(timeout)
318318
# connect to the running omc instance using ZMQ
@@ -414,7 +414,7 @@ def _getuid(self):
414414
"""
415415
return 1000 if sys.platform == 'win32' else os.getuid()
416416

417-
def _set_omc_command(self, omc_path_and_args_list):
417+
def _set_omc_command(self, omc_path_and_args_list) -> list:
418418
"""Define the command that will be called by the subprocess module.
419419
420420
On Windows, use the list input style of the subprocess module to
@@ -461,9 +461,9 @@ def _set_omc_command(self, omc_path_and_args_list):
461461
if self._interactivePort:
462462
extraFlags = extraFlags + ["--interactivePort=%d" % int(self._interactivePort)]
463463

464-
self._omc_command = omcCommand + omc_path_and_args_list + extraFlags
464+
omc_command = omcCommand + omc_path_and_args_list + extraFlags
465465

466-
return self._omc_command
466+
return omc_command
467467

468468
def _get_omhome(self, omhome: str = None):
469469
# use the provided path

0 commit comments

Comments
 (0)