Merge lp:~brendan-donegan/checkbox/bug926104_cli_password_prompt into lp:checkbox

[Brendan Donegan]

Remove dots from CLI to avoid them getting in the way of the password prompt

[Zygmunt Krynicki (zyga)]

Heh, simple and easy!

Looks good to me

review: Approve

[Brendan Donegan (brendan-donegan)]

Thanks for the ack, but since the branch isn't actually *fixing* the problem so much as avoiding it, i'll put it as work in progress until JeffL can review it

[Marc Tardif (cr3)]

Would you mind if I have a look at the problem too?

[Brendan Donegan (brendan-donegan)]

Of course not

[Brendan Donegan (brendan-donegan)]

Already got a fix for this

review: Disapprove

Unmerged revisions

1694. By Brendan Donegan on 2012-09-25

Merged from trunk.

1693. By Brendan Donegan on 2012-09-25

checkbox_cli/cli_interface.py - remove dots from CLI to avoid them getting
in the way when being prompted for the sudo password (LP: #926104)

=== modified file 'checkbox_cli/cli_interface.py'
--- checkbox_cli/cli_interface.py	2012-06-12 15:03:42 +0000
+++ checkbox_cli/cli_interface.py	2012-09-25 08:28:23 +0000
@@ -286,8 +286,7 @@
 
         if progress != None:
             self.put("\r%u%%" % (progress * 100))
-        else:
-            self.put(".")
+        
         sys.stdout.flush()
 
 

=== modified file 'debian/changelog'
--- debian/changelog	2012-09-24 15:05:45 +0000
+++ debian/changelog	2012-09-25 08:28:23 +0000
@@ -127,6 +127,8 @@
   * [FEATURE] jobs/power-management.txt.in, jobs/touchpad.txt.in,
     jobs/sniff.txt.in - Fix incorrect formatting of job descriptions to
     allow steps to be displayed by the UI (LP: #1054208)
+  * checkbox_cli/cli_interface.py - remove dots from CLI to avoid them getting
+    in the way when being prompted for the sudo password (LP: #926104)
 
   [Sylvain Pineau]
   * jobs/suspend.txt.in: Fixed suspend/suspend_advanced dependencies to avoid