launchpad-reviewers team mailing list archive

[LaMont Jones <lamont.jones@xxxxxxxxxxxxx>]

LaMont Jones has proposed merging lp:~lamont/launchpad/lp-buildd-79 into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~lamont/launchpad/lp-buildd-79/+merge/72182

correctly delivers sudoers.d/buildd.

as released revision 79 of launchpad-buildd
-- 
https://code.launchpad.net/~lamont/launchpad/lp-buildd-79/+merge/72182
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~lamont/launchpad/lp-buildd-79 into lp:launchpad.

=== modified file 'lib/canonical/buildd/debian/changelog'
--- lib/canonical/buildd/debian/changelog	2011-04-07 05:57:59 +0000
+++ lib/canonical/buildd/debian/changelog	2011-08-19 13:35:30 +0000
@@ -1,3 +1,9 @@
+launchpad-buildd (79) hardy-cat; urgency=low
+
+  * Fix sudoers.d/buildd permissions
+
+ -- LaMont Jones <lamont@xxxxxxxxxxxxx>  Fri, 19 Aug 2011 07:31:54 -0600
+
 launchpad-buildd (78) hardy-cat; urgency=low
 
   * Correctly update sudoers files when needed.  LP: #742881

=== modified file 'lib/canonical/buildd/debian/postinst'
--- lib/canonical/buildd/debian/postinst	2011-04-07 04:45:45 +0000
+++ lib/canonical/buildd/debian/postinst	2011-08-19 13:35:30 +0000
@@ -45,6 +45,8 @@
 %lpbuildd ALL=(ALL) NOPASSWD: ALL
 EOF
 		fi
+		chmod 440 /etc/sudoers.d/buildd
+		chown root:root /etc/sudoers.d/buildd
 	fi
 
         install -d -o${USER} -m0755 /home/${USER}