MAAS

Merge lp:~andreserl/maas/packaging_lp1381603 into lp:~maas-maintainers/maas/packaging

  1. packaging_lp1381603
  2. Merge into packaging
Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: 393
Merged at revision: 391
Proposed branch: lp:~andreserl/maas/packaging_lp1381603
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 13 lines (+0/-2)
1 file modified
debian/maas-common.postinst (+0/-2)
To merge this branch: bzr merge lp:~andreserl/maas/packaging_lp1381603
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Gavin Panella Pending
Review via email: mp+259104@code.launchpad.net

This proposal supersedes a proposal from 2015-05-14.

Commit message

Ensure that only maas.log gets permissions changed.

To post a comment you must log in.
Revision history for this message
Gavin Panella (allenap) wrote : Posted in a previous version of this proposal

Looks good, but with a few comments.

review: Approve
Revision history for this message
Andres Rodriguez (andreserl) : Posted in a previous version of this proposal
Revision history for this message
Andres Rodriguez (andreserl) : Posted in a previous version of this proposal
review: Approve
Revision history for this message
Adam Collard (adam-collard) : Posted in a previous version of this proposal
Revision history for this message
Andres Rodriguez (andreserl) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/maas-common.postinst'
2--- debian/maas-common.postinst 2015-05-14 11:02:42 +0000
3+++ debian/maas-common.postinst 2015-05-14 11:28:58 +0000
4@@ -10,9 +10,7 @@
5 fi
6
7 # Give appropriate permissions
8- chown -R maas:maas /var/lib/maas/
9 chown syslog:syslog /var/log/maas/maas.log
10-
11 }
12
13 add_user_group(){

Subscribers

People subscribed via source and target branches

to all changes: