Tweaks

src/huggingface_hub/repository.py

@@ -493,7 +493,7 @@ def clone_from(self, repo_url: str, use_auth_token: Union[bool, str, None] = Non
         # For error messages, it's cleaner to show the repo url without the token.
         clean_repo_url = re.sub(r"https://.*@", "https://", repo_url)
         try:
-            run_subprocess("git lfs install", self.local_dir)
+            run_subprocess("git lfs install".split(), self.local_dir)
 
             # checks if repository is initialized in a empty repository or in one with files
             if len(os.listdir(self.local_dir)) == 0:

src/huggingface_hub/utils/__init__.py

@@ -16,8 +16,17 @@
 
 import subprocess
 
+from .logging import get_logger
+
+
+logger = get_logger(__name__)
+
 
 def run_subprocess(command, folder, check=True) -> subprocess.CompletedProcess:
+    if isinstance(command, str):
+        logger.error("`run_subprocess` should be called with a list of strings.")
+        command = command.split()
+
     return subprocess.run(
         command,
         stderr=subprocess.PIPE,