[00:38] wgrant: Did you spy bug 1186050? [00:38] <_mup_> Bug #1186050: Remove Packages-arch-specific use from Soyuz

[00:52] Yeah [00:53] I think I have a branch that destroys it [00:55] Just deciding if I want to completly destroy lp.soyuz.pas, and if so where determineArchitecturesToBuild will live. [01:23] StevenK: It even fixes a critical [01:23] Ah, no [01:23] We still use dpkg-architecture for parsing the hint list, of course [01:24] Yeah [01:26] wgrant: So I want to destroy lp.soyuz.pas (and I think I do, since the name gives me nightmares), I'm still unsure where to stuff it [01:31] StevenK: The new code is only 80 lines. I'd do one of two things: a) move it to model.publishing, or b) switch the pubrec arg to an archive and an SPR, and move the whole thing to model.bpb or similar. [01:32] Or c) switch the pubrec argh and move it to a new small module in adapters [01:32] Probably c. [01:32] Yeah, adapters.arch or dpkgarch was what I thinking [01:33] adapters.buildarch, perhaps [01:33] Right [01:33] wgrant: 'Switch the pubrec argument' ? [01:33] To spph? [01:33] StevenK: Replace it with an archive and an spr [01:33] There's no point having it depend on a publication [01:33] When all it needs is an archive and an SPR [01:34] Right, I see that. [01:34] You could even give it a hintlist rather than an SPR [01:34] To simplify tests further [01:34] Now that we no longer need to do source/binary name matching [01:42] steven@undermined:~/launchpad/lp-branches/destroy-pas-yay% bzr di | diffstat -s [01:42] 6 files changed, 41 insertions(+), 371 deletions(-) === micahg_ is now known as micahg [04:07] wgrant: https://code.launchpad.net/~stevenk/launchpad/destroy-pas-yay/+merge/166639 [04:10] StevenK: Could I convince you to name determineArchitecturesToBuild legally? [04:11] Also, order should be archive, distroseries, legal_archseries, I think [04:13] wgrant: Sure, rename to what, since there's like two callsites [04:13] StevenK: determine_architectures_to_build [04:13] It's currently disguised as a method [04:13] Also, I thought there was only one non-test callsite [04:14] Right, the second callsite is the test [04:16] wgrant: http://pastebin.ubuntu.com/5718806/ [04:16] This will mean that things that go through unapproved or new might actually get the same sort of builds as autoapproved ones :) [04:16] And we can remove the stupid builddmaster dirs from haetae and pepo [04:16] Yeah [04:17] :) [04:18] wgrant: Any other issues? [04:19] I think that's about it [04:19] It was just a matter of deleting code until I convinced you to fix issues that had been bothering me for years :) [04:19] Well. [04:20] I made the mistake of telling infinity I was looking for stuff to do, and he suggested that since I like deleting stuff ... [04:21] wgrant: Just making sure test_buildarch and test_publishing are happy before I push. [04:40] wgrant: Can haz +1? [04:58] StevenK: Sec, cats needed attention [04:58] wgrant: Duh, that's their default state. [04:59] StevenK: r=me === tasdomas_afk is now known as tasdomas [05:41] wgrant: Is that your QA I see? [05:42] Since Thunderbird is convinced it must keep reading the staging mailbox [05:45] StevenK: I just rejected lots of stuff [05:45] To clear out raring's queue === wedgwood_away is now known as wedgwood === alexlist` is now known as alexlist === wedgwood is now known as wedgwood_away === matsubara is now known as matsubara-lunch === wedgwood_away is now known as wedgwood === matsubara-lunch is now known as matsubara === wedgwood is now known as wedgwood_away === wedgwood_away is now known as wedgwood === tasdomas is now known as tasdomas_afk === wedgwood is now known as wedgwood_away === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha === cjwatson_ is now known as cjwatson === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha