[11:30]  * Laney is testing importing the sync blacklist into dogfood
[12:13] <Laney> http://people.ubuntu.com/~laney/sync-blacklist.new.txt
[12:15] <Laney> http://people.ubuntu.com/~laney/import-blacklist.py http://people.ubuntu.com/~laney/sync-blacklist.py
[12:15] <Laney> please excuse my python
[12:16] <tumbleweed> probably better than my haskell / mono :P
[12:17] <tumbleweed> Laney: launchpadlib knows how to deal with _link
[12:18] <tumbleweed> comment_author = comment.comment_author.name
[12:18] <Laney> oh
[12:18] <Laney> i did it like that because when i tried ot call.name on it to print it out I just go tthe whol elinkdd
[12:18] <Laney> lag
[12:19] <Laney> silly microwave
[12:20] <nigelb> IRC-ing from a microwave?
[12:20] <Laney> it's warm in here
[12:22] <tumbleweed> Laney: python also has builtin for filtering: comment_text = [c for c in comment.body_text.splitlines() if c and not c.startswith("Ignored")]
[12:22] <Laney> is that better than using filter?
[12:23] <tumbleweed> I think it's faster in some cases, but also more readable (filter isn't used much in modern python)
[12:23] <tumbleweed> obviously nothing wrong with what you had :)
[12:24] <Laney> in haskell that's [ c | c <- comment.body_text.splitlines(), c, not c.startswith("Ignored") ]
[12:24] <Laney> but filter is also very common
[12:24] <Laney> that's why I was more than happy to use it
[12:25] <Laney> thank fuck i havent had to do any of that
[12:25] <Laney> erm, excuse me
[12:26] <Laney> . o O ( seriously, /one day/ I will figure out how to disable the x clipboard )
[12:26] <tumbleweed> Laney: the current blacklist has comments at the top of blocks of packages, it doesn't look like you import those
[12:26] <Laney> correct
[12:27] <Laney> I didn't fancy complicating it much more, thought it would be easier to just copy them down
[12:27] <Laney> patches welcome etc
[12:28] <tumbleweed> my one in is_blacklisted does, if you want an example
[12:28] <Laney> there were also missing DSDs on dogfood, but I didn't see the same on production
[12:28] <Laney> so some stuff couldn't be blacklisted at all
[12:28] <tumbleweed> yeah, the copy of lp data is ancient
[12:30] <Laney> your stuff looks applicable. fancy integrating it?
[12:31] <tumbleweed> oh, all right :P
[12:31] <Laney> :-)
[12:31]  * Laney has to do Real Work™ for a bit
[12:48] <tumbleweed> Laney: http://people.ubuntu.com/~stefanor/tmp/
[12:49]  * Laney didn't know about current_series
[12:50] <Laney> good work
[12:50] <Laney> you could make that change in sync-blacklist too
[13:08] <Laney> tumbleweed: do you think we should start using this now?
[13:08] <Laney> or at least import the data into production
[13:10] <tumbleweed> Laney: remaining issue I see is determining which comment is the blacklist reason, and I don't think there is any way to do that (which is why we are using all the comments, everywhere)
[13:10] <Laney> right, that's a launchpad presentation thing
[13:11] <tumbleweed> cjwatson said he wanted to manually check the list when importing, to remove the mismatched-tarballs, as they don't need to be imported if the sync tool can detect them
[13:11] <Laney> yep
[13:11] <Laney> whoever runs the script can sanity check the input
[13:11] <tumbleweed> I don't know what else is impacted by the blacklist. mom?
[13:11] <Laney> if we keep on generating it for now from launchpad then old stuff will keep working
[13:12] <tumbleweed> yup
[13:12] <Laney> apart from things which are removed for the import
[13:12] <tumbleweed> they can be statically added to the export, if necessary
[13:13] <cjwatson> probably won't be necessary; I don't expect to use sync-source -a again
[13:15] <tumbleweed> any progress on the new mass sync tool?
[13:15] <Laney> cjwatson: want to take a look at running http://people.ubuntu.com/~stefanor/tmp/import-blacklist.py ?
[13:15] <jdstrand> ScottK: hi! I'd like to sync rails from Debian for a security update. Daviey tells me this is desirable from the server team's perspective as well. is this ok?
[13:15] <cjwatson> Laney: not today, but could you mail that to me and I will?
[13:16] <cjwatson> tumbleweed: not yet, expected later this week though
[13:19] <jdstrand> ScottK: (bug #842667)
[13:24] <Laney> k
[13:38] <ScottK> jdstrand: Go for it.
[13:49] <jdstrand> ScottK: thanks!
[15:56] <Daviey> Hmm, Server ISO hasn't been built since beta.. Can someone flip the switch please?
[15:58] <cjwatson> wonder why it was commented out
[15:58] <cjwatson> uncommented
[15:58] <cjwatson> and running a build for you now
[20:17] <Daviey> cjwatson: Ah, thanks!  Just noticed.