Skip to content

Commit

Permalink
fix: changed Launcher.launch_sherlock() to properly append sherlock_c…
Browse files Browse the repository at this point in the history
…ommand_args (#487)

Co-authored-by: pyansys-ci-bot <[email protected]>
  • Loading branch information
ansjmoody and pyansys-ci-bot authored Jan 20, 2025
1 parent 6227ba1 commit 85a6193
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions doc/changelog.d/487.documentation.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
fix: changed Launcher.launch_sherlock() to properly append sherlock_command_args
6 changes: 3 additions & 3 deletions src/ansys/sherlock/core/launcher.py
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ def launch_sherlock(
try:
_is_port_available(host, port)
except Exception as e:
print(str(e))
LOG.error(str(e))
raise e

_server_version = None
Expand All @@ -95,8 +95,8 @@ def launch_sherlock(
args.append("-singleProject")
args.append(single_project_path)
if sherlock_command_args != "":
args.append(f"{shlex.split(sherlock_command_args)}")
print(args)
args.extend(shlex.split(sherlock_command_args))
LOG.info(f"Command arguments: {args}")
subprocess.Popen(args)

sherlock = connect_grpc_channel(port, _server_version)
Expand Down

0 comments on commit 85a6193

Please sign in to comment.