launchpad-reviewers team mailing list archive

[William Grant <me@xxxxxxxxxxxxxxxxxx>]

William Grant has proposed merging lp:~wgrant/launchpad/remove-archivepublisher-config into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #739992 in Launchpad itself: "Remove obsolete archivepublisher config keys"
  https://bugs.launchpad.net/launchpad/+bug/739992

For more details, see:
https://code.launchpad.net/~wgrant/launchpad/remove-archivepublisher-config/+merge/56873

archivepublisher.{root,base_url,copy_base_url} have been moved into the DB, and were removed from the production configs yesterday. This branch removes them from the config schema.
-- 
https://code.launchpad.net/~wgrant/launchpad/remove-archivepublisher-config/+merge/56873
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~wgrant/launchpad/remove-archivepublisher-config into lp:launchpad.

=== modified file 'lib/canonical/config/schema-lazr.conf'
--- lib/canonical/config/schema-lazr.conf	2011-03-23 18:29:09 +0000
+++ lib/canonical/config/schema-lazr.conf	2011-04-08 04:30:55 +0000
@@ -24,12 +24,6 @@
 # datatype: string
 dbuser: archivepublisher
 
-# XXX: wgrant 2011-03-22 bug=739992: These three keys are obsolete and
-# should be removed after 11.04 is released.
-root: none
-base_url: none 
-copy_base_url: none
-
 
 [binaryfile_expire]
 dbuser: binaryfile-expire