/srv/irclogs.ubuntu.com/2011/12/01/#ubuntuone.txt

=== gatox_xbox is now known as gatox
=== eu is now known as Guest87301
JamesTaitI can't believe it's that time already, but Happy December everyone! :)09:33
j0nrrye: re. yesterday 14:01 that's Ok, just happy its being looked at. Glad to have found a bug!09:46
mandelmorning all!10:11
mandeldobey, I though I did change all of the, I grep in ubuntuone tests returns all imports from testcases and not testcase...10:12
* mandel reboots10:43
=== yofel_ is now known as yofel
gatoxgood morning!10:58
* mandel back11:07
ralsinagood morning mandel11:07
mandelralsina, morning!11:07
gatoxralsina, hi!11:08
gatoxmandel, hello11:08
mandelgatox, hello11:08
gatoxmandel, can you believe that i was using unity3d when unity2d was all the time the love of my life????? jejejeje11:09
ralsinaunity2d rocks!11:09
gatoxralsina, trueeeeeeeee!!!!!!11:10
mandelralsina, gatox yeah, unity2d is the oly one that works with my two screens...11:10
ralsinaI mean, it *works*! ;-)11:10
mandelhahaha11:11
gatoxralsina, and it's fastttttttt...... believe..... my machine wasn't working very well..... and now is really something different...... it doesn't has all the compiz effects.... i think that help :P11:11
mandelralsina, may I have a review for https://code.launchpad.net/~mandel/ubuntuone-client/fix-817582/+merge/8392511:11
ralsinagatox: nothing is free11:11
ralsinamandel: on it11:11
mandelralsina, thx11:12
gatoxralsina, well..... but i didn't want compiz..... so..... WIN-WIN :P11:12
gatoxralsina, do you know how to configure the number of desktop in unity2d?? i'm not being able to find it with dconf and gconf-editor11:20
ralsinagatox: no idea, I am a 1-dsktop guy11:20
gatoxralsina, ahhhhh...... ok...... i always need to have 6 :P11:20
ralsinaI tend to have everything maximized except editors (usually side-by-side)11:20
ralsinaso desktops don't buy me much11:21
mandelralsina, oh, but I use vsplit in vim that does the same with diff buffers :)11:22
mandelgatox, you need to install the funny compiz settings tool to do it, which is not included by default, I think is called ccmanager11:23
gatoxmmmmm compizconfig-settings-manager?11:23
mandelgatox, that is the one11:24
gatoxmandel, i have that..... but that allows me to configure desktops in unity3d.... not 2d11:24
ralsinagatox: isn't unity2d using metacity?11:24
mandelgatox, oh, in 2d I have no idea..11:24
gatoxralsina, yes11:25
ralsinagatox: in which case, maybe this helps: http://superuser.com/questions/276372/multi-row-layout-for-workspaces-in-metacity11:25
gatoxralsina, crap.... i don't have that key11:31
ralsinagatox: maybe you have to install desktop switcher?11:32
gatoxralsina, maybe..... i'll take a look into that later11:32
gatoxbut i like unity2d! :D11:32
gatoxmandel, can you review this branch please: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/fix-double-navigation/+merge/8397311:46
gatoxeasy one11:46
mandelgatox, sure, you give me one yesterday and I forgot, do you remember which?11:49
gatoxmandel, nop.... but you can start with that :P11:50
gatoxjeje11:50
mandelgatox, ok11:50
mandelstupid question, but I read a comment in a review and I'm doubting myself, x of False == x, right?11:51
gatoxmandel, a review about my branch?11:51
* gatox is lost11:51
mandelgatox, no, about one of mine11:52
mandelgatox, so, x or False == x, right?11:52
ralsinamandel: not guaranteed11:53
mandelralsina, ein?11:53
mandelralsina, why?11:53
ralsina[] or False  da False11:53
ralsinax or False == x if and only if x is "true"11:53
ralsinaso it fails for '' [] {} and a bunch of other stuff11:54
mandelralsina, ok, the correct statements should have been x or False == bool(x)11:55
ralsinahmmmmmm11:55
mandelralsina, which has the same meaning in an if..11:55
ralsinamaybe11:55
mandelralsina, since the or operatior will do bool(x) or False == bool(x)11:55
ralsinait depends on how you are using it in an IF11:55
ralsinabut yes, that one seems to be true11:56
mandelralsina, for example mocker gives errors when you do if object because bool() is called and you did not expected it.. stupid mocker..11:56
ralsinamandel: well, mocker is right :-)11:56
nessitahello everyone!11:56
ralsinamandel: and without the bool, it may or may not have the same result in the if depending on how you use that thing11:56
ralsinahello nessita11:57
mandelralsina, yeah.. true, but I don't want all of use to be writing if bool(blah) when python does it for you :P11:57
mandelnessita, buenos dias!11:57
ralsinamandel: for example: if x is None is nto the same as if x or False is None :-)11:57
mandelnessita, all the branches where you added the a needs fixing have been updated11:57
mandelnessita, and sorry for yesterday, I though I did change the ubuntuone-client tests, what was the issue?11:58
ralsinamandel: maybe I am not getting the idea of *why* you are doing "x or False" in the first place11:58
nessitamandel: regarding u1client tests in windows vs udevtools?11:58
mandelralsina, you where doing it: https://code.launchpad.net/~mandel/ubuntuone-control-panel/auto-update-looping-call/+merge/8101511:58
mandelnessita, yes11:58
nessitamandel: they were not running if you don't have pyxdg installed in windows11:59
mandelnessita, I though I did remaplace testcase for testcases everywhere.. I suppose I forgot about that project..11:59
nessitamandel: yeah, is fixed now :-)12:00
mandelnessita, cool, nevertheless sorry :)12:00
gatoxralsina, do you want to make a ui review?? https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/misaligned-icons/+merge/8362012:01
mandelnessita, by the way is this list correct: https://bugs.launchpad.net/ubuntuone/+bugs?field.searchtext=&orderby=-importance&search=Search&field.status%3Alist=NEW&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&assignee_option=choose&field.assignee=mandel&field.bug_reporter=&field.bug_supervisor=&field.bug_commenter=&field.12:01
mandelsubscriber=&field.tag=&field.tags_combinator=ANY&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on12:01
mandelnessita, ouch, sorry let me get you a smaller url12:01
ralsinagatox: yay, fun!12:01
ralsinagatox: does it have pretty pictures?12:01
gatoxralsina, mmmmmm nop..... do you want me to add it?? .P12:02
gatox:P12:02
mandelnessita, this: http://tinyurl.com/c7nyqhq I'm working with that as the list of bugs I have to tackle, is it correct?12:02
nessitamandel: looking...12:02
ralsinagatox: usually it's a good idea, because it's hard to see before/after12:02
nessitaAAAAHHHH what happened to launchpad?!?!?!12:02
mandelgatox, in https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/fix-double-navigation/+merge/83973 can we use %r for the logging?12:02
ralsinanessita: new bug list UI?12:02
gatoxralsina, ok..... give me just a sec12:02
mandelnessita, yeah, is ugly12:02
mandelnessita, it looks like a web 2.0 now :(12:03
nessita:-/12:03
gatoxmandel, ok..... changing that12:04
mandelgatox, also, in the test you have things like  self.assertEqual(self.controller.view.count_back, 4) what does that 4 mean? can we have it as visitedPages()[0] or something like that?12:04
mandelgatox, mainly so that variable name gives some information of what is being asserted :)12:05
gatoxmandel, ok, i'll made: times_visited = 412:05
mandelgatox, much nicer :)12:05
mandelgatox, same for  self.patch(self.controller.view, "current_user_page_id", 20)12:06
gatoxmandel, great..... on it12:07
mandelnessita, I believe that bug 806701 after your SDToolProxy refactoring, am I right?12:07
ubot4Launchpad bug 806701 in ubuntuone-client "Windows: if SyncDaemonTool fails, the error is not propagated to the caller (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80670112:07
nessitamandel: indeed, sorry I got distracted buy a private  ping, one sec12:07
mandelno worries :)12:08
gatoxralsina, now it has pretty pictures: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/misaligned-icons/+merge/83620 :D12:10
nessitamandel: that's the list I also have, I will add more items soon, seems like you're running out of them? :-)12:11
mandelnessita, I've been working hard :)12:11
nessitanice!12:11
nessitamandel: there is one issue I would need you to debug, there are a couple of reports of people saying that U1 will not work as non-admin user12:12
nessitamandel: is there any chance you install U1 using the latest installer, and then on a non-admin account you try to set it up, using the wizard and all?12:12
mandelnessita, do you remember the branch in which you did the refactoring of SDToolProxy, I'd like to link bug 806701 to that one so we keep track of it12:12
ubot4Launchpad bug 806701 in ubuntuone-client "Windows: if SyncDaemonTool fails, the error is not propagated to the caller (affects: 1) (heat: 6)" [Medium,Fix released] https://launchpad.net/bugs/80670112:12
mandelnessita, sure, I have vms ready for that, give me the bug and I'll work with it12:13
nessitamandel: sure! go ahead. The name was, I think, u1sdtool-multiplatform12:13
mandelnessita, thx12:13
gatoxmandel,  you caan re-review this one now: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/fix-double-navigation/+merge/8397312:15
mandelgatox, on it!12:15
nessitamandel: 2 things: 1- what do you mean in the last comment of https://code.launchpad.net/~mandel/ubuntuone-control-panel/auto-update-looping-call/+merge/81015 ? minimized is a parameter so it can be either true or false, and we can't assume it will be one specific12:20
mandelnessita, I agree with that statement, but just before that line we have a 'if not minimized:'12:21
nessitalooking12:21
mandelnessita, so, for use to go inside the if we must have a bool(minimized) == False, right?12:21
nessitamandel: never ever use bool() on something12:22
nessitathat indicates API dirtiness12:22
mandelnessita, I was saying that we have a minimized == False12:22
mandelnessita, what is a API dirtiness?12:22
nessitamandel: you're right, that logic is messed up :-)12:22
nessitamandel: when you allow people passing anything and treating that as a bool12:23
mandelnessita, oh, true :)12:23
mandelnessita, If you want I can put it back, but I though it was cleaner without the or..12:23
nessitano no12:24
nessitalet me see the context, you're right12:24
duanedesigno/12:24
nessitamandel: while I see that, the next thing is that I asked you to change the args in https://code.launchpad.net/~mandel/ubuntuone-windows-installer/put-migration-together/+merge/78995 but then I proposed a branch when I was eoding yesterday because I needed that working for something else12:25
nessitamandel: so is likely you'll have a conflict if you merge trunk in... but a simple one12:25
mandelnessita, ok, let me merge and see if it brakes :)12:25
mandelnessita, just did the merge with trunk and pushed it, revno 11112:28
nessitamandel:  your change to controlpanel re: minimized looks good, and makes sense, sorry for missing the context (I was just looking a diffed lines)12:34
mandelnessita, no problem, happens to all of use :)12:34
nessitamandel: question, why do you do "icon.auto_update_lc = auto_update_lc"12:35
mandelnessita, let me look12:36
mandelnessita, to have access to the looping call from the object in case i want to stop it, I could have done that in a single line12:37
mandelin line 5912:37
mandelnessita, I'll change that12:37
mandelis ugly like that12:37
mandelnessita, I have to have an early lunch so I'm off now, please can you write any comments and I'll fix them before the weekly meeting12:39
nessitamandel: yeah, anyways I will wait for alecu to finish the review first12:39
mandelnessita, ok12:39
* mandel lunch12:41
=== Guest87301 is now known as adorilson
alecuhello!13:29
alecumandel, ping: was this the branch that needed reviews? https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/8326713:29
nessitaHOLA alecu!13:30
alecuHOLA nessita! how were your vacations???13:30
nessitaalecu: THE AWESOMEST OF AWESOME13:30
alecunessita, ultramegacool!13:31
* alecu can't wait for his vacations. They won't be as awesomest for sure, but can't wait anyway.13:31
nessitaalecu: when are you taking them?13:31
alecunessita, starting Dec 16th, till January 1st13:32
gatoxalecu, this branch needs a re-review from you: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/not-validated-account :P13:46
alecugatox, thanks, looking.13:47
pmatulishave a puzzling situation here13:53
alecugatox, I'm getting an error running the tests on that branch:13:53
alecugatox, I've pasted it in the proposal: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/not-validated-account/+merge/8305513:53
nessitamandel: https://code.launchpad.net/~mandel/ubuntuone-windows-installer/put-migration-together/+merge/78995 is approved but it needs some cleanup before you land it, ok?13:53
gatoxalecu, :S checking....13:53
alecunessita, ralsina: I'm taking Amelia to kinder, will be back before our team meeting.14:05
nessitaalecu: ack!14:05
nessitagatox: is this still opened for some reason? https://bugs.launchpad.net/ubuntuone-client/+bug/89099014:12
ubot4Launchpad bug 890990 in ubuntuone-client "Error: UnicodeDecodeError ascii (affects: 16) (dups: 9) (heat: 84)" [Critical,In progress]14:12
nessitaralsina: quick question, did you manage to fix installer suite on husdon?14:13
ralsinahudson?14:13
ralsinahusdon?14:13
gatoxnessita, sorry..... i didn't see it in my queue: https://bugs.launchpad.net/~diegosarmentero/+assignedbugs?field.searchtext=&orderby=-importance&field.status%3Alist=NEW&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.bug_reporter=&field.bug_supervisor=&field.bug_commenter=&field.subscriber=&field.tag=u1-14:13
gatoxzomg-windows&field.tags_combinator=ANY&field.status_upstream-empty-marker=1&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on&search=Search14:13
nessitagatox: weird, it should be14:14
nessitalet me check my gatox's queue link :-)14:14
nessitagatox: this is the queue I have for you: goo.gl/aXzLb14:19
gatoxnessita, new one?14:19
nessitagatox: no the same one I always had...14:19
nessitagatox: perhaps you're still filtering for u1-zomg-windows?14:19
gatoxnessita, this is the last one i have: http://is.gd/ZaEsDa14:20
gatoxsaving the new one now14:20
nessitagatox: oh, yes, that's filtering for zomg-windows14:20
nessitagatox: perhpas I created it and forgot to give you the link?14:20
gatoxnessita, ok..... i'll check that bug once i fix a branch i have for review..14:21
nessitagatox: sure, the bug perhpas just needs closing14:22
gatoxnessita, yes....14:22
gatoxnessita, but i'll test it before14:22
* mandel back14:37
mandelalecu, ping14:44
alecumandel, pong14:56
mandelalecu, lets talk after the weekly call :)14:56
* alecu runs to fetch the laptop to start mumble for the team meeting.14:56
mandelnessita, done the cleaning and approved, thx for eh reviews!14:57
nessitamandel: ack!14:57
ralsinateam call in 3'!14:57
ralsinanessita, alecu, dobey, mumble!15:02
nessitaralsina: can you hear me? (I can't hear you)15:02
ralsinaI haven't said anything :-)15:03
mandelnessita, did you try to say something? I could just see the red lips15:03
nessitamandel: can you hear me?15:03
mandelnessita, no :(15:03
mandelnessita, yo te escucho, no te oigo jejeje15:03
* mandel has a terrible sense of humor..15:04
mandelalecu, https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/83267 fixed :)15:05
joshuahooverdobey: is there anything outside of u1 that's blocking us in p right now? (prepping a weekly ubuntu release report)15:30
dobeyjoshuahoover: twisted reactor; but i'm working on it15:30
joshuahooverdobey: thx15:31
nessitaralsina: you left? :-)15:59
ralsinagoing back then :-)15:59
duanedesignwas the fix for -  Web Client Error 400 'Invalid consumer.' - removing the token from the  windows box?16:00
* gatox lunch...... brb16:03
dobeyralsina: 1:1 in ~22 min right?16:09
ralsinadobey: yes16:09
dobeyok; stupid google16:09
ralsinadobey: I seem to not have it scheduled (again)16:13
dobeyralsina: google tells me it is at 9:30 AM16:14
ralsinadobey: it's just not n my calendar16:14
dobeyralsina: it's in the online services calendar, no?16:14
dobeysigh, and the dual sign-in thing is dumb16:15
dobeywhy is it in the online services calendar? none of the other 1:1s are :P16:16
mandelalecu, dobey: telnet miku.acm.uiuc.edu16:24
* mandel is back at the non-stop nym cat ...16:24
alecumandel, lol!!!!16:24
dobeyeh16:25
dobeynessita: hrmm, i think we need to change the version on all the stable-3-0 branches to 2.99; otherwise all the releases will have to be 3.0.x16:36
nessitadobey: what's the problem with releasing 3.0.x?16:39
dobeyfew minutes; 1:1 now16:41
nessitadobey: sure!16:43
nessita@ping16:45
ubot4pong16:45
dobeynessita: well, releasing it as 3.0.x implies it's already finished and we're only fixing bugs in it :)16:49
nessitadobey: I see. Let me think about that a couploe of minutes16:52
nessitadobey: so, since the numbering is a convention, can't we ship new features the same on 3.0.x versions?16:53
dobeynessita: technically we can do whatever we want, yes; but that's not the convention16:54
dobeynessita: it feels weird for me to move code from one project to another, in a 3.0.x release, for example :)16:55
nessitadobey: what do you mean exactly with moving code from one project to another? between stable branches?16:56
dobeynessita: yes; i want to move libsyncdaemon into libubuntuone for example (as well as refactor it to be more async and easier to use)16:56
dobeyso moving code, breaking abi, changing UI, etc… all feels weird16:57
nessitadobey: ok, let's downgrade then, makes sense16:57
dobeynessita: should i do them all, or do you want to do some of them too?16:58
nessitadobey: I can do some :-), wanna split the list?16:59
dobeynessita: if you wish; doesn't matter. they are trivial changes, so i can do them all if you have other stuff to do17:00
nessitadobey: ok then, let's do that17:00
nessitaso I can finish setting up all these VMs17:01
dobeyok17:01
* mandel needs to learn how to cache the windows updates so that all his vms get updated fast..17:09
dobeybbiab, lunch and appointment17:23
nessitalunchtime!17:25
mandelok, EOD for me17:36
gatoxmandel, ok..... bye mandel!17:36
mandelgatox, we do alcus reviews tom and work on FF then?17:36
gatoxsee you tomorrow on FF17:36
gatox:P17:36
mandelgatox, I want to release it tom if we can :)17:36
gatoxmandel, roger that17:36
mandel:D17:36
gatoxmandel, yes we can! :D17:36
gatoxmandel, hey! do you want me to get up earlier so we have more time to work together tomorrow?17:37
gatoxit's not trouble for me17:37
mandelgatox, nah, don't worry :)17:40
mandelgatox, I'll do reviews in the morning for alecu17:40
gatoxmandel, ok17:41
=== beuno is now known as beuno-lunch
ralsinaalecu: ping 1-1 when you are ready18:05
alecuralsina, pong18:05
ralsinaalecu: in mumble?18:05
ralsinaalecu: lost you!18:07
gatoxralsina, do we have in the calendar our 1-1 yet?? i don't remember if you create it18:07
alecuralsina, I don't see it connecting at all...18:07
ralsinagatox: I should have, let me check18:07
ralsinaalecu: you appear and diappear quickly18:07
=== beuno-lunch is now known as beuno
nessitawho do I owe reviews to?19:33
gatoxnessita, i have a branch for review about a bug you mentioned once if you want19:35
gatoxit's short19:36
nessitagatox: shot!19:36
gatoxnessita, https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/fix-double-navigation/+merge/8397319:36
gatoxbefore you tell me: TESTSSSSS!!!..jejje with a little modification in the fake... the previous test works fine19:36
gatoxi wrote the previous test too19:37
nessitagatox: the simple fact you start with a disclaimer makes me doubt :D19:37
gatoxand added one more test now19:37
gatoxnessita, jejeje trust me.... i'm telling you to avoid stress19:37
gatoxjejee19:38
nessitalol19:38
dobeyin soviet russia, test runs you!19:39
gatoxjejjee19:39
dobeynessita: ok, i think all the stable-3-0 branches are version 2.99 now :)19:48
nessitadobey: great, thanks!19:49
gatoxEOD..... see you tomorrow!! :D20:16
alecubye gatox!20:21
nessitabye gatox20:21
alecugatox, I'll re-review your branch, see you tomorrow.20:21
gatoxalecu, ok.... let me know if the change has sense for you20:22
karniralsina: What's the gastronomic adventure :D?20:28
dobeykarni: exotic food20:32
karnidobey: sounds exciting. and scary ;)20:32
dobey brb; gotta run to post20:40
dobeyback20:54
nessitadobey: can I have a quick review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/package-utils/+merge/84173 ?>20:55
nessitadobey: I will update the packaging dailes after that20:55
dobeymeh, the trac for twisted is so slow no20:56
dobeynow20:56
dobeynessita: +121:03
nessitathanks!21:03
nessitaok, I'm off21:17
nessitabye all!21:17
=== gatox is now known as gatox_away
=== gatox is now known as gatox_away

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