Merge lp:~thisfred/ubuntuone-control-panel/lp-746489 into lp:ubuntuone-control-panel

Proposed by Eric Casteleijn on 2011-03-31
Status: Merged
Approved by: Eric Casteleijn on 2011-03-31
Approved revision: 122
Merged at revision: 120
Proposed branch: lp:~thisfred/ubuntuone-control-panel/lp-746489
Merge into: lp:ubuntuone-control-panel
Diff against target: 18 lines (+3/-3)
1 file modified
bin/ubuntuone-control-panel-gtk (+3/-3)
To merge this branch: bzr merge lp:~thisfred/ubuntuone-control-panel/lp-746489
Reviewer Review Type Date Requested Status
Natalia Bidart Approve on 2011-03-31
Manuel de la Peña (community) 2011-03-31 Approve on 2011-03-31
Review via email: mp+55769@code.launchpad.net

Commit message

Added proper defaults to the command line arguments.

Description of the change

Added proper defaults to the command line arguments.

To post a comment you must log in.
review: Approve
122. By Eric Casteleijn on 2011-03-31

consistent quoting

Natalia Bidart (nataliabidart) wrote :

Works ok!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/ubuntuone-control-panel-gtk'
2--- bin/ubuntuone-control-panel-gtk 2011-03-30 16:06:33 +0000
3+++ bin/ubuntuone-control-panel-gtk 2011-03-31 16:44:53 +0000
4@@ -38,13 +38,13 @@
5 usage = "Usage: %prog [option]"
6 result = OptionParser(usage=usage)
7 result.add_option("", "--switch-to", dest="switch_to", type="string",
8- metavar="PANEL_NAME",
9+ metavar="PANEL_NAME", default="",
10 help="Start the Ubuntu One Control Panel (GTK) in the "
11 "PANEL_NAME tab. Possible values are: "
12 "dashboard, volumes, devices, applications")
13 result.add_option("-a", "--alert", dest="alert", action="store_true",
14- help="Start the Ubuntu One Control Panel (GTK) alerting "
15- "the user to its presence.")
16+ default=False, help="Start the Ubuntu One Control Panel "
17+ "(GTK) alerting the user to its presence.")
18 return result
19
20

Subscribers

People subscribed via source and target branches