Skip to content

Commit d486ad1

Browse files
authored
Merge pull request #40 from DavidCEllis/cleanup_launch
Make sure lockfile installs are done with --no-deps
2 parents 069f58f + e7ae420 commit d486ad1

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/ducktools/env/catalogue.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -301,6 +301,7 @@ def _create_venv(
301301
"install",
302302
"--python",
303303
env.python_path,
304+
"--no-deps",
304305
"-r",
305306
requirements_path,
306307
]
@@ -310,6 +311,7 @@ def _create_venv(
310311
"--python",
311312
env.python_path,
312313
"install",
314+
"--no-deps",
313315
"-r",
314316
requirements_path,
315317
]

src/ducktools/env/manager.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -340,10 +340,6 @@ def _launch_script(
340340
# Update environment variables for access from subprocess
341341
os.environ.update(env_vars)
342342

343-
def sigint_handler(signum, frame):
344-
# Parent process should *not* terminate on keyboard interrupt
345-
pass
346-
347343
# Ignore the keyboard interrupt signal in parent process while subprocess is running.
348344
with _ignore_keyboardinterrupt():
349345
result = _laz.subprocess.run(

0 commit comments

Comments
 (0)