[08:49] <mandel> morning all!
[09:09] <fagan> Morning sorry about being a bit late had to make like 8 phone calls to locate my package
[09:09] <fagan> but found it
[09:09] <fagan> \o/
[09:10] <fagan> The 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 house
[09:10] <fagan> so weird
[09:18] <mandel> fagan: don't worry I think I'm the only one who notices at this time :P
[09:19] <mandel> fagan: fancy to see a good code review?
[09:25] <fagan> mandel: sure
[09:25]  * fagan was grabbing tea
[09:25] <mandel> fagan: https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/68530 nessita does really good ones :)
[09:26] <fagan> mandel: 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 detail
[09:26] <fagan> holy shit though she doesnt like your code :D
[09:26] <mandel> fagan: nah, dont worry, this is just for you to learn, part of the internship ;)
[09:26] <mandel> fagan: is more the dialog, since I do have my comments as well hehehe
[09:27] <fagan> mandel: haha well being nessita would be pretty hard to be in 6 months
[09:28] <mandel> fagan: will certainly be impossible
[09:28] <fagan> mandel: im sure there would be some mind altering drugs or something that would help
[09:30] <mandel> nah, to dangerous
[09:54] <mandel> fagan: 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 min
[09:54] <fagan> mandel: cool
[09:54] <fagan> Ill hold the fort
[09:56]  * fagan takes 10 minutes to write up some notes to talk about with matthew later
[11:25]  * fagan takes a break 
[11:26] <nessita> hello everyone!
[11:27] <fagan> nessita: hey, if you are looking for mandel he is at a cyber cafe doing updates and cant get onto freenode
[11:27] <nessita> fagan: ack, thanks for the info
[11:27] <fagan> nessita: he had to reboot but should be back soon on the private irc
[11:55] <ralsina> good morning!
[11:55] <fagan> morning ralsina
[11:56] <fagan> ralsina: im looking down the bug list to have a look at the triage policies if thats cool
[11:57] <fagan> wanted to see how the big bugs get attention
[11:59] <fagan> policy is a bad word I mean more what logs are needed for specific things and all that
[11:59] <fagan> procedure is a better one
[12:00] <ralsina> fagan: good idea
[12:00] <nessita> hi ralsina! you feeling better?
[12:00] <ralsina> nessita: MUCH!
[12:00] <ralsina> nessita: even slept 8 hours and everything
[12:12] <fagan> welcome back mandel
[12:12] <fagan> :)
[12:12] <mandel> fagan: yes :)
[12:13] <mandel> there was something failing in the handsake.. I hate the internet :P
[12:13] <fagan> mandel: lets make a replacement :D
[12:14] <mandel> fagan: I do have a theory that proves that spaniards had wireless 1000 years ago :)
[12:14] <fagan> mandel: smoke signals?
[12:15] <mandel> fagan: yelling!
[12:15] <fagan> im good at that :D
[12:17] <james_w> hi, I've just set up a new oneiric  machine, but don't have an ~/Ubuntu One folder on it
[12:18] <james_w> other folders are synced fine, and the control panel says that it is synced
[12:18] <james_w> I've even seen notifications of files being downloaded
[12:18] <fagan> james_w: thats really weird
[12:18] <james_w> double clicking the folder in the control panel does nothing, but works for other folders
[12:18] <james_w> any clues?
[12:19] <fagan> nessita: ^
[12:19] <fagan> I dont have a clue I didnt even thing the verson in oneiric was actually working properly yet
[12:21] <fagan> james_w: on the website is the folder there?
[12:22] <james_w> hmm, seems not
[12:22] <fagan> james_w: thats even more weird
[12:22] <fagan> check your bin
[12:23] <james_w> but it's on my natty machine
[12:23] <fagan> james_w: hmmm and there is stuff in it?
[12:23]  * fagan realised that doesnt really matter 
[12:23] <james_w> there are two files in the trash on the oneiric machine, both of which would be somewhere under my ~/Ubuntu One
[12:24] <fagan> james_w: aha
[12:25] <james_w> it's still on my natty machine, but some files may be missing, and there's a u1conflict dir
[12:25] <james_w> which would be the dir containing the files in the trash on the oneiric machine
[12:26] <james_w> and there's a whole bunch of the files in the trash on the natty machine
[12:26] <fagan> james_w: well it sounds like some sort of sync issue but id need rye or someone to help debug it
[12:26] <james_w> it seems like a rather unfortunate failure mode
[12:26] <fagan> yeah its definitely something strange
[12:27] <rye> fagan, reading
[12:29] <rye> james_w, no Ubuntu One folder in ~home ???
[12:29] <james_w> correct
[12:31] <mandel> nessita: morning, answering on the MP the questions and commenting on the tests
[12:36] <nessita> mandel: I answered, did you see that?
[12:40] <mandel> nessita: yes, ill get an answer for that in some min :)
[12:40] <ralsina> mandel: could we mumble in 5'? Short!
[12:40] <nessita> mandel: in the mean time, I'll make a branch for the rename of the remove_windows_illegal_chars only
[12:40] <nessita> mandel: so your branch gets smaller
[12:41] <mandel> ralsina: can't, can it way 'til after the tandup, I'm having lunch :)
[12:42] <ralsina> mandel: sure
[12:42] <mandel> nessita: very well, pass me the merge proposal if you will :)
[12:44] <rye> james_w, well, i am pretty sure ubuntuone cannot work w/o a root folder
[12:45] <rye> james_w, ummm... could you please check whether ~/.cache/ubuntuone/log/syncdaemon-exceptions.log exists and whether there is something interesting there?
[12:46] <james_w> Failure: twisted.internet.error.ConnectionDone: Connection was closed cleanly.
[12:47] <james_w> TypeError: unsupported operand type(s) for +: 'int' and 'NoneType'
[12:48] <fagan> Im going to give my standup now since ill be on skype during
[12:48] <fagan> me
[12:48] <fagan> DONE
[12:48] <fagan> * Read the triage process for high and critical bugs
[12:48] <fagan> TODO
[12:48] <fagan> * More reading of triage stuff (unless someone has some task)
[12:48] <fagan> BLOCKED
[12:48] <fagan> * Nope
[12:48] <fagan> NOTE: On skype at the moment talking about bug assigning will be a few mins
[12:48] <fagan> next probably nessita
[12:49] <james_w> rye, there's lots of that second exception. I also see it creating the root dir in the start of syncdaemon.log
[12:50] <rye> james_w, could you please pastebin it?
[12:50] <james_w> rye, syncdaemon.log?
[12:51] <james_w> the first one, or the current one?
[12:51] <james_w> (it's been rotated three times)
[12:51] <rye> jamalta, syncdaemon exceptions, with int + NoneType
[12:53] <james_w>  :q
[12:56] <ralsina> fagan dobey alecu nessita DiegoSarmentero mandel:standup in 4'
[12:56] <fagan> ralsina: already did mine
[12:57] <fagan> and I though DiegoSarmentero's contract is up
[12:57] <ralsina> fagan: why did you do it 12 minutes early?
[12:58] <fagan> ralsina: going on skype in a minute for lp usability testing
[12:58] <ralsina> And yes, his contract is up. He's still doing some stuff though, and he's welcome to join :-)
[12:58] <ralsina> fagan: ok
[12:58] <mandel> ralsina: ping, and ingnore the other channel one :)
[12:58] <ralsina> alecu is not going to make it but I have his notes
[12:58] <ralsina> mandel: pong
[12:58] <mandel> ralsina: so, we do mumble after stan up?
[12:58] <ralsina> yes please
[12:59] <mandel> ralsina: or can we multitask, I don't think I can :P
[12:59] <ralsina> me neither
[12:59] <mandel> ralsina: so after standup then :)
[12:59] <alecu> ralsina, I made it!
[12:59] <ralsina> alecu: welcome and say me then :-)
[12:59] <nessita> alecu: heeeeeey!
[12:59] <alecu> ralsina, the connection on the ferry is better than I thought
[13:00] <nessita> me
[13:00] <mandel> me
[13:00] <alecu> me
[13:00] <ralsina> me
[13:00] <nessita> mandel: https://code.launchpad.net/~nataliabidart/ubuntuone-client/rename-method/+merge/68676 (trivial)
[13:00] <nessita> dobey: ?
[13:00] <nessita> DONE: bug #813099, bug #813157, reviews, meetings, QT learning
[13:00] <nessita> TODO: ultimate details for bug #813099, make syncdaemon branches land
[13:00] <nessita> BLOCKED: nopes
[13:00] <nessita> NEXT: mandel
[13:00] <ubot4> Launchpad 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/813099
[13:00] <ubot4> Launchpad bug 813157 in ubuntuone-client "Windows: no need to customize expanduser (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/813157
[13:01] <mandel> DONE: 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 paths
[13:01] <mandel> TODO: Probably chat a little with nessita to see where we agree :), review her branch. Reporpose fix-os-hlper and others
[13:01] <mandel> BLOCKED: no
[13:01] <mandel> alecu, go go
[13:01] <alecu> DONE: almost ready branches for bug #803669, using the windows user id
[13:01] <alecu> TODO: finish branches for bug #803672
[13:01] <alecu> BLOCKED: need one more review for the unjelly branch:
[13:01] <alecu> https://code.launchpad.net/~alecu/ubuntuone-client/unbreak-connect/+merge/68309
[13:01] <alecu> NEXT: ralsina
[13:01] <ubot4> Launchpad 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/803669
[13:01] <ubot4> Launchpad 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/803672
[13:01] <dobey> me
[13:01] <ralsina> DONE: calls. I was not feeling/coding well, went to the doctor, took the evening off because of that. TODO: reviews, installer fixes. BLOCKED: no
[13:01] <ralsina> dobey, please
[13:02] <dobey> λ DONE: Registered ubuntuone-credentials and ubuntuone-client-gnome projects, started reorg splitting work
[13:02] <dobey> λ TODO: reorg work, shim code, applly for motu, apply for ~ubuntu-desktop
[13:02] <dobey> λ BLCK: None.
[13:02] <ralsina> ok, comments?
[13:02] <mandel> ralsina: feeleing ok today?
[13:02] <alecu> uhhh
[13:02] <alecu> lots of lag
[13:02] <ralsina> mandel: feeling perfect
[13:03] <mandel> that is what she said
[13:03]  * mandel is getting very good at this :)
[13:03] <ralsina> they *always* say that to me :-)
[13:03] <fagan> mandel: not really
[13:03] <alecu> ralsina, stop eating crap!
[13:03] <alecu> mandel, ^
[13:03] <ralsina> alecu: definition of crap varies.
[13:03] <mandel> hahaha
[13:03] <mandel> alecu: tha tis all yours :)
[13:04] <mandel> ralsina: launching mumble right now
[13:04] <ralsina> ok, so I assume there are no comments relative to the standup, thus EOM
[13:06] <nessita> eom!
[13:08] <alecu> nessita, 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:09] <nessita> alecu: ack
[13:09] <mandel> ralsina: felt from mumble :(
[13:09] <mandel> ralsina: but ok, I'll got to the admin page and will add it
[13:13] <ralsina> mandel: cool
[13:16] <mandel> nessita: can you please take a llok at my comment in the MP and let me know what you think
[13:20] <nessita> mandel: sure
[13:21] <mandel> alecu, ralsina: one more: https://code.launchpad.net/~nataliabidart/ubuntuone-client/rename-method/+merge/68676
[13:21] <ralsina> mandel: on it!
[13:28] <nessita> mandel: 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:29] <nessita> mandel: since in a test, it doesn't make much sense to have an assertFalse and before exiting a builtin assert
[13:29] <mandel> nessita: why is thaT?
[13:29] <nessita> mandel: yes?
[13:29] <mandel> nessita: we can swap the around, right?
[13:30] <nessita> mandel: not really.... as far as I can see
[13:30] <nessita> mandel: wanna show me a code snnipet?
[13:30] <mandel> nessita: sure, give me a sec
[13:30] <nessita> of what are you thinking on, I may be missing something
[13:30] <ralsina> IMVHO: if the function replaces, then you should test replacement, not removal
[13:31] <nessita> ralsina: very good point
[13:31] <nessita> mandel: also the rename of the method should be to replace_foo
[13:31] <mandel> nessita: ok
[13:31] <nessita> ralsina: ^ so I guess I can change that in my branch about renames
[13:33] <mandel> nessita: 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] <mandel> for example
[13:33] <nessita> mandel: no, 2 methods is always preferred and more pythonic
[13:34] <mandel> nessita: but then you have issues if one method changes and the other does not...
[13:34] <ralsina> yeah, replace_foo and 2 methods makes sense
[13:34] <nessita> mandel: what do you mean?
[13:34] <nessita> mandel: with proper tests, that should not happen
[13:34] <mandel> really? I guess my brain is wired in a diff way
[13:35] <nessita> mandel: 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 it
[13:36] <nessita>  - do the replace in the unicode string
[13:36] <nessita>  - return that
[13: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 it
[13:36]  * fagan back 
[13:36] <nessita> sorry, that last is not correct
[13:37] <nessita>  - grab the windows path, which is unicode, do the inverse replace and *then* create a bytes sequence with it
[13:37] <nessita>  - return that
[13:37] <nessita> mandel: yes?
[13:38] <mandel> nessita: yes, that is correct
[13:38] <ralsina> so they return different types. That's enough to have two methods
[13:39] <mandel> nessita: I was thinking of this order for the tests: http://paste.ubuntu.com/649110/
[13:39] <mandel> ralsina: so.. are we using python as if it was typed?
[13:39] <nessita> mandel: not at all
[13:40] <nessita> mandel: but you need to have a clear API
[13:40] <ralsina> mandel: taking different types is good. Returning different types depending on unclear assumptions is not. Remember listdir :-)
[13:40] <mandel> nessita: honestly, I dont care that much, lets leave it in two
[13:41] <mandel> ralsina: so listdir is not pythonic?
[13:41] <nessita> mandel: is sad that you don't care :-(
[13:41] <ralsina> mandel: it's confusing, is what it is :-)
[13:41] <nessita> ralsina, 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_unicode
[13:42] <ralsina> nessita: sounds good
[13:42] <nessita> ralsina: and now that I see those names, I wonder if path_name_from_linux path_name_to_linux isn't a better choice :-P
[13:43] <ralsina> hehe
[13:43] <nessita> I guess no, I don't like the name 'linux' in the method names
[13:43] <mandel> nessita: 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] <nessita> is too restrictive
[13:43] <ralsina> the long ones are mre explicit, and "linux" reeks of "you know what I mean"
[13:43] <mandel> nessita: don't use linus, in mac os x is the same, as with bsd etc...
[13:43] <mandel> use unix if you like
[13:44] <nessita> mandel: let's stick with the one I pushed already
[13:44] <mandel> nessita: ok, specially because we removed windows from the name for a reason, right?
[13:44] <nessita> mandel: yes
[13:45] <nessita> mandel: 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] <nessita> mandel: so, when you merge, you will need to fix some conflicts...
[13:45] <mandel> nessita: 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:46] <nessita> mandel: I understand what you said, but worries me that you consider having a single method is a good option
[13:46] <mandel> nessita: 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 one
[13:47] <mandel> to make it even cleaner you could hide the maps in the implementation so that people are not tempted to use it
[13:47] <fagan> mandel: bsd sucks man :D
[13:47] <nessita> mandel: if you have 2 internal, different, implementations then you're not encapsulating any logic! :-)
[13:47] <nessita> mandel: of course, +100 to hide the char map
[13:48] <mandel> nessita: 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:50] <nessita> mandel: 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 replacing
[13:50] <nessita> mandel: so I would advice using the 2 code snippets I added in the MP for the 2 tests
[13:53] <ralsina> mandel: 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] <alecu> ralsina, are we mumbling soon?
[13:54] <ralsina> alecu: good question. Oh, the time!
[13:54] <nessita> ralsina: I'm there
[13:54] <ralsina> alecu, nessita, mandel, Chipaca: quick windows status mumble?
[13:54] <alecu> nessita, It’s Time to Activate!
[13:55] <alecu> nessita, got that mail?
[13:55] <ralsina> Activia?
[13:55] <nessita> alecu: yes....
[13:55] <nessita> IT ENDS... TONIGHT
[13:55] <alecu> hahaha
[13:56] <alecu> I love this line: "Best wishes for your software development projects."
[13:56] <nessita> alecu, mandel: mumble?
[13:56] <alecu> yeah, right.
[13:56] <mandel> ralsina: going sorry
[13:56] <ralsina> mandel: I forgot too :-)
[13:56]  * ralsina feels left out and did not get that email
[13:57] <mandel> ralsina, nessita: this is waht I mean: http://paste.ubuntu.com/649128/
[13:58] <mandel> ralsina: main idea, ensure that the function has the commutative property
[13:58] <Chipaca> ralsina: OTP
[13:58] <mandel> nessita: look at my comment ^
[14:14] <mandel> nessita: I cannot hear you… :(
[14:14] <mandel> nessita: but I think its my network the problem
[14:15] <nessita> mandel: can you hear ralsina?
[14:15] <mandel> nessita: mumble is done, I'll be back asap
[14:15] <mandel> ralsina, can hear! what do I need to know?
[14:26] <mandel> nessita: 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] <mandel> nessita: did the decorator explanation help you?
[14:28] <nessita> mandel: yes to the decorator explanation
[14:29] <nessita> mandel: 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:30] <mandel> nessita: ok, and merge with trunk and resolve merge issues, ok?
[14:34] <nessita> yes
[14:34] <ralsina> DiegoSarmentero: welcome to Canonical! (just got the official "new starter" email :-)
[14:34] <DiegoSarmentero> ralsina, THANKS!! :D
[14:35] <nessita> DiegoSarmentero: heeeeeeeey! welcome!
[14:35] <fagan> welcome DiegoSarmentero
[14:35] <DiegoSarmentero> nessita, thanks everyone! :D I'm very happy!
[14:35] <nessita> DiegoSarmentero: I have send an email confirming your flight to the agency, we'll have the electronic ticket tomorrow
[14:35] <DiegoSarmentero> nessita, awesome
[14:35] <ralsina> DiegoSarmentero: traé alfajores al sprint
[14:35] <mandel> DiegoSarmentero: bienvenido!!!
[14:36] <DiegoSarmentero> ralsina, will do!
[14:36] <DiegoSarmentero> mandel, gracias!
[14:36] <alecu> DiegoSarmentero, bienvenido!
[14:36] <DiegoSarmentero> alecu, gracias!!
[14:36] <alecu> mandel, ralsina, DiegoSarmentero: please, I need one more review on this branch: https://code.launchpad.net/~alecu/ubuntuone-client/unbreak-connect/+merge/68309
[14:36] <ralsina> And today, canonicaladmin magically looks ok on chromium. yay!
[14:37] <ralsina> alecu: on it!
[14:37] <mandel> nessita, alecu, ralsina: did you get an email from msdn?
[14:37] <ralsina> alecu: I tested it yesterday and forgot to approve it, sorry!
[14:37] <nessita> mandel: yeap
[14:37] <ralsina> I did not, but I have windows licenses already
[14:37] <alecu> mandel, yes!
[14:38] <alecu> ralsina, ok... please, approve it!!!!!!
[14:38] <mandel> superb!
[14:38] <ralsina> alecu: done!
[14:51] <dobey> man, ppa builds are slow today
[14:53] <nessita> ralsina: I'm leaving to sign that thingy at the university
[14:53] <nessita> I'll be back ASAP
[14:53] <ralsina> nessita: ok
[14:53] <ralsina> nessita: have lunch too
[14:53] <nessita> (but it will take one hour approx)
[14:53] <ralsina> just coordinate mandel's handoff...
[14:54] <nessita> mandel: how long are you staying?
[14:54] <mandel> nessita: 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 guys
[14:55] <nessita> mandel: I can be here in approx one hour, depending on the traffic
[14:55] <mandel> nessita: I can take a break and connect back at my 8 for example
[14:55] <nessita> mandel: or I can do this tomorrow...
[14:55] <nessita> let me do some phone calls
[14:56] <mandel> nessita: look dont worry, do your stuff at uni, worst worst case scenario, tom I start at you 9 and we do then the hand off
[14:57] <mandel> nessita: If not, we can do it later 2night, I have no problem what so ever
[14:57] <nessita> mandel: I'm making some phone calls
[14:59] <mandel> nessita: ok, but you seriously don't have to
[15:01] <nessita> mandel: ok, I moved my thing for tomorrow
[15:01] <mandel> nessita: ok
[15:02] <nessita> mandel: let me know as soon as you can make the hand off to me
[15:02] <nessita> mandel: I'm not in a hurry, but maybe is late for ytou
[15:02] <mandel> nessita: sure
[15:03] <mandel> nessita: is not, I started one hour later today, so we are fine :)
[15:03] <nessita> mandel: I imagine, I read you were drinking with 18-years old last night until 4am your time!
[15:03] <mandel> nessita: I'm pushing the changes to fix-os-helper, will run tests on linux and will let you know
[15:03] <nessita> ;-)
[15:03] <nessita> mandel: ack
[15:04] <mandel> nessita: yes, but I sleep an average of 4 hours… and with their conversation, I felt sleep earlier than usual :)
[15:05] <nessita> lol
[15:12] <mandel> so… 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 :P
[15:12] <ralsina> mandel: so are cured meats, sadly
[15:13] <ralsina> BTW: expect marianna's mail soonish, but the hotel will be http://www.hoteljacaranda.com.ar
[15:13] <alecu> mandel, a knife from toledo?
[15:13] <mandel> ralsina: really? that is lame...
[15:13] <nessita> mandel: corn noddles, they are sold in "Al campo", and they are very tasty and gluten free!
[15:13] <DiegoSarmentero> ralsina, it seems nice! :P
[15:13] <mandel> alecu: hahaha I'll be in madrid so no idea i I could :)
[15:13] <mandel> nessita: I can get you those for sure!
[15:14] <alecu> mandel, oh, then perhaps I can ask you for some board game.
[15:14] <nessita> mandel: you sure? only if it's not that much trouble
[15:14] <mandel> alecu: shoot, but dont make it too complicated
[15:14] <nessita> mandel: of course I will pay for them, they cost something like 1.5 euros the 500g the package
[15:14] <mandel> nessita: no trouble at all, and you have to get something out from the reviews, right?
[15:15] <mandel> nessita: but do send me an email with the details, I don't want to bring the wrong stuff
[15:15] <dobey> ralsina: oh nice
[15:15] <ralsina> nessita: 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] <alecu> mandel, 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] <ralsina> nessita: I may go there this saturday and get you amaranth noodles. I always wanted to ask somene to try those ;-)
[15:15] <alecu> mandel, I can trade that for asado
[15:16] <nessita> ralsina: bring them on!
[15:16] <alecu> mandel, I mean, using you as a mule: I'll pay for the game :-)
[15:16] <mandel> alecu: sure, I think I know where to get that...
[15:16] <DiegoSarmentero> alecu, noooo, battlestar galactica is awesome!!!!
[15:16] <DiegoSarmentero> jejje
[15:16] <alecu> cool
[15:17] <alecu> DiegoSarmentero, where did you play it? at toote's?
[15:17] <ralsina> nessita: we went there with rayis last saturday. I'll try to convince the wife to go again :-)
[15:17] <dobey> i am sad i won't be there
[15:17] <DiegoSarmentero> nono...... i didn't play the board game, i mean the tv show........ and........... everything related :P
[15:18] <ralsina> dobey: 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] <DiegoSarmentero> alecu, i answer above... :P
[15:18] <alecu> DiegoSarmentero, hahaha
[15:18] <ralsina> alecu: amanith noodles would be so much fun! :-)
[15:18] <DiegoSarmentero> answered*
[15:20] <alecu> mandel, so, remember "battlestar galactica". That or a leg of "pata negra"
[15:21] <mandel> alecu: 5 J I suppose...
[15:25] <alecu> mandel, right! jajaja
[15:28] <mandel> alecu, nessita, ralsina: please: https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/68530
[15:28] <mandel> DiegoSarmentero, nessita: which airport are you arriving to and when? we might be able to share a taxi...
[15:29] <DiegoSarmentero> nessita, ?
[15:29] <DiegoSarmentero> mandel, i don't know yet
[15:29] <ralsina> mandel: they arrive at the local one, you at the internationa
[15:29] <nessita> mandel: we arrive to Aeroparque, at almost 11pm
[15:29] <nessita> mandel: and you arrive to Ezeiza, at 20pm, right?
[15:29] <ralsina> mandel: looking
[15:29] <mandel> nessita: indeed, so no point to share
[15:30] <nessita> mandel: right
[15:34]  * DiegoSarmentero is away: Not here... send me an e-mail! :P
[15:35] <fagan> mandel: what IRC client do you use on your mac?
[15:35]  * fagan was wondering what is a good one 
[15:35] <mandel> fagan: adium
[15:35] <mandel> fagan: I use it for a set of diff protocols
[15:35] <fagan> nice
[15:35] <fagan> googled but I got bad ones
[15:36] <ralsina> mandel: expect to pay $250 for the car to the hotel (about 45 euros or so)
[15:36] <nessita> mandel: I strongly advice to arrange the price before hiring the service, inside the same airport
[15:36] <nessita> right after the custom control
[15:36] <mandel> ralsina: joder!!! how far is it?
[15:36] <ralsina> nessita, DiegoSarmentero: you can expect to pay about $100 from your airport depending on how much they rob you :-(
[15:36] <ralsina> mandel: 50km
[15:36] <mandel> nessita: yeah, I did learn about that last time :)
[15:36] <nessita> ralsina: right
[15:37] <nessita> ralsina: 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 min
[15:38] <ralsina> nessita: you can call (011) 47435658
[15:38] <ralsina> nessita: that's one from near my home, so not too far from the hotel
[15:38] <nessita> right
[15:38] <ralsina> nessita: I will ask them
[15:41] <ralsina> nessita: $85 but no factura
[15:41] <nessita> ralsina: we need factura... is it more expensive with the receipt?
[15:41] <ralsina> nessita: they can make you an unofficial one, but if they have to do an official one, it's 100 so... no point
[15:43] <nessita> ralsina: ok, is a good reference to negociate there
[15:43] <ralsina> If 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 late
[15:44] <ralsina> that's about half of the way
[15:44] <dobey> yay
[15:44] <dobey> sso nightlies on maverick working again
[15:44] <dobey> ok, time to get some lunch. bbiab
[15:45] <ralsina> same here!
[15:45] <ralsina> see you all later
[15:45] <ralsina> mandel, nessita, alecu: I am finishing the review after lunch (fix-os-helper)
[15:45] <mandel> ralsina_lunch: ack
[15:45] <alecu> ralsina_lunch, cool
[15:47] <alecu> mandel, oh, and remember to say that you are *not* coming to work here :-)
[15:48] <alecu> or you'll have to get a visa.
[15:49] <mandel> alecu: yes, I know… I'm more worried about the diff between the passport pict and my current looks
[15:52] <alecu> mandel, oh, so you've finally went ahead with your sex-change operation?
[15:52] <mandel> alecu: call me mari carmen please
[15:53] <alecu> "le llamaban juana la loca"!
[16:01] <nessita> mandel: you around?
[16:01] <mandel> nessita: around I am
[16:02] <nessita> mandel: so, my head is close to explode. I'm seeing something in your branch that... if I got it right, scares me :-D
[16:02] <nessita> mandel: can you please look the test in
[16:02] <nessita> 75     def test_replace_unicode_with_illegal_chars(self):
[16:02] <nessita> do you see something odd here?
[16:02] <mandel> nessita: what?!
[16:02] <nessita> 84+        original_path = 'test'.join(LINUX_ILLEGAL_CHARS_MAP)
[16:02] <nessita> 85+        path = replace_unicode_with_illegal_chars(original_path)
[16:04] <mandel> nessita: so you get test+ unicode_chars and then yo get test+illeglachars, right?
[16:04] <nessita> mandel: not sure what that means
[16:04] <mandel> nessita: since LINUX_* is a unicode to illegal char map
[16:05] <nessita> mandel: so, in both tests, which tests different stuff, you're building the same original path:
[16:05] <nessita> original_path = 'test'.join(LINUX_ILLEGAL_CHARS_MAP)
[16:05] <nessita> original_path = 'test'.join(LINUX_ILLEGAL_CHARS_MAP)
[16:05] <nessita> those are from diff lines 84 and 97
[16:05] <nessita> mandel: that has no sense, one path should have the windows unicode chars, the other the linux bytes chars
[16:07] <mandel> nessita: joder… que dia… no wonder the tests pass
[16:07] <nessita> mandel: I can take over your branch, maybe that's the best option?
[16:08] <nessita> listo, eso fue todo :-D
[16:08] <nessita> mandel: I can take over your branch, maybe that's the best option?
[16:09] <nessita> mandel: so, maybe you should call it a week already, and just do the hand off of your branches to me
[16:09] <nessita> mandel: I'd like to schedule my lunch, but I want to coordinate the hand off from you first
[16:10] <mandel> nessita: have lunch, I have a coffee to wake up and we handle the branches of, is that ok?
[16:13] <mandel> nessita: just pushed the correct test
[16:14] <nessita> mandel: I'll have lunch then, I'll be back before the next o'clock (ie in less than 45 minutes)
[16:14] <mandel> nessita: I'll have coffee then
[16:29] <nessita> lunchtime!
[16:46]  * alecu will be out as well
[16:47] <mandel> nessita: 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 much
[16:47] <alecu> mandel, will you be pushing more changes to the branch?
[16:48] <alecu> mandel, I'll probably be online in an hour to update that branch and probably mumble with you and nessita.
[16:48] <alecu> mandel, or are you done for the day?
[16:49] <alecu> allright, I'm off for an hour or less.
[16:51] <nessita> mandel: I'm back
[16:54] <dobey> nessita: here's a fine start for you: https://code.launchpad.net/~dobey/ubuntuone-client/less-gnome/+merge/68713
[16:54] <dobey> :)
[16:54] <nessita> alecu: we will be taking over mandel's branches, including the one for review
[16:54] <nessita> dobey: ack
[17:15]  * DiegoSarmentero is back (gone 01:41:41)
[17:18] <ralsina_lunch> nessita: 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:21] <nessita> ralsina_lunch: is not deprecated. What needs fixing should be there that are not?
[17:21] <ralsina_lunch> nessita: 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:22] <nessita> ralsina: ah, yes, mandel pushed that fix before I could comment in the MP
[17:22] <ralsina> And I was not sure if that was already pushed or not (the branch is big enough that it's not obvious :-)
[17:22] <ralsina> ok then, back to reading
[17:23] <nessita> ralsina: let me know how it goes
[17:23] <ralsina> nessita: slow! :-D
[17:23] <mandel> nessita: I'm back, how do you want to do the handover
[17:23] <nessita> mandel: yes please
[17:23] <nessita> mandel: mumble?
[17:23] <ralsina> I keep getting to check the original sources because the diff lacks context, but it's moving
[17:24] <mandel> nessita: though 3G wont be very good… sorry :(
[17:24] <nessita> ralsina: can you join mandel and me, so the modules names start to sound familiar to your ears?
[17:24] <ralsina> sure
[17:24] <nessita> mandel: but just like this morning?
[17:24] <dobey> yay, and sso nightlies building again on lucid too
[17:25] <mandel> nessita:I was missing half of the conversation this morning, give me 5 min and I'll try to use the neighbords wifi
[17:25] <nessita> mandel: nice!
[17:25] <nessita> I'll grab some tea
[17:25] <mandel> nessita: will ping when I'm in a decent connection
[17:25] <mandel> or can use mumble, one of those
[17:27] <dobey> watch out for solar flares
[17:32] <nessita> dobey: is all red! :-D
[17:33] <mandel> nessita: I'm back, neighbor is not there, lets try with 3g
[17:34] <nessita> ack
[17:34] <nessita> mandel: we're there
[17:34] <mandel> ok
[17:38] <dobey> nessita: mostly, yep :)
[17:47] <mandel> nessita: se me oye?
[17:49] <nessita> mandel: sip
[17:50] <nessita> alecu: if you can, we're mumbling right now re: mandel's branches
[17:59] <alecu> nessita, I'm having lunch, so I'll join on listen only mode.
[18:03] <mandel> nessita, ralsina, alecu: http://paste.ubuntu.com/649338/
[18:57]  * alecu runs
[18:58] <ralsina> nessita: which one takes unicode, "replace_unicode_with_illegal_chars" ?
[18:58]  * ralsina is slightly dizzy with this branch
[18:59] <nessita> ralsina: replace_unicode_with_illegal_chars takes unicodes
[19:00] <nessita> as params
[19:00] <nessita> returns bytes
[19:00] <ralsina> okis
[19:12] <ralsina> nessita: then there is something very wrong in test_replace_unicode_with_illegal_chars
[19:12] <ralsina> nessita: we are doing "unicodething in path" where path is bytes. That is always false.
[19:13] <ralsina> oh, wait, the keys a re bytes
[19:13] <nessita> ralsina: 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:14] <nessita> ralsina: LINUX_ILLEGAL_CHARS_MAP's keys are unicodes
[19:14] <nessita> so, 'test'.join(LINUX_ILLEGAL_CHARS_MAP) is unicode
[19:14] <nessita> of course the test legibility will improve if we did
[19:14] <nessita> u'test'.join(LINUX_ILLEGAL_CHARS_MAP)
[19:14] <nessita> and probably picking better names for the mappings
[19:15] <ralsina> nessita: I was about to put a needsfixing about u'test' to be obvious that is unicode
[19:15] <nessita> ralsina: you do that, I'll take over this branch once all the needs fixing are in place
[19:16] <ralsina> but then again line 99 seems wrong, because path is unicode, and the keys in WINDOWS_ILLEGAL_CHARS_MAP are strings
[19:16] <nessita> ralsina: so, I first panicked like you when I saw the latests changes :-D
[19:16] <nessita> ralsina: line 99 is this for me
[19:16] <ralsina> hehe
[19:16] <nessita> 99+        assert not any(c in path for c in WINDOWS_ILLEGAL_CHARS_MAP)
[19:16] <ralsina> yes
[19:16] <nessita> ralsina: you're right, path is unicode
[19:16] <ralsina> that path is the output of replace_illegal_chars_with_unicde, which is unicode, right?
[19:17] <nessita> and  WINDOWS_ILLEGAL_CHARS_MAP's keys are chars
[19:17] <ralsina> it may work, but mostly by accident
[19:17] <nessita> indeed
[19:17] <ralsina> I am not sure what would be a meaningful way to do that though
[19:23] <ralsina> Those 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] <ralsina> I will make another pass, though
[19:24] <ralsina> Of course, the exact same problem happens in the other test. We are comparing a bytes path with unicode keys of LINUX_ILLEGAL_CHARS_MAP
[19:26] <dobey> just allow the lowest common denominator of legal filename characters?
[19:27] <ralsina> dobey: we can't choose the user's filenames :-)
[19:27] <ralsina> I wish we could!
[19:27] <dobey> we can choose what filenames to not work with :)
[19:27] <nessita> dobey: and we need to be able to sync your valid filenames in linux to windows
[19:28] <ralsina> nessita: posted needsfixing on the MP
[19:28] <dobey> nessita: you misunderstood maybe. my suggestion was to reduce the set of supported filenames to something that works on both :)
[19:28] <nessita> ralsina: great! let's wait for alecu's
[19:28] <dobey> so that would be a non-issue
[19:28] <nessita> dobey: ah... oh
[19:28] <nessita> dobey: I think that is a no-no, at least from previous discussions
[19:29] <dobey> i don't see why not. ubuntuone != linux. we are our own filesystem. *shrug*
[19:30] <dobey> don't know why you'd have \ or ? in filenames anyway
[19:30] <dobey> it's not like they work valiantly well in linux
[19:30] <ralsina> dobey: I get that often by using wget to download stuff
[19:31] <ralsina> I get "?" and "&"  pretty often :-)
[19:31] <dobey> yes, well, wget is not the smartest piece of software in the world
[19:31] <nessita> dobey: I have several images called wtf?.png :-)
[19:31] <ralsina> no kidding. It's dumb as a dumb rock
[19:32] <dobey> nessita: stop that. :)
[19:32] <nessita> dobey: but is fun!
[19:32] <dobey> nessita: the ? is redundant there
[19:32] <nessita> dobey: well, I have others called ugly!.png
[19:33] <nessita> so...
[19:33] <dobey> ! is ok on windows i think, so that's not a problem :)
[19:33] <ubot4> dobey: Error: I am only a bot, please don't think I'm intelligent :)
[19:33]  * ralsina still has "ñandu ñoño.png"
[19:33] <dobey> ubot4: trust me, i don't
[19:33] <ubot4> dobey: Error: I am only a bot, please don't think I'm intelligent :)
[19:35] <dobey> i have 我的光碟.zip
[19:35] <ralsina> see? You *must* sync that
[19:36] <ralsina> ok, I am done looking at that branch. My eyeballs are bleeding a bit.
[19:36] <dobey> hrmm
[19:36] <dobey> i just hit the thisfred bug
[19:37] <thisfred> hey!
[19:37] <ralsina> is it the one where you start tagging things as "songs that have thunder sounds in them"?
[19:37] <thisfred> HEY!
[19:37] <dobey> no, the notification bubble showed an old filename
[19:37] <dobey> when i copied the chinese zip file into my u1 folder
[19:38] <dobey> hmm, and it didn't upload
[19:39] <thisfred> Yeah, still no idea what is causing this.
[19:39] <thisfred> I proposed a branch that doesn't fix it though
[19:41] <dobey> oh, it is still WORKING
[19:41] <dobey> but my network usage is nil
[19:41] <dobey> only hits i get are typing in irc
[19:44] <thisfred> this is unlikely to be the notifications' fault ;)
[19:46] <dobey> hrmm
[19:46] <dobey> it is definitely stuck
[19:46] <dobey> but i don't see any evidence of why in the log
[19:46] <dobey> it called MakeFile like 20 times, and there's not even an empty file on the server
[19:54] <thisfred> Lots of MakeFile calls is also something I also saw in facundo's logs when he had the issue
[19:54] <thisfred> I still think there are at least two different problems here
[19:55] <thisfred> Well obviously, but I mean two bugs behind the two problems
[19:56] <dobey> sure
[19:56] <dobey> i just happened to hit both of them :(
[19:56] <thisfred> well, one may be triggering the other
[19:56] <facundobatista> thisfred, when I had which issue?
[19:56] <thisfred> facundobatista:  the wrong filename in the notification
[19:57] <dobey> facundobatista: my syncdaemon is stuck
[19:57] <facundobatista> dobey, debug logs?
[19:57] <ralsina> nessita, 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:58] <dobey> facundobatista: 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:59] <dobey> facundobatista: the filename was chinese, but also valid utf-8, if that matters
[20:00] <nessita> ralsina: FOR SURE!
[20:00] <nessita> ralsina: I need to download some stuff (?)
[20:00] <ralsina> also, from 512Kb to 1.5Mb upload
[20:02] <facundobatista> dobey, and it's stuck doing what?
[20:03] <dobey> facundobatista: 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] <dobey> facundobatista: just kept doing that
[20:03] <dobey> hrmm, though now it won't connect, because apparently ubuntu-sso-client got broken
[20:04] <facundobatista> dobey, what do you mean won't connect? it says it's connected
[20:05] <dobey> facundobatista: 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 reconnect
[20:06] <dobey> and now it is reconnected, since new sso works again
[20:06] <dobey> and now i got a notification with the right filename
[20:07] <dobey> but is still not uploading the file :-/
[20:09] <facundobatista> dobey, that is very strange indeed... can you pass me the logs?
[20:10] <nessita> ralsina: you available for an IU review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/unify-account-and-service/+merge/68743
[20:10] <ralsina> nessita: sure
[20:10] <dobey> hrmm, now it says idle
[20:10] <dobey> and the file appears on the web ui
[20:11] <dobey> i guess it finally uploaded it
[20:11] <dobey> weird
[20:11] <dobey> facundobatista: i must have hit a bug that was fixed already :)
[20:11] <dobey> hmmmmmmmmmm
[20:13] <facundobatista> dobey, the simple magical process of sending me the logs most of the times fixes the problem, that's why I always ask for them :D
[20:14] <dobey> :)
[20:14] <ralsina> nessita: hahaha self.ui.password_lineedit.setText("swordfish")
[20:14] <nessita> ralsina: all credits to alecu :-)
[20:15] <ralsina> too bad you are removing it :-)
[20:20] <ralsina> nessita: +1 on that branch
[20:21] <nessita> thanks!
[20:30] <nessita> dobey: you may know this: I accidentally removed the 'systray' from my panel, and now I can find it within the applet list to restore it
[20:31] <nessita> dobey: 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 sonata
[20:32] <dobey> nessita: notification area
[20:33] <nessita> ah! THANKS!
[20:33]  * nessita hugs sonata back
[20:38] <thisfred> nessita: so you use mpd?
[20:38] <nessita> thisfred: I do. I love it, except when I have to use both mpd and mumble
[20:38] <nessita> they both go crazy
[20:38] <thisfred> heh, well, that would be mumble's fault I'm guessing
[20:39] <nessita> thisfred: pretty likely, mpd only gets broken by mumble
[20:39] <thisfred> I really like mpd too. I used to use it a lot in my previous job
[20:39] <thisfred> where I set up an office radiostation with it
[20:39] <nessita> thisfred: we use it a lot in the 'office' ;-)
[20:40] <thisfred> yeah, it's really cool when anyone can add or skip music
[20:40] <nessita> yeap
[20:40] <nessita> I threat people that skips my choosing of music, but yeah :-P
[20:41] <thisfred> hehe
[20:41] <nessita> thisfred: I bet you're amazed by that ;-)
[20:42] <thisfred> I 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:43] <thisfred> I wonder if there are android clients for mpd
[20:43] <thisfred> yep
[20:53] <dobey> alright. am out. have a good evening all
[20:54] <ralsina> EOD for me, mail me if you need me!
[22:13] <nessita> ok, I'm off for the day
[22:13] <nessita> see ya all tomorrow!