=== yofel_ is now known as yofel [08:26] hello everyone! [08:26] oh, man I hate updating OSes... [08:48] * mandel out for 15 min errands [08:54] Goooood mooorning all. ;-) [09:03] good morning! [09:11] ralsina, morning! [09:11] hello mandel [09:35] ralsina, I'm of to do some errands in the bank :( [09:35] ralsina, I should be back in 20 mins or so [09:35] go err [09:35] before you do, any prgress on the watches? [09:36] ralsina, yes, should have an MP after mumble or so [09:37] mandel: awesome [09:46] mandel, are you still needing a review? sorry to forget about that [10:35] gatox, I have not pushed the fixes that nessita asked for, I'll ping you when done [10:35] I was hacing fun with something else... [10:35] mandel, ok! [10:35] :P [11:18] Holaaaaaa [11:29] * mandel walking the beast [11:42] good morning everyone! [11:57] mandel: ping [11:58] @ping === ehw is now known as ehw|asdf === ehw|asdf is now known as ehw [12:08] bug #123456 [12:11] ralsina: ping [12:14] nessita: pong [12:22] ralsina: hi there! [12:23] god morning nessita [12:23] have the link handy to the bug report about the double to_errdict? [12:23] I have to change work locations and I'm with my laptop [12:23] so I lost the "tab" :-) [12:23] nessita: I can probably find it :-) [12:24] I can't find it under ussoc, which is weird, I think you filed it there? [12:24] nessita: https://bugs.launchpad.net/ubuntuone-windows-installer/+bug/812468 [12:24] nessita, when you have a second please: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/close-on-license/+merge/73292 [12:24] gatox: sure! [12:24] nessita: no idea why I filed it under installer [12:24] ralsina: thanks [12:29] nessita, ping [12:30] mandel: pong [12:30] nessita, hello, you were after me, right? [12:30] mandel: I was pinging you a little while ago [12:30] mandel: yeah, I don't see the MP fixed, is LP correct? [12:31] nessita, lp is correct that is no the laste version, let me push it [12:31] I was looking at other things and forgot [12:31] ok [12:36] ralsina: how can I reproduce the bug? I mean, is the description of the bug the same thing you tried to have the second trace? [12:37] nessita: yes, the same thing, using ffff@ as email address and everything [12:42] nessita, lp finished parsing th enew version, feel free to take a look [12:47] gatox: comment added [12:47] nessita, ok === m_conley_away is now known as m_conley [13:00] me [13:00] me [13:00] alecu, ping [13:00] good morning all! [13:00] me [13:00] hi alecu [13:01] me [13:01] ralsina: standup? [13:02] eso! [13:02] DONE: reviews, almost done bug #834900, daily meeting [13:02] TODO: more reviews, finish bug #834900 [13:02] BLOCKED: nopes [13:02] NEXT: mandel [13:02] DONE: Fixed new line errors in https://code.launchpad.net/~mandel/ubuntuone-client/add-method-name/+merge/73400. Worked on the ancestors watches issue but I dont thing there is a bug for that, is there? [13:02] TODO: Finish fixing a number of tests related to the addition of the UDF ancestors in watches. [13:02] BLOCKED: no [13:02] next: gatox! [13:02] DONE: [13:02] Reset Password Page and Refactoring of password operations between Installer and SSO complete. [13:02] TODO: [13:02] Bunch of UI bugs. [13:02] BLOCKED: [13:02] No. [13:02] ralsina, go [13:03] DONE: release, installer test suite runs on linux, a few fixes, reviews, talked with lisette, fixed parrino's handoff bug in theinstaller. TODO: think about/implement systray icon, fix stderr BLOCKED: no [13:03] I forgot: worked on adding syncdaemon to autostart [13:03] And I am kinda BLOCKED in that I have my kid at home today, we'll see how that turns out [13:04] alecu: go? [13:04] me [13:04] alecu: go! [13:05] DONE: tested IRL the delayed hashing branch and worked on broken tests in it [13:05] TODO: review and check events workflow in windows [13:05] BLOCKED: got a sick kid today at home, may be unresponsive at times [13:05] mandel, pong? [13:06] ralsina, I've got a few ideas on the systray icon as well, please let me know when you start working about it. [13:07] mandel: the dot should not be added in add_method_name but in the strings itself. The code style recommends adding an ending dot to every comment/message/log message, so usually a dev will write that in the same string, and we'll end up with 2 dots [13:07] alecu: sure! [13:07] alecu: we can stay talking after the mumble [13:07] alecu: but just FYI I am aiming at _minimal_ ;-) [13:07] ralsina, mostly on how we should adapt the notifications/messaging menu code to work with a systray icon. [13:07] ralsina, +1 [13:08] alecu: cool, was not intending to implement that now, but it's good to not do something that makes it impossible later :-) [13:08] mandel: comment added to the MP [13:08] nessita, I guess I can add and extra sentence rather than appending the calling method to it for future changes... [13:09] alecu, did you file a bug for the accentors issue? [13:09] ralsina, nessita another review please (Both branches are related): https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/reset-password-page/+merge/73522 - https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/reset-password-page/+merge/73523 [13:09] mandel, don't know what an "accentor" is, but I will fill it right now. [13:09] accentors? Are those the bad guys in the new Harry Potter? ;-) [13:09] alecu, hahaa [13:10] alecu, adding a watcher to the parent :P [13:10] gatox: on it in about 10' [13:10] mandel: you can strip the '.' if it's present and append the method name [13:10] ralsina, thanks [13:10] ralsina: when you have a minute, I would like to ask you about the errdict bug [13:10] nessita: is now ok? [13:10] sure! [13:10] mumble or here? [13:10] ralsina: here, typing now [13:12] so, if I recall the IRC backlog from Monday correctly (I read it while I was very sleepy), I think you mentioned this bug was related to the changes I made for login-email-password-for-everyone. I'm looking at the trace, and I'm not sure I can make that connection, maybe you saw something I'm missing? [13:12] nessita, uh? well I can do that too, a little ugly though. I prefer: Path should be unicode. Callign method 'blah' [13:12] nessita, with a . at the end ;) [13:13] mandel: sounds good [13:13] nessita: uness I misunderstood bzr blame, I think you added the except_to_errdict to the lambda in that branch [13:13] ralsina: checking [13:15] * mandel super quick lunch [13:17] ralsina: that's what bzr blame says, yes, though that was revno 764 (we're now in 771), I'm trying to find what branch was it [13:18] ralsina: right, I just fixed the indentation: [13:18] http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntu-sso-client/trunk/revision/766#ubuntu_sso/main/windows.py [13:18] ralsina: that code is in place since revno 675, added by mandel [13:18] Then it's probably something I added when doing the login_with_user_email [13:18] oh, right, it's mandel's fault! ;-) [13:18] ralsina: I can debug, no problem [13:19] ralsina: actually I like to do it, so I can clean up this a bit [13:19] nessita: cool, it's kinda icky [13:19] ralsina: but I would like to queue this up after I finish the controlpanel UI bugs? otherwise I may not hold my estimate for this Friday :-/ [13:19] nessita: ok [13:19] I will assign the bug to me [13:22] ralsina: when you have a new "free" slot I would like help debugging why a button does not get visually enabled when called setEnabled(True) (it can be clicked though) [13:22] nessita: if you can click it, it's enabled [13:22] ralsina: but is "visually" disabled [13:22] nessita: it's a styling issue, you will have mre luck with gatox [13:22] is greyed out [13:23] ralsina: the button is orange until I call setEnabled(False) [13:23] nessita: if you click it, it gets orange? [13:23] then, is greyed out. When later I call setEnabled(True), it can be clicked but never turns orange again [13:23] I can ask gatox to help me :-) [13:24] nessita: yes, he had the same problem while at the sprint [13:24] gatox: you remember how you solved it? [13:25] * gatox reading... [13:26] nessita, yes... where are you having that problem? [13:26] nessita, you have to refresh the style with unpolish and polish [13:26] gatox: in the controlpanel [13:26] isn't there any other "cleaner" solution? [13:27] nessita, nop...... that is the cleaner solution... the qt doc say it :P [13:27] the other solution is slower [13:27] gatox: have a link handy please? [13:27] nessita, to the doc? [13:27] yes [13:28] nessita, let me look for it... i usually ready it locally in the qt assistant...... wait..... [13:30] that certainly work, but I don't like it (can live with it though) === m_conley is now known as m_conley_away [13:31] ralsina, Chipaca, alecu, mandel, gatox: I'm using my laptop today, that has a for-ever-broken mumble, so I apologize and I would ask that we use skype for today, please [13:32] nessita: no problem [13:32] skype? are we pro-microsoft now? [13:32] * Chipaca trolls, randomly [13:35] nessita, i can find it in the doc...... but i'm sure i read it.... basically it says that you can reapply the style using setStyleSheet(styleSheet()) but this is slower because force several widgets to reapply the style... and the proper solution would be to call unpolish and polish for the widget that we want to reapply the style [13:36] gatox: ok, I'll google if I need more info, thanks! it worked, you know :-) [13:36] nessita, so the widget only will use the styles that are assigned to that kind of widget [13:36] right [13:37] nessita, ah wait... you wanted a link of how implemented or about what i was talking about (performance stuff)?? [13:37] i was looking for the second one [13:37] gatox: the second one, yes [13:37] ahhhhh....... well.... that one i couldn't find it [13:38] is ok [14:05] nessita, ok, sle tme start skype [14:06] alecu, nessita, ralsina, gatox have you guys started the cLL [14:06] not yet [14:07] ah.. ok [14:07] Chipaca: if you can attend to the meeting, can you start the call, please? [14:07] alecu, ralsina, gatox: skype? [14:07] nessita: starting it [14:07] nessita, ack [14:13] facundobatista, hope you don't mind that I've added you as reviewer for https://code.launchpad.net/~alecu/ubuntuone-client/hash-bash-dash-rash/+merge/73540 [14:13] alecu, ok === m_conley_away is now known as m_conley [14:43] alecu, Chipaca, ralsina: https://code.launchpad.net/~damiano-albani/ubuntu-sso-client/proxy-support/+merge/71170 [14:49] looking [14:52] nessita, this branch is ready for re-review: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/close-on-license/+merge/73292 [14:54] gatox: ack [15:11] alecu, ping [15:12] facundobatista, pong [15:12] Ok, off to lunch === ralsina is now known as ralsina_lunch [15:12] alecu, if _Hasher is in the middle of a time.sleep(), and I do cancel_if_running() on it, what will happen? [15:13] ah, you don't have the mutex there [15:13] facundobatista, the thread will wait for the sleep to end, then it will process the cancel as usual [15:13] facundobatista, and then pick the next item from the queue [15:13] facundobatista, by definition, the next item should be hashed *later* than the one waiting. [15:16] no_timestamp = None [15:16] item = (self.end_mark, no_timestamp) [15:16] alecu, ^, create a local variable just for readability? isn't better a comment? [15:18] facundobatista, yes: I always prefer readability in the code than in comments. Comments tend to get out of date easily; code stops working instead :-) [15:19] alecu, how you test that? [15:20] facundobatista, which assertion? [15:20] alecu, don't know, it's "useless" production code [15:21] facundobatista, most of the tests on the hash queue code is high level: it tests the integration of all the bits from outside the hash thread. [15:21] facundobatista, so it's not so easy to test just that bit of code. [15:22] facundobatista, perhaps I should make "no_timestamp" a constant? [15:22] facundobatista, also: what's your concern with "no_timestamp" being a local? cpu usage? memory consumption? [15:24] alecu, conceptual uglyness, but we agreee that we don't agree there [15:24] facundobatista, would making it a module level constant look better to you? I think it does to me. [15:25] alecu, as you wish [15:38] Any reason why if I copy over an updated version of a file, I no longer can use the link that I previously published? [15:42] ralsina_lunch, gatox: what should I tweak in order to set the focus in a lineedit in a screen? can I configure that in designer? [15:42] hi all [15:42] hi have a question [15:43] i can't restore my firefox bookmarks, how can i do ? [15:43] snap-l: what kind of file is it? [15:43] ubuntu one says "all files synchronized" [15:43] but no bookmarks in firefox [15:43] ... [15:46] nosedrum: can you please file a support request here? https://one.ubuntu.com/suppo [15:46] sorry, here: https://one.ubuntu.com/help/ [15:46] this not ubuntu one chatroom here ? [15:46] nosedrum: yes, it is [15:47] rye: you around? [15:47] facundobatista, I've fixed it by making it a constant, and added a new test for the patched OrderedDict. [15:47] facundobatista, I'm still not sure what you mean by "Conceptual Uglyness"... perhaps you would have liked the end mark to not carry a timestamp, or something like that. [15:47] nosedrum, could you please PM me your e-mail so that I could look up the account? [15:47] alecu: did we find any package to backport ordereddict? [15:48] nessita, set the focus?? [15:48] nessita, I ended up fixing the one that comes with twisted, so it works a bit more similar to the one in 2.7 [15:48] nessita, setFocus() is not what you need? [15:48] gatox: can I do that from designer? [15:48] alecu, no, I understand the needing of key/value in the new structure [15:48] gatox: I prefer not doing it programatically if possible [15:49] nessita, i'm not sure... but probably not from the designer, because you are grouping together several widgets... [15:49] nessita, so there isn't a rule of which one is going to gain focus over other widgets [15:49] gatox: that sucks! ;-) [15:50] nessita, mandel, ralsina_lunch: perhaps one of you guys wants to review my branch: https://code.launchpad.net/~alecu/ubuntuone-client/hash-bash-dash-rash/+merge/73540 [15:50] nessita: It was a PDF file that I copied over the previous file. [15:51] facundobatista: can you please refresh my memory re: public files links? I think we keep the same link for the same node, right? [15:51] nessita, AFAIR, the public link points to the node_id, but I'm not 100% sure [15:51] alecu: queued [15:51] facundobatista: so in which case will the node_id change? remove and re-add? [15:52] ralsina_lunch, buen provecho! [15:52] nessita, yes, and that happens for some editors [15:52] * alecu will be afk cooking for amelia [15:52] snap-l: how did you update the pdf file? [15:53] I copied over another file on top of it [15:54] facundobatista: shouldn't that preserve node_ids? ^ [15:55] snap-l, how did you copy it? [15:57] lunch === gatox is now known as gatox_lunch [16:01] facundobatista: Via nautilus === beuno is now known as beuno-lunch [16:05] nessita, this is what nautilus does for a "copy t.txt over other t.txt in /tmp/foo": https://pastebin.canonical.com/52054/ [16:05] nessita, as you can see, it's a new file the resulting one, that explains the behaviour snap-l sees [16:06] right [16:07] snap-l: so, nautilus copied the file using a temp file in between, which generates file system events that makes syncdaemon upload the file as a new file [16:07] facundobatista: And that's not the behavior I would expect. [16:12] ralsina_lunch: seems like your branch from yesterday (I finished the review late last night) is not landing [16:12] ralsina_lunch: did you check with sidnei if that tarmac is landing windows-installer branches? [16:15] nessita: haven't. Let's ask! === ralsina_lunch is now known as ralsina [16:17] nessita: it isn't but he will add it [16:18] mandel: ping [16:19] gatox_lunch: did you mention that close-on-license was ready for re-review? [16:19] nessita, yes [16:19] gatox_lunch: I don't see any new change to it. Not in LP nor branching. [16:19] mandel: same on your u1client branch ^ [16:30] @ping === gatox_lunch is now known as gatox [16:37] nessita, mmmmmm let me check [16:38] nessita, now is ready...... the server send me an error before [16:38] nessita, about my other 2 branches....... they should be merged together because they are related [16:38] nessita, really? [16:39] butI'm sure I pushe the changes, let mecheck [16:40] mandel: maybe I mis-checked? let me know [16:40] gatox: ok [16:43] nessita, I need to go for a little, 'Ill be back to check what is wrong [16:43] mandel: ok === mandel is now known as mandel_afaik [16:45] gatox: +1 on the 1st branch [16:47] ralsina, thanks [16:50] gatox: how shall I call setFocus? I'm trying [16:50] nessita, object.setFocus() [16:50] + self.ui.email_entry.setFocus(QtCore.Qt.OtherFocusReason) [16:50] but it has no effect [16:50] nessita, there is no need to pass a parameter [16:50] send* [16:51] nessita, just: [16:51] gatox: with no parameters it does not work either [16:51] self.ui.email_entry.setFocus() [16:51] right, already tried that, but the focus is not in that entry [16:51] nessita, where are you setting that code?? is the widget visible at that moment? [16:52] gatox: not necesarily [16:53] gatox: I'm now calling it in showEvent, and the focus is not set... any ideas? [16:53] nessita, i think that might be the problem...... you should set the focus to a visible widget [16:53] gatox: I'm now calling it in showEvent, and the focus is not set... any ideas? :-) [16:53] yes, you can't setFocus on invisible widgets [16:53] on showEvent, it's not "shown" yet [16:53] ralsina: I'm calling setFocus in showEvent... [16:53] it will be shown after you accept the event [16:54] nessita, are you calling it after forwarding the show event to the parent? [16:54] gatox: no [16:54] nessita, so it is still invisible [16:54] nessita, you should do something like: [16:54] def showEvent(self, event): [16:55] super(MyClass, self).showEvent(event) [16:55] widget.setFocus() [16:55] gatox: I have that exactly, and the focus is not there... [16:56] or Qtimer.singleShot(0, widget.setFocus) [16:56] ralsina, wowwwwww...... that is nassty! :P [16:56] I prefer the widget not have focus before doing that! ;-) [16:56] gatox: yes, but if that works, then we know it'sa timing problem :-) [16:58] nessita, do you have that code you are testing in any branch where i can take a look? [16:58] gatox: yes, cleaning it up before piushing [16:59] ralsina, i should leave right now if it is possible, i've to run to the university (just for today at this time)... i'll be submitting my changes at night if that is ok..... [16:59] gatox: sure [16:59] ralsina, thanks! i'll send you an email tonight [17:00] nessita, send me a link to your branch..... i'll take a look at it as soon as i can [17:00] bye all [17:00] gatox: bye, thanks [17:15] alecu: +1 on bash-mash-rash-flash [17:15] ralsina, naish, thanksh! [17:15] yoush welcomesh === beuno-lunch is now known as beuno [17:18] * nessita -> quick lunch [17:28] ok, the kid situation is getting complicated [17:28] I am going to take a break, and will work intermittently the rest of the day, but until late [17:28] I will probably get more done later when he's tired. I hope :-( [17:29] I will still be around, so talk to me or ask for reviews, or whatever, I will get it [17:48] ralsina: if you can, whenever you can: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/disconnect-logic/+merge/73555 [17:48] nessita: sure! [17:49] nessita, I've added needs fixing plus comments to https://code.launchpad.net/~damiano-albani/ubuntu-sso-client/proxy-support/+merge/71170 [17:49] alecu: thanks! [17:51] nessita, the code in that branch considers a few simple cases of proxy support; but it looks like it will fail with other simple cases, and it ignores the complex cases. [17:51] nessita, I've only commented on the failing cases; I won't comment on the complex ones because we won't be able to tackle them while using stdlib [17:52] alecu: right, my big concern is if this code will be an issue when we try to deal with the complex ones [17:52] nessita, I think that if we fix the other simple cases we can land this branch. [17:52] nessita, for the complex ones we should switch to libsoup. [17:53] which is not cross platform, afaik [17:53] nessita, probably :-( [17:54] ok, let's see what this contributor responds [17:54] nessita, perhaps we should try whatever qt provides that's similar to libsoup. [17:54] alecu: perhaps, yes [17:55] nessita, but anyway, till we get to that point, we can support this solution that will help a significant percentage of proxy users. [17:55] sounds good [17:55] nessita, my only concern is that sso will probably work, but we are not sure if syncdaemon will. [17:55] and that sounds even worse to me. [17:56] alecu: well, this code will only affect sso for now... [17:57] nessita, yup. But consider a user that uses u1 behind a proxy on a given computer for the first time. He is able to login or register, but then SD won't sync. [17:57] nessita, that's a lousy experience. [17:58] alecu: indeed, but since SSO is an "independent" project from u1client I think it makes some sense (for example, software center will be able to function for rating/reviews behind a proxy) [18:00] right [18:28] nessita: +1 on disconnect-logic based on tests + IRL testing but only a quick read of the code [18:28] ralsina: thanks! [18:35] ok, I'm leaving to uni [18:38] * nessita waves [18:38] see ya later crowd [18:39] hi all [18:40] as I can see, symlink doesn't supported yet?.. [18:43] Alukardd: yes, not supported yet [18:44] that's a pity [18:47] karni: and there is no way to sync with u1 files from partion other than home folder?! [18:51] hmm, rye / duanedesign ↑ ? [18:51] Alukardd: From what I know, only folders from your users home directory. [18:52] karni: heh :-( Thanks... [18:52] Alukardd, no, at the moment only folder within /home/ are supported [19:12] hi guys [19:13] just curious, what is that small gray horizontal bar in the middle of the U1 icon at the vertical menu in unity? [19:13] afeijo: that is a progress indicator [19:13] afeijo: shows how far along your sync is [19:13] ralsina, the whole day? [19:14] it is totally synced [19:14] should'nt show it then [19:14] afeijo: ok, that is probably a bug [19:14] thisfred: are you around? [19:14] I thought it would be [19:14] ralsina: I am [19:14] thisfred: if yes, afeijo's problem is a known one, right? [19:14] * thisfred reads backlog [19:15] ralsina: not really known to me, no :( [19:15] That's not to say it's not filed as a bug already [19:16] But I have not been looking at the client at all [19:16] if I may ask, when the windows client will be released? any date yet? [19:17] thisfred: I think I remember a bug where after you start u1cp, then you close it, the bar got stuck because the connection to unity was lost... it's foggy [19:17] afeijo: soonish [19:17] hehe [19:18] new accts get 2gb or 5gb? [19:18] ralsina: ah, yes, I remember something like that (different symptom though, but may have the same cause: clicking the unity launcher kills the progress bar) [19:19] thisfred, I confirm that, the bar is gone [19:19] afeijo: 5 [19:19] ralsina, I had 2 gb, about 1 month ago, it up to 5 [19:20] yeah, but then it will never come back. That *is* a known bug. I'm not really sure a stuck progress bar is related [19:20] thisfred, I will watch it after my next reboot, thanks === elopio is now known as elopio-lunch === m_conley is now known as m_conley_away === elopio-lunch is now known as elopio [23:21] * nessita is back to catch up with news