mandelmorning all!08:49
faganMorning sorry about being a bit late had to make like 8 phone calls to locate my package09:09
faganbut found it09:09
faganThe post system in ireland is so weird they didnt recognise my address even though they delivered to the same place the week before so they sent it to my mother's house09:10
faganso weird09:10
mandelfagan: don't worry I think I'm the only one who notices at this time :P09:18
mandelfagan: fancy to see a good code review?09:19
faganmandel: sure09:25
* fagan was grabbing tea09:25
mandelfagan: https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/68530 nessita does really good ones :)09:25
faganmandel: well I dont think I was asked to do anything more than ./run_tests and IRL tests too so never needed to go into that much detail09:26
faganholy shit though she doesnt like your code :D09:26
mandelfagan: nah, dont worry, this is just for you to learn, part of the internship ;)09:26
mandelfagan: is more the dialog, since I do have my comments as well hehehe09:26
faganmandel: haha well being nessita would be pretty hard to be in 6 months09:27
mandelfagan: will certainly be impossible09:28
faganmandel: im sure there would be some mind altering drugs or something that would help09:28
mandelnah, to dangerous09:30
mandelfagan: will be out some mins, I need to go to the internet cafe to be able to do some updates, should be back online in 15/20 min09:54
faganmandel: cool09:54
faganIll hold the fort09:54
* fagan takes 10 minutes to write up some notes to talk about with matthew later09:56
=== teknico is now known as teknico_away
* fagan takes a break 11:25
nessitahello everyone!11:26
fagannessita: hey, if you are looking for mandel he is at a cyber cafe doing updates and cant get onto freenode11:27
nessitafagan: ack, thanks for the info11:27
fagannessita: he had to reboot but should be back soon on the private irc11:27
ralsinagood morning!11:55
faganmorning ralsina11:55
faganralsina: im looking down the bug list to have a look at the triage policies if thats cool11:56
faganwanted to see how the big bugs get attention11:57
faganpolicy is a bad word I mean more what logs are needed for specific things and all that11:59
faganprocedure is a better one11:59
ralsinafagan: good idea12:00
nessitahi ralsina! you feeling better?12:00
ralsinanessita: MUCH!12:00
ralsinanessita: even slept 8 hours and everything12:00
faganwelcome back mandel12:12
mandelfagan: yes :)12:12
mandelthere was something failing in the handsake.. I hate the internet :P12:13
faganmandel: lets make a replacement :D12:13
mandelfagan: I do have a theory that proves that spaniards had wireless 1000 years ago :)12:14
faganmandel: smoke signals?12:14
mandelfagan: yelling!12:15
faganim good at that :D12:15
=== teknico_away is now known as teknico
james_whi, I've just set up a new oneiric  machine, but don't have an ~/Ubuntu One folder on it12:17
james_wother folders are synced fine, and the control panel says that it is synced12:18
james_wI've even seen notifications of files being downloaded12:18
faganjames_w: thats really weird12:18
james_wdouble clicking the folder in the control panel does nothing, but works for other folders12:18
james_wany clues?12:18
fagannessita: ^12:19
faganI dont have a clue I didnt even thing the verson in oneiric was actually working properly yet12:19
faganjames_w: on the website is the folder there?12:21
james_whmm, seems not12:22
faganjames_w: thats even more weird12:22
fagancheck your bin12:22
james_wbut it's on my natty machine12:23
faganjames_w: hmmm and there is stuff in it?12:23
* fagan realised that doesnt really matter 12:23
james_wthere are two files in the trash on the oneiric machine, both of which would be somewhere under my ~/Ubuntu One12:23
faganjames_w: aha12:24
james_wit's still on my natty machine, but some files may be missing, and there's a u1conflict dir12:25
james_wwhich would be the dir containing the files in the trash on the oneiric machine12:25
james_wand there's a whole bunch of the files in the trash on the natty machine12:26
faganjames_w: well it sounds like some sort of sync issue but id need rye or someone to help debug it12:26
james_wit seems like a rather unfortunate failure mode12:26
faganyeah its definitely something strange12:26
ryefagan, reading12:27
ryejames_w, no Ubuntu One folder in ~home ???12:29
mandelnessita: morning, answering on the MP the questions and commenting on the tests12:31
nessitamandel: I answered, did you see that?12:36
mandelnessita: yes, ill get an answer for that in some min :)12:40
ralsinamandel: could we mumble in 5'? Short!12:40
nessitamandel: in the mean time, I'll make a branch for the rename of the remove_windows_illegal_chars only12:40
nessitamandel: so your branch gets smaller12:40
mandelralsina: can't, can it way 'til after the tandup, I'm having lunch :)12:41
ralsinamandel: sure12:42
mandelnessita: very well, pass me the merge proposal if you will :)12:42
ryejames_w, well, i am pretty sure ubuntuone cannot work w/o a root folder12:44
ryejames_w, ummm... could you please check whether ~/.cache/ubuntuone/log/syncdaemon-exceptions.log exists and whether there is something interesting there?12:45
james_wFailure: twisted.internet.error.ConnectionDone: Connection was closed cleanly.12:46
james_wTypeError: unsupported operand type(s) for +: 'int' and 'NoneType'12:47
faganIm going to give my standup now since ill be on skype during12:48
fagan* Read the triage process for high and critical bugs12:48
fagan* More reading of triage stuff (unless someone has some task)12:48
fagan* Nope12:48
faganNOTE: On skype at the moment talking about bug assigning will be a few mins12:48
fagannext probably nessita12:48
james_wrye, there's lots of that second exception. I also see it creating the root dir in the start of syncdaemon.log12:49
ryejames_w, could you please pastebin it?12:50
james_wrye, syncdaemon.log?12:50
james_wthe first one, or the current one?12:51
james_w(it's been rotated three times)12:51
ryejamalta, syncdaemon exceptions, with int + NoneType12:51
james_w :q12:53
ralsinafagan dobey alecu nessita DiegoSarmentero mandel:standup in 4'12:56
faganralsina: already did mine12:56
faganand I though DiegoSarmentero's contract is up12:57
ralsinafagan: why did you do it 12 minutes early?12:57
faganralsina: going on skype in a minute for lp usability testing12:58
ralsinaAnd yes, his contract is up. He's still doing some stuff though, and he's welcome to join :-)12:58
ralsinafagan: ok12:58
mandelralsina: ping, and ingnore the other channel one :)12:58
ralsinaalecu is not going to make it but I have his notes12:58
ralsinamandel: pong12:58
mandelralsina: so, we do mumble after stan up?12:58
ralsinayes please12:58
mandelralsina: or can we multitask, I don't think I can :P12:59
ralsiname neither12:59
mandelralsina: so after standup then :)12:59
alecuralsina, I made it!12:59
ralsinaalecu: welcome and say me then :-)12:59
nessitaalecu: heeeeeey!12:59
alecuralsina, the connection on the ferry is better than I thought12:59
nessitamandel: https://code.launchpad.net/~nataliabidart/ubuntuone-client/rename-method/+merge/68676 (trivial)13:00
nessitadobey: ?13:00
nessitaDONE: bug #813099, bug #813157, reviews, meetings, QT learning13:00
nessitaTODO: ultimate details for bug #813099, make syncdaemon branches land13:00
nessitaBLOCKED: nopes13:00
nessitaNEXT: mandel13:00
ubot4Launchpad bug 813099 in ubuntuone-control-panel "QT UI: tweak account tab to match GTK's account tab (affects: 1) (heat: 47)" [High,Triaged] https://launchpad.net/bugs/81309913:00
ubot4Launchpad bug 813157 in ubuntuone-client "Windows: no need to customize expanduser (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/81315713:00
mandelDONE: More fix-os-helper changes, saw nessitas replay, started writing a reply for questions and comments about the tests. Added an smaller branch that adds asserts for the events returned from the fs so that the do use the correct data type in paths13:01
mandelTODO: Probably chat a little with nessita to see where we agree :), review her branch. Reporpose fix-os-hlper and others13:01
mandelBLOCKED: no13:01
mandelalecu, go go13:01
alecuDONE: almost ready branches for bug #803669, using the windows user id13:01
alecuTODO: finish branches for bug #80367213:01
alecuBLOCKED: need one more review for the unjelly branch:13:01
alecuNEXT: ralsina13:01
ubot4Launchpad bug 803669 in ubuntuone-client (and 1 other project) "Get the port numbers from the users' registry (affects: 1) (heat: 6)" [Low,Triaged] https://launchpad.net/bugs/80366913:01
ubot4Launchpad bug 803672 in ubuntuone-client (Ubuntu) (and 2 other projects) "Do not start two instances of this process (affects: 1) (heat: 6)" [Undecided,Invalid] https://launchpad.net/bugs/80367213:01
ralsinaDONE: calls. I was not feeling/coding well, went to the doctor, took the evening off because of that. TODO: reviews, installer fixes. BLOCKED: no13:01
ralsinadobey, please13:01
dobeyλ DONE: Registered ubuntuone-credentials and ubuntuone-client-gnome projects, started reorg splitting work13:02
dobeyλ TODO: reorg work, shim code, applly for motu, apply for ~ubuntu-desktop13:02
dobeyλ BLCK: None.13:02
ralsinaok, comments?13:02
mandelralsina: feeleing ok today?13:02
aleculots of lag13:02
ralsinamandel: feeling perfect13:02
mandelthat is what she said13:03
* mandel is getting very good at this :)13:03
ralsinathey *always* say that to me :-)13:03
faganmandel: not really13:03
alecuralsina, stop eating crap!13:03
alecumandel, ^13:03
ralsinaalecu: definition of crap varies.13:03
mandelalecu: tha tis all yours :)13:03
mandelralsina: launching mumble right now13:04
ralsinaok, so I assume there are no comments relative to the standup, thus EOM13:04
alecunessita, ralsina: we're arriving at the port soon; I'll find a bar with wifi and I'll try to get to the meeting. But it's very crowded today because of the winter holidays, so please start if I don't show up.13:08
nessitaalecu: ack13:09
mandelralsina: felt from mumble :(13:09
mandelralsina: but ok, I'll got to the admin page and will add it13:09
ralsinamandel: cool13:13
mandelnessita: can you please take a llok at my comment in the MP and let me know what you think13:16
nessitamandel: sure13:20
mandelalecu, ralsina: one more: https://code.launchpad.net/~nataliabidart/ubuntuone-client/rename-method/+merge/6867613:21
ralsinamandel: on it!13:21
nessitamandel: I read your comment, and my proposal would be to have the 2 tests I outlined there, plus 2 more with only the assertFalse(any())13:28
nessitamandel: since in a test, it doesn't make much sense to have an assertFalse and before exiting a builtin assert13:29
mandelnessita: why is thaT?13:29
nessitamandel: yes?13:29
mandelnessita: we can swap the around, right?13:29
nessitamandel: not really.... as far as I can see13:30
nessitamandel: wanna show me a code snnipet?13:30
mandelnessita: sure, give me a sec13:30
nessitaof what are you thinking on, I may be missing something13:30
ralsinaIMVHO: if the function replaces, then you should test replacement, not removal13:30
nessitaralsina: very good point13:31
nessitamandel: also the rename of the method should be to replace_foo13:31
mandelnessita: ok13:31
nessitaralsina: ^ so I guess I can change that in my branch about renames13:31
mandelnessita: yeah, we should.. I wonder if it is better to merge both methods in a single one, that way the logic of swtiching from one to other is encapsulated and we call it replace_illegal_chars(path, platform='windows')13:33
mandelfor example13:33
nessitamandel: no, 2 methods is always preferred and more pythonic13:33
mandelnessita: but then you have issues if one method changes and the other does not...13:34
ralsinayeah, replace_foo and 2 methods makes sense13:34
nessitamandel: what do you mean?13:34
nessitamandel: with proper tests, that should not happen13:34
mandelreally? I guess my brain is wired in a diff way13:34
nessitamandel: they are instrinsically different, let me explain:13:35
nessita* when you have a valid path in linux and you need to mangle it to be valid in windows, you need to:13:35
nessita - grab the linux path, which is bytes, and create a unicode string with it13:35
nessita - do the replace in the unicode string13:36
nessita - return that13:36
nessita* when you have a valid path in windows and you need to mangle it to be valid in linux, you need to:13:36
nessita - grab the windows path, which is unicode, and create a bytes sequence with it13:36
* fagan back 13:36
nessitasorry, that last is not correct13:36
nessita - grab the windows path, which is unicode, do the inverse replace and *then* create a bytes sequence with it13:37
nessita - return that13:37
nessitamandel: yes?13:37
mandelnessita: yes, that is correct13:38
ralsinaso they return different types. That's enough to have two methods13:38
mandelnessita: I was thinking of this order for the tests: http://paste.ubuntu.com/649110/13:39
mandelralsina: so.. are we using python as if it was typed?13:39
nessitamandel: not at all13:39
nessitamandel: but you need to have a clear API13:40
ralsinamandel: taking different types is good. Returning different types depending on unclear assumptions is not. Remember listdir :-)13:40
mandelnessita: honestly, I dont care that much, lets leave it in two13:40
mandelralsina: so listdir is not pythonic?13:41
nessitamandel: is sad that you don't care :-(13:41
ralsinamandel: it's confusing, is what it is :-)13:41
nessitaralsina, mandel: new rename was pushed: restore_illegal_chars is now replace_unicode_with_illegal_chars, and remove_illegal_chars is now replace_illegal_chars_with_unicode13:41
ralsinanessita: sounds good13:42
nessitaralsina: and now that I see those names, I wonder if path_name_from_linux path_name_to_linux isn't a better choice :-P13:42
nessitaI guess no, I don't like the name 'linux' in the method names13:43
mandelnessita: aint sad, I'm being pragmatic, you have your point of view, I have mine. It is already implemented by me in two methods, so why discussing? I did consider the two methods already, ergo I know its good and bad points. I'm challenging that implementation because we are writing very similar tests, which smells...13:43
nessitais too restrictive13:43
ralsinathe long ones are mre explicit, and "linux" reeks of "you know what I mean"13:43
mandelnessita: don't use linus, in mac os x is the same, as with bsd etc...13:43
mandeluse unix if you like13:43
nessitamandel: let's stick with the one I pushed already13:44
mandelnessita: ok, specially because we removed windows from the name for a reason, right?13:44
nessitamandel: yes13:44
nessitamandel: in the last rename, some lines go over 80 columns. I did not fixed that to avoid creating more conflicts with your branch, ok?13:45
nessitamandel: so, when you merge, you will need to fix some conflicts...13:45
mandelnessita: so, do you understand why I say I don't care? is more of a 'I consider both to have ts good and bad points'13:45
nessitamandel: I understand what you said, but worries me that you consider having a single method is a good option13:46
=== m_conley` is now known as m_conley
mandelnessita: yes, 'cause in ensures the encapsulation of the logic in a single location, rather than in several places, even if internally in that method is implemented in 2 diff one13:46
mandelto make it even cleaner you could hide the maps in the implementation so that people are not tempted to use it13:47
faganmandel: bsd sucks man :D13:47
nessitamandel: if you have 2 internal, different, implementations then you're not encapsulating any logic! :-)13:47
nessitamandel: of course, +100 to hide the char map13:47
mandelnessita: yes, since both operations are within the same method, let me show you (although this is just for the mental exercise, we wont be changing a thing :) )13:48
nessitamandel: regarding the code snippet, I still see it and think the goal of the test is not the last assert. But I guess that got cleared up after ralsina pointed out that we're not removing but replacing13:50
nessitamandel: so I would advice using the 2 code snippets I added in the MP for the 2 tests13:50
ralsinamandel: the concern about the methods getting out of sync is fixed by roundtrip tests and maybe a comment on each method mentioning that the roundtrip should be invariant.13:53
alecuralsina, are we mumbling soon?13:53
ralsinaalecu: good question. Oh, the time!13:54
nessitaralsina: I'm there13:54
ralsinaalecu, nessita, mandel, Chipaca: quick windows status mumble?13:54
alecunessita, It’s Time to Activate!13:54
alecunessita, got that mail?13:55
nessitaalecu: yes....13:55
nessitaIT ENDS... TONIGHT13:55
alecuI love this line: "Best wishes for your software development projects."13:56
nessitaalecu, mandel: mumble?13:56
alecuyeah, right.13:56
mandelralsina: going sorry13:56
ralsinamandel: I forgot too :-)13:56
* ralsina feels left out and did not get that email13:56
mandelralsina, nessita: this is waht I mean: http://paste.ubuntu.com/649128/13:57
mandelralsina: main idea, ensure that the function has the commutative property13:58
Chipacaralsina: OTP13:58
mandelnessita: look at my comment ^13:58
mandelnessita: I cannot hear you… :(14:14
mandelnessita: but I think its my network the problem14:14
nessitamandel: can you hear ralsina?14:15
mandelnessita: mumble is done, I'll be back asap14:15
mandelralsina, can hear! what do I need to know?14:15
mandelnessita: so what is the summary of the changes to be made in fix-os-helper, add the tests as you mentioned + a test that simply tests self.assertFalse(any for each of them, is that right?14:26
mandelnessita: did the decorator explanation help you?14:26
nessitamandel: yes to the decorator explanation14:28
nessitamandel: pending stuff: fix the tests as per my code snippets, no need to add an extra test for assertFalse(any(...)) since we agreed that is not needed because of the nature of the function (it does not perform removes but replacements)14:29
mandelnessita: ok, and merge with trunk and resolve merge issues, ok?14:30
ralsinaDiegoSarmentero: welcome to Canonical! (just got the official "new starter" email :-)14:34
DiegoSarmenteroralsina, THANKS!! :D14:34
nessitaDiegoSarmentero: heeeeeeeey! welcome!14:35
faganwelcome DiegoSarmentero14:35
DiegoSarmenteronessita, thanks everyone! :D I'm very happy!14:35
nessitaDiegoSarmentero: I have send an email confirming your flight to the agency, we'll have the electronic ticket tomorrow14:35
DiegoSarmenteronessita, awesome14:35
ralsinaDiegoSarmentero: traé alfajores al sprint14:35
mandelDiegoSarmentero: bienvenido!!!14:35
DiegoSarmenteroralsina, will do!14:36
DiegoSarmenteromandel, gracias!14:36
alecuDiegoSarmentero, bienvenido!14:36
DiegoSarmenteroalecu, gracias!!14:36
alecumandel, ralsina, DiegoSarmentero: please, I need one more review on this branch: https://code.launchpad.net/~alecu/ubuntuone-client/unbreak-connect/+merge/6830914:36
ralsinaAnd today, canonicaladmin magically looks ok on chromium. yay!14:36
ralsinaalecu: on it!14:37
mandelnessita, alecu, ralsina: did you get an email from msdn?14:37
ralsinaalecu: I tested it yesterday and forgot to approve it, sorry!14:37
nessitamandel: yeap14:37
ralsinaI did not, but I have windows licenses already14:37
alecumandel, yes!14:37
alecuralsina, ok... please, approve it!!!!!!14:38
ralsinaalecu: done!14:38
dobeyman, ppa builds are slow today14:51
nessitaralsina: I'm leaving to sign that thingy at the university14:53
nessitaI'll be back ASAP14:53
ralsinanessita: ok14:53
ralsinanessita: have lunch too14:53
nessita(but it will take one hour approx)14:53
ralsinajust coordinate mandel's handoff...14:53
nessitamandel: how long are you staying?14:54
mandelnessita: more or less when will you be back, we can coordinate it so I have a small break and I'm back to talk with you guys14:54
nessitamandel: I can be here in approx one hour, depending on the traffic14:55
mandelnessita: I can take a break and connect back at my 8 for example14:55
nessitamandel: or I can do this tomorrow...14:55
nessitalet me do some phone calls14:55
mandelnessita: look dont worry, do your stuff at uni, worst worst case scenario, tom I start at you 9 and we do then the hand off14:56
mandelnessita: If not, we can do it later 2night, I have no problem what so ever14:57
nessitamandel: I'm making some phone calls14:57
mandelnessita: ok, but you seriously don't have to14:59
nessitamandel: ok, I moved my thing for tomorrow15:01
mandelnessita: ok15:01
nessitamandel: let me know as soon as you can make the hand off to me15:02
nessitamandel: I'm not in a hurry, but maybe is late for ytou15:02
mandelnessita: sure15:02
mandelnessita: is not, I started one hour later today, so we are fine :)15:03
nessitamandel: I imagine, I read you were drinking with 18-years old last night until 4am your time!15:03
mandelnessita: I'm pushing the changes to fix-os-helper, will run tests on linux and will let you know15:03
nessitamandel: ack15:03
mandelnessita: yes, but I sleep an average of 4 hours… and with their conversation, I felt sleep earlier than usual :)15:04
mandelso… while I wait for the branch to be done over 3g… ralsina, nessita, alecu, DiegoSarmentero want anything from Spain that I can legally  travel with from Spain, *?/: are illegal by the way :P15:12
ralsinamandel: so are cured meats, sadly15:12
ralsinaBTW: expect marianna's mail soonish, but the hotel will be http://www.hoteljacaranda.com.ar15:13
alecumandel, a knife from toledo?15:13
mandelralsina: really? that is lame...15:13
nessitamandel: corn noddles, they are sold in "Al campo", and they are very tasty and gluten free!15:13
DiegoSarmenteroralsina, it seems nice! :P15:13
mandelalecu: hahaha I'll be in madrid so no idea i I could :)15:13
mandelnessita: I can get you those for sure!15:13
alecumandel, oh, then perhaps I can ask you for some board game.15:14
nessitamandel: you sure? only if it's not that much trouble15:14
mandelalecu: shoot, but dont make it too complicated15:14
nessitamandel: of course I will pay for them, they cost something like 1.5 euros the 500g the package15:14
mandelnessita: no trouble at all, and you have to get something out from the reviews, right?15:14
mandelnessita: but do send me an email with the details, I don't want to bring the wrong stuff15:15
dobeyralsina: oh nice15:15
ralsinanessita: too bad you are not here on saturday. 2 minutes from the hotel a friend of mine has a organic producers market, I am sure they have any kind of noodles made of weird grains :-)15:15
alecumandel, try to see if you can get the main "battlestar galactica" board game in spanish. The base one, make sure it's not one of the expansions :-)15:15
ralsinanessita: I may go there this saturday and get you amaranth noodles. I always wanted to ask somene to try those ;-)15:15
alecumandel, I can trade that for asado15:15
nessitaralsina: bring them on!15:16
alecumandel, I mean, using you as a mule: I'll pay for the game :-)15:16
mandelalecu: sure, I think I know where to get that...15:16
DiegoSarmenteroalecu, noooo, battlestar galactica is awesome!!!!15:16
alecuDiegoSarmentero, where did you play it? at toote's?15:17
ralsinanessita: we went there with rayis last saturday. I'll try to convince the wife to go again :-)15:17
dobeyi am sad i won't be there15:17
DiegoSarmenteronono...... i didn't play the board game, i mean the tv show........ and........... everything related :P15:17
ralsinadobey: next time!15:18
* alecu had to look up "Amaranth", because he pictured "Amanith", and it did not made sense for a noodle.15:18
DiegoSarmenteroalecu, i answer above... :P15:18
alecuDiegoSarmentero, hahaha15:18
ralsinaalecu: amanith noodles would be so much fun! :-)15:18
alecumandel, so, remember "battlestar galactica". That or a leg of "pata negra"15:20
mandelalecu: 5 J I suppose...15:21
alecumandel, right! jajaja15:25
mandelalecu, nessita, ralsina: please: https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/6853015:28
mandelDiegoSarmentero, nessita: which airport are you arriving to and when? we might be able to share a taxi...15:28
DiegoSarmenteronessita, ?15:29
DiegoSarmenteromandel, i don't know yet15:29
ralsinamandel: they arrive at the local one, you at the internationa15:29
nessitamandel: we arrive to Aeroparque, at almost 11pm15:29
nessitamandel: and you arrive to Ezeiza, at 20pm, right?15:29
ralsinamandel: looking15:29
mandelnessita: indeed, so no point to share15:29
nessitamandel: right15:30
* DiegoSarmentero is away: Not here... send me an e-mail! :P15:34
faganmandel: what IRC client do you use on your mac?15:35
* fagan was wondering what is a good one 15:35
mandelfagan: adium15:35
mandelfagan: I use it for a set of diff protocols15:35
fagangoogled but I got bad ones15:35
ralsinamandel: expect to pay $250 for the car to the hotel (about 45 euros or so)15:36
nessitamandel: I strongly advice to arrange the price before hiring the service, inside the same airport15:36
nessitaright after the custom control15:36
mandelralsina: joder!!! how far is it?15:36
ralsinanessita, DiegoSarmentero: you can expect to pay about $100 from your airport depending on how much they rob you :-(15:36
ralsinamandel: 50km15:36
mandelnessita: yeah, I did learn about that last time :)15:36
nessitaralsina: right15:36
nessitaralsina: I may call a remis company from here just to see how much they will charge. Do you recommend any from your area?15:37
* mandel small brake, will be back in 10 min15:37
ralsinanessita: you can call (011) 4743565815:38
ralsinanessita: that's one from near my home, so not too far from the hotel15:38
ralsinanessita: I will ask them15:38
ralsinanessita: $85 but no factura15:41
nessitaralsina: we need factura... is it more expensive with the receipt?15:41
ralsinanessita: they can make you an unofficial one, but if they have to do an official one, it's 100 so... no point15:41
nessitaralsina: ok, is a good reference to negociate there15:43
ralsinaIf you don't mind walking 6 blocks or so, you can go to the Maipú station of the "tren de la costa" and then $4 for the train, but I am not sure if they work so late15:43
ralsinathat's about half of the way15:44
dobeysso nightlies on maverick working again15:44
dobeyok, time to get some lunch. bbiab15:44
ralsinasame here!15:45
ralsinasee you all later15:45
ralsinamandel, nessita, alecu: I am finishing the review after lunch (fix-os-helper)15:45
=== ralsina is now known as ralsina_lunch
mandelralsina_lunch: ack15:45
alecuralsina_lunch, cool15:45
alecumandel, oh, and remember to say that you are *not* coming to work here :-)15:47
alecuor you'll have to get a visa.15:48
mandelalecu: yes, I know… I'm more worried about the diff between the passport pict and my current looks15:49
alecumandel, oh, so you've finally went ahead with your sex-change operation?15:52
mandelalecu: call me mari carmen please15:52
alecu"le llamaban juana la loca"!15:53
nessitamandel: you around?16:01
mandelnessita: around I am16:01
nessitamandel: so, my head is close to explode. I'm seeing something in your branch that... if I got it right, scares me :-D16:02
nessitamandel: can you please look the test in16:02
nessita75     def test_replace_unicode_with_illegal_chars(self):16:02
nessitado you see something odd here?16:02
mandelnessita: what?!16:02
nessita84+        original_path = 'test'.join(LINUX_ILLEGAL_CHARS_MAP)16:02
nessita85+        path = replace_unicode_with_illegal_chars(original_path)16:02
mandelnessita: so you get test+ unicode_chars and then yo get test+illeglachars, right?16:04
nessitamandel: not sure what that means16:04
mandelnessita: since LINUX_* is a unicode to illegal char map16:04
nessitamandel: so, in both tests, which tests different stuff, you're building the same original path:16:05
nessitaoriginal_path = 'test'.join(LINUX_ILLEGAL_CHARS_MAP)16:05
nessitaoriginal_path = 'test'.join(LINUX_ILLEGAL_CHARS_MAP)16:05
nessitathose are from diff lines 84 and 9716:05
nessitamandel: that has no sense, one path should have the windows unicode chars, the other the linux bytes chars16:05
mandelnessita: joder… que dia… no wonder the tests pass16:07
nessitamandel: I can take over your branch, maybe that's the best option?16:07
nessitalisto, eso fue todo :-D16:08
nessitamandel: I can take over your branch, maybe that's the best option?16:08
nessitamandel: so, maybe you should call it a week already, and just do the hand off of your branches to me16:09
nessitamandel: I'd like to schedule my lunch, but I want to coordinate the hand off from you first16:09
mandelnessita: have lunch, I have a coffee to wake up and we handle the branches of, is that ok?16:10
mandelnessita: just pushed the correct test16:13
nessitamandel: I'll have lunch then, I'll be back before the next o'clock (ie in less than 45 minutes)16:14
mandelnessita: I'll have coffee then16:14
* alecu will be out as well16:46
mandelnessita: I need to walk we dog, he tried to scape because he needs to do his things, I might be back a little later than the next o'clock but not by much16:47
alecumandel, will you be pushing more changes to the branch?16:47
alecumandel, I'll probably be online in an hour to update that branch and probably mumble with you and nessita.16:48
alecumandel, or are you done for the day?16:48
alecuallright, I'm off for an hour or less.16:49
nessitamandel: I'm back16:51
dobeynessita: here's a fine start for you: https://code.launchpad.net/~dobey/ubuntuone-client/less-gnome/+merge/6871316:54
nessitaalecu: we will be taking over mandel's branches, including the one for review16:54
nessitadobey: ack16:54
=== beuno is now known as beuno-lunch
* DiegoSarmentero is back (gone 01:41:41)17:15
ralsina_lunchnessita: if I understand correctly, https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/68530 is deprecated or already has needsfixings that are not in the MP?17:18
nessitaralsina_lunch: is not deprecated. What needs fixing should be there that are not?17:21
ralsina_lunchnessita: I was checking the backlog and saw your comment to mandel about a test that was not done correctly (using the same string to check both path replacement methods)17:21
=== ralsina_lunch is now known as ralsina
nessitaralsina: ah, yes, mandel pushed that fix before I could comment in the MP17:22
ralsinaAnd I was not sure if that was already pushed or not (the branch is big enough that it's not obvious :-)17:22
ralsinaok then, back to reading17:22
nessitaralsina: let me know how it goes17:23
ralsinanessita: slow! :-D17:23
mandelnessita: I'm back, how do you want to do the handover17:23
nessitamandel: yes please17:23
nessitamandel: mumble?17:23
ralsinaI keep getting to check the original sources because the diff lacks context, but it's moving17:23
mandelnessita: though 3G wont be very good… sorry :(17:24
nessitaralsina: can you join mandel and me, so the modules names start to sound familiar to your ears?17:24
nessitamandel: but just like this morning?17:24
dobeyyay, and sso nightlies building again on lucid too17:24
mandelnessita:I was missing half of the conversation this morning, give me 5 min and I'll try to use the neighbords wifi17:25
nessitamandel: nice!17:25
nessitaI'll grab some tea17:25
mandelnessita: will ping when I'm in a decent connection17:25
mandelor can use mumble, one of those17:25
dobeywatch out for solar flares17:27
nessitadobey: is all red! :-D17:32
mandelnessita: I'm back, neighbor is not there, lets try with 3g17:33
nessitamandel: we're there17:34
dobeynessita: mostly, yep :)17:38
mandelnessita: se me oye?17:47
nessitamandel: sip17:49
nessitaalecu: if you can, we're mumbling right now re: mandel's branches17:50
alecunessita, I'm having lunch, so I'll join on listen only mode.17:59
=== beuno-lunch is now known as beuno
mandelnessita, ralsina, alecu: http://paste.ubuntu.com/649338/18:03
* alecu runs18:57
ralsinanessita: which one takes unicode, "replace_unicode_with_illegal_chars" ?18:58
* ralsina is slightly dizzy with this branch18:58
nessitaralsina: replace_unicode_with_illegal_chars takes unicodes18:59
nessitaas params19:00
nessitareturns bytes19:00
ralsinanessita: then there is something very wrong in test_replace_unicode_with_illegal_chars19:12
ralsinanessita: we are doing "unicodething in path" where path is bytes. That is always false.19:12
ralsinaoh, wait, the keys a re bytes19:13
nessitaralsina: I thought so too, then I discussed it with Cheepaca, and we agreed the test is right but what is misleading is the name of the mappings.19:13
nessitaralsina: LINUX_ILLEGAL_CHARS_MAP's keys are unicodes19:14
nessitaso, 'test'.join(LINUX_ILLEGAL_CHARS_MAP) is unicode19:14
nessitaof course the test legibility will improve if we did19:14
nessitaand probably picking better names for the mappings19:14
ralsinanessita: I was about to put a needsfixing about u'test' to be obvious that is unicode19:15
nessitaralsina: you do that, I'll take over this branch once all the needs fixing are in place19:15
ralsinabut then again line 99 seems wrong, because path is unicode, and the keys in WINDOWS_ILLEGAL_CHARS_MAP are strings19:16
nessitaralsina: so, I first panicked like you when I saw the latests changes :-D19:16
nessitaralsina: line 99 is this for me19:16
nessita99+        assert not any(c in path for c in WINDOWS_ILLEGAL_CHARS_MAP)19:16
nessitaralsina: you're right, path is unicode19:16
ralsinathat path is the output of replace_illegal_chars_with_unicde, which is unicode, right?19:16
nessitaand  WINDOWS_ILLEGAL_CHARS_MAP's keys are chars19:17
ralsinait may work, but mostly by accident19:17
ralsinaI am not sure what would be a meaningful way to do that though19:17
ralsinaThose are the only needfixes I found, probably due to ignorance of context in other places. The abundance of non-unicode strings in the tests makes me nervous.19:23
ralsinaI will make another pass, though19:23
ralsinaOf course, the exact same problem happens in the other test. We are comparing a bytes path with unicode keys of LINUX_ILLEGAL_CHARS_MAP19:24
dobeyjust allow the lowest common denominator of legal filename characters?19:26
ralsinadobey: we can't choose the user's filenames :-)19:27
ralsinaI wish we could!19:27
dobeywe can choose what filenames to not work with :)19:27
nessitadobey: and we need to be able to sync your valid filenames in linux to windows19:27
ralsinanessita: posted needsfixing on the MP19:28
dobeynessita: you misunderstood maybe. my suggestion was to reduce the set of supported filenames to something that works on both :)19:28
nessitaralsina: great! let's wait for alecu's19:28
dobeyso that would be a non-issue19:28
nessitadobey: ah... oh19:28
nessitadobey: I think that is a no-no, at least from previous discussions19:28
dobeyi don't see why not. ubuntuone != linux. we are our own filesystem. *shrug*19:29
dobeydon't know why you'd have \ or ? in filenames anyway19:30
dobeyit's not like they work valiantly well in linux19:30
ralsinadobey: I get that often by using wget to download stuff19:30
ralsinaI get "?" and "&"  pretty often :-)19:31
dobeyyes, well, wget is not the smartest piece of software in the world19:31
nessitadobey: I have several images called wtf?.png :-)19:31
ralsinano kidding. It's dumb as a dumb rock19:31
dobeynessita: stop that. :)19:32
nessitadobey: but is fun!19:32
dobeynessita: the ? is redundant there19:32
nessitadobey: well, I have others called ugly!.png19:32
dobey! is ok on windows i think, so that's not a problem :)19:33
ubot4dobey: Error: I am only a bot, please don't think I'm intelligent :)19:33
* ralsina still has "ñandu ñoño.png"19:33
dobeyubot4: trust me, i don't19:33
ubot4dobey: Error: I am only a bot, please don't think I'm intelligent :)19:33
dobeyi have 我的光碟.zip19:35
ralsinasee? You *must* sync that19:35
ralsinaok, I am done looking at that branch. My eyeballs are bleeding a bit.19:36
dobeyi just hit the thisfred bug19:36
ralsinais it the one where you start tagging things as "songs that have thunder sounds in them"?19:37
dobeyno, the notification bubble showed an old filename19:37
dobeywhen i copied the chinese zip file into my u1 folder19:37
dobeyhmm, and it didn't upload19:38
thisfredYeah, still no idea what is causing this.19:39
thisfredI proposed a branch that doesn't fix it though19:39
dobeyoh, it is still WORKING19:41
dobeybut my network usage is nil19:41
dobeyonly hits i get are typing in irc19:41
thisfredthis is unlikely to be the notifications' fault ;)19:44
dobeyit is definitely stuck19:46
dobeybut i don't see any evidence of why in the log19:46
dobeyit called MakeFile like 20 times, and there's not even an empty file on the server19:46
thisfredLots of MakeFile calls is also something I also saw in facundo's logs when he had the issue19:54
thisfredI still think there are at least two different problems here19:54
thisfredWell obviously, but I mean two bugs behind the two problems19:55
dobeyi just happened to hit both of them :(19:56
thisfredwell, one may be triggering the other19:56
facundobatistathisfred, when I had which issue?19:56
thisfredfacundobatista:  the wrong filename in the notification19:56
dobeyfacundobatista: my syncdaemon is stuck19:57
facundobatistadobey, debug logs?19:57
ralsinanessita, alecu: it's less than $100 (U$S25) to upgrade the hotel's internet from 4Mb to 10Mb for a week. I think I will do that ;-)19:57
dobeyfacundobatista: nothing in exceptions.log, which is old. and syncdaemon.log has a bunch of MakeFiles, and a few Uploads for the same file, but nothing that jumps out as the problem.19:58
dobeyfacundobatista: the filename was chinese, but also valid utf-8, if that matters19:59
nessitaralsina: FOR SURE!20:00
nessitaralsina: I need to download some stuff (?)20:00
ralsinaalso, from 512Kb to 1.5Mb upload20:00
facundobatistadobey, and it's stuck doing what?20:02
dobeyfacundobatista: 2011-07-21 15:55:03,262 - ubuntuone.SyncDaemon.Main - NOTE - ---- MARK (state: <State: 'QUEUE_MANAGER'  (queues WORKING  connection 'With User With Network')>; queue: 1; hash: 0) ----20:03
dobeyfacundobatista: just kept doing that20:03
dobeyhrmm, though now it won't connect, because apparently ubuntu-sso-client got broken20:03
facundobatistadobey, what do you mean won't connect? it says it's connected20:04
dobeyfacundobatista: that's what it was stuck doing. i did a quit, to check if there were updates, and apparently the last time i updated sso, it was broken, so now it won't reconnect20:05
dobeyand now it is reconnected, since new sso works again20:06
dobeyand now i got a notification with the right filename20:06
dobeybut is still not uploading the file :-/20:07
facundobatistadobey, that is very strange indeed... can you pass me the logs?20:09
nessitaralsina: you available for an IU review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/unify-account-and-service/+merge/6874320:10
ralsinanessita: sure20:10
dobeyhrmm, now it says idle20:10
dobeyand the file appears on the web ui20:10
dobeyi guess it finally uploaded it20:11
dobeyfacundobatista: i must have hit a bug that was fixed already :)20:11
facundobatistadobey, the simple magical process of sending me the logs most of the times fixes the problem, that's why I always ask for them :D20:13
ralsinanessita: hahaha self.ui.password_lineedit.setText("swordfish")20:14
nessitaralsina: all credits to alecu :-)20:14
ralsinatoo bad you are removing it :-)20:15
ralsinanessita: +1 on that branch20:20
nessitadobey: you may know this: I accidentally removed the 'systray' from my panel, and now I can find it within the applet list to restore it20:30
nessitadobey: any idea what is the name of that applet? I know is supposed to go away, but I still need it for stuff like skype and sonata20:31
dobeynessita: notification area20:32
nessitaah! THANKS!20:33
* nessita hugs sonata back20:33
thisfrednessita: so you use mpd?20:38
nessitathisfred: I do. I love it, except when I have to use both mpd and mumble20:38
nessitathey both go crazy20:38
thisfredheh, well, that would be mumble's fault I'm guessing20:38
nessitathisfred: pretty likely, mpd only gets broken by mumble20:39
thisfredI really like mpd too. I used to use it a lot in my previous job20:39
thisfredwhere I set up an office radiostation with it20:39
nessitathisfred: we use it a lot in the 'office' ;-)20:39
thisfredyeah, it's really cool when anyone can add or skip music20:40
nessitaI threat people that skips my choosing of music, but yeah :-P20:40
nessitathisfred: I bet you're amazed by that ;-)20:41
thisfredI had an mpd+autoqueue+icecast internet radiostation at one point, but since I was the only listener, there wasn't that much of a point :)20:42
thisfredI wonder if there are android clients for mpd20:43
dobeyalright. am out. have a good evening all20:53
ralsinaEOD for me, mail me if you need me!20:54
=== m_conley is now known as m_conley_away
nessitaok, I'm off for the day22:13
nessitasee ya all tomorrow!22:13

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