[12:19] <ddaa> BjornT: pong
[12:26] <BjornT> ddaa: i'm reviewing svn-peg-revision now. could you explained briefly what peg_revision does?
[12:27] <ddaa> I'm writing test cases for this just now.
[12:27] <ddaa> briefly, it works around brain-damaged defaults in pysvn
[12:27] <ddaa> so the code actually does what it was intended to do all along
[12:28] <ddaa> the test cases are quite massively larger that the code, though
[12:29] <ddaa> IOW, please don't review this right now :)
[12:30] <BjornT> ddaa: ok, cool, i'll wait for the tests then :)
[12:41] <ddaa> It so happens that I need test cases anyway to make sure that the pyrex work will no regress this. This bug has a pretty nasty silent failure mode :(
[12:50] <lifeless> ddaa: can you shepard https://launchpad.net/products/libsyncml/trunk ?
[12:51] <ddaa> lifeless: I'm not going to have the time tonight
[12:51] <ddaa> oh well, nevermind
[12:53] <ddaa> lifeless: I'll finish it tomorrow
[12:53] <lifeless> ddaa: thanks
[12:53] <ddaa> spiv: oh, btw, you've got twisted
[12:53] <lifeless> test failed ?
[12:54] <ddaa> mh
[12:54] <ddaa> weird one
[12:54] <ddaa> https://macquarie.warthogs.hbd.com/roomba/status/libsyncml-trunk/events/6/log
[12:55] <ddaa> lifeless: looks like you've got the url a bit wrong
[12:55] <ddaa> you're sure it's not http://svn.opensync.org/trunk/libsyncml instead?
[12:55] <ddaa> also, it smells of svn:externals all over the place
[12:56] <lifeless> ddaa: http://libsyncml.opensync.org/wiki/download
[12:56] <lifeless> svn co http://svn.opensync.org/libsyncml/trunk libsyncml
[12:57] <ddaa> if it works, then I bet it's full of svn:externals
[12:57] <lifeless> it checks out ok
[12:57] <ddaa> in this case, then you're toast, at least in the short term
[12:57] <lifeless> no
[12:58] <lifeless> svn.opensync.org/libsyncml is the root of a repository
[12:59] <ddaa> the import spews lots of warnings like
[12:59] <ddaa> WARNING:root:ignored change outside branch: action=M path=u'/trunk/configure.ac' copyfrom_revision=None copyfrom_path=None
[12:59] <ddaa> WARNING:root:ignored change outside branch: action=M path=u'/trunk/libsyncml/parser/sml_xml_parse.c' copyfrom_revision=None copyfrom_path=None
[01:00] <lifeless> yes, I see
[01:00] <ddaa> mh
[01:01] <ddaa> I do not have to diagnose this tonight anyway.
[01:01] <ddaa> Might be a simple fix.
[01:01] <ddaa> Can this wait until Monday?
[01:03] <lifeless> ok, no problem.
[01:03] <lifeless> I think I know
[01:03] <lifeless> the root detection logic is probably finding http://svn.opensync.org/ as the root of the repository, which is wrong :(
[01:03] <ddaa> might be
[01:03] <lifeless> svn seems able to tell us.
[01:03] <lifeless> so we should start doing that.
[01:03] <ddaa> there's a fixed root detection logic in the pyrex branch
[01:04] <lifeless> cool
[01:04] <ddaa> just need somebody to just fucking review it :)
[01:04] <ddaa> before I've finished porting the whole code to pyrex, preferrably...
[01:05] <lifeless> well four reviewers are at sprint, its a little hard :)
[01:05] <ddaa> well, the assigned reviewer for this is SteveA...
[01:33] <ddaa> this is scary
[01:34] <ddaa> I spent four hours writing tests that somehow manage _not_ to exercice the bug I wanted to test...
[01:34] <kiko-fud> heh
[01:34] <kiko-fud> man it is FREEZING here today
[01:34] <kiko-fud> my hands are toast
[01:35] <ddaa> the scary thing is that I know that the code fixed the bug, because it made actual failing imports pass
[01:35] <ddaa> but I must just not be bending svn in quite the right way
[01:36] <salgado> kiko-fud, it's freezing here too. must be around 15C outside
[01:36] <salgado> kiko-fud, btw, do you have any other concerns with my mirror-reassignment branch or can I merge it?
[01:51] <carlos> kiko-fud: did you see danilo's comment on using google to index Rosetta??
[01:59] <carlos> kiko-fud: nevermind
[02:05] <Ubugtu> New bug: #71303 in launchpad "https://launchpad.net/people/<team>/+branches could do with status filtering" [Undecided,Unconfirmed]  http://launchpad.net/bugs/71303
[02:07] <kiko-fud> carlos, hmmm, not yet
[02:07] <kiko-fud> salgado, it is -5C here
[02:07] <kiko-fud> I just came back from a 2h bike ride
[02:07] <kiko-fud> my fingers HURT
[02:10] <kiko> carlos, I don't really care much about timeouts myself.. if rosetta isn't fast enough to render the suggestions, it's a bug anyway
[02:10] <carlos> kiko: well, if google gets most of the pages as timeouts
[02:10] <carlos> kiko: then it's useless ;-)
[02:10] <kiko> carlos, but it won't, surely. we don't really time out that much
[02:11] <carlos> and yes, it's a bug, but it's still useless :-P
[02:11] <kiko> heh
[02:11] <carlos> kiko: well, google will have a bunch of pages to index and our timeout reports will go out of scale
[02:11] <kiko> carlos, I somehow doubt it...
[02:11] <carlos> it's also a good way to catch the timeouts ;-)
[02:12] <carlos> kiko: if we don't merge a fix for the suggestions problem, I think so
[02:12] <kiko> well
[02:12] <carlos> btw, how's that going?
[02:12] <kiko> depends on how we want to fix suggestions
[02:12] <kiko> it's not going at all right now
[02:13] <kiko> carlos, how hard is it to do the DB refactoring you guys are planning on doing?
[02:13] <carlos> kiko: I don't think it should be too complex
[02:13] <carlos> but the rollout will take a while....
[02:13] <kiko> carlos, a lot of database code will need to change right?
[02:13] <kiko> heh
[02:14] <carlos> sure, but most of the time will be change the table name
[02:14] <carlos> or removal unneeded queries
[02:14] <kiko> mmm 
[02:14] <kiko> ok
[02:14] <carlos> to use a class property
[02:15] <kiko> yeah
[02:15] <kiko> ok!
[02:15] <kiko> I'm outta here for a bit
[02:16] <stub> Can we please finish the spec before changing the rosetta database schema?
[02:16] <kiko-zzz> stub?
[02:16] <kiko-zzz> what are you talking about?
[02:20] <stub> The rosetta db performance spec so sort suggestions.
[02:20] <stub> And other things
[02:22] <kiko-zzz> heh
[02:23] <kiko-zzz> stub, you're funny
[02:26] <carlos> kiko-zzz: night
[04:02] <BHSPitMonkey> mdke, thanks
[06:01] <Ubugtu> New bug: #71382 in launchpad-bazaar "Add the ability to push branches in SVN trunk" [Undecided,Unconfirmed]  http://launchpad.net/bugs/71382
[06:10] <kiko-zzz> wow, people will file bugs on anything