[00:58] <dobey> adorilson: hi
[01:15] <adorilson> Do you have fixed this bug https://bugs.launchpad.net/ubuntuone-client/trunk/+bug/801938 ?
[01:15] <ubot4> Launchpad bug 801938 in ubuntuone-client (Ubuntu Oneiric) (and 3 other projects) "Make command gets error: variable 'result' set but not used (affects: 1) (heat: 8)" [High,Fix released]
[01:17] <adorilson> I remember that you ask to me if I can do a merge proposed. alrigth?
[01:52] <dobey> adorilson: yes
[01:53] <dobey> adorilson: i made a proposal and attributed the fix to you
[01:53] <dobey> anyway, time to finish the beer and get some sleep :)
[02:15] <adorilson> dobey: ok.
[02:15] <adorilson> time to sleep for me too
[02:15] <adorilson> see you. o/
[10:08] <ralsina> morning!
[10:21] <fagan> brb just heading down to the shop for some supplies and ill grab some lunch too
[10:46] <duanedesign> morning all
[10:48] <karni> morning duanedesign
[10:53]  * fagan  back 
[10:53] <fagan> morning duanedesign and karni
[10:53] <karni> \o
[10:54] <duanedesign> o/
[10:54] <fagan> \o/
[11:08] <fagan> ralsina: have you made a different version of that script to bundle the branches that looks at the site_packages
[11:08] <fagan> Or do I just have to import everything thats missing and give the script back
[11:11]  * fagan is just wondering am I testing or actually fixing it 
[11:20] <fagan> ooh its working
[11:21] <fagan> ralsina: ok its working fully now
[11:23] <fagan> oh actually there is 1 think that I dont know about its looking for a module called configglue.inischema
[11:23] <fagan> (for syncdaemon)
[11:23] <fagan> oh its just config glue my bad
[11:31] <fagan> Seems like there is something rotten in syncdaemon when you run it from the exe http://paste.ubuntu.com/643178
[11:31] <fagan> ralsina: ^
[11:32] <fagan> other than that everything is working fine
[11:42] <nessita> hello everyone!
[11:43] <fagan> yo nessita
[11:43] <nessita> hola fagan, how is it going?
[11:43] <fagan> nessita: good good just got the py2exe thingy working
[11:43] <fagan> other than for syncdaemon
[11:44] <fagan> but isnt that borked still?
[11:44] <fagan> I pastebinned the error just in case
[11:46] <fagan> Other than that its a nice sunny day in ireland :)
[11:46] <nessita> fagan: what's the pastebin?
[11:46] <fagan> http://paste.ubuntu.com/643178
[11:47] <fagan> Oh it might have something to do with the account not being set up either
[11:47] <nessita> fagan: nopes
[11:47] <nessita> fagan: you need to pass 2 arguments to syncdaemon
[11:48] <nessita> fagan: the syncdaemon.conf and the logging.conf, you can grab both from ubuntuone-client/data
[11:48] <fagan> nessita: ak ok I just ran it and was hoping just not to get an error about a module not found
[11:48] <fagan> So thats cool then
[11:52] <nessita> ralsina: ping
[12:21] <nessita> ralsina: you around?
[12:22] <fagan> nessita: he said morning must have went off for a bit
[12:29] <ralsina> nessita: here now!
[12:29] <nessita> ralsina: hi there!
[12:29] <ralsina> nessita: took thekid to school frgot to say it
[12:30] <ralsina> goodmorningfagan, nessita
[12:30] <nessita> ralsina: I made a mistake in allhands and I submit only one goal (I clicked on the wrong button, I guess. That UI sucks)
[12:30] <ralsina> he
[12:30] <ralsina> nessita: you can't add any more?
[12:30] <nessita> ralsina: can you do something to give them back to me so I expand?
[12:30] <nessita> ralsina: no, I can't edit any more
[12:30] <ralsina> nessita: no idea, but I can try!
[12:31] <fagan> hey ralsina, got the thingy done what do I do now?
[12:31] <ralsina> fagan: really? cool!
[12:31] <ralsina> fagan: now, zip the dist folder and put it somewhere I can get it
[12:31] <fagan> ralsina: cool ill put it on u1
[12:32] <fagan> ralsina: you want the script too with the changes to the imports
[12:32] <ralsina> fagan: what changes did you do?
[12:32] <fagan> ralsina: well I added the missing imports to the py2exe path bit
[12:32] <ralsina> which were...
[12:33] <fagan> like oauth httplib2..etc
[12:33] <fagan> simplejson
[12:33] <ralsina> fagan: put that in a pastebin
[12:33] <fagan> Will do
[12:33] <ralsina> nessita: it seems I can add objectives for you
[12:34] <ralsina> nessita: but I don't see how I can open them up for you again. Maybe if I reject them, but that's a bit scary :-)
[12:35] <nessita> ralsina: can you send them back to me?
[12:35] <ralsina> nessita: I can only reject, edit and add more
[12:35] <nessita> let me ask in #hr
[12:37] <nessita> ralsina: hr will fix this for me
[12:38] <ralsina> nessita: cool
[12:44] <fagan> ralsina: I pmed the paste
[12:45] <ralsina> fagan: that one doesn'thaveyour additions. I remember you had to add oauth and it's not there
[12:46] <ralsina> fagan, you added "import oauth"? I meant add them to the includes list! Please try again?
[12:47] <ralsina> fagan, in fact, my instructions were "fagan,could you add "oauth" in the includes list of the script and rerun?"
[12:47] <fagan> ralsina: I thought you said just add the import to the path bit so it would pull it in and its working.
[12:47] <fagan> ralsina: so I just have to move it up to the top import list
[12:47] <fagan> ?
[12:48] <ralsina> No, I told you that,22 hours ago.
[12:48] <ralsina> NO
[12:48] <ralsina> Look for something that is a "includes list"
[12:48] <fagan> ralsina: ok
[12:48] <ralsina> Tip, it's a list called includes in line 200 and something ;-)
[12:50] <ralsina> fagan, alecu, nessita, dobey, mandel, standup in 10'
[12:50] <ralsina> And DiegoSarmentero of course
[12:50] <DiegoSarmentero> ralsina, :P
[12:50] <ralsina> (this once it was on purpose, I swear ;-)
[12:51] <DiegoSarmentero> ralsina, jejeje
[12:51] <fagan> ralsina: mandel isnt around is he on a swap or something?
[12:51] <ralsina> fagan: no idea, I have no swap requests from him
[12:51] <fagan> ralsina: weird
[12:51] <ralsina> fagan: he is on shorter hours for the week, though
[12:52] <fagan> ah maybe he will get in a while
[12:54] <fagan> ralsina: ok added them to the includes bit
[12:55] <fagan> ralsina: and its working
[12:55] <ralsina> fagan: cool, generate the bundles, zip, upload and put the script in there as well
[12:55] <fagan> ralsina: cool
[12:55] <fagan> will do
[13:00] <fagan> moi
[13:00] <dobey_> hmm
[13:00] <fagan> ralsina, nessita, DiegoSarmentero, dobey standup
[13:00] <nessita> me
[13:00] <DiegoSarmentero> me
[13:00] <ralsina> me
[13:01] <dobey> me
[13:01] <fagan> DONE
[13:01] <fagan> * get ralsina's script working
[13:01] <fagan> TODO
[13:01] <fagan> * upload the dist folder and script
[13:01] <fagan> BLOCKED
[13:01] <fagan> * no
[13:01] <fagan> nessita: go
[13:01] <nessita> alecu is not here?
[13:01] <fagan> nessita: nope
[13:01] <nessita> DONE: added tony fixes and landed DiegoSarmentero's branch for control panel styling, ran the whole u1client test suite in windows, took screenshots for the bosses, meetings, tcp-activtion reviews
[13:01] <nessita> TODO: tweak control panel UI a bit more bug #809363, windows reviews
[13:01] <nessita> BLOCKED: nopes
[13:01] <nessita> NEXT: DiegoSarmentero
[13:01] <ubot4> Launchpad bug 809363 in ubuntuone-control-panel "FileSyncStatus needs to be updated to the current design (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/809363
[13:01] <DiegoSarmentero> DONE:
[13:01] <DiegoSarmentero> Assets added to u1 windows installer. Improves in style. Tests added.
[13:01] <DiegoSarmentero> TODO:
[13:01] <DiegoSarmentero> Apply new design from Lisette. Check and Improve tests. Review fonts and specs definition.
[13:01] <DiegoSarmentero> BLOCKED:
[13:02] <ralsina> I assume BLOCKED: yes? :-)
[13:02] <nessita> that blocked?
[13:02] <nessita> :-D
[13:03] <ralsina> DONE: created windows installer (actual one) and sent to design and QA, reviews, calls. TODO: signed URLs, embedded webkit, review wizard styling, BLOCKED: no
[13:03] <fagan> Wow very blocked
[13:03] <ralsina> and dobey! :-)
[13:03] <dobey> λ DONE: bug #809561 (mostly), thought about shim design
[13:03] <dobey> λ TODO: start the shim
[13:03] <dobey> λ BLCK: None.
[13:03] <ubot4> Launchpad bug 809561 in ubuntuone-dev-tools "Support ignoring paths in u1lint (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/809561
[13:03] <nessita> ralsina: did we got any feedback from design yet?
[13:03] <ralsina> nessita: not a peep
[13:03] <ralsina> nessita: they must be shocked by our brilliance I assume, and scared to say anything
[13:04] <fagan> ralsina: obviously :)
[13:04] <nessita> or they passed out :-/
[13:04] <ralsina> yeah, or that
[13:04] <ralsina> ok, comments that are not about DiegoSarmentero's strange disappearance?
[13:05] <ralsina> Or mandel
[13:05] <ralsina>  's strange disappearance ;-)
[13:07] <fagan> ralsina: we are dropping like flies
[13:07]  * fagan checks under his bed for the programmer eating monster 
[13:07] <ralsina> Diego got split, itseems
[13:08] <ralsina> mandel had connectivity problems yesterday, was around too early for me today
[13:08] <ralsina> ok, eom!
[13:09] <ralsina> Now everyone, go DO that TODO that you DO so well.
[13:09] <nessita> eom!
[13:10]  * fagan dodothetodo
[13:15] <alecu> good morning!
[13:15] <fagan> morning alecu
[13:17] <nessita> alecu: gooooooood morning. And go! when you're ready
[13:17] <ralsina> I forgot about *alecu's* strange disappearance, too! :-)
[13:17] <FxIII> hi all
[13:17] <ralsina> Hello FxIII
[13:17] <FxIII> there is some api to build services based on ubuntuone?
[13:17] <fagan> hmmmmm the web client doesnt want to accept the zip
[13:18] <FxIII> *is there
[13:18] <fagan> FxIII: yes we do! :)
[13:18] <fagan> https://one.ubuntu.com/developer/
[13:19] <fagan> FxIII: if you need any help with anything ask away
[13:19] <FxIII> fagan: wow! the page i was lookin for
[13:20] <alecu> me
[13:20] <alecu> DONE: branch for tcp activation in syncdaemon; started working on bug #806539
[13:20] <alecu> TODO: finish branch for that bug
[13:20] <alecu> BLOCKED: no
[13:20] <ubot4> Launchpad bug 806539 in ubuntuone-client (and 1 other project) "Cleanup API to connect to services (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/806539
[13:21] <alecu> ralsina, sorry about my strange disappearance; got abducted while I was "yendo de la cama al living" :P
[13:23] <FxIII> fagan: i'm interested in couchdb storage, but i cant find the base url for my user
[13:23] <ralsina> alecu: damn livingroom aliens
[13:24] <fagan> FxIII: well we have an easy way of using couchdb in Ubuntu called desktopcouch
[13:25] <fagan> give me a sec ill find a link to some tutorials
[13:25] <FxIII> cant be accessed directly?
[13:25] <FxIII> I mean if i understood right the desktopcouch is a local couchdb that is sincronized with the remote one
[13:25] <FxIII> where the remote one is?
[13:27] <fagan> FxIII: yep your correct
[13:27] <dobey> FxIII: ubuntuone-couch is probably what you want
[13:27] <dobey> FxIII: it's a python lib to talk to the remote couchdb on the server
[13:28] <FxIII> is there difference with the standard python library for couchdb?
[13:32] <fagan> wow the file upload on the web of a 20 mb file seems to be taking a month
[13:32] <fagan> or its after hanging
[13:32] <ralsina> fagan: use chromium, it has an upload progress bar
[13:33] <fagan> ralsina: ah ok I was using ff
[13:33] <ralsina> Or wait for it :-)
[13:42] <alecu> nessita, ralsina: are we meeting in 5 minutes?
[13:42] <ralsina> alecu: well, we are doing it without mandel, it seems
[13:43] <nessita> alecu: I hope so, I pinged mandel thru twitter and got no response
[13:45] <nessita> Chipaca`, ralsina, alecu: mumble?
[13:46] <Chipaca`> nessita: give me 5
[13:46] <nessita> sure!
[13:50] <nessita> ralsina, alecu: me escuchan?
[13:54] <fagan> ralsina: pmed both the dist and the script
[13:55] <fagan> ralsina: and dont mind its called build. I just thought the problem uploading was a naming issue
[14:19] <joshuahoover> dobey: ping
[14:19] <dobey> joshuahoover: hi
[14:19] <joshuahoover> dobey: hi! bug #776836 ...we've had a few users run into this...looks like we have some debug output, any ideas?
[14:19] <ubot4> Launchpad bug 776836 in libubuntuone (Ubuntu) (and 1 other project) "Pressing on the "Subscribe" button to download my purchased music didn't work. (affects: 6) (dups: 2) (heat: 77)" [Medium,New] https://launchpad.net/bugs/776836
[14:24] <dobey> joshuahoover: hrmm, well, mterry's debug output is suggesting that ubuntuone-syncdaemon crashed when dbus was trying to start it
[14:26] <dobey> and it doesn't check connectivity, and treats errors as "you aren't subscribed" which is a problem, but requires ui/string change to fix :(
[14:27] <joshuahoover> dobey: hmmm...that's no good :(
[14:46] <ralsina> nessita's phone died, she will be back eventually
[14:51] <dobey> doh
[14:53] <ralsina> alecu: could you point me towards the timeout value in sso?
[14:55] <alecu> ralsina, sure.
[14:55] <alecu> ralsina, ubuntu_sso/main/__init__.py:TIMEOUT_INTERVAL = 1000000
[14:56] <ralsina> alecu:cool, thx
[14:57] <alecu> ralsina, it used to say 10000, meaning ten seconds; I should have updated the comment when adding the two zeroes
[14:58] <ralsina> alecu: ok,trunk has that.
[15:09] <dobey> nessita!
[15:10] <alecu> hola nessita!
[15:11] <nessita> dobey: I'm very sad right now, my internet provider decided I cna't have phone (ergo internet) until the 22nd
[15:11] <alecu> wtf!
[15:11] <alecu> nessita, you should pay your bills on time!
[15:12] <nessita> alecu: "there is some damage caused by a thrid party and we're replacing cables until the 22"
[15:12] <dobey> nessita: :(
[15:12] <alecu> :-(
[15:13] <nessita> I'm stealing some wifi from someone around, but I need to seek a more permanent solution
[15:13] <dobey> nessita: well, good news, i fixed the path vs filename issue in my devtools branch, so you can -i path/ and it will work
[15:13] <nessita> dobey: ok, I'll try it that soon
[15:13] <alecu> nessita, catch a bus and bring matias, you can both stay at my office :-)
[15:14] <nessita> alecu: it sounds great, really :-) I will count that as an option
[15:16] <dobey> ok, lunch time. bbiab
[15:51] <ralsina> damn, the credentials the wizard gets don't work :-(
[15:55] <nessita> ralsina: any error?
[15:56] <ralsina> nessita: I get that the token is invalid.
[15:56] <nessita> ralsina: can I see the error?
[15:56] <ralsina> nessita: in a few minutes, sure
[15:57] <ralsina> nessita: I am trying to see if the debug on screen is similar for the installer and get_credentials, since get_credentials's credentials work
[15:57] <nessita> ralsina: how are you getting credentials?
[15:57] <nessita> in the wizard, I mean
[15:58] <ralsina> By calling SSO with the same parameters control panel uses
[15:58] <ralsina> I can't use the u1cp credentialsmanager directly because of UI problems
[16:02] <nessita> ralsina: you should not be calling SSO directly, let's try to fix the second issue you mentioned
[16:02] <ralsina> Looks like it's not doing the ping at the end. That could be the problem, right?
[16:02] <nessita> ralsina: there is no u1cp credentialsmanagement thingy
[16:02] <nessita> ralsina: you meant the u1client credentialsmanagement?
[16:02] <nessita> ralsina: all you should be calling is:
[16:02] <nessita> creds = CredentialsManagementTool()
[16:03] <nessita> result = yield creds.find_credentials()
[16:03] <nessita> is result == {} -> no credentials in the system
[16:03] <nessita> ralsina: did you try that ^?
[16:04] <ralsina> brb
[16:04] <ralsina> nessita: that's for checking if there are credentials, right?
[16:04] <nessita> ralsina: yes
[16:05] <nessita> ralsina: as far as I understand, that is all you need, right?
[16:05] <ralsina> no, I need to get credentials
[16:05] <ralsina> I *do* get credentials, but it seems they are wrong
[16:05] <nessita> ralsina: I'm confused
[16:05] <nessita> get credentials == find credentials
[16:05] <nessita> you mean register/login?
[16:05] <ralsina> nessita: yes
[16:06] <ralsina> sorry if I was using wrong terminology
[16:06] <nessita> is ok, I misundertood
[16:06] <nessita> ralsina: then you need to call yield CrendetialsManagementTool().register(), does that work?
[16:06] <ralsina> nessita: No idea.
[16:07] <nessita> you never used it?
[16:07] <ralsina> nessita: no, I am using ubuntu_sso.credentials.Credentials because there I can pass the UI as an argument
[16:08] <nessita> oh hum, that is not good
[16:08] <nessita> but I understand
[16:08] <nessita> you probably are not passing the proper ping url to the call, but ideally your code should not be aware of that
[16:08] <ralsina> I am missing some call CredentialsManager does, should check and do it right.
[16:08] <nessita> that's why we have the  ubuntuone.credentials module for
[16:09] <nessita> ralsina: can we instead have the wizard not calling SSO directly?
[16:09] <ralsina> nessita: then it breaks the whole UI
[16:09] <nessita> and that way use the U1 abtraction layer?
[16:09] <nessita> ralsina: can't we add something to the U1 abstraction layer so the wizard does not break?
[16:10] <nessita> I'm trying to see if we can keep the abstraction layer being such
[16:10] <ralsina> nessita: don't know.Just found out about this 5 minutes ago :-)
[16:12] <ralsina> nessita: credentials.register uses the gtk feature of getting a windows ID and embedding the widgets there. That's done completely different on the Qt side of sso
[16:13] <nessita> ralsina: but you have the Credentials.register for the QT side
[16:14] <nessita> ralsina: what I'm saying is: we may have a design issue that is coming up to light now, so, before workaround it, I would like to evaluate if it's fixable in a timely manner
[16:14] <nessita> if it's a big chance, we may need to accept the workaround
[16:14] <ralsina> nessita: ok. The API for Credentials.register's support for embedding the SSO UI is based on two things:
[16:15] <ralsina> 1) That the UI provided by SSO is correct. This is not the case with the current design we are trying to implement.
[16:15]  * fagan EOD
[16:15] <ralsina> 2) that to embed the UI you have to pass a window_id and then you can embed arbitrary widgets there. That doesn't work on Qt at all.
[16:16] <nessita> ralsina: the window_id is optional so you can not pass that
[16:16] <nessita> I still don't understand (1)
[16:16] <ralsina> nessita: yes, and you get a SSO popup
[16:16] <nessita> ralsina: right, my point is:
[16:17] <nessita> ralsina: can't we abstract the callers for SSO (the way you're calling it) so they call ubuntuone.credentials instead of sso directly? we may need to add a new method, yes, but not sure if you understand what I mean
[16:17] <ralsina> nessita: I need to style, theme, add extra buttons, and change texts on SSO, for the installer. If I use the default SSO popup, I don't see how to do that.
[16:17] <ralsina> That's the explanation for 1) above
[16:18] <nessita> ralsina: I'm not saying you use the popup
[16:18] <nessita> I'm saying let's try to hide the auth details in the proper place
[16:18] <ralsina> nessita: ok, I'm all for it. I am not getting how to do it though :-)
[16:19] <nessita> ralsina: what I'm saying is that the windows-installer should not be importing the constants APP_NAME, TC_URL, DESCRIPTION and  PING_URL in its code
[16:19] <nessita> ralsina: those constants are, somehow, private to ubuntuone.crendetials
[16:20] <nessita> ubuntuone,credentials should offer to you the API you need, hiding the auth detail we need t hide
[16:20] <nessita> for example, the ping url
[16:20] <ralsina> ok. I need ubuntuone.credentials to accept a ui_module parameter and pass it nto ubuntu_sso.credentials.Credentials
[16:21] <nessita> ralsina: so, maybe is too late, but from my POV, we should have a U1-SSO-widget that you can embed anywhere, but the workflow is defined in the widget in isolation, so we have the same user experience every time, every where we use it
[16:22] <nessita> ralsina: and the popup becomes only a dialog that embeds that dialog
[16:22] <nessita> and the wizard is a qwizard with that widget in it
[16:22] <ralsina> nessita: no, that's not good enough I am afraid
[16:22] <nessita> ralsina: why not? (I know is not short)
[16:22] <dobey> quacky qwizards
[16:23] <nessita> dobey: quack?
[16:23] <nessita> :-)
[16:23] <dobey> :)
[16:25] <ralsina> nessita: because wording is too strict(you can only add snippets in the middle of texts, and not on all pages), because you can't add a "No thanks I will setup later" button, because the app can't change widget positions, or sizes.
[16:26] <ralsina> nessita: and because the SSO widget *is* a qwizard, so you would have a wizard that pops up a wizard.
[16:26] <nessita> ralsina: yes, I agree to that. And I think all the wizard coding should be in sso not in the installer
[16:26] <nessita> anyways, we won't be changing that now :-(
[16:26] <ralsina> nessita: but the wizard sets up things that are not SSO related
[16:26] <nessita> the change is too big
[16:27] <nessita> ralsina: I mean other thing
[16:27] <ralsina> ok
[16:27] <nessita> I meant: the process of registering/logging in is a wizard in its own, and inside the SSO project, we should provide a single sso-widget that is embeddable into other widget
[16:28] <ralsina> ok, we can do that eventually :-)
[16:28] <nessita> yeap
[16:28] <nessita> not now, I know
[16:28] <nessita> is worth talking about it though
[16:28] <ralsina> Right now, I am going to make a sandwich, and spend my lunch break looking at this :-D
[16:28] <dobey> so we can put a wizard in a wizard? :)
[16:28] <nessita> dobey: of course!!!
[16:28] <ralsina> dobey: indeed, that's how wizards are made.
[16:28] <nessita> :-D
[16:28] <csgeek> I'm running Kubuntu 11.04 and I was wondering how you'd add a folder sync outside of the Ubuntu One folder.
[16:29] <nessita> csgeek: ou have ubuntu one running already?
[16:29] <nessita> you*
[16:29] <csgeek> nessita: yup.  the service is running and its  synching my Music + UbuntuOne folder atm
[16:29] <dobey> ralsina: i'm pretty sure morgan was made by putting a king inside a wizard ;)
[16:29] <nessita> csgeek: are you familiar with a command line terminal?
[16:29] <ralsina> But then, because of that, there was a wizard inside a wizard :-)
[16:29] <csgeek> nessita: yup.. I prefer cli
[16:30] <dobey> ralsina: eh, these jokes aren't the same without mandel on the internet :)
[16:30] <nessita> csgeek: so, you can use the u1sdtool command. To start syncing a new folder outside Ubuntu One (but inside your home), you can do
[16:30] <nessita> csgeek: u1sdtool --create-folder=/home/your-username/The/Folder/You/Want/To/Sync
[16:30] <nessita> use absolute path, JIC
[16:31] <dobey> haha
[16:31] <csgeek> okay.  Thank you nessita
[16:31] <dobey> this song title is a lisp statement
[16:31] <dobey> (defun botsbuildbots () (botsbuildbots))
[16:32] <ralsina> same as wizards!
[16:32] <dobey> bug wizards aren't megalomaniacal AIs
[16:36] <duanedesign> rye: is their a way to recover a users lost(deleted) tomboy notes?
[16:41] <nessita> ralsina: can a QLabel have both text and a pixmap set? (and show both?)
[16:42] <nessita> as per my tests no, it can't, but want to rule out PICNIC
[16:42] <ralsina> nessita: yes if using richtext
[16:42]  * ralsina is guessing though
[16:42] <nessita> ralsina: I mean calling setText and setPixmap
[16:42] <nessita> and having a pixmap and next to it a text
[16:42] <nessita> or shall I use 2 labels?
[16:42] <ralsina> no, just using setText with <img> taf in it
[16:42] <ralsina> tag*
[16:42] <nessita> guh, ugly
[16:42] <ralsina> or two lables, yes
[16:43] <nessita> ralsina: is cleaner to use 2 labels, I think, but what do you think?
[16:43] <ralsina> I woulduse two labels.
[16:43]  * nessita too
[16:43] <nessita> thanks!
[17:03]  * nessita -> lunch
[17:27] <ralsina> Ok, found the problem
[17:27] <ralsina> And with 9 minutes to spare in my lunch break, too ;-)
[17:28] <ralsina> Turns out, for some reason, even if you set the ping_url, ubuntu_sso.credentials.Credentials doesn't ping it.
[17:28] <ralsina> The ping is actually done as a callback from a callback from a callback from show_ui which makes no sense whatsoever AFAICS
[17:29] <dobey> ralsina: you put a callback in your callback because you like callbacks?
[17:29] <ralsina> dobey: not my code!
[17:29] <dobey> ralsina: you're the manager. it's your code by proxy ;)
[17:30] <ralsina> but anyway, the Credentials._login_success_cb is only connected by _show_ui
[17:30] <ralsina> dobey: oh mine you are right
[17:31] <ralsina> And I am not sure if that's intentional. Since other users of SSO call Credentials.login or Credentials.register, which make SSO show the UI, then it works.
[17:31] <dobey> heh
[17:31] <dobey> don't know
[17:32] <ralsina> Anyway, I am surely using sso "wrong".
[17:36] <ralsina> OTOH, why Credentials doesn't ping the ping_url on login success by default, I don't know
[17:37] <dobey> i thought it was something the server hit, not the client?
[17:39] <nessita> ralsina: can you please point me a some code? I may be able to answer
[17:39] <ralsina> nessita: sure
[17:40] <ralsina> nessita: http://pastebin.ubuntu.com/643406/ line 252 and following
[17:41] <ralsina> In line 262 is where the login_success_callback is connected
[17:41] <ralsina> without that connection, the ping is not done
[17:41] <dobey> nessita: did you get chance to re-review my devtools branch?
[17:41] <nessita> dobey: not yet, sorry, will start now
[17:41] <dobey> nessita: ok, thanks
[17:41] <nessita> ralsina: right, so
[17:41] <ralsina> Which means the only way to use the class is entering through _register_or_login() which pretty much makes it impossible to use a custom UI
[17:42] <nessita> ralsina: when we designed this module, the use case that made sense is what I mentioned before
[17:42] <ralsina> nessita: right
[17:42] <ralsina> but this would not work for the embedded widget we talked about
[17:42] <nessita> ralsina: the main goal of this module was to explictely avoid other use cases
[17:42] <nessita> ralsina: yes it would
[17:42] <nessita> ralsina: this code does not fix what UI you can use
[17:42] <nessita> ralsina: the login_or_register uses the UI you tell it to
[17:43] <nessita> the class you pass to be used in
[17:43] <ralsina> nessita: hmmmm
[17:43] <nessita>         gui = sys.modules[self.ui_module]
[17:43] <nessita>        self.gui = getattr(gui, self.ui_class)(app_name=self.app_name,                         tc_url=self.tc_url, help_text=self.help_text,                         window_id=self.window_id, login_only=login_only)
[17:43] <nessita> should connect the login_success_callback properly
[17:44] <ralsina> ok, there is a problem calling login but it's tricky and probably could be fixed redoing bits of the installer
[17:44] <nessita> and then the backend knows what to do in that case
[17:45] <nessita> ralsina: so, in your UI code, you should callback login_success_callback so is called when the login finished
[17:45] <nessita> ralsina: is your code doing that? /me checks
[17:45] <ralsina> that is the fix I am testing, and it works
[17:45] <ralsina> it means a call to a private methd though
[17:45] <nessita> nonono :-)
[17:45] <nessita> the private methods are not meant to be called! :-)
[17:45] <ralsina> oh, right
[17:46] <ralsina> yes it's called, you just assign it first ;-)
[17:46] <nessita> ralsina: wait
[17:46] <nessita> before you modify any more code, please follow me on this
[17:46] <ralsina> I have modified no code at all
[17:46] <nessita> ralsina: you defined the UbuntuSSOClientGUI class in embedded_sso, right?
[17:46] <ralsina> Ok, tested if it was really that missing call :-)
[17:47] <ralsina> yes
[17:47] <nessita> so, that class has the responsability to define 3 callbacks:
[17:47] <nessita> login_success_callback, registration_success_callback, and user_cancellation_callback
[17:48] <ralsina> ok
[17:48] <nessita> ralsina: and that class need to call those methods when those action, which are detectedat UI level, occur
[17:48] <nessita> so the backend can now, when login_success_callback is called, do a ping, or 2 pings, or 0 pings and store files in disk, whatever is needed
[17:49] <nessita> but what the backend does should be hidden from the UI
[17:49] <nessita> ralsina: this is why I think we need to hide all these details from upper layers in the credential management process...
[17:52] <ralsina> Ok, so the ClientGUI class has a controller and a view. When there is a successful login, controller makes the view emit loginSuccess. That signal should be connected to login_success_callback on the ClientGUI? Right?
[17:53] <ralsina> Which is set by _show_gui to credentials._login_success_cb
[17:54] <ralsina> I think I got it now
[17:54] <nessita> ralsina: forget the _something methods :-) I'm not sure I understand what you say above about controllers and views, but the UbuntuSSOClientGUI should call
[17:54] <dobey> ralsina: we'll be using MSIs for updates on windows, right?
[17:54] <nessita> call as in execute login_success_callback (and the other 3) when those events happends
[17:54] <dobey> ralsina: and for the initial install?
[17:54] <ralsina> dobey: it's an installer but it's not aMSI
[17:55] <ralsina> not done anything about updates yet
[17:55] <nessita> ralsina: does that answer your question?
[17:55] <ralsina> nessita: yes
[17:55] <dobey> hmm, ok
[17:55] <nessita> ralsina: :-)
[17:57] <ralsina> the part about controllers and views is how the ui replacement actually shows stuff
[17:58] <nessita> dobey: can you please give me the url of your branch? I can't browse it thru LP
[17:58] <dobey> nessita: https://code.launchpad.net/~dobey/ubuntuone-dev-tools/lint-ignores/+merge/67758
[17:58] <nessita> thanks
[18:06] <nessita> dobey: does this mean is not working or that am I too upset due this connectivity issue I'm having? http://pastebin.ubuntu.com/643416/
[18:06] <nessita> too upset to test it properly :-)
[18:07] <dobey> nessita: does "./bin/u1lint -i bin/" result in no errors in the devtools branch itself?
[18:07] <nessita> let me try it
[18:07] <nessita> dobey: nopes
[18:08] <nessita> I will branch again, JIC, but this is a clean branch
[18:08] <dobey> nessita: hrmm, ok, so it is working right in devtools
[18:08] <dobey> let me check again with cp
[18:08] <nessita> dobey: maybe you missed a push?
[18:08] <nessita> LP is misbehaving today a bit
[18:09] <dobey> nah, it's pushed, and even rescanned
[18:09] <nessita> dobey: http://pastebin.ubuntu.com/643421/ <- the diff I'm using and the results
[18:10] <dobey> nessita: ok, that's very weird :)
[18:10] <dobey> nessita: this is what i got in cp with it: http://pastebin.ubuntu.com/643422/
[18:11] <nessita> dobey: you pícaro!
[18:11] <nessita> dobey: you don't have the generated ui ;-)
[18:11] <dobey> nessita: yeah i just realized that
[18:11] <nessita> dobey: run ./setup.py build inside cp
[18:12] <nessita> ralsina: how would you translate pícaro?
[18:12] <ralsina> nessita: no idea, sorry
[18:12] <dobey> nessita: smartass? bastard?
[18:12] <nessita> nonono
[18:13] <dobey> wow it is slow now
[18:14] <nessita> dobey: we, argetinians, use it like "you tricky person" but in a friendly way
[18:14] <dobey> ah ok
[18:14] <nessita> dobey: is very slow, and what puzzles me, is much, much slower that pylint
[18:14] <nessita> for the same amount of files
[18:14] <dobey> hrmm
[18:14] <dobey> nessita: for me the slow thing was pylint
[18:15] <nessita> ...
[18:15] <dobey> because it was hitting the ui files :(
[18:15] <nessita> weird
[18:15] <dobey> pylint used up 240M before exiting
[18:15] <dobey> RES
[18:15] <dobey> which is not fun
[18:15] <dobey> and it was eating a lot of cpu
[18:19] <nessita> dobey: so, did your u1lint work in cp?
[18:21] <dobey> no
[18:21] <dobey> :(
[18:21] <dobey> but i see what's wrong
[18:21] <dobey> and also another problem
[18:21] <csgeek> my u1sdtool -s reports the state being QUEUE_MANAGER, and when I try to create a folder it seems to hang indefinitely
[18:22] <dobey> nessita: i am an idiot. sorry. will fix :)
[18:23] <nessita> dobey: is the day of the week, don't worry. /me is there too
[18:23] <nessita> csgeek: besides QUEUE_MANAGER, what do you have under 'queues'?
[18:23] <nessita> csgeek: either WORKING or IDLE
[18:24] <dobey> nessita, ralsina: btw, could you guys get me a list of all the assets we're using in u1-windows-installer that we're duplicating from somewhere else (graphics, fonts, etc sort of stuff)?
[18:24] <ralsina> dobey: ask DiegoSarmentero
[18:24] <ralsina> right now, u1-windows-installer has n assets
[18:24] <DiegoSarmentero> dobey, yes?
[18:24] <ralsina> sorry "no assets"
[18:24] <nessita> dobey: all the assests used in the control panel are located in the data/ dir
[18:24] <csgeek>     queues: WORKING
[18:24] <dobey> or in control panel
[18:25] <nessita> csgeek: your syncdaemon is working on stuff. What does u1sdtool --waiting | wc -l says?
[18:25] <csgeek> 1038
[18:25] <csgeek> okay.. I guess its thinking about stuff..
[18:25] <DiegoSarmentero> dobey, as nessita says, all the assets are inside data/
[18:26] <dobey> ok
[18:26] <nessita> csgeek: is doing, you can do u1sdtool --waiting | grep "running=True" | wc -l
[18:26] <csgeek> that gives me 25 items
[18:27] <csgeek> hmm.. I thought it was synched already.. I guess I can just wait it out..
[18:27] <ralsina> nessita: I can't pass to Credentials a module that is not in the SSO bundle, because of course, we can't import that. Which means that, when built into exes, the whole "pass a ui module" feature won'treally work.
[18:27] <nessita> csgeek: probably there is like 10 things happening in parellel
[18:28] <nessita> csgeek: yes, you should wait. Are you familiar with bzr?
[18:28] <nessita> ralsina: wait
[18:28] <csgeek> nessita: once I add a folder via the u1sdtool, as long as the daemon is running, it should keep the folder in sync, is that about right?
[18:28]  * ralsina waits
[18:28] <csgeek> nessita: yes..
[18:28] <csgeek> I've cloned/copies branches before
[18:28] <nessita> ralsina: the import happens at SSO run time. So we won't have an ImportError until the sso backend actually wants to load your UI
[18:28] <nessita> csgeek: one second
[18:29] <nessita> ralsina: so, the code where the sso backend tries to import a module from the installer will be trigger by the installer
[18:29] <nessita> ralsina: so, how come the class will not be available?
[18:29] <ralsina> yes, triggered, but it takes place in the sso process. That process doesn't have access to the modules in the installer process.
[18:30] <nessita> csgeek: bzr branch lp:magicicada; ./magicicada/bin/magicicada will give you a GTK UI with all the details about what syncdaemon is processing
[18:30] <nessita> csgeek: of course is optional and not official :-)
[18:30]  * ralsina may be confused by how show_gui works though, where the GUI isshown by ussoc process
[18:30] <nessita> csgeek: but if you have the gtk runtime installed, you may give it a try
[18:30] <nessita> ralsina: I'm happy to explain
[18:30] <ralsina> :-)
[18:30] <csgeek> nessita: that's fine.. visual tools are nice
[18:30] <nessita> ralsina: wanna skype/talk by phone? (no mumble in this laptop)
[18:30] <dobey> nessita: i'm sure he has gtk+, since so many things still require it, like sso :)
[18:31] <ralsina> sure, I'll call you
[18:31] <dobey> (or [s]he. /me doesn't presume to know)
[18:34] <csgeek> nessita: visual tool is empty and it throws an exception.    'SyncDaemonTool' object has no attribute 'get_shares_dir'
[18:34] <csgeek> (also.. thank you for all your help )
[18:43] <nessita> csgeek: oh, I know...
[18:43] <nessita> csgeek: in order to run latest magicicada, you need latests ubuntuone-client, which is available in our PPA
[18:44] <nessita> csgeek: you can certainly install our PPA and update the packages we provide, but you must know that since there may be some metadata upgradel, there is not wasy way "back"
[18:44] <nessita> csgeek: to install our PPA: sudo apt-add-repository ppa:ubuntuone/nightlies; sudo apt-get update; sudo apt-get upgrade
[18:45] <DiegoSarmentero> ralsina, hi! do you have news from lisette about the new design?
[18:45] <ralsina> DiegoSarmentero: there was an email a while ago, I havenot been looking really
[18:46] <DiegoSarmentero> ralsina, ahh you are right! sorry
[18:46]  * DiegoSarmentero reading...
[18:46] <csgeek> ah..  I think I'll wait it out for now.  I might upgrade once its done with the sync
[18:48] <nessita> csgeek: ok then
[18:48] <nessita> csgeek: if really curious, u1sdtool --waiting | less will show the detail of what is processin
[18:48] <nessita> g
[18:49] <nessita> DiegoSarmentero, ralsina: can I have reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-809363/+merge/67875 ?
[18:49] <DiegoSarmentero> nessita, ok... reviewing
[18:49] <ralsina> nessita: queuing
[19:06] <nessita> DiegoSarmentero: ping
[19:06] <DiegoSarmentero> nessita, pong
[19:06] <DiegoSarmentero> nessita, i was reading the changes
[19:06] <nessita> DiegoSarmentero: hi there! question, when you mentioned moving the folder from a tree view to a list, what widget in particular you mentioned?
[19:07] <DiegoSarmentero> nessita, it was from tree to table, it was the tree inside the folder tab
[19:08] <DiegoSarmentero> nessita, unless you want to collapse and expand each section
[19:08] <DiegoSarmentero> nessita, in that case it would be better to maintain the tree
[19:08] <rye> duanedesign, re: tomboy notes recovery - yes it is possible - tomboy keeps deleted notes in Backup/ folder, then there is a ubuntuone-couchdb-undelete.py script
[19:08] <nessita> DiegoSarmentero: I think we want to be able to collapse and expand. But suppose we don't need that, what other widget you suggested?
[19:09] <DiegoSarmentero> nessita, to expand and collapse i think that tree would be the best, it has that function already implemented... but with a table you can add that function through the code.... but it doesn't seems to have much sense
[19:10] <DiegoSarmentero> nessita, if you don't need that
[19:10] <nessita> DiegoSarmentero: my question is
[19:10] <DiegoSarmentero> nessita, i would said the best should be a qtablewidget
[19:10] <nessita> ah, I see
[19:10] <nessita> DiegoSarmentero: thanks! I look into this, since I need to fix the ugly buttons
[19:11] <duanedesign> rye: thank you
[19:11] <DiegoSarmentero> nessita, which buttons?
[19:12] <nessita> DiegoSarmentero: let me show you by PM
[19:12] <ralsina> nessita: got 5 minutes for another call? sorry :-(
[19:12] <nessita> ralsina: sure! just dial up
[19:12] <nessita> :-)
[19:25] <ralsina> nessita: I like your branch, looks good to me
[19:25] <nessita> ralsina: thanks!
[19:26] <DiegoSarmentero> nessita, yep, +1... but i could only review it looking at the changes, i wasn't able to download it here (my bad)
[19:28] <nessita> DiegoSarmentero: is ok, we can land this with one review but I wanted to start including you in this activities ;-)
[19:28] <DiegoSarmentero> nessita, thanks :D
[19:28] <nessita> DiegoSarmentero: can you set your vote in the merge proposal?
[19:29] <DiegoSarmentero> yep
[19:30] <ralsina> nessita: the equivalent of Ctrl-R on windows is F8 :-)
[19:30]  * ralsina remembers nessita asking
[19:31] <nessita> ralsina: uh?
[19:31] <nessita> ah!!!!
[19:31] <nessita> ralsina: thanks
[19:33] <ralsina> nessita: could I get an ok on the extremelytrivial branch with the officialist name: https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_810096/+merge/67878
[19:39] <nessita> ralsina: sure!
[19:39] <nessita> ralsina: is trivial, yes. But does it break any windows tests?
[19:40] <ralsina> nessita: not here
[19:40] <nessita> ralsina: I can't launch the remote VM since this connection is.... well, "borrowed"
[19:40]  * dobey wonders if the - is installed upside down on keyboards in argentina
[19:40] <ralsina> it's a noop unless you are passing the argument
[19:40] <ralsina> and before the branch, that was an exception :-)
[19:40] <nessita> dobey: no, ralsina just like to name his branches fix_something instead of fix-something :-)
[19:41] <ralsina> dobey: is _ bad? I can CHANGE!
[19:41] <nessita> ralsina: ok I will trust you on this one
[19:41] <dobey> ralsina: it can cause RSI :)
[19:41] <nessita> dobey: good point :-P
[19:41] <nessita> ralsina: approved
[19:41] <ralsina> nessita: cool, thx
[19:50] <alecu> nessita, ralsina: please review? https://code.launchpad.net/~alecu/ubuntu-sso-client/clean-up-disconnects/+merge/67882
[19:51] <nessita> alecu: sure!
[19:51] <ralsina> alecu: sure
[20:02] <nessita> alecu: code looks great, all green in linux, but I can't IRL test until I have an internet connection of my own. Approving, but someone else needs to run the tests in windows and try it IRL, I think
[20:03] <alecu> nessita, great, thanks.
[20:05] <alecu> nessita, ralsina: here's the exact same fix for ubuntuone-client: https://code.launchpad.net/~alecu/ubuntuone-client/clean-up-disconnects/+merge/67887
[20:06] <ralsina> alecu, nessita: I ran the tests on windows and they pass. What would be a good IRL test?
[20:06] <alecu> ralsina, make a sso bundle, point control panel at this sso, run control panel, see that it gets the user name from the webservice. Now close control panel and open it again, see that it gets the username again.
[20:06] <alecu> that means it worked.
[20:07] <ralsina> alecu: ok
[20:07] <alecu> nessita, I'll tackle #806655 now
[20:07] <alecu> nessita, did you do anything about it?
[20:07] <alecu> I mean: bug #806655
[20:07] <ubot4> Launchpad bug 806655 in ubuntuone-control-panel (and 1 other project) "Windows: SyncDaemonTool should provide a way of registering a callback for status changed notifications (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/806655
[20:08] <nessita> alecu: nothing at all!
[20:08] <nessita> alecu: do you know where to start?
[20:09] <alecu> nessita, any pointers would be appreciated.
[20:10] <nessita> alecu: so, syncdaemon will emit SYS_STATUS_CHANGED every time its status changes. Right now, the dbus_interface (linux) will register a listener for the events and define a handle_SYSY_STATUS_CHANGED
[20:10] <alecu> nessita, right, I remember that much.
[20:10] <nessita> alecu: I was thinking we need t do the same in whatever the equivalent of dbus_interface is on the windows side
[20:10] <nessita> alecu: then, instead of emitting a signalo like in the dbus side, we shuold call a callback
[20:11] <nessita> alecu: that the control panel (in particular) can set with whatever method it needs to be called with
[20:11] <alecu> great.
[20:11] <nessita> that makes sense?
[20:11] <alecu> perfect sense.
[20:11] <nessita> great
[20:12] <alecu> nessita, should I work on that, or should we work together on the unicode stuff we discussed earlier?
[20:12] <alecu> http://bazaar.launchpad.net/~mandel/ubuntuone-client/pyinotify-non-unicode/revision/1053/ubuntuone/platform/windows/os_helper.py
[20:12] <nessita> alecu: I would say you work on that, and see how far you can go today
[20:12] <nessita> alecu: I would very much like to sync with mandel before we dig into that code
[20:13] <alecu> nessita, it makes sense.
[20:21] <ralsina> alecu: +1 on the first one
[20:21] <alecu> goot
[20:30] <nessita> alecu: approving with the same disclaimer as before :-)
[20:37] <ralsina> alecu: what's a good test IRL for this second one?
[20:38] <alecu> ralsina, the same, but point the control panel to a syncdaemon bundle of this branch.
[20:38] <ralsina> alecu: ok then +1
[20:38] <alecu> ralsina, start control panel, see that it gets the settings from syncdaemon. Shut the control panel, restart it... see that the settings are read from syncdaemon again...
[20:38] <alecu> ralsina, cool!
[20:45]  * DiegoSarmentero is away: Not here... send me an e-mail! :P
[20:46]  * DiegoSarmentero is back (gone 00:00:04)
[20:46] <dobey> ugh, away announcements :)
[20:57] <dobey> alright, am off. have a good evening peoples!
[21:00] <nessita> ralsina: any idea why if I copy the same code that loads the qss in the main/windows.py file to the main/linux.py file, the css is not applied?
[21:00] <nessita> ralsina: but if I apply a string with the css, it's applied?
[21:00] <ralsina> nessita: missing import of the resource file?
[21:00] <ralsina> oh
[21:00] <nessita> ah, yes!
[21:00] <nessita> maybe!
[21:00] <ralsina> no, then no part of the css would work
[21:01] <nessita> no, the css is not load
[21:02] <nessita> is the css some how platform specific?
[21:02] <ralsina> nessita: shouldn't
[21:04] <nessita> ralsina: would you have some mins to help me debug?
[21:05] <ralsina> nessita: of course!
[21:05] <nessita> bzr branch lp:~nataliabidart/ubuntuone-control-panel/fix-810100
[21:05] <ralsina> nessita: could you push to a junk branch so I can see?
[21:05] <ralsina> ok
[21:05] <nessita> ralsina: you have a linux available?
[21:05] <ralsina> I have to start my linux though
[21:05] <nessita> ah
[21:05] <ralsina> shouldbe 1'
[21:05] <nessita> can you or is it too much trouble?
[21:06] <ralsina> none whatsoever. After dragging you across half of sso'snasty bits I owe you ;-)
[21:06] <ralsina> BTW: I have that working, I just need to figure out testing
[21:15] <nessita> ralsina: great!
[21:16] <ralsina> nessita: the qss is not being loaded (stylesheet is '' an empty string)
[21:16] <nessita> hum
[21:16] <nessita> I have no idea why :-(
[21:17] <nessita> the same code is used on windows
[21:17] <nessita> is not like I want to commit these changes, but they were useful to test in windows some style fixes
[21:18] <ralsina> yes, let me try a couple of things, it may not be too hard to fix
[21:22] <ralsina> nessita: if you load it like this, it works: http://pastebin.ubuntu.com/643537/
[21:22]  * ralsina has no idea why
[21:22] <nessita> ....
[21:22] <nessita> ok!
[21:22] <nessita> ralsina: thanks
[21:22] <ralsina> And it's a bit nicer, IMHO
[21:24] <nessita> yes, indeed
[21:24] <nessita> it works here too
[21:24] <nessita> thanks!
[21:33] <nessita> ok, what a "moved" day
[21:33] <nessita> (qué día movidito)
[21:34] <nessita> I will grab some fresh air, and will hunt some good luck :-D
[21:34] <ralsina> nessita: do you have 15 minutes before EOD?
[21:34] <nessita> (my phone line is still dead)
[21:34] <nessita> ralsina: sure!
[21:34] <ralsina> oh ok, no problem
[21:34] <ralsina> good :-)
[21:35] <nessita> ralsina: te leo
[21:35] <ralsina> With the wizard fix, I would love some testing tips, the changes are not very large, mostly on how things are called
[21:35] <ralsina> branch is here: http://pastebin.ubuntu.com/643537/
[21:35] <nessita> no is not! :-)
[21:36] <ralsina> Sorry, my laptop has turned into cold polenta for some reason
[21:36] <ralsina> argh
[21:36] <ralsina> better: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix-810053/+merge/67894
[21:37] <nessita> opening
[21:38] <nessita> ok, i quickly looked at it
[21:38] <nessita> you need testing guides for this branch?
[21:38] <ralsina> please
[21:39] <nessita> ralsina: you would like them today? :-)
[21:39] <ralsina> I can check that the callbacks in the SSOGui work
[21:39] <ralsina> nessita: ha
[21:39] <ralsina> nessita: I guess not ;-)
[21:40] <nessita> I can give you some, wanna call me? (or I can call you)
[21:40] <nessita> thing is I want to go to a public phone to make more reclamos to telecom
[21:40] <ralsina> sure, calling you!
[21:40] <nessita> before they stop answering the hot line
[21:42] <thisfred> nessita: why is it important again to put the expected value first in an assertEqual() ? My brain has a hd failure
[21:46] <alecu> thisfred, readability?
[21:47] <thisfred> alecu: yeah, but is that all? Since that's rather a matter of taste I guess
[21:47] <alecu> thisfred, no idea then :-(
[21:47] <thisfred> if you think of assertEqual as analogous to ==, then x, 3 reads more natural than 3, x
[21:47] <ralsina> I am stopping now for a little while. Nessita, expect the report very late tonight or very early tomorrow morning
[21:48] <alecu> ralsina, "nessita has quit"
[21:48] <thisfred> anyway, not super important, just getting yelled at by jdo for making unnecessary changes ;)
[21:49] <thisfred> and by 'yelled at' I mean politely questioned
[21:50]  * ralsina is lagged
[22:26] <alecu> aeodus!