kallithea Changeset - 08d439bfbd8c

Changeset - 08d439bfbd8c

Parent rev.

Child rev.

[Not reviewed]

beta

0 2 0

Marcin Kuzminski - 13 years ago 2013-05-09 13:29:03
marcin@python-works.com

fixed handling shell argument in subprocess calls, it always was hardcoded even when passed properly in arguments

2 files changed with 4 insertions and 3 deletions:

rhodecode/lib/vcs/backends/git/repository.py

rhodecode/lib/vcs/subprocessio.py

0 comments (0 inline, 0 general)

rhodecode/lib/vcs/backends/git/repository.py

➞

Show inline comments

@@ @@ -125,10 +125,11 @@ class GitRepository(BaseRepository): @@
         cmd = [_git_path] + _copts + cmd
         if _str_cmd:
             cmd = ' '.join(cmd)
         try:
             _opts = dict(
                 env=gitenv,
-                shell=False,
+                shell=True,
+            )
             _opts.update(opts)
             p = subprocessio.SubprocessIOChunker(cmd, **_opts)

rhodecode/lib/vcs/subprocessio.py

➞

Show inline comments

@@ @@ -342,10 +342,10 @@ class SubprocessIOChunker(object): @@
             input_streamer.start()
             inputstream = input_streamer.output
         _shell = kwargs.get('shell', True)
         if isinstance(cmd, (list, tuple)):
             cmd = ' '.join(cmd)
         _shell = kwargs.get('shell') or True
         kwargs['shell'] = _shell
         _p = subprocess.Popen(cmd,
             bufsize=-1,
@@ @@ -353,7 +353,7 @@ class SubprocessIOChunker(object): @@
             stdout=subprocess.PIPE,
             stderr=subprocess.PIPE,
             **kwargs
+            )
+        )
         bg_out = BufferedGenerator(_p.stdout, buffer_size, chunk_size, starting_values)
         bg_err = BufferedGenerator(_p.stderr, 16000, 1, bottomless=True)

0 comments (0 inline, 0 general)