[00:26] * nessita -> eod'd [01:17] hi, dobey [09:46] *yawn* [09:46] morning all [10:38] morning! [10:38] So to speak. [11:51] * fagan break [11:54] rye: ping [11:54] duanedesign, pong [11:54] hello rye ! [11:54] duanedesign, hello! [11:55] rye: umm. had a post i have been putting off answering...because i have never downgrdaded an account so I am unsure of the expected behaviour [11:55] duanedesign, by downgrading you mean? [11:56] rye: so if you had a 20 GB pack and where downgrading to the free 2 GB account [11:56] duanedesign, the data will stay intact, if your usage is more than 2Gb then it will simply prevent you from uploading more [11:57] ok, great. that is exactly what I needed to know [11:57] rye: thank you much [12:25] * mandel walking dog [12:39] hi duanedesign [12:39] hi rye [12:41] hello everyone! [13:02] o/ [13:04] hello nessita! [13:04] hola ralsina! [13:04] and duanedesign [13:05] :) [13:07] nessita: I should have fixed your needsfixings at https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800376/+merge/66194 [13:07] yes you should have :-) [13:10] nessita: hmmm... I may have? ;-) === teknico is now known as teknico_away [13:11] Anyway, I think I fixed them :-) [13:11] ralsina: you lost me. Shall I re-review? :-) [13:11] ok [13:11] nessita: yes, please [13:11] * nessita on it [13:15] ralsina: last night I was tired, and I missed that the docstring for CalculateSize is not pep-257 compliant. Can you please fix? [13:15] nessita: sure. Sorry about that. [13:16] np :-) [13:16] can I have a second review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/isolate-create-folder/+merge/66220 ? [13:17] ralsina: same for docstring for show_hide_offer [13:17] nessita: yes, I am checking all docstrings now [13:18] is there a chance that there us a typo here? "The folders you have selected to sync take over your %s space. You can remove some folders or add some extra space" [13:18] this part sounds weird " take over your 2GB space" [13:18] nessita: that is copied from the wireframe, I will file a bug to re-check them all with the final texts I have in a google doc [13:19] thanks! [13:20] hello, hello. Hola! [13:20] hola alecu! [13:21] hi alecu! [13:21] alecu: so, I've had a talk with Chipacaa last night about reactors, and we decided that we should go with the socket scenario by using a fixed port without any further port handling until we have all the pieces working together [13:22] alecu: given what you said last night in your email, we should be start moving to a working solution today [13:22] alecu: does that make sense? [13:22] nessita, sounds reasonable. === teknico_away is now known as teknico [13:23] nessita: pep-257-ized [13:23] alecu: so, for now, no special port handling but a fixed (hardcoded) one, aiming to have the 3 main pieces (sso, sd, cp) working together [13:23] ralsina: re-looking [13:24] alecu: and finger cross, as well :-P (figuratively, we need to type) [13:24] nessita, jajajajajajajajaa [13:26] nessita: pep257says we should leave a whole blank before the closing """ in multiline docstrings? No wonder I was not writing compliant ones :-( [13:26] nessita: so they are still not compliant. Re-fixing :-( [13:26] ralsina: the extra blank line is optional, and I'm +1 to have it but some others don't like it [13:27] (optional only when closing, not between the summary and the description) [13:27] nessita: Oh, ok, then they are compliant (/me double-checks) [13:36] ralsina: did you push the change to docstring in show_hide_offer? [13:37] LP still shows a non pep257 [13:38] I have """Show or hide the offer to buy space according\nto the size of the currently selected folders.\n""" [13:38] It's too long for a oneliner but not enough for a summary/blank/description [13:39] ralsina: the docstring needs to be either the one liner, or the summary/blank/description [13:39] ralsina: that docstring, as is, is not pep257 [13:39] nessita: grmbl, ok, making it shorter [13:42] nessita: pep257 suggests blank lines before docstrings for classes, and we are not doing that anywhere, I think. [13:43] ralsina: we should, one empty line before and one after [13:44] nessita: I am looking at control panel, and it's not done, same in SSO. I will add them in the wizard, but we seem to need a pep257.py and a day off to fix them everywhere. [13:45] ralsina: +100 [13:45] ralsina: where can I delete my credentials in w? [13:46] nessita: open control panel, search for credentials, open it, look arund, click on the one you want to delete, click "remove" [13:46] nessita, ralsina: if you guys have time: https://code.launchpad.net/~mandel/ubuntuone-client/fix-802738/+merge/66292 [13:46] mandel: on it! [13:47] * mandel super quick lunch [13:48] nessita: just as personal opinion, I agree we need to follow a convention and this is the one we have, but the PEP257 style is really ugly :-( [13:48] ralsina: I accept that as you personal opinion :-). I, personally, love it. [13:49] nessita: also, our module docstrings are really lacking by that standard. [13:50] After things settle I may take a dy or two for docstring checks [13:50] ralsina: you mean at syntax level or content? [13:50] nessita: content [13:50] True [13:54] standup in 5', crowd [13:54] mandel: I've had several issues trying to test your normpath branch, but none directly related to the code [13:54] mandel: still trying [13:54] (mostly VM issues) [13:55] nessita: ok, that I guess is good.. [13:55] mandel: well, I haven't been able to run the code yet :-) [13:56] mandel: can you do a review, please? [13:56] nessita: ofcourse [13:56] https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/isolate-create-folder/+merge/66220 [13:56] thanks! [13:56] np === m_conley_away is now known as m_conley [13:57] * ralsina gets coffee, updates notes [13:58] nessita: in line 38, is_link does exist in platform, why does it complain? [13:59] mandel: it complains because the definition of is_link is not in that module (but imported to it) [13:59] ah, ok, pylint is stupid ;) [14:00] moi [14:00] me [14:00] me [14:00] me [14:00] alecu, mandel, dobey? [14:00] me [14:00] me [14:01] alecu ? [14:01] me [14:01] WORK IN PROGRESS [14:01] rst port of the tutorial pages [14:01] Blocked [14:01] * nope [14:01] go thisfred [14:01] DONE: Bug #803062 Bug#762722 TODO: wrap Bug#762722 BLOCKED: no NEXT: ralsina [14:01] Launchpad bug 803062 in ubuntuone-control-panel (and 2 other projects) "Unsurprisingly, sys.platform == "linux2" check no longer works on linux 3 (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/803062 [14:01] DONE: mgmt call, windows call, administrivia, fixed things/worked on branches proposed for bug #800376 and bug #800381 [14:01] TODO: The other half of 800376, polish the workflow, ping design about assets, start fixing UI of SSO client [14:01] BLOCKED: no [14:01] Launchpad bug 800376 in ubuntuone-windows-installer "Add "syncing your computer to the cloud" page to the installer (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/800376 [14:01] Launchpad bug 800381 in ubuntuone-windows-installer "Add "sync now or later" page to the installer (affects: 1) (heat: 19)" [Undecided,In progress] https://launchpad.net/bugs/800381 [14:01] next nessita [14:01] DONE: Bug #800161, Bug #802517, lots of reviews, meetings, attempts to do windows testing but VM is not being nice to me [14:01] TODO: Bug #803228, run syncdaemon a lot on w7 testing, mostly, mandel's branches [14:01] BLOCKED: the VM I'm using is, suddenly, very slow [14:01] NEXT: dobey [14:01] λ DONE: reviews, fixed distutils-extra backport to lucid, started on autotriage tool, discovered broken deps on lucid for lazr [14:01] Launchpad bug 800161 in ubuntuone-control-panel "Do not allow folder creation in a folder containing an UDF (affects: 1) (heat: 12)" [Medium,Triaged] https://launchpad.net/bugs/800161 [14:01] Launchpad bug 802517 in ubuntuone-control-panel "QT UI: Split the "add folder" button out from the folders panel (affects: 1) (heat: 57)" [High,In progress] https://launchpad.net/bugs/802517 [14:01] Launchpad bug 803228 in ubuntuone-control-panel "Make test run and pass on windows (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/803228 [14:01] λ TODO: more bug fixing/triage [14:01] mandel [14:01] λ BLCK: None. [14:02] DONE: finished tests for bug 802738 moved to look at bug 708652 and found how to solve it. [14:02] TODO: review for nessita, but 802726 and bug 802628 [14:02] BLOCKED: no [14:02] Launchpad bug 802738 in ubuntuone-client "KeyError on the root folder inside volume manager (affects: 1) (heat: 6)" [Critical,In progress] https://launchpad.net/bugs/802738 [14:02] Launchpad bug 708652 in ubuntuone-client "Tritcask does not support the use of long paths on windows (affects: 1) (heat: 7)" [Medium,Triaged] https://launchpad.net/bugs/708652 [14:02] Launchpad bug 802628 in ubuntuone-client "Shares dir does not use the long path prefix: \\?\ (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/802628 [14:02] bug 802726 [14:02] alecu: go [14:02] Launchpad bug 802726 in ubuntuone-client "KeyError in ubuntuone\syncdaemon\filesystem_notifications.py (affects: 1) (heat: 8)" [Medium,Triaged] https://launchpad.net/bugs/802726 [14:03] any comments? [14:03] DONE: compared twisted 10 and 11 behaviour with txnamedpipes, researched txnamedpipes+iocpreactor+threadedselect reactor, found more txnamedpipes issues [14:03] TODO: change ubuntu-sso-client to use qtreactor back (bug #803433) [14:03] BLOCKED: no [14:03] Launchpad bug 803433 in ubuntu-sso-client "Use qt-reactor (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/803433 [14:03] mandel, ralsina, alecu, Chipaca: meeting "a las y media"? [14:03] nessita, ack [14:03] nessita: ok for me [14:03] nessita: genial [14:03] ok, thanks everyone! [14:03] eom [14:03] * alecu has just tagged bug #803062 with "funniest-bug-titles" [14:03] Launchpad bug 803062 in ubuntuone-control-panel (and 2 other projects) "Unsurprisingly, sys.platform == "linux2" check no longer works on linux 3 (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/803062 [14:04] mandel, ralsina: does the check ys.platform == "win32" works on non-32 windows? [14:04] nessita: good question. let me check! [14:05] yes [14:05] for now [14:05] That is, yes, if I am on windows 7 64 bits, which I am not sure of ;-) [14:05] ralsina: right click on the My Computer in the menu and look at properties [14:05] I discovered linux2 also still works on linux 3, though it may change soon, so it's good we fixed it [14:06] ralsina: will tell you what os is running [14:06] yep, 7 home premium 64 [14:06] thisfred: misleading bug! [14:07] yep, I misremembered the thread on python-dev ;) [14:08] ralsina: to fix: docstring in test_credential_parameters (is the only ds left to fix) [14:08] ralsina: fixed the bug title and description [14:09] nessita: line 69 in the diff, can that path have \\?\ in it, if it does we have an issue when we expand ~ because the expansion won't have \\?\ and therefore will not work [14:10] looking [14:10] nessita: there is a expanduser function in os_helper that will do that for you, that is expand the user and add \\?\ but I dont know if you need it in this case [14:11] mandel: well, I guess we'll need it since the volume info, sent from syncdaemon, will have the \\?\ prefix [14:11] brb [14:11] mandel: can I file a bug a fix that in a second branch, and check all the project to handle that? [14:12] nessita: sure, if there are more places to check it makes sense [14:13] bug #803437 [14:13] Launchpad bug 803437 in ubuntuone-control-panel "Confirm that paths are properly handled using ubuntuone.platform os helper functions (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/803437 [14:18] nessita: besides code review and tests, is there anything I should do? [14:22] mandel: nopes [14:23] +1 then [14:23] thanks! [14:27] mandel: I can't have syncdaemon running, I'm getting errors such as: [14:27] mandel: ah, no, sorry, PICNIC [14:28] my credentials are not in place... somehow syncdaemon is not handling that, I may file a but later [14:28] no worries, PINIC happens to everyone, even to nessita [14:28] nessita: not it places as in the sso is not getting them? [14:28] and lately very often :-/ [14:29] alecu, mandel: I need a second review for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800376/+merge/66194 [14:29] mandel: syncdaemon is just calling find_credentials, and sso says CredentialsNotFound. Yet, syncdaemon keeps going and send the empty credentials to the server [14:29] alecu: I'm on ralsina branch before I do context switch to a diff bug [14:29] mandel: thanks! [14:30] mandel, alecu, Chipaca: mumble? [14:30] nessita: surely we should be getting that error and die.. file a bug and lets see if we can get your sd running [14:30] yeap [14:30] running show_gui now [14:30] ralsina: I hate your merge proposal descriptions ;) [14:31] mandel: I can change! Why? ;-) [14:31] ralsina: is sooooo much work to do hehehe :) [14:32] mandel: I would make them easier if I could, trust me [14:49] alecu: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800381/+merge/65502 [15:00] ralsina, ping [15:00] ralsina, I'm trying to test irl, but I'm getting the sso screen with no labels [15:00] ralsina, what it was that am I doing wrong???? :-( [15:00] alecu: did you start the sso client before this? [15:00] ralsina, yes [15:00] alecu: no idea then ;-( [15:01] s/;/:/ [15:01] I only get that when I don't start SSO [15:01] you could try the page of this branch by moving it to the beginning of the wizard [15:03] wait:the "login as existing user" page has no labels on the lineedits. It should have "sign in" in the button, though [15:03] alecu: ^ [15:03] ralsina, right [15:04] ralsina, well, I'm seeing some pb errors on the log, so it might be the iocp reactor acting up. [15:04] alecu: darn [15:04] ralsina, ok, since run-tests.bat has run successfully, I'll approve stating that my dev env is borked. [15:05] alecu: ok [15:05] ralsina, and hopefully we'll switch the wizard to use qtreactor tomorrow too. [15:05] alecu: if there are IRL problems they will show up in the next branch for this bug anyway [15:05] cool [15:06] alecu: does that involve anything other than changing the main? I can do that. [15:06] ralsina, approved. [15:06] alecu: yay! [15:09] ralsina, we need to change the main to use the qtreactor, and change the calls to sso to use some code that will start sso if it's not running. I still need to think a bit on how to do that. [15:09] yes, the activation is tricky [15:09] ralsina, I'll first change -sso and u1-client, and then we'll use the same on u1cp and the wizard. [15:10] alecu: sure [15:10] anyway, I'll probably do the activation after seeing all the pieces working together. [15:26] mandel: on your branch, the os_helper tests pass, the code is OK, but I have no idea if it fixes the underlying problem because I don't know if you caught it everywhere that it happened. So, I am not sure if my approve is worth anything. [15:26] ralsina: lets get ask verterok [15:27] nessita, mandel, alecu, thisfred, dobey: OBJECTIVES! [15:27] ah, yes [15:28] nessita, mandel, alecu, thisfred, dobey: can each of you make a first draft, then we chat a bit to polish? Due tomorrow? [15:28] oh yeah, we have to do that thing today [15:28] I misread that as 'chat a bit in Polish', and thought, wow they don't make it easy [15:28] ralsina: objective => holidays, survive windows [15:28] bother. i totally forgot about that :( [15:28] ralsina: is that ok? [15:28] lol [15:28] mandel: doesn't fill the year ;-) [15:29] ralsina: holidays can surely fill the year :) [15:29] surviving hopefully does ;) [15:29] ralsina: AAAAAAAAAAAAAAAAHHHHHHHHHHHHHHHHHH [15:29] nessita: agreed :-( [15:30] ralsina: draft in the admin system or in email? [15:31] mandel, what does "survive windows" mean? not falling thru any? [15:32] alecu: and gates too [15:33] thisfred: email would be easier for me [15:34] kk [15:34] damn 30-day months! [15:40] hi adorilson [15:41] dobey: can you see my proposal merge? [15:41] https://code.launchpad.net/~adorilson/ubuntuone-control-panel/fix-802069/+merge/66227 [15:41] adorilson: i haven't seen one yet, but i wanted to discuss with you, because i haven't been able to find the default compiler flags for fedora anywhere on google [15:42] adorilson: and i would very much like to know whwat those are [15:43] dobey: ok, but this patch is not about this. is about another bug. [15:43] alecu: this is surviving windows: http://www.youtube.com/watch?v=zlMUswe8wY8 [15:44] oh [15:44] adorilson: oh, this seems incorrect [15:47] ralsina: is there any chance you buy us some time re: goals? [15:47] nessita: sure [15:48] nessita: monday is good? I can try to get that much [15:48] ralsina: well, Tuesday, since MOnday I'm not coming. Which reminds me, I need to swap this friday swap to later in time [15:49] nessita: ok, will try to get tuesday. [15:52] ralsina: ping [15:52] Chipaca: pong [15:54] dobey: this one is for you http://i.imgur.com/aqzMu.png [15:54] mandel: ^5 [15:54] :D [15:55] ralsina: uper quick easy review: https://code.launchpad.net/~mandel/ubuntuone-client/fix-708652/+merge/66322 [15:55] s/uper/super [15:57] mandel: checking! [15:58] ralsina: one question, when you call the __init__ of your parent class, should that always be the first thing you call? [15:58] mandel: no [15:58] mandel: but you can't use anything from the parent class before that [15:59] mandel: I am under strict suggestions to not approve anything without tests ;-) [15:59] ralsina: how the hell do i test that? [15:59] ralsina: that was in a polite tone, no screaming etc.. [16:01] mandel: hahaha, beats me. Let me see some context. [16:01] nah, no way to test it, +1 [16:02] :D [16:02] it's a temporary value right in the middle of a method, so whatever [16:02] ralsina: I m having a terrible time tryng to have things running in the VM. Was alecu able to run your branch? [16:05] nessita: he got reactor errors [16:05] nessita: In any case, the IRL for that branch can be re-done after the next branch for the same bug [16:07] nessita: but the tests passed! ;-) [16:07] ralsina: right, I ll approve then [16:08] nessita: col, thx [16:08] ralsina: wait, did you fix test_credential_parameters docstring? LP may be lying [16:09] (it doesn't look fixed after a reload) [16:09] * ralsina checks [16:09] nessita: no, will fix before merging [16:09] always one gets away :-( [16:10] ack [16:10] nessita: pushed [16:12] gaaaaah I can't use ussoc twice in a row!!! [16:13] nessita: 'stale reactor" error? [16:13] ralsina: no, ussoc just do not answer any request [16:13] ... [16:13] anyways, let's start over again! :-) [16:14] I'll grab some snack first [16:16] food! good idea! === ralsina is now known as ralsina_lunch [16:16] oh it's about that time [16:19] ralsina_lunch: ping? [16:20] ralsina_lunch: well, for later, seems that you approved https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800376/+merge/66194 before you had a second review (I was on it) I have approved it but I suppose that tarmac will complain, but I'm not sure [16:21] mandel: oh, damn, didn't notice [16:22] * ralsina_lunch confused two branches :-( [16:22] mandel: could you actually see it? [16:22] ralsina_lunch: yes, development has moved from dependency hell to branch hell [16:22] ralsina_lunch: the approve or the dialog? [16:22] mandel: the dialog? [16:23] ralsina_lunch: I mean, the dialog :-) [16:23] ralsina_lunch: yes, I had some issue but I think is because I did not follow the instructions [16:23] let me recheck [16:24] ralsina_lunch: yes, works [16:24] mandel: ok, great [16:24] dobey: I saw your comment. [16:24] so, after luch, I can add the last page, then hack it all together [16:24] yay! [16:26] dobey: You means that we write just one line? [16:26] adorilson: i also commented on the make error bug. can you provide the default compiler flags on that bug report, from fedora? [16:27] adorilson: no, i mean the string literal should remain the way it was before your change, and you should only change the second line of that function call, to use the % (foo, bar) instead [16:28] adorilson: or preferrably use .format() notation instead of %-notation [16:28] since %-notation is deprecated [16:31] mandel: good news, seems like your fix to normapth is working (I still need to go thru the logs to confirm no exceptions are there) [16:31] ok, am off to lunch. bbiab [16:31] mandel: also, images are not corrupted, so yey!!!! [16:31] mandel: one down side is that I deleted a folder in the web UI, but that change is not being reflected locally in windows\ [16:32] mandel: any clues? [16:32] dobey: ok. I'll fix [16:32] nessita: first time I see that… is it a udf or a folder inside root? [16:33] mandel: folder inside root, I haven't tested UDF's yet. Hum, looking at the logs I see an exception [16:33] OSError [16:33] mandel: will file bug [16:33] nessita: ok, that is good :) [16:33] mandel: yes! [16:33] you need reviews from me, for the normpath thingy? [16:34] nessita: let me check [16:34] (or anything else) [16:34] nessita: yes please normpath is here: https://code.launchpad.net/~mandel/ubuntuone-client/fix-802738/+merge/66292 [16:35] ralsina_lunch: swap day re-swapped [16:35] mandel: looking [16:35] nessita: there is an explanation of why abspath is used as a TODO, the branch that fixes tritcask is approved so I'll file a low level bug pointing to those todo comments [16:36] mandel: please! [16:36] nessita: also, can you let me know your bug, I think i'll fix that one faster than the one of the events [16:36] which seems to be easy.. and event mask is missing [16:37] mandel: the OSError? yes, will report that before the review, on sec [16:37] nessita: do you know which changes where made regarding pyinotify? it could be realted since I use the same masks [16:37] mandel: no idea, no [16:43] mandel: bug #803507 [16:43] Launchpad bug 803507 in ubuntuone-client "OSError when deleting a folder (windows version) (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/803507 [16:45] brb [16:45] nessita: great! that is a piss of cake, os.listdir is appending / instead of os.path.sep, we have to apend it so that listdir does not behave like a retard :) [16:57] :-) [16:58] even when he's not trying to be offensive... ;) [16:58] o wait, strike that I didn't read the whole sentence. [17:02] thisfred: I should have said piece, I dont know what I was thinking... [17:02] * mandel looks at the second monitor… no piss porn there... [17:03] mandel: for future coding, I've notice you use different names for the test cases. I've seen FooTestCase, FooTests and TestFoo [17:03] mandel: if possible, please let's use FooTestCase [17:03] nessita: ok [17:03] mandel: setUp in TestSpecialOSCalls is not needed, can you please remove? [17:04] nessita: yes [17:07] mandel: that would be all from eye-review, now I'll branch and run tests [17:07] nessita: I'll wait to push the branchin case you have more omments [17:12] mandel: while tests run, I would like you to analize http://pastebin.ubuntu.com/635142/, that log is from this last run in w7. The OSError is the reported above, but there are several WindowsError: [Error 2] The system cannot find the file specified: '\\\\?\\C:\\Users\\Test\\Ubuntu One\\empty' [17:14] nessita: sees to be a diff issue… I wonder did you get those paths in your machine when you performed the sync? [17:14] yes [17:15] nessita: hm.. interesting, is there a way were we can find a better trace for one of those? [17:15] nessita: I'd say that is a diff bug altogether, but at leadt we not that _idx_path has the correct info [17:16] mandel: what do you mean by better trace? those logs are in DEBUG mode, not sure what else I can provide :-) [17:17] nessita: true… I'd like to know who is callig the stat method [17:17] seems to be VM, I'll grep for it [17:19] mandel: good news is that the only other Error there is the already known [17:19] 23:736:exceptions.KeyError: 'Unhandled Event in INotify: ' [17:19] mandel: so, :-) [17:19] mandel: I'll work a little on running the control panel tests on windows, to detect all possible paths issues. Then I'll do some testing with UDF and shares [17:20] nessita: yes, I do you know why is that one? we are dealing with opening folders, which is something that pyinotify does not care about :) [17:20] nessita: I'm working on that event one right now [17:21] mandel: not sure what you're asking me if I know about. Can you please re-ask? Also, branch approved, so please push fixes before landing (beh, actually seeking another review) [17:22] nessita: nothing, I was saying I know how to fix it :) [17:22] great! [17:28] nessita: swap-swap-day ap-ap-proved [17:28] :-) [17:29] no, lunchtime! [17:29] * nessita -> brbs === ralsina_lunch is now known as ralsina [17:42] ralsina, alecu: I need one more review here: https://code.launchpad.net/~mandel/ubuntuone-client/fix-802738/+merge/66292 [17:43] mandel: on it ASAP [17:44] ralsina: I think you already did it, is the one you were not sure about [17:44] yes, want to see the new diff and that's all [17:45] mandel: +1 [17:46] cool [17:58] * mandel afternoon dog walk [18:18] meh [18:18] * nessita is back [18:19] nessita: small problem using the "add folder" button: https://pastebin.canonical.com/49186/ [18:20] looking [18:20] ralsina: right, we don't have (yet) integration with syncdaemontool in windows [18:21] ralsina: let me build a branch for that right now [18:21] nessita: ok, then it works ;-) === alecu is now known as alecu-lunch [18:24] ralsina: can you plaese file a bug so I work on that? [18:27] nessita: sure, bug #803560 [18:27] Launchpad bug 803560 in ubuntuone-control-panel "AddFolderButton calls unimplemented get_root_dir method in sd_client (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/803560 [18:27] thanks [18:29] ralsina: great work setting the tag properly! :-) [18:29] * ralsina learns eventually [18:29] You Android guys! Check out Ubuntu One Files for Android :) https://market.android.com/details?id=com.ubuntuone.android.files [18:38] nessita: alecu-lunch dobey: no idea why or for how long, but the launcher does not work for me on natty in that I never get to see the progress bar, or anything else I set on it, even though the code does not fail. Even when I don't use our own code. Can any of you test this script: http://paste.ubuntu.com/635202/ [18:38] did we change/break the .desktop file maybe? [18:38] thisfred: well i don't have an 11.04 system with unity. does the same problem occur in oneiric? [18:39] thisfred: not that I know of [18:39] dobey: please try it on oneiric too [18:39] I'll do the same [18:39] (ie I don't think we broke the .desktop file) [18:39] thisfred: I can test when I boot my natty VM, where I run unity [18:39]