=== eu is now known as Guest5442 [09:37] Happy Fix It Friday everyone! :D [10:11] mandel, ping [10:14] davidcalle, pong! [10:15] mandel, hi. I have a Soup question you might be able to answer. When I need to set a timeout if the server takes time to answer, do I use timeout or idle_timeout? [10:20] davidcalle, so, idle_timeout is how long you want to keep the connection alive with the server without doing a thing, while timeout is the timeout you will have for the blocking IO [10:20] davidcalle, in your case you should use timeout [10:21] mandel, thanks, I wasn't sure about the difference between the two. :) [10:22] davidcalle, yeah, the names are confusing, idle_timeout is for long living connections, some apps need those.. which is a pita with proxy because some are configured not to let you have one [10:23] mandel, I see [11:14] good morrning! === gatox_ is now known as gatox [11:27] gatox: hola! [11:28] Chipaca, buenas [11:32] gatox, morning! [11:32] mandel, hi there! [11:33] mandel, congratulations about the proxy support! :P i forgot to tell you that [11:34] gatox, well, is just control panel, we have to do some serious work about auth proxies for sd, but we are getting there :) [11:34] gatox, and I'm really please that we are doing it on time hehe [11:34] mainerror, you have to start somewhere :P [11:35] gatox, autocomplete hehe [11:35] gatox, stop using ninja as your irc client ;) [11:35] OMG!! AGAIN! [11:36] mandel, jejejeje nono, i use xchat..... but i'm lazy and i tend to type only the first 2 chars of the name [11:36] gatox, :) [11:36] duanedesign, published the fixed maverick->lucid version to https://launchpad.net/~rye/+archive/ubuntuone-support [12:02] wtf!! unity just decided to log off [12:05] mandel, maybe it saw you tired :P [12:06] gatox, hehe I don't think we are at that point yet [12:06] mandel, jeje [12:09] ok, restarting system.. [12:19] you rock rte [12:19] rye* [12:20] duanedesign, well, now I will wait for dobey to see whether it can be integrated. The thing is if it is reverted, then the nautilus will hang for 2 minutes on startup, this cannot be fixed easily, If it stays, nautilus does not hang but sd does not work if the amount of files is big [12:28] buen día! [12:29] nessita, hi [12:29] nessita, finishing with your review [12:30] gatox: how is it looking? [12:30] (besides long :-/) [12:31] nessita, fine..... i only found one thing that i would suggest to change..... do you want me to tell you here or put it as need fixing? [12:31] gatox: tell me, I may start working on that [12:31] nessita, - In setup_account_page.py you are calling self._set_line_edits_validations() inside the initializePage method, and the function "_set_line_edits_validations()" has 2 connect operations inside, that lines (the connect ones) should be moved to _connect_ui, initializePage is called every time the wizard need to show that page, and i would recommend to not make the connects there. [12:32] gatox: looking! [12:32] (it I moved that initializePage, it was not my intention) [12:34] nessita, it or if? [12:34] if :-) [12:35] nessita, ahhhh yes.... that's the only thing i found [12:35] gatox: I'm looking at my diff (with meld :-P) and self._set_line_edits_validations() was added to def _set_translated_strings(self): [12:35] nessita, i used meld too! :P [12:36] nessita, ahhhh you are right..... too many colors confuse me [12:36] gatox: congrats! :-D let me show you a screenshot [12:36] ah, you saw it already? [12:36] nessita, yes [12:37] nessita, either way....... it's not a problem of your branch because it was like this before, but i think that would be better to do all the connects, in _connect_ui [12:37] gatox: when doing that code, I wasn't sure if [12:37] self._set_line_edits_validations() [12:37] self._register_fields() [12:37] should go into _set_translated_strings or _connect_ui [12:38] gatox: I even would like to get rid of those 2 functions, and have a single _setup_page [12:38] gatox: that would be called from the setup_page in SSOWizardPage [12:38] gatox: but if I do that, branch will grow up :-/ [12:39] nessita, yes... i'm not sure about putting everythingg together..... it's easy to read if you can identify clearly the scope of each function [12:40] some classes has little code in there, but for example setup_account has a couple of lines [12:40] gatox: yes, that also... anyways, another thing I was in doubt was that [12:40] title_page = TITLE_STYLE % TITLE.format(app_name=self.app_name) [12:40] self.setTitle(title_page) [12:40] self.setSubTitle(self.help_text) [12:40] should not be called in initializePage... what do you think? [12:41] nessita, mmmm if exist the chance that someone is going to do: setup_accont.app_name = "something else"; move_to_setup_account [12:41] yes [12:42] if not..... it's not necessary [12:42] i mean [12:42] the app_name should never ever change :-) but I will not change that in this branch [12:43] if that text could change during the execution of the same application, and we need that to be refresh just in case every time that page is shown, yes..... but in any other case, i would leave that in the __init__ or any inialization method and execute it only once [12:44] that's my opinionn [12:44] I understand [12:44] gatox: another thign I just see and is very wrong (and is causing a bug assigned to High to you) is this: [12:44] self.set_up_button.clicked.connect(self.overlay.show) [12:44] self.set_up_button.clicked.connect(self.set_next_validation) [12:44] inside initializePage [12:45] nessita, yes, that is not going to connect several times the same signal..... but i prefer to avoid that anyway [12:46] gatox: but it also wrong that both self.overlay.show and self.set_next_validation are connected to self.set_up_button.clicked [12:46] only the first one should [12:46] and on_user_registered should do self.set_next_validation [12:47] nessita, ahhhh.... i understand [12:47] gatox: that's causing bug #934502 [12:48] gatox: there is no need to do all the setup for the self.set_up_button there, right? that could be done in _connect_ui, no? [12:49] nessita, yes [12:49] ack, thanks! [12:53] nessita, review please https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/uife-padding/+merge/94450 [12:58] I think we should switch to php.. https://bugs.php.net/bug.php?id=61095 [13:00] gatox: yes! [13:00] mandel: hola! I started doing reviews for you last night... added a comment to the first one [13:00] nessita, sweet, let me take a look [13:01] mandel: is trivial, I must confess I saw that and stopped (it was very late here, will continue now) [13:01] nessita, no worries, I'm happy you saw that and wrote the comment :) [13:01] nessita, I hate to leave #HACK comments around.. [13:15] ralsina: got my email for the review? [13:25] hello! [13:25] hola alecu [13:41] mandel: ping [13:41] nessita, pong [13:42] mandel: can you please mark all the strings in https://code.launchpad.net/~mandel/ubuntu-sso-client/ssl-dialog/+merge/94012 to not be translated? so the generated method retranslateUi is empty, like in the rest of the uis [13:43] mandel: after my fix to the strings every method in the _ui.py file has this: [13:43] 66 def retranslateUi(self, Form): [13:43] nessita, sure, sorry this was an old branch, let me fix that [13:43] 67 pass [13:43] thanks! [13:43] nessita, quick errands and I'm on that (3 min) [13:43] mandel: no rush [13:44] * nessita will make some mate [13:57] neats a preview [13:57] rye: revert what? [13:59] dobey, UniqueInterface in maverick was added to allow syncdaemon start w/o being noticed by nautilus plugin. When plugin saw syncdaemon on the bus, it started communicated with the latter. In case metadata loading was long, nautilus and desktop was hanging waiting for the response. [13:59] dobey, but it turned out that dbus waits only for 2 minutes for the autostarted app to claim the required bus name and it kills it after that [14:00] ok [14:21] mandel: question when you get back [14:21] are we still ralsinaless? [14:24] you'll have to ask ralsina [14:25] good morning! [14:25] You are now ralsinafull [14:27] ralsina: hola! everything ok? [14:28] (besides kid-schedule-issues) [14:29] nessita: just a lot of kid-related scheduling issues [14:29] nessita: but have no more of those starting today :-) [14:30] nice [14:31] nessita: I started the monster review, but I am not finished yet [14:31] ralsina: hey! is not monster [14:31] most of that is red@! [14:31] :-) [14:31] nessita: yeah, we are supposed to read the red parts too! ;-) [14:32] * nessita looks at her branch with care and tells to it "you're not a monster" [14:32] nessita: mostly trying to understand where everything went [14:33] ralsina: to a common place. A little info: I removed no code at all (except for the first "choose sign in" screen), all the red went to a common class as a little green. We had a lot of duplication... [14:33] nessita: yes, I am checking that it was really all the same code, just in case [14:33] perfect [14:48] ralsina: if you can, i'd like you around in #ubuntu-meeting at 16:00 utc ...i'll kick things off and point out the freeze exceptions but if there are specific tech questions that come up it would be good to have you or someone from desktop+ there to help answer :) [14:49] joshuahoover: sure! [14:49] dobey, nessita, please hang around as well if you can just in case ;-) [14:49] joshuahoover, I can join too [14:49] alecu: cool, the more the merrier ;) [14:51] going [14:52] ok [14:59] me [14:59] me [14:59] nessita, dobey: after the standup I'd like to ask you both on your input on how to add qt4reactor based tests to u1-client [14:59] me [14:59] me [15:00] meh [15:00] alecu: ack (though my first reaction was "don't, let's have integration tests somewhere else" :-P) [15:00] ralsina, mandel? [15:00] me [15:00] nessita, and back, sorry there was a huge q in the bank,, [15:01] mandel: ack, after the standup let's talk about your ssl-dialog branch [15:01] ok, I guess ralsina will join us [15:01] DONE: weekly call, fixed bug #939173 and uploaded a patch to Ubuntu, proposed branch for ussoc's part of bug #933576 [15:01] TODO: branch to fix u1cp of bug #933576, reviews [15:01] BLOCKED: nopes [15:01] NEXT: gatox [15:01] DONE: [15:01] Proposed padding and sizes branch for SSO, and apply some refactor to that. Review nessita's branch. Working on Bug #938604. [15:01] TODO: [15:01] Finish with Bug #938604. [15:01] BLOCKED: [15:01] No [15:01] alecu, please [15:01] me [15:01] nessita, sure [15:02] alecu, ? [15:02] DONE: pipelined tunnel client and server branches, initial cleanup [15:02] TODO: more cleanup, propose for merge, still bug #929207 [15:02] BLOCKED: need to add qt4reactor based tests to SD [15:02] NEXT: briancurtin [15:02] DONE: proposed notification branch, started on fixing Windows' read_link [15:02] TODO: read_link, any reviews i can chime in on [15:02] BLOCKED: nada [15:02] NEXT: dobey [15:02] λ DONE: finished releases, team call, bug #939691, bug #867803 [15:02] λ TODO: finish releases, deal with some twisted upstream stuff [15:02] λ BLCK: none. [15:03] mandel [15:03] DONE: Read about SSL certificates. How do I get the errors from libsoup and qnetwork. Some FF work. [15:03] TODO: Update ssl cert branch to not translate QDesigner text. Some work in FF and a windows file system notifications issue. [15:03] BLOCKED: no [15:03] ralsina, please [15:03] DONE: fixed minor network detection bug, team call, read about Mac development, calls, kid-issues TODO: fix a few bugs, misc BLOCKED: no [15:03] oops [15:03] also DONE: gave a bit of a hand in a couple of Qt-related things to dobey and nessita (yay, felt smart for 15 minutes!) [15:04] ralsina, and 1-1 with manuel :) [15:04] for the todo I mean :P [15:04] ralsina: and with nessita [15:04] :-) [15:04] any comments anyone? [15:04] mandel: and that. And with alecu. and nessita. All of you were in "calls" :-) [15:04] alecu, mandel: will join mumble to discuss all those stuff in 5 minutes [15:04] EOM? [15:05] EOM! [15:05] ralsina: when is our 1-1 supposed to be exactly? [15:05] nessita, , all those stuff means? [15:05] So, Start of Lunch for me :P [15:05] * gatox lunch [15:05] dobey: let me check [15:05] * alecu is at a loss regarding "all those stuff", too. [15:05] nessita, ^ [15:05] dobey: yesterday :-) [15:06] * mandel joins mumble nevertheless [15:06] ralsina: right; but i'm pretty sure i didn't agree to the time that the calendar thinks it is [15:06] dobey: re-schedule at will [15:07] mandel: all those stuff == your branch [15:07] because i don't generally agree to things being that early, especially if they recur weekly :) [15:07] alecu: all those stuff == the qt4reactor in u1client [15:08] qt in u1client is going to be a problem. [15:08] dobey: ok then, choose any time I am marked as free, and it's yours [15:08] ralsina: you have free time? surely you jest [15:08] dobey: I figured that it would be a problem. [15:08] dobey: I have time marked as free, it's similar. [15:09] alecu: a problem in regards to ubuntu i mean. whether or not we run qtreactor based tests isn't really a problem so much [15:10] but pulling ubuntuone-client off the CD will not be fun [15:12] dobey: I think we can manage qt as an optional dependency; so if pyqt is not installed then the proxy support is not used. [15:12] nessita, screen-shots: http://ubuntuone.com/4ryoeN4idT7WvdlkinX7XB http://ubuntuone.com/6AkotJRrh6AAPiWrTls6iQ [15:12] nessita, that is opened dialog and closed [15:13] dobey: can you join mumble, please? we are with nessita discussing this. [15:13] dobey: could you join us in mumble? alecu is explaining why he needs this [15:26] nessita, branch updated [15:27] ralsina, 1-1? [15:29] mandel: in 15 [15:29] sure [15:32] ralsina: I'm mumbe, let me know when is my turn :-P [15:32] nessita: mandel, then you, so maybe in 30 or so [15:32] no prob [15:32] nessita, we are all waiting in mumble for him hahaha [15:33] gatox: ping, Why you removed the PASSWORD_HELP constant from ubuntu_sso/qt/setup_account_page.py ? [15:33] ah, was duplicated :-) [15:33] gatox: unping [15:33] nessita, ye [15:33] gatox: well, ping again [15:33] the one you should remove is the second one [15:33] nessita, what happend? [15:34] so the constants get alphabetically ordered [15:34] I guess. [15:34] though I'm not sure :-/ [15:34] nessita, i let you choose :P [15:34] gatox: leave it like that [15:34] nessita, ok [15:36] gatox: looks good! [15:36] gatox lol mate. :D [15:36] mainerror, sorry.... really sorry :P [15:37] hehe [15:37] No problem. It is kinda funny. :D [15:38] gatox, was that for me? [15:38] mandel, hehe no, it was for the real mainerror [15:38] ok, 'cause I added an alarm for his nickname in my client just in case hehe [15:38] hehehehe [15:39] rofls [15:42] finally: mandel, 1-1 [15:43] mandel: can't hear you [15:43] ralsina, weird, I just had a chat with nessita and alecu with no problems.. [15:43] ralsina, I did not understand a word.. [15:44] argh [15:44] ralsina, you sound like one of those little robots from doctor who [15:44] EXTERMINATE! [15:47] mandel: dalkes? [15:55] nessita: 1-1 when you want [16:03] dobey: any word on bug #939797 getting in for beta1? [16:04] https://bugs.launchpad.net/ubuntu/+source/ubuntuone-installer/+bug/939797 [16:04] joshuahoover: not yet. haven't figured it out yet. [16:04] it's pretty odd [16:04] might be a bug in aptdaemon even [16:04] dobey: weird [16:09] alecu: haha [16:10] joshuahoover: hopefully i can figure it out today, or monday at latest. [16:10] dobey: k, thanks [16:14] gatox: btw, have you seen https://bugs.launchpad.net/bugs/939869 ? [16:15] dobey, that is already fixed [16:15] gatox: i knew you were working on it, and thought you already had a bug for it. which is why i'm pinging you with it :) [16:15] dobey, ah, but that is control panel [16:16] nessita, ^ [16:16] that will need styling in the control panel [16:16] looking [16:16] now the padding is added to the style of sso [16:16] if we are going to use the same frame for sso, problem solved [16:16] gatox: no, we're not... [16:16] if we are going to take only the wizard from sso and put it in control panel, a styling is needed [16:17] dobey, gatox: this is the design we have for windows as well.... and we have no task in the roadmap to change this [16:17] ack === yofel_ is now known as yofel [16:17] gatox: this is related to the "Account info" tab, no sso involved [16:17] ah ok [16:17] is the same styling as we had before, nothing changed in that front [16:18] ah ok [16:18] i thought you were fixing cp as well [16:18] I can guess people will complain a lot about different things in that panel... anyways, my plan is to gather all those bugs reports, put together in a documents with screenshots from all platforms, and take to our design team [16:19] sure. i will complain a lot :) [16:19] I think we need to review the whole Qt controlpanel design, which is not trivial not quick, so we need to start kicking that process [16:19] dobey: I don't expect less from you ;-) [16:25] ok i should get some lunch, and do yet more debugging of aptdaemon afterward :-/ [16:26] bbiab [16:38] ralsina: can a window set as transient of another x window id in Qt? [16:38] nessita: good question. I am weak in answering it though! Give me 1' to check [16:38] sure! [16:44] nessita: looks like no [16:45] buuuu. But thanks! [16:45] nessita: you may haveto set the WM_TRANSIENT_FOR XAtom manually [16:46] nessita: ugly hack: xprop -set WM_TRANSIENT_FOR abcdfgh [16:46] -id lkjhgf [16:46] but yuck [16:47] yuck indeed [16:51] nessita: the Xlib needed to do it is not terribly complicated, and is doable via ctypes [16:51] ralsina: have an example? [16:51] nessita: let me check [16:54] nessita: all the examples I see of similar things are done using python-xlib [17:02] hi, im trying to save a note in the web interface but the save button doesn't work! [17:02] cancel works just fine though [17:03] ok it works for editing an old note [17:11] ralsina, joshuahoover: did online services already happened in the ubuntu-meeting? [17:11] nessita: if it did I missed it [17:11] me too [17:11] I've been reading lots of boring stuff [17:12] waiting for our turn :-D [17:13] nessita: yeah, not sure...seems we got overlooked somehow [17:13] joshuahoover: would you ping or we're better like that? :-P [17:14] nessita: i sent out an email with FEs and the bug about u1-installer so there's nothing new there: https://lists.ubuntu.com/archives/ubuntu-release/2012-February/000876.html [17:14] * joshuahoover needs to remember to copy ubunet on that [17:14] ok then [17:14] nessita: i thought they'd REALLY want to chat with us today after we dropped 14 FEs on them the past 2 weeks [17:14] gatox: I noticed that since you branch landed, the qt sso is not longer centered on the screen [17:15] joshuahoover: I guess they forgot [17:15] nessita: or they removed us from the distro ;) [17:15] gatox: can we do something about that? [17:15] joshuahoover: ah, yes, that too :-D [17:15] oh right; totaly forgot aobut ubuntu-meeting [17:15] nessita, did you put some code to do that and it was reverted? [17:15] dobey: they no longer care about us :-D [17:15] nessita: you are going to love the branch I am doing. I am adding tests for arg parsing in u1cp and fixing two bugs at the same time ;-) [17:15] gatox: not that I'm aware of [17:15] ralsina: YEY [17:16] oh well :P [17:16] nessita: maybe 3, because we are not supporting switch_to in the Qt frontend [17:16] nice [17:16] nessita, so, it was centered just by luck :P because there wasn't code for that..... i'll add that to the sso branch i'm working on right now [17:17] gatox: thanks! please use this bug for that bug #934173 [17:18] ack [17:19] * dobey wouldn't mind being in spain next week [17:21] dobey: we're sptrinting to spain next week? :-D [17:21] i don't know if i can run that far [17:21] joshuahoover: hrmm, i think we might need a UIFe for https://bugs.launchpad.net/ubuntu/+source/rhythmbox-ubuntuone/+bug/933010 ; it's a regression, but the (full and proper) fix will introduce a new string [17:22] nessita: i wish [17:22] * nessita whishes too [17:22] ok, I will have lunch [17:22] dobey: i was hoping you would say that...14 FEs was kind of an odd number ;) [17:23] joshuahoover: let's add UIFe until we get a prime number [17:23] prime numbers are interesting [17:23] joshuahoover: well i can do it without marking the string for translation as well. it's fine by me :) [17:23] nessita: good idea! [17:23] dobey: heh [17:23] dobey: do you know what text change(s) will need to be made? [17:23] is infinity a prime number? [17:24] dobey: no, is even, clearly (?) [17:24] dobey: no [17:24] gatox, ralsina: branch https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/use-login-only/+merge/94455 is now updated from trunk, would you please finish the review when you have a chance? [17:24] dobey: infinityis divisible by any integer, so it's *very* not prime [17:24] joshuahoover: it's not a change, but a new string. we need to add a .desktop file [17:24] nessita, ok [17:24] I will have lunch now! [17:24] ralsina: it's only divisible by itself and 1 :) [17:25] nessita: +1 I had finished itandforgot tocomment [17:25] dobey: k, if you think we need it to fix that bug properly, can you update the description with details on that and i'll take it from there [17:25] ok [17:25] dobey: nope. [17:25] dobey: consider the set of integer numbers. It's obviously possible to divide it in2, orin 3 [17:25] ralsina: infinity always represents the next prime number higher than the value you think it has :) [17:25] dobey: So, if by infinity you meant Aleph(0), not prime ;-) [17:26] gatox: please make sure to have the revno 899 [17:26] dobey: divisibility is a matter of cardinality, so no ;-) [17:26] * ralsina stops arguing [17:26] dobey: is --help output covered by string freeze? [17:28] yes (and/or feature freeze) [17:29] dobey: ok, so there is another exception for u1cp [17:30] sigh [17:30] eh? [17:30] are you fixing it to do the right thing for arg parsing? [17:30] dobey: ubuntuone-control-panel-qt --help is completely wrong [17:31] dobey: I am fixing a bug where it ignored Qt arguments (like -style or whatever) that was needed for QA [17:31] dobey: also, I noticed it ignores --switch-to and --alert [17:31] right [17:31] dobey: also, the help of --switch-to is wrong [17:33] altø altø wik [17:33] * elopio goes to buy the beer for ralsina. [17:33] beware the møøse [17:50] gatox, ninja does no support proxies! [17:51] mandel, hehehehe..... you win there [17:51] gatox, it should not be a problem, I mean, if you cannot connect to the internet my ide should not block, atm is in the loading thing.. [17:52] mandel, what? are you using ninja and it gets block at loading? [17:52] does it handle unicode properly though? [17:53] gatox, yes... and I have to kill it from the process explorer [17:53] dobey, YES! :P (in most situations.... not the crazy hacks for windows) [17:54] mandel, weird..... i'll test that later [17:54] gatox, where do you keep logs? [17:54] gatox, so I can give you some info [17:55] mandel, na.... the logs doesn't have very useful information..... only the things from the plugins are being logged..... we need to improve that [17:55] gatox, ok, look at when ever you get a connection to the outside world and there is the problem [17:56] mandel, ok..... we are going to have a minisprint tomorrow..... so between today and tomorrow i'll test that [17:57] mandel, thanks for the heads up [17:59] gatox, no problem, if you need any help with it let me know [17:59] mandel, ok, thanks! [18:00] gatox, is probably picking the system proxy settings and not handeling the 407 correctly [18:09] EOW for me, see you all! [18:09] mandel, bye! [18:09] bye mandel! [18:13] bye mandel [18:16] ttyl, all [18:20] nessita, gatox, dobey: maybe a couple of reviews? https://code.launchpad.net/~ralsina/ubuntuone-control-panel/opt-parsing/+merge/94599 [18:20] ralsina: ack [18:21] yes [18:22] ralsina: i guess it needs ffe right? [18:22] dobey: this one looks like a plain bug to me [18:22] dobey: it doesn't fix the broken options [18:22] dobey: it just makesithandleonesit should already handlebefore [18:23] argh, damn spacebar I HATE YOU [18:23] ok; the bugs need to be marked as affecting the ubuntu packages, and targeted to the trunk and stable-3-0 series for the project, and precise for the package [18:24] dobey: ok, will do it now [18:24] also, i have https://code.launchpad.net/~dobey/ubuntuone-installer/fix-939797/+merge/94598 just now; which joshuahoover will love :) [18:24] dobey: trade you! ;-) [18:24] :) [18:25] ralsina: fix your bug targets :) [18:25] dobey: I am doing it now [18:27] dobey: so, it should affect distribution ubuntuone-control-panel (ubuntu)? [18:27] And series precise? [18:28] dobey: take a look at bug #910834 and tell me if I missed anything please, since I never did this before :-) [18:32] ralsina: yep, that's right [18:32] dobey: cool, thanks, doing the other bug then [18:35] dobey: I feel stupid, but I don't have a stable-3 series offered in https://bugs.launchpad.net/ubuntu/+source/ubuntuone-control-panel/+bug/938102 [18:36] Oh, ok, it only offers the series for the last project! [18:37] And now Ihave all the series to nominate EXCEPT P [18:37] * ralsina goes burn launchpad, brb [18:38] Ok, it nominated itself WTF [18:38] whatever, done [18:39] dobey: +1 code review because I can't test that IRL right now [18:39] ralsina: yeah, lp is annoying sometimes in that regard :) [18:39] dobey: in any case, it seems to be pointed/targeted/nominated/offered/candidated/annointed in all the right clubs now ;-) [18:41] can I haz reviews? https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fallback-to-gtk/+merge/94601 [18:42] ralsina: humm. does the branch even fix the second bug? [18:44] hrmm, i guess so [18:46] dobey: yes [18:46] dobey: it's a subtle fix ;-) [18:46] wow. --with-icon really should be disabled on linux :) [18:46] dobey: no, please. It's useful on KDE ;-)\ [18:47] you shouldn't be using kde [18:47] you should be using unity [18:47] dobey: plus, I want to use it as a playground for indicator work [18:47] ralsina: after the sso branch, would you review https://code.launchpad.net/~nataliabidart/ubuntuone-client/use-sso-qt/+merge/92632 ? [18:47] dobey: please take notes that in the next release, u1client 2.99.90 must depend on ussoc 2.99.90 (because of the UI fallbacking if the Qt UI is not available) [18:47] nessita: sure [18:47] nessita: I already did the SSO branch, if we are talking of the large one [18:48] ralsina: no, the one I pasted a couple of mins ago: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fallback-to-gtk/+merge/94601 [18:48] nessita: ok, missed that one [18:48] is 84 diff line! [18:48] nessita: ok :-) [18:48] ralsina: to compensate ;-) [18:51] ralsina: any chance you migrate to argparse while you're at that branch? [18:52] nessita: sure! [18:52] * ralsina has no idea how to use argparse [18:53] ralsina: should be simple, you can check out ussoc-gtk and ussoc-qt for reference [18:53] ralsina: thing is that optparse is deprecated, afaik [18:53] nessita: sure, will doit asap [18:53] no rush [18:54] optparse is deprecated only that it is no longer recommended, but it wont be removed (maybe in python 4?) [18:54] but argparse is nice [18:58] briancurtin: let's futureproof this baby for 2050 then! ;-) [19:00] ralsina: so remove all the arguments, and just replace it with "import skynet" ? [19:00] dobey: #!/bin/skynet -c "sync my files,please" [19:02] "die human." [19:03] nonono, skynet, I said "sync"! Retry! [19:13] nessita: typo in line 80 of the diff, other than that, nice! [19:14] nessita: in fallback-to-gtk [19:14] loooooking [19:16] ralsina: typo "fixied" [19:16] nessita: +1d already :-) [19:16] nice [19:17] dobey: would you please review that? (https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fallback-to-gtk/+merge/94601) [19:37] nessita: updated opt-parsing with argparse (which works remarkably similar to OptParser, to be honest) [19:37] ralsina: nice! [19:39] nessita: needs a couple of english fixes [19:39] dobey: ack [19:39] "the english fix" sounds like a very boring drug name. [19:39] Or brit-pop band. [19:39] dobey: nice correction! though I liked my invention fallbacking [19:39] :-D [19:43] heh [19:43] ralsina: it's the prequel to "the french connection" [19:43] dobey: all fixed and pushed to revno 893. [19:45] dobey: the string freeze is even if the strings are not translatable? [19:46] dobey: that's about the --help [19:50] ralsina: if they're not translated then you're probably fine; though changing the help options would be an api or ui change still; maybe you should ask in #ubuntu-release [19:50] dobey: cool, thanks [19:51] oh man [19:51] a 6 character change, and the qt control panel looks like 20x better [19:51] at least, on default ubuntu theme it does [19:51] dobey: really? which one? :-) [19:51] rm -rf ./? [19:52] - background-color: #aea79f; [19:52] + background-color: #3c3b37; [19:52] oh, that's 7 [19:52] ralsina: so, I can't run the test as the branch is, I'm getting http://pastebin.ubuntu.com/855751/. I know if I setup.py build it gets fixed, but run-tests should work from scratch. And the setup.py build should only be called before the qt tests (where is now). Would you please debug why I'm getting this "new" failure? [19:53] nessita: sure, I do have it builtso I didn't see it [19:53] nessita: I moved a delayed import out of main() [19:53] nessita: I will move it back in (sigh) [19:53] dobey: +100. Would you ping the design people about this? [19:54] nessita: english fixes approved :) [19:54] yey [19:55] dobey, ralsina: http://ubuntuone.com/1lVbN6HjbyyO7blROfk0rg from the branch I'm proposing soon [19:55] nessita: not bad! [19:56] actually, much better [19:56] nessita: what is that? [19:56] dobey: wanna file bug, so we run it by cparrino and rtgrant, and then we may ask the UIFe? [19:57] dobey: the controlpanel's new first screen (current first screen has no option for register. Yes, we have an approved UIFe for this) [19:57] nessita: I would hide the3rd button [19:57] nessita: and have the user just close the window. But that's just personal taste [19:57] ralsina: why? (and maybe, but I guess we need a complete design-review on that) [19:58] nessita: that shows up always if the token gets deleted or they haven't set up u1 yet? [19:58] ralsina: this is taken from the installer... I would prefer not changing the current design [19:58] nessita: because it's just a "close this window" button and those are usually a bad idea. Agreed on not doing it, though. [19:58] dobey: yes (I know you'll complain about the "Ubuntu One has installed") [19:58] dobey: if there is no token [19:58] nessita: yes :) [19:59] i do not like that at all [19:59] dobey: I'm not sure about that either (is in the original mocks). But is also true that that screen will be shown after installation [19:59] and it is confusing [19:59] dobey: yeah, may be [19:59] nessita, dobey: and talking about logging in, someday we have to make it so ROOT_MISMATCH doesn't happen or gives a friendly warning or something. [19:59] ralsina: add that to the roadmap, make room for it, and you got it :-D [19:59] nessita: I think that is one of the copy changes we should ask about. Makes no sense on linux [20:00] ralsina: well, if the user just installed U1 from the installer... it might, no? [20:00] nessita: we are close to the end of our current roadmap! There is a whole lot of space after that! ;-) [20:00] nessita: yes, but we have no way to know if that's the case [20:00] and it's all full of turtles [20:00] ralsina: and I bet parrino already has some crazy tasks for that :-P [20:00] turtles all the way down [20:00] ralsina: we could if the installer runs the controlpanel with --installed [20:00] nessita: our next task is the always glamorous backport effort ;-) [20:01] but that will require an exception, I'd guess [20:01] nessita: yes, option controlpanel doesn't have and we need a FFe for [20:01] right [20:01] ralsina: like i said; turtles. [20:01] ok, so, summing up: [20:01] nessita: OTOH, we could fix the --help in the same FFe ;-) [20:01] * nessita removes the "has installed blah blah" string [20:01] yes [20:02] nessita: or just change it to "Welcome to Ubuntu One." or something [20:02] dobey files the bug for the UIFe to change the background color to present to bosses [20:02] dobey: yes, like in the current GTK one [20:02] nessita: and center it, because it looks really weird being the only thing left aligned [20:02] also, padding [20:02] dobey: I was thinking on padding... not center [20:02] center will be "too much" (image is centered, buttons are also...) [20:02] there is way too much whitespace in that window [20:03] yes [20:03] centered would be better than stuck left. Padding at least [20:03] ask design for quick input [20:03] yes, I will take some screenshots of this and will email the peopl [20:03] e [20:04] too many teaks to make [20:04] yep [20:04] also, the orange looks different [20:04] (comparing images vs button) [20:04] also, are the twitter andacebook buttons misaligned or they just look that way because of the colors? [20:04] the button is a gradient, it will always look lighter [20:04] ralsina: the latter, I'd guess [20:05] ralsina: I have no changed that at all (facebook and twitter links) [20:05] buttons* [20:05] amazing how the twitter/facebook logos are vertical mirror images of each other [20:06] That page looks like a robot's face [20:06] haha [20:08] ralsina: yeah, xmag shows the follow icons are same size [20:08] nessita: in my options branch: if I move the "import start" into the main(), I will have to remove a couple of tests. Maybe I should try to see why importing that module breaks the tests instead? [20:08] ralsina: sure! [20:08] dobey: since you're in such a good mood, would you also please? (last one of the week I swear) https://code.launchpad.net/~nataliabidart/ubuntuone-client/use-sso-qt/+merge/92632 [20:09] nessita: didn't you already change DEBUG to be U1_DEBUG there? [20:09] dobey: no :-/ [20:10] dobey: well, yes, but in that branch, which I had on hold for more than a week now [20:11] nessita, ralsina i've proposed this branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/show-errors/+merge/94617 [20:11] whenever you can [20:11] ack [20:12] joshuahoover: btw, the installer fix is uploaded, and just waiting approval now [20:12] dobey: excellent [20:15] nessita: it's either move "from ubuntuone.controlpanel.gui.qt.ui import mainwindow_ui" or moving "from ubuntuone.controlpanel.gui.qt.gui import start " into one. Doing the second means losing a little bit of test coverage. The first one has no side effects AFAICS. [20:15] I mean "move one of those into a function" [20:16] EOW here! [20:17] bye people [20:17] Or put one import in a try/except so it doesn't fail when we didn't do a build [20:17] bye gatox! [20:18] ralsina, see you later [20:19] nessita: or, do the build before running the tests [20:20] sigh; more bugs i need to file it seems :( [20:20] ralsina: ./run-tests does the build before running the tests [20:21] ralsina: perhaps we're not ignoring the qt properly? [20:21] the qt part* [20:21] nessita: could be [20:22] the test is in ubuntuone/controlpanel/gui/qt/main/tests/test_main.py maybe I should rename it or change the ignores [20:23] nessita: right, that is not in QT_TESTS [20:23] QT_TESTS_PATH [20:34] ralsina: so, is fixable? :-) [20:34] nessita: thinking about it because it's working for me when I try to debug run-tests :-) [20:35] ralsina: what id you do ./setup.py clean before that [20:35] nessita: yep [20:35] joshuahoover: do i need to update the description too, or is comment good enough? [20:35] but trying again, it's late and may have had a senior moment [20:35] i wish we used filesystem links rather than shell links on windows :/ [20:35] dobey: comment should be good enough [20:35] briancurtin: feel free :-) [20:36] joshuahoover: cool. just added that [20:36] briancurtin: that is probably just our ignorance showing [20:36] ralsina: we'll still have to support shell links which were previously created, but i might propose to move away [20:36] briancurtin: we may even change/fix those links on startup if needed [20:37] briancurtin: or on installation of upgrade [20:37] good point [20:37] dobey: thanks [20:43] nessita: could you try running the tests again with revno 274? They like, totally pass for me [20:43] ralsina: sure! [20:55] ralsina: nopes, is not working [20:55] ralsina: what if you branch from scratch? [20:55] nessita: grmbl [20:55] nessita: ok, I'll try [20:56] maybe clean is not cleaning enough or something [20:58] nessita: hey,got the error at least [20:59] :-) [20:59] nessita: yes, have to add an ignore [21:03] nessita: done in revno 275 [21:03] nessita: but it surely can wait until monday if you are EODing [21:03] you mean Tuesday? ;-) [21:04] O course [21:04] Tuesday is the new Monday [21:10] dobey: will you review https://code.launchpad.net/~nataliabidart/ubuntuone-client/use-sso-qt/+merge/92632 ? I was hoping to implicitly test the nightlies during the long weekend :-) [21:11] nessita: https://bugs.launchpad.net/ubuntu/+source/ubuntuone-control-panel/+bug/940620 is the color build [21:11] do i have to? [21:11] dobey: I would like if you do [21:12] dobey: thanks for the bug report, will put that together with some screenshots [21:12] (on Tuesday, of course) [21:13] nessita: it has a branch even :) [21:13] yeap, saw it :-) [21:15] ralsina: it runs all green! [21:15] nessita: yay [21:15] ralsina: may I finish on Tuesday? I have pilates and need to buy meat for my coming-back-home-half-orange [21:15] nessita: it has 2 other approves ;-) [21:16] nessita: but of course it can wait [21:16] ralsina: thanks [21:16] nessita: I would like to have it before wednesday though so QA can use the testability driver [21:16] i really don't want to review this branch; because i am against the idea anyway. [21:16] nessita: does it not require a feature freeze exception to do this? [21:16] dobey: I don't think so... does it? [21:17] is not an API change, nor it adds functionality, nor it removes functionality... [21:17] nessita: well, it's using new functionality from a dependency [21:18] nessita: also, the bug report has no information. it's just a "we must do this" statement, and doesn't help understand why we need to do it [21:18] dobey: I can re-phrase the bug. And, using a new func from a dep requires a FFe? [21:19] nessita: i'm not entirely sure if it does or not. but i think so [21:21] dobey: ok, will check. If you don't want to review, is fine, I guess it can be landed with ralsina's review only (would you agree ralsina?) [21:23] nessita: either way, i'd prefer the ffe question was answered first, and the bug was understandable :) [21:23] nessita: I have no problem with that, I leave it to your opinion [21:24] ok, will rephrase the bug after pilates and ask in ubuntu-release weather that requires and FFe, and then will land if not [21:24] I'm off crowd [21:24] thanks nessita [21:24] thank you! [21:24] ok, bye all! have a great weekend! [21:24] (3-day weekend for some of us :-)) [22:00] have a good weekend all [22:00] you too [22:12] ok, this ends my week [22:12] see you guys on tue [22:23] hello [22:24] i've got problem with ubuntuone [22:24] i run it but ubuntuone sso does not appear [22:24] previously i've got problem with all ubuntuone but i pugre it and reinstall [22:24] and now i cant configure my accountu [22:24] could you help me? === facundobatista_ is now known as facu_bar6