explosion/spaCy
1
1
Fork 0
mirror of https://github.com/explosion/spaCy.git synced 2025-07-15 02:32:37 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix command substitution

Browse Source
This commit is contained in:
Ines Montani 2020-06-30 20:35:51 +02:00
parent 3383d1c822
commit 6da3500728
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
spacy/cli/project.py
View File

@ -15,7 +15,7 @@ from ._app import app, Arg, Opt, COMMAND, NAME
from .. import about from .. import about
from ..schemas import ProjectConfigSchema, validate from ..schemas import ProjectConfigSchema, validate
from ..util import ensure_path, run_command, make_tempdir, working_dir from ..util import ensure_path, run_command, make_tempdir, working_dir
from ..util import get_hash, get_checksum from ..util import get_hash, get_checksum, split_command
CONFIG_FILE = "project.yml" CONFIG_FILE = "project.yml"
@ -588,7 +588,7 @@ def run_commands(
) -> None: ) -> None:
"""Run a sequence of commands in a subprocess, in order. """Run a sequence of commands in a subprocess, in order.
commands (List[str]): The split commands. commands (List[str]): The string commands.
variables (Dict[str, str]): Dictionary of variable names, mapped to their variables (Dict[str, str]): Dictionary of variable names, mapped to their
values. Will be used to substitute format string variables in the values. Will be used to substitute format string variables in the
commands. commands.
@ -597,6 +597,7 @@ def run_commands(
for command in commands: for command in commands:
# Substitute variables, e.g. "./{NAME}.json" # Substitute variables, e.g. "./{NAME}.json"
command = command.format(**variables) command = command.format(**variables)
command = split_command(command)
# Not sure if this is needed or a good idea. Motivation: users may often # Not sure if this is needed or a good idea. Motivation: users may often
# use commands in their config that reference "python" and we want to # use commands in their config that reference "python" and we want to
# make sure that it's always executing the same Python that spaCy is # make sure that it's always executing the same Python that spaCy is