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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 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

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux