[10:23] <wgrant> cjwatson: Thanks for the review. Didn't get to yours today, I'm afraid. Too much buildd and related amusement.
[10:23] <cjwatson> NP, I have plenty to do with splitting out the next in the series and working out why it makes one of the sharingservice tests fail.
[10:25] <wgrant> Heh
[10:26] <cjwatson> Which would be fine if we didn't also have simultaneous drain blockage and tumble-dryer failure because why does everything hate me.
[10:26] <cjwatson> So I think I will be working from the launderette this afternoon.
[10:29] <wgrant> You should move to Australia, where we have air conditioner failures but it's warm enough to avoid tumble dryers.
[10:30] <cjwatson> I think Kirsten would expire on the spot.
[10:31] <wgrant> Likely.
[10:31] <cjwatson> She's one of the few people I know who can be too hot in the middle of an English winter.
[10:31] <wgrant> ...
=== elmo_ is now known as elmo
[21:37] <cjwatson> wgrant: Just noticed a problem with the default-repository methods in git-basic-model.  My code requires that a target default must also be the default for the combination of the repository owner and its target, to avoid what I think would be the very confusing situation where lp:launchpad is owned by ~launchpad-pqm but lp:launchpad != lp:~launchpad-pqm/launchpad.  But the project owner is allowed to set the default repository for ...
[21:37] <cjwatson> ... their project even if they don't own that repository, to handle the bot-owned repository case; and in general setting an owner-target default requires launchpad.Edit on the owner, which I think is sensible in isolation.  So do I (a) give special dispensation to setTargetDefault to set the owner-target default (still provided that there isn't one already set) even if you aren't the owner/admin, (b) require you to set the ...
[21:37] <cjwatson> ... owner-target default first in this case (but that would be awkward in the bot case which is the whole point here) (c) something else?
[21:38] <cjwatson> Oh or (d) compute owner-target defaults dynamically, by looking first for an explicit owner-target default but falling back to a target default with the same owner
[23:00] <wgrant> cjwatson: A good question. I'll have to think about that one...