powerd

Merge lp:~sforshee/powerd/cli-fix-display-args into lp:powerd

  1. cli-fix-display-args
  2. Merge into trunk
Proposed by Seth Forshee
Status: Merged
Approved by: Matt Fischer
Approved revision: 79
Merged at revision: 81
Proposed branch: lp:~sforshee/powerd/cli-fix-display-args
Merge into: lp:powerd
Diff against target: 12 lines (+1/-1)
1 file modified
cli/powerd-cli.c (+1/-1)
To merge this branch: bzr merge lp:~sforshee/powerd/cli-fix-display-args
Reviewer Review Type Date Requested Status
Matt Fischer (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+178289@code.launchpad.net

Commit message

Fix bug in argument processing for 'powerd-cli display'

Description of the change

Fix bug in argument processing for 'powerd-cli display'

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:79
http://jenkins.qa.ubuntu.com/job/powerd-ci/101/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-ci/55
        deb: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-ci/55/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins:8080/job/powerd-ci/101/rebuild

review: Approve (continuous-integration)
Revision history for this message
Matt Fischer (mfisch) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cli/powerd-cli.c'
2--- cli/powerd-cli.c 2013-07-29 16:05:34 +0000
3+++ cli/powerd-cli.c 2013-08-02 13:14:45 +0000
4@@ -821,7 +821,7 @@
5 return 0;
6 }
7 else if (!strcmp(argv[1],"display")) {
8- if (argc < 4) {
9+ if (argc < 3) {
10 fprintf(stderr,"display requires other arguments\n");
11 usage(argv[0]);
12 return -1;

Subscribers

People subscribed via source and target branches