/srv/irclogs.ubuntu.com/2012/01/09/#ubuntuone.txt

Kikehello04:09
=== AlanChicken is now known as AlanBell
mandelmorning all!09:36
JamesTaitHappy Monday, everyone! :D10:11
* mandel short coffee making break10:17
* mandel back10:41
gatoxgood morning!11:04
mandelgatox, hola!11:11
torporhello11:28
torporis there any way to get the verification code for a new Ubuntu One account re-sent?11:28
torporit appears i have not received the verification code because the account was not set up before ubuntu tried to send it - but i have now set up the account and it works.11:29
gatoxmandel, do you have time for some reviews?11:36
mandelgatox, sure, I'm a little block while updating the windows vm11:36
gatoxmandel, thanks.... first this one: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/8660511:36
gatoxmandel, and then this one that is incredibly trivial :P https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/remove-network-detect/+merge/8779511:37
mandelgatox, oh, by the way, path_exists will brake all the metadata methods if the user decided to create a lnk there with the same name as the folders used by the metadata11:37
mandelgatox, is the fault of the user to do that.. yet it could happen11:37
gatoxmandel, yep.... we talk about that prroblems.....11:38
mandelgatox, what was the conclusion?11:39
gatoxmandel, documentation.... we know that happend.... until there isn't a better solution, we are going to document what the problems might be11:40
gatoxif i understand correctly...... ralsina ^11:41
mandelgatox, ah, ok..11:41
mandelgatox, why this: 'Remove network detect from sso.' ?11:46
mandelgatox, was it a bug?11:46
gatoxmandel, because we decideed to change the implementation of network detect in the installer, to not depend in sso..... is related to this branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704911:49
mandelgatox, I think you can do a lot better for test_links_target_without_lnk_extension and test_links_target_with_lnk_extension to write less code, both tests do the same and only change the destination path, what about reusing the code?11:49
mandelgatox, write an _assert_read_link(self, target) and pass the target accordingly11:50
gatoxmandel, yep.... right.... changinng that11:52
mandelgatox, also, instead of exists = os.path.exists(path) or is_link(path) and later do   return exists11:52
mandeljust do return  os.path.exists(path) or is_link(path) :)11:52
mandelgatox, do I make sense?11:53
gatoxmandel, yes yes11:55
ralsinamandel, gatox: yes that was the conclusion11:59
ralsinamandel, gatox: sorry about the delay, was buying groceries11:59
gatoxralsina, no problem!11:59
mandelralsina, np :)12:00
mandelralsina, did you get me milk?12:00
ralsinamandel: I did12:00
ralsinamandel: and blueberry jam and bread12:01
mandelralsina, thx ;)12:01
nessitahello everyone!12:01
ralsinahello nessita!12:01
=== teknico is now known as teknico_away
mandelnessita, morning!12:01
mandelralsina, oh, question, none of us is in the desktop rally, right?12:01
nessitahello ralsina, mandel!12:01
gatoxnessita, hi12:02
mandelgatox, this guy is approved: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/remove-network-detect/+merge/8779512:02
gatoxmandel, great.... i'm updating the other one12:03
ralsinamandel, gatox, nessita: anyone needs reviews? (must be less than 6500 lines ;-)12:03
mandelgatox, ok, let me know since I'm wasting a little time waiting for the stupid windows vm to update...12:03
gatoxi need a review from nessita :P https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704912:03
gatoxfor IRL it should use this sso: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/remove-network-detect/+merge/8779512:04
nessitagatox: I will do both I owe you (inclidung fix-links)12:08
gatoxnessita, thanks! :D12:08
mandelherb, what I bloody irony, couchdb creator leaves the project and wants to port more and more of the code from erlang to c/c++ ..12:10
ralsinamandel: it's not called irony, it's called finding the right medication12:11
mandelralsina, hehe true :)12:11
mandelralsina, at least we ran away on time12:12
ralsinamandel: no we didn't. But anyway, I'll get back to coding a bit.12:12
herbmandel: hi12:12
mandelI'll get back to check wtf is windows doing :)12:12
mandelherb, hello12:12
herboh sorry. I just saw the hilight. didn't read that it was about couchdb and not directed at me.12:13
mandelherb, haha sorry, I did not know that my funny sound was a nickname :P12:14
herbor, you know, my real name too. :)12:14
mandelherb, yeah, it was more of a spanish onomatopoeia that anything else :)12:15
mandelsorry :)12:15
herbno worries12:15
* mandel hates windows updates...12:37
ralsinagatox: answer this question if you can: http://askubuntu.com/questions/93458/ubuntu-one-not-work-in-windows-7-russian-edition (no rush)12:40
* gatox checking....12:41
nessita@ping12:46
ubot4pong12:46
ralsinamandel: if you can take a look at https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_809873/+merge/67911 again...12:48
mandelralsina, right on it!12:48
mandelralsina, in line 300 of the diff, is that needed?12:55
mandelfinalize_options == pass ?12:56
ralsinamandel: yes12:56
ralsinamandel: if it's not defined, it throws an exception12:56
ralsinamandel: and the parent class doesn't define it. No idea why12:56
mandelralsina, we are still using os.system to call python rather than subprocess...12:57
mandel357 is an example12:58
ralsinamandel: let me look. Must confess I didn't search for it...12:58
mandelralsina, not big deal :)12:58
mandelralsina, is just not to have two diff ways to do something in the same code, is confusing to a new comer12:58
ralsinamandel: but subprocess [way, to, call, things] is ugly! ;-)12:59
mandelralsina, I know... but is what we have..13:00
ralsinamandel: let it pass, please13:00
ralsinamandel: I am too bored of that branch ;-)13:01
mandelralsina, If you give me a bug number for it I'll let it pass :)13:01
ralsinamandel: file at will ;-)13:01
* ralsina assigns bug to mandel13:01
mandelralsina, ok..13:01
mandelralsina, FYI bug 913765 is assigned to you, feel free to manage it and give it to someone else :)13:03
ubot4Launchpad bug 913765 in ubuntuone-windows-installer "Be consistent in the use of subprocess and os.system (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/91376513:03
ralsinamandel: it's ok, I promise to fix it myself13:03
ralsinamandel: I want this to go away :-)13:03
mandelralsina, I understand, one question, the vista dll, is it needed in all the diff versions of windows, or just vista?13:05
ralsinamandel: needed for vista an 7, is a noop on xp13:05
mandelralsina, ok.. I was wondering if we could be smart and tell the installer not to copy it to the system when is not needed, is it possible?13:06
ralsinamandel: we can even delete it after install13:06
mandelralsina, I think it will be nice to do so, so that we just use the really needed stuff, I'll file a diff bug for it :)13:06
ralsinamandel: it's just that wizard can't do it (because it's not running as admin) and installbuilder can't do it (because it's finished ;-)13:07
ralsinamandel: maybe put it in a temporary location so it deletes itself eventually13:07
mandelralsina, in temp or there is a way to tell the system to delete a file in the next reboot13:07
mandelralsina, I'll just add the bug so that we can land this13:07
ralsinamandel: right13:08
ralsinamandel: in any case, it's less than 100KB13:08
mandelralsina, I'll say is more completeness than anything else, it can be flagged as a minimun bug or even not fixing, is just to keep track of this things13:09
ralsinamandel: cool13:09
alecuhello all!13:10
gatoxalecu, hi13:10
mandelralsina, how come there are paths that use the linux separator in the xml, example: <windowsIcon>${installdir}/ubuntu_one.ico</windowsIcon>13:10
nessitagatox: several tests in the windows-installer branch are failing... will I be missing something?13:10
mandelralsina, line 56513:10
nessitahola alecu!13:10
mandelalecu, buenos dias!13:10
ralsinamandel: it's what InstallBuilder expects13:10
gatoxnessita, mmmmmmmm let me check again..... but it shouldn't13:10
nessitagatox: on linux this is13:10
mandelralsina, and then we have this: <windowsExec>${installdir}\dist\ubuntuone-installer-qt.exe</windowsExec>13:11
alecuhey guys, today I'm at the in-laws, and with a shitty wi-fi13:11
nessitagatox: also,13:11
nessitaubuntuone_installer/gui/qt/tests/__init__.py:13:11
nessita    178:  [C0111, FakeWizardButtonStyle] Missing docstring13:11
mandelralsina, if it is like that.. WTF installBuider!13:11
ralsinamandel:  hmmmm both work,I can make them more consistent. Some of those I added editing XML (have \), some were added by the GUI (have /)13:11
mandelralsina, it just got my attention that we are using both13:12
nessitagatox: the lint issue comes from trunk, if you could fix it, if qould be great13:12
nessitawould*13:12
gatoxnessita, yep... i'm checking..... because i had fix that one......13:12
ralsinanessita, dobey: looks like we don't have lint checks on tarmac on windows-installer, can we add that?13:12
nessitaralsina, mandel, alecu, gatox: remember that tarmac is still not running any verify_command for windows installer, so please, when doing reviews, run the installer tests in windows and linux. Thanks! (ATM we have a lint issue in trunk)13:13
ralsinanessita: ack13:13
nessitaralsina: ah, just see what I pasted ^ :-)13:13
mandelnessita, ok13:13
alecunessita, ack13:13
mandelralsina, regarding line 399, can we add that text in a const var, it will look nicer, right?13:14
nessitagatox: pasted test errors in the MP so you can tell me if I'm missing something13:15
ralsinamandel: 399? os.path.join("data", "revnos.txt") ?13:15
gatoxnessita, ok13:15
mandelralsina, yes, what is written inside the with statement13:15
ralsinamandel: the revnos are obtained a few lines before, and only used there13:15
mandelralsina, also, I think it will be nice to look at clientdefs.py.in and try to generate that too, rather than a copy of the windows folder13:16
mandelralsina, I mean, REVNO_TEXT = 'ubuntu-sso-client:%s'13:16
ralsinamandel: good idea. File one for that too :-)13:16
mandelralsina, ok13:17
ralsinamandel: I meant file one for the clientdefs13:17
mandelralsina, I suppose you are talking about the clientdefs.py.in :)13:17
mandelack13:17
ralsinamandel: I don't quite like the constant for that, it just seems verbose13:17
mandelralsina, well, we can leave it like that then13:17
ralsinamandel: let's ask for a third opinion to break the tie :-)13:17
mandelralsina, nah, es una majaderia, it is more a personal style thing than anything else :)13:18
ralsinamandel: ack13:18
mandelralsina, FYI bug 91377213:19
ubot4Launchpad bug 913772 in ubuntuone-windows-installer "Generate clientdefs.py in the package like it is done on linux (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/91377213:19
ralsinamandel: thanks13:19
ralsinamandel: you know that's generated by autotools, right? ;)13:20
mandelralsina, we do this: "..", "..", "installed", "Lib", "site-packages", a lot, I think in this case we should store that in a var13:20
mandelralsina, yes, I know, is not an easy task, but we should unify that process a little13:20
ralsinamandel: looks like just a dumb template so it should be doable. I do it already for logging.conf13:21
mandelralsina, yeah, I though so, I do remember a little how the file looks like13:22
nessitaalecu: were you able to move forward with the review, on last Fri?13:22
ralsinamandel: agreed about making that a variable13:22
mandelralsina, is not something for now, but I think in the long term is worth doing it, right?13:22
ralsinamandel: please put a needs fixing for that13:22
mandelralsina, sure13:22
ralsinamandel: well, I would prefer not making ubuntuone-client's python stuff need automake, but we are not having that discussion yet ;-)13:23
mandelralsina, I know, it is a recurrent comment we have hehehe13:24
ralsinamandel: fixed the repeated path13:29
mandelsweet13:30
gatoxnessita, mmmmmmm something doesn't have any sense..... the lint issue.... if you look at the code.... the docstring iis there.... you can see it in the diff13:30
alecunessita, no, I didn't got to the big review it on friday, I'll be doing it today13:30
mandelralsina, did you push it?13:31
ralsinamandel: wait, push failed13:32
ralsinamandel: damn pageant that doesn't open on start13:32
ralsinamandel: pushed13:32
mandelralsina, ok13:33
mandelralsina, my typing is contagious -> borked13:35
ralsina?13:35
mandelralsina, in the branch mp :)13:36
ralsinalooking...13:36
ralsinamandel: I don't see any new comments13:37
mandelralsina, oh, do you want me to add it in the needs fixing?13:37
mandelsure13:37
ralsinamandel: I don'tknow what you are talking about :-)13:38
mandelralsina, line 37913:38
mandelralsina, ->  # Copying by hand because the install is borked13:38
nessitagatox: ping13:38
ralsinaoh, "borked" is a real word ;-)13:38
mandelborked :)13:38
gatoxnessita, pong13:38
gatoxnessita, did you get my mmessage?13:38
ralsinamandel: http://www.urbandictionary.com/define.php?term=borked13:38
nessitagatox: no13:38
nessitagatox: can  you please repeat?13:39
gatox<gatox> nessita, mmmmmmm something doesn't have any sense..... the lint issue.... if you look at the code.... the docstring iis there.... you can see it in the diff13:39
ralsinamandel: borked "Specifically used to describe technology that is broken."13:39
mandelralsina, ah.. did not know that :)13:39
ralsinamandel: you have been typing correctly all along ;-)13:39
nessitagatox: I will re branch and re test. Re the fix-links branch: the new make_link impl has an except for AttributeError13:39
nessitagatox: why did you leave that there?13:39
* gatox checckiing....13:40
mandelralsina, new years goal :)13:40
nessitagatox: you removed the comment that says "This try-except is required at least for the current..." but the except should have be removed as well, I think13:40
gatoxnessita, ahhh sorry13:41
nessitagatox: but you tell me if leaving that there makes sense :-)13:42
mandelralsina, rest look perfectly ok, I'll go for lunch and will run the lint checks against it as nessita recommended and will approve then, is that ok?13:42
gatoxnessita, checking..13:42
ralsinamandel: pylint is probably going to have a seizure with this code13:42
ralsinamandel: pep8 likes it though ;-)13:42
gatoxnessita, nop.... no sense13:42
gatoxremoving....13:42
mandelralsina, ok, we will take a look, hopefully is not much :)13:42
nessitaralsina: we run u1lint over windows-installer in linux, so you can do that13:43
nessitaralsina: or I can run it for you and pass you the output13:43
ralsinanessita: I can do it, np13:43
mandelok, lunch for me13:43
* mandel lunch13:43
ralsinanessita: it complained about weird stuff last I checked (like, importing classes with too many methods)13:44
nessitaralsina: on windows or linux?13:44
ralsinanessita: on windows, but it was correct ;-)13:44
nessitaralsina: in my env, u1lint will work fine for windows-installer. On windows, may not be using the correct pylintrc?13:44
ralsinanessita: I can run it on linux too and do a lint-check pass there13:45
ralsinanessita: these are new files, so13:45
nessitaralsina: so...? (am I oflfine again :-))13:46
ralsinanessita: so it's new problems :-)13:47
nessitagatox: ah, the lint issue appears only in trunk,  you're right. The thing is that I also run the tests there to confirm the errors from your branch were new13:49
gatoxnessita, ok..... fixing that13:50
nessitagatox: but the test failures are still there13:50
nessitagatox: fixing what? :-)13:50
gatoxnessita, the test13:50
nessitagatox: ack, let me know13:50
gatoxnessita, ok!13:51
nessita@ping13:55
ubot4pong13:55
gatoxnessita, still network issues?13:55
nessitagatox: is wireless adapter issues... seems like the wireless driver sucks bug time13:56
gatoxahhhhh13:56
nessitagatox: it just hangs and I have to rmmod + modprobe it13:56
gatox:S13:56
nessitagatox: which is a much nicer workaround that rebooting13:56
gatoxyep13:57
nessitagatox: I run u1lint directly on your windows-installer branch and got what I added as last comment in https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704913:58
ralsinacome on, pylint, "f" for a file is perfectly reasonable and traditional!13:59
gatoxnessita, :S ok....13:59
ralsinanessita: u1lint is failing for me (https://pastebin.canonical.com/57879/), but I am getting 10.0 score from pylint on setup.py of branch https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_809873/+merge/67911  could you do a quick pass?14:04
ralsinanessita: mind you, that would be scripts/setup.py14:04
nessitaralsina: sure!14:06
nessitaralsina: and the naming convention are setting we put ion pylintrc14:06
ralsinanessita: it's ok, I put reasonable meaningful names now, was just venting ;-)14:06
alecunessita, I'm having the same issue with my wi-fi driver in P.14:07
nessitaralsina: :-)14:07
alecubut that's the main complain with P actually, so it's not such a big deal.14:07
nessitaralsina: u1lint will not take arguments, FYI14:07
ralsinanessita: ok, that explains it14:07
nessitaalecu: is it also a realtek driver?14:07
nessitaalecu: this will "fix" it for me: sudo rmmod r8192se_pci && sudo modprobe r8192se_pci14:08
alecunessita, the weird thing is that the *ethernet* driver malfunctions similarly too.14:08
alecunessita, :-)14:08
nessitaalecu: FYI, I'm still in natty in the laptop14:08
nessitabut seems like realtek driver are not good, no matter what version :-P14:08
alecunessita, the driver that P is using is rtl8192se, but probably it was renamed in the kernel.14:09
alecunessita, lspci identifies my wi-fi like this:14:09
alecu03:00.0 Network controller: Realtek Semiconductor Co., Ltd. RTL8191SEvB Wireless LAN Controller (rev 10)14:10
nessita02:00.0 Network controller: Realtek Semiconductor Co., Ltd. RTL8191SEvB Wireless LAN Controller (rev 10)14:10
nessitathe same s***ty one!14:10
nessita:-P14:10
alecufu**ing cheap*$$es lenovo!14:10
nessitaalecu: got my question re: my sso branch?14:13
alecunessita, yes, and I answered it, and looking in the logs it seems like you disconnected just after it.14:14
nessitaalecu: very likely :-/14:14
alecunessita, <alecu> nessita, no, I didn't got to the big review it on friday, I'll be doing it today14:14
nessitaawesome, thanks!14:15
alecunessita, I'll probably trade that review for one or two of mine :-)14:15
nessitaalecu: shoot!14:15
ralsinanessita, alecu: I think I have a reliable USB wifi dongle somewhere. I could send it to one of you :-)14:16
nessitaralsina: actually, I have 2, I should try that14:16
nessitaralsina: so feel free to send to alecu :-)14:16
* alecu had one usb dongle that got broken of carrying it in the backpack14:17
ralsinanessita, alecu: well, it's in buenos aires, so I can send it next saturday ;)14:18
nessitagatox: can you please add to the fix-link MP instructions to install the new dependency?14:19
gatoxnessita, i added that to the wiki..... should i include the link of the wiki?14:19
nessitagatox: just the new dependency in the MP, please14:19
gatoxnessita, okas14:19
nessitaralsina: (valid) lint issues for your branch http://pastebin.ubuntu.com/798237/14:21
alecunessita, the first branch I need reviews on is this: https://code.launchpad.net/~alecu/ubuntu-sso-client/proxy-integration-tests/+merge/8753814:21
ralsinanessita: thanks!14:22
nessitaalecu: looking14:22
alecunessita, you may need to put devtools trunk in PYTHONPATH14:22
nessitaalecu: what revno?14:22
alecunessita, I have revno 52 here. It's the one where the branch mentioned in the MP was merged.14:23
nessitaI have 3.1+r52-14~natty114:23
nessita(installed)14:23
nessitaso seems like I should not need devtools trunk, no?14:23
alecunessita, cool. I don't have nightlies in this P14:23
alecunessita, right14:24
ralsinanessita: I am disabling W0404 because it's a false positive (I am not importing anything in line 35, it's a def)14:27
ralsinanessita: pushed revno 95 for another try when you have a minute (no rush)14:28
=== m_conley_away is now known as m_conley
dobeyhrmm14:33
nessitaralsina: you should also disable the following so we get a clean run on linux too:14:33
nessita    100:  [F0401, get_py2exe_extension] Unable to import 'py2exe.mf'14:33
nessita    103:  [F0401, get_py2exe_extension] Unable to import 'win32com'14:33
ralsinanessita: sure14:33
dobeyralsina: we can add the lint checks to tarmac, but i suspect it will be a massive bucket of failures on linux, without all the win32 libs we need14:34
ralsinadobey: right. But everything windows-specific is on files called windows.py :-)14:34
nessitadobey: actually no, I run that all the time in linux, is not failing14:34
nessitadobey:  we have the proper disables in place14:35
dobeyralsina: lint doesn't care if they're called os2.py :)14:35
ralsinadobey: but we can ignore them14:35
dobeynessita: ah, ok14:35
nessitagatox: you added the instructions to the MP so I can use that?14:35
gatoxnessita, yep14:35
nessitadobey: hello, btw :-)14:35
dobeyhola nessita14:35
nessitagatox: where?14:36
gatoxnessita, in the description14:36
nessitagatox: all I have is "To test in real life, you need to execute the installer with the "--installer param.14:36
nessitaAnd configure the VM enabling the net adaptar for the traditional flow  and then disabling the net adapter to see the Network Detection page."14:36
ralsinanessita: pushed that ignore14:36
dobeymandel: you need to fix your bzr config to use the long e-mail address, or add the short one to launchpad :)14:37
ralsinadobey: fixed that this morning :-)14:38
nessitagatox: so? :-)14:38
nessita(am I missing messages?)14:38
gatoxnessita, here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/8660514:39
gatoxthe description of the change14:39
nessitagatox: oh, you're right, I was reloading the network-detect proposal :-P14:39
gatox:P14:39
nessitaalecu: ./ubuntu_sso/utils/webclient/tests/test_gsettings.py:56:17: E202 whitespace before ')'14:40
nessitagatox: I have 5 tabs with MP in them :-D14:40
gatoxnessita, fun morning! :P14:40
nessitayou say fun, I say...14:41
nessita:p14:41
gatoxfun was sarcastic :P14:41
nessitagatox: ;-). One tiny note: you don't need to duplicate the same thing you put as commit message to be also merge description. You can either leave the merge descritpion empty, or add relevant info to test IRL, for example14:42
nessitabut the field can be empty14:42
gatoxnessita, ahhhh ok14:42
nessitaralsina: seems clean re: u1lint your branch now14:43
ralsinanessita: cool14:44
ralsinanessita: thanks!14:44
alecunessita, gatox: I've been always copying merge description and commit message.... should we only put commit message?14:44
gatoxalecu, the same here14:45
ralsinaalecu, nessita, gatox me too (when I put long IRL explanations, manuel complains, BTW ;-)14:47
nessitaalecu, ralsina: from my POV, there is no gain into duplicating the commit message... no?14:48
nessitathe field can be empty, so I don't see why putting the same as commit message14:48
ralsinanessita: well, the empty space on the page makes me feel guilty? ;-)14:48
nessitaand I'm +1 to IRL tetsing instructions, almost always! :-)14:48
nessita(except for trivial branches)14:49
ralsina+1 on using it for IRL instructions, if they are unusual ("run it and see that the bug is not there" is not unusual)14:49
nessitagatox: can't install comtypes, I'm getting "The filename, directory name, or volume label syntax is incorrect. Could not open archive"14:51
nessitagatox: that's when running the .exe14:51
gatoxnessita, :S weird...... can you try with the .zip?14:51
nessitagatox: did you install it from your weird-unicode-username account?14:51
nessitagatox: is an .exe14:51
nessitagatox: ah, the zip, will try14:52
gatoxnessita, there is also the .zip in the linux instruction.... can you try that on windows....14:52
gatoxnessita, if that work, i'll remove the .exe reference..... and leave only the zip instruction14:52
mandeldobey, I've done both14:55
mandeldobey, ralsina told me that I made you loose some time, sorry14:55
mandel/me back14:55
mandelralsina, did you run lint on the branch for the installer?14:55
ralsinamandel: yes, should be lint-clean on windows and linux14:56
ralsinamandel: verified by nessita14:56
mandelralsina, cool, I'll quickly run it in my system and will approve then :)14:57
ralsinamandel: awesome!14:57
dobeymandel: thanks14:58
nessitaralsina: un clon de qué en dónde?14:58
ralsinanessita: un clonazepam... para el panico ;-)14:59
mandeldobey, sorry again, I did not noticed that I used the short version14:59
nessitaralsina: lol14:59
nessitaralsina: this laptop has no cdrom :-P14:59
mandeldobey, on p, is there a new lint version?15:00
dobeyhttp://www.japansubculture.com/2012/01/its-no-ordinary-sale-its-a-fuckin-sale/15:00
dobeymandel: there is a new pylint it seems, yes15:00
mandelputo puto puto15:00
dobeywe need to switch to pyflakes only :)15:01
mandelralsina, nessita I'll have to trust you regarding the lint state of this lp:~ralsina/ubuntuone-windows-installer/fix_809873 and approve, is that ok?15:01
nessitame15:01
nessitamandel: yes15:01
mandelme15:01
ralsiname15:02
dobeyme15:02
nessitaalecu, gatox: standup?15:02
gatoxme15:03
nessitaDONE: freaky friday: continued removing markers from u1client, proposed for merging the branch for fixing IPC in sso15:03
nessitaTODO: reviews, apparently a lot :-D, help new hire brian get tickets to sprit15:03
nessitaBLOCKED: nopes15:03
nessitaNEXT: mandel15:03
mandelDONE: Reyes Magos on Friday. Fixed my brz whoami to avoid annoying other team members, sorry again. Done reviews for ralsina and gatox. Work on bug 907511. Had to fight a little with vmware and windows.15:03
mandelTODO: run tests for fix of 907511 and hopefully propose it. Sync with alecu regarding which proxy bugs I can tackle.15:03
mandelBLOCKED: no15:03
ubot4Launchpad bug 907511 in ubuntuone-client "Windows: events from directories where no watch has been added should be ignored (affects: 1) (heat: 31)" [High,In progress] https://launchpad.net/bugs/90751115:03
mandelralsina, please15:04
ralsinaDONE: finished with the windows packaging branch! sprint details finished (so far), canonicaladmin, working some more on the buildout (almost there), general VM housecleaning TODO:  propose buildout for merging after packaging branch, bug triage, hiring, etc. BLOCKED: no15:04
ralsinadobey?15:04
dobeyλ DONE: hackday, some gireactor work15:04
dobeyλ TODO: write proposal for u1 packageset, finish gireactor work, start music store/libsyncdaemon work15:04
dobeyλ BLCK: No.15:04
dobeygatox:15:04
gatoxDONE:15:04
gatoxReimplement network detect in the installer an sso. Review some of the bugs in my queue.15:04
gatoxTODO:15:04
gatoxFix several test in linux for the installer branch15:04
gatoxBLOCKED:15:04
gatoxNo15:04
gatoxalecu, go15:04
ralsinagatox: another duplicate for your unicode homedir branch, I think: https://bugs.launchpad.net/bugs/91350715:04
ubot4Launchpad bug 913507 in ubuntuone-client "ubuntu one-windows crahes (affects: 1) (heat: 6)" [Undecided,New]15:04
nessitacomments anyone?15:05
gatoxralsina, roget that15:05
ralsinaEOM?15:05
nessitaeom! (I guess alecu will paste his standup when he sees this)15:05
gatoxralsina, can i assign that to me?15:06
ralsinagatox: of course15:06
alecudamn15:06
ralsinagatox: but maybe you should just mark it duplicate15:06
alecumy irc beep is not working at all15:06
gatoxralsina, ah ok15:06
alecu@ping15:06
ubot4pong15:06
alecuubot4, ping15:06
ubot4pong15:06
ubot4another contentless ping... sigh...15:06
mandelubot4, ping15:07
ubot4pong15:07
mandelubot4, puto?15:07
ubot4Factoid 'puto?' not found15:07
mandelhehe15:07
mandelEOM?15:08
alecuDONE: worked on restful client replacement, and on another branch to use it15:11
alecuTODO: fix restful on P, do reviews15:11
alecuBLOCKED: no15:11
alecumandel, I need your review on this branch: https://code.launchpad.net/~alecu/ubuntu-sso-client/proxy-integration-tests/+merge/8753815:13
mandelalecu, on it15:13
alecumandel, and probably this one too: https://code.launchpad.net/~alecu/ubuntu-sso-client/silence-dbus-logger15:14
* mandel looks15:14
nessitagatox: I was taking to you a lot, and realized my connection was stuck :-)15:15
nessitaI was saying:15:15
nessita(12:06:36 PM) nessita: gatox: why not mark it as duplicate of the master bug?15:16
nessita(12:09:28 PM) nessita: gatox: in fix-links, you're modifying run-tests.bat15:16
nessita(12:09:32 PM) nessita: gatox: any idea why?15:16
nessita(12:10:34 PM) nessita: gatox: (in fact, that change makes running test fail on windows)15:16
gatoxnessita, i already mark it as duplicate15:16
nessitagreat15:16
gatoxnessita, i don't recall exactly the reason...... i think that maybe mandel or dobey knows..... but it was failing in the other way.... i'll test it again15:17
gatoxnessita, about run-tests.bat   ^^15:18
dobeyhuh?15:18
mandelnessita, when ubuntu-one-dev tools was changed to use the twisted option parser for some reason we don't know we had to make that change, right dobey ?15:18
dobeymandel: oh, the argument ordering?15:19
mandeldobey, yes15:19
nessitagatox: but your branch is "reverting" the change needed to run properly15:19
nessitagatox: the directory with tests should be at the end of the test command line15:19
dobeymandel: yeah, it only became obvious after we landed those changes15:19
gatoxnessita, ok.... i'll change that now15:19
nessitagatox: and your branch is moving it back to the middle15:19
gatoxnessita, reverted15:21
nessitagatox: added a needs fixing with several tets failures I got in that branch15:21
nessitagatox: I ran the suite under my complex-unicode-username account15:21
gatoxbrb.... need to pay some bills!15:21
=== gatox is now known as gatox_brb
mandelalecu, are this two the same? 'parsed_value = True if value == 'true' else False' and 'parsed_value = value == 'true'15:22
mandelalecu, line 93 of https://code.launchpad.net/~alecu/ubuntu-sso-client/proxy-integration-tests/+merge/8753815:22
alecumandel, yes, it's the same construct. I think the former looks nicer, though.15:25
mandelalecu, may I know why? just curious :P15:26
alecumandel, "looks nicer" is a purely subjective appreciation.15:26
alecumandel, for instance, you mom may say you "look nice", but most people would disagree.15:27
mandelalecu, ah, but you must have a reason, this car looks nicer that the other because I like convertibles instead of sub15:27
mandelalecu, same with the mother, I look nicer because I'm her son :)15:27
alecumandel, I don't like " x = y == z"15:28
alecumandel, too many equals together15:28
mandelalecu, ah, see, you had a reason :)15:29
alecumandel, I made that reason up just now!15:29
alecumandel, like with all "nicer" stuff, it's just gut feeling15:30
mandelalecu, I don't care about the timing ;)15:30
alecumandel, so, I may change it if you still think = == looks nicer.15:30
mandelalecu, I have no preference, I was wondering why typing more, that is all15:30
alecumandel, let's ask the team:15:31
alecuhey all15:31
alecuwhat do you prefer?15:31
alecu+            parsed_value = True if value == 'true' else False15:31
alecuor15:31
alecu+            parsed_value = value == 'true'15:31
alecunessita, gatox_brb, ralsina, dobey ^15:32
nessitaalecu: definitely the second15:32
ralsinaalecu: a needle in my left eye? ;-)15:32
ralsinathe second, with () around the comparison15:32
dobeyalecu: i prefer foo = bar == 'true' ? True : False15:32
dobeybut alas15:32
alecudobey, lol15:32
ralsinaparsed_value = (value =='True')15:32
nessitaalecu: in the university, we chop off students hands when they do:15:33
nessitaif (something == true) { ...}15:33
nessitawhich kinda applies to this case15:33
dobeybut what ralsina said15:33
alecuyup, I like the version with parenthesis the most too.15:34
alecumandel, I'm fixing that.15:34
mandelack15:34
alecumandel, damn: the tests in that branch are now failing with squid315:36
alecumandel, (I used to ran those tests in Oneiric)15:36
mandelalecu, I was going to mention that, but we know hot to fix that :)15:37
mandelalecu, it has landed in ubuntuone-dev-tools and I think we should provide MockWebserver for the tests15:37
alecuyay, got accepted! http://wiki.qt-project.org/QtonPi/Device_program/Accepted15:41
mandelalecu, what is that project for?15:42
ralsinaalecu: Qt on the raspberry pi? Cool! :-)15:45
ralsinaalecu: didn't know you were such a big Qt fan! ;-)15:45
alecuralsina, in fact, I'm a huge fan of cheap fanless hardware.15:46
alecu"User:alecu (Alejandro J. Cura), port Ubuntu One to Raspberry Pi. I work on the client side of Ubuntu One, and I'd like to see these small devices have access to all kinds of big files stored in the cloud. So I'm proposing to make the Ubuntu One Qt based client work with the Raspberry Pi."15:46
dobeyi still haven't even turned this i.mx53 board on15:46
=== gatox_brb is now known as gatox
gatoxback15:46
elopiogatox: is there an open bug for ascii issues on windows?15:49
gatoxelopio, ascii no.... for unicode you have this: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/85135615:50
ubot4Launchpad bug 851356 in ubuntuone-control-panel "QT UI: Folder list may show garbage for user homes non-ascii (affects: 2) (dups: 1) (heat: 15)" [High,In progress]15:50
gatoxamong others..... all related15:50
elopioright, that's what I meant :p15:51
elopiogatox: thanks for the link15:51
gatoxelopio, np15:51
ralsinaalecu: the pi has very limited storage, so syncing is not going to be useful, unless I am missing something15:52
alecuralsina, perhaps not running a full syncdaemon, but probably will make sense using the storage api15:52
ralsinaalecu: hopefully you can figure out something cool :-)15:53
dobeywell, you can add storage to it easily15:53
dobeyi see USB ports :)15:53
ralsinaOff for lunch!16:07
mandelalecu, I'm going to the pharmacy to buy some drugs I need, I'll be back in 10' if you need me16:09
alecumandel, great, thanks.16:09
mandelalecu, I'm back16:25
nessitaneed to reboot16:26
nessitawill have lunch in the mean time16:26
alecumandel, I think we should remove this:16:27
alecu"""        # we disable the timeout in the tests, we will deal with it manually.16:27
alecu        self.timeOut = None"""16:27
dobeylunch time, bbiab16:27
mandelalecu, sure, does it work without setting the timeout to None?16:27
alecumandel, I think it should, because one is the timeout we force of the server socket, while the other is...16:31
alecumandel, nevermind :P16:32
mandelalecu, looping call thingy?16:32
alecumandel, no, I thought that you were disabling the timeout of the tests, but it's just the timeout of the "Site" object, whatever that is.16:33
alecumandel, so don't worry about removing that timeout, my mistake.16:33
mandelalecu, yes, it has nothing to do with the tests timeouts, is to prevent some __timeout looping call used in the Site which is used unless you set it to None which then de-activates it16:34
alecumandel, that's right, cool.16:34
alecumandel, so we should keep it.16:34
gatoxsomethinggggggg is really weird with this tests..... :S16:36
mandelgatox, they pass:?16:36
alecu"I want to shoot... The whole day down, down, down... Shoot it all down!"16:36
gatoxmandel, no.... they dont..... and all the code is commented.... the function only has the docstring16:36
gatoxsomething with some defer...... really weird......16:37
* gatox debugging16:37
mandelgatox, may I see the code?16:45
gatoxmandel, here is the paste of the error: http://pastebin.ubuntu.com/798185/16:46
gatoxmandel, this is the branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704916:46
gatoxmandel, the last error in the paste16:46
gatoxmandel, for some reason a defer is failing or somehitng16:46
gatoxmandel, i'm looking of what might be wrong...... in windows works16:47
mandelgatox, so it fails on ubuntu?16:47
mandelgatox, which version?16:47
gatoxmandel, O16:47
mandelgatox, I'm getting: RuntimeError('To make asynchronous calls, receive signals or export objects, D-Bus connections must be attached to a main loop by passing mainloop=... to the constructor or calling dbus.set_default_main_loop(...)',)16:54
gatox:S16:54
mandelgatox, from here: /usr/local/lib/python2.7/dist-packages/ubuntu-sso-client/ubuntu_sso/networkstate/linux.py(129)find_online_state()16:54
mandelgatox, know the issue, or at least how it looks to me, is that the dbus mainloop is not set up for it to work correctly, now, I have not look at the code, but I'd take a look if, first you are using a test that suports dbus16:55
mandelgatox, that is, DBusTestCase should be the base of it so that the dbus service gets started16:55
mandelgatox, but before you get down to all that work, use  dbus.set_default_main_loop(...) in the test and test if it works correctly16:56
gatoxmandel, ok! thanks16:57
mandelgatox, try with something like this: http://pastebin.ubuntu.com/798388/16:58
gatoxmandel, in the test?16:58
nessitamandel: how are you trying to run that? gatox should not be needing to set any dbus loop as default16:58
dobeyno16:58
mandelwait, waith16:59
dobeythe DBusTestCase needs to be used16:59
mandelI'm telling him to see if the main loop is running for dbus to work, because it is not16:59
gatoxnessita, u1trial --gui --reactor=qt4 -t test_check_connection_with ubuntuone_installer/gui/qt/tests/test_gui.p16:59
mandeldobey, I mentioned that already a few mins ago16:59
gatoxnessita, with export PYTHONPATH:=.....16:59
gatoxwindows-installer16:59
nessitamandel: that code should not be used in modules that are libraries, only in executale scripts17:00
nessitamandel: what's the problem specifically?17:00
nessitagatox: ^17:00
nessita@ping17:00
ubot4pong17:00
gatoxnessita, the errors in your paste: http://pastebin.ubuntu.com/798185/17:00
gatoxthe last one that involves a lot of tests17:01
mandelnessita, the problem is that his tests are failing because when it calls the sd to ask for the network state it fails, the exact failure is from /usr/local/lib/python2.7/dist-packages/ubuntu-sso-client/ubuntu_sso/networkstate/linux.py(129)find_online_state()17:01
dobeywhy is ubuntuone-windows-installer using dbus anyway?17:01
mandeldobey, good question..17:01
nessitagatox: you should patch that, no?17:01
mandeldobey, is using sso to know the network state, which uses dbus17:01
dobeyon linux, right; hrmm17:01
nessitamandel: the problem is that gatox should patch that in the windows-installer test17:01
nessitadobey: the network_state is a multiplatform method that will use dbus in linux, and something else on windows17:02
dobeyyeah, patch it17:02
mandelnessita, yes, sounds like that17:02
gatoxi see......17:02
gatoxok..... patching it17:02
nessitamandel: remember to never set the main loop default unless you're building a script that starts a dbus service, or that connects to a dbus service17:02
mandelnessita, it was for debugging, not a solution17:03
mandelnessita, mainly, it was, set it, does it work, bingo you know the issue, now, fix it17:03
nessitamandel: but even if for debugging, let's suppose it worked with that... what to do next? :-)17:03
mandelnessita, use the DBusTestCase that tell u1trial to start dbus for you, and mock the object on the other side of dbus, or patch the original call and just state is called17:04
nessitagatox: understand the problem? your installer branch is querying the real network service :-)17:04
gatoxnessita, yep.... i see it now also with a print17:05
nessitamandel: right, but since we're building multiplatform code, using DBustestCase is not an option17:05
nessitabecause the multiplatform specifics are already isolated in sso17:05
mandelnessita, uh, you are right, I forgot about that detail, sorry, but it was just a way to point gatox in the right direction17:05
nessitamandel: thanks... I must confess I got scared when I saw you advising that :-P17:06
mandelnessita, I understand, out of context it does sound terrible :)17:06
gatoxnessita, mandel thanks! now i have more idea what is going on17:06
dobeynessita: me too :)17:07
nessita@ping17:33
ubot4pong17:33
mandelralsina, nessita, gatox, dobey  so we lost alecu due to energy issues17:51
nessitaoh17:51
gatoxyep17:51
nessitaI'm about to be next17:51
gatoxor he wanted just to take the day off17:51
gatoxjejeje17:52
nessita(very likely, temperature is almost 40C)17:52
gatoxnessita, yep....17:52
gatox:S17:52
gatoximposible to go outside17:52
ralsinaoh, right "my notebook is overheating" <quit> ;-)17:52
dobeynessita: get a block of ice for your computer17:52
gatoxralsina, :P17:52
nessitaepec will shutdown the power any minute now :-)17:52
nessitadobey: I'm with the AA on, but the power supplier usually explodes when th whole city uses it17:53
nessitadobey: and given the current C, I bet energy will go out any minute now17:53
* beuno cranks the A/C down to 16 deg just un case that happens here17:53
gatoxnessita, move to nueva cordoba17:54
gatoxnessita, we always have power17:54
dobeynessita: solar panels :)17:54
dobeybeuno: *brilliant*17:54
nessitagatox: my mean side is hoping that epec will leave *you* without any A/C17:54
nessita:-D17:54
gatoxnessita, jeje17:56
gatoxnessita, it was an honest suggestion mine.... not to make you angry17:56
beunogatox, FIGHT!17:56
nessitagatox: the power supplier should provide the same "quality" service to the whole city17:57
gatoxnessita, obviusly!!.... but we live in argentina :P17:57
nessitaheh17:57
ralsinaelopio: in general, bugs < 500000 are not very useful  because we are not using that code anymore ;-)17:57
mandelnessita, gatox that is the same in spain, plus they cut the water in some areas17:58
mandelwhich is annoying when you need to use the loo17:59
gatoxno water...... and to hot because there is no power for fan or air conditionar......17:59
gatoxmmmmmmm complicated situation17:59
gatoxjeje17:59
elopioralsina: I agree, but more than 500 bugs open is not manageable. If we are not going to fix some of those bugs, please mark them as "Won't fix".17:59
ralsinaelopio: cool18:00
ralsinaelopio: most of them are "that thing doesn't exist anymore" :-)18:00
ralsinamandel: check last comment on https://bugs.launchpad.net/bugs/88392618:00
ubot4Launchpad bug 883926 in ubuntuone-client "[windows] Breaks on system directories My Music, My Videos dirs in ~\Documents folder on Win7 (affects: 7) (dups: 3) (heat: 46)" [Critical,Fix released]18:00
elopioralsina: yes, invalid sounds even better :D18:00
mandelralsina, elopio if does are u1 windows related, send them to me :)18:00
ralsinamandel: I think I already closed all the first-beta bugs18:01
mandelralsina, which revno is the release he is talking about?18:01
elopiomandel, ralsina, I'm not sure where to start. Today I'll focus on lucid.18:01
gatoxbrb....... need to buy citric :P18:01
elopioWindows is on wednesday, because if I start using windows on mondays, by thursday I'll quit.18:02
ralsinaelopio: evything you find pre-lucid, mark as invalid18:02
mandelralsina, 2.0.3 release... I have no idea which code was sent there18:02
mandelelopio, reasonable ;)18:02
ralsinamandel: I can tell you the revnos in a minute18:02
mandelralsina, please18:02
ralsinamandel: well, I have to install it first ;-)18:03
ralsinamandel: it's revno 1169 of ubuntuone-client18:10
mandelralsina, it should be fix... AFAIK18:11
mandelralsina, we need to get the logs, I'll take care of it18:11
ralsinamandel: what I thought18:11
ralsinamandel: could be he is still running 2.0.2 after upgrading becuase it doesn't restart itself18:11
mandelralsina, sounds possible18:12
mandelralsina, but It will be very very strange18:14
ralsinamandel: not really. Aslong as the user doesn't logout, it will keep running the old version18:15
ralsinamandel: and I log out about once a week.18:15
gatoxnessita, branch updated: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704918:22
nessitaack18:22
elopiohey ralsina, should we have a separate master bug for CPU usage?18:24
ralsinaelopio: could be.18:25
ralsinaelopio: yes18:25
dobeywhee18:25
elopioralsina: I'll open one and try to send as duplicate all the bugs I find.18:25
dobeydirspec MIR filed.18:25
ralsinaelopio: cool18:25
nessitadobey: wanna do a 2x2 and file the qt4reactor MIR? ;-)18:26
dobeynessita: i thought you were going to do that one18:26
nessitadobey: yeap, I was, but tons of other sutff came up18:26
dobeyi really want to get the gi reactor stuff finished today18:27
mandelralsina, nessita, dobey, gatoxok, EOD for me, I'll be around later on irc doing other things yet feel free to ping me18:27
nessitadobey: sure, it makes sense. Is there any chance you queue that up for after that?18:27
dobeywhich i can hopefully finish in within the next hour or so. and then start working on the libsyncdaemon/libu1/musicstore mess18:28
dobeynessita: maybe. qt4reactor scares me :)18:28
dobeybecause i basically know nothing about it18:28
nessitaheh, why? :-)18:28
dobeyi've never talked to upstream. i don't know if there is a release schedule or what. or what the maintenance requirements of it might or might not be.18:29
ralsinamandel: did you approve my branch?18:29
nessitadobey: hum, none of us know that18:30
dobeyso it's not simple, like dirspec is18:30
dobeynessita: right, and that's a problem for an MIR :)18:30
nessitayes18:30
mandelralsina, I think I did, let me check18:30
nessitaralsina: how can we move forward with the qtreactor MIR given that? ^18:30
dobeyalso, i really don't like qt :)18:30
ralsinanessita: well, we can contact upstream18:31
nessitadobey: that's orthogonal18:31
dobeyyes18:31
dobeybut i had to say it :)18:31
dobeyit's in a contract18:31
ralsinadobey: felt good? ;-)18:31
mandelralsina, ha, I wrote the comment but I did not hit save18:31
mandelralsina, should be approved now18:31
ralsinamandel: awesome! thanks!18:31
ralsinanessita: we have the big branch for windows packaging with +1 from diego and manuel.Approve or you want to take a look? It's really not fun to look at ;-)18:32
nessitaralsina: go ahead. Did you got the MIR question?18:33
mandeldobey, so... you are not pushing the qt reactor base on Pyside upstream... common you are the only one that has pushed a reactor already!18:33
dobeyhttp://www.youtube.com/watch?feature=player_detailpage&v=5A0-u85aAYg#t=157s18:33
ralsinanessita: got it and answered it18:33
dobeymandel: huh?18:33
ralsinamandel: we are not even using PYside18:33
nessitaralsina: ah, sorry, missed that. Upstream was contacted when we did the packaging, and they basically answered "I won't be maintaining this anymore"18:34
ralsinanessita: then I say we package it and support it as well as we can, since we have nothing else18:34
ralsinanessita: in any case, it's temporary because we are moving to a replacement IIRC what you and mandel mentioned a while ago, right?18:34
ralsinamandel: the branch still has a needs fixing from you18:35
dobeygtk+?18:35
nessitaralsina: no... no replacement for at4reactor as far as I know18:35
mandelis just to make dobey work on qt :)18:35
ralsinanessita: ok, then we have to live with it, and support it if noone else does18:35
nessitaralsina: so, we need to write a MIR for qt4reactor, and I'm not sure that with the sentence "we'll support it as well as we can" we're gonna get it :-/18:36
mandelralsina, here: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_809873/+merge/6791118:36
nessitaralsina: you familiar with what a MIR is? perhaps I'm assuming you're and that's a mistake from me18:36
dobeywe can make a gtk3 cp for linux ;)18:36
ralsinamain inclusion request?18:37
nessitayes18:37
ralsinawe can say we own it now and are the new upstream18:37
ralsinaans set it up on launchpad18:37
nessitaralsina: would you do that? (JIC, https://wiki.ubuntu.com/MainInclusionProcess)18:37
ralsinanessita: sure, but not today, I started at 6:30 AM :-P18:38
nessitaralsina: sure, I wasn't meaning today :-)18:38
ralsinanessita: ok, added to my todo list18:38
nessitaralsina: awesome :-)18:38
Surlent777hello; would someone be able to attempt to help me with an issue re: Tomboy and Ubuntu One?18:38
dobeynessita: btw, i worked around the precise breakage in nightlies, so you can install new u1client on precise now18:39
nessitanice18:39
Surlent777basically, I have a fresh Kubuntu 11.10 installation here, and Tomboy will download notes, but never upload, and as such local changes are overwritten. Yet, on my other computer running 11.10, it all works fine. What can I do to attempt to fix this?18:43
dobeyjoshuahoover, duanedesign, rye: ^^ any ideas for Surlent777's issue with tomboy?18:45
duanedesignhi Surlent77718:45
Surlent777hi; if it helps, I already made a debug log dump, as per a certain forum post, and I could throw it up on pastebin if that would be any help18:48
duanedesignSurlent777: I was just about to send you this link :)  http://pastebin.com/yiubL41U18:49
duanedesignmight be what you already did18:49
Surlent777duanedesign: yep; http://pastebin.com/JA2eacEV, set to expire in an hour, as I don't want to be one of Those Guys that leave useless crap on there for eternity18:53
dobeyah the "New Note Template" issue18:54
duanedesignYou will need to delete the New Note Template18:55
Surlent777okay; it's not listed in the main collection of notes. How would I do that?18:56
duanedesignok18:57
duanedesigni think what you do is create the note18:58
Surlent777huh, just noticed that not all of my notes seem to have downloaded, judging from what I have on one.ubuntu.com18:58
Surlent777anyway, so make a note entitled "New Note Template", I assume, then what? try to sync, delete, sync again?18:59
duanedesignbug #84825019:00
ubot4Launchpad bug 848250 in tomboy (Ubuntu) (and 1 other project) "Tomboy sync fails on Oneiric, New Note Template already exists (affects: 33) (dups: 1) (heat: 177)" [Undecided,Confirmed] https://launchpad.net/bugs/84825019:00
duanedesigncreate note "New Note Template" then sync and rename the local note19:01
duanedesignSurlent777: comment 10 in the above bug. That seems to do the trick for most users19:01
ralsinanessita: th only requirement forMIR that's shaky for qtreactor is " To support a package, we must be reasonably convinced that upstream supports and cares for the package". As a tech lead,do you think we can be the upstream? Because if we can't we need to rethink things ;-)19:03
nessitaralsina: exactly my point. I personally don't think we can be upstream of this, which is an issue, yes19:04
nessitaralsina: personally but as teach lead. I don't think we have the expertise to maintain a twisted reactor19:04
nessitaralsina: nor the time in the roadmap19:05
nessitaralsina: what do you think?19:05
ralsinanessita: ack19:05
ralsinanessita: well, I don't know how much maintenance this would need19:05
ralsinanessita: don't have the knowledge to have an informed opinion19:05
nessitaralsina: me neither, and I think we both don't know because we don't understand it that much :-)19:06
ralsinanessita: agreed19:06
Surlent777duanedesign: I used the "create the note on u1's site" variant, and that seems to have done the trick. Thank you very much; I had no idea that template had anything to do with it19:06
duanedesignSurlent777: great!19:06
ralsinagatox: does this log look like a unicode bug to you?19:08
ralsinagatox: https://bugs.launchpad.net/ubuntuone-client/+bug/883926/+attachment/2664809/+files/syncdaemon.log19:08
ubot4Launchpad bug 883926 in ubuntuone-client "[windows] Breaks on system directories My Music, My Videos dirs in ~\Documents folder on Win7 (affects: 7) (dups: 3) (heat: 50)" [Critical,Fix released]19:08
* gatox looking....19:08
gatoxralsina, yep..... it's looks like it19:10
ralsinagatox: could you reply to him and point him to the right bug?19:10
gatoxralsina, ok..... only because "unicode" is my middle name19:10
gatoxjejeje19:10
ralsinahehe19:11
ralsinaI see unicode, I think of you, Diego!19:11
gatoxjejjej19:11
alecumandel, nessita: I've pushed some fixes to the branch you guys were reviewing: https://code.launchpad.net/~alecu/ubuntu-sso-client/proxy-integration-tests/+merge/8753819:21
nessitaalecu: ack!19:24
nessitaralsina: so, I'm thinking in background about the qt4reactor MIR19:25
ralsinanessita: yes?19:25
nessitaralsina: and I remembered what you suggested about us talking about the possibility of not depending on it on linux19:25
nessitaralsina: and we *could* not depend on it once we use the webclient alecu and mandel are working on, but I'm not 100% sure, so19:26
nessitaralsina: is alecu attending to the sprint, at least a couple of days? I would love to analyze this in depth with him19:26
ralsinanessita: yes, he is at least 2 or 3 days19:26
ralsinanessita: we can move it to a full week if it's ok with him, too19:27
nessitaralsina: well, then I would put on hold the qt4reactor MIR until he and I analyze this and put a plan together to remove that dep in linux19:27
nessitaralsina: how does that sound?19:27
ralsinanessita: cool. Timing scares me a bit19:27
nessitaralsina: to me too19:27
nessitaalecu: what do you think?19:27
nessitaralsina: do you know brian IRC nickname?19:29
nessita@ping19:29
ubot4pong19:29
alecunessita, ralsina: I think it's feasible to get rid of the qt3.99reactor on linux, but we need to check around to make sure.19:29
* nessita is afraid to loose connection any time19:29
ralsinanessita: nope. You have his email though :-)19:29
nessitaralsina: yes, but I'd need better throughput than that, just for now that we're coordinating his plane ticket19:29
ralsinanessita: I meant, I don't, but we can ask him ;-)19:30
nessitaralsina: yeah, but seems like he's not onto his gmail email while at work19:30
nessita(how dare he? :-P)19:30
ralsinanessita: probably a slightly more fascist work environment than ours ;-)19:31
ralsinaOK, goota run. I may be slightly late tomorrow. Mail me if you need anything early.19:45
gatoxralsina, ack!19:51
dobeyyay, almost done19:56
dobeysigh :(20:08
gatoxEOD!!! see you tomorrow!!20:10
gatoxalecu, ahhhhhhh you are here..... maybe you would enjoy to review this: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704920:11
* gatox run...20:12
gatoxjejej20:12
alecugatox, run away! I'm reviewing a 7k-line branch by naty20:12
gatoxalecu, :S ok...... i'm not going to bother you.......20:13
alecugatox, I may review it tomorrow... sorry!20:13
gatoxalecu, no problem!! i'm in eod.....20:13
gatoxalecu, tell nessita that she is not thinking in your mental health :P20:14
nessitagatox: aux le contraire!20:14
gatox:P20:15
dobeygah, this sucks. twisted hates me.20:26
beunodobey, don't worry, it hates everybody20:28
dobeyyes, but *i* have to figure out what broke, and is causing it to infinite loop and eat all my ram.20:41
alecudobey, mandel: do you have any idea why this is happening???21:00
alecuhttps://bugs.launchpad.net/ubuntuone-dev-tools/+bug/91396821:00
ubot4Launchpad bug 913968 in ubuntuone-dev-tools "Could not locate suitable squid.conf.in (affects: 1) (heat: 6)" [High,New]21:00
dobeydoh21:02
dobeysetup.py21:02
dobeyalecu: https://code.launchpad.net/~dobey/ubuntuone-dev-tools/fix-913968/+merge/8798321:06
alecudobey, +121:15
alecunessita, if you are still around, can you please review the trivial branch above by dobey?21:16
=== JanC_ is now known as JanC
nessitaalecu: sure!21:16
dobeynot necessary21:17
dobeyi already set it to approved :)21:17
alecunessita, ^21:17
nessitaack!21:17
dobeyit might take a while for the nightlies to get built once it lands though. i saw earlier in #launchpad that apparently PPA builders are pretty backed up :(21:18
alecudobey, :-(21:19
dobeywell that's not as bad as my luck with twisted is right now, anyway21:20
=== yofel_ is now known as yofel
alecudobey, what's the problem now with it?21:22
dobeythe test suite is hitting an infinite loop and eating all my ram, it seems, with my latest changes to refactor gireactor/gtk2reactor into a single base class that both use instead21:25
alecudobey, do you want me to take a look at that branch?21:25
dobeyif you want. i'm not sure what i could have changed to cause it :(21:26
dobeylp:~dobey/twisted/gi-support21:26
dobeyalecu: "PYTHONPATH=. bin/trial -r gi twisted" will show the problem when it gets to the manhole tests21:29
=== gatox is now known as gatox_away
mandelalecu, I was going to review dobey  branch.. I suppose is not longer needed :P21:39
mandelalecu, if you don't mind, I'll finish your review tom21:40
* mandel reboots21:40
nessitaso, I'm of21:43
nessitaoff*21:43
nessitawill do fixed reviews tomorrow21:43
dobeymy brain hurts and i am upset that this isn't working :-/21:51
alecudobey, I've started commenting all the testcases that hang up, and most of them have "Stdio" in their names21:54
alecuwell, not everyone:21:54
alecuSystemEventTestsBuilder_GIReactor.test_callWhenRunningOrder21:55
dobeytestClassDefinition doesn't have "Stdio" in it :)21:57
dobeywell, so the tests all worked fine in the previous revision :(21:57
alecuright, I see that they are working fine in the previous revision after a "merge from trunk"21:59
dobeywell they were working fine before the merge from trunk as well22:00
alecuwell, it's a non-trivial branch by far, so I'm not understanding it at this time.22:07
alecudobey, I'll take another look tomorrow.22:07
dobeyalecu: ok, thanks22:09
dobeyalecu: it's not super trivial, but it's not terribly complicated either. it is large though, because of having to keep the gi/gobject bits separate22:10
dobeyif there was some way to simplify the diff view, it would probably look a lot more trivial than it does now :)22:11
=== m_conley is now known as m_conley_away
aquarius_at_confralsina, ping22:29
aquarius_at_confrye, ping22:29
aquarius_at_confduanedesign: ping22:30
duanedesignhello aquarius_at_conf22:30
qwebirc716586duanedesign: hey22:32
=== qwebirc716586 is now known as aquariusatconf
dobeyheh22:32
aquariusatconf:)22:32
dobeyhi aquariusatconf22:32
duanedesigno/22:33
dobeyaquariusatconf: what do you need?22:33
dobeyyou're obviously not at pub, so you're either sick, or you need something :)22:34
dobeyi bet he timed out22:36
aquarius_atconfgrrr22:36
dobeyindeed22:36
aquarius_atconfduanedesign: having a problem with u1 for Windows22:36
aquarius_atconfduanedesign: File Sync error: local and server roots are different (ROOT_MISMATCH)22:36
aquarius_atconfwhat's that? how do I fix it?22:37
duanedesignhmm, ok22:37
dobeyaquarius_atconf: someone logged in with one account, and then a different one22:37
duanedesignaquarius_atconf: two accounts have been used on the same machine22:37
duanedesignaquarius_atconf: need to delete the metadata22:37
duanedesignwhich version of windows?22:37
aquarius_atconfokokok22:37
aquarius_atconfk22:37
aquarius_atconfheh, sorry, browser probs too :)22:38
aquarius_atconfok, so...22:38
aquarius_atconfhow do I fix it?22:38
aquarius_atconfam happy to wipe the account details and sign in again22:38
aquarius_atconfbut I don't knoiw how22:38
dobeydelete the metadata. or just ignore it, and make a new user on the system to use instead :)22:38
dobey(if that's feasible to do anyway)22:38
dobeywindows 7?22:39
dobeyi guess yes22:39
duanedesignaquarius_atconf: https://pastebin.canonical.com/57909/22:40
duanedesignaquarius_atconf: that should do it^22:41
* dobey bets he got dropped again22:42
duanedesigno.O22:43
duanedesigndobey sees the future :)22:43
dobeyheh22:43
dobeyindeed22:43
dobeyand i am off22:43
dobeylater :)22:43
duanedesigno/22:43
aq_at_conf_2foir crying out l.oud22:44
aq_at_conf_2grr22:44
duanedesignhttps://pastebin.canonical.com/57909/22:44
aq_at_conf_2dobey, duanedesign, sorry, if you said something I missed it22:44
duanedesignthat was the link i posted if you did not get it :)22:44
aq_at_conf_2cheers22:44
aq_at_conf_2can you paste it to paste.ubutnu.com22:44
aq_at_conf_2am not on my own laptop22:44
duanedesignyep22:45
duanedesignaq_at_conf_2: http://paste.ubuntu.com/798746/22:45
aq_at_conf_2duanedesign: ok doing that cheers22:51
=== gatox_away is now known as gatox
=== gatox is now known as gatox_swim

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