Merge lp:~patrick-hetu/golxc/fix-1238541 into lp:golxc

Proposed by Patrick Hetu on 2014-01-08
Status: Merged
Approved by: Nate Finch on 2014-01-16
Approved revision: 6
Merged at revision: 6
Proposed branch: lp:~patrick-hetu/golxc/fix-1238541
Merge into: lp:golxc
Diff against target: 12 lines (+1/-1)
1 file modified
golxc.go (+1/-1)
To merge this branch: bzr merge lp:~patrick-hetu/golxc/fix-1238541
Reviewer Review Type Date Requested Status
Nate Finch (community) 2014-01-08 Approve on 2014-01-16
Review via email: mp+200845@code.launchpad.net

Commit message

fix 1238541: lowercase when parsing lxc output

To post a comment you must log in.
Nate Finch (natefinch) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'golxc.go'
2--- golxc.go 2013-06-14 01:27:52 +0000
3+++ golxc.go 2014-01-08 14:48:08 +0000
4@@ -432,7 +432,7 @@
5 for _, line := range lines {
6 parts := strings.SplitN(line, sep, 2)
7 if len(parts) == 2 {
8- kv[parts[0]] = strings.TrimSpace(parts[1])
9+ kv[strings.ToLower(parts[0])] = strings.TrimSpace(parts[1])
10 }
11 }
12 return kv

Subscribers

People subscribed via source and target branches

to all changes: