| === doko_ is now known as doko | ||
| * Laney is testing importing the sync blacklist into dogfood | 11:30 | |
| Laney | http://people.ubuntu.com/~laney/sync-blacklist.new.txt | 12:13 |
|---|---|---|
| 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:15 |
| tumbleweed | probably better than my haskell / mono :P | 12:16 |
| tumbleweed | Laney: launchpadlib knows how to deal with _link | 12:17 |
| 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:18 |
| Laney | silly microwave | 12:19 |
| nigelb | IRC-ing from a microwave? | 12:20 |
| Laney | it's warm in here | 12:20 |
| 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:22 |
| 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:23 |
| 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:24 |
| Laney | thank fuck i havent had to do any of that | 12:25 |
| Laney | erm, excuse me | 12:25 |
| 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:26 |
| 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:27 |
| 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:28 |
| Laney | your stuff looks applicable. fancy integrating it? | 12:30 |
| tumbleweed | oh, all right :P | 12:31 |
| Laney | :-) | 12:31 |
| * Laney has to do Real Work⢠for a bit | 12:31 | |
| tumbleweed | Laney: http://people.ubuntu.com/~stefanor/tmp/ | 12:48 |
| * Laney didn't know about current_series | 12:49 | |
| Laney | good work | 12:50 |
| Laney | you could make that change in sync-blacklist too | 12:50 |
| Laney | tumbleweed: do you think we should start using this now? | 13:08 |
| Laney | or at least import the data into production | 13:08 |
| 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:10 |
| 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:11 |
| 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:12 |
| cjwatson | probably won't be necessary; I don't expect to use sync-source -a again | 13:13 |
| 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:15 |
| cjwatson | tumbleweed: not yet, expected later this week though | 13:16 |
| jdstrand | ScottK: (bug #842667) | 13:19 |
| Laney | k | 13:24 |
| ScottK | jdstrand: Go for it. | 13:38 |
| jdstrand | ScottK: thanks! | 13:49 |
| Daviey | Hmm, Server ISO hasn't been built since beta.. Can someone flip the switch please? | 15:56 |
| cjwatson | wonder why it was commented out | 15:58 |
| cjwatson | uncommented | 15:58 |
| cjwatson | and running a build for you now | 15:58 |
| === warp11 is now known as warp10 | ||
| === Ursinha is now known as Ursinha-nom | ||
| === Ursinha-nom is now known as Ursinha | ||
| Daviey | cjwatson: Ah, thanks! Just noticed. | 20:17 |
| === Ursinha is now known as Ursinha-afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!