/srv/irclogs.ubuntu.com/2011/12/19/#ubuntuone.txt

=== M0hi is now known as IAmNotThatGuy
=== yofel_ is now known as yofel
ryemorning08:33
=== pedronis__ is now known as pedronis
ralsinagood morning!12:18
gatoxralsina, good morning!12:19
ralsinahello gatox12:19
gatoxralsina, how are you? how do you spend your weekend? relaxing?12:20
ralsinaI slept!12:20
ralsinaActually deactivated the alarm!12:21
gatoxjeje good!12:22
gatoxneed to restart13:07
gatoxback13:15
gatoxbrb13:44
nessitabuenos días!14:21
gatoxnessita, buenas14:22
nessitadowntown is crazy today, I had to run some errands and that took more than expected14:22
dobeybuenas nessita14:23
nessitahola dobey14:23
dobeynessita: should we bug ralsina? or after standup?14:41
nessitadobey: after standup? so we can all organize14:44
nessitaralsina: can we have a talk with dobey about stable/trunk relationship after standup?14:44
ralsinanessita: sure14:45
dobeybig TODO list today14:48
dobeynessita: do we need an MIR for qt4reactor btw?14:50
nessitadobey: nopes, wanna do it? :-)14:51
dobeynessita: i think you misread the question? i asked if we need one, not if we have one :)14:51
nessitadobey: ah yes, I misread. Yes, I think we need one, since we want our qt packages on the main repo along with the rest14:52
dobeynessita: but control panel doesn't use qt4reactor does it, except for tests?14:53
nessitadobey: nopes, the qt controlpanel uses it to run. And so will use it the qt sso dialog.14:53
dobeyi thought it used the plain twisted reactor14:54
dobeyin the backend, and the gui doesn't use a reactor14:54
nessitadobey: the qt version does not access the backend thru dbus, it just imports it and use it. And to "mix" a twisted reactor and qt, we use at4reactor14:55
nessitaqt*14:55
dobeyoh, ugh14:55
dobeyso the deps are all wrong14:55
nessitadobey: FYI, I just confirm the qt controlpanel works (but yes, I have all the needed deps already installed)14:57
nessitame15:00
gatoxme15:00
dobeynessita: everything works right and it's not broken at all?15:01
dobeyme15:01
nessitadobey: in my env, yes15:01
nessitadobey: but again, I have all the needed deps because I develop here15:01
nessitaralsina: standup?15:01
nessitaDONE: started fixing all the test suites for bug #83473015:02
nessitaTODO: try to finish the aforementioned bug, talk with dobey and ralsina re: stable-vs-trunk, teach leads call15:02
nessitaBLOCKED: nopes15:02
nessitaNEXT: gatox15:02
ubot4Launchpad bug 834730 in ubuntuone-client (and 1 other project) "Use SignalBroadcaster and RemoteMeta from ubuntu sso client (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/83473015:02
gatoxDONE:15:02
gatoxFigure it out how to solve the windows shortcut problem, modify make_link in u1-client.15:02
gatoxTODO:15:02
gatoxFix 4 broken tests, add some tests to check the link creation.15:02
gatoxBLOCKED:15:02
gatoxNo15:02
gatoxdobey, go15:02
dobeyλ DONE: hack day, dirspec initial code for xdg base dirs spec15:02
dobeyλ TODO: discuss stable v trunk, call congress, expenses, dirspec nightlies, make devtools use dirspec, some initial music store fixups for P, backport some stuff to stable-3-0 branches, release prep, fixes in gireactor for twisted review15:02
dobeyλ BLCK: No.15:02
dobeyralsina15:02
dobeynessita: with or without -gtk installed?15:03
ralsinaDONE: vacation, windows release. TODO: arrange hiring interviews, tech leads call, XML file for autoupdate,talk with nessita&dobey BLOCKED: nopes15:03
nessitadobey: with15:03
dobeynessita: can you try without please?15:03
nessitadobey: sure15:03
nessitaany comments anyone?15:03
gatoxnop15:03
ralsinano comments15:03
nessitaok, eom!15:04
nessitaralsina, dobey, shall we mumble? or do you guys prefer something else?15:04
* gatox lunch15:04
=== gatox is now known as gatox_lunch
dobeyi like irc; but whatever is fine15:04
ralsinaeither works for me15:05
dobeynessita: pick15:06
nessita1oops, net issues15:09
=== nessita1 is now known as nessita
dobeynessita: you pick :)15:09
nessitadobey: mumble, so we expedite the thing :-)15:09
dobeyok15:10
dobeyyou better not have net issues ;)15:10
=== gatox_lunch is now known as gatox
dobeynessita, ralsina: can you review https://code.launchpad.net/~dobey/dirspec/add-tests/+merge/86130 please? :)15:56
nessitadobey: sure!15:57
ralsinadobey: on it!16:00
dobeyalright, need to get lunch; bbiab16:38
gatoxbrb17:27
nessitalunchtime!17:39
nessitadobey: is dirspec meant to be used in windows as well?18:05
dobeynessita: yes18:07
dobeynessita: and it should work in windows, as-is18:08
nessitadobey: tests are not running for me...18:09
nessitaC:\Python27\\python.exe: can't open file 'C:\Python27\\Scripts\trial': [Errno 2]18:09
nessita No such file or directory18:09
nessitadobey: I do have a trial.py in Scripts18:09
dobeyah18:09
dobeynessita: ok, i didn't know that it wasn't just "trial" on windows18:10
dobeyi'll fix that18:10
nessitadobey: now, if I change the run-tests.bat to use trial.py, no failure but no test is ran18:10
nessitadobey: http://pastebin.ubuntu.com/775539/18:11
dobeyhuh18:11
dobeynessita: what version of twisted is this btw?18:12
nessitadobey: let me confirm18:12
nessitadobey: 11.0.018:12
dobeyweird18:13
dobeynessita: how do you set the python include path on windows? (ie PYTHONPATH=. on linux)18:15
nessitadobey: set PYTHONPATH-.18:15
nessitasorry18:15
nessitaset PYTHONPATH=.18:15
nessitadobey: which I did not run on this branch :-) good catch!18:15
dobeynessita: well %PYTHONPATH% in run-tests.bat is that path to where python is installed, afaict, so i guess we can't just set it to that?18:16
nessitadobey: right, I just set it by hand and test ran18:16
nessita(and we set it by hand to every project)18:16
dobeyso we can't set it inside run-tests.bat?18:16
nessitadobey: well, we usually set the env with other name, PYTHONPATH is not the happiest name18:17
nessitait should be PYTHON_EXEC_PATH18:17
nessitaor something like that18:17
dobeynessita: you mean what is currently in run-tests.bat should be that instead?18:17
nessitadobey: yes, see for example the run-tests.bar of ussoc18:18
dobeyit's weird that trial behaves different on windows than on linux though :-/18:19
=== nessita1 is now known as nessita
dobeynessita: so change that to PYTHONEXECPATH and add SET PYTHONPATH=. in run-tests.bat should work?18:22
nessitadobey: yeap, plus using trial.py18:22
dobeyright18:22
nessitadobey: you use trial and not u1trial to not depend on udevtoools, right?18:23
dobeyright, to avoid the circular dep, since devtools will need to depend on dirspec18:24
dobeyalso why i'm running pyflakes instead of u1lint18:25
dobeyit /is/ just "pyflakes" on windows right? not pyflakes.py or .exe or something?18:25
ralsinadobey: yes, just pyflakes18:26
dobeyyay18:27
dobeynessita: pushed the fix; can you update and try run-tests.bat without adding to the env yourself? :)18:27
nessitadobey: of curse18:29
nessitadobey: but give a couple of minutes18:29
dobeynessita: sure; thanks for testing it on windows :)18:29
nessita:-)18:30
nessitadobey: this is what I'm getting now http://pastebin.ubuntu.com/775566/18:40
nessitaand I do have a python installed, under C:\Python2718:41
dobeynessita: humm; did i screw up the s/PYTHONPATH/PYTHONEXECPATH/g bit somehow?18:41
dobeyah, yes i missed a bit18:42
nessitadobey: the wow thingy?18:43
dobeynessita: fixed/pushed18:43
dobeynessita: the bit that checks the registry key was setting PYTHONPATH still18:44
dobeynessita: fixed it to be PYTHONEXECPATH also18:44
nessitaah18:44
dobeyso should work now :)18:45
nessitatesting18:53
nessitadobey: unseting the PYTHONPATH on the terminal before running the suite ends up in no test being run18:56
dobeynessita: :(18:57
dobeynessita: run-tests.bat has SET PYTHONPATH=.; so it should work afaik18:57
dobeydoes it have to be quoted?18:58
nessitadobey: honestly, no idea :-/18:58
nessitadobey: weren't you a windows dev? :-P18:58
dobeyno :)18:58
dobeynessita: well you have to set it before doing run-tests.bat for all the other projects right?18:59
nessitadobey: yeap18:59
dobeynessita: so i say let's figure it out later :)18:59
nessitadobey: +118:59
dobeyneed to get it packaged up and in ubuntu with the releases tomorrow19:00
dobeynessita: all the tests pass without skipping any, on windows, right?19:01
nessitadobey: yesir19:01
dobeythen i am very happy with the current state of dirspec, with that branch :)19:02
ralsinaNo, doesn't need to be quoted19:02
nessitadobey: nice!19:02
nessitadobey: question, how shall I use bzr to merge trunk into stable but without a specific commit?19:02
ralsinamostly, on windows, you have to set PYTHONPATH maually for all the other projects too19:03
dobeyonly one of the tests actually needs windows; and we just skip it on ubuntu :)19:03
dobeynessita: you can specify a range of commits with -r19:03
nessitadobey: and so I need to do 2 merges, right?19:04
dobeynessita: for?19:04
dobeyyou can do -r 1..3,5 to skip 4, i think19:04
nessitadobey: ah!19:04
dobeybzr revision specs can be quite complicated19:05
dobeybzr help revspec iirc19:05
dobeyor not19:06
dobeyoh, but 1..3 doesn't include 119:06
dobeyoh, but it might screw up the history19:07
nessitadobey: so? :-D19:09
nessitadobey: I'm lost what I should run19:09
dobeynessita: finish the review of the dirspec branch, and i'll help you with the specifics19:10
nessitadobey: you're not choosing your battles wisely :-P19:11
dobeyi just set up nightlies recipe for dirspec and it would be nice to have it building in the background :)19:12
dobeyblah i don't even know who my congress people are :-/19:13
nessitadobey: review finished19:19
dobeybah :(19:21
dobeyoh well, so what are you trying to merge from trunk to stable exactly?19:22
nessitadobey: to start with the stable-3-0 process, I'm updating ussoc. My idea is to have everything except the revno 82119:23
nessitadobey: which brokes sso linux19:23
nessitaI checked the rest19:23
dobeyso nightlies are broken on linux?19:26
nessitadobey: yes for a specific (and not common) use case19:26
nessitadobey: I'm working on a fix, will not be ready before tomorrow19:26
nessitadobey:  bzr merge -r 813..820,821..824 lp:ubuntu-sso-client definitely does not work19:30
nessita(nor  bzr merge -r 813..820,822..824 lp:ubuntu-sso-client)19:30
* nessita will try 2-step update19:31
dobeyfor some reason it is merging r821 anyway19:32
dobeyyou don't want r814 included19:33
dobeythere's already a separate commit for that in stable-3-019:33
nessitadobey: you're right19:33
dobeyalso, for some reason it seems to be pulling in the different setup.py :(19:33
nessitadobey: is not doing that for me...19:33
nessitadobey: what difference do you see?19:33
dobeybzr merge -r 814..820,822..824 shows a conflict in setup.py, and bzr st -v shows r 821 included19:34
nessitadobey: the conflict is easy, is the version number (as you predicted)19:34
dobeyhrmm19:35
dobeyi don't think bzr likes M,N19:35
nessitadobey: I'm doing two merges, and presto19:36
dobeyif you do 814..82019:36
dobeycommit, then do 821..824, i think it works ok19:36
nessitadobey: indeed19:36
dobeyand doesn't pull in the setup.py change either19:36
dobeymake sure 821 doesn't show up in the second merge though, by checkint bzr st -v19:37
dobeysince it likes to collapses the messages it seems19:38
dobeyhrmm, actually it seems like it may not even keep the history in that case19:38
dobeyblah19:39
dobeyoh well19:39
nessitadobey: so, -r 821..824 does bring 821 :-/, so I had to do 822..82419:41
nessitaI checked the diff against trunk and is correct19:41
dobeyhuh19:41
nessitathe only missing bits are the 821 revno19:41
dobey821..824 shouldn't. i guess the docs are wrong in bzr :(19:41
nessitadobey: right (I read the same as you)19:42
gatoxnessita, ping19:47
nessitagatox: pong19:47
gatoxnessita, can you do me a favor?19:47
dobeynessita: is __setitem__ better to use than just set?19:47
gatoxnessita, can you open volume_manager in u1/syncdaemon/volume_manager19:48
dobeynessita: or using a lambda instead perhaps?19:48
gatoxnessita, line 1495...... _guess_metadata_version  and tell me if that IF has sense to you?19:48
nessitadobey: I personally prefer __setitem__, the cleaner solution is using operator19:48
nessitadobey: from operator import setiten19:49
nessitasetitem*19:49
nessitadobey: so, the addCleanup would be:19:49
nessitaself.addCleanup(setitem, os.environ, var_name, old_value)19:49
nessitagatox: looking19:49
nessitagatox: yes, it does make sense. Some migration happens from user not having that dir19:51
nessitagatox: why?19:51
gatoxnessita, this is what i see:19:51
gatox        if path_exists(self._shares_md_dir) \19:51
gatox           and path_exists(self._shared_md_dir):19:51
gatoxahhhhhhhhhhhhhhhhh19:51
nessitagatox: me too19:51
gatox:P19:51
gatoxshares..... shared19:51
gatoxnot so good at this hour19:51
nessitagatox: ah, yes, that too (yes is confusing)19:51
gatoxsorry19:52
nessitais ok :-)19:52
nessitaralsina, dobey: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/stable-3-0-update/+merge/8628319:53
ralsinanessita: I won't finish that review today...19:54
nessitaralsina: is there any chance you do it later today or tom morning?19:55
ralsinatomorrow morning sure19:55
nessitaralsina: so we can release tomorrow. It does not need to be a code review in detail, since all that code has been reviewed already19:55
ralsinaI can try to do a IRL test at least19:55
nessitaralsina: yes please, on windows ideally (I made it on Ubuntu)19:55
ralsinanessita: ack, I'll have it done before beginning of workday tomorrow19:56
dobeynessita: canonical e-mail addresses19:57
dobeyi will yell at mandel tomorrow :)19:57
nessitadobey: ok, I'll change it in the MP19:57
dobeyi wish bzr handled history properly19:58
dobeymeh, i also have to review mandel's devtools branch still :(19:59
dobeyit makes me sad19:59
nessitadobey: cheer up, you learnt about operator today :-) (when I did I was thrilled!)20:00
ralsinanessita: can you give me a really really trivial review here? https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/quote_autostart/+merge/8495220:00
nessitaralsina: looking20:00
ralsinanessita: just in case, that did get a IRL test from QA as part of the latest release20:01
nessitaralsina: well, that branch is not exactly trunk20:01
nessitaralsina: I can not merge a branch from gatox that breaks linux when user is not validated20:01
ralsinanessita: I mean my little branch ;-)20:01
nessitaralsina: ah!20:01
gatoxnessita, sorry...... which branch?20:02
nessitagatox: the change you made to user_validated_cb (I already mentioned this, so nothing new)20:03
gatoxnessita, ah yes......20:03
nessitagatox: but we can't release that to Ubuntu20:03
gatoxok....... eod for me! see you tomorrow20:10
nessitaralsina: approved20:11
ralsinanessita: cool, thx20:11
ralsinanessita: I am leaving the stable branch syncing, will eod and give the review late tonight20:11
dobeynessita: refactored the environment modifying tests20:13
nessitadobey: ack.20:29
nessitadobey: tweak_env looks great, and replied to your comment20:46
nessitaralsina, dobey: https://code.launchpad.net/~nataliabidart/ubuntuone-client/stable-3-0-update/+merge/86289 (test pass on all OS, IRL tested on Ubuntu)20:50
nessitaralsina, dobey: MP resubmitted, I did not choose stable-3-0 as target https://code.launchpad.net/~nataliabidart/ubuntuone-client/stable-3-0-update/+merge/8629420:53
dobeyah ok21:06
dobeyoh, other project21:07
nessitadobey: hum?21:07
dobeyi didn't realize you did one for u1client also21:07
nessitadobey: yes, and I'm about to have the controlpanel one21:09
dobeybah it's already 16:10 :(21:10
nessitadobey: still 2 more hours to go! :-)21:12
dobeywhat?21:13
nessitadobey: right, from 9am to 6pm21:17
nessitano?21:17
dobey9+8 = 1721:18
nessitadobey: as far as I was told, we work from 9am to 6 pm having one hour~ for lunch21:24
dobeygah, stupid congressional contact web forms; why do they all require a stupid prefix21:26
dobeyi think i will put "GOD" as my prefix21:26
dobeyhrmm, i guess i am not getting my truck back today either21:27
dobeymeh21:27
nessitadobey: one more... when you can: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/stable-3-0-update/+merge/8629621:29
nessitadobey: last one, at least for today: https://code.launchpad.net/~nataliabidart/ubuntuone-windows-installer/stable-3-0-update/+merge/8629721:35
dobeya bunch of the windows fixes also need backported to stable-2-0 don't they?21:36
dobey(in all the projects that is)21:36
nessitadobey: yeap -- but since on Feb we'll try to have stable-3-0 on stable-2-0, that will do it21:41
nessitabye all! be back later21:49
dobey21:54
mandeldobey, still here?22:15
dobeyhi mandel22:16
dobeymandel: how long is the train from madrid to malaga? :)22:18
mandeldobey, hm.. depends on which train you take, if the speed train goes there, 2 hours else 4 or something like that, why?22:19
dobeymandel: was pondering this: http://gstreamer.freedesktop.org/wiki/GStreamerHackfest201222:19
mandeldobey, if that was the reason why you wanted to yell at me, I'm happy :)22:19
mandeldobey, hm.. that is super tempting!22:20
dobeyno; i wanted to yell at you because apparently you used your personal e-mail in some commits :)22:20
mandeldobey, oh, bullocks!!! where?22:20
dobeyhttp://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntu-sso-client/trunk/revision/816.2.1322:21
dobeyhttp://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntu-sso-client/trunk/revision/816.1.122:22
mandeldobey, hm... I know that was the email I used to use when I was out from canonical..22:22
dobeyyes22:22
mandeldobey, maybe the machine I used is badly configured... is there a way I can help you with thaT?22:23
mandelups, capital T :P22:23
dobeymandel: and your devtools branch22:23
dobeymandel: i don't think it's easy to go through and edit all the commits; but you can fix your config so you use your canonical e-mail on the canonical projects :)22:23
mandeldobey, certainly!!! sorry for that, I guess I used the desktop to edit those.. sorry :(22:24
mandeldobey, did anything interesting happened today?22:24
dobeymandel: for instance, i have this in my ~/.bzr/locations.conf : https://pastebin.canonical.com/57413/22:25
dobeymandel: and i manage all the canonical projects under ~/Projects/canonical/ :)22:25
dobeymakes things easy22:25
dobeynothing particularly interesting, no22:26
mandeldobey, I have the same, a ~/Projects/Canonical, I guess that I screwed up on of the machines at some point and did not notice..22:26
dobeyi totally failed to get all the things done today, that i wanted to do, and really need to get done for tomorrow, though :(22:26
mandeldobey, ok, I'll be working tom so if you need any help let me know and I do my best (for example, fixing my desktop bzr config :P )22:27
mandeldobey, and regarding malaga, if you go, I'll join you :)22:28
mandeldobey, and you are more than welcome to spend some day here in madrid :)22:28
dobeyi probably won't go22:29
dobeyam a bit tired of travel, at the moment :)22:29
mandeldobey, ha, tell me about it!22:29
dobeyugh, and i'm pretty tired in general right now22:30
mandeldobey, well, I'm going to bed, I just opened this to check if anything bad/good happened, I'll see you tom here (its 11:30 pm CET :P)22:31
dobeybuenos noches22:33
mandeldobey, good night to you too :)22:35
* mandel watches tv22:35
dobeyneed to get some food23:05

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