[02:31] <lalejand> Hi, is there a way to flush the my contacts on U1 servers (and on my computer) ?
[09:39] <karni> morning
[09:45] <JamesTait> Happy Friday, everyone!
[10:12] <ralsina> good morning!
[10:24] <karni> ralsina: morning
[10:24] <ralsina> good morning karni!
[10:24] <karni> verterok: has the server bug about fileSize=0 been fixed perhaps? (I don't have a bug number, we talked about it some time ago)
[10:25] <karni> verterok: fileSize in the getDelta items, that is.
[10:26] <verterok> karni: I'm not at home ATM, but I think it's fixed
[10:26] <karni> verterok: ACK!
[10:26] <karni> you guys are up early..
[10:27] <verterok> karni: naaa, I'm running some errands
[10:27] <verterok> :)
[10:28] <karni> :)
[12:09] <duanedesign> morning all
[12:12] <karni> hi duanedesign
[12:19] <duanedesign> hey karni !
[12:27] <jderose> aquarius: you want to skype about ratings and reviews?  i'll be up for next few hours you're free
[12:33] <aquarius> jderose, heya. I got the email. Sure, I'm around
[12:34] <jderose> aquarius: cool... just ring me on skype when your ready
[13:04] <ralsina> What happens if I publish ~/Ubuntu One/Shared With Me/somefile ? I can publish it (get no error) but I can't copy the public URL either.  And it's nowhere in the web UI (because it was not uploaded, I think)
[13:04] <ralsina> notice that somefile is not a share, but something I created.
[13:18] <aquarius> you're not on skype, comrade jderose :)
[13:18] <jderose> aquarius: hmm, actually i am... but i don't see you as online either
[13:19] <aquarius> jderose, says you're offline. Ringing you says you're offline :)
[13:19] <jderose> aquarius: hmmm, does ufw cause problems with skype
[13:20] <aquarius> jderose, nfi. Can you see anyone else online?
[13:20] <jderose> no
[13:20] <aquarius> I can see loads of people. So I thik the problem is at your end :P
[13:20] <jderose> okay... i have a kernel update anyway, perhaps a reboot in case skype is confused... brb
[13:51] <ralsina> mandel dobey thisfred standup in 10'
[13:52] <ralsina> nessita is running errands, so she probably won't make it in time
[13:52] <ralsina> But I have her notes
[13:52] <thisfred> kk
[13:55] <alecu> thisfred, it seems that the libunity error was fixed... but some other parts are broken :P
[13:56] <karni> alecu: is your REST api work published somewhere (lp perhaps) ?
[13:56] <karni> o wait.. was it vds?
[13:56] <ralsina> karni: vds
[13:56] <karni> alecu: sry ;)
[13:56] <thisfred> alecu: ah. There seems to be a new X and linux kernel, so *maybe* nvidia drivers will work again, but probably not
[13:56] <alecu> karni, no problem
[13:56] <vds> karni, hello
[13:56] <karni> vds: is your REST api work published somewhere (lp perhaps) ?
[13:57] <karni> hi :)
[13:57] <alecu> thisfred, are you using the binary drivers or the open source ones?
[13:57] <vds> karni, that's a question for aquarius :)
[13:57] <karni> vds: aha ;)
[13:57] <karni> aquarius: /me wants to see the REST progress :)
[13:59] <aquarius> karni, we're very close to havin the basic API available :)
[13:59] <karni> aquarius: I see
[14:00] <mandel> me
[14:00] <aquarius> karni, it's not yet working, but it nearly is
[14:00] <karni> aquarius: I'm RESTless about handling that SSL connection ;) heheh
[14:00] <aquarius> and I'll be happy to show it to you once we've got the last couple of bits tidied up :P
[14:00] <karni> aquarius: very cool.
[14:01] <ralsina> me
[14:01] <thisfred> me
[14:02] <ralsina> me_as_nessita
[14:02] <alecu> me
[14:02] <ralsina> mandel?
[14:03] <ralsina> alecu, sorry I forgot you on the standup ping!
[14:03] <mandel> DONE: Fixed errors with os_helper branch. Create a XP testing vm while launchpad was done. Refactored IDbusInterface logic so it can be reused on Windows.
[14:03] <mandel> TODO: Refactor test to test code in IDbusInterface without Dbus. Include namedpipe sever for IPC. Create named pipe client for IPC. Chase people to get reviews for file system notifications code.
[14:03] <mandel> BLOCKED: no
[14:03] <mandel> ralsina: please
[14:03] <ralsina> DONE: reviews, canonicaladmin, worked on bug #712674 (yay, I coded!)
[14:03] <ralsina> TODO: reviews, management stuff, sprint planning, triage another 8 tons of bugs
[14:03] <ralsina> BLOCKED: no
[14:03] <ralsina> thisfred!
[14:03] <ubot4> Launchpad bug 712674 in ubuntuone-client (Ubuntu Natty) (and 5 other projects) "Nautilus offers publishing files within shares while it's not allowed on the server (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/712674
[14:04] <thisfred> DONE: u1couch API thinking, libunity trying and failing
[14:04] <thisfred> TODO: u1couch API. Bug #702183, Bug #702116, make UDF notifications aggregate
[14:04] <thisfred> BLOCKED: Unity does not work on my laptop
[14:04] <thisfred> HATED: nvidia drivers
[14:04] <ubot4> Launchpad bug 702183 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to open the control panel in the background and change the launcher icon to urgent when the user exceeds their quota (affects: 1) (heat: 120)" [Medium,Triaged] https://launchpad.net/bugs/702183
[14:04] <ubot4> Launchpad bug 702116 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to show and update a progressbar in the Ubuntu One launcher for pending operations (affects: 1) (heat: 120)" [Medium,Triaged] https://launchpad.net/bugs/702116
[14:04] <dobey> me
[14:04] <thisfred> ralsinessita
[14:04] <ralsina_as_nessi> DONE: worked on bug #715713, but #715804, bug #716678, bug #716499, bug
[14:04] <ralsina_as_nessi> #715776 and bug #716431. Reviews. Weekly call. 'What to do with shares'
[14:04] <ralsina_as_nessi> call.
[14:04] <ralsina_as_nessi> TODO: control panel bug triage, Daniel Fore from design team has
[14:04] <ralsina_as_nessi> reported decens of UI bugs (yey!). Maybe work on some of them. Talk with
[14:04] <ralsina_as_nessi> aquarius to see what I would work on next week (re APIs).
[14:04] <ubot4> Launchpad bug 715713 in ubuntuone-control-panel (Ubuntu) (and 2 other projects) "Place the usage bar label on top of the bar instead of beside it (affects: 1) (heat: 6)" [Low,Triaged] https://launchpad.net/bugs/715713
[14:04] <ralsina_as_nessi> BLOCKED:
[14:04] <ubot4> Launchpad bug 716678 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "Set name to widgets to style properly (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/716678
[14:04] <ubot4> Launchpad bug 716499 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "clicking a folder should open it (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/716499
[14:04] <ralsina_as_nessi> NOTES: Monday I'm not coming. I need reviews for:
[14:04] <ubot4> Launchpad bug 716431 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "Shares to me path looks aweful (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/716431
[14:04] <ralsina_as_nessi> https://code.launchpad.net/~evfool/ubuntuone-control-panel/fix715710/+merge/49290
[14:05] <ralsina_as_nessi> https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/add-widget-names/+merge/49305
[14:05] <ralsina_as_nessi> https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/open-folder/+merge/49287
[14:05] <ralsina_as_nessi> alecu!
[14:05] <alecu> DONE: weekly meeting. went thru android/eclipse sdk hoops, got DroidCouch running on the emulator and on my phone
[14:05] <alecu> TODO: find out how our public couch instance authenticates users. Make DroidCouch use that. Talk with aquarius re: DroidCouch and desktopcouch
[14:05] <alecu> BLOCKED: no
[14:05] <ralsina> damn, that didn't work :-)
[14:05] <ralsina> ok, comments?
[14:05] <ralsina> 1: the sprint is approved!
[14:05] <dobey> λ DONE: finished pth migration in nightlies, bug 669204, helped ralsina with bug 712674
[14:05] <dobey> λ TODO: shotwell plug-in research, finish bug 673012, evaluate SRUs for maverick
[14:05] <ubot4> Launchpad bug 669204 in couchdb-glib "Add Vala bindings to Couchdb-GLib (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/669204
[14:05] <ralsina> oops, sorry dobey
[14:05] <ubot4> Launchpad bug 712674 in ubuntuone-client (Ubuntu Natty) (and 5 other projects) "Nautilus offers publishing files within shares while it's not allowed on the server (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/712674
[14:05] <thisfred> I'll do the reviews
[14:05] <dobey> λ BLCK: None.
[14:05] <ubot4> Launchpad bug 673012 in desktopcouch (Ubuntu) (and 1 other project) "Async monitoring of _changes (affects: 1) (heat: 43)" [Undecided,Confirmed] https://launchpad.net/bugs/673012
[14:05] <mandel> I need a couple of reviews for windows done from linux, so please take a look
[14:06] <mandel> thisfred: ^ please..
[14:06] <mandel> :D
[14:06] <ralsina> Anyone that wants to laugh at my C, please review https://code.launchpad.net/~ralsina/ubuntuone-client/fix_712674/+merge/49332
[14:06] <thisfred> mandel: will do
[14:06] <mandel> thisfred: thx! just look at the code and run on linux
[14:06] <ralsina> And I repeat: THE SPRINT IS APPROVED ;-)
[14:07] <thisfred> ralsina: so we should book flights?
[14:07] <alecu> ralsina, where?
[14:07] <ralsina> thisfred alecu: the place is not approved
[14:07] <thisfred> ah
[14:07] <alecu> :-)
[14:07] <ralsina> Technically, it should be in western europe
[14:07] <mandel> Spain!
[14:07] <ralsina> Because 5 attendants are from there
[14:07] <thisfred> oh, the netherlands would be good :)
[14:08] <ralsina> But it may still be Buenos Aires.
[14:08] <dobey> thisfred: it is supposed to be where they live, not any random place
[14:08] <ralsina> the problem is that 3 of them are from London, and London is apparently crazy expensive.
[14:08]  * mandel seraches for a vide of fready mercury screamming barcelona
[14:09] <dobey> 5? chipaca, vds, mandel. who else?
[14:09] <ralsina> dobey: 2 UX people
[14:09] <mandel> we have 2UG people?
[14:09] <dobey> well, 1 GBP is like 1.8x USD
[14:09] <ralsina> mandel: for the sprint, yes
[14:09] <ralsina> At least that'sthe plan
[14:09] <mandel> uh, cool
[14:10] <jderose> aquarius: just sent you a happy friday email :)
[14:10] <alecu> ralsina, I just realized that having 2 new UX people would be no help after the ui freeze.
[14:10]  * karni short lunch break
[14:10] <ralsina> alecu: good point!
[14:10] <thisfred> I'm still secretly hoping for BA in which case I will take a week of vacation there after the sprint with my wige
[14:10] <thisfred> wife
[14:10] <thisfred> even
[14:11] <ralsina> thisfred: I will even make arrangements for your vacations if it's here ;-)
[14:11] <ralsina> thisfred: I can fix you up with a week in a typical farm picking peppers. It pays about U$S 9 a day, too! ;-)
[14:12] <alecu> ralsina, March 24th "UserInterfaceFreeze"
[14:12] <ralsina> alecu: right, so probably no UX people. I wonder why Chipaca told me that then.
[14:12] <ralsina> So, if no UX people, it's Argentina
[14:13] <thisfred> haha, that might not go over too well.
[14:13] <Chipaca> ralsina: me? sidetrack you? nevah
[14:13] <ralsina> thisfred: all the peppers you can eat!
[14:13] <ralsina> Chipaca: hey, I am just confused :-)
[14:15] <nessita> hello everyone
[14:15] <nessita> did you eom already?
[14:15] <thisfred> Holanessita!
[14:15] <ralsina> hello again nessita!
[14:15] <ralsina> almost eom
[14:16] <ralsina> nessita: sprint is approved
[14:16] <nessita> yey!
[14:16] <nessita> where is it?
[14:16] <ralsina> and you have a golden chance to laugh at me by reviewing https://code.launchpad.net/~ralsina/ubuntuone-client/fix_712674/+merge/49332
[14:16] <Chipaca> nessita: belgrade
[14:16] <thisfred> beirut?
[14:16] <ralsina> nessita: the BAD side of belgrade
[14:16] <ralsina> thisfred: beirut is actually a lovely place when there's no war going on.
[14:17] <thisfred> I hear Egypt's nice this time of year :)
[14:17] <Chipaca> ralsina: wah, *two* needfixings
[14:17] <Chipaca> ralsina: enough to drive you to qt, i tell you
[14:17] <thisfred> ralsina: Yeah I totally want to go there
[14:17] <ralsina> Chipaca: I never claimed to be good at this programming thing
[14:17] <nessita> ralsina: reviewing it
[14:17] <thisfred> Dammit Jim, I'm a doctor not a C programmer
[14:17] <Chipaca> ralsina: that is true. Neither did I; I said you were prolific.
[14:17] <nessita> Chipaca: you already home? wow, public transport suck :(
[14:18] <ralsina> Chipaca: these are my slowest 10 lines ever
[14:18] <ralsina> Then again C is not a real language, it can't even add one string to another wthout an argument ;-)
[14:18] <Chipaca> nessita: I drove my anger out. Now I'm tranquil as a bidimensionalized dog.
[14:18] <nessita> Chipaca: breathe in, breathe out
[14:19] <ralsina> dobey: I see you don't appreciate my love of suymmetry
[14:19] <nessita> ralsina: so, shall be book tickets to Serbia then?
[14:19] <nessita> shall we*
[14:19] <Chipaca> nessita: animal bidimensionalization is great therapy
[14:19] <alecu> Chipaca, nessita: let me guess... you had a mobile company morning.
[14:19] <ralsina> nessita: AFAIK the place is not official yet.
[14:20] <nessita> Chipaca: I shall do the same, I'm still trying to choke someonw
[14:20] <ralsina> Chipaca: were you suggested to install the phones into a bodily cavity?
[14:20] <dobey> belgrade's not too bad, just don't speak albanian
[14:20]  * ralsina speaks albanian like a native! Like a native argentinian!
[14:21] <dobey> so lets not do belgrade
[14:21] <nessita> ralsina: you have 2 needs fixing already, you sure you want a potential third?
[14:21] <ralsina> ok, eom for the standup.
[14:21] <thisfred> mandel: what branch did you need review for?
[14:21] <ralsina> nessita: hahaha
[14:21] <ralsina> nessita: I already fixed rye's so I am down to one now!
[14:21] <nessita> heheh
[14:22] <nessita> did you all see my hunger of reviews?
[14:22] <rye> belgrade?
[14:22] <dobey> no
[14:22] <mandel> thisfred: this is one: https://code.launchpad.net/~mandel/ubuntuone-client/windows_reuse_pyinotify/+merge/49062
[14:22] <thisfred> thx
[14:23] <dobey> is called a joke ppls
[14:23] <mandel> thisfred: and if you have time, this too: https://code.launchpad.net/~mandel/ubuntuone-client/implement_notifier/+merge/49064
[14:24] <thisfred> sure\
[14:24] <mandel> thisfred: I'm sorry for the merge size, is kind of hard to keep them small yet meaningful
[14:24] <mandel> specially due to the tests…
[14:25] <thisfred> Sometimes it's just not possible
[14:25] <thisfred> I'll take a fat branch with lots of tests over no tests
[14:26] <nessita> can I have a trivial review for https://code.launchpad.net/~evfool/ubuntuone-control-panel/fix715710/+merge/49290 ?
[14:28] <ralsina> dobey: re-review please?
[14:28] <ralsina> rye: re-review of https://code.launchpad.net/~ralsina/ubuntuone-client/fix_712674/+merge/49332 please?
[14:30] <ralsina> mandel: got the invite for the monday call?
[14:31] <mandel> ralsina: let me check
[14:32] <mandel> ralsina: yes
[14:32] <ralsina> ok
[14:35] <Chipaca> ok, so not belgrade. Huahine?
[14:36] <dobey> Pristinĕ
[14:36] <dobey> err, wrong accent, but eh
[14:42] <ralsina> I am kinda against going to spain because of bad experiences on the airport by friends. Basically, they would rather go through groping on USA than the mistreatment they got at Barajas again.
[14:43] <dobey> i had no problems in barcelona
[14:44] <ralsina> dobey: you are not from Argentina
[14:45] <dobey> but we should do orlando or buenos aires anyway. we don't have 5 people in spain
[15:00] <ralsina> dobey: g_build_filename doesn't add a trailing "/" unless I put it in the last element, right?
[15:00] <dobey> ralsina: right, that's why i put the "" there before the NULL
[15:00] <ralsina> dobey: sadly, gi.repository.GLib has only build_filanemv so I can't  test it quickly
[15:00] <ralsina> dobey: that is ignored, according to the doc
[15:01] <dobey> ralsina: the "" is? or the NULL is?
[15:01] <ralsina> dobey: both?
[15:01] <ralsina> dobey: "The number of trailing copies of the separator on the result is the same as the number of trailing copies of the separator on the last non-empty element."
[15:02] <ralsina> That is for g_build_path and g_build_filename is supposed to "act like g_build_path with the right separator"
[15:03] <dobey> well you can replace the "" in my example with G_PATH_SEPARATOR probably
[15:03] <dobey> or whatever the correct macro name is
[15:03] <ralsina> Or just add a trailing /. This code is unix only anyway
[15:03] <dobey> no, use the macro
[15:04] <ralsina> ok
[15:04] <dobey> just becuase *we* aren't building it on windows, doesn't mean it can't be :)
[15:04] <ralsina> dobey: this is a nautilus plugins. I can count the number of windows nautilus users with no hads :-)
[15:07] <dobey> yes, well, you are a kde person
[15:07] <dobey> so i can see how you are biased
[15:09] <CardinalFang> These distro updates are a pain.
[15:10] <aquarius> jderose, ping
[15:10] <jderose> aquarius: pong
[15:10] <aquarius> jderose, want some code to log into a website with Ubuntu One?
[15:10] <jderose> aquarius: that would be awesome to experiment with, yes please!
[15:11] <aquarius> jderose, http://paste.ubuntu.com/565904/
[15:11] <aquarius> jderose, implemented as very, very, very stupid Python CGIs.
[15:11] <jderose> shows how APIs work, that's all i need. thanks!
[15:12] <jderose> aquarius: so do i need any kind of application key or whatever to try this, or will that basically work as is?
[15:21] <rye> erm, ralsina should the latest version have "Share..." in ~/.local/share/ubuntuone/syncdaemon ?
[15:21] <ralsina> rye: no
[15:21] <ralsina> it does?
[15:21] <ralsina> Oh, wait, this is not about Share, it's about Publish
[15:22] <ralsina> If there is a problem showing "Share" in the wrong place it's another bug :-)
[15:22] <rye> erm
[15:22] <dobey> woah
[15:22] <rye> ralsina, ok, re: publish: it works fine
[15:23] <ralsina> dobey, I think I addressed all your comments in the new revision. Sorry to waste so much of your time, I am a newbie at gnomeish development ;-)
[15:27] <dobey> hrmm
[15:27] <dobey> what happened to the internets today
[15:27] <dobey> people are trying to depose the connectivity it seems
[15:27] <ralsina> dobey: one more commit because I should have used g_free instead of free for g_build_path result.
[15:28] <dobey> ralsina: you should always use g_free () in glib apps, because g_free (NULL); doesn't segfault like free (NULL); does
[15:28] <ralsina> g_build_filename*
[15:28] <dobey> or at least, it shouldn't
[15:29] <ralsina> dobey: but I have to use free for the result of realpath
[15:29] <ralsina> "It's important to match g_malloc() with g_free() , plain malloc() with free()" according to the g_free docs :-)
[15:29] <dobey> ralsina: you should use g_free ()
[15:29] <dobey> *shrug*
[15:30] <dobey> glib really should have a g_path_resolve () function, but alas
[15:30] <ralsina> At least it says that "Otherwise bad things can happen, since these allocators may use different memory pools"
[15:31] <dobey> maybe since the glib switch to using GSlice for everything
[15:31] <dobey> you definitely shouldn't use free () on g_malloced things
[15:31] <ralsina> dobey: right, that's why I fixed that
[15:55] <dobey> rye: can you re-review ralsina's branch please?
[15:56] <ralsina> dobey: what, no needs fixing anymore? Have I SUCCEEDED????
[15:58] <nessita> ralsina: can I have your eyes on https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/add-widget-names/+merge/49305?
[15:58] <ralsina> nessita: shipping eyes to said address
[15:58] <nessita> yey!
[16:00] <ralsina> nessita: is a code review good enough for this one? I am in the midle of a 260MB update and the internet feels sick.
[16:01] <nessita> ralsina: yes
[16:01] <ralsina> nessita: ok
[16:01] <nessita> those a 'trivial' fixes
[16:02] <ralsina> nessita: +1
[16:02] <nessita> ralsina: thanks!
[16:32] <CardinalFang> Wow, nice job, Egypt.
[16:34] <dobey> CardinalFang: did they annuonce a partnering with Microsoft too?
[16:47] <CardinalFang> dobey, They haven't picked what's next, just decided to abandon the old platform.
[16:53] <dobey> lunch time :)
[17:03] <ralsina> What is this dialog supposed to do? Honest question! http://ubuntuone.com/p/clw/
[17:14] <karni> facundobatista: hi! sorry to jump at you already. could you tell me why here (Download ActionQueueCommand method) http://paste.ubuntu.com/565957/ we iterate over both Download *and* Upload ?
[17:14] <karni> facundobatista: I assume it's probably some marginal case.
[17:15] <facundobatista> karni, yes, it's marginal, but there's no point in be uploading a file if you're downloading it
[17:15] <facundobatista> and viceversa
[17:15] <karni> facundobatista: so it's kind of precaution, right?
[17:15] <facundobatista> yeap
[17:16] <karni> facundobatista: thanks
[17:17] <rye_> ralsina, 671 revid:tarmac-20100902185055-41zos28bp0g2dr3z - Fix dbus interface get_metadata methods to use the realpath.
[17:17] <ralsina> rye_: that would have fixed that bug, yes
[17:17] <rye_> ralsina, so i am reassigning this to verterok and marking as fix released
[17:17] <ralsina> rye_: cool
[17:18] <rye_> bzr-explorer is a nice thing
[17:18] <gord> hi all, just wondering if i were to shut ubuntu one down, copy over a directory of files someone has shared to me to another drive and sym link it back over, would ubuntu one accept that or freak out and start re-downloading everything?
[17:20] <rye_> yay!
[17:20] <rye_> gord, based on my previous tests it would download the files again
[17:21] <gord> oh that's not fun :(
[17:29] <rye_> gord, what ubuntuone client version are you running so that i could test it with 100% confidence?
[17:29] <gord> rye_, Installed: 1.5.3-0ubuntu3
[17:29] <rye_> gord, ok, and OS ?
[17:29] <karni> facundobatista: sorry to bother. how does def _start(self): of Download command relate to def start(self): of it's super type (ActionQueueCommand). I suppose former is private, latter public. but do they have anything in common?
[17:30] <gord> rye_, natty, fairly up-to date
[17:30] <rye_> gord, okay, let me see...
[17:31] <karni> double underscore is private.. so this one is not. that's even more confusing.
[17:35] <facundobatista> karni, ActionQueueCommand does not have a start anymore
[17:36] <karni> facundobatista: I just saw it's _start (not start), my bad. was it removed in some newer revision.. ?
[17:37] <karni> facundobatista: I'm looking at unleash-the-queues-6 (which may be already old..)
[17:40] <facundobatista> karni, mmm... in unleash-the-queues-6 there's no "start" in AQC
[17:40] <facundobatista> karni, I'm not following you
[17:40] <karni> facundobatista: ... o_O I'm confused, let me check..
[17:41] <karni> facundobatista: rev 830, action_queue.py line 1160 - am I missing something?
[17:42] <karni> facundobatista: and I mean _start, like I mentioned ↑
[17:42] <karni> facundobatista: does the Download command override the AQC _start method?
[17:43] <facundobatista> karni, ahhhhhhhhhhh, so you want to know the relation of Download._start with ACQ._start, ok, *now* I get you
[17:43] <karni> facundobatista: I'm sorry, yes..
[17:43] <facundobatista> karni, the _start method is for commands that want to do something *once* in their lifecycle, before they're actually run
[17:43] <facundobatista> karni, ACQ just provides a default that does not do anything, so you don't need to define it in every method
[17:44] <karni> makes sense, like acquiring the semaphore
[17:44] <facundobatista> karni, :)
[17:51] <karni> facundobatista: thank you :')
[17:51] <karni> If __means a private method, what do you guys use the _for ? I'm sorry to ask lame Python questions.
[17:51] <karni> Java is seriously syntactically impaired.
[17:52] <ralsina> karni: _ is for "please don't use this" :-)
[17:52] <karni> ralsina: seriously.. ?
[17:52] <ralsina> karni: __ is for "you can't use this" :-)
[17:53] <karni> ralsina: ack :)
[17:53] <beuno> karni, right, so a non-enforced private method
[17:53] <karni> I should be reading Python manuals instead of sleeping ;<
[17:53] <ralsina> karni: one of the python slogans is "we are all consenting adults here"
[17:53] <karni> ralsina: hahahah :)
[17:53] <beuno> karni, I think sleeping is probably more valuable in the long term
[17:53] <ralsina> In fact, you can easily use the __foo methods if you really want to.
[17:54] <karni> beuno: I'm getting much sleep (especially now that classes have finished). but I feel impaired without knowing those nitty gritty details/tips about Python syntax
[17:54] <karni> if only day could have 36 hours
[17:54] <beuno> karni, this is how you learn
[17:54] <beuno> asking!
[17:55] <karni> I feel I ask way to many questions here :F
[17:55] <beuno> karni, no such thing as too many question
[17:55] <karni> One thing I know. In few monts time, Python will be my languange of choice.
[17:56] <beuno> that's the one true path
[17:58] <rye_> facundobatista, so, when i tried putting the same files online with syncdaemon shut down and copy the same files to the same syncdaemon dirs upon startup syncdaemon marked local files as u1conflicts and started downloading them from the server, why?
[17:58] <rye_> the same files have the same hashes and they are basically the same
[17:58] <rye_> gord, basically that's the answer for now re move of the files ^
[17:59] <rye_> ralsina, ______method means it will immediately abort the execution?
[17:59] <ralsina> rye_: and kill the calling object, yes
[17:59] <gord> rye_, okay then, thats a shame, a friend of mine with a large share is starting to eat up a sizeable amount of storage on my home partition (small fast ssd) - thanks for testing for me :)
[18:00] <____rye> gord, that's not how it should be, i will stalk facundobatista with this
[18:01] <facundobatista> ____rye, #711389
[18:06] <alecu> nessita, ping
[18:06] <nessita> alecu: pong
[18:06] <alecu> nessita, got a minute? I'm trying to think the dbus interface for enabling/disabling notifications, to make it extensible to support per type enabling/disabling
[18:07] <alecu> nessita, but only support "all" right now.
[18:07] <alecu> nessita, I was thinking of using a dict
[18:07] <alecu> dict of string:bool
[18:07] <nessita> alecu: hum.Why not a dedicated method for each?
[18:07] <alecu> nessita, enable_notifications: {"all": True}
[18:08] <alecu> nessita, or {"file_sync": False, "out_of_space_errors": True}
[18:08] <nessita> alecu: I think the bets way is to have a call per setting
[18:09] <nessita> enable_file_sync_notifications
[18:09] <nessita> enable_out_of_space_notifications
[18:09] <nessita> etc
[18:09] <alecu> nessita, all that *plus* enable_all_notifications ?
[18:09] <ralsina> nessita: with a dict, enable_all_notifications is simpler
[18:09] <nessita> nopes, without the _all_
[18:10] <ralsina> oh, without the all
[18:10] <nessita> ralsina: is less pythonic, from my POV
[18:10] <ralsina> nessita: a list of methods? :-)
[18:10] <nessita> once I read about that, I would like to find the link
[18:10] <alecu> nessita, it's easier to add notification groups by adding to that dict.
[18:10] <alecu> nessita, we are talking dbus, not python :-)
[18:11] <nessita> ralsina, alecu, having set_flag(boolean) is less recommended that set_fla()g + unset_flag()
[18:11] <nessita> with proper typing, of course
[18:11] <nessita> :-)
[18:11] <ralsina> I'd say it depends on how many methods we are talking about, and how often a new one could appear
[18:12] <nessita> alecu: since dbus has such a strict API (can't be changed), I think dedicated methods is easier to maintain
[18:12] <nessita> ralsina, alecu: I understand your point. I will not strongly fight for dedicated methods :-)
[18:12] <nessita> if you both think is best to have a dict, let's. But this is inconsistent with what we talked about file sync status notifications
[18:12] <ralsina> nessita: think of how much longer the doc you will write for dedicated methods will be ;-)
[18:13] <ralsina> j/k
[18:13] <nessita> ralsina: good point (to my favor). Adding an isolated entry for a new method is easier that extending an existent entry
[18:13] <nessita> mainly becasue the new entry is very likely not to be updated ;-)
[18:13] <ralsina> nessita: hahaha
[18:13] <nessita> old entry, I meant :-)
[18:14] <ralsina> nessita: can I ask a question about something you did in a branch that's already landed?
[18:14] <alecu> nessita, "what we talked about file sync status notifications": can you point me at that?
[18:14] <alecu> hmmm....
[18:14] <nessita> I think we're abusing dicts in dbus to have flexible APIs (maybe we have no other choice)
[18:14] <nessita> alecu: when I implemented FileSyncStatusSyncing, FileSyncStatusDisconnected, etc
[18:15] <nessita> alecu: you wanted a single dict with the status
[18:15] <nessita> ralsina: shoot
[18:15] <alecu> nessita, right.
[18:15] <ralsina> In https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/improve-share-name/+merge/49270
[18:15] <ralsina> Why not use os.path.realpath instead of looking up two directory names via dbus and replacing?
[18:16] <ralsina> Oh, forget it, you are going the other way around, of course :-)
[18:16] <nessita> ralsina: exactly
[18:16]  * ralsina often understands when he asks things
[18:16] <alecu> nessita, ok: I'll be only adding "enable_all_notifications" right now (and disable_), and get_all_notifications_shown. We will add more methods later.
[18:16] <nessita> I have no idea (in the control panel) where Shared With Me is linked to
[18:17] <nessita> alecu: ok, like I said, I'm not feeling particularly strong about this :-)
[18:17] <nessita> alecu: I still find discomfort when defining APIs in dbus...
[18:18] <alecu> :-)
[18:18] <ralsina> nessita: well, it's linked to os.path.realpath(whatever/Shared With Me)
[18:18] <nessita> alecu: another point (that I thought but didn't mentioned), is that already existent syncdaemon dbus API has dedicated methods for each setting
[18:19] <nessita> ralsina: right, but at first I don't know what 'whatever' is
[18:19] <nessita> it can be anything
[18:19] <ralsina> it's shares_dir_link?
[18:19] <ralsina> But anyway, it was just idle curiosity :-)
[18:19] <nessita> :-)
[18:19] <nessita> ralsina: do some work if you're IDLE!!! :-D
[18:19] <ralsina> I am procrastinating because I don't want to patch nautilus on stable-1.4
[18:19] <alecu> nessita, right. But this will be a group of settings, so I thought it made sense to group them. I'm thinking of the empathy screenshot that ry-ee posted
[18:20] <nessita> alecu: right
[18:20] <nessita> ralsina: do patch! we need you. Be a good boy/boss
[18:20] <ralsina> 1.4 is maverick or lucid?
[18:20] <alecu> anyway: I'll go with the simplest route for now, and that's the one "all" method
[18:20] <ralsina> Because it sure as hell doesn't build on natty
[18:21] <alecu> nessita, and also I see that all sd settings are like you said (enable_share_autosubscribe + disable_share_autosubscribe + share_autosubscribe_enabled)
[18:21] <alecu> nessita, so I'll try to follow the same pattern.
[18:21] <nessita> ralsina: maverick
[18:22] <ralsina> alecu: generate them from a dict ;-)
[18:22] <nessita> alecu: sounds like a plan. Thanks for asking opinions on this!
[18:22] <alecu> nessita, well... not all.
[18:22] <nessita> alecu: not all?
[18:23] <alecu> nessita, for instance: enable_share_autosubscribe has separarated enable and disable, but set_autoconnect_enabled has a boolean arg
[18:23] <alecu> so :P
[18:24] <alecu> 3 flags are done one way, 2 the other.
[18:25] <alecu> I'll go the first route anyway.
[18:25] <nessita> alecu: ack
[18:29] <thisfred> alecu: nessita very simple branch that adds the icons https://code.launchpad.net/~thisfred/ubuntuone-client/lp-717311/+merge/49443
[18:29] <nessita> thisfred: YEYEY
[18:30] <nessita> ok, I need to switch back to classic desktop, unity desktop is making me (more) crazy
[18:33] <dobey> heck, i switched back to metacity :(
[18:34] <thisfred> heck, I'm on xmonads :)
[18:34] <thisfred> well, not on my naty machine. That one just doesn't work at all
[18:35] <alecu> thisfred, I'm testing your branch by running "PYTHONPATH=. ./bin/ubuntuone-syncdaemon", but I still can't see the icon on notifications.
[18:35] <dobey> well, you're weird
[18:35] <thisfred> so? :)
[18:35] <thisfred> alecu: hmmm. I tested by commenting out the .patch() lines in the test_aggregator tests, and that worked...
[18:36] <thisfred> only the notification patch line that is
[18:36] <alecu> ok, I'll try that.
[18:38] <alecu> thisfred, that's not working for me either :-( I get the notifications, but no icons.
[18:39] <thisfred> alecu: bah. I only tested on maverick. I hope they haven't changed the API
[18:39] <thisfred> testing on natty
[18:39] <alecu> thisfred, I was missing the ubuntuone-client-gnome package. Let's see now.
[18:39] <thisfred> ah
[18:39] <thisfred> that would have the icon I guess
[18:40] <alecu> thisfred, that did it :-)
[18:40] <thisfred> yay!
[18:41] <alecu> thisfred, somehow the u1 messaging menu entry has disappeared.
[18:41] <alecu> thisfred, this happened since today, I believe.
[18:41] <thisfred> hmmm
[18:42] <thisfred> that's weird. Maybe someone removed the link to the .desktop file again?
[18:43] <dobey> alecu: did you uninstall something?
[18:44] <thisfred> dobey: I also don't have it
[18:45] <alecu> dobey, I did a bit of dbusmenu uninstalling yesterday to try to fix the gir issue
[18:45] <dobey> thisfred: i thought we just declared that you're weird :)
[18:45] <alecu> dobey, but that was solved today
[18:45] <nessita> ok, I'm back
[18:45] <thisfred> dobey: doesn't mean I don't deserve love and attention :)
[18:46] <dobey> doh, this whole internets having issues thing is annoying
[18:47] <alecu> dobey, will ubuntuone-client-gnome be installed by default on Unity desktops?
[18:48] <dobey> alecu: it is installed by default on ubuntu, yes
[18:48] <dobey> why?
[18:48] <dobey> mandel: lint issues
[18:50] <thisfred> alecu: for me ls /usr/share/indicators/messages/applications/ does not show ubuntuone-control-panel-gtk anymore
[18:50] <nessita> thisfred: for me neither :-(
[18:51]  * thisfred looks at the make file
[18:51] <nessita> dobey: did we loose the .desktop file in the messaging on nightlies?
[18:51] <alecu> dobey, cool. I was worried it would not be installed by default, and only ubuntuone-client
[18:52] <nessita> thisfred: the ICON is there! yey! /me approved
[18:52] <dobey> nessita: i didn't remove it. did we never add it?
[18:53] <nessita> dobey: I think we did...
[18:53] <dobey> maybe not in nightlies?
[18:53] <thisfred> dobey: not sure, I think so, since it worked for a while
[18:53] <nessita> dobey: maybe. Could you please confirm and/or fix, when you have a slot?
[18:53] <dobey> thisfred: for a while narwhal package was newer than nightlies because i didn't realize i needed to bump the version
[18:53] <dobey> nessita: can you make an ubuntuone-control-panel release/upload for narwhal?
[18:54] <nessita> dobey: yes, as soon as I land a branch I'm coding
[18:54] <nessita> dobey: which is needed for design people
[18:54] <dobey> ok
[18:55] <alecu> thisfred, two approvals :-)
[18:55] <thisfred> awesome
[18:57] <ralsina> my first full crash in a week :-(
[18:57] <alecu> ralsina, how so?
[18:57] <alecu> ralsina, testing windows?
[18:58] <ralsina> well, the screen went black, everything stopped and the HD ligth became a laserbeam :-)
[18:58] <ralsina> alecu: nope, never had a windows crash on this notebook
[18:58] <alecu> :-)
[18:58] <alecu> a new gir1.2-unity-3.0 .... let's try it!
[19:02] <mandel> dobey: i've seen them.. i keep using make test instead of make check… im stupid
[19:03] <dobey> heh, ok
[19:05] <dobey> mandel, thisfred, alecu: any branches that are must-have for a u1client release?
[19:05] <thisfred> dobey: the icons are a nice to have
[19:05] <mandel> dobey: from my side, they are all windows related, so not really ;)
[19:06] <alecu> thisfred, we should subscribe to: https://bugs.launchpad.net/libunity/+bug/709240
[19:06] <ubot4> Launchpad bug 709240 in unity (and 1 other project) "libunity support gobject-introspected languages (affects: 3) (heat: 18)" [Medium,Triaged]
[19:06] <thisfred> alecu, thx done!
[19:13] <dobey> mandel: yeah. the question for your branches is do they break linux :)
[19:13] <mandel> dobey: they should not, but in those parts where we do not have unit tests is hard to say….
[19:14] <mandel> dobey: but you know what they same about braking eggs to make omelets :P
[19:17] <dobey> heh
[19:20] <thisfred> or monkeys to make rugby
[19:23] <dobey> it's the infinite rugby you have to watch out for
[19:28] <thisfred> Is that the one where the monkeys have typewriters?
[19:28] <dobey> yes
[19:40] <mandel> hehe
[19:46] <mandel> he, some of the tests have very interesting names: 'test_deep_and_wide'
[19:46] <mandel> I think i've heard about a movie with that name
[19:47] <ralsina> mandel: one about alien abductions? ;-)
[19:48] <mandel> ralsina: it was certainly alien hehe
[19:48] <dobey> mandel's movie collection is NSFW
[19:52] <dobey> https://code.launchpad.net/~dobey/ubuntuone-client/fix-distcheck/+merge/49455
[19:52] <dobey> can i get 2 quick reviews on that?
[19:57] <thisfred> dobey:  you have 1
[19:58] <mandel> dobey: on it
[19:59] <mandel> dobey: +1
[20:00] <duanedesign> wow, someone has been busy with bugs today :)
[20:01] <mandel> duanedesign: i though ralsina was going throw them…
[20:01] <duanedesign> nice
[20:01] <dobey> great
[20:01] <ralsina> mandel: not this week, I got snookered into hacking C and other similarly unpleasant things. By you, in some case ;-)
[20:03] <mandel> ralsina: i really though you liked the test_deep_and_wide, will not happen again...
[20:03] <mandel> hehe I've got a problem
[20:03] <ralsina> mandel: yes you do :-)
[20:05] <mandel> ralsina: but it was a good one :)
[20:06] <ralsina> oh, yes, I am all for testing jokes ;-)
[20:09] <dobey> test_joke_is_funny
[20:10] <mandel> dobey: I fixed the lint issues, do you have any problem with the release if i approve the branches, should I wait?
[20:11] <dobey> mandel: lets wait until after i do the release
[20:11] <mandel> dobey: sure, I'll approve them on monday then :)
[20:33] <nessita> dobey: will the branch from thisfred get in?
[20:34] <nessita> the one that adds the icon name to the notifs
[20:34] <dobey> it's already in
[20:35] <nessita> awesome
[20:37] <nessita> dobey: do I need to do some special magic with the newly added .pth file when building the new u1cp package?
[20:37] <dobey> oh
[20:37] <dobey> nessita: you need to change the .install files a bit
[20:37] <nessita> define: a bit
[20:38] <dobey> http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-control-panel/packaging-dailies/revision/8
[20:38] <dobey> like that
[20:39] <nessita> ack
[21:18]  * alecu hates default values in ubuntuone/syncdaemon/config.py
[21:27] <ralsina> EOD for me. Have a nice weekend!
[21:35] <karni> facundobatista: what's AQ_DOWNLOAD_COMMIT :) ? what does Nanny do, foo.partial -> foo ?
[21:37] <karni> facundobatista: wait... it had something to do "when the systems ready", but I can't recall the details :<
[21:50] <facundobatista> karni, it's about not finishing the download when you have a file open, otherwise file editors may go crazy if you touch them the file underneath
[21:51] <karni> facundobatista: aha! gracias
[21:51] <facundobatista> karni, so the download ending message is captured by the DownloadNanny, who holds it if the file is open and releases it when it's closed
[21:51] <facundobatista> karni, de nada! :)
[21:51] <karni> facundobatista: perfect :) thanks!
[22:05] <dobey> ok, my day is done
[22:05] <dobey> later all!
[22:38] <alecu> bye all!
[23:55] <karni> night guys