[01:34] <cjwatson> Huh, sqlobject doesn't have GROUP BY?
[01:35] <wgrant> cjwatson: SQLObject has just about nothing.
[01:35] <wgrant> cjwatson: Stormify that stuff.
[01:35] <wgrant> But you probably want DISTINCT distroseries, pocket anyway.
[01:35] <cjwatson> Stormifying xPPH seems like it might take a while
[01:36] <wgrant> Hm, what are you doing?
[01:37] <cjwatson> Looking at the next stage of optimising publisher phase A, so changing getPending*Publications to return everything; seemed like it'd be easiest to group it by series to get roughly the same ordering (hence logging) as before
[01:38] <wgrant> That's ORDER BY, not GROUP BY.
[01:38] <cjwatson> Oh, er, cough.  I speak SQL good, I learn him from a book.
[01:38] <wgrant> Heh
[01:39] <cjwatson> And SQLObject has ORDER BY, good.
[01:39] <wgrant> Also, getPendingPublications doesn't use any of the other xPPH methods; you can Stormify it alone without a problem.
[01:40] <cjwatson> Oh, I guess that's true, yes
[01:41] <wgrant> Hm
[01:42] <wgrant> Tempting to fix all the suite references while I'm touching almost the same code for archive references.
[01:50] <wgrant> Also not sure whether to leave the reference for primary archives as 'ubuntu', or to make it 'ubuntu/primary' like the rest.
[01:51] <cjwatson> I always found ubuntu/primary weird.  I think just ubuntu is an improvement.
[01:51] <cjwatson> Marginally.
[06:59]  * wgrant tries to track down all of the ancestry implementations.
[07:00] <wgrant> There's NascentUpload.get(Source|Binary)Ancestry and lp.soyuz.adapters.overrides, which I think are the two that matter, but then I'm pretty sure there's at least another two elsewhere that are used for varying purposes.
[07:01] <wgrant> PackageUploadSource.getSourceAncestryForDiffs
[07:01] <wgrant> And check_copy_permissions
[07:04] <wgrant> I think all except getSourceAncestryForDiffs are relevant for ubuntu-rtm, as check_copy_permissions assumes a package will require manual approval if no overrides are found.
[08:35] <wgrant> cjwatson: Oh, https://code.launchpad.net/~wgrant/launchpad/archive-references-everywhere/+merge/226061 basically includes your pcj-repr changes.
[08:35] <wgrant> I can back those bits out, I guess.
[08:35] <cjwatson> wgrant: Oh, I can just withdraw my branch
[08:36] <wgrant> That's probably easier. I had tests running overnight after I basically grepped through and replaced almost every reference to archive.owner.name and archive.name.
[08:36] <wgrant> So it wasn't pushed yet, sorry.
[08:37] <cjwatson> Not a problem.
[08:40] <wgrant> cjwatson: For the ubuntu-rtm needing to inherit ubuntu's overrides problem, I'm thinking we might just want a flag on DistroSeriesParent which lets overrides fall back to a parent series' primary archive.
[08:40] <wgrant> Also, did we decide that they wanted to skip NEW in the fallback case?
[08:46] <cjwatson> A flag would seem sensible.  I thought we wanted to just do NEW as normal in the fallback case though.
[08:46] <cjwatson> I believe we agreed it was OK for #ubuntu-release to manage that.
[08:46] <wgrant> I couldn't remember, either works.
[08:46] <wgrant> getBinaryAncestry and getSourceAncestry die tonight.
[08:48] <wgrant> http://paste.ubuntu.com/7769369/ is my notes for the various cases atm.
[08:49] <cjwatson> We only want to force component to universe/multiverse in some limited cases.
[09:10] <wgrant> cjwatson: Don't we always want to do that if there's no ancestry in either archive?
[09:15] <cjwatson> New binaries uploaded to Ubuntu with source in main default to main.
[09:15] <cjwatson> For source, yes.
[09:15] <wgrant> Er yeah, forgot that binary override complexity, good point.
[09:27] <Eisbrecher_xnox> is it still the case that backports can't build-depend on other backports? or has that been fixed?
[09:28] <cjwatson> That's been fixed.
[09:28] <Eisbrecher_xnox> cool.
[09:29] <cjwatson> https://bugs.launchpad.net/launchpad/+bug/888665
[09:29] <_mup_> Bug #888665: Backports can't build-depend on other backports <soyuz-build> <Launchpad itself:Fix Released by adconrad> <https://launchpad.net/bugs/888665>
[12:15] <wgrant> cprov: Thanks.
[12:16] <cprov> wgrant: no problem, I am still owning you few reviews.