[00:01] mandel, I think that with your fix we could remove the references to WebClient.proxy_instance [00:02] alecu, I honestly do not have the balls to make that change at 1 am when it works :P [00:02] :-) [00:02] alecu, sounds like an improvement we can do later hehe [00:02] I thought you were much braver :-) [00:03] alecu, you mean a lot more stupid, which I am, but I fail under peer pressure :P [00:03] good point :-) [00:04] alecu, if we get this on beta 2 we will have enough user to be sure it works, later we need to remove that instance thing and test it on windows etc.. [00:06] mandel, approved! [00:06] alecu, toma! [00:06] ralsina, ping! [00:10] alecu, lol: 'and code looks as good as if I had written half of it.' [00:10] alecu, there are a number of files around in the project that can match that, like windows file events [00:11] alecu: pong [00:12] ralsina, review review review: https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-alecus-mess/+merge/98308 [00:12] mandel: hehe, ok, on it [00:13] BTW: sorry I disappeared, but had 2 doctor's appointments and could not get wifi in between [00:13] ralsina, I added the QApplication 'cause I saw it using libsoup.. just in case :) [00:13] mandel: haha [00:13] mandel: NEEDSFIXING! [00:13] mandel: nah [00:13] ralsina, no me undas!!! [00:14] mandel: line 27, str(domain)? [00:14] if domain comes from the cli, it could be unicode (but then the user is evil) [00:15] +1 [00:15] So, alecu, mandel, what did I miss? [00:16] ralsina, hm.. good point, but yes, he is evil setting a proxy with unicode [00:17] mandel: and it's going to be *really* obvious in the logs [00:17] ralsina, I was not paying much attention to irc to be hones, I was a little blocked trying to get that working with a factory.. this area of qt is buggy, and the bug is 3 years old O_o [00:18] mandel: ok, things happen [00:18] ralsina, I'm off to bed, catch you tom :) [00:18] mandel, alecu: did you guys find any clues about creds-qt on windows? [00:19] ralsina, will this land in the beta 2? [00:19] ralsina, I can take a look at that early in the morning tom :) [00:19] bye mandel [00:19] ralsina, no: no clues [00:20] alecu: ok [00:20] hahaha must be annoying to google a problem and get a page in your manager's blog, and he's not even around [00:21] mandel: we are probably going to do a release on windows that doesn't match exactly with ubuntu's [00:21] because we don't have time to merge the windows fixes before the ubuntu release [00:22] mandel: if you can take a look, and have nothing else, I would really really appreciate it [00:22] ralsina, ok, I'll take a look to see what is going on, can you send me an email with how I have to merge the branches for windows? [00:25] mandel: already did it yesterday ;-) [00:25] mandel: will resent just for you [00:26] ralsina, hmm if you sent it I have it.. I was updating my email client and there are some filter rules missing.. [00:26] mandel: I will resend just in case [00:26] ralsina, we get a ridicoulus about of mail per day.. [00:27] mandel: yes we do, I get 3 times what you get, too ;-) [00:27] mandel: resent [00:27] ralsina, which is stupid.. important ones get lost in the noise.. [00:28] ralsina, I'll take a look at that bug tom morning as well as getting jenkins set up to scream at us when we brake windows [00:28] ralsina, but with no emails, irc or twitter or something funny :P [00:28] mandel: cool, thanks [00:28] ralsina, like a chuck norris fact when you brake one of the platforms hehe [00:29] mandel, that already exists! [00:29] https://wiki.jenkins-ci.org/display/JENKINS/ChuckNorris+Plugin [00:29] I think I am going to be sane and just do one more review then rest [00:30] instead of the original plan of banging my head against windows for 3-4 hours [00:30] ralsina, lol [00:33] mandel, alecu: if this means proxy is done, great job, guys [00:33] * ralsina sees he wrote "done" and laughs histerically [00:33] let's say "shipped" ;-) [00:33] ralsina, there are some corners to make smoother, and the credits goes to alecu [00:34] ralsina, while I'll take the money :P [00:34] mandel: go to sleep! [00:34] * alecu should stop reading this article and get some food and sleep too. [00:39] ralsina, alecu I think I lost it: http://www.youtube.com/watch?v=bLr4pCg4PNk [00:39] lol, I'm going to bed :P [00:40] mandel: yes please ;-) [00:48] alecu, we should get a tecno viking plug in, is scarier that chuck :P [00:48] ok, of to bed :) [00:48] see you all tom! [00:49] bye mandel! [00:57] annnnnnnd I am gone too, will be working early tomorrow. [01:03] bye too! === JanC_ is now known as JanC [09:21] morning all! [09:32] Good morning all! :D [09:47] Good morning! :) === jml` is now known as jml === mpt_ is now known as mpt [11:14] good morning! [11:16] gatox, morning! [11:17] mandel, hi! how are you? [11:17] gatox, did you watch the season finale then? (Walking dead) [11:17] mandel, yes... i said EOD and run to watch the episode! :P [11:17] gatox, tired.. ended up working 'til late yesterday and now sorting out the windows code to make sure proxy works there :) [11:18] gatox, it is no an ending.. I cannot wait for season 3! [11:18] mandel, me too........ [11:18] mandel, yes, i saw your twit about the review last night [11:19] gatox, hehe [11:20] gatox, have you watch the show in english? [11:20] mandel, yes, of course, i download it in english and 720 :D [11:21] gatox, then be ready for the wtf: https://www.youtube.com/watch?v=ZblD1BNudAk [11:21] gatox, I cannot believe his is from... I'll let you watch it :P [11:22] mandel, let me see [11:36] buenos días! [11:37] nessita, buenas [11:37] hola gatox [11:37] nessita, buenos dias! [11:37] hola mandel! [11:56] ralsina: when you can, could you please fix the conflict in https://code.launchpad.net/~ralsina/ubuntuone-control-panel/tweaks/+merge/97244 ? [12:41] nessita: did you see my comment re: string formattun [12:41] formatting, even [12:41] thisfred: I did, I was not aware that .format is much slower [12:41] thisfred: have a link I can read? :-) [12:42] good morning! [12:42] nessita: it was a thread recently on python-dev, let me look [12:42] nessita: of course! [12:42] ralsina: thanks [12:42] thisfred: *how* slow are we talking here? ;-) [12:43] nessita: the more imortant point was it ties us to a recent python, thou [12:43] ralsina: in practice I'm sure it won't matter [12:43] thisfred: and how "recent" is that? [12:43] thisfred: I guessed so [12:43] ralsina: 2 weeks or so? [12:43] thisfred: no, I meant how recent a python :-) [12:44] ah, I think it may be 2.7 [12:44] mandel, urbanape: how nice would it be to have this? http://blog.rburchell.com/2012/03/qt-51-aka-when-qfilesystemwatcher-might.html [12:44] thisfred: yes, I also read that, but I'm not concerned about that... it ties us to 2.6, right? [12:44] ralsina: also since the old formatting is never going away, and already used everywhere [12:45] thisfred: yeah. But the new formatting is much less error prone [12:45] ralsina, really nice! the actual qfilesystemwatcher of qt sucks right now [12:45] thisfred: although since we d have the plan to go back in time up t maverick (hopefully) we should not do stuff that's not available in maverick's python [12:45] gatox: yes it does [12:46] ralsina: also we make it harder than necessary to port to legacy distros ; [12:46] ) [12:47] ralsina, thisfred .format is available since 2.6 ;-) [12:47] and lucid has 2.6, so no worries on that front [12:47] ralsina, thisfred: http://docs.python.org/library/stdtypes.html#str.format [12:47] yes 2.6 is good enough [12:47] nessita: yep 2.6, so not so big a pro [12:47] blem [12:48] ralsina, looking [12:48] with legacy distros I was referring to red hat which I think is still stuck on 2.5 [12:48] not enough of an incentive maybe [12:48] thisfred: let me dig deep in my sould to see how much I care... [12:48] I'm sure we already require 2.6 for other stuff [12:49] thisfred: I know it's there somewhere [12:49] heh [12:49] also, we'll probably be going to 3 soon [12:49] ralsina, it states the problem we already have with move on linux.. is a delete created if you are looking at the creating dir.. [12:49] thisfred: fedora is 2.7, RHEL is so not our target [12:49] * mandel hates file systems.. [12:50] nessita: ok, so critique retracted ;) but then I approved anyway [12:50] thisfred: thanks :-) [12:51] ralsina, mandel: I still think Apple made the right choice with iCloud. But it's orthogonal to a model like Dropbox. [12:52] Simpler, but less room for fuckupery on the part of the filesystem [12:52] urbanape: since they own the freaking FS they have little excuse [12:52] urbanape: but yes, there is a diminishing returns thing on how much effort you can put there [12:53] There's also the belief that hierarchical approach is outdated/doesn't fit the users' minds so much. [12:53] It would be hard to get Ubuntu apps to play along without a (more?) common underlying framework behind the apps, too. [12:54] urbanape: yep [12:55] ralsina, urbanape I wonder what I they using instead of ReadDirectoryChangesW in qt [12:56] and it uses kqueue for the time being [12:56] mandel: why not ask the guy? [12:58] nessita, for completeness' sake: http://python.6.n6.nabble.com/Status-regarding-Old-vs-Advanced-String-Formating-td4503327.html [12:59] ralsina, so, I fixed the issue with the dialog not working on windows and... then I hit this bug: http://twistedmatrix.com/trac/ticket/2226 [12:59] nessita, how much control do we have over patched to the qt reactor? [12:59] mandel: looking... [12:59] also: http://pyvideo.org/video/948/pragmatic-unicode-or-how-do-i-stop-the-pain [12:59] ralsina, python keeps running so do int a sys.exit(return_code) does not return the correct code.. bummer [12:59] thisfred: thanks! [13:00] mandel: not sure what you mean... we can propose patches to upstream thru github [13:00] nessita, ok.. [13:00] mandel: ugh [13:00] * mandel goes to github [13:00] mandel: why? [13:00] mandel: on windows we can build qtreactor from whatever we want [13:01] mandel: and we don't use it on linux [13:01] ralsina: we use it to run tests [13:01] mandel: we are literally building it out of a branch tarball on github [13:01] (on both) [13:01] nessita: ok, but that won't be affected by this, because it's a bug on exiting the process [13:03] ralsina, nessita the issue i the following, all the thread that are used by deferToThread with qt reactor get glocked if you use exec_ for a QDialog (one bug in the reactor) [13:03] nessita: while fixing the conflict in tweaks, I think the gotowebbutton __init__ is kinda broken. Specifically, self.legend is never going to be not None [13:03] ralsina: why not? [13:03] ralsina, nessita my work around was to use show and connect to finish which will get the return code, then do a sys.exit(return_code) and reactor.stop() but that does not work (an other reactor bug) [13:03] nessita: oh, right, subclasses, nevermind [13:04] mandel: I would seriously consider bringing those to upstream... [13:04] ralsina, nessita a third way to fix this is do a callFromThread in the windows keyring so that we do not use a thread but execute the call in the main loop of Qt, then return a deferred that gets fired when done, which does work and does not require to fix the reactor [13:05] nessita, sure, I can loop at what is being done in the reactor.. [13:06] mandel: did you talk with alece about this? he is our master when it comes to reactor stuff [13:06] nessita, will do when he wakes up.. we work 'til late yesterday [13:07] nessita, is this upstream => https://github.com/ghtdak/qtreactor [13:07] ?? [13:08] mandel: yes [13:08] nessita, thx [13:09] and alecu is in uruguay [13:13] ralsina: I got a little lost regarding your tweaks branch... is it ready? [13:14] nessita: yes, pushed the conflicts fix [13:14] ack [13:16] * gatox brb..... goes to buy fruit salad to try to unblock the brain with the backend issue :S [13:16] if fruit salad doesn't work.... i'm going to buy a gun [13:18] gatox: you may also try getting help. You know. Professional help ;-) [13:21] ralsina, how well does the python threading and the pyqt interact? I'm looking at http://twistedmatrix.com/trac/browser/tags/releases/twisted-8.1.0/twisted/python/threadpool.py#L27 which is what is used as a threadpool in twisted and I wonder how well does that work [13:21] mandel: not very well AFAIK [13:22] ralsina, he, then I guess there is some way to easy improve this :) [13:22] mandel: or rather, python threads and pyqt work, as long as you don t do anything pyqt-ish in the non-main threads [13:23] back! [13:27] hello #ubuntuone! [13:27] hello alecu! [13:27] hola alecu [13:28] alecu, buenas [13:28] ralsina, mandel, alecu, gatox: isn't the issue in windows, related to the keyring using deferToThread and not working, the thing that is making the 'Sign In' button not being enabled? it makes sense [13:29] since that would mean that ussoc can not progress and can't tell if there is no credentials in the system [13:30] is there any branch about that, that i could test? [13:30] nessita, yes, that looks similar to the problem I see in the proxy creds dialog (which is a smaller simpler app) [13:30] nessita, deferToThread on the qt reactor is not work at the moment, nor is callInThread [13:30] gatox: apparently there is no fix yet, AFAIK [13:30] nessita: may be! [13:30] nessita, yest callFromThread does (which means, q in the main loop) [13:30] mandel: how come this worked before? [13:30] right now i'm seeing like the app get stuck when trying to do: reactor.connectTCP and ask if it's listening [13:31] mandel: it may be the exact same thing i the -ui-qt script! [13:31] nessita: this never worked on windows with that on a separate process [13:31] ralsina: aaaaahhhhhhh uuuuhhhhhhh craaaaaaaap [13:31] nessita, I'd need to look at all the changes, I've noticed that when a dialog uses exec_ it does not work, but if you use show it does [13:32] nessita: before it was either in u1cp r in -installer, so it worked [13:32] mandel: exec_ starts a secondary event loop, that may break reactor [13:32] mandel: we're displaying the "main" sso UI with show.... [13:32] mandel: no, I lie, it uses exec_ [13:32] mandel, ralsina: but is a separated process... why would that interfere with the "first" mainloop? [13:33] nessita: lost me there. May not be the same thing then. [13:33] in any case, I'm incline to think that the "backend" not ready bug is related to this [13:34] nessita, who is working on this? [13:34] gatox: a couple of tests you could do on Linux, is to lock your keyring, and then try to sign in [13:34] nessita, in a diff process it should not be an issue.. I now exec_ brakes things I have not looked any deeper atm [13:34] gatox: mandel I guess, but not sure. Ideally we'd need advice from alecu_, being him our reactor master [13:35] nessita, sorry: the reactor master you are looking for is dobey :-) [13:35] alecu_: can you mumble? [13:35] nessita, I know how to *use* reactors, not how they work internally. [13:35] que pasa? [13:35] could we get rid of deferToThread and just block on those calls? The windows keyring should be quick anyway [13:35] alecu_: I have some ideas I would like to run by you [13:35] nessita, I can try mumbling, but not sure if mumble will work for me today, since I'm not working from home. [13:35] ralsina: what if it's locked, like the linux keyring can be? [13:35] nessita: you can't lock the windows keyring [13:36] nessita: AFAIK at least [13:36] alecu_: let's try... mandel, gatox, can you do a quick mumble? [13:36] nessita, yes! [13:36] ralsina: will research. I know you're in a meeting, but if you want/can, mumble? [13:36] nessita, sure [13:36] on mumble [13:36] I am at the design call, in mumble so no [13:36] :-( [13:37] gatox: would you drag me to the channel you're in? I can't [13:38] alecu_: we're in mumble if you can [13:44] nessita, this connection is very flaky, so no mumble today. [13:45] alecu_: :-/ [13:45] nessita, so, how can I help? [13:46] nessita, lp:~mandel/ubuntu-sso-client/proxy-creds-windows [13:46] alecu_: we're mumbling, will get back to you with concrete questions [13:46] thanks! [13:46] ack [13:48] nessita, here is the 'interesting' changes: http://bazaar.launchpad.net/~mandel/ubuntu-sso-client/proxy-creds-windows/view/head:/ubuntu_sso/qt/proxy_dialog.py#L158 [13:50] ralsina: i don't think you saw it yesterday, but syncing with a proxy works fine. still trying to figure out what the on_upload_finished issue is with getting the wrong number of arguments [13:50] briancurtin: awesome! [13:50] briancurtin: if you can pop into mumble, they are covering some windows-related bugs now [13:50] ralsina: logging in now [13:51] briancurtin, did anybody else got that error on XP? [13:52] alecu_ it turns out that same thing happens on Win7, just way way faster (i noticed it on XP since its a really slow VM) [13:52] so its all windows, apparently [13:53] briancurtin, so, it's a relief knowing it's not a platform dependent bug. [13:53] yep [13:53] makes it sound a lot less scary :-) [13:53] yeah, if that was XP only, it was caused by martians [13:54] ralsina, martians that should be extinct long ago! [13:54] EOLife'd martians, yes [13:55] "Your race is doomed!!!". 'No, we *will* prevail' it said while climbing back in the saucer. [14:01] briancurtin, nessita, gatox very important regarding spawing a ui that uses the qt4 reactor and expecting the return code: http://twistedmatrix.com/trac/ticket/2226 [14:02] sys.exit(3) reactor.stop() with the qtreactor does not work.. :( [14:16] mandel: so, let me know when you come back from lunch and when the branch is ready for inspection :) [14:19] so, good news? [14:29] ralsina: kinda... I'm very confident gatox's backend issue is caused by the Qt SSO UI not running the qt4reactor in windows [14:29] ralsina: regarding mandel's issue, need to see his branch [14:29] nessita: that is good news [14:30] the stuff about "the app doesn't end', we have seen before [14:30] or at least very similar things [14:32] nessita, yes, it was reproducible from sources also..... i'm adding the qtreactor now [14:32] gatox: yey! [14:36] alecu_, ralsina, mandel: there are like 4 branches from mandel that will not make it for this release.... were you aware of that? [14:37] nessita, can you please point me in the direction of the qtreactor on u1-cp? [14:37] gatox: yes, first look at the bin/ubuntuone-control-panel-qt [14:37] ahhhh ok [14:37] 25 if sys.platform == 'win32': [14:37] 26 import qt4reactor [14:37] 27 qt4reactor.install() [14:37] 28 [14:37] 29 from ubuntuone.controlpanel.gui.qt import main [14:38] gatox: then, go to ubuntuone.controlpanel.gui.qt.main [14:38] nessita: I piged him about one yesterday but it has a broken prerequisite [14:38] thanks! [14:38] gatox: in there, see the windows.py file :-) [14:38] gatox: I advice doing something similar to the qt sso ui [14:38] nessita, will do [14:54] * mandel back [14:54] nessita, yes, we know is not a problem, is a feature that we did that won't be used in our case [14:55] nessita, I'm making the changes to the branch to see the different issues [14:56] mandel: got lost, what thing is not a problem? [14:56] nessita, branches not landing, we know they wont and we are not in a hurry for those [14:57] nessita, and they should have merge issues.. [14:57] mandel: ack then [15:00] me [15:00] me [15:00] me [15:01] me [15:01] me [15:01] ralsina, alecu, dobey, thisfred? [15:01] meh [15:01] me [15:01] me(no notes yet) [15:01] briancurtin: go! [15:01] DONE: debugging, proxy IRL testing [15:01] TODO: hooking up VS to debug on_download_finished TypeError now that i know it's not XP specific, get any other windows stuff out of the way to get back with urbanape for a bit [15:01] BLOCKED: XP kinda sucks [15:01] NEXT: urbanape [15:01] DONE: looked into /dev/fsevents reading for file-level event granularity [15:01] TODO: Tests, land branch, for god's sake! [15:01] BLOCK: my brain. [15:01] gatox, go. [15:01] DONE: [15:01] Propose the unicode fixes branch for sso and fixed the test_main suit. Investigate the backend issue. [15:01] TODO: [15:01] Finish with the backend problem. [15:01] BLOCKED: [15:01] No [15:02] nessita, go [15:02] DONE: proposed computer-to-cloud-page u1cp branch, reviews [15:02] TODO: land branches for release, help with stucked apps to mandel and gatox, leave early due to teaching duties [15:02] BLOCKED: nopes [15:02] NEXT: mandel [15:02] * mandel finishing notes [15:03] DONE: We got the webclient working correctly by fixing bug (late night work). Looked at issues with prxoy creds dialog on windows. [15:03] TODO: more creds dialog debugging. Look at jenkins. Write sso script example for aquarius. [15:03] BLOCKED: no [15:03] dobey, please [15:03] λ DONE: askubuntu, bug #932103 (pinging right people, and testing) [15:03] λ TODO: bug #953119, releases, uploads [15:03] λ BLCK: none. [15:03] me? [15:03] Launchpad bug 932103 in gstreamer0.10-fluendo-plugins-partner (Ubuntu Precise) "Cannot install MP3 playback support; silently fails" [High,Fix released] https://launchpad.net/bugs/932103 [15:03] Launchpad bug 953119 in ubuntuone-client-gnome (Ubuntu) "valgrind invalid read error" [Undecided,Triaged] https://launchpad.net/bugs/953119 [15:03] alecu_: go! [15:03] ALL: is there any branch that *needs* to be in tomorrow's release that hasn't landed yet? [15:04] nessita: I don't think so, if we don't mind the windows release being done from another tarball [15:04] DONE: got a few windows-specific branches merged and the focus tweaks branch, reviews, tech leads call, mgmt call (in progress), design call, missed part of the day because of doctor TODO: help with windws fixes, start windows release process BLOCKED: not really [15:05] gatox, briancurtin, alecu_, mandel, dobey, thisfred, urbanape: is there any branch that *needs* to be in tomorrow's release that hasn't landed yet? [15:05] DONE: reviews | gave up on segfault for now TODO: u1db filter functions in C | u1db and other reviews | turn 40 | go to Savannah BLOCKED: no NEXT: ralsina! [15:05] nessita: not from me [15:05] nessita, not from me AFAIK [15:05] nessita: is there a branch that fixes everything to be how i wante it to be? :) [15:05] nessita: windows specific, but we need this one in whenever the windows release happens (1 approval already): https://code.launchpad.net/~brian.curtin/ubuntuone-client/930398-windows-is_root/+merge/96224 [15:05] nessita, i don't have branches for review.... working in the qtreactor thing right now [15:05] thisfred: happy birthday! (you turn 40 today? :-)) [15:05] nessita: tomorrow :) [15:06] ack to all the branch info, thanks [15:06] thisfred: will try remember saying happy bday tomorrow then :-) [15:06] ALL: so I won't be here the rest of the week [15:06] dobey: no :-) [15:06] nessita: I'll be in Georgia :) [15:06] thisfred: sorry, I shot first. Like Han Solo [15:06] np [15:07] the state, or the country? [15:07] thisfred, 40!?! If you where a british northern girl you could be my mother! [15:07] I'd have given you up for adoption for sure [15:08] mandel: your mother gave birth to you at the age of 13? [15:08] lol [15:08] explains a lot [15:08] dobey, that is why I said 'british northern girl' my mother is spanish :P [15:08] that and the heavy drinking she did during the pregnancy [15:09] heh [15:09] lol === alecu_ is now known as alecu [15:10] * gatox lunch! [15:14] nessita: we have a new comment in the color chage bug from robert, is that good enough to move forward/ [15:14] ? [15:14] ralsina: looooooking === gatox is now known as gatox_lunch [15:16] ralsina: may be enough.... I guess we can start the process. Asking joshuahoover for confirmation is the best thing to do right now [15:16] nessita: ack [15:16] ralsina: also, I proposed https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/license-page/+merge/98432 so we can add the strings before releasing (though the license page is never shown, for now) [15:17] nessita: looking [15:17] nessita: yeah, that should be good...i'll move it forward as soon as i get off this current call i'm on [15:17] joshuahoover: thanks!!! [15:17] nessita: I can do the --installer branch later myself if you want [15:17] nessita: since I have been hacking on command lines lately [15:17] ralsina: you can try, but it will not be trivial [15:18] nessita: sure, I can try, if I fail I'll cry and let you know ;-) [15:18] ralsina: since the wizard is created 'automatically' from the setupUi, and thus I don't know how to pass custom arguments to custom widgets [15:18] nessita: I'm siure there is a way ;-) [15:19] briancurtin: your last note in the standup reminded me: Should I get W7 for my VMware? [15:19] I'm currently running XP [15:19] ralsina: nice [15:22] nessita, btw: was it useful, the info on the package dependencies I provided yesterday ? [15:22] nessita: +1 on that [15:22] alecu: yes, the -proxy package is already available (I mentioned this at noon yesteday). Have you try it? :-) [15:23] nessita: do you want to do the release of ubuntuone-client as well? [15:24] dobey: I can if you teach me :-) [15:24] dobey: make dist I guess and then upload that tarball? [15:24] nessita, I updated lp:~mandel/ubuntu-sso-client/proxy-creds-windows I have added a new arg for testing, if you use --use-exec the diaog will be launched using exec_ otherwise a show + finish signal solution is used (which 'works') [15:24] ralsina, ^ [15:24] oh, right. it's a bit more complicated than that, because of gtk-doc [15:24] ralsina, and if you have time we can take a look to see if I did something stupid [15:25] mandel: I will be available in about 15 minutes [15:25] mandel: so, I just confirmed that the reactor has to be installed *before* the qapp is created [15:25] nessita: i'll do the tarball release, and you can do the upload [15:25] dobey: sounds great [15:26] nessita, before? then the doc is a lie: https://github.com/ghtdak/qtreactor [15:26] urbanape: yeah i think you might as well have it [15:26] mandel: that stinks of "QDialog.exec_()" breaks the qtreactor to me [15:26] mandel: so I am ok with using show() [15:26] nessita, from the docs in github: '= Using the Qt4Reactor = [15:26] In your own code, BEFORE you import the reactor... [15:26] app = QApplication(sys.argv) [15:26] import qt4reactor [15:26] qt4reactor.install() [15:26] ' [15:26] nessita: if you install the reactor first, it will create a QCoreApplication, IIRC [15:26] ralsina, does the trick, except for the fact that I cannot return the code via sys.exit :( [15:27] briancurtin: do we have an MSDN account or do I just grab a copy somewhere? [15:27] (legit, natch) [15:27] mandel: we had that problem before, didn't we? [15:27] ralsina, I have not seen that.. [15:27] urbanape: that's a question for ralsina (i'd like the answer as well, since my XP VM was created with an MSDN i got elsewhere) [15:27] ralsina, mandel: then we have it wrong in the controlpanel... the reactor is installed *before* the qapp [15:27] ralsina: is u1cp working on windows? [15:28] nessita: yes [15:28] nessita: which is why I am pretty confused right now [15:28] urbanape, I do :) [15:28] ralsina: well, we're going it the other way around there (first installing the reactor, then creating a qapp) [15:28] mandel: wanna test having the other way around? (first installing the reactor, then creating a qapp) [15:28] nessita: so, let's do it like you say and let's see :-) [15:28] urbanape, and is legit, gatox alecu nessita and I have msn accounts :) [15:28] nessita, sure! [15:29] if that wouldn't work, QApp will complain [15:37] ralsina, nessita updated chaging the order, that is, first import & install then create instance, it works, wtf? also, look at the logs, the qt reactor.qApp is a QCoreApplication and not a QApplication.. [15:37] branch is the same [15:37] mandel: awesome and weird [15:38] ralsina: perhaps a Qapp can be created after a qcoreapp and the latter gets "upgraded"??? [15:38] nessita: never heard of such a thing [15:38] nessita: but let's pick our battles ;-) [15:38] gatox_lunch: may I have a review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/license-page/+merge/98432 ? [15:40] ralsina, nessita , I think that is a QCoreApplication because the reactor is calling QCoreApplication.instance() which return the QApplication instance casted as a Core app [15:40] mandel: you lost me [15:42] nessita, nothing I was guessing.. and is wrong.. if you do QApplication(sys.argv) and then QCoreApplication.instance() you get the QApplication [15:43] dobey: do you know if u1client is ready to be updated from trunk? [15:44] nessita: i don't know of any pending branches we need to land for today [15:44] dobey: besides the tarball, are you doing the update branch as well, or shall I? [15:44] you can do that [15:44] dobey: ack [15:45] ralsina, from here: http://cep.xor.aps.anl.gov/software/qt4-x11-4.2.2-browser/dd/d94/qdialog_8cpp-source.html I see that exec uses a QEventLoop which might be the culprit, can you have a QEventLoop without a QApp? [15:45] mandel: probably not [15:46] mandel: you can have one with a QCoreApp though [15:46] ralsina, and it can deal with UI, right? [15:46] mandel: no [15:47] mandel: you need a real qapp before you can create a qpainter [15:49] mandel: what's your branch again/ [15:49] ? [15:49] ralsina, branch: lp:~mandel/ubuntu-sso-client/proxy-creds-windows [15:50] mandel: ok, looking now [15:53] ralsina, fancy a major wtf? I have one for you :) [15:53] ralsina, execute this on linux and windows http://paste.ubuntu.com/892348/ [15:53] mandel: why not [15:54] ralsina, if I did not screw it up you are going to me like me [15:54] * mandel FUUUUUUUUUUUU [15:54] Apple Store Genius Bar appt, part deux! [15:54] biab [15:54] mandel: ok, so you can exec_ a dialog without exec_ ing the app. That's a bit surprising [15:55] ralsina, you did it on windows only? [15:55] mandel: yes [15:55] ralsina, try linux now [15:55] mandel: can't do it on linux now [15:55] mandel: so explain :-) [15:55] ralsina, well, I can tell you => segfault [15:55] mandel: ha! [15:56] ralsina, I don't think our problem is related to that, but is a serious what the holy frack windows? [15:56] ok, i really need to get lunch. bbiab kids [15:57] ralsina, and exec_ is probably executing the QEventLoop correctly as long as there is a QApplication, that means that the reactor does not run the dialog ergo no deferToThread [15:57] mandel: well, you prefer the segfault? [15:57] mandel: right, the reactor is only moved by the main event loop [15:57] mandel: so, we use show(modal=True) and that's it [15:58] ralsina, yes, and also + finish.connect, right? [15:58] mandel: I am looking now at your exit_code function [15:58] ralsina, a diff story is returning via sys.exit, which does not work [15:58] mandel: you can't do stuff after sys.exit! [15:58] guys, I gotta run to the university [15:58] mandel: let me take a whack at it :-) [15:58] ralsina, gatox_lunch: call me to my cell if anything comes up [15:59] nessita: sure! [15:59] ralsina: I will propose the stable-3-0 updates branches during the afternoon [16:00] ralsina: actually, I have one update ready now: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/stable-3-0-update-2.99.91/+merge/98440 [16:00] the rest will come during the day, will email those [16:00] mandel: it works perfectly if you exit with QtCore.QCoreApplication.instance().exit(code) [16:01] ralsina, sweet! [16:01] nessita: will review them after I boot to linux. [16:01] ralsina, so we have a dirty fix :) [16:01] mandel: what's dirty? ;-) [16:01] mandel: have not tried exiting with QApplication.instance() but should be the same thing [16:01] mandel: it's even crossplatform! ;-) [16:04] ok, I'm soon to be off [16:04] bye all! [16:04] mandel: so, go ahead, ping me when you have a branch to review :-) [16:04] ralsina, hm.. have you done an IRL [16:04] mandel: yes [16:04] ralsina, when you store the creds it does not exit.. at least in my system [16:04] mandel: running the script, not the whole thing [16:05] mandel: works perfectly, even stores the credentials [16:05] mandel: let me give you a diff [16:05] ralsina, or push to a branch and I merge (me lazy! ) [16:06] mandel: ok, let me try a slightly cleaner fix [16:06] ralsina, must say, it is a cute bug :) [16:08] mandel: fix at lp:~ralsina/+junk/mandel [16:08] mandel: ok, rather, it's being pushed there [16:09] mandel: ok, let me push it to a ot-junk place because that takes forever === gatox_lunch is now known as gatox [16:09] mandel: here it is: lp:~ralsina/ubuntu-sso-client/mandel notice that we *still* have to stop the reactor [16:10] ralsina, ah.. I was missing that detail [16:11] mandel: why we have to do that after we stop the event loop, I have no idea, but as long as we have a recipe, I am happy [16:13] ralsina, it is a very weird thing.. that qt reactor scares me.. [16:13] mandel: just walk carefully and try to look bigger than you are [16:14] mandel: we only need it because of IPC and this defertothread right? [16:15] ralsina, in most of the stuff ipc, in this case, defertothread [16:15] ralsina, I know that the qt login ui implementation if it has IPC is going to have this problem [16:15] gatox, ^ [16:15] unless exec_ is not used [16:15] mandel: ok, let's think about how to get rid of it for next year [16:16] mandel, gatox: talk with each other ;-) [16:16] ralsina, so many things we need to fix for next year! [16:16] gatox, are u single? [16:16] mandel, ok...... you are scaring me now [16:16] ralsina, that ^ kind of chat? :P [16:16] mandel, gatox: because this fix needs to be done in the UI too [16:16] mandel, ralsina i'm trying to add the call to the qtreactor in the ui of sso right now [16:17] ralsina, yes, it does, so we should make him do one of the reviews to say wtf and learn about it :) [16:17] gatox: you will also be hit by the exit code bug we just fixed [16:17] ralsina, eh? what was the problem about that? [16:18] gatox, there are several things, first doing this: http://paste.ubuntu.com/892348/ [16:19] gatox, well esecute the dialog and not run the twisted reactor O_o [16:19] gatox, second, you will need to use QApplication.exit() to set the return code and after reactor.stop [16:20] i didn't use QApplication.exit() before because it wasn't working..... you have it working with that now? [16:21] gatox, yest, let me clean up a little the branch and will ask for a review [16:21] mandel, so.... let me understand..... this thing that you are doing with the qapplication, qcoreapplication.... is related in anyway with the problem i'm having with the sso ui? [16:21] mandel, do i need to do the same? or the qtreactor approach is the correct one' [16:21] ? [16:22] gatox, you have to use the qtreactor which brings a long some bugs that we have found how to work around [16:26] ralsina: i'm spending too much time right now to even get debugging setup for this on_download_finished problem. since this isn't a showstopper, are there other things i should work in order to push forward the release? [16:29] briancurtin: we are waiting for two branches (mandel & gatox) and we may have something that works [16:29] briancurtin: so, maybe lunch? ;-) [16:29] cool, well i'll keep going on this for now and await their branches [16:30] yeah i should do that soon [16:55] briancurtin: if you are around, 1:1? [17:03] ralsina: logging in now [17:03] briancurtin: cool [17:04] if mumble will ever start running... [17:12] ralsina, gatox FYI fixing tests.. is not that easy to ensure the changes we made are tested [17:12] mandel, i'm having some troubles with the reactor already installed.... i'm looking at that now.... [17:12] ugh the pollen [17:16] gatox, let me know if you need a hand :) [17:17] mandel, yes, thanks.... i already fix that.... but now there is another problem (as always jeje) [17:18] gatox, unicode? [17:18] mandel, jeje not this time [17:26] polen, unicode, you guys are complainers [17:26] * ralsina goes do something manager-like [17:26] ralsina, my bottom itches! [17:26] alecu_: it's all the hat sitting [17:26] ralsina, no tall the tests pass won windows, right? [17:26] lol [17:27] mandel: on sso? [17:27] ralsina, yes, I'm getting a dirty reactor.. [17:27] ralsina: well, snot and keyboards don't mix [17:27] mandel: on u1cp things get stuck, and yes we have dirty reactors like chernobyl [17:27] mandel, I'm replying to rm-cbride mail on proxies. [17:27] dobey: but do they BLEND? [17:27] mandel: one of these days we need to debug that [17:28] mandel: the reactors are very dirty [17:28] ralsina: almost certainly [17:29] alecu_, which mail? [17:29] alecu_, oh, ubuntunet-discuss, got it [17:30] also, the new theme changes are weird [17:30] to light-themes [17:31] ralsina, briancurtin can you add a bug report regarding the reactors, I'll try to clean them asap [17:31] mandel: sure [17:31] dobey: you mean our theme changes, or the light-theme changes? [17:32] ralsina: light-themes; some fonts look fuzzier now, and the desaturation when not focused is a bit odd [17:32] dobey: ack [17:32] and doesn't seem to work for all widgets [17:38] man. some apps really do not like multiple screen setups [17:45] mandel: how's that branch coming? [17:46] ralsina, getting test running on both platforms, maybe 10 mins more and I'm done [17:46] mandel: cool! [17:47] * dobey discovers a unicode bug in apport [17:51] dobey: assign to gatox! [17:51] heh [17:52] mandel: bug #960436 [17:52] Launchpad bug 960436 in Ubuntu One Control Panel "tests from trunk fail on windows" [Undecided,New] https://launchpad.net/bugs/960436 [17:52] my tear ducts are itchy :( [17:55] dobey: too much unicode? [17:56] too much plant reproductive code [17:57] brb!! === gatox is now known as gatox_brb [18:04] * mandel forgot how slow is too work on a windows vm.. [18:14] i should port my theme to gtk3 [18:17] hi davidcalle [18:18] hi dobey! [18:18] how are you? [18:18] dobey, fine and you? [18:19] davidcalle: good. have you got any further with finishing the rb scope? [18:20] dobey, it's waiting to be merged. [18:20] dobey, https://plus.google.com/u/0/117867558830601601230/posts/j6ikKygHS48 [18:20] to be merged ... in the music lens* === gatox_brb is now known as gatox [18:21] back [18:27] ralsina, do you have a bug for the proxy issue? [18:27] mandel, are you eod already? [18:27] mandel: no [18:27] ralsina, I'll create one then [18:27] gatox, nearly, do you need help? [18:27] mandel: you can use the one about the script not being crossplatform [18:28] mandel, if you have a minute.... i'm really stuck with this...... i've installed the qtreactor, but the problem persist [18:28] gatox, which is the problem? [18:28] davidcalle: ah, is there a different branch now? the rb-scope one says "work in progress" for its status, and hasn't had any changes since Mar 13 :) [18:29] ralsina, created a quick one, I can also add it to that one so that you just close it when all scripts are done [18:29] ralsina, do you have the bug number? [18:29] dobey, indeed, it's https://code.launchpad.net/~scopes-hackers/unity-lens-music/rb-scope/ [18:29] mandel: nah, the quick one is enough [18:29] mandel, the reactor seems not to be working or something.... twisted get stuck on connectTCP [18:29] ah [18:30] mandel, the same problem we discuss early today.... that it was going to be fix probably with the qtreactor..... can i show you a branch on how i use the qtreactor and you can tell me if that is ok? [18:31] gatox, of course you can, shoot! [18:31] mandel, uploading the branch [18:33] ralsina: for the autostart piece moving into CP, is there any preferred location it should go? i'm guessing just somewhere in main so it gets added on startup? [18:34] it used to be at the end of the wizard in u1-win-installer [18:34] gatox, ralsina please: https://code.launchpad.net/~mandel/ubuntu-sso-client/proxy-creds-windows/+merge/98481 [18:34] briancurtin: somewhere after the user has logged in, I suppose. [18:34] mandel, on it [18:34] mandel, this is the branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/backend-problem [18:34] ralsina: ah, better idea :) [18:34] briancurtin: maybe at the end of the wizard in ubuntuone/controlpanel/gui/qt/wizard.py [18:34] oh :) [18:35] briancurtin: in the done() method [18:35] mandel, the bug can only be reproduce on windows..... if you start the ubuntu-sso-login-qt ui..... with --login_only..... and complete the form, the login button never became enabled [18:35] mandel: on it! [18:35] gatox, the issue is on windows not getting signals etc, right? [18:35] mandel: also looking [18:36] gatox, ack [18:36] briancurtin, thx! is a little hacky, or hackier than i like. The branch is related to the fact that using exec_ in qt uses a QEventLoop, that starts a new event loop and blocks the reactor go not allowing the deferreds to be fired [18:37] mandel, the issue is that the backend get stuck so when the setup of the pages try to use it fails, and the ui is not configured.... but the problem is not the ui, but the backend being stuck on connectTCP [18:37] briancurtin, the work around is to use show on windows and connect to the finished signals so that only event loop present is the one from the reactor [18:40] gatox, a priori thins look good [18:40] gatox, how are you testing it? [18:40] mandel: and since i'm not sure of what didn't work before this branch, what should i notice in an IRL test of it? [18:40] (still waiting for the diff to show up, so i don't know *anything* yet :) [18:41] mandel, trying to execute the ui..... you can execute python bin/ubuntu-sso-login-qt --app_name="u2" --login_only [18:41] briancurtin, you can go either to the keyring on linux of the credential manager on windows and see a test_domian entry with the username and password :) [18:41] mandel, or also, run first, bin/ubuntu-sso-login.... and then the ui in another process [18:41] both things gave the same result [18:42] briancurtin, do you know how to use meld with bzr? you can branch and then do 'bzr diff --old lp:ubuntu-sso-client --using meld' [18:42] briancurtin, a lot nicer than lp diff :) [18:44] gatox, do you mind if I walk the dog and give you a hand after? [18:44] mandel: ah, didn't know about that [18:44] have the dog do a review [18:44] mandel, yes! no problem.... [18:44] mandel, also if it's your eod go and enjoy the world! :P [18:44] briancurtin, he is not that smart hehe [18:44] no worries! [18:45] gatox, sure? I'd like to take a rest to be honest, but you can send me an email and I'll take a look while watching bad tv :) [18:45] mandel, yes!! no problem!! don't worry, really [18:46] gatox, ok, then do send me the email! [18:46] all, EOD here [18:46] mandel, ok! see you [18:46] mandel, bye! [18:46] ralsina, briancurtin let me know about the review in the comments, I get the lp email on my phone [18:46] will do [18:46] * mandel walks dog and freezes ass [18:47] briancurtin, and be harsh :) [18:47] mandel: ack [19:05] gatox: can you do a test/lint run of lp:~ralsina/ubuntuone-control-panel/installer-option on linux? [19:05] ralsina, yep.... [19:05] gatox: thanks! [19:08] https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/update-from-trunk/+merge/98483 [19:09] ralsina, i'm getting this: http://paste.ubuntu.com/892625/ [19:10] gatox: thanks! === yofel_ is now known as yofel [19:10] mandel needs to actually read the review comments though :P [19:13] gatox: another one please? [19:14] dobey: which specific one? [19:14] ralsina: https://code.launchpad.net/~mandel/ubuntu-sso-client/ssl-checkbox/+merge/97619 [19:14] i reviewed it on friday even :P [19:15] and it's the branch that's blocking his other one from landing [19:15] dobey: that branch is not going into release [19:15] dobey: it's a nixed feature (or at least a delayed one) [19:15] dobey: turns out noone uses ssl-to-proxy connections [19:16] well, few people do i guess [19:16] anyway [19:16] ralsina, yes shoot [19:16] ralsina, the same one updated? [19:16] update the related proposals to reflect that then? :) [19:17] gatox: yes [19:17] dobey: talk to the mandel :-) [19:17] * ralsina is jut the manager [19:18] * dobey delegates management to his manager [19:18] ralsina, http://paste.ubuntu.com/892645/ [19:19] gatox: thaks, and sorry I use you as a buildbot ;-) [19:19] gatox: but the tests are broken here [19:19] ralsina, jejej no problem..... is just 2 commands [19:20] gatox: I can't believe I have not added a bit of lint yet :-) [19:22] :P [19:22] oh, fun,tests are full of magic numbers [19:23] And I don't mean the band "The Magic Numbers" [19:26] gatox: uno mas y no jodemos mas! (a menos que fallen los tests) [19:26] ralsina, jejeje ok [19:28] ralsina, bad news: http://paste.ubuntu.com/892669/ (and it seems you left a print somewhere in the code) [19:28] gatox: it's moving in the right direction! ;-) [19:29] jeje yes [19:34] gatox: again! [19:35] on it [19:39] ralsina, tihs was kind of aggresive. http://paste.ubuntu.com/892692/ :P [19:41] it's working! it's working..... [19:41] gatox: oops [19:41] gatox: forgot the self. :-/ [19:42] ralsina, happens :P..... it seems that the qtreactor thing blocking the backend is working.... i should test it now with py2exe [19:42] gatox: wooohoo! [19:42] briancurtin: we may build a release today yet :-) [19:42] briancurtin: for QA purposes only ;-) [19:42] ralsina, at least from sources...... that was failing before, it's working now [19:43] i'm putting together the autostart thing right now, adding in the tests [19:43] briancurtin: awesome, I have almost finished adding the --installer page [19:43] gatox: one more! [19:44] ralsina, running [19:44] briancurtin: another missing bit to migrate to u1cp is "uninstall when the user clicks on cancel on the license" but that one can wait until tomorrow I think [19:45] ralsina: ok [19:45] cancel on what license? [19:46] ralsina, http://paste.ubuntu.com/892709/ [19:46] dobey: on windows, we show the gpl on installation [19:46] gatox: argh [19:46] only the gpl? [19:46] dobey: yes [19:47] also, the gpl doesn't really say anything about use. it's all about distribution [19:47] so uninstall is probably the "wrong" action at that point [19:47] as is requiring a user to click "accept" to the gpl [19:47] dobey: yes, it's about accepting it before you copy the software into your system. If you don't accept it, it doesn't install [19:47] dobey: platform expectations, also known as "all my friends liked jumping off bridges" [19:48] but all of our code isn't GPL :) [19:48] dobey: please don't make my life harder today. We can do this tomorrow after I have a nice rest ;-) [19:48] hehe [19:48] :) [19:49] oh [19:49] speaking of making life harder. legal agreed we should add the exception for openssl right/ [19:49] dobey: awesome [19:49] so, for next release [19:49] ok [19:53] https://code.launchpad.net/~dobey/ubuntuone-installer/update-from-trunk/+merge/98497 [19:53] gatox: one moreplese [19:53] https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/update-from-trunk/+merge/98483 [19:53] dobey: got it! [19:53] running.... [19:53] ^^ can i get some reviews on those two? [19:53] dobey: yep [19:53] nessita donde es? [19:54] uh, python crashes on "test_animation_is_active" in u1cp :/ [19:54] dobey: teaching [19:54] briancurtin: on windows? [19:54] ralsina: yeah [19:54] briancurtin: tests on windows seem to have devolved lately [19:54] bah [19:54] dobey: trunk is open already, right? [19:54] ralsina, congrats! \o/ just a minor lint issue: [19:54] ubuntuone/controlpanel/gui/qt/wizard.py: [19:54] 332: [W0201, UbuntuOneWizard.check_settings] Attribute '_next_id' defined outside __init__ [19:54] gatox: hey, close :-) [19:55] ralsina: trunk of what? [19:55] dobey: sso [19:55] ralsina: well, my opinion is that trunk is always open; and we can cherry pick :) [19:55] ralsina: i don't know if nessita did an sso release or not [19:55] dobey: cool, just wanted not to bother [19:56] dobey: +1 on -installer since it's exactly like trunk [19:56] Bugs targeted: 30 New, 2 Triaged [19:56] on sso-client [19:57] * dobey wonders why that says 10 new, 2 triaged [20:02] gatox: hopefully last one? [20:03] running tests [20:03] dobey: +1 on rb [20:03] ralsina: gracias [20:03] ralsina, everything perfect [20:04] gatox: awesome, proposing [20:08] ralsina, yes! it's working [20:08] ralsina, the reactor thing [20:09] gatox: awesome! [20:09] ralsina, let me check if this requires test..... and i'll propose [20:10] So, the delta between linux and windows seems to have been about 1.5 developer-week. That's not so much. [20:10] as in, it took 10 developer-days to get windows working again [20:10] we just have to spread that across the cycle [20:11] gatox: if, after you propose, you could review https://code.launchpad.net/~ralsina/ubuntuone-control-panel/installer-option/+merge/98503 and its prerequisite branch, it would be great [20:11] ralsina, yes! no problem [20:14] dobey: your -installer branch is bouncing horribly [20:15] yeah i know why [20:15] dobey: ack [20:22] wtf tarmac [20:23] how did this branch land [20:24] also, how did i not get these lint errors yesterday [20:25] dobey: good questions [20:25] guess i have to do another branch to get in the release [20:25] I am going to take a break [20:26] briancurtin: before you leave, if you have that branch ready, please give me a report [20:26] I will do (very late tonight) a patchwork binary with all the branches we fixed today [20:26] so QA has something to start testing tomorrow [20:26] ralsina: i will. i'm currently fudging OpenKey in the tests so it will work [20:26] briancurtin: awesome [20:29] gatox, dobey, briancurtin: And I am gone. Review requests, on the mail please, I will do some tonight. [20:29] ralsina: have a good evening [20:30] ralsina, ok.... i'll send you mine in a while.... and review yours [20:32] briancurtin, thisfred: one of you care to do a quick/trivial review on https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/fix-lint/+merge/98511 ? [20:34] dobey: +1 [20:35] thanks [20:39] ralsina, here is the branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/backend-problem/+merge/98513 (i'm sending you an email with this too) [20:40] thanks gatox! [20:40] ralsina, reviewing yours and nessita's now [20:46] gatox: looks like you introduced some controlpanel module there! [20:46] ralsina, really?!..... removing that [20:46] gatox: line 122 o the diff [20:48] ralsina, i've approved your branch and nessita's..... not globally approved [20:48] gatox: golbally apprve nessita's [20:48] argh, can't type anymore :-) [20:48] mine can wait for another review [20:49] ralsina, ohh.. i confuse the modules..... fixing that [20:49] gatox: something tells me you did not IRL this ;-) [20:49] ralsina, yes..... and it works! [20:50] but if the controlpanel is in the pythonpath is going to work [20:50] gatox: right, fails for me on windows because I have no controlpanel on PYTHONPATH [20:50] because those modules are the same [20:52] gatox: docstring on qt/main/__init__ says "gtk" [20:53] gatox: and why remove all of tests_main ? [20:54] ralsina, fixed..... and added the missing tests to the new folder [20:54] gatox: ack, will rereview [20:54] ralsina, i moved the test_main inside main folder [20:54] and added some more [20:56] ralsina, eod for me..... but let me know if there is something that i need to fix in the branch i'll do it [20:56] gatox: cool, good work, BTW [20:56] ralsina, thanks :D [21:02] briancurtin, thisfred: same changes again, but for stable-3-0, if you can do a quick review please https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/fix-lint-stable-3-0/+merge/98517 ? [21:02] on it [21:02] dobey: looking [21:03] +1 [21:03] gatox: congrats [21:03] thanks [21:03] ralsina, :D [21:03] gatox: it works and everything [21:03] dobey: +1 [21:04] and bye all! [21:04] ralsina, awesome.... the working-thing part was important! :P [21:04] ralsina, bye! [21:04] enjoy [22:06] hello [22:06] hey [22:07] briancurtin: hola! [22:07] ralsina: were you able to do my review? [22:10] nessita, i think ralsina is not here [22:10] gatox: holas! bu, I asked him to do a review for me [22:10] gatox: were you able to review the one I asked before leaving? [22:10] yes! [22:11] nessita, i review a branch of yours.... i don't know if that is what you mean [22:11] that one [22:11] nessita, i propose the branch that fix the reactor thing also [22:11] gatox: yes, thank you for the review. I will review and land the reactor one tomorrow, so it does enters this (ubuntu) release [22:12] nessita, ok, great [22:15] alright, i am off. got installer and rb-u1 uploaded to ubuntu at least. and so much for doing any yard work tonight it seems. :-/ [22:15] later all [22:15] dobey, bye [22:17] gatox: you still working? [22:17] briancurtin: you up for a very trivial review? https://code.launchpad.net/~nataliabidart/ubuntuone-windows-installer/stable-3-0-update-2.99.91/+merge/98445 [22:18] nessita: looking now [22:18] nessita, npo.... just hanging around :P [22:18] gatox: ja [22:18] nessita: oh that was easy :) [22:18] briancurtin: ;-) [22:38] i'm out of here, see everyone tomorrow === gatox is now known as gatox_tv_brb40 [23:18] * nessita needs reviews [23:21] ralsina: hey!!! [23:21] ralsina: you around? === gatox_tv_brb40 is now known as gatox [23:39] ralsina: ok, sent you email with review request, as soon as those are done, will move forward creating the tarballs [23:39] now, to have dinner [23:39] bye all!