Re: [Launchpad-reviewers] [Merge] lp:~mbp/launchpad/mbp-trivial into lp:launchpad

2010-08-10 Thread Martin Pool
wrong: apparently they don't have an errno... -- https://code.launchpad.net/~mbp/launchpad/mbp-trivial/+merge/32173 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~mbp/launchpad/mbp-trivial into lp:launchpad. ___

Re: [Launchpad-reviewers] [Merge] lp:~lifeless/launchpad/registry into lp:launchpad/devel

2010-08-10 Thread William Grant
I think all POSTs redirect before displaying the subsequent page, so that should be fine. However, I'm very suspicious of widely deploying @cachedproperty... I'd be much happier if it was populated only by the explicit prejoin, and operated uncached in the normal case. --

Re: [Launchpad-reviewers] [Merge] lp:~jelmer/meta-lp-deps/lpreview-body into lp:meta-lp-deps

2010-08-10 Thread Max Bowsher
Granted this is only a Recommends not a hard Depends, but bzr-lpreview-body is only published in the PPA for lucid. It ought to be available in all populated distroseries. Also, please mention bzr-lpreview-body on https://dev.launchpad.net/LaunchpadPpa. --

[Launchpad-reviewers] [Merge] lp:~jelmer/meta-lp-deps/lintian into lp:meta-lp-deps

2010-08-10 Thread Max Bowsher
The proposal to merge lp:~jelmer/meta-lp-deps/lintian into lp:meta-lp-deps has been updated. Status: Needs review = Work in progress -- https://code.launchpad.net/~jelmer/meta-lp-deps/lintian/+merge/31745 Your team Launchpad code reviewers is requested to review the proposed merge of

[Launchpad-reviewers] [Merge] lp:~maxb/meta-lp-deps/pg8.4 into lp:meta-lp-deps

2010-08-10 Thread Max Bowsher
Max Bowsher has proposed merging lp:~maxb/meta-lp-deps/pg8.4 into lp:meta-lp-deps. Requested reviews: Launchpad code reviewers (launchpad-reviewers) -- https://code.launchpad.net/~maxb/meta-lp-deps/pg8.4/+merge/32200 Your team Launchpad code reviewers is requested to review the proposed

Re: [Launchpad-reviewers] [Merge] lp:~maxb/meta-lp-deps/pg8.4 into lp:meta-lp-deps

2010-08-10 Thread Jelmer Vernooij
Review: Needs Fixing code Production still uses 8.3, so we can't drop 8.3 support just yet. The other changes are fine. -- https://code.launchpad.net/~maxb/meta-lp-deps/pg8.4/+merge/32200 Your team Launchpad code reviewers is subscribed to branch lp:meta-lp-deps.

Re: [Launchpad-reviewers] [Merge] lp:~jelmer/meta-lp-deps/lpreview-body into lp:meta-lp-deps

2010-08-10 Thread Jelmer Vernooij
On Tue, 2010-08-10 at 13:26 +, Max Bowsher wrote: Granted this is only a Recommends not a hard Depends, but bzr-lpreview-body is only published in the PPA for lucid. It ought to be available in all populated distroseries. Also, please mention bzr-lpreview-body on

[Launchpad-reviewers] [Merge] lp:~jelmer/meta-lp-deps/lintian into lp:meta-lp-deps

2010-08-10 Thread noreply
The proposal to merge lp:~jelmer/meta-lp-deps/lintian into lp:meta-lp-deps has been updated. Status: Approved = Merged -- https://code.launchpad.net/~jelmer/meta-lp-deps/lintian/+merge/31745 Your team Launchpad code reviewers is requested to review the proposed merge of

[Launchpad-reviewers] [Merge] lp:~jml/launchpad/readme into lp:launchpad/devel

2010-08-10 Thread Jonathan Lange
Jonathan Lange has proposed merging lp:~jml/launchpad/readme into lp:launchpad/devel. Requested reviews: Launchpad code reviewers (launchpad-reviewers) I thought, you know what would be a good idea? A README file! It should explain what Launchpad is, where one could go to find more

[Launchpad-reviewers] [Merge] lp:~jelmer/launchpad/publisher-use-debian-1 into lp:launchpad/devel

2010-08-10 Thread Jelmer Vernooij
Jelmer Vernooij has proposed merging lp:~jelmer/launchpad/publisher-use-debian-1 into lp:launchpad/devel. Requested reviews: Launchpad code reviewers (launchpad-reviewers): code Refactor lp.archivepublisher.publishing to use the functionality in the new debian Python module to generate

[Launchpad-reviewers] [Merge] lp:~bac/launchpad/bug-590813 into lp:launchpad/devel

2010-08-10 Thread Brad Crittenden
Brad Crittenden has proposed merging lp:~bac/launchpad/bug-590813 into lp:launchpad/devel. Requested reviews: Launchpad code reviewers (launchpad-reviewers) Related bugs: #590813 ShortListTooBigError configuring project https://bugs.launchpad.net/bugs/590813 = Summary = Some products

Re: [Launchpad-reviewers] [Merge] lp:~sinzui/launchpad/deactivate-deactivated-account-0 into lp:launchpad/devel

2010-08-10 Thread Robert Collins
Hi, this looks ok to me. Nothing to do with your patch per se - I have a question about the test technology being used: create_initialised_view - I presume that that lets you get in under the publishing machinery? That means that things setup by publications - like oops handling etc - won't be

[Launchpad-reviewers] [Merge] lp:~jcsackett/launchpad/plus-participation-timeouts into lp:launchpad/devel

2010-08-10 Thread j.c.sackett
j.c.sackett has proposed merging lp:~jcsackett/launchpad/plus-participation-timeouts into lp:launchpad/devel. Requested reviews: Launchpad code reviewers (launchpad-reviewers) Related bugs: #607879 ~person/+participation timeouts https://bugs.launchpad.net/bugs/607879 Summary Partial

[Launchpad-reviewers] [Merge] lp:~jcsackett/launchpad/member-icons-title into lp:launchpad/devel

2010-08-10 Thread j.c.sackett
j.c.sackett has proposed merging lp:~jcsackett/launchpad/member-icons-title into lp:launchpad/devel. Requested reviews: Launchpad code reviewers (launchpad-reviewers) Related bugs: #614649 Title for member of icons unhelpful https://bugs.launchpad.net/bugs/614649 Summary Fixes a trivial

Re: [Launchpad-reviewers] [Merge] lp:~sinzui/launchpad/deactivate-deactivated-account-0 into lp:launchpad/devel

2010-08-10 Thread Curtis Hovey
This catches exceptions like other tests. The issue of recording an oops (which is an act of adaption) is a separate concern. I do not think TestBrowser records oopes either because it uses a similar mechanism to instantiate the view. --

Re: [Launchpad-reviewers] [Merge] lp:~sinzui/launchpad/deactivate-deactivated-account-0 into lp:launchpad/devel

2010-08-10 Thread Robert Collins
Thanks! -- https://code.launchpad.net/~sinzui/launchpad/deactivate-deactivated-account-0/+merge/32264 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~sinzui/launchpad/deactivate-deactivated-account-0 into lp:launchpad/devel.

[Launchpad-reviewers] [Merge] lp:~wgrant/launchpad/bug-616154-guessPackageNames-order-fix into lp:launchpad

2010-08-10 Thread William Grant
William Grant has proposed merging lp:~wgrant/launchpad/bug-616154-guessPackageNames-order-fix into lp:launchpad. Requested reviews: Launchpad code reviewers (launchpad-reviewers) Related bugs: #616154 Distribution.guessPackageNames needs to return the source for the latest published binary