Re: [PATCH 2/3] Log commands as a string instead of as a list of strings.

[Chris Lumens <clumens redhat com>]

> diff --git a/iutil.py b/iutil.py
> index 3fb4057..872915d 100644
> --- a/iutil.py
> +++ b/iutil.py
> @@ -105,7 +105,7 @@ def execWithRedirect(command, argv, stdin = None, stdout = None,
>      elif stderr is None or not isinstance(stderr, file):
>          stderr = sys.stderr.fileno()
>  
> -    program_log.info("Running... %s" % ([command] + argv,))
> +    program_log.info("Running... %s" % (" ".join([command] + argv),))
>  
>      #prepare os pipes for feeding tee proceses
>      pstdout, pstdin = os.pipe()
> @@ -201,7 +201,7 @@ def execWithCapture(command, argv, stdin = None, stderr = None, root='/'):
>      elif stderr is None or not isinstance(stderr, file):
>          stderr = sys.stderr.fileno()
>  
> -    program_log.info("Running... %s" % ([command] + argv,))
> +    program_log.info("Running... %s" % (" ".join([command] + argv),))
>  
>      env = os.environ.copy()
>      env.update({"LC_ALL": "C"})
> @@ -273,7 +273,7 @@ def execWithCallback(command, argv, stdin = None, stdout = None,
>      elif stderr is None or not isinstance(stderr, file):
>          stderr = sys.stderr.fileno()
>  
> -    program_log.info("Running... %s" % ([command] + argv,))
> +    program_log.info("Running... %s" % (" ".join([command] + argv),))
>  
>      p = os.pipe()
>      p_stderr = os.pipe()

Yes, please.

- Chris