=== theshining_ is now known as theshining === m_conley_away is now known as m_conley === m_conley is now known as m_conley_away [07:56] morning all! [08:23] Good morning all! :-D === cparrino is now known as cparrino-QBR [11:13] good morning! [11:14] ralsina, morning! [11:14] ralsina, and.. I hate unity trunk, they changed again something in the preview container that broke the payment preview.. I already fixed it but it is a PITA [11:17] mandel: :-( [11:17] mandel: well, our trunk is broken too [11:17] ralsina, what?! how come? [11:18] something in the sync menu breaks when there is no sync menu installed [11:18] ralsina, unity trunk I don't care,, but ours? how did that happen? [11:18] ralsina, mierda :( [11:19] mandel: surely an easy fix, I am about to do it [11:19] mandel: plus nightlies are broken in Lucid, which affects the server guys, so I am looking into that as well [11:19] mandel: and apparently windows is very broken [11:19] mandel: so, FUN DAY! [11:20] ralsina, I'm not that busy with the preview, send some bugs my way :) [11:29] good morning! [11:30] mandel: sure will [11:30] good morning gatox [11:30] ralsina, hi [11:35] gatox: do you have a lucid box? [11:35] gatox: it seems like the sync menu breaks when there is no sync menu: http://pastebin.ubuntu.com/1273005/ [11:37] ralsina, no...... no lucid..... i can install one if you want [11:38] gatox: if you can just start a branch fixing that one, and pass it onto facundobatista to test, that's good enough [11:38] ralsina, ok.... i'll do that [11:39] the problem seems to be with dbusmenu, not the sync menu [11:40] gatox: yes, but dbusmenu is only used by the sync menu, isn't it? (I don't know) [11:41] ralsina, yes [11:41] gatox: the import is guarded, but then the usage is not [11:41] gatox: so the import fails, sets a flag, and the flag is ignored :-) [11:42] ralsina, we are using the flag for the other part of the menu, i'll extend it to take care of that part too [11:43] gatox: awesome [11:47] gatox: also, trunk on P: http://pastebin.ubuntu.com/1273024/ [11:47] gatox: I think you can do both in one branch [11:47] ralsina, yap :S === mmcc_ is now known as mmcc [12:01] ralsina, any bus numbers for me [12:02] mandel: let me find a special one :-) [12:02] ralsina, I don't want bug #1 thx ;-) [12:02] Launchpad bug 1 in Ubuntu "Proprietary operating systems have a majority market share" [Critical,In progress] https://launchpad.net/bugs/1 [12:03] mandel: ok bug #3 ;-) [12:03] Launchpad bug 3 in Launchpad itself "Custom information for each translation team" [Low,Fix released] https://launchpad.net/bugs/3 [12:03] * ralsina loves vague mission statement bugs [12:04] ralsina, I write by bugs as todos :) [12:04] ralsina, can you review it? https://code.launchpad.net/~diegosarmentero/ubuntuone-client/menulucid/+merge/129169 [12:04] gatox: sure [12:05] let me know if we need two reviews for that [12:07] gatox: I would like facundobatista to take a look [12:07] ralsina, ah right [12:07] facundobatista, can you please review this one? https://code.launchpad.net/~diegosarmentero/ubuntuone-client/menulucid/+merge/129169 [12:08] ralsina, gatox, do you guys have any idea of why the following is not merging: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/u1-cp-publishapi/+merge/128316 [12:08] mandel, i was about to check that..... because those tests are working here [12:08] gatox, from the diff, if you do [12:09] + Menuitem = object [12:09] and then [12:09] gatox, yes, same here, which is weird [12:09] + self.go_to_web = Menuitem() [12:09] facundobatista, i know.... fixing that [12:09] flakes fail [12:09] gatox, you know what? [12:10] facundobatista, i thought you were talking about something else [12:10] facundobatista, what happend? [12:10] gatox, from the diff, if you do [12:10] + Menuitem = object [12:10] and then [12:10] + self.go_to_web = Menuitem() [12:10] gatox, how do you then do [12:10] 41 self.go_to_web.property_set(Dbusmenu.MENUITEM_PROP_LABEL, GO_TO_WEB) [12:10] ? [12:10] facundobatista, if you look at the bottom of the file, we have: UbuntuOneSyncMenu = UbuntuOneSyncMenuLinux if use_syncmenu else DummySyncMenu [12:11] so, if we reach the point to do: [12:11] gatox, I mean, object() has no property_set [12:11] Menuitem = object [12:11] that part of the code is never used [12:11] gatox: shouldn't that be TransfersMenu()? [12:12] gatox: so, you replace the base class of TransfersMenu but you use the actual class everywhere? [12:13] ralsina, the only place where TransfersMenu is being used, is inside UbuntuOneSyncMenuLinux..... and if the import fail and we have use_syncmenu = False the real use_syncmenu = False is not used..... so we are not using TransfersMenu either [12:13] gatox: yes, but that code feels bad :-) [12:13] the real UbuntuOneSyncMenuLinux i mean [12:13] gatox, ralsina, what was the exact error, getting an import error from gi.repository? [12:13] gatox, if you have code that is broken, and never used, *delete it* [12:13] gatox: if it's not used, then why set it? [12:14] facundobatista, it is used...... not in lucid [12:15] gatox: is there any chance of Menuitem being instantiated from object? [12:15] ralsina, the other thing was to declare TransfersMenu and DummyTransfersMenu inside an if using use_syncmenu..... but it seems ugly...... maybe it just my opinion [12:15] ralsina, sorry.... didn't understand that question [12:16] gatox: you are setting sometimes Menuitem = object and in other cases Menuitem is something else [12:17] gatox: so you have a class that sometimes has an API and sometimes doesn't [12:17] gatox: which is an accident waiting to happen :-) [12:17] gatox: I would be happier if it were sometimes the real thing and sometimes the dummy but always with a consistent api [12:18] ralsina, yes..... if we assume that TransfersMenu is going to be used for someone else instead of UbuntuOneSyncMenuLinux...... do you prefer the option with the class declared inside an if? [12:18] gatox: yes [12:18] ralsina, ok.... i'll change that [12:19] gatox: or declare it always and use an if to assign it to the "real" name? [12:19] gatox: either way is good [12:19] gatox, wasn't simpler to do this? class TransfersMenu(Dbusmenu.Menuitem if use_syncmenu else object) [12:20] facundobatista, but that has the same problem that ralsina was saying [12:20] * facundobatista reads backlog [12:21] gatox, +0 to the both classes definition inside a big if [12:22] facundobatista, i don't like that option either..... but...... [12:22] facundobatista: I don't love it either but having an instantiable object feels just bad :-( [12:22] facundobatista, ralsina gatox, why not leave it like it is, do the import from within the UBuntuOneSyncMenuLinux with a try except and then if you got an except set it to none and check when needed [12:23] gatox, ralsina, the cleanest way to do it is to import the class you'll use from two separated files, but this is good enough [12:23] mandel, +1 [12:23] i like mandel suggestion [12:23] facundobatista, gatox: I trust your good taste guys :-) [12:27] ralsina, I'm done with the preview changes, any bugs for me.. please... [12:30] mandel: in 1' [12:31] mandel: bug #1065272 [12:31] Error: Launchpad bug 1065272 could not be found [12:31] mandel: should be easy-ish [12:32] ralsina, facundobatista here is the branch using mandel 's suggestion (which i think is the cleanest one): https://code.launchpad.net/~diegosarmentero/ubuntuone-client/menulucid/+merge/129169 [12:32] mandel: then bug #1062729 which is semi-trivial too [12:32] Launchpad bug 1062729 in Ubuntu One Client "sync menu uses wrong upload size for percentages" [Undecided,Confirmed] https://launchpad.net/bugs/1062729 [12:33] gatox: much nicer [12:33] * mandel ego +1 [12:33] hehehe [12:33] thx mandel [12:33] jeje [12:34] gatox: still feels bad to declare a whole class we will never use :-) [12:34] ralsina, but we use it! [12:34] gatox: not if it inherits object we don't [12:34] ralsina, in that case goes back to None [12:34] at the end [12:34] gatox: exactly, so, we define a class that inherits object and then we set it to None :-) [12:35] gatox: that code is like pedaling on a fixed bike! (but yes, alternatives are verbose and worse) [12:37] yay...... tests just ends..... couple of fails.... fixing [12:43] ralsina, ouch..... i'm having problems with the new OrderedDict :S...... fixing.... [12:46] gatox: really? [12:46] ralsina, yap..... i'm trying to figure it out why [12:47] maybe is something related to other stuff...... but failing with that (and just a coincidence) [12:51] ok, time for my lunch === mandel is now known as mandel|lunch [12:56] ralsina, i found the problem.... it seems that the OrderedDict from twisted doesn't have the proper method implemented to be used with for [13:04] gatox: how could that not trigger any test failures? [13:04] gatox: in any case, maybe t can be used with for k in d.keys() [13:04] ralsina, that's what i did [13:04] facundobatista, ralsina now is ready: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/menulucid/+merge/129169 and with the orderedict from twisted fixed too [13:05] gatox: cool [13:09] gatox, you keep changing the branch? [13:09] facundobatista, i found another problem with the OrderedDict, that's why [13:10] facundobatista, but now is final [13:10] gatox, sure? [13:10] facundobatista, 100% [13:10] gatox, ok [13:10] gatox: sorry about the ordereddict breakage! [13:11] ralsina, no problem.... it wasn't a big thing [13:11] pretty easy to find and fix [13:14] gatox: it's what happens when I try to fix bugs I can't see :-) [13:20] and ... http://t.co/VdYiSl8Z [13:20] congrats gatox, mandel, and the rest of the team who's late ;-) [13:20] \o/ [13:20] congrats everyone! [13:32] mandel|lunch, ralsina when you have a moment please: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/clickshare/+merge/129188 [13:37] gatox: sure [13:37] gatox: you proposed it against the wrong package ;-) [13:37] gatox: therefore 77390 lines (+76238/-0) 209 files modified (has conflicts) [13:37] ralsina, ohhhhhhh [13:38] ralsina, brain fart [13:38] ralsina, is a good thing that the weekend is close [13:39] ralsina: remind me the command to run u1sdtool on mac? [13:40] chaselivingston: well, it's u1sdtool I just have no idea where it lies ;-) [13:40] ralsina: i've found it, but when i try to run it i just get -bash command not found [13:40] chaselivingston: if you are in that folder, do ./u1sdtool [13:41] ralsina: ah, I think that's right [13:41] ralsina, mandel|lunch now in the proper project: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/clickshare/+merge/129199 [13:41] ralsina: hiya! my friend can't sign in to Mac beta. did check his username/pass by signing in to o.u.c. on mac, getting "The authentication failed" [13:41] hey, I forgot to say "hi everyone!" [13:41] ralsina: any hints? [13:41] alecu: hello! [13:42] karni: not really [13:42] ralsina: any logs he can provide/turn on? [13:42] ralsina: ok.. he changed his password, and it worked this time o_O [13:42] karni: MAGIC [13:42] ralsina: I don't know where the problem was, but yeah.. problem solved ;] [13:43] he did sign in to o.u.c with previous password no prob. [13:43] Kewl [13:44] FWIW he changed his password for the same one, and it worked afterwards. [13:44] also, congratulations on Mac beta launch, team! [13:51] dobey, ping [13:51] alecu: no need for 1-1, right? We talked already? [13:52] dobey, do you know why this branch is bouncing?? tests are ok for mandel and me but failing on tarmac: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/u1-cp-publishapi [13:53] ralsina: right [13:55] gatox: dobey is on vacation [13:55] ralsina, oops....... [13:56] ralsina, do you know if tarmac is running with an older version of qt or something? [13:56] gatox: we should try asking sidnei regarding tarmac [14:00] i am on holiday, but the version of qt in quantal is 4.8.3 [14:01] gatox: those are the same 67 failures in QThread I asked about in another cp branch of yours a week or two ago [14:01] dobey, thx.... keep enjoying your vacation.... i'll check what is going on..... [14:03] dobey: go away and vacate [14:04] dobey: or however the thing one does on a vacation is called [14:04] i am [14:05] gatox: +1 but get a good second review, mine was soft :-/ [14:06] ralsina, yes.... i would ask mandel|lunch when he finish his lunch === mandel|lunch is now known as mandel [14:06] now [14:06] jeje [14:06] gatox, you want what? [14:06] gatox, I finished in u1-internal first hehe [14:07] mandel, a small review please: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/u1-cp-publishapi/+merge/128316 [14:07] gatox, I already gave a +1 to that, tarmac is complaining about qthread [14:08] mandel: here's a real bug that's been pissing me for a while bug #1065513 [14:08] Launchpad bug 1065513 in Ubuntu One Control Panel "Double gathering info screen " [Medium,Triaged] https://launchpad.net/bugs/1065513 [14:08] ralsina, ok, I'll tackle the other one first, then will more to that one :) [14:08] mandel, sorry, this one: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/clickshare/+merge/129199 [14:09] mandel: ok [14:13] gatox, would be great if connect form pyqt could take extraparameters that would be appended to the ones emitted, like self.button.clicked.connect(self.set_name, '') I hate seeing lambdas everywhere ;) [14:14] mandel, i'm starting to notice that..... [14:14] jejee [14:14] mandel, but you can't do that in pyqt [14:15] mandel, or i'm not understanding what you say [14:15] gatox, I know.. that is why I used the second conditional, would be great if... [14:15] ahhhhhhh [14:15] gatox, it would make the api nicer :) [14:15] mandel, i thought you were requesting a change [14:15] no no no [14:15] * gatox start doing a pull request into pyqt because mandel ask [14:15] jejjee [14:15] lol [14:16] mandel: can you give me an example? Because there is quite a bit of flexibility in connect :-) [14:17] ralsina, for example: http://paste.ubuntu.com/1273294/ [14:18] ralsina, just to avoid using lambdas, but I agree with gatox, there is nothing we can do, right? [14:19] alecu, can you merge the following: lp:~mandel/avani/unity-lens-music.ubuntuone-purchase-take2 with your branch? [14:20] alecu, some of the string are wrong (the ui ones) and I fixed them according to the given design [14:23] mandel: ack [14:24] gatox, branch lp:~diegosarmentero/ubuntuone-control-panel/clickshare have you ran the tests on ! [14:24] mandel: did you see the new checkbox in th design? it's called "forget payment details" (or something like that) [14:24] gatox, on Q I meant, sorry for the ! [14:24] gatox: I was not planning on including that. [14:24] sorry [14:25] mandel, yes..... i'm on Q [14:25] mandel: I was not planning on including that. [14:25] alecu, wtf? when was there? and no, I have a design without it [14:25] mandel, what are you seeing? [14:25] gatox, let be > to a file [14:26] alecu, and I agree, we are not doing that, we don't even have that in the rest api, do we [14:26] * mandel does not want to have to do yet another ui element in nux [14:26] mandel: right! [14:26] mandel: it's on "the one and only 12.10 screen" [14:26] alecu, ok, you can ignore that [14:27] mandel: it's called "remember my credit card details" [14:27] alecu, let me fwd the real design [14:27] alecu, I already said no to that in the sprint :) [14:27] mandel: oh, awesome. [14:27] alecu, just sent you the real thing [14:29] gatox, http://paste.ubuntu.com/1273319/ [14:29] gatox, but maybe is my system that got bananas [14:30] mandel, i'm not being able to run lint for u1-cp in Q neither [14:30] mandel, but the tests pass [14:30] mandel, alecu: scared me there for a bit :-) [14:30] gatox, yes, that I know.. [14:31] ralsina, nah, I already made sure we were not doing that hehehe [14:31] I was very very clear [14:31] * mandel said no in nearly all languages => NO! [14:35] gatox, +1 hoping that if there are real lint issues tarmac will tell us we are stupid [14:36] mandel, na, they are not real if you check [14:36] gatox, I got lost in the verbal diarrhea that lint has. === m_conley_away is now known as m_conley [14:46] gatox, that sync menu has lots of blackguard compatibility issues => bug #1065272 [14:46] Error: Launchpad bug 1065272 could not be found [14:47] easy to fix though === m_conley is now known as m_conley_away [14:58] mumble in 2'? [15:00] briancurtin: just the question i was going to ask [15:00] brb [15:00] oh,mumble, right [15:00] sorry, been a little distracted [15:02] sorry, joining [15:02] gatox, alecu, mmcc mumble [15:22] mmcc, ralsina when you have a moment, another review: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/deflated-size/+merge/129215 (now i'm moving to windows) [15:23] brb, need to grab some coffee [15:23] briancurtin, i'm going to have lunch now..... and we can talk after that what do you need..... or tell me what you have been trying so i start debugging that here too.... do you agree? [15:30] briancurtin, alecu: I have assigned to you a few bugs that are the ones support is getting questions about [15:30] briancurtin: yours look easy [15:30] alecu: yours looks hard [15:31] ralsina: ack [15:31] ralsina: I don't see a welcome page bug in my assigned bugs, did you say you assigned it to me? [15:31] briancurtin: one interesting bit is that we are still getting valicert questions. I am not sure if we ever released with the cert bundled on windows. If we did, then it's not working thugh [15:31] mmcc: let me track it [15:31] also, gatox I am looking at your deflated-size branch now [15:32] mmcc, great, thankx [15:32] mmcc: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/1065514 [15:32] Ubuntu bug 1065514 in Ubuntu One Control Panel "Mac welcome screen" [Undecided,New] [15:32] btw, I think I'd like to look at client test running soon. it's slow everywhere, and totally broken on osx [15:33] mmcc: fixing it on osx and then getting a darwin jenkins souds like something we really should be doing [15:33] ralsina: yes. [15:34] and remember, team, the bug situation can always be worse http://feeds.gawker.com/~r/gawker/full/~3/1gnDhJEMjp4/heres-footage-of-the-live-roach+eating-contest-that-killed-the-winner [15:34] * mmcc is NOT clicking that link [15:34] thank god for readable URLs [15:34] ralsina: once we get this other win stuff figured out, i'll look at how the installer is built in regards to certs [15:35] briancurtin: awesome [15:35] briancurtin: since we are not going to do another release before the 4-beta ... [15:36] ralsina: that welcome screen bug -> I think he just means the control panel main window. it has a rounded white rect inside a grey background. Although I agree that it's not like the default mac look, it is on-brand, and there isn't a real guideline for 'mac look' anymore anyway. [15:37] gatox: i've been poking around the code and following log files and trying to see how this CredentialsError is coming up. i'm going to go back and look at the dirspec changes to binary finding, then try to debug SSO in isolation (inside credentials.py) again [15:39] briancurtin, ok..... lunch and i'll start trying to execute u1 in windows and track the bug === gatox is now known as gatox_lunch [15:39] gatox_lunch: sounds good [15:41] gatox +1 on that deflated-size branch, thanks [15:42] briancurtin: how are you running things when you hit problems? I'm about to kill my credentials and try running just control-panel [15:42] mmcc: i'll put my details on a paste in 1', too much to type here [15:43] briancurtin: cool ,thanks === m_conley_away is now known as m_conley [15:50] where do the logs go in windows? I see the special folder names in dirspec but I don't know where to find those in the filesystem :\ [15:51] mmcc: C:\Users\brian\AppData\Local\xdg\cache\ubuntuone\log [15:51] oh, that's hidden in the explorer? [15:52] mmcc: yeah i think AppData is hidden [15:52] mmcc: ok! [15:52] oh it's even hidden from 'dir', tricky! [15:52] mmcc: put that same explanation and a wontfix status [15:53] ralsina: ok [15:53] mmcc you can use %LOCALAPPDATA% in windows explorer (or something similar) [15:54] mmcc: still working on your doc...now u1cp is being weird as i try to document what im doing [15:54] ralsina: oh on second thought he probably means this welcome screen: https://lh3.googleusercontent.com/-wpAFFGeAPzKkY7KFyiNmj01uz7KKrnkzVqoy25LuoJuIdXU1tRKfOnjDASSKo6ycwLmAiCyL7RmjFufhU1KsCQmrdBkhikgC8fxRTY2CTH9R_W0hKI [15:55] the one that says welcome :) [15:55] hahaha [15:55] same response though [16:00] ralsina, gatox_lunch, please can you review the following: https://code.launchpad.net/~mandel/ubuntuone-client/spawn-cp/+merge/129230 [16:01] mandel, will do after lunch [16:02] mmcc: here's the setup: https://pastebin.canonical.com/76354/ [16:02] gym, I have no logs in %LOCALAPPDATA%\ubuntuone [16:02] gym was supposed to be hrm. [16:04] so briancurtin, are environment variables inherited by child processes on windows? [16:04] mmcc: this is what i was just thinking about 1' ago...no, you need to pass it on, otherwise apps get a clean slate [16:04] the import errors you see when just running cp by itself should be avoided because it uses 'python ' + sso whatever [16:04] oh snap [16:04] well there you go [16:05] the whole deal with pre-pending 'python' to the binary paths was so that we get the buildout python [16:05] yeah it kind of clicked in my head when i saw it complain about twisted [16:05] but if the child process doesn't have the path set right, then boom [16:05] and that's why sso-qt isn't working either [16:05] because sso uses the same trick to use the buildout python to run sso-qt [16:06] well that's what you get when a guy who has avoided windows for 30 years tries to write windows code :\ [16:07] mmcc: not your fault. i should have been on the lookout (also should have been able to diagnose this a while ago...) [16:07] mmcc: your changes were in dirspec, or was that just finding the binaries and not running them? [16:07] briancurtin: dirspec finds 'em, other code runs it [16:08] briancurtin: I'll dig up a list of places now. I think there is one in sso and one in cp or client [16:10] sso-client - utils/__init__.py, line 126 , and ubuntuone-client syncdaemon/utils.py [16:11] mmcc: yep i'm looking at the sso one now [16:11] just those two places. control panel imports from those places [16:11] should be the exact same fix in both places. they're almost identical [16:11] ok, brb with more coffee [16:44] ok, EOD, catch you all tom! [16:45] bye mandel! === gatox_lunch is now known as gatox [16:45] mandel, bye [16:47] why do we only have upload info for the sync menu and not downloads also? [16:48] mmcc, you should ask the designers i think [16:49] gatox: not just the ubuntu sync menu but also the control-panel sys tray one… [16:49] mmcc the data is the same [16:50] mmcc: syncdaemon collects the data, describes a menu and either u1cp or sync-menu draws it [16:50] ralsina: yeah, I was thinking maybe the decisions had been made differently for each case [16:50] mmcc, yes, what i mean is that the design team creates the spec like that..... let me give you the design doc in pm [16:51] since the sys tray one was older… [16:51] thanks gatox [16:52] that osx screenshot is just photoshop, right? [16:53] mmcc, yes [16:53] hmm, that slide set just says 'files that are currently in transfer', it doesn't say if they're uploads or downloads [16:55] ok, so as far as I can tell, using QWidgetAction to show a progress bar doesn't work on osx [16:55] mmcc, maybe lisettte can clarify that [16:55] even if I just show the progress bar (no layout, no label, just the bar), it shows nothing [16:56] gatox: what do you want me to clarify? [16:56] mmcc, can you send me an screenshot..... maybe we are having the same problem as in ubuntu where we needed to use the number instead the progress bar [16:56] lisettte, mmcc has some question about the upload/download info in the menu [16:57] lisettte: the flyout menu that shows current transfers, in the code now it only shows uploads, no downloads. was that intended, or should it show both? [17:00] gatox: I will get a screenshot for you in a minute, but it's easy to imagine - as it is, it will show the label and text percentage but no progress bar, and no space for one. if I only put in the progress bar, it just shows nothing (there's vertical space for a menu item but nothing is drawn there) [17:01] mmcc, yes..... the same problem..... so we need to use the saame code in mac as in ubuntu [17:02] mmcc: the mac client is for intel macs only right? no power pc support? [17:03] chaselivingston: yes, no ppc [17:03] mmcc: cool, thanks [17:04] gatox: which code is that? [17:05] mmcc, gatox: on mac we have the same problem as in ubuntu, the tray menu is actually out of process [17:06] ralsina: that seems unrelated, I'm confused [17:07] mmcc, is actually the same code.... if the progress bar doesn't work it should show just the text..... i'm debugging u1 on windows now..... if you can send me the screenshot it would be helpful [17:08] gatox: it does show just the text, that's the problem. I want it to show the progress bar [17:09] gatox: what are you debugging on windows now? [17:10] I have a branch mostly done that fixes the sync transfers menu on osx, and need to test it on windows. I was just hoping to also get the progress bars showing, but it sounds like that's a known problem with qt? [17:11] mmcc, ahhhh..... well.... i think ralsina is right about that...... and maybe is not possible to do it when integrating with the systray [17:11] mmcc, the credentials problem.... as was discussed in the weekly call [17:12] gatox: check with briancurtin, I think we figured that out while you were at lunch [17:12] mmcc, briancurtin really? please tell me [17:13] gatox, I still don't understand what ralsina said! :) The progress bar seems to be an issue with qt menus on macs, but whether or not it's in a separate process shouldn't matter [17:13] gatox: see the scroll back, it's because windows apps don't inherit environment variables, so prepending 'python' to the path does not give us the buildout python as intended [17:14] mmcc: i'm an idiot and mixed myself up...they *do* inherit [17:14] gatox, so if you start sso separately like briancurtin did, control panel will find it via the TCPactivation, but it'll eventually fail when sso tries to launch sso-login-qt [17:14] briancurtin: oh no :( and there I was all convinced it was solved [17:14] mmcc, the thing that is a different process it's because it seems that mac handle the systray menus in a different way (as ubuntu does)...... so you probably are not having a qt menu there...... but something that makes the qt menu compatible with mac, so you can't use all the things from qt [17:15] briancurtin, so i should keep debugging this? [17:15] mmcc: yeah, im sure its something around that, though. it really cant be anything else [17:15] briancurtin: well, I still think that's the source of the issues. I'd poke at the get_bin_cmd stuff and see if maybe prepending 'python -v ' gives clues or whatever [17:15] let me know if I can be of more use. for now I'll go back to sys tray stuff [17:16] gatox: i think for now you can probably go back to whatever else. if *this* doesn't turn out to be the real issue, then maybe i'll try to bring you back to the windows dark side [17:16] briancurtin, ok, no problem... i'll keep fixing syncmenu and shares issues...... let me know as soon as you need me :D [17:43] alecu, ping [17:44] ralsina, ping [17:47] * briancurtin lunch === yofel_ is now known as yofel [18:04] ok, so I can get the cp systray menu to work on osx the same way it should on linux - except that nothing will update while you hold the menu open like it does on linux, because of how Qt interacts with the osx menu system [18:04] mmcc, :S [18:05] gatox: yup [18:12] also, the menu items can't be set to disabled, because of a qt bug that will be fixed in 4.8.4: https://bugreports.qt-project.org/browse/QTBUG-25544 [18:13] * gatox is waiting for Qt5...... so excited! \o/ [18:16] hrm, I'm not convinced that their fix actually fixes this [18:17] mmcc, well.... at least digia has declared to improve the integration with mac [18:17] gatox, that's good news I guess. Doesn't help me fix this stuff now, though :\ [18:18] mmcc, jeje no..... for that you have to blame the people who didn't invent the time machines yet [18:19] gatox or I could blame the people who broke menu enabling in qt/mac, because apparently it used to work :) [18:19] or that [18:25] interesting, qt's systemtrayicon can be compiled with support for growl [18:26] can't tell if it was in the bottle from brew though [18:29] gatox: pong [18:30] oh yeah, I see what's happening here. Qt is re-creating the menu items on every update, even when I don't re-create the python menu items :( [18:31] alecu, hi....... i'm looking at this bug: https://bugs.launchpad.net/ubuntuone-client/+bug/1065272 [18:31] Error: ubuntu bug 1065272 not found [18:32] alecu, it seems that spawn_command_line_async is not supported in older versions..... but that was there from before..... although that report is not using the last version because we are not spawning ubuntuone-installer anymore [18:32] alecu, do you know what should be used for backward compatibility instead of spawn_command_line_async? [18:33] gatox: no. But I see that mandel has proposed a branch for that bug.... what do you think of it? [18:35] alecu, ahhhhh..... i had that on hold to start working on it.... i didn't know that mandel propose a branch for that because he show me the issue and it didn't has anything at that moment [18:35] * gatox refresh the page [18:36]