[1/1] pwclient: Fix pwclient am output formatting

Message ID 20180726071114.12838-1-petr.vorel@gmail.com
State Accepted
Headers show
Series
  • [1/1] pwclient: Fix pwclient am output formatting
Related show

Commit Message

Petr Vorel July 26, 2018, 7:11 a.m.
repr() print unicode prefix for string:
$ pwclient git-am N
Applying patch #N using u'git am'

Remove it:
$ pwclient git-am N
Applying patch #918868 using "git am"

git mixes single and double quotes, use double quotes which are more
frequently used.

Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
 patchwork/bin/pwclient | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Daniel Axtens Aug. 26, 2018, 9:21 a.m. | #1
Hi Petr,

Applied, thank you!

Regards,
Daniel

> repr() print unicode prefix for string:
> $ pwclient git-am N
> Applying patch #N using u'git am'
>
> Remove it:
> $ pwclient git-am N
> Applying patch #918868 using "git am"
>
> git mixes single and double quotes, use double quotes which are more
> frequently used.
>
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
>  patchwork/bin/pwclient | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/patchwork/bin/pwclient b/patchwork/bin/pwclient
> index 79137b0..2020a8c 100755
> --- a/patchwork/bin/pwclient
> +++ b/patchwork/bin/pwclient
> @@ -327,8 +327,8 @@ def action_apply(rpc, patch_id, apply_cmd=None):
>          print('Applying patch #%d to current directory' % patch_id)
>          apply_cmd = ['patch', '-p1']
>      else:
> -        print('Applying patch #%d using %s' %
> -              (patch_id, repr(' '.join(apply_cmd))))
> +        print('Applying patch #%d using "%s"' %
> +              (patch_id, ' '.join(apply_cmd)))
>  
>      print('Description: %s' % patch['name'])
>      s = rpc.patch_get_mbox(patch_id)
> -- 
> 2.18.0
>
> _______________________________________________
> Patchwork mailing list
> Patchwork@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/patchwork

Patch

diff --git a/patchwork/bin/pwclient b/patchwork/bin/pwclient
index 79137b0..2020a8c 100755
--- a/patchwork/bin/pwclient
+++ b/patchwork/bin/pwclient
@@ -327,8 +327,8 @@  def action_apply(rpc, patch_id, apply_cmd=None):
         print('Applying patch #%d to current directory' % patch_id)
         apply_cmd = ['patch', '-p1']
     else:
-        print('Applying patch #%d using %s' %
-              (patch_id, repr(' '.join(apply_cmd))))
+        print('Applying patch #%d using "%s"' %
+              (patch_id, ' '.join(apply_cmd)))
 
     print('Description: %s' % patch['name'])
     s = rpc.patch_get_mbox(patch_id)