The patch looks nice, but could you provide us a scenario where this patch is needed? In which situation are you trying to run a command and you need to specify a different set of environment variables other than the default ones (i.e. the env inherited by the main process)? I couldn't figure this out by reading the patch's description. On Fri, Apr 17, 2015 at 4:29 PM Jose Ricardo Ziviani < joserz@linux.vnet.ibm.com> wrote:
- The called may want to pass a custom environment variable to run_command and now this is possible with this new parameter.
Signed-off-by: Jose Ricardo Ziviani <joserz@linux.vnet.ibm.com> --- src/kimchi/utils.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/kimchi/utils.py b/src/kimchi/utils.py index d71338a..2017cbe 100644 --- a/src/kimchi/utils.py +++ b/src/kimchi/utils.py @@ -165,11 +165,12 @@ def check_url_path(path): return True
-def run_command(cmd, timeout=None): +def run_command(cmd, timeout=None, env=None): """ cmd is a sequence of command arguments. timeout is a float number in seconds. timeout default value is None, means command run without timeout. + env is the environment variable passed to the command. """ # subprocess.kill() can leave descendants running # and halting the execution. Using psutil to @@ -193,7 +194,8 @@ def run_command(cmd, timeout=None):
try: proc = subprocess.Popen(cmd, stdout=subprocess.PIPE, - stderr=subprocess.PIPE) + stderr=subprocess.PIPE, + env=env) if timeout is not None: timer = Timer(timeout, kill_proc, [proc, timeout_flag]) timer.setDaemon(True) -- 1.9.1
_______________________________________________ Kimchi-devel mailing list Kimchi-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel