launchpad-reviewers team mailing list archive

[Julian Edwards <julian.edwards@xxxxxxxxxxxxx>]

Julian Edwards has proposed merging lp:~julian-edwards/maas/packaging into lp:~maas-maintainers/maas/packaging.

Commit message:
  [ Julian Edwards ]
  * debian/maas-region-controller.install: Remove installation of maas-gc; it
    is no longer required as upstream no longer stores files in the filesystem.
    (LP: #1069734)


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1052805 in MAAS: "Packaging build broken: no file to patch"
  https://bugs.launchpad.net/maas/+bug/1052805

For more details, see:
https://code.launchpad.net/~julian-edwards/maas/packaging/+merge/131584
-- 
https://code.launchpad.net/~julian-edwards/maas/packaging/+merge/131584
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~julian-edwards/maas/packaging into lp:~maas-maintainers/maas/packaging.

=== modified file 'debian/changelog'
--- debian/changelog	2012-10-19 15:42:58 +0000
+++ debian/changelog	2012-10-26 10:52:46 +0000
@@ -4,7 +4,12 @@
   * debian/maas-dns.postinst: fix permissions and group ownership of
     file /etc/bind/maas/named.conf.rndc.maas. (LP: #1066935)
 
- -- Andres Rodriguez <andreserl@xxxxxxxxxx>  Wed, 17 Oct 2012 20:12:59 -0400
+  [ Julian Edwards ]
+  * debian/maas-region-controller.install: Remove installation of maas-gc; it
+    is no longer required as upstream no longer stores files in the filesystem.
+    (LP: #1069734)
+
+ -- Julian Edwards <julian.edwards@xxxxxxxxxxxxx>  Fri, 26 Oct 2012 12:50:20 +0200
 
 maas (0.1+bzr1269+dfsg-0ubuntu1) quantal-proposed; urgency=low
 

=== modified file 'debian/maas-region-controller.install'
--- debian/maas-region-controller.install	2012-10-16 14:30:48 +0000
+++ debian/maas-region-controller.install	2012-10-26 10:52:46 +0000
@@ -40,9 +40,6 @@
 # Install new preseed files
 debian/tmp/usr/share/maas/preseeds
 
-# Install cronjobs
-debian/tmp/etc/cron.d/maas-gc
-
 # Install all other stuff
 debian/extras/maas usr/sbin
 debian/extras/maas-region-celeryd usr/sbin