[Piuparts-commits] [SCM] piuparts git repository branch, piatti, updated. 0.44-827-g219614b

[Andreas Beckmann]

The following commit has been merged in the piatti branch:
commit dc034a97ee651d81ab0b04fb850081f35fde80ea
Author: Andreas Beckmann <debian at abeckmann.de>
Date:   Wed Jun 20 09:54:28 2012 +0200

    p-s.deb: comment the shipped sudoers entries
    
    require explicit editing to activate
    
    Signed-off-by: Andreas Beckmann <debian at abeckmann.de>

diff --git a/Makefile b/Makefile
index 8a5006a..3231651 100644
--- a/Makefile
+++ b/Makefile
@@ -52,6 +52,7 @@ install-conf:
 
 	install -d $(etcdir)/sudoers.d
 	install -m 440 conf/piuparts.sudoers $(etcdir)/sudoers.d/piuparts
+	sed -i -r '/^[^#]+/s/^/#/' $(etcdir)/sudoers.d/piuparts
 
 	install -d $(etcdir)/apache2/conf.d
 	install -m 0644 conf/piuparts.apache $(etcdir)/apache2/conf.d
diff --git a/TODO b/TODO
index c044aed..3d08720 100644
--- a/TODO
+++ b/TODO
@@ -18,6 +18,9 @@ for 0.46:
 
 - make piuparts-master + piuparts-slave packages work out-of-the-box
 
+- ship sane minimal sudoers in p-s.deb than can be enabled by default
+  do we really need anything else than running piuparts as root?
+
 - verify that all scripts in $user/bin/ also work from master-slave packages
   eventually enable some conjobs in the packages
 
diff --git a/debian/changelog b/debian/changelog
index 3ab6ef6..bb64baa 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -168,6 +168,8 @@ piuparts (0.45) UNRELEASED; urgency=low
     --warn-on-leftovers-after-purge is not used.
     (Closes: #668752)
   * Ship custom scripts in the piuparts package.
+  * Comment sudoers and crontabs shipped in the master/slave packages,
+    requiring explicit editing after installation to activate them.
 
   [ Leo Iannacone ]
   * piuparts.py: add --existing-chroot option, specify a directory as source

-- 
piuparts git repository