/srv/irclogs.ubuntu.com/2016/01/12/#launchpad-dev.txt

cjwatsonwgrant: Thanks00:51
* cjwatson requests a deployment00:53
=== blr_ is now known as blr
=== jamesh_ is now known as jamesh
=== danilos` is now known as danilos
cjwatsonwgrant: Thanks for the reviews so far.  I obviously can't actually land git-recipe-model yet, since db-git-recipes needs to go in first and isn't approved.12:35
cjwatsonSo I'll just get the rest of the stack proposed today.12:35
wgrantYep12:35
wgrantHow much is left?12:35
wgrantI was waylaid by a ridiculous ISP transparent proxy bug today, so didn't get through them all.12:36
cjwatsonI just proposed browser code to edit existing recipes.  After that I have ~1300 lines to sort out - I think one branch for a bit of BranchTarget/GitNamespace refinement, and one or two more for listing recipes and creating new ones.12:37
wgrantOh, not so bad.12:37
cjwatsonWould prefer not to land the last until the buildd stuff's in place, of course.12:38
wgrantIndeed.12:38
cjwatsonI could add an FF; not sure it's worth it in this case.12:39
cjwatsonwgrant: The recipe series is complete now.15:33
=== blr_ is now known as blr
wgrantcjwatson: Re. Build-Depends-Arch, I'm not sure that we shouldn't use user_defined_fields.22:25
wgrantI'm not unhappy either way, but the specific fields exist because JSON didn't.22:26
cjwatsonIf you think that it's reasonable to special-case certain elements of user_defined_fields, I don't object to reworking things.22:27
wgrantI have no problem with only displaying things that are interesting to display.22:28
wgrantThe only downside is the inconsistency.22:29
wgrantvs. the existing fields22:29
cjwatsonDisplay, and the archivepublisher ordering tweak, though the latter's fairly insignificant.22:30
wgrantOh they're not seriously sorted that way are they22:31
cjwatsonIt's manual22:31
cjwatsonWe could pick out B-[DC]-A from user_defined_fields22:31
wgrantI personally think it makes sense to do that.22:32
wgrantWe don't win much by having dedicated columns.22:32
wgrantUnless you consider increased database patch numbers to be a win.22:32
cjwatsonA little cumbersome in lp.archivepublisher.indices, but not horribly.22:32
cjwatsonNo, my recent MP history notwithstanding :-)22:32
wgrantHeh22:32
cjwatsonI will note that upgrade.py breaks when we hit 100.22:33
cjwatsonMight be worth a consolidation at some point ...22:33
cjwatson(Or fixing the glob)22:33
wgrantI think the last one was in like 2010.22:33
wgrantCould get stub to do a fresh dump once your latest patch is out.22:34

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!