/srv/irclogs.ubuntu.com/2011/06/23/#ubuntuone.txt

=== spm` is now known as spm
karnialecu: I'll be happy if you would take a look at this for me lp:~karni/+junk/ubuntu-sso-java-api and let me know what you think/comments. It's very fresh. The API is pretty straight forward, but I'll make a sample android app anyway. As far as SSO is concerned, it'll be an Android library using that ubuntu-sso-java-api.01:18
karniCardinalFang: Whenever you have time, I'll be happy to hear your thoughts on lp:~karni/+junk/ubuntu-sso-java-api01:18
CardinalFangkarni, I'll look tonight or after a sleep.01:19
karniCardinalFang: Awesome. No rush.01:19
karniMan time passes by fast. And MAN it's hot in here!01:20
* karni opens the window winder01:20
alecukarni, looks very nice.01:25
alecukarni, one comment regarding pingUbuntuOne01:25
alecuperhaps it should be moved out of the SingleSignOnAPI01:26
alecuI think it should reside in a class that inherits from UbuntuSingleSignOnAPI, in a com.ubuntuone package01:27
alecusince it's very specific to our use case.01:27
karnialecu: yes, I will move it out definitely01:27
karnialecu: and it uses HMAC singer, whilst SSO uses Plaintext singer01:27
alecusigner, right.01:27
karnialecu: btw thanks :)01:27
karnihaha01:27
karnisigner :)01:27
karniat 2:30 AM I feel like singing! ;)01:28
alecukarni, I've only looked at the SingleSignOnAPI interface, though :-)01:28
alecuhahahaa01:28
karnialecu: haha :) cool. take your time, I just wanted to update you I'm making progress with it.01:28
alecukarni, another thing: perhaps pingUbuntuOne should be called silently, by the validateEmail in the UbuntuOne class01:29
karnialecu: upon success. good idea :)01:29
alecukarni, so it gets called automatically and the user of the library does not need to remember to do it.01:29
alecuyes.01:29
karnialecu: however, it may be the case the email is properly validated, but the ping fails (say, U1 is down ;) )01:29
karnialecu: what then? should we throw?01:30
alecukarni, good point.01:30
karnialecu: we have to be careful. I wanted to include the ping to make our life easier, but, like you notied, it's not integral part of SSO itself.01:30
alecukarni, perhaps we should ask nessita tomorrow for the details of it, to see how the ubuntu-sso-client does it.01:30
karnialecu: sure. I referenced ubuntu-sso-client code quite a few times, but I don't recall when/how ping details.01:31
karnialecu: validate_email() does not call ping directly01:31
karniin ubuntu-sso-client01:32
karniprobably somewhere higher01:32
karnihigher as "not in accounts.py" :D01:32
alecuok01:32
karnialecu: it pings upon successful login ( credentials.py : _login_success_cb() )01:34
alecuoh, right!01:34
alecukarni, you should definitely ask nessita about that, because she'll remember the details01:35
karnialecu: ok :)01:35
alecukarni, I think it had to do with updating the sso pinged credentials on every login... or something. but it makes sense to do it there.01:35
karnialecu: yeah, I'll definitely ask her to be sure01:36
karniI don't know the details.01:36
karnialecu: We're still caching SSO tokens in app servers, aren't we?01:36
karnialecu: I believe this is what we want to fix (avoid) in the future.01:36
karnialecu: But that doesn't concern us (us playing with Android apps)01:37
alecukarni, we've been doing it at least for the past six months, and we'll probably be doing it in the foreseeable future.01:37
karniright01:37
=== DanRabbit is now known as DanRabbit|spaces
=== DanRabbit|spaces is now known as DanRabbit
=== lifeless_ is now known as lifeless
tntchey guys. I'm having an issue with playlists on ubuntuone music for android. Every time I tap the playlist button, I get an error. "A network error occurred. Try again later."03:49
tntceverything else works. I can view my music library and stuff, and play music03:49
tntcany thoughts on what could be causing this?03:50
karnitntc: Hi!05:16
karnitntc: So, we're having problems with the database that backed this functionality.05:16
karnitntc: The programmer who was looking into it left just a while ago05:16
karnitntc: But I'll ask him tomorrow if there's anything new on the subject.05:17
karnitntc: Sorry for any inconveniance you have due to this issue!05:17
duanedesignmorning all09:27
JohnTeddyIs this a beta program?09:30
duanedesignJohnTeddy: some clients are, like the windows client. The Linux client is not. afaik.09:33
* duanedesign wonders if mr. rye is around?10:00
ryeduanedesign, yes, today i am around10:00
JohnTeddyduanedesign: I mean server side.10:00
JohnTeddyI looked on the status page, there are service disruptions and service outages.10:01
duanedesignrye: hope all is well today. Trying to track the possible reason for an error. Thought I might ask if you had seen a common reason for this. http://ubuntuforums.org/showthread.php?t=178145110:02
duanedesignJohnTeddy: i read their was a problem with a database that backed a particular function causing a temporary outage.10:03
duanedesignbrb10:29
* mandel getting some coffee 11:19
* mandel back11:30
* duanedesign waves at mandel 11:39
mandelduanedesign: hello :)11:40
* mandel walking dog12:22
* fagan break 12:26
nessitahello everyone!12:29
faganyo nessita12:29
nessitafagan: you're awake! :-P12:30
fagannessita: yeah for hours :)12:30
nessitalol12:30
facundobatistaHola nessita12:32
nessitahola facundobatista12:32
duanedesignrye: /512:51
duanedesignoops12:51
ryeduanedesign, so, i downgraded one of my system to natty default/updates and my evolution is working properly. I wonder whether the user is using nightlies12:53
* mandel early lunch13:06
ralsinahola13:18
* ralsina is officially an old man. He fell down on the street last night13:19
faganawh ralsina you ok?13:21
ralsinaAnyway, I didn't break my hip, but I twisted my knee. So I have a pressure bandage, taking some magic pain pill and can't walk. Yay.13:22
ralsinafagan: worse case, I'll use a peg leg and live a pirate's life. Best case it will stop hurting in a few hours.13:22
ralsinaBTW: jumping up to my 1st floor home on one leg... not as easy as it sounds13:23
faganpirate legs are cool13:23
ralsinafagan: but they make you use those rubber thingies at the bottom soyou don't scratch the floor and kills all the tap dance effect13:23
faganno fun :/13:24
duanedesignrye: hmmm. Ok, thank you! All the bug reports I can find with this error are Fix Released...13:27
ralsinaok, back to coding what I should have coded last night :-(13:28
nessitacan I have a couple of reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/greet-and-report-quota/+merge/65598 ?13:30
* thisfred is on it13:31
nessitaralsina: also, question. Is there any easy way to do transparent overlays in QT? (if you need to google, don't, I'm asking to the top of your head)13:31
nessitathisfred: thanks!13:31
thisfredwow, we can handle YiBs! :)13:32
ralsinanessita: yes there is, but I can't give you a recipe. Look for setMask in the docs13:36
ralsinanessita: reviewing...13:37
ralsinanessita: is it for the spinner thingy? I need one too for the wizard13:37
ralsinanessita: OTOH I may not have understood what you want13:38
nessitaralsina: if you look lisete's windows share, go to:13:39
nessitaU1_design_work / Windows / client / PSD :13:39
nessitaand see:13:39
nessitawin_client_008_please_wait.png13:39
nessitawin_client_008_please_wait002.png13:39
nessitaralsina: she proposed those, and before agreeing to that, I wanted to see if there is a way to implement it13:40
nessitaralsina: I'll google13:40
nessitaand see if I can create a widget13:40
ralsinalooks like I don't have lisette's only Ivanka and Claire's13:40
ralsinalisette: ping13:40
nessitaralsina: did you subscribed the share?13:40
nessitaralsina: go to the linux control panel and subscribe it13:40
ralsinanessita: I'll check13:40
nessita(maybe you have to accept it first?_13:41
nessita)13:41
lisetteralsina: hi13:41
ralsinalisette, I don't seem to have your share on Ubuntu One, could you re-invite me?13:41
ralsinanessita: nope, no email about it either13:41
* ralsina may have lost it13:42
lisetteralsina: sure, i will uninvite and reinvite you :)13:42
ralsinalisette: cool, thx13:42
ralsinalisette: got it, thanks!13:43
lisetteralsina: done13:43
facundobatistanessita, dobey, I fixed the problem of ubuntuone-client in Oneiric, the one from bug #79897913:44
ubot4Launchpad bug 798979 in ubuntuone-client (Ubuntu Oneiric) (and 2 other projects) "ubuntuone-syncdaemon crashed with KeyError in add_watch(): '/home/caravena/Ubuntu One' (affects: 7) (dups: 7) (heat: 68)" [High,Triaged] https://launchpad.net/bugs/79897913:44
nessitafacundobatista: yey?13:45
ralsinanessita: the second is really easy13:45
nessitaralsina: from my pov both are the same13:45
facundobatistanessita, dobey, note that after the quick fix it now fails with NoAccessToken: CredentialsNotFound13:45
ralsinanessita: basically, disable the whole tab, add a child widget to the tab (the square), place it in the middle, that's it13:46
ralsinanessita: the first one is the same, but don't disable, make the child widget cover the whole tab13:46
nessitaralsina: ack, I'll try. I'll build the widget so you can later reuse13:46
ralsinaThe child widget can be just a plain QLabel with a QMovie in it13:47
nessitaralsina: so we don't duplicate work13:47
ralsinanessita: here's a C++ example http://www.developer.nokia.com/Community/Wiki/CS001434_-_Creating_a_loading_animation_with_GIF,_QMovie,_and_QLabel13:47
dobeyfacundobatista: that's fine. i'm working on a fix for that :)13:48
ralsinanessita: but that seems quite complicated for what it does, though13:49
ralsinastandup in 10' people13:50
* mandel back13:51
ralsinanessita: +1 on greet-and-report-quota13:51
nessitathanks!13:52
facundobatistadobey, great, thanks13:53
faganstandup in  313:57
alecuhello!13:59
faganyo alecu13:59
faganmoi13:59
facundobatistaHola alecu14:00
nessitame14:01
faganmandel, thisfred, dobey, ralsina, alecu standup14:01
ralsiname14:01
mandelme14:02
nessitaalecu, dobey?14:02
nessitathisfred: ?14:02
dobeyme14:02
thisfredme14:02
alecume14:02
faganok go me14:02
faganWORK IN PROGRESS14:02
faganrst port of the FAQ pages14:02
faganBlocked14:02
fagan* nope14:02
fagannessita: go14:02
nessitaDONE: bug #800683, failed attempts to have syncdaemon running on win XP due to SSO failures, meetings14:02
nessitaTODO: Review Lisette's proposal for "loading" messages, try to have XP working with SSO? nail some other UI bug?14:02
nessitaBLOCKED: freaking out14:02
nessitaNEXT: ralsina14:02
ubot4Launchpad bug 800683 in ubuntuone-control-panel "Storage use and user greetins should be shown in the QT control panel (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/80068314:02
ralsinaDONE: Windows catchup call, Windows+design call, bunch'o'reviews, fixed bug #800381 (needs 2 reviews), fixed #800715 (needs 2 windows reviews), did most of bug #800359 (SSO on wizard)14:03
ralsinaTODO: Finish the SSO on wizard integration, start syncdaemon on wizard stuff, hopefully get that mostly done, then whatever I can fit in.14:03
ralsinaBLOCKED: not yet, but if syncdaemon doesn't work on windows today I will be14:03
ralsinaNEXT: mandel14:03
ubot4Launchpad bug 800381 in ubuntuone-windows-installer "Add "sync now or later" page to the installer (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/80038114:03
ubot4Launchpad bug 800359 in ubuntuone-windows-installer "Add sign in page to the installer (affects: 1) (heat: 6)" [Undecided,Triaged] https://launchpad.net/bugs/80035914:03
mandelDONE: Partially complited all tests for window_sdtool. Looked at my windows XP vm and updated the setup env to later test txnamedpipes on it.14:03
mandelTODO: Complete the test and provide them as an example for nessite to use sdtool.14:03
mandelBLOCKED: no14:03
mandelCOMMENT: Will be here longer than usually so I can help14:03
mandeldobey: go14:03
dobeyλ DONE: update for amz on server, more work on bug #800294, closed old sso milestones14:03
ubot4Launchpad bug 800294 in ubuntu-sso-client (Ubuntu) (and 2 other projects) "Support new fdo secrets API property names (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/80029414:03
=== mbarnett` is now known as mbarnett
dobeyλ TODO: finish bug #800294, fix more stuff, chase people14:03
dobeyλ BLCK: None.14:03
dobeythisfred: your turn14:04
dobeyla la la14:04
thisfredDONE: Rethinking the launcher and doing some experiments with counters/quicklists TODO: turn that thinking into bugs and branches BLOCKED: nope NEXT: alecu14:05
alecuDONE: calls and more calls, worked on bug #800669, found bug #800887, a few reviews14:05
alecuTODO: calls, debug CredentialsManagementTool14:05
alecuBLOCKED: no kinder still; but less planned interruptions than yesterday14:05
alecuNEXT!14:05
thisfredsry14:05
ubot4Launchpad bug 800669 in ubuntu-sso-client "InsecureJelly exception thrown when SD requests credentials (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80066914:05
ubot4Launchpad bug 800887 in ubuntu-sso-client "UbuntuSSOClient fails when starting the SSO process (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80088714:05
alecuthisfred, "doing some experiments with counters/quicklists" <- cooool!14:05
thisfredalecu: yeah, for new udfs I think counters may make more sense than the messaging menu for instance14:06
ralsinathisfred: nice!14:06
thisfredand the quicklists for all subscribed folders, I'm thinking14:06
nessitaany closing comments?14:07
fagannope14:07
thisfredBe excellent to each other!14:08
dobeylooking for closure?14:08
faganok dont be nice be excellent14:08
ralsinadobey, thisfred we need to talk about keeping u1 working on oniric, could you guys give me a status update on that by email?14:08
faganralsina: its broken on my machine in 11.1014:08
thisfredralsina: uh ok: I have not been able to install O yet14:08
thisfredwill try again on the laptop14:09
ralsinaI mean talk between you then send me a short list of things that are completely broken. Ok, thisfred, then just dobey I guess ;-)14:09
ralsinaor if you can get it installed today, send me the report late today14:09
dobeyralsina: uh, after i fix these tests again in my branch, and it gets approved/landed, we should be good to go if facundobatista's fix has already landed14:09
thisfredsure14:09
ralsinadobey: cool then, it should be a very short report14:10
ralsinadobey: also we need to squeeze "port everything that stays gtk to gtk3" somewhere in the next 4 or 5 months :-(14:10
ralsinaand from pygtk to the (currently semibroken, right) gtk-via-introspection-thing14:11
dobeywhich is mostly not really an issue at this point14:11
dobeycouple of minor things to change, but we should be good for the most part14:11
=== m_conley_away is now known as m_conley
ralsinadobey: cool, not bad there then. Hopefully things won't diverge too quickly14:12
dobeythe only big thing with that is the banshee/rhythmbox issue14:13
dobeybecause banshee is not gtk3 yet, and rbox is14:13
nessitaralsina, mandel, alecu, Chipaca: mumble in 2 minutes?14:13
thisfredralsina: 4 or 5 months? I thought we had like 3 more weeks or so :)14:13
ralsinanessita: ack14:13
Chipacanessita: no can do, got the web thing in 2 minutes14:13
ralsinathisfred: well... that's one way to look at it. gtk2 is leaving the CD n the next cycle14:13
mandelnessita: at 15 pass, sure14:13
dobeythisfred: well, more like 2 months14:13
nessitaChipaca: ah, ok. How long then? we can reschedule?14:13
ChipacaI've got the checkpoint on my calendar at :3014:14
Chipacathat's a bit tight, i'd rather :4514:14
ralsinathisfred: OTOH the nautilus plugin probably will have to be ported in three weeks yes14:14
nessitaChipaca: ok, :45 it is14:14
dobeyand there's that whole *other* issue to deal with14:14
nessitamandel, alecu, ralsina: mumble at :45  (in 30 minutes)14:14
dobeyralsina: the nautilus plug-in already works, it just gets installed to the wrong directory, i think14:14
thisfreddobey: you mean the shiv? ;)14:14
ralsinanessita: ack^214:14
nessita:-)14:15
dobeythisfred: yes14:15
alecuack14:15
mandelnessita: sure14:15
dobeyshivs and shanks14:15
ralsinadobey: well, that is good news14:15
* ralsina should get oneiric installed too :-(14:15
facundobatistanessita, dobey, https://code.launchpad.net/~facundo/ubuntuone-client/avoid-double-watch/+merge/65661 (feel free to review)14:16
dobeyfacundobatista: interesting diff. :) are there no unit tests for that block of code?14:19
facundobatistadobey, it's just a removal of a duplicated line... but the duplicated line is not even in that file14:19
facundobatista(this is just dirtiness left ages ago, when VM took the responsibility on setting up 'root')14:20
dobeyright14:20
alecumandel, any ideas on this? bug #80114514:20
ubot4Launchpad bug 801145 in ubuntu-sso-client "I won't stop right on windows/txnp (affects: 1) (heat: 6)" [Critical,Confirmed] https://launchpad.net/bugs/80114514:20
dobeyfacundobatista: approved14:21
mandelalecu: let me take a look14:21
dobeyralsina: ^^ one branch down for oneiric :)14:22
ralsinadobey: yay!14:22
facundobatistadobey, thanks14:23
mandelalecu: the bug is in line 56 of bin/windows-ubuntu-sso it is telling the reactor to stop when it should be telling that to the QApplication and not the reactor14:24
mandelalecu: shutdown function here: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntu-sso-client/trunk/view/head:/bin/windows-ubuntu-sso-login14:24
alecumandel, nice, thanks14:24
mandelalecu: np14:24
ralsinamandel: I am integrating the SSO into the wizard and it is working, but I have the strange feeling that I am using it completely wrong. Could you take a quick look at https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800359 ?14:27
mandelralsina: sure,, on it14:28
ralsinamandel, basically http://bazaar.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800359/view/head:/ubuntuone_installer/gui/qt/gui.py14:29
mandelralsina: looks good to me, you are passing the controller to the ui so that it hooks it up, that is the idea14:31
ralsinamandel: ok14:32
ralsinamandel: I am not really happy because I am putting lots of logic in something that depends on Qt but well, it's much harder the other way around.14:32
mandelralsina: what do you mean?14:32
ralsinaoh, wait, all the controller stuff in sso is qt-specfic! Nevermind :-)14:33
facundobatistadobey, "There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions."14:35
facundobatistadobey, and that's only because nessita gave me an approve after you approved the MP14:35
facundobatistadobey, I'm approving the MP again... but it seems that Otto is wrong there14:36
* nessita wanted to help14:37
mandelralsina: that is the entire point of the controller, they have no notion of the framework, they just want to add cb to the signals :)14:37
ralsinamandel: cool, hacking it some more then ;-)14:38
dobeyfacundobatista: no i think what happened was that the empty commit you added for --fixes showed up after i'd approved the branch (so the old revision was approved, and not the new), and launchpad was being a bit slow, and i wasn't expecting another revision to show up :)14:39
dobeynessita: had nothing to do with your +114:39
nessitaok14:39
facundobatistadobey, nop, my empty commit is 22 minutes ago, and your approve is 18 minutes ago14:40
dobeyfacundobatista: yes, but that has nothing to do with when launchpad actually completed the rescanning14:40
facundobatistadobey, mmm... and Otto checks when the scanning was done, and not when the commit was done?14:41
dobeyfacundobatista: if the lp page had no idea of the second commit at the time, then when i approved, it couldn't have chose the later revno, but only the latest one it knew at the time.14:41
dobeyfacundobatista: tarmac just tries to merge the revision that is approved in launchpad. if that revision is not the tip of that branch, it complains14:42
facundobatistadobey, ok14:42
dobeyfacundobatista: and launchpad can be slow at times, and it doesn't help when unexpected revisions pop up :)14:43
dobeyHUZZAH!14:43
dobeyPASSED (skips=1, successes=494)14:43
dobeyfacundobatista: also, there are two rescan processes. the rescan of the branch for the bracnh data page in lp, and the rescan/regeneration of the diff for the merge proposal. and the merge proposal one happens after the branch one, and is the one we have to wait for, so that the revisions are all up to date there.14:46
dobeynessita: can you re-review https://code.launchpad.net/~dobey/ubuntu-sso-client/new-kr-props/+merge/65419 please? :)14:47
alecunessita, mandel, ralsina: I see that Chipaca is already on our mumble channel14:47
alecushall we?14:47
adorilsonrye: hi14:47
ralsinaok, let's!14:48
Chipacait's :45 :)14:48
Chipacabah, it was when i mvoed in14:48
adorilsoncan you help me with this  https://wiki.ubuntu.com/UbuntuOne/RunClientFromTrunk ?14:48
ryeadorilson, hi14:48
ryeadorilson, hm, this is first time i see this page but make install will most likely not work14:49
ryeadorilson, it can be ran w/o installation, that's what chicharra team does, actually14:49
dobeyhrmm, yeah, i'd avoid the make install bit there14:50
ryeThanks to Roman??? Hmmmm14:50
adorilsonrye: someone told me you had written this page. did not you?14:51
dobeymaybe he was talking about Pilot14:51
dobeyi don't think rye wrote that page :)14:51
dobeyat least, i don't think he's crazy enough yet to be thanking himself on his own wiki pages14:52
thisfredhmm, update-manager fails wil an OSError (No such file or directory) ...14:52
ryeadorilson, i am listed there as "Thank you" but i don't recall having the discussion of running the client from trunk, other than that it is really simple - bzr fetch for protocol and client, build client specifying the protocol location and then run the client specifying correct PYTHONPATH14:53
thisfredand now, my laptop is AGAIN in limbo between natty and oneiric14:54
ryefacundobatista, by the way, i have now a test client (natty, latest updates, no nightlies) that cannot upload flv file because it claims ubuntuone.SyncDaemon.StorageClient - TRACE - start - processMessage: id: 13 type: ERROR error {   type: UPLOAD_CORRUPT   comment: "Deflated size does not match hint." }14:54
facundobatistarye, size?14:55
ryefacundobatista, 96380073 bytes, BUT, this is completely different to my current issue of 180Mb file which keeps being uploaded on 100% and hangs, this just fails with upload corrupt and requires client restart to try uploading again14:59
facundobatistarye, and it always happen?14:59
facundobatistaverterok, ^ we did have an issue with persistent corruptness in resumable, is it fixed now?15:00
adorilsonrye, see this http://paste.ubuntu.com/631200/ please15:00
verterokfacundobatista: yes, we have...and isn't fixed yet15:00
verterokrye: could you add the node details to bug #77880515:01
ubot4verterok: Bug 778805 on http://launchpad.net/bugs/778805 is private15:01
verterok?15:01
facundobatistaverterok, ok, thanks15:01
verterokrye: upload_id, share_id, node_id and hash please :)15:01
=== karni_ is now known as karni
ryeverterok, added15:04
verterokrye: thanks!15:04
ryeadorilson, apt-get install gobject-introspection15:05
dobeyhmm15:07
* fagan break for an hour15:15
faganjoshuahoover: Looks like ill take a long day to get this done. I have a few done now but there are good few pages15:16
joshuahooverfagan: ok, thanks for the update...can you make sure you commit what you have done to your branch by eod (for you)?15:16
faganjoshuahoover: sure will do, ill hang around till around 9 UTC id say.15:17
joshuahooverthanks :)15:17
dobeyalecu: fix the lint warnings before setting back to approved!15:17
adorilsonrye: installing. but I need to tell a secret: I'm on Fedora 1515:19
dobeyadorilson: it may be prudent to make RPMs in proper order :)15:20
ryeadorilson, another thing is that you will need to have ubuntu-sso package too since syncdaemon depends on it to get the credentials15:21
dobeywell, will need a lot of things15:21
adorilsondobey: I searched, but I dont found ubuntuone's RPMs15:21
ryeadorilson, or, if you like hack-driven approach to make it work first and think how make it better later you may want to use the command line script to get the credentials from Ubuntu SSO and put them to oauth= config variable15:22
dobeyadorilson: right. keyword in my statement was "make" :)15:22
adorilsondobey: sure. but I dont have a minimal idea how to make RPMs15:25
adorilsonrye: gobject-introspection is already installed.15:26
adorilsonrye: any ideia?15:27
ryeadorilson, do you have pkg-config? try pkg-config  --print-errors "gobject-introspection-1.0" --modversion15:29
adorilsonrye15:31
adorilson$ pkg-config  --print-errors "gobject-introspection-1.0" --modversion15:31
adorilsonPackage gobject-introspection-1.0 was not found in the pkg-config search path.15:31
adorilsonPerhaps you should add the directory containing `gobject-introspection-1.0.pc'15:31
adorilsonto the PKG_CONFIG_PATH environment variable15:31
adorilsonNo package 'gobject-introspection-1.0' found15:31
alecudobey, great, thanks.15:32
verterokrye: one way to workaround the upload corrupt bug, is to move the file out of u1, wait for the client to delete the file on the server, and move it back15:32
ryeverterok, this file has not been uploaded to the server for this account15:32
verterokrye: the file was created, but not uploaded15:33
verterokrye: the "node" exists in the server, but with no content15:33
adorilsonrye: gobject-introspection version is 0.10.815:33
verterokrye: moving the file out of u1 will trigger a unlink of the node, so the stalled upload will be cleanedup15:33
dobeyadorilson: you need the -devel packages of things15:33
adorilsondobey: installing gobject-introspection-devel...15:35
ryeverterok, RROR error {   type: UPLOAD_CORRUPT   comment: "Deflated size does not match hint."15:37
ryeverterok, the file is already uploaded under a different user15:38
dobeyhrmm15:38
verterokrye: if the same file is already uploaded it shouldn't upload it again15:38
verterokrye: magic uploads!15:38
verterokrye: so, it might be in a different shard15:39
ryeverterok, the original file was uploaded this february, before magic15:39
verterokrye: ah, ok15:39
* alecu will be afk for 30'15:39
verterokrye: try moving it out of u1, wait for the client to delete/unlink it, and move it back15:40
ryeverterok, for the original account?15:41
verterokrye: from the account you'r getting the upload corrupt error15:43
adorilsonrye, dobey: what is lndir ?15:43
ryeverterok, already done that, unlink sent to the server, sd got IDLE, moved the file back in... UPLOAD_CORRUPT15:44
verterokrye: it tried to resume the upload?15:45
verterokrye: look for BEGIN_UPLOAD message in the logs, offset should be 015:45
dobeyadorilson: install xutils-devel15:45
ryeverterok, BEGIN_CONTENT begin_content {   offset: 015:46
dobeyalthough we can probably remove that at this point15:46
verterokrye: ok, so this isn't the same bug...something else is going on15:46
verterokfacundobatista: ^ isn't resuming the upload...so isn't the stalled resumable uploads issue :/15:47
rodrigo_can someone on natty send me some screenshots of the contacts picker?16:04
ryerodrigo_, contact picker from ubuntuone?16:05
faganjoshuahoover: the links will probably be wrong is that ok16:05
rodrigo_rye, yes16:05
joshuahooverfagan: links to?16:06
faganjoshuahoover: the other pages16:06
mandelalecu: ping16:06
joshuahooverfagan: maybe muffinresearch has ideas on how to handle links (if there is any way to handle it better than just leaving them be wrong for now)16:07
faganjoshuahoover: yeah he would be the one to ask16:07
faganmuffinresearch: ^16:07
faganoh and I just finished the first 2 sections16:07
muffinresearchfagan: do you have examples?16:08
faganmuffinresearch: sec16:08
ryerodrigo_, http://ubuntuone.com/p/10om16:08
faganmuffinresearch: https://wiki.ubuntu.com/UbuntuOne/Tutorials/Setup/Maverick#Already%20have%20an%20Ubuntu%20One%20account16:09
dobeyrye: ugh, contacts picker --16:09
fagantaht kind of thing, should I just leave it or should I guess what the link would be or something16:09
rodrigo_rye, thanks, but don't you have more contacts? It's for showing other people what it does :)16:10
dobeyrodrigo_: i think he was pointing out the disconnect between the search field contents, and that there are any results at all :)16:11
rodrigo_dobey, :)16:11
muffinresearchfagan: the faq itself will be at /help/faq/ - The table of contents as shown on that page should be auto generated by ReST using the directive for that. http://docutils.sourceforge.net/docs/ref/rst/directives.html#table-of-contents16:11
faganmuffinresearch: hmmmm well in the interest of speed ill leave them and sort out all the links tomorrow16:12
faganill give it a shot anyway and if they are wrong you guys can sort it since its a small thing to fix16:13
ryerodrigo_, well, i can create more contacts16:15
rodrigo_rye, please, if it's not too much hassle :)16:15
rodrigo_rye, although I'd prefer to get contacts with photos16:15
rodrigo_dobey, do you have?16:15
dobeyrodrigo_: i have no friends dude :)16:16
rodrigo_heh16:16
rodrigo_no even ennemies with photos in your addressbook? :D16:16
rodrigo_so, anyone else please?16:16
ryerodrigo_, hm, the photo part is tricky, replication is disabled for my shard16:16
dobeyno16:16
facundobatistarye, can you send me the logs of that session? thanks16:16
dobeybut i don't see why it matters16:16
dobeythe bug i'm seeing in that screenshot has nothing to do with photos16:17
rodrigo_rye, don't worry then, I'll try to find someone with contact photos16:17
rodrigo_dobey, it's for showing people designing a contact picker for gnome how ours look like16:17
rodrigo_so yes, I'd prefer photos16:17
rodrigo_I have photos myself for lots of contacts, but u1 doesn't work on oneiric yet16:18
rodrigo_joshuahoover might have it16:18
rodrigo_joshuahoover, do you have photos for contacts that are displayed in the contacts picker in nautilus?16:18
dobeyrodrigo_: so rye wasn't showing you a bug in that screenshot?16:19
ryedobey, nope16:19
dobeyoh, because all i saw was "look at this bug" :)16:19
joshuahooverrodrigo_: let me check16:19
rodrigo_dobey, no, not a bug16:19
joshuahooverrodrigo_: doesn't look like it...hmmm16:20
rodrigo_ok16:20
dobeyrodrigo_: the thing is, almost nobody has photos of contacts in evolution16:21
rodrigo_dobey, I have, they get imported from facebook16:21
rodrigo_not for u1 contacts though16:21
dobeyrodrigo_: the only people that do are people who added photos to demo the fact that photos can be used in evo :)16:21
rodrigo_:)16:21
dobeyrodrigo_: how did you get fb contacts in evolution if not through u1?16:22
rodrigo_joshuahoover, ok, so can you take a screenshot please?16:22
rodrigo_dobey, via pidgin plugin, iirc16:22
dobeyoh, weird16:23
joshuahooverrodrigo_: i don't have any photos or contacts here :(16:23
rodrigo_joshuahoover, I'll replace it with one with photos as soon as u1 works here16:23
joshuahooverrodrigo_: only via the web16:23
rodrigo_joshuahoover, doesn't mind, if you have more than 1 contact, it's enough :)16:23
joshuahooverrodrigo_: ah, ok...let me get that for you then16:23
dobeyrodrigo_: didn't the design mock-up have fake contacts with photos?16:23
rodrigo_joshuahoover, cool, thanks16:23
rodrigo_dobey, the design mockups are different from what we implemented16:24
rodrigo_and they are not public, afaik16:24
rodrigo_aren't they?16:24
dobeyi thought they were, but no idea16:24
dobey*shrug*16:25
dobeyi'm surprised gnome isn't just using the OSX design16:25
rodrigo_https://live.gnome.org/action/subscribe/Design/Apps/Contacts <- this is what's up for now16:25
dobeyalthough, i'm sure they will settle on that, but it will have the gtk3 theme16:25
dobeyyeah i've seen that16:26
dobeybut i never did like os x16:26
rodrigo_me neither :)16:26
dobeywhat happened to nessita?16:28
joshuahooverrodrigo_: http://ubuntuone.com/p/10ow/16:28
nessitadobey: I'm here, why?16:28
dobeynessita: can you re-review https://code.launchpad.net/~dobey/ubuntu-sso-client/new-kr-props/+merge/65419 please?16:29
nessitadobey: sure, in a couple of minutes16:29
rodrigo_joshuahoover, thanks! do you have any problem in making that screenshot more public? that is, adding it to https://live.gnome.org/action/subscribe/Design/Apps/Contacts ?16:29
dobeynessita: you've been silent for the last hour. it is unlike you :)16:29
dobeynessita: thanks16:29
joshuahooverrodrigo_: no, that's fine...just names shown there16:29
nessitadobey: I'm coding like a monkey16:29
rodrigo_joshuahoover, ok!16:29
dobeylunch time, bbiab16:32
adorilsondobey: yum dont found xutils-devel. And I can't found in another place. :-/16:33
thisfredruntime directory /run/udev not writeable. That sounds ungood.16:35
duanedesignverterok:do you know what is the bug number for 'persistent corruptness in resumable'? I have a couple bugs with that issue and have been looking for a bug to dupe them too.16:36
verterokduanedesign: depends on what 'persistent corruptness in resumable' means :), looks like we have 2 different issues16:36
duanedesignverterok: aha16:36
duanedesignverterok: glad you said that. i was thinking we might looking at the conversation16:36
verterokduanedesign: if the client get a BEGIN_CONTENT with offset > 0 and the upload fails with UPLOAD_CORRUPT, it's bug #77880516:37
ubot4verterok: Bug 778805 on http://launchpad.net/bugs/778805 is private16:37
verterokduanedesign: you need TRACE logs to see the BEGIN_CONTENT msg16:37
duanedesignahh. you predicted my next question :)16:39
duanedesignthank you for info16:40
verterokheh, np :)16:40
mandelnessita: running the test on linux and will ask you to re-review16:44
nessitamandel: ack16:44
thisfredok, my laptop is severly screwy now. can anyone with a working oneiric send me the output of dpkg --get-selections ?16:47
alecumandel, pong. sorry, was afk16:52
mandelalecu: did the change in the shutdown function work?16:55
alecumandel, I'm just working on that right now. I'll let you know in 15'16:55
mandelalecu: cool, the idea is that you just get the QApp instance and close it there, and there should be no need to tell the reactor to do it at all16:56
alecumandel, cool, I understood.16:57
mandel:)16:57
nessitaralsina: ping16:59
ralsinanessita: pong16:59
nessitaralsina: in GTK I can connect a signal to a widget and pass extra arguments like this: button.connect('clicked', my_callback, data1, data2, data3, ...) and then, when the button is clicked, my_callback will be called with the follwoing params: button, data1, data2, data3, ...17:00
nessitaralsina: can I do something like that in PyQt? I've tried with QtCore.QObject.connect(button, ... but I can't pass extra args17:01
ralsinano, no extra args17:01
ralsinayou can use a lambda17:01
nessitaralsina: hum... no I can't...17:01
ralsinabutton.clicked.connect(lambda : recipient(bla, foo, bar))17:02
nessitaI'm inside a loop, and I need the extra  params (bla, foo, bar) be those from the loop17:02
nessitaso, I have code that looks like:17:02
nessitafor bla, foo, bar in data:17:02
nessita    QtCore.QObject.connect(button, QtCore.SIGNAL('clicked()'), lambda: my_callback(bla, foo, bar))17:02
nessitathat will not work17:02
nessitabla, foo, bar will always be the last ones from the loop run17:03
ralsinanessita: pass them as arguments to the lambda: lambda foo=foo, bar=bar: recipient(foo, bar)17:03
ralsinathat way, the foo inside the lambda is the name of the arg, which is bound to the current value in the loop17:03
nessitatrying that, but is EXTREMELY DISGUSSING :-)17:04
nessitadisgusting *17:04
ralsinanessita17:04
nessitayes?17:05
ralsinanessita, at least it's not lambda bar=foo, foo=bar: f(foo,bar) ;-)17:05
nessitadoes not seems to work... and I can see why17:05
nessitaah no, wait17:05
ralsinathere are cleaner solutions using signalmapper but it's really verbose17:06
nessitait works17:06
nessitathanks17:06
* ralsina remembers doing this17:06
nessitais uglier than mestre17:06
ralsinaglad to help!17:06
ralsinait's poor man's python closures17:07
nessitadobey: reviewing17:08
alecumandel, so, after calling app.quit it just wont finish17:09
alecumandel, I even tried calling a shutdown function that calls app.quit and then reactor.stop, just in case.17:10
mandelalecu: bummer… that looks like there is a thread being evil somewhere...17:11
alecumandel, absolutely. It must be the threaded reactor thread.17:11
alecumandel, but for now, we'll raise the value of the sso timeout to infinity-117:11
alecumandel, so the sso client will keep running forever and that's it.17:12
alecumandel, we'll fix this properly later.17:12
mandelalecu: sure.. it should not be a major issue for the time being17:13
mandelnessita, alecu: please can you look at https://code.launchpad.net/~mandel/ubuntuone-client/windows_sdtool/+merge/6549317:25
nessitayes17:26
nessitadobey: branch works very good, approved17:29
nessitadobey: I would like alecu's doing the second review17:29
dobeynessita: great, thanks17:29
dobeysure17:29
dobeyalecu: can you review https://code.launchpad.net/~dobey/ubuntu-sso-client/new-kr-props/+merge/65419 please?17:29
alecusure17:29
alecudobey, why do you abbreviate collection as CLXN?17:33
dobeyalecu: PEP817:35
alecumandel, I'll review your megabranch after lunch.17:35
mandelalecu: sure, np17:35
nessitamandel: instead of raise Exception('Not implemented.')17:36
nessitamandel: can you please raise NotImplementedError?17:36
mandelsure17:36
nessitamandel: and class ErrorSignal(Exception): needs a docstring (and then remove the "pass")17:38
mandelnessita: can you write all this in a comment so that we can keep track?17:39
nessitamandel: I could, but adding a needs fixing and waiting for your fixes use to take too much roundtrip17:39
nessitamandel: if I stare at you while you fix this it goes faster :-D17:39
alecudobey, I don't see anything on pep 8 regarding that kind of abbreviations. But I find it confusing that in some constants it's spelled COLLECTION, and on others CLXN17:40
mandelnessita: ok17:40
alecudobey, would you mind changing the CLXN to COLLECTION?17:40
dobeyalecu: line length limit17:41
nessitamandel: let me know when those 2 are fixed and pushed17:42
nessitamandel: should be quick, right?17:42
dobeyalecu: having to do 'FOO = \\\n"some long string"' is lame17:42
dobeyalecu: and there isn't really any raeson for those to be used outside of txsecrets.py anyway, aside from the fact that the tests also use them17:43
mandelnessita: yes, just done it and pushing, I'll ping you when done17:43
nessitathanks!17:43
nessitaI'll have lunch after that17:43
mandelnessita: done, r106117:45
ralsinanessita: ran into some trouble making SSO show the right buttons on the wizard, will take another 60, 90 minutes17:49
nessitaralsina: OH NO! what are we doing now! NOOOOOO :-P17:50
nessitaralsina: ok, let me know :-)17:50
nessitaralsina: need help?17:50
ralsinanessita: sure. I am trying to make it good enough not to have to work on it again tomorrow after design sees it ;-)17:50
nessitagood choice17:50
karninessita: Every time we authenticate the user (and then save oauth tokens, naturally) we add some, name it, host token - "Ubuntu One @ foobar". Is there a way to authenticate a user and reuse such token instead of creating new oauth tokens if the user, say, uninstalls U1 software, and then installs it again?17:50
alecudobey, why did you change the out_signature of SecretServiceMock.CreateCollection ?17:51
karninessita: I'm trying to be precise, but it's a tricky question that blajk just asked.17:51
nessitamandel: approving. When I get a windows env I'll test it IRL, and I'll file bugs if necessary17:51
nessitakarni: nopes17:52
karninessita: Say, we authenticated with "Ubuntu One @ foobar" token once. We remove the software completely. Then we install it and re-authenticate - can we request previously generated oauth tokens or we need to create new once?17:52
mandelnessita: ok, I'm writing the script you asked atm17:52
karninessita: So a new one is neccessary, right?17:52
karninessita: That's what I thought. Thanks!17:52
nessitakarni: you're welcome!17:53
karni:)17:53
alecudobey, the code looks fine otherwise. The only doubt I have is with that out_signature change.17:53
* alecu will have lunch now.17:53
dobeyalecu: one second my firefox went nuts17:53
faganjoshuahoover: did a big push of a few pages17:54
faganjoshuahoover: about halfway there17:54
joshuahooverfagan: cool, thank you!17:54
faganjoshuahoover: you know where the branch is?17:55
joshuahooverfagan: yep17:56
faganjoshuahoover: cool ill do some more then and should be almost done for tomorrow morning17:56
dobeyalecu: hrmm, no idea. changed it back, and pushed17:56
joshuahooverfagan: excellent17:56
faganok changing location brb17:57
fagan]/quit17:57
mandelalecu: this is new: http://paste.ubuntu.com/631291/ is it on purpose?17:59
* mandel walks dog, will me back in 30 min17:59
nessitaok, lunchtime!18:11
nessitaok, no luch yet, firs, merge proposal: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/explore-folder/+merge/6569718:18
nessitacan I have a couple of reviews for? ^ please18:18
nessitanow yes, lunchtime!18:18
* thisfred reviews18:24
dobeyi'm not good at doing reviews. rebuking on the other hand, i am ace at.18:26
alecudobey, there are a few error messages shown while running the tests that are not shown on trunk: http://pastebin.ubuntu.com/631316/18:32
alecudobey, the tests pass fine though.18:33
dobeyalecu: maybe that's why i changed it to v. no idea why those are happening, or why they're not causing failures18:33
dobeyhrmm, though those must be coming from a different call18:35
ralsinamandel: ping18:35
mandelralsina: pong18:35
ralsinamandel: not for now, not for soon, but using buttons inside the pages for controlling the wizard is not really good for integrating it with other code18:36
mandelralsina: I agree, the issue here is the design..18:36
ralsinafor example, the same effect can be achieed (as long as you hae less than three "action" buttons, using the custom buttons of the wizard view18:37
mandelralsina: the problem is the location, not the number of buttons, that I know how to do18:37
mandelralsina: the actual best thing for you to do if it is a problem is to reuse the controllers and just provide the same interface for the UI18:38
ralsinamandel: well, the thing is ... I prefer them to be located in the wizard :-)18:38
ralsinamandel: yes, I will probably do that eventually18:38
mandelralsina: the problem is that certain buttons they placed in the UI cannot be in the wizard, but have to be in the page, remember the wireframes?18:39
ralsinamandel: yes for some, but for example "create account" on the signup page is clearly a "Next" in disguise18:39
ralsinaanyway, I think it's looking ok, een if it doesn't quite follow the wireframes to the letter18:40
mandelralsina: if you but it as a next, how would you ensure that is placed where they wanted?18:40
mandelralsina: you would have to play with the button layout of the wizard that can be everything but flexible18:41
ralsinait is at the bottom right corner, where the next button usually is. I mean in the page with the captcha18:41
ralsinabut yes, the "choose sign in" page's wireframes mean no wizard buttons there18:42
alecudobey, it looks like it's the "Get" dbus methods on the Mock, being called with the wrong propname. They don't match the propname, and end up returning None, which can't be converted to a dbus value, and this error is caught by your new code and ignored (and that's fine). Please open a bug for this, and assign it to me.18:42
mandelralsina: exactly, so we have to fight between design and the QWizard implementation18:42
ralsinamandel: exactly18:42
ralsinawhich is why, I will probably (later) sit down with design and move a few buttons around :-)18:43
dobeyalecu: ah right. guess I should make it raise an exception18:43
mandelralsina: feel free to do so :)18:44
ralsinamandel: hahaha18:44
alecudobey, right, rising a new "NotFoundException" would be the right thing there.18:44
dobeyalecu: well i was thinking a custom "InvalidProperty" exception18:46
mandelalecu: ping18:46
dobeyor DBusException("Invalid properties") or something18:46
alecudobey, "InvalidPropery" sounds fine. It will be translated by dbus automatically into a DBusException on the other side.18:47
dobeyright18:47
* dobey fixes that18:47
alecudobey, approved18:49
alecumandel, still need the huuuuuge review?18:50
mandelalecu: yes, but is not for that, go a question, when I try to run sd on my machine, which used to work, I now get the following: http://paste.ubuntu.com/631291/18:50
mandelalecu: looks like credentials related, which is strange.. sd should be able to run even without the credentials, the state machine does include the state (with no user, with network)18:51
alecumandel, oh, right.18:51
dobeyalecu: great, thanks18:52
mandelalecu: oh, right? suena peligroso ;)18:53
alecumandel, "peligroso" is my middle name.18:53
nessitaalecu, mandel: syncdaemon has a setting which is autoconnect18:54
nessitaso it will ask the keyring for the credentials18:54
nessitamandel: do you have SSO working?18:54
nessitarunning, I meant18:54
mandelnessita: no, it was not running, but I wanted to check if the sd would work from trunk, it should be able no matter if there are creds or not18:55
* mandel tries with sso18:55
nessitaalecu: I know what's mandel issue, he updated his syncdaemon to the latest, which now handles credentials "properly" (after your fix)18:55
nessitamandel: the thing is that before alecu's branch, the connect() method was buggy and was not doing what is should, which is, seek for credentials18:56
mandelnessita: so why do we have a state in the state machine that allows to have no user with network, it does not make much sense..18:56
mandeldoes it?18:57
nessitamandel: so, it was working before for you because connect() was doing kinda nothing18:57
nessitamandel: it does not make sense, but you can not connect by setting autoconnect to False in the conf file18:57
nessitaI does* make sense, I mean :-D18:57
mandelalecu, nessita: ok, so now I get the jelly serialization error.. whic I'm guessing we have to fix18:58
nessitamandel: please do not overlap work with alecu. I think you should work on having the start() method implemented18:59
nessitaalecu: you're dealing with the jellyfish, right? :-)19:00
alecuI am19:00
alecuI am the walrus19:00
nessitagreat19:00
alecuchoo-choo-choo-choo19:00
nessitamandel: so, you work now on the SDTool.start()19:00
nessitayes?19:00
mandelnessita: yes, I wanted to test sdtool agains trunk, but seems impossible…19:01
nessitamandel: WELCOME TO MY WORLD :-P19:01
nessitaand alecu's19:01
* mandel cries 'but it was working….'19:02
nessitamandel: using hard code credentials does not mean it was working ;-)19:03
nessitacoded*19:04
nessitaor hard core credentials, as well19:04
mandeldetails, details...19:07
nessitaheh19:07
alecumandel, I found the solution to the jelly, but the branch is missing some bits still19:07
alecumandel, anyway, I'm reviewing your megabranch right now.19:08
mandelalecu: awesome, what was the issue for the jelly thing?19:08
alecumandel, self.sso_proxy.find_credentials(APP_NAME, {}, reply_handler=reply_handler, error_handler=error_handler)19:09
alecumandel, both reply_handler and error_handler were set to NO_OP19:10
alecumandel, but sso_proxy was a pb proxy, not a dbus one19:10
ralsinamandel, alecu, nessita: Can I get two kind reviews for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800359/+merge/6570319:10
alecumandel, so both NO_OP ended being sent thru pb as keyword parameters19:10
mandelalecu: yes, but there is a decorator that should ensure that the callbacks are Referenceable objects, right?19:10
nessitaralsina: I'll get one19:11
alecumandel, well, they were being *sent* thru pb19:11
nessitamandel: let's trade https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/explore-folder/+merge/6569719:11
ralsinanessita: there is no test for the constants because the tests don't work on any platform. On Linux SSO doesn't install teh qt stuff, and on windows I get a reactor problem when trying to use u1trial19:11
alecumandel, instead, they should not be sent thru pb, but instead the corresponding callbacks are already being connected to the remote signals.19:12
alecumandel, so, some bits are missing there, and a bit more debugging.19:12
alecumandel, but please let me finish your megareview first.19:12
nessitaralsina: the tests DO work on linux, we ran them yesterday19:12
mandelalecu: true, the pb returns a deferred19:12
nessitaralsina: what error you mean by "On Linux SSO doesn't install teh qt stuff"?19:12
alecumandel, pb returning a deferred has nothing to do with this!19:12
ralsinanessita: importing sso qt stuff breaks them19:12
mandelralsina., nessita: I'll take a look at your proposals19:12
nessitaralsina: let's set a pythonpath19:13
nessitaralsina: and on windows, what reactor problem are you having?19:13
ralsinanessita: since ubuntu-sso-client doesn't install the SSO things, tests don't run. It doesn't even compile the .ui files unless you hack setup.py19:13
alecumandel, that deferred returned by pb only means that the find_credentials was correctly handled on the other side.19:13
alecumandel, it is not the return value of the find_credentials.19:13
ralsinanessita: will show you in 2'19:13
ralsinaoops/SSO things/qt things/19:13
alecumandel, the return value of find_credentials is sent thru a signal; but that signal should be "subscribed" first with "register_to_signals".19:14
ralsinanessita: I get "The specified reactor is not supported." on windows. With both qt4 and txnp reactors19:14
mandelalecu: yes, you are right19:14
alecumandel, and btw: register_to_signals/unregister_to_signals is braindead to me as an API.19:14
nessitaralsina: are you u1devtools updated?19:14
mandelralsina: you do not have them installed, you are getting an import error and is printing that19:15
alecumandel, so, bits and pieces missing. I'll keep working on it later.19:15
ralsinanessita: trunk19:15
dobeyralsina: are you pulling stuff from wrong PYTHONPATH?19:15
mandelalecu: ok, I'll get back to reviews and sdtool19:15
nessitaralsina: do you have them installed, like mandel says?19:15
ralsinamandel: ok, will look into the tests then19:15
nessitaralsina: right, no test is not an option (still! :-))19:15
mandelralsina: also, as dobey said, check your path is ok19:15
ralsinaI am pretty sure I have the qt4 one installed since it works ;-)19:15
ralsinabut yes, could be PYTHONPATH crazyness19:16
ralsinaI would like some IRL testing to know if it only works for me though :-)19:17
nessitaralsina: what thing may work only for you?19:17
ralsinanessita: the whole ubuntuone-windows-installer thing19:18
nessitaralsina: I'll do an IRL, but I would like you to work on adding tests, and having them running. Since the next work is going to be more complex and we need to have tests for those for sure.19:19
ralsinanessita: yep, tests the rest of the day, in principle19:19
ralsinaand if sdtool is looking good, start the syncdaemon/cntrolpanel integration tonight19:20
nessitaralsina: well, ideally I would expect you to have the tests done before you eod, and then start integrating u1cp widgets. You think that's doable?19:21
ralsinanessita: what I said19:21
nessitaralsina: ah, I understood something else19:21
ralsinaI will have the tests on before EOD, and then in my couple of late hours will look at u1cp widget integration19:21
* ralsina has two eods a day now ;-)19:22
dobeygrr lp19:22
nessitaralsina: ah, thanks for the clarifiaction, I misunderstood before19:22
nessitaralsina: sounds great then19:22
dobeyyay finally19:22
ralsinaanyone need a second review on something? I dn'twant to code for 3 0 minutes :-)19:23
nessitaralsina: did you do mine? :-) (LP does not show any review from you)19:24
ralsinanessita: probably got an idea on how to fix something and forgot. Link?19:25
nessitahttps://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/explore-folder/+merge/6569719:26
ralsinanessita: ok, on it for real this time19:26
nessitathisfred: I share your concerns from the review, but all those will be cleared up with the design team when they see this19:27
mandelnessita: I just finished this : https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/explore-folder/+merge/6569719:27
thisfrednessita: that's what I figured, so I approved19:27
nessitamandel: oh thanks19:28
nessitaralsina: no need then19:28
ralsinanessita: oooook then.19:29
nessitahow can I have a cmd.exe on w7? start -> run command is gone and I can't find it in less than 3 seconds19:29
nessita(so I better ask the experts here)19:29
ralsinanessita: yes, windows key -> r and type cmd19:29
ralsinaor type cmd in the search line and press enter.19:30
nessitagreat19:30
nessitaworked19:30
ralsinaAny other important reviews pending?19:30
mandelralsina: line 253 in https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800359/+merge/65703 is that print on purpose?19:30
ralsinamandel: surely not.19:30
alecunessita, if the windows key is captured by unity, try using "Ctrl-Esc"19:30
nessitaalecu: is not, but thanks!19:31
mandelralsina: also, line 268, if you do nothing with the exception, at least log it so that we know it occurs, right?19:31
nessitaralsina: most of the news docstrings have no ending dot, can you please add those19:31
ralsinamandel: ok. The exception is a bug in PyQt when nothing is connected to the signal19:32
ralsinanessita: ok.19:32
mandelralsina: oh, then we should have a comment so that the next guy knows about it19:33
ralsinamandel: it's ok, I will log it and add a comment19:33
mandelralsina: lines 434 and 441, why did you comment out the imports?19:33
mandelI mean, if you do not need them then dd, right?19:34
nessitaralsina: going back to have the tests running on linux, is there a chance to have the running if we set PYTHONPATH to the proper ussoc source code, where all the QT stuff is available?19:34
ralsinamandel: yes, will delete19:34
ralsinanessita: possibly, but you need to edit setup.py to make it compile the .ui files19:35
mandelralsina: the title passed to the controllers should be translated via getted, lines 469 and 48319:35
nessitaralsina: but... ./setup.py is al;ready building the qt ui for sso. What do you mean exactly?19:36
ralsinanessita: last I checked, on Linux, it didn't19:36
ralsinanessita: I may be remembering wrong, of course19:37
alecumandel, def is_running(bus=None): does not use bus for anything, right?19:37
nessitamandel, ralsina: if you use the proper constant from u1client no translation should be needed, right?19:37
mandelalecu: no, left it to be compatible with linux, no other reson19:37
mandelnessita: true, but is the header of the page, I don't know if we have that in the contansts… probably we do19:38
alecumandel, oh, so the api is the same, right_19:38
alecu?19:38
ralsinanessita: there is no constant for this, I think19:38
nessitaralsina: which string is it?19:38
ralsina"Sign In"19:38
mandelalecu: yes, that is the idea, so that nessita does not have to do weird things with it19:39
alecumandel, but start() in (the huge and perhaps wrongly named) SyncDaemonTool sends a self.bus to it. Which is not used in any other place...19:39
alecumandel, (it should have been named SyncDaemonSwissTool)19:39
mandelalecu: hahaha19:40
alecuthank you, thank you. I'll be here for the remaining of the week.19:40
mandelalecu: ein?19:41
alecumandel, SyncDaemonTool.start: "if not is_running(self.bus)"... but self.bus is not defined anywhere!19:42
ralsinamandel, nessita: added periods, removed commented code, added comment for exception catch, translated "Sign In".19:43
mandelalecu: oh, that is a bug, let me check 'cause I though that was fixed...19:43
thisfreddog is acting weird I think she needs to go, bbiab19:43
alecumandel, another thing: reply_handler inside get_current_downloads already has access to "d" (thru the closure), so no need to receive it as an arg, and no need for the lambda that calls reply_handler19:46
mandelalecu: ok, let me check19:47
mandelalecu: true, that is ugly, I would remove the use of the lambda, what do you think?19:48
alecumandel, that sounds perfect.19:48
alecumandel, also, inside get_current_downloads too: since no error handlers are attached to downloads_d, any error would be lost.19:49
alecumandel, it would be *much* safer to rewrite all this using inlineCallbacks.19:49
alecumandel, but perhaps nessita will agree that we should better do this as a refactor later.19:49
nessitaalecu: if those are 15-20 minutes changes, let s do them now19:50
nessitaalecu: I agree with your suggestions, they will ease tracking busg later if needed (let's hope not!)19:50
alecumandel, same thing with the lambdas and inlineCallbacks in get_current_uploads19:51
alecumandel, and in pretty much every function in that module.19:51
mandelalecu: ok, I'll move all the possible methods to use the inlineCallbacks decorator19:52
alecuwell, not every function :P19:52
mandelalecu: well, you understood what I meant :P19:52
alecuright!19:52
alecumandel, so, every function that's creating an returning a Deferred is candidate to rewrite with inlineCallbacks, and it would be much safer written that way.19:54
alecumandel, please let me know if you need help with that. And if you consider that it's too much trouble let's file a bug and do it next week.19:55
mandelalecu: then all those that are using the pb code directly to, like all the config ones19:57
mandelthat is why there are lots of them19:57
ralsinaA few minutes ago I tried to stand up and my knee didn't let me. I am going to see another doctor now, I should be back in IRC in 30 minutes or so from the cell phone on the waiting room, and  back at coding in 90 minutes.19:57
nessitaralsina: looking your branch19:58
ralsinanessita: cool, thanks.19:59
nessitaralsina: good luck in the doctor19:59
ralsinanessita: even more thanks :-)19:59
alecuralsina, you should only stand up at 10AM sharp!20:04
ralsinaalecu: seems doing it twice a day is too much formy old bones ;-)20:05
* ralsina will live in a bath tub from now on20:05
ralsinaBTW: quasseldroid rocks!20:06
nessitamandel: ping20:13
mandelnessita: pong20:13
nessitamandel: I'm trying to install bzr, but for that I need to install its dependencies, but for that I need pycurl-ssl-bla, but when trying to install I'm getting an error:20:13
thisfredback20:14
nessita"The program can't be started  becasue MSVCR71.dll is mising from your computer"20:14
nessitamandel: what the heck is that?20:14
mandelnessita: nest thing is to install the bundle version, the complain is about a dll that comes with vs20:14
nessitamandel: ok, the wiki should said so, I'll add it20:15
mandelnessita: yes, mea culpa20:15
nessitais ok, we'll improve it together20:15
mandelnessita, alecu, ralsina: I'm going to call it a EOD since its 21:20 here.. I might be back later, but I'm not sure20:20
mandellaters!20:20
nessitamandel: question20:21
nessitamandel: did you finish the branch alecu was reviewing?20:21
ralsinamandel: c u tomorow!20:22
dobeytrivial review on https://code.launchpad.net/~dobey/ubuntuone-client/fix-headers/+merge/65714 please?20:23
thisfredon it20:33
thisfreddobey: done20:33
dobeythanks20:34
alecudobey, approved.20:34
jeroen-ping rye20:35
ralsinaok, gt x-rays, turns out I have what the doctor calls "heck of a bump" in the knee, and it's swelling, and it's probably nothing, except it's going to hurt like crazy for a few days.21:12
dobeyralsina: morphine ftw21:13
ralsinadobey: good band!21:13
ralsina;-)21:13
nessitawhat program can I use to tar xzv?21:19
nessita7zip!21:20
dobeyalso have https://code.launchpad.net/~dobey/ubuntuone-client/fix-headers-1-6/+merge/65723 and https://code.launchpad.net/~dobey/ubuntuone-client/fix-headers-1-4/+merge/65728 that are trivial if anyone can take a quick look and approve :)21:27
ralsinadobey: on it!21:30
dobeyand thisfred needs another review on https://code.launchpad.net/~thisfred/ubuntuone-client/better-progress-bar/+merge/6488721:30
thisfrednot urgent though, any windows work should go first21:31
dobeywell, would be nice to put it in the release for oneiric21:33
ralsinathisfred: I really want to use Linux for 5' so, reviewing ;-)21:33
ralsinadobey +1 on the two trivial ones21:34
thisfredawesome!21:34
dobeythanks!21:34
ralsinathisfred: so this makes the progressbar go by bytes instead of files?21:35
thisfredralsina: exactly21:36
ralsinathisfred: nice!21:36
ralsinathisfred: works too! ;-)21:37
thisfredyay!21:37
ralsinathisfred: +1'd21:38
thisfredthx!21:38
ralsinathisfred: now if the unity folks make a visible progressbar we are good with it! ;-)21:39
thisfredheh21:39
ralsinaI made a zoom in mine, it has 4 pixels of border and 2 pixels of actual progress thingie between them.21:39
dobeynoice21:42
ralsinaInsecureJelly exception? Some developers should really joke less and give descriptive names more :-(21:53
thisfredsounds like the bad old twisted days. They had error msges like: "Your opulent throne will be your undoing."22:00
thisfredSlightly less informative than "guru meditation #00000004.0000AAC0"22:01
thisfredalso: http://www.youtube.com/watch_popup?v=EVwlMVYqMu4&vq=medium#t=12522:02
dobeyalright, i'm out. later all!22:07
thisfredlater dobey22:07
ralsinathisfred: that error is from twisted, so it's still the bad old days over there ;-)22:26
thisfredhe22:31
thisfredok, gonna walk the dogs, may be back later, ping me if anyone needs reviews or anything22:32
ralsinaok, first EOD, tests are almost there, will resume work in a couple of hours22:46
=== m_conley is now known as m_conley_away
nessitabye all22:54

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!