/srv/irclogs.ubuntu.com/2012/02/22/#ubuntuone.txt

=== beuno_ is now known as beuno
karniLater all00:53
=== m_conley is now known as m_conley_away
ryeso, mandel FILESYNC_ERROR is str and WARNING_MARKUP is unicode07:33
mandelrye, cute... I wonder why..09:46
ryemandel, are you using spanish locale?09:47
mandelwhere, in my ubuntu? no]09:47
mandeloh, and good morning!09:47
JamesTaitGood morning all!09:48
ryemandel, ¿Y U NO USE ESPAÑOL?09:52
mandelrye, cause is a stupid language for computing :P09:53
ryemandel, but anything non-latin1 is good for catching unicode issues09:53
mandelrye, I've got enough with running a prerelease hehe09:55
ryemandel, I am running precise AND spanish. I understand 30% of the labels :)09:56
mandelrye, sorry, that is too much for me09:58
karniMorning guys o/10:05
* mandel restarting due to updates10:11
=== mandel is now known as p
=== p is now known as Guest63379
gatoxgood morningg!!!11:04
ryegatox, moarninggsss mwahaha!11:04
ryegatox, https://bugs.launchpad.net/ubuntu/+source/ubuntuone-control-panel/+bug/93780911:04
gatoxrye, mmmmmm...... i didn't like that laugh jejeje11:04
gatoxralsina, more unicode? unbelievable! :P11:05
ryegatox, are you using spanish locale on your development machine?11:07
gatoxrye, it's an english version of win, with a chinese/spanish username11:11
ryegatox, well, the bug is for ubuntu, gettext returns 'str' which i don't know whether is used in windows version, i guess it is11:11
ryegatox, the windows version does not have translations11:11
ryei suppose11:11
Guest63379rye, we are using gettext wrong, that is the provlem11:12
Guest63379problem*11:12
gatoxrye, ahhh i didn't see that was for linux.... just saw unicode aand start hitting my head with the table jeje11:12
gatoxshould be easy to fix anyway11:12
Guest63379we should be using ugettext11:13
ryeGuest63379 is mandel btw11:13
Guest63379oh, sorry :P11:13
=== Guest63379 is now known as mandel
mandelmuch better :)11:13
ryemandel, i could not figure out how to drop-in-replace gettext with ugettext11:13
ryeyet11:14
gatoxmandel, ahhhhh you are the mistery guy! jeje11:14
gatoxmandel, did you see this: https://plus.google.com/115212051037621986145/posts/6x1BrUNgTuA ???11:14
* mandel looking11:17
mandelrye, I think we have been using gettext instead of ugettext everywhere.. so is a pita11:17
ryemandel, i wonder why it exploded only now11:18
mandelgatox, nice!11:18
gatoxmainerror, yeah! :P11:18
mandelrye, a mistery..11:18
mandelgatox, I'm here!11:18
mainerrorHuh?11:19
gatoxmandel, ahhhhhh11:19
mandelhehe11:19
gatoxmainerror, sorry.... code completion always confuse mandel with mainerror :P11:19
mandelmainerror, our nicknames are too close and people get confused with the autocmplete hehe11:19
mainerrorheh11:19
gatoxcode completion jeje11:19
gatoxjust completion11:19
ryeunless by removing gtk we changed something related...11:19
ryegtk control panel11:20
ryewell, the net result is that control panel for qt does not work for any languages that use something outside latin111:20
ryeor ascii, to be precise11:20
nessitahello everyone!11:20
mandelrye, super cute!11:20
mandelrye, :P11:20
mandelnessita, morning!11:20
nessitarye: hola! where is that?11:20
mandelsweet! I got my ubuntu machine back11:21
* mandel switches machine11:22
ryenessita, in ubuntu, now, gettext returns 'str' but our language files for e.g. spanish are in utf-811:22
ryenessita, https://bugs.launchpad.net/ubuntu/+source/ubuntuone-control-panel/+bug/93780911:22
gatoxnessita, https://bugs.launchpad.net/ubuntu/+source/ubuntuone-control-panel/+bug/93780911:22
gatox:P11:22
nessitarye: awesome, will propose a fix. I just fixed that for sso last week :-)11:23
ryenessita, oh, shiny!11:23
nessitarye: thanks for catching this!11:23
ryenessita, you are very welcome11:24
mandelI'm back!!! hurray!11:25
gatoxnessita,  i was thinking about the UIFe for the bug about how to handle showing the errors inside the wizard..... but, we don't need a UIFe for that (i think), because in the wizard from sso we are just showing the errors in a new dialog, the inside display of the errors is only affecting the windows wizard11:29
nessitagatox: but we want to stop showing the dialogs in the sso screens, no?11:30
nessitaand integrate the errors with the same wizard pages11:30
gatoxnessita, ah yes, if that is the plan, so we need a UIFe that involve to move the errors inside the pages, handling the correct way to handle that is just consequencce..... am i right?11:31
facundobatistanessita, is that gettext issue the same we talked last time in your place?11:32
facundobatistanessita, hola, btw :)11:32
nessitagatox: yes11:32
gatoxnessita, ok, i'll create that bug now!11:32
nessitafacundobatista: hola! yes, though is not in sso (is fixed), but now is also in controlpanel, so will apply the same fix11:33
nessitagatox: wait!11:33
gatoxnessita, what?11:33
nessitagatox: wasn't the bug already created?11:33
gatoxnessita, i don't remember to have seen any bug related to move the errors inside the pages......11:33
facundobatistanessita, ah, ok11:34
nessitagatox: you were supposed to file the UIFe for that along with the margin changes11:34
gatoxnessita, i report the one with the padding11:35
nessitagatox: let me re-read our chat from last fri 'cause now I'm confused about what we agree11:36
* gatox too11:36
gatoxnessita, i file the uife bug about the padding..... only the one about the errors remain... but i found about that we are not using that in that way in linux..... now that you said that the idea is to move that into the pages, what we really need to file is the bug to move the errors inside the pages..... that's what i understand11:38
nessitagatox: as per my logs, you were supposed to create that bug asking for the UIFe on Monday :-P11:38
mandelgatox, alecu, nessita FYI I have control panel working through a proxy atm \o/11:39
nessitamandel: yey! ah, btw11:40
nessitamandel: saw my comment on the bug you filled yesterday re: strings for sso dialog?11:40
mandelnessita, no, let me check11:40
mandelnessita, yeah, I though about the doc being private.. I'll provide more details regarding the strings to be used11:41
nessitamandel: ok, and consider we need a UIFe for that, since we're making releases today11:41
mandelnessita, yes, there are several branches of mine for review that will require that, I'll coordinate with josh about this11:42
nessitamandel: thanks!11:42
mandelno problem :)11:42
ralsinagatox: yes, more unicode :-(11:56
ralsinagood morning everyone!11:57
gatoxralsina, hi! :P11:57
nessitahola ralsina!12:03
ralsinahola nessita! I did very early reviews of the stable branches, just for you!12:04
nessitaralsina: thanks! already merged, so I'm moving forward with the releases :-)12:05
ralsinanessita: cool12:05
nessitaralsina: so, I will have to leave for an hour approx after lunch :-/ (I'm filled with hate :-D)12:05
ralsinanessita: ack12:06
nessitarye: would you be able to review this? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/unicode-gettext/+merge/9413612:15
ryenessita, interesting12:17
ryenessita, muy bien! it works for spanish/russian for sure12:22
nessitarye: YEY (can you please try both, -gtk and -qt?)12:22
ryenessita, well, yes, both -qt and -gtk work equally good, gtk one has more strings translated though12:23
ryenessita, also, qt one does not want to get killed by Ctrl+C12:23
ryebut that is not relevant12:23
nessitarye: right to the first one, 'cuz -qt just got into the ubuntu repo last week, so translator are still getting the new strings. About the second... yes, we know but have no idea why (and have not debugged yet)12:24
ryenessita, well, here qt one seems to be killed KeyboardInterrupt but event loop continues, in case of u1sdtool this is sigkill only12:26
mandelnessita, ralsina that branch will brake if you receive a http://developer.qt.nokia.com/doc/qt-4.8/qevent.html#Type-enum of type LanguageChange because the retranslateUi methopd provided by qt designer will be using QtGui.QApplication.translate instead of gettext12:28
mandelthat happens when the QTranslate is changed.. I don't know when that happens12:29
nessitamandel: what branch, the gettext-unicode? if so, how the code that was before there was any different?12:30
mandelnessita, the compiler modification you changed was the one that ensured that the correct method was used, do a python setup.py build and compare to _ui files, one from trunk and one from that branch12:30
mandelthis guy: _wrapuic12:31
nessitamandel: doing it ATM, but then why is working without that code?12:31
mandelnessita, because we are manually setting the labels in the widgets using setText(GETTEXT_TEXT) but that is done during the __init__ of the widgets, if we get the LanguageChange event, which is when the QTranslater changes the retranslateUi method won't use gettext12:32
nessitamandel: so this only affects when someone changes the language while the app is opened?12:33
mandelnessita, yes, only then AFAIK12:33
nessitamandel: so, we can workaround by not setting any string from the designer files, no?12:33
mandelnessita, the it will set everything to ''12:34
nessitamandel: hum? I mean, settings the button's string using code12:34
mandelnessita, read the following: http://developer.qt.nokia.com/doc/qt-4.8/qcoreapplication.html#installTranslator12:36
nessitamandel: the thing is that we need to do something that works without having to import the _12:36
nessitamandel: since we need to install it with gettext.install so we can force to have unicodes from it12:36
mandelnessita, I'd say, remove it, branch that code and then add a bug saying that we do not support dynamic-translation and we won't12:37
mandelnessita, I mean, not branch, land12:37
nessitamandel: but I see another issue now, using what you showed me: there are strings that will never get translated, for example:12:38
nessita        self.edit_profile_button.setText(QtGui.QApplication.translate("Form", "Edit personal details online", None, QtGui.QApplication.UnicodeUTF8))12:38
nessitamandel: we used to have _("Edit personal details online") and that will make the string show up for translation12:38
nessitanow I guess it won't?12:38
mandelnessita, indeed12:38
mandelnessita, but is nicer, since it would be confusing for translators.. since we have the common ones to translate and then the ones from the QDesigner, so duplication everywhere12:39
nessitamandel: the problem is that some strings are only set once in designer, so now I need to move those out and set them by code12:40
mandelnessita, best thing I can think of, use your code and allow us to use strings in designer, so that we can see what we are doing12:40
mandelthen state, dynamic translation is not supported12:40
mandelif you change the lang, close and open the app12:40
nessitamandel: does gettext support dynamic translation?12:41
mandelnessita, very very good question.. and I have no idea, maybe dobey  knows it12:41
Chipacabuen día, gente12:42
mandelnessita, maybe Chipaca knows :)12:42
mandelChipaca, buenos dias!12:42
nessitahola Chipaca!12:42
Chipacatapping into the proxies experience we have, if I wanted a glib app to support proxies, should i make it use libsoup? (this is for the remote videos scope)12:42
mandelChipaca, yes, although you have to write code dealing with the 407 and the 401 coming from the proxy, I'm playing with the libsoup code atm so I should be able to point you to our webclient for that12:44
Chipacawhat does dynamic translation mean? gettext is rather crufty, but you can call it twice in a row with a little setup in between and it'll switch languages12:44
nessitaChipaca: is my understanding that using libsoup is the best option, though you still need to do some not trivial tasks to proxy actually being used. Among those are retrieving proxy credentials, validating ssl cert and handling any errors, etc12:44
Chipacaplain ol' http, here12:45
Chipacait's a lovely place where responses are fast and uncomplicated (?)12:45
Chipacamandel: i'd appreciate those pointers :)12:45
mandelChipaca, so, it depends how libsoup was compiled, if it was compiled with the --gnome-support or something like that flag, it will pick the proxy settings, otherwhise you have to manually set it, which is a pita12:46
mandelChipaca, I should be done with the code this afternoon, can you wait that long?12:46
nessitaChipaca: there is an email from alecu in reply to "Preferred C HTTP Library", that has a little more info... you saw that?12:46
Chipacathis is python :)12:46
nessitaChipaca: ah!12:46
nessitaChipaca: then the lens may use the ussoc's webclient (new) library?12:47
Chipacaurl! url! :)12:47
mandelindeed, if you re using python you can use our code12:47
mandelwhich has integration tests with squid etc..12:47
* Chipaca pulls ussoc trunk12:48
nessitaChipaca: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntu-sso-client/trunk/files/head:/ubuntu_sso/utils/webclient/12:48
Chipacayep, bzr and grep got me that far :)12:48
nessitaChipaca: the choosing of which webclient is "automagic", so you just import the factory and start enjoying12:49
Chipacadavidcalle: ping, about lens, dependencies, and proxies12:49
mandelChipaca, well.. no in trunk there is not proxy support yet12:49
ralsinanessita,mandel: about the translation stuff, I really really hate .ui files that have no text in them, so unless we actually know how to crash it, please don't remove them :-/12:49
davidcalleChipaca, ?12:49
mandelChipaca, lp:~mandel/ubuntu-sso-client/webclient-use-dialog12:50
Chipacadavidcalle: hi. The remote video lens does not support proxies, and uses urllib2 to get things12:50
mandelChipaca, no 100% ready yet..12:50
Chipacadavidcalle: would you be ok with depending on an external library (part of the ubuntu sso client) for that?12:50
davidcalleChipaca, yes12:50
nessitaralsina: the problem is that those strings are not getting translated... I guess we can leave some dummy strings in place, as long as we always override them with code (and thus set a proper string instead)12:50
mandelralsina, +1 on not removing the strings, lets use them but not translate them using gettext, so I thin nessita code is 100% correct, we simply say that if you change the lang of the system you will have to close and  open12:51
davidcalleChipaca, if it doesn't affect the speed.12:51
mandeldavidcalle, it will affect the speed if you don't have the proxy settings ;)12:51
Chipacadavidcalle: the other option is to use libsoup, but there are some corner cases with proxies that it alone doesn't cover12:51
mandelChipaca, davidcalle the API is asyn using deferreds, so it should not block, and the overhead should be in the interaction with the user to get the creds, either from him or from the keyring12:53
Chipacadavidcalle: would you rather the lens interrupted the user to ask for the proxy credentials if they aren't set up, or would you rather it just bailed?12:53
Chipacathat might be a design question12:53
nessitamandel: since you already looked at the branch, would you please review (I filed bug #938626 to fix the other things we discussed)? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/unicode-gettext/+merge/9413612:54
ralsinamandel,nessita: +1 on that12:54
mandelnessita, yes :)12:54
davidcalleChipaca, I didn't knew that urllib2 was an issue with proxies. Anyway, interrupting the lens for credentials is pretty bad. I might be missing something, why the lens with what we currently have, would need credentials?12:54
ralsinadavidcalle: authenticated proxies12:55
Chipacadavidcalle: proxies can require credentials12:55
davidcalleOk12:55
mandeldavidcalle, my nightmare.. :)12:55
Chipacadavidcalle: I agree interrupting from the lens is bad. I think we should just let it bail.12:56
davidcalleWe have the possibility to display an error message in the lens if no results have been fetched : "No credentials found for your proxy " or something, and let everything being handled by the keyring.12:57
davidcalleHum, I'm not saying that 'No credentials found for your proxy" is a godd user facing string, but you get the idea :)12:58
mandelnessita, +112:59
mandelnessita, I'll try to research at some point in my life how to deal with this in the future.. only thing I can think of is a generic QDesigner extension, but would be a PITA to work with.. and maybe not worth the effort12:59
gatoxnessita, did you see the ubuntudev twit? :D13:01
mandelok, does anyone understand this:13:02
mandel<Message object at 0x20a1690 (SoupMessage at 0x1b69540)>13:02
mandel<SessionAsync object at 0x20a15f0 (SoupSessionAsync at 0x1b69460)>13:02
mandellibsoup-CRITICAL **: soup_session_pause_message: assertion `item != NULL' failed13:02
nessitagatox: no! link?13:02
gatoxnessita, i retweeted13:02
davidcalleChipaca, I just need to investigate libsoup then. I'm used to urllib2 for calls and lxml if I need to parse something (faster than libsoup).13:02
nessitagatox: :-D13:02
gatoxnessita, grosa! :D13:03
nessitaheheheh13:03
mandellol13:03
mandeldavidcalle, Chipaca so, the deal is, if you don't have the creds you can tell the user to go to the system settings and use user:password@domain, gsetting will lie to you because it will say no auth required, then if you use that for the proxy url in the libsoup session you will be fine13:05
mandeldavidcalle, Chipaca if you get a auth error and use auth.is_for_proxy() you will know if it was a 401 from the proxy or from the source13:05
mandelwhere source I mean which ever server thing you are using13:06
davidcalleOk13:06
Chipacamandel: you lost me there a bit13:14
mandelChipaca, ok, open your system settings and go to network, you will see that the ui does not provide a single thing for auth creds13:14
mandelChipaca,  to work around that you set the url of the proxy to be user:password@domain13:14
mandelChipaca, for example mande:proxy@daredevil.marvel13:15
Chipacamandel: but if the user has done that, things just work via libsoup?13:15
mandelChipaca, yes13:15
mandelChipaca, well, if libsoup was compiled with the gnome flag13:15
mandelChipaca, you could get a libsoup that uses nothing from gnome :)13:17
Chipacamandel: yah, yah. Which it is in ubuntu if anybody is sane, yes? :)13:17
mandelChipaca, yes13:17
Chipacamandel: you say "compiled with the gnome flag", but you still have to add the SoupGNOMEProxyWhatever thing to the session, yes?13:17
mandelChipaca, automatic proxy detection should be there: http://mail.gnome.org/archives/libsoup-list/2009-May/msg00008.html13:19
Chipacahttp://developer.gnome.org/libsoup/2.37/libsoup-client-howto.html13:21
Chipacamandel: ^ adding the gnome proxy thing (in C)13:21
mandelChipaca, which is exactly the same that we do via pygobject13:22
Chipacamandel: yep13:22
Chipacamandel: so you're saying it's automatic once you've manually set it up :)13:23
mandelChipaca, well, yes, but you just have to set the proxy-uri thing13:24
mandelok, I'm off to have lunch13:33
davidcalleChipaca, about "Recently Viewed", have you discussed it with John Lea?13:37
Chipacadavidcalle: yes13:37
Chipacadavidcalle: only filed the bug after getting his go ahead13:37
davidcalleChipaca, cool, merging then.13:37
Chipacadavidcalle: ditto with the others13:37
Chipacabah, i haven't checked the "refresh" one with him, but only because i can't imagine a world where we *don't* want to refresh :)13:38
Chipacadavidcalle: I'll be fixing most of these today13:38
Chipacadavidcalle: unless you want to work on any of them?13:38
Chipaca(the "refresh" one is one that can wait, because it's clearly not a feature nor ui change :)13:39
davidcalleChipaca, I'm currently writing a lens tuto for dev.ubuntu. I don't mind if you want to work on them. About the refreshing, when do you want to do it? On each search?13:42
Chipacadavidcalle: nah, once an hour tops13:42
davidcalleChipaca, great :)13:42
davidcalleChipaca, I'm merging the i18n branch, and adding your string change with it.13:43
=== yofel_ is now known as yofel
dobeynessita: what do you mean by "dynamic translations" ?14:10
nessitahola dobey!14:10
dobeyhola14:10
nessitadobey: it was a term that mandel was using, I think he means changing the language while having an app opened (and thus expect that the strings change to teh new language)14:10
dobeyit's certainly possible, but not easy, and not worth the trouble of implementing, i don't think14:11
nessitadobey: right14:11
briancurtinhi team14:11
nessitahola briancurtin!14:11
gatoxbriancurtin, hi14:12
ralsinahi briancurtin! So, you didn't get selected?14:12
briancurtinralsina: ha. i was in the list of 40 people to be questioned for a trial, but i know the lead attorney14:13
ralsinabriancurtin: lucky break!14:13
briancurtini'm still on the hook through friday, but they said once you've been called in it's unlikely they call you back in during that period14:14
ralsinanessita: we have our 1-1 but I am feeling kinda crappy, so I'd rather we had it tomorrow14:28
nessitaralsina: is ok, let's have it tomorrow14:28
* ralsina doesn't want to puke on mumble (yes, too much information)14:28
mandelnessita, dobey open and close the app is good enough, is not that people keep switching from one lang to another all the time14:52
nessitaargh, standup!14:59
ralsiname14:59
gatoxme14:59
nessita16 seconds to go!14:59
briancurtinme14:59
mandelme15:00
nessitame15:00
nessitaalecu, dobey?15:01
dobeymeh15:01
alecume15:01
nessitaralsina: go!15:01
ralsinaDONE: calls, reviews, interviews, canonicaladmin, misc, reading about OSX development, installed a hackintosh, minor hacking TODO: feeling ill, so maybe go to bed BLOCKED: by feeling like crap15:01
ralsinagatox, please15:01
gatoxDONE:15:02
gatoxRelaxing a lot during holiday (monday-tuesday), File some UIFe issues, Propose a branch for UI styling in SSO. Start working in SSO padding problem (almost done!). Happy about tihs: https://plus.google.com/115212051037621986145/posts/6x1BrUNgTuA (yes, i needed to say it)15:02
gatoxTODO:15:02
gatoxFinish with the padding issue, work in the other UIFe issue.15:02
gatoxBLOCKED:15:02
gatoxNo15:02
gatoxbriancurtin, go15:02
briancurtinDONE: holiday, federal jury duty15:02
briancurtinTODO: 851810, figure out the very last level of exposing the notification since i have the lower levels figured out15:02
briancurtinBLOCKED: none15:02
briancurtinNEXT: mandel15:02
mandelDONE: Finished and proposed branches with the ssl dialog. One review for nessita. Debugged why libsoup webclient is giving issues. Fixed my P machine that went to hell.15:02
mandelTODO: Talk with josh about exceptions. More proxy. Mumble with alecu, urbanape and ralsina15:02
mandelBLOCKED: no15:02
mandelnessita, please15:02
nessitaDONE: lots of holidays15:02
nessitaTODO: ubuntu releases, fix bug #938626 and bug #937809 and build patches for packages15:02
nessitaBLOCKED: nopes15:02
nessitaNEXT: dobey15:02
dobeyλ DONE: releases, pyqt pokery15:02
dobeyλ TODO: finish releases, deal with some twisted upstream stuff15:02
dobeyλ BLCK: none.15:02
dobeyalecu15:02
alecuDONE: got a working tunnel server, started with tunnel client15:02
alecuTODO: mumble with mandel & urbanape, work on tunnel client (bug #929207)15:02
alecuBLOCKED: no15:02
alecuEOM?15:02
nessitacomments anyone?15:02
gatoxnop15:03
nessitaI was wondering if is everyone familiar with the freeze exception process... JIC15:03
briancurtinralsina: speaking of OSX development, i looked around and can't find the install CD so i'll need to get a new one15:03
mandelnessita, I'm a little, but I've never had to deal with it15:03
ralsinabriancurtin: ok, let me check what we can do about that15:03
briancurtinnessita: it wouldn't seem to affect me for now, but for the future i should probably know15:03
mandelnessita, any wiki I can get the infor from?15:03
nessitabriancurtin, mandel: https://wiki.ubuntu.com/FreezeExceptionProcess, any question about that, please ask. Yes, is mostly FYI, what I care the most is that we don't land branches that violates some of the freezes that are happening these weeks15:04
mandelok15:04
nessitadobey: is our precise tarmac blocked?15:05
dobeyi don't think so15:06
nessitadobey: hum, this is not landing https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/unicode-gettext/+merge/9413615:07
nessita(approved 57 minutes ago and has commit message)15:07
dobeynessita: i can't get to it to verify anything anyway15:08
aquariusChipaca, dead bip?15:09
Chipacaaquarius: yes. just sent you a message.15:10
mandelralsina, should be briancurtin in the mac os x chat?15:11
mandelbriancurtin, FYI mac os x chat in 17 mins aprox15:13
briancurtinmandel: mumble or somewhere in IRC?15:13
mandelbriancurtin, mumble so that you can hear or horrible accents :)15:13
ralsinamandel: briancurtin: yes15:14
briancurtinmandel: i'll have to make up an accent, or get one of those voice encoders like movie criminals15:15
mandelbriancurtin, If I were you I'd fake a mexican accent to blend in the team hehehe15:16
dobeynessita, mandel, gatox: btw, you have earned scorn for https://code.launchpad.net/~diegosarmentero/ubuntuone-client/network-detect/+merge/8854515:17
aquariusChipaca, I have no message?15:17
gatoxdobey, why?15:18
mandeldobey, scorn or corn? :P15:19
dobeygatox: you commented out two constant imports, and use arbitrary numbers instead of the constants, where they were being used before15:19
Chipacaaquarius: whatsapp15:19
dobeygatox: and mandel and nessita approved it!15:19
nessitadobey: why?15:20
aquariusChipaca, ah. blew away my phone and haven't reinstalled it ;)15:20
mandeldobey, bloody true: 10+#    NM_STATE_CONNECTED_GLOBAL,15:20
mandel11+#    NM_STATE_DISCONNECTED,15:20
Chipacaaquarius: ah. Well. "bip server down" was pretty much the extent15:20
dobeymandel: yes, that15:20
dobeyand the related changes to that15:20
aquariusChipaca, heh, ok :)15:21
nessitadobey: besides the commented lines, that should be removed, what else?15:21
gatoxdobey, that's why we were removing that things.... but couldn't remove everything yet15:21
mandeldobey, I'm up to emend that error..15:21
nessitadobey: the replacemente of that is not a fetaure adding, but using an existent feature15:21
dobeynessita: they shouldn't be removed.15:21
nessitadobey: yes they should (along with the fake network manager, there is a bug for that)15:21
nessitadobey:  bug #93538915:22
nessitaoh no bot15:22
dobeynessita: then they should have been removed in that, rather than simply replacing them with magic numbers in this branch15:22
nessitahttps://bugs.launchpad.net/ubuntuone-client/+bug/93538915:22
nessitadobey: yes, agreed that the magic number replacing is bad. But is in test_dbus.py, so I filed the bug to fix in another branch15:23
dobeynessita: branches that replace well defined and descriptive constants, with magic numbers, should not be approved15:23
mandelnessita, I agree with dobey we should have removed it all at once, but since its going away, is not a huge issue15:24
dobeyit is an issue when i propose a backport branch for making the release, and it includes that change, and a reviewer asks me why it is doing that15:24
nessitadobey: I agree, and we should not do that again. Though for this particular case, since is a test file, is not that bad. I know gatox was trying to remove the fakednetworkmanager in that branch, but since it got hairy, we left that for another branch15:24
nessitadobey: ok, that's bad, yes15:25
nessitadobey: want gatox to propose a revert for that?15:25
dobeyno. i want to get the release done. and for you all to know to never do that again :)15:25
nessitadobey: duly noted. gatox, got that? ^15:27
gatoxyes15:27
nessitadobey: thanks!15:27
ralsinaalecu, dobey: you are both confirmed for UDS15:55
alecuralsina, cool15:55
dobeyoh right, i've been meaning to ping about that anyway15:56
ralsinadobey: so, preemptively ponged15:56
dobeyralsina: and no sprint or some-hands or whatever the week prior, right? uds only? :)15:57
ralsinadobey: AFAIK just UDS15:59
dobeyok, good16:00
mandelalecu, suertudo!16:00
dobeyi wonder if i can get a flight that's not too insane.16:01
briancurtindobey: flying there won't be too bad. flying back is another story :/16:02
dobeybriancurtin: ? probably wouldn't be as bad for you. it seems i'd have at least 2 stops on the way16:03
briancurtindobey: ok i change my answer then16:04
dobeynessita: care to do https://code.launchpad.net/~dobey/ubuntuone-client/update-from-trunk/+merge/94053 ?16:09
nessitadobey: not at all16:09
* gatox lunch!16:23
dobeylunch, bbiab16:34
* mandel wants to kill himself..16:35
ralsinamandel: first! called it! ;-)16:36
ralsinamandel: why?16:36
ralsinaDamn, lost the opportunity to call shotgun instead16:37
mandelralsina, stupid proxy raising Unauthorised exceptions that are cough by the backend of control panel which then cleans the creds.. then everything explotes!16:38
ralsinamandel: libsoup or qt?16:39
mandelralsina, atm qt16:39
mandelralsina, I'm fixing it atm on the sso code.. but it has made me waste some time..16:40
nessitadobey: approved16:42
Scunizigood morning all.. I'm having issues syncing tomboy across several ubuntu/kubuntu machines. When I manually sync the system asks if I want to make the copy on my machine marked as (old) and get the server version. Now all my machines have duplicate notes half of which are labeled (old).  How do I remedy this sync process?16:42
nessitamandel, ralsina: can I haz a review please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/all-ui-from-code/+merge/9422016:42
ralsinanessita: of course16:43
nessitathanks!16:43
mandelnessita, sure16:43
briancurtinugh, i forgot to submit per diem from the sprint. quick question if anyone knows: 1) is it too late, 2) what is the max value?16:44
Chipacabriancurtin: it's not too late, and there is a table for what you claim, it's a fixed amount per day16:46
Chipacabriancurtin: um ... maybe in private with your manager :)16:47
briancurtinChipaca: ah yes i'll find the table, and oops16:47
mandelnessita, this looks funny: if getattr(self.ui, 'label', None) is not None: why is that?16:49
mandelbriancurtin, I know that 1 trillion dollars is not acceptable as per diem.. :(16:50
Chipacamandel: that looks like The Right Way to do hasattr(self.ui, 'label')16:51
Chipacamandel: well, assuming self.ui.label cannot be None. Otherwise, the not-so-good-way :)16:51
nessitamandel: test blowing up in my face 'cause we're faking the object that setupUI returns :-/16:51
nessitaChipaca: why not so good way?16:51
Chipacanessita: if self.ui.label can be None, then getattr(self.ui, 'label', None) can't tell "self.ui has no attribute 'label'" from "self.ui.label is None"16:52
nessitaChipaca: ah, yes. Anyways, if it's None I can't call what's next, so the check is what I want (tm) :-)16:53
Chipacanessita: if self.ui.label can be None, you'd do: marker=object(); if getattr(self.ui, 'label', marker) is not marker16:53
mandelnessita, looks good, I would have been A LOT lazier than you and I would have done the following, created a dict with the name of the label/button/blah as key and the string as value16:53
Chipacanessita: yes, of course :)16:53
ralsinamandel: only if you had dinner in pluto16:53
mandelnessita, then I would have iteritems and would have done a setText in each16:53
ralsinamandel: half a trillion for lunch in pluto, though16:53
Chipacanessita: not-so-good only in the "right way to do hasattr", not necessarily bad when you can opportunistically do less to do more16:54
nessitamandel: but sometimes is setText, sometimes is other method16:54
nessitaChipaca: :-)16:54
mandelnessita, yes, setTitle, but that is 6 methods vs the rest, nevertheless is because I'm lazy, now that it has been typed, who cares :)16:55
mandel+1 on the branch16:55
nessitamandel: thanks!16:55
ralsinanessita: isn't it more pythonic in lines ~688 to just do the remove and catch KeyError?16:58
nessitaralsina: looking16:58
ralsinasorry, I meant "do the setText"16:58
ralsinanessita: also, why would it not have a button?16:59
nessitaralsina: sorry, got lost. What would be more pythonic than what?16:59
ralsinanessita: lines 688-689 of the diff16:59
nessitaralsina: the button may not be the 'cause in tests self.ui is a extremely generic fake17:00
nessitaralsina: what would you use instead of the getattr?17:00
ralsinanessita: I don't understand why that may happen. And if it may happen, why do a getattr instead of a try/except KeyError17:00
nessitaralsina: exception are expensive17:00
ralsinanessita: having the code do strange things because of a test seems backwards to me17:00
nessitaralsina: yes, though I need to put this in ASAP before I build the packages (will be a vendor patch)17:01
ralsinanessita: but hey, I am nitpicking17:01
ralsinaok17:01
nessitaralsina: I agree, but fixing the test suite will take me some time that I don't have (today)17:01
ralsinacan you add a FIXME, maybe? It you say no, it's fine too17:01
nessitaralsina: I will create a bug report, is that ok?17:01
nessitaor a FIXME< as you prefer17:01
ralsinanessita: wonderfultastic17:02
ralsinanessita: +117:02
nessitathanks17:02
nessitamandel, ralsina: disclaimer comment added in each case17:04
mandelI'm not sure exceptions are expensive in python.. how do you get out of iterators?17:06
briancurtinraise StopIteration17:07
nessitamandel, ralsina: equivalent branch for ussoc: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/all-ui-from-code/+merge/9422817:10
ralsinanessita: on it!17:11
nessitamandel: just FYI, whena adding constants to the ubuntu_sso/utils/ui.py, we try to add it in alphabetical order (sometimes we fail just because we're human). I corrected the PROXY_* constants (just FYI)17:11
mandelnessita, they are corrected in one of my proposed brancjes17:11
mandelbranches*17:12
nessitamandel: ok, then you will need to merge trunk in :-/17:12
mandelnessita, they are in the same place, I'd be surprised if we have issues17:12
nessitaack17:12
mandelnessita, why the new param in 416917:13
nessitalooking17:13
nessitamandel: every wizard page needs to know which app_name is being displayed for, so we can build proper messages like 'In order to connect to %(app_name)s please have an internet connection foo bar'17:14
nessitamandel: see lines 4216 and 421717:14
mandelnessita, so doing something like page.wizard().appname is not a good idea?17:14
nessitamandel: not at all, we're trying very hard to remove the dependency on the parent wizard in each page17:15
nessitamandel: we get tons of AttributeError because .wizard() is None17:15
nessitamandel: so our code turns into a complex workflow where we can only do stuff when the page has been added to a wizard17:16
nessitawhich, from my POV, is not good, since we want the pages to be "self-sufficient"17:16
nessitaso we can re-use them from other places, such as embed those in other  UIs17:16
mandelnessita, ok, I don't know how you guys are dealing with the ui its you code :D17:18
mandelbut maybe doing a widget instead of a wizard page is a better idea.. seems odd to say you have a wizard page with no wizard..17:18
mandelwhat does a wizard page add to a widget?17:18
nessitamandel: that's the next step, but we couldn't do all the desired refactoring in time17:19
mandelok17:19
ralsinamandel: a wizard page has a wizard() method and has a method where it can do things like setting up the wizard's button layout17:21
ralsinamandel: but of course we can split that into a widget and a wizardpage with the widget, or have the wizard setup its own buttons on next() and other ways17:22
mandelnessita, forgotten_password.ui changed sizes, is that an issue?17:36
mandel446 => 23017:37
nessitamandel: no, I just adjusted the form size with "adjust size" (it was big for no reason)17:37
mandelok17:37
mandelnessita, I see you added a spacer for setup_account.ui, we had resizing issues?17:39
nessitamandel: hum, that was not intentionally... let me check17:39
mandelnessita, the object name is verticalSpacer_417:40
mandelnessita, and verticalSpacer_517:40
nessitamandel: what diff line?17:42
mandelnessita, in lp is 344917:43
nessitamandel: how are you seeing the diff?17:44
mandelnessita, meld17:44
nessitamandel: in LP the whole file is diffed since I changed the EOLs, and in meld I don't get that difference. Is your base branch (trunk) up to date?17:45
mandelnessita, double checking17:45
mandelnessita, sorry false alarm something landed from a few hours, sorry17:47
nessitamandel: ack17:47
mandelnessita, +117:49
nessitamandel: thanks!17:50
nessitadobey: hey, I have a non trivial question re-packaging...17:50
nessitadobey: both u1cp-gtk and u1cp-qt have some shared dependencies on some data under usr_prefix/ubuntuone-control-panel/*.png... how shall we deal with this at packaging level?17:51
nessitadobey: ATM, the -gtk binary is installing those... but if someone installs only the -qt package the images will be missing17:52
dobeynessita: make a new binary package called ubuntuone-control-panel-common, with those files, and have both depend on it17:55
nessitadobey: ack, thanks17:56
gatoxbrb!17:58
=== gatox is now known as gatox_brb
nessitadobey: another question, shall I remove the -gtk package? (I don't think so since we have not resolved the cd-yadda issue yet)18:05
nessitadobey: but perhaps you have a special plan in your head :-)18:05
dobeynessita: i don't see any reason not to remove it18:05
dobeynessita: other than the fact that we need a ffe to do so i guess18:05
nessitadobey: wouldn't desktops "break"? the launcher icon, the messaging icon18:05
dobeynessita: no18:05
nessitadobey: you already released to ubuntu ubuntuone-installer? if so, I need to bump the depends on controlpanel18:06
dobeynessita: ubuntuone-installer has been installed by default for probably 6 or 7 months already :)18:07
nessitadobey: yeah, I know that. But I was asking about the latest change you made, where it checks for -qt instead of -gtk18:07
dobeynessita: i haven't released it yet. am about to do that18:07
nessitadobey: ack, what version will that be?18:08
dobeynessita: same as everything else. 2.99.518:08
nessitagreat, thanks. So I will not remove the -gtk controlpanel yet, until we get the FFe18:08
dobeynessita: we need to get ubuntuone-control-panel-gtk removed from Depends of ubuntu-desktop package as well (which is main reason for ffe)18:12
nessitadobey: right18:12
nessitaok, I gotta go to run an unpleasant errand... will be back in one hour I hope18:13
nessitaralsina: ^18:13
=== nessita is now known as nessita-away
ralsinanessita-away: ack18:14
dobeythis is why i hate parking my car on the street in the city18:15
mandelok, EOD for me, laters!18:17
=== JanC_ is now known as JanC
=== gatox_brb is now known as gatox
briancurtinnessita-away: on bug #851810, i'm kind of at a dead-end. i'm not totally sure how to move forward exposing this signal. i have it doen and tested in VM and interaction_interfaces, but where above that does it need to be exposed?18:44
briancurtin(whenever you get back, of course)18:44
dobeyhttps://code.launchpad.net/~dobey/ubuntuone-installer/update-from-trunk/+merge/9424618:57
dobeycan have a review please?19:05
briancurtindobey: i looked and it seems reasonable to me19:07
ralsinadobey: +2 with mine19:09
dobeysigh, gtk-doc is evil19:39
=== nessita-away is now known as nessita
nessitahey briancurtin19:45
briancurtinnessita: hio19:46
nessitabriancurtin: so, you need to also add the signal to linux/dbus_interface.py and windows/ipc.py and ipc_client.py19:46
nessitabriancurtin: that may be the only bit left for you19:46
briancurtinhm, so i have it in there. i guess i just need to test those pieces19:46
nessitabriancurtin: you have that there and is not working?19:47
briancurtinnessita: well i have it in there because i had been following other similar types of notifications, but i havent written the tests yet to ensure that they work (i did it slightly backwards)19:47
nessitabriancurtin: OH NO.... how dare you? :-)19:48
briancurtinthe world didnt end yet, but i'll be on the lookout19:48
briancurtin:)19:48
nessita;-)19:48
dobey1 hr left.20:00
nessitadobey: what for?20:03
dobeyoh no20:03
dobeythat'd be 25 hrs wouldn't20:04
dobeyam thinking it is thursday already :P20:04
dobeynessita: ui freeze20:04
nessitadobey: right, 25 hours, stop scaring me :-)20:04
nessitadobey: I'm adding the new u1cp-common bin package... I see that debian/tmp/usr/share/indicators/messages/applications/ubuntuone-control-panel should also be installed by that package, am I getting that right? (the content of that file is the path to ubuntuone-installer.desktop)20:14
* gatox wants to kill a qlabel20:14
dobeynessita: yes; and you can make that package depend on ubuntuone-installer20:14
nessitadobey: right, and remove that dep for the -gtk one20:16
dobeyright; well -gtk and -qt should depend on the -common20:16
dobeyso will get it that way20:16
nessitadobey: should we consider having the com.ubuntuone.controlpanel.gui.service provided by also the -qt UI? (will require further fixings, but now that I see that, I wonder...)20:17
alecurye, the file you published at the end of your vala+dbus post was very funny!20:18
ryealecu, i suppose this should trigger a natural reflex of opening in every geek20:18
aleculol20:18
ryePLAINTEXT PASSWORD ME WANT!20:18
ryealecu, even i went forward to open it, because it looked so interesting :)20:20
dobeynessita: well i guess not. are we actually using it anywhere?20:20
nessitadobey: I think the we're using that to integrate with the launcher...20:20
dobeythe indicator. we open to the folders tab20:24
dobeywe'll have to make it run the qt panel directly i guess20:24
nessitadobey: and we also set the urgency on the control panel by using that from the launcher, I think20:24
dobeynessita: no20:25
dobeynessita: the urgency is set using the unity launcher api20:25
dobeythough i am not sure it works at the moment (though it should)20:26
nessitabut we also make the controlpanel do something when the --alert is passed20:26
dobeynah, it was just checking if the launcher hint was set, and then unsetting it, when it gets launched20:26
dobeyof course, it can't do that now, becuase of the gi issues20:26
nessitaah. Boo.20:27
dobeythe qt panel could; or the installer could20:27
dobeywe should probably have the installer do that20:27
dobeysince it should be getting run when you click on the launcher, or on a message; and it then opens the control panel20:28
gatoxok..... i'm off for today..... EOD.... see you tomorrow!20:29
nessitadobey: right20:32
j0nrhey folks...21:05
j0nrI am on a machine I want to connect log into my ubuntu one account. But I don't want my ~/Music to try and download (its big).21:06
j0nrwill I gte the chance to not sync that folder?21:06
nessitadobey: is this correct for the new u1cp-common package? http://pastebin.ubuntu.com/853236/ my question is mainly regarding the Depends: ${misc:Depends} which I'm not 100% sure what's that for21:10
mandelj0nr, if that folder is a UDS yes, no problem what so ever21:11
dobeynessita: i think in general, people like the long description to be a bit longer.21:11
nessitadobey: have a suggestion, to understand what other info may be relevant in that desc?21:12
dobeynessita: and you need appropriate Breaks/Replaces; and you might also want to Recommends: indicator-messages or whatever the package is for it21:12
dobeynessita: i don't. i don't really like the needlessly long descriptions21:12
nessitadobey: breaks/replaces for ubuntuone-control-panel-gtk, right?21:13
dobeyj0nr: if it's outside of ~/Ubuntu One/ then it isn't synced by default21:13
dobeynessita: yes21:13
dobeyoh am tired now :-/21:16
j0nrdobey: so if I log in to the U1 client, by default it will only try to sync ~/Ubuntu One ?21:16
nessitadobey: why the Recommends on indicator-messages? so the icon in on the envelop and can display the  shares/udfs/etc messages?21:17
dobeyj0nr: yes; you'll have to subscribe the other folders you want synced21:21
dobeynessita: because the package provides a file that integrates with it21:21
nessitadobey: right, but then why is not a Depends?21:22
dobeynessita: because it's ok if it's not there21:22
nessitaack21:22
nessitaok, I'm gone to pilates, will get back in one hour and will finish the uploads to ubuntu (still waiting on some nightlies build to confirm proper packaging)21:51
nessitabrb21:51
dobeylater all22:14
=== gatox is now known as gatox_away
=== lifeless_ is now known as lifeless
* nessita is back23:24
=== m_conley_away is now known as m_conley

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