Merge lp:~joetalbott/charms/trusty/snappy-proposed-image-builder/fix_user_info into lp:~canonical-ci-engineering/charms/trusty/snappy-proposed-image-builder/trunk

Proposed by Joe Talbott on 2015-06-05
Status: Merged
Approved by: Joe Talbott on 2015-06-05
Approved revision: 11
Merged at revision: 11
Proposed branch: lp:~joetalbott/charms/trusty/snappy-proposed-image-builder/fix_user_info
Merge into: lp:~canonical-ci-engineering/charms/trusty/snappy-proposed-image-builder/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
hooks/actions.py (+1/-1)
To merge this branch: bzr merge lp:~joetalbott/charms/trusty/snappy-proposed-image-builder/fix_user_info
Reviewer Review Type Date Requested Status
Celso Providelo (community) 2015-06-05 Approve on 2015-06-05
Review via email: mp+261259@code.launchpad.net

Commit message

Fix user and group info accesses.

Description of the change

Fix user and group info accesses.

To post a comment you must log in.
Celso Providelo (cprov) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/actions.py'
2--- hooks/actions.py 2015-06-04 21:59:21 +0000
3+++ hooks/actions.py 2015-06-05 18:01:14 +0000
4@@ -81,7 +81,7 @@
5 # make sure the user owns their home.
6 group_info = grp.getgrnam(username)
7 user_info = pwd.getpwnam(username)
8- os.chown(home, user_info['pw_uid'], group_info['gr_gid'])
9+ os.chown(home, user_info.pw_uid, group_info.gr_gid)
10
11
12 def install_python_packages(service_name):

Subscribers

People subscribed via source and target branches