Phablet Tools

Merge lp:~robru/phablet-tools/stupid-printf into lp:phablet-tools

  1. stupid-printf
  2. Merge into trunk
Proposed by Robert Bruce Park on 2015-03-26
Status: Merged
Approved by: Robert Bruce Park on 2015-03-30
Approved revision: 336
Merged at revision: 336
Proposed branch: lp:~robru/phablet-tools/stupid-printf
Merge into: lp:phablet-tools
Diff against target: 21 lines (+2/-2)
1 file modified
citrain (+2/-2)
To merge this branch: bzr merge lp:~robru/phablet-tools/stupid-printf
Reviewer Review Type Date Requested Status
Tony Espy 2015-03-26 Approve on 2015-03-30
PS Jenkins bot continuous-integration Approve on 2015-03-26
Ubuntu Phablet Team 2015-03-26 Pending
Review via email: mp+254180@code.launchpad.net

Commit Message

Stop interpreting existing zero-pad as octal.

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Tony Espy (awe) wrote :

Looks good to me, and a quick sanity check using silo 013 works correctly now.

review: Approve
Robert Bruce Park (robru) wrote :

Thanks Tony!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'citrain'
2--- citrain 2014-10-10 10:14:05 +0000
3+++ citrain 2015-03-26 00:04:30 +0000
4@@ -27,7 +27,7 @@
5 device-purge Not implemented.
6
7 SILO-NUMBER:
8- 1..20
9+ 0..30
10
11 DISTRIBUTION: (optional)
12 ubuntu
13@@ -56,7 +56,7 @@
14 echo "$1" | egrep -q "^[0-9]{1,3}$" || usage
15
16 # Read the second positional argument.
17-SILO=landing-$(printf "%03d" "$1")
18+SILO=landing-$(echo "000$1" | rev | cut -c -3 | rev)
19 [ $# -gt 0 ] && shift || usage
20
21 case "$COMMAND" in

Subscribers

People subscribed via source and target branches