[06:11] I just reinstalled Ubuntu. Completely clean install, home and all. I then synchronized with u1. Just now, I opened Gnome Activity Journal, and all the files that were synced is displayed there. I don't think it should. At least not by default. Because you haven't really worked with those files. And it made the application very slow. So, the question is whether Ubuntu One logs to Zeitgeist in an inappropriate way or if it's Gnome Activit [06:11] y Journal that should be fixed? === OutOfControl is now known as benonsoftware [08:58] Morning all! :D [09:04] hi [09:06] just asking before doing damage: is it possible to move Ubuntu One folder elsewhere and replace it with a symlink in my /home? [09:17] andtorg, I would check with mandel but I guess it is possible, stop the sync daemon first [09:17] that's timing! :) [09:18] maybe rye ? :) [09:18] lol, thanks vds === gord_ is now known as gord [10:00] vds: just to let you know, it worked [10:00] andtorg, cool === yofel_ is now known as yofel [10:59] vds, it should work with the root folder, but if you put it in a diff volume it will go bannanas, udfs and folders do not follow symlinks :) [11:08] good morning! [11:10] mandel, yep [11:11] gatox, morning! [11:11] mandel, hi [11:11] gatox, I'm in my review day, can you please update the following mp: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 [11:11] * gatox looking.... [11:26] gatox, so, about the mp, will you update it? [11:26] mandel, yes..... i'm changing that..... [12:11] hello! [12:13] alecu, morning! [12:13] alecu, I have updated the webserver branch here: https://code.launchpad.net/~mandel/ubuntuone-dev-tools/mocked-webserver/+merge/101224 [12:13] mandel, awesome, looking. [12:14] alecu, also, I've started with my review day and there are some branches that you are currently as a reviews, can you take a look when possible, otherwise there is not that much I can review today :( [12:14] alecu, hi [12:15] mandel, got links to those branches? [12:15] alecu, we also have issues with the following branches you proposed: https://code.launchpad.net/~alecu/ubuntuone-client/timestamp-autofix-1-4/+merge/82755 and https://code.launchpad.net/~alecu/ubuntuone-client/timestamp-autofix-1-6/+merge/82754 [12:15] alecu, it seems that tarmac does not like the fact that we have to point to other branches.. so those are blocked until we have the chance to talk with dobey [12:16] alecu, let me grab a list [12:17] alecu, I think this is the only one: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 and gatox is making the required changes [12:19] mandel, yes, that branch only needs a fix that nessita requested. After that you may do the re-review. [12:19] alecu, ok, great! [12:22] mandel, please review this: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 [12:22] let me know if that is ok [12:22] (about the implementation... the code is working) [12:22] alecu, FYI I've talk with nessita, I'll push people around to update their branches and will let her know when we can re-review [12:27] gatox, I'll do it after lunch, is that ok? [12:27] mandel, no problme [12:27] gatox, ok! [12:27] * mandel lunch [12:28] alecu, mandel, it seems i don't have review day [12:28] gatox, monday! [12:28] :P [12:28] mandel, that's yours :P [12:28] alecu, ? [12:28] gatox, I know hehehe [12:31] 6/14 [12:39] good morning! [12:40] gatox: you have to claim your review day on the calendar, I think [12:44] ralsina, morning..... ok..... but monday to friday is already taken recursively.... that's why i was asking :P [12:44] gatox: pick a day, and that day there's 2 [12:44] ralsina, ok! [12:45] gatox: probably friday is a good idea since it's release-freeze day [12:45] ralsina, good [12:45] gatox: and since I am the one that does second reviews all days, I spend fridays on other stuff [12:48] gatox, mandel, alecu: I have a doctor's appointment i 1 [12:48] argh [12:48] in 1:45 so I will miss standup [12:48] ralsina, ack [12:48] I will stay late to compensate [12:50] In other news, 117.0 and still losing weight as planned :-) [12:50] although half the time I could eat a horse [12:50] ralsina, good!! I'm trying to do the same.... not success yet [12:50] gatox: did you go see a doctor? [12:51] gatox: it does make a difference [12:51] ralsina: guttenmorgenflerfen! [12:51] Hello Chipaca! [12:51] ralsina, no, i was just eating healthy and doing a lot of exercise [12:51] ralsina: ping me when you've finished booting :) [12:51] gatox: eating healthy pigs is not enough [12:51] Chipaca: well, about now is good :-) [13:09] morning, all! [13:09] urbanape, hi [13:18] morning urbanape! [13:18] urbanape: how was vacation? [13:19] urbanape: while you were gone, we finished the macport and put you in charge of the Amiga port. [13:19] urbanape: your Amiga 500 + external floppy will be arriving shortly [13:19] restauración [13:20] woohoo! [13:20] urbanape: on real news, brian is going back mostly to windows and mandel is going to help you more [13:21] and alecu? [13:21] urbanape: alecu as well [13:22] urbanape: and gatox can do reviews every now and then [13:22] excellent [13:22] urbanape: but alecu is new tech lead while natalia is on rotation, so he's not going to do aything fulltime [13:22] right right [13:24] urbanape: also, we instated review days [13:24] I saw the calendar email [13:24] urbanape: right, pick a day, you will have to share though [13:25] urbanape: if I may suggest tuesday, it's my busiest day, and would free me some [13:25] sure, that sounds good [13:27] mandel: hmm? [13:28] ralsina, btw: remember the SIGSEGV problem with SSO? It was a PyQt issue, so I've sent a script to reproduce the problem to the PyQt list, and the fix is now being packaged and proposed for precise in bug #334757 [13:28] Launchpad bug 334757 in KDE Bindings "update-notifier-kde.py crashed with SIGSEGV in QSocketNotifier::setEnabled()" [High,Confirmed] https://launchpad.net/bugs/334757 [13:28] alecu: saw the emails on PyQt, congrats :-) [13:28] alecu: i was just about to ping you about that [13:28] It was a *very* old bug. [13:29] alecu: can you test the fix that's in precise-proposed? [13:29] dobey, sure. How should I install the package in precise-proposed? [13:30] dobey, is it a ppa? [13:30] alecu: no. you can enable it in the "Updates" tab of the software sources config dialog [13:30] software-properties-gtk [13:30] awesome, thanks! [13:31] alecu: be careful though as there may be other updates in there, which might break stuff :) [13:31] right :-) [13:35] alecu: there is another fix that's coming in PyQt soon about disconnect, but we have workarounds in place for that one [13:35] ralsina, yup, it's usually harder to find workarounds for SEGFAULTS in python :-) [13:36] alecu: hehe [13:36] alecu: do we still want to put faulthandler in main? I can ask, you know. [13:37] ralsina, I'm checking the proposed pyqt packages, and if they solve it we may not need it. [13:37] alecu: cool [13:41] * mandel back [13:41] dobey, you need me? [13:43] Anyone needs any reviews? And if you don't whathave you all been doing????? ;-) [13:43] dobey, I've been looking at https://code.launchpad.net/~alecu/ubuntuone-client/timestamp-autofix-1-6/+merge/82754 and https://code.launchpad.net/~alecu/ubuntuone-client/timestamp-autofix-1-4/+merge/82755 do you know how we can make them land? [13:44] ralsina, che, todays is my review day, don't steal my job! [13:44] mandel: I am on review day every day (except tue and fri) [13:44] * ralsina is the goalkeeper of reviews [13:46] mandel: ignore them for now [13:47] ralsina: i've been trying to enjoy the weekend, thank you very much :P [13:47] dobey, ok, on other news, I added you for a couple of MP from a colleague from canonical regarding strings being liked in ubuntuone-client-gnome and ubuntuone-client [13:47] dobey: good :-) [13:47] mandel: right, i saw those and was going to look at them today :) [13:48] dobey, sweet, I did one of them, will move to the second one in a few mins [13:49] dobey, I also updated the ubuntuone-dev-tools one (mocked-webserver) removing the code duplication that you mentioned [13:49] and talk with nessita about the comments you added for an MP to ubuntuone-dev-tools, dobey she told me she will chat with you [13:52] yes [13:59] Hi urbanape and ping [14:06] pong [14:06] Dr_Who, ^ [14:08] Hi urbanape, are you the right person to talk to about the iOS ubuntuone projects for files, music and such? Reason for the q, I started working on the files app about a week ago (fixed a few bugs and such) and am doing a file things with the music app too, be good to connect with others who are working on it [14:08] Dr_Who: yup. I'm the primary (now) developer on both [14:09] ah cool ! [14:10] I'm just working on things in my spare time but happy to help out [14:10] on the music app, I've got an initial implementation to move the settings into the iOS settings app [14:10] We started that way, but there's no way using the normal settings bundles to do conditional disclosure of settings. [14:11] as I've been doing that, I've wondered a bit on cache policy as well as what seems to be a design point to have someone always connected into ubuntu one [14:11] Apple obviously can, but 3rd party devs are limited to a single plist describing the settings. [14:11] Not sure what you mean [14:12] the policy is this: Cache everything as it's played, up to being within 10% of the device's storage remaining. Purge LRU. [14:12] I'm off to the doctor, so he can tell me I am still fat. Will be back in about 90 minutes === ralsina is now known as ralsina_doctor [14:12] tracks explicitly downloaded (pinned via the buttons on album view) are exempt from both storage limits and LRU purge. [14:12] so if you have an option to turn off caching for instance, you can register to be notified of it, but your'e right the music app wouldn't be clearning caches until it starts up again, effectively resumed [14:13] ah ok [14:13] so what i mean by "always wanting to be signed in" is if you go to the settings view, hit sign out of ubuntuone, it'll prompt you immediately to sign right back in [14:14] well it used to ? I modified that slightly :-) [14:14] What's the use of it not talking to U1? [14:14] I mean, if you've got tracks downloaded, it works just fine in offline mode (airplane mode) [14:14] if we already have music local, I'd like to think it's reasonable to just limit the selection of what you'd be able to play [14:15] right [14:15] (or in No Service areas) [14:15] exactly [14:15] it does that. You can't navigate to music that doesn't exist locally if you're offline [14:15] right and not suggesting that you should be able to either [14:16] tho I am suggesting that signing out of u1 == airplane mode or lack of wifi or whatever without having to turn on airplane mode for instance [14:16] I think we're on the same page [14:17] Maybe. Is the point just to keep the app from talking to the server? [14:17] effectively [14:17] using network? I guess if you've got a really, really horrible data plan, that could save you a couple HTTP requests. [14:18] and being signed out would keep your selection of music confined to what you have locally [14:19] yeah, except we don't purge the Core Data entities just because we can't reach the server. [14:19] So, there's still gonna be all the greyed out entries in Artists, Albums, and Songs [14:21] you could still display them tho I think the correct thing to do would be similiar to what iCloud does, they have a setting to keep or omit them from the list [14:21] I'd suggest if you're signed out or without network, there's not much sense in displaying them [14:21] tho I suppose that could have a downside if you're on an bad network [14:22] anyway all that said, is there some areas that would be quite helpful to you ? [14:23] I've got one outstanding bug about resuming the auto-download queue when reacquiring network connectivity [14:23] so, probably need to have something triggered on the Reachability events that starts it back up. [14:24] o that one? [14:24] https://bugs.launchpad.net/ubuntuone-ios-music/+bug/972574 [14:24] Ubuntu bug 972574 in Ubuntu One Music App for iOS "Problem with Caching Songs - moving from online to offline mode" [High,Triaged] [14:24] there's one bit of detail there, I added some code to address it [14:24] o nm that's not the one I'm thinking [14:25] ah ok ? sure I'll see what i can find [14:26] the files app I think it was had an issue where it needed permission to location services in order to function. you could answer no and be left wondering why the app wasn't functioning [14:26] anyway I added a bit of logic and a UIAlert or two to instruction users how of the situation and how to correct [14:26] that's a different topic however [14:28] gatox, +1 for https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 [14:28] bleah. Yeah, I hate that whole aspect. [14:28] it would be better that if you disallowed it, you'd just get those stripped out of the EXIF data. [14:28] but that's not up to me [14:31] yeah agreed, it's a bummer of a design [14:32] briancurtin, ping [14:32] mandel, yeyyyyy [14:32] mandel pong [14:33] briancurtin, can you take a look at https://code.launchpad.net/~brian.curtin/ubuntuone-client/upoad-spellcheck/+merge/97656 and add what nessita required? [14:33] mandel yeah i'll see if i can figure that out [14:34] briancurtin, I think that ensuring that the argparse gets the correct params should be ok, but I might be wrong [14:35] briancurtin, let me know when done and I'll get you the second review you need [14:36] mandel: the change in tunnel_runner shouldn't be there, i probably pushed it there by accident. i'll look to fix that typo part though [14:36] mandel: what he heck is "iri" ? [14:37] mandel, ahhhhh but i still need alecu and nessita's approval [14:37] dobey, http://en.wikipedia.org/wiki/Internationalized_Resource_Identifier [14:38] alecu, can you abstain in gatox branch? [14:38] mandel: but why do we need a "get_iri()" here? and why a get_uri() that just returns the same thing? [14:40] dobey, one returns a unicode based url, the reason is that the webclient from ubuntu_sso expects all its urls to be unicode [14:40] they both return a unicode object [14:40] briancurtin: i've been following the ticket for the windows release but can't tell if it's ready for upload? doesn't appear that the new release is available at /windows/release [14:41] oh, no i see you do bytes() [14:41] ugh [14:41] joshuahoover: its not. waiting on the people who have access to do that to carry out the upload [14:41] dobey, yeah: bytes(iri2uri(iri)) [14:41] briancurtin: k, but it's all ready to be uploaded to the site, correct? [14:42] yep, we're in a queue for an admin to execute that [14:43] joshuahoover: after that's up, we will also need to have them bring the update.xml up to the latest, but ralsina wanted there to be some time between uploading the installer and updating the update.xml [14:43] briancurtin: right, got ya...and this version is 3.0.0, correct? [14:43] joshuahoover: correct [14:43] cool, thanks! [14:44] mandel: i don't think we should do bytes() ever. if we need ascii we should still return a unicode object, but assert that it fits in ascii [14:46] dobey, iri2uri will return unicode, how would you prefer to pass it to bytes? [14:46] mandel: we shouldn't pass it to bytes [14:46] mandel: think about py3 for example :) [14:47] dobey, urllib2 likes bytes, as well as twisted :( [14:47] mandel: even on python 3? [14:48] dobey, if you do an HTTPClientFactory with something that is not bytes you get an error, I think is the same with libsoup [14:48] qt is fine because it uses QStrings [14:49] mandel: those errors need to be handled where those points of error can happen i think. we shouldn't try to protect the world from the future in arbitrary places in our code like this :) [14:49] dobey, I've installed pyqt from proposed, and my script is not segfaulting anymore. [14:49] alecu: hooray! [14:50] dobey, so you propose that I go though every instance of get_uri used in the tests and call the iri2uri there? [14:50] mandel: no [14:51] mandel: what happens if you try to carete an HTTPClientFactory in python 3, and try to pass it bytes, given py3 defaults strings to unicode and not bytes. [14:52] dobey, when we pass to py3 we would brake, how would you do it? [14:54] mandel: i would start by never returning bytes(somethingelse) [14:57] dobey, so using decode, is that what you mean? [14:57] no [14:58] well, if the function is supposed to return only ASCII, then i would say fix the docstring to say so, and use unicode(foo.encode('sscii')) i guess? [14:58] i think that's the right way to do that in python [14:59] i think we need to expose and understand all the cases where unicode doesn't work first, though [14:59] and i don't think that is well known [15:00] dobey, well, I know twisted does not like it, urllib2 and libsoup [15:00] me [15:01] me [15:01] me [15:02] dobey, the only case in which get_uri is used is in the tests of that webserver in ubuntuone-dev-tools and in the timestamp code in ubuntu-sso-client [15:02] dobey, we could get rid of it in the tiestamp so that it get unicode and just leave it for the tests of the webserver that use twisted [15:03] ralsina_doctor, gatox, alecu, dobey stand up? [15:03] me [15:03] me [15:05] meh [15:05] briancurtin, go [15:05] DONE: signing stuff, hacking on the build_installer.py script to work around some buildout oddities [15:05] TODO: fixing the upoad-spellcheck branch [15:05] BLOCKED: return None [15:05] NEXT: urbanape [15:05] DONE: vacation TODO: Catch up, merge some of mandel's branches, push the proposal, BLOCK: NO LONGER ON VACATIOn [15:06] mandel: please [15:06] Reviews Day: [15:06] DONE: Reviews: Looks like there are issues when trying to merge https://code.launchpad.net/~alecu/ubuntuone-client/timestamp-autofix-1-4/+merge/82755 and https://code.launchpad.net/~alecu/ubuntuone-client/timestamp-autofix-1-6/+merge/82754 due to dependencies not being added. We have to ignore them atm, right dobey The following review is blocked: https://code.launchpad.net/~nataliabidart/ubuntuone-dev-tools/add-recorder/+merge/99058 - Need upda [15:06] te from nessita due to comments form dobey, [15:06] https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 - is blocked due to needed reviews form alecu and nessita [15:06] TODO: Reviews as needed, finish fixed for mocked-webserver (talking with dobey about it). Review urbanapes mac branch. [15:06] BLOCKED: no [15:06] 3. Branches that I cannot review (they are mine): [15:06] https://code.launchpad.net/~mandel/ubuntu-sso-client/remove-duplication/+merge/101724 [15:06] [15:06] [15:06] gatox, go [15:06] stupid tomboy paste.. [15:06] DONE: [15:06] Swap on Thursday and Friday, Fix u1sdtool -q branch tests, worked on how to handle text wrapping for Checkbox and widgets without wrapping, fix done, writing tests now. 1-1 with ralsina on Thursday, suggest a possible fix for Bug #824574 (waiting for design feedback). [15:06] TODO: [15:06] Finish with wrapping tests and propose, keep fixing ui stuff. [15:06] BLOCKED: [15:06] Launchpad bug 824574 in Ubuntu One Control Panel "QT UI scroll bar overlaps list area bottom border" [Low,In progress] https://launchpad.net/bugs/824574 [15:06] No [15:06] alecu, go [15:06] DONE: a script to reproduce the pyqt segfault which got fixed upstream, some reviews, making plans for UDS [15:06] TODO: get back to bugfixing [15:06] BLOCKED: no [15:06] NEXT: gatox [15:07] alecu, no.... i already did my standup [15:07] λ DONE: reviews, bug #682954, bug #956407 [15:07] λ TODO: SRUs, finish reviewing mandel's branch, couple small reviews, u1db packaging [15:07] Launchpad bug 682954 in Ubuntu One Client stable-1-6 "Typo in man page for u1sdtool, "path's" to "paths"" [Medium,Triaged] https://launchpad.net/bugs/682954 [15:07] λ BLCK: none. [15:07] Launchpad bug 956407 in Ubuntu One Client stable-1-6 "Apport recipe attaches old/useless log files" [Low,Triaged] https://launchpad.net/bugs/956407 [15:07] sorry [15:07] NEXT: dobey [15:07] thisfred: ? [15:07] me [15:08] DONE: test improvements u1db TODO: same + start work on u1db example app? BLOCKED: no [15:10] is Harry around? [15:10] he's meeting Sally. [15:10] briancurtin, was the installer finally signed by the l-sas? [15:11] alecu: it's signed, just waiting on them to do the actual upload. apparently they're a bit backed up this morning [15:12] briancurtin, awesome anyway. [15:12] mandel: a feature of the jenkins automation will be that everyone has to install these on their computer, then if they make tests fail the lights turn on until you fix it: http://compare.ebay.com/like/251025562457?var=lv<yp=AllFixedPriceItemTypes&var=sbar [15:12] briancurtin, is that the installer with the fixed autoupdate? [15:12] dobey, so, what do we do with that bytes, also you whould know that a similar code is present under ubuntu-sso-client webclient [15:13] briancurtin, haha great idea! [15:13] alecu, you mean the re-added autoupdate? [15:13] brb....... reboot [15:13] mandel, that, right :-) [15:13] alecu, yes, should have been merged in trunk for 3.0.0 AFAIK [15:14] briancurtin, and I thought the Chuck Norris plugin for jenkins was cool. Not anymore! [15:14] alecu: yep, that's in there. it's stable 3.0 plus one hacky branch to allow the Cloud To Computer dialog on Windows to work [15:14] mandel: i'm not entirely sure. maybe barry has some idea about how to handle that compatibly for both py2 and py3 [15:15] mandel: i also needs fixing'ed your branch again, for string format stuff. we should use .format() notation for all new strings now [15:15] dobey, ein, are we already moving all the way to python 3? [15:15] dobey, briancurtin, all: do we need anything else from the l-sas these days? [15:15] ralsina_doctor, too [15:15] alecu, not from me [15:16] cheepaca was asking us about that. [15:16] alecu: after the windows installer is uploaded, we will need them to change the update.xml on the website, but we want there to be some time in between the file being uploaded and it being set as the update version (so we can do any last minute tests) [15:16] alecu, ein? is that about my python 3 comment or losas? [15:18] alecu: i don't know of anything other than the windows stuff. [15:19] dobey, I'll take a look at the format() , but I did not know we were already movin to python3 (do they want to kick out python2?) [15:19] briancurtin, right. [15:19] * gatox lunch! [15:19] mandel: we need to try and avoid creating more work for us in the future when we do make python3 the default [15:19] dobey, same with exceptions then, Exception as e ? [15:20] mandel: did something change for exceptions for py3? [15:20] briancurtin, do we have any checks planned for the time between the file goes up, and the time update.xml is changed? [15:21] mandel, we can already start using "Exception as e" in 2.7 afaik [15:21] dobey, yes: http://docs.python.org/release/3.0.1/whatsnew/3.0.html#changed-syntax [15:21] mandel, I think we can already use it in lucid, but lemme check. [15:21] alecu: no lucid is python 2.6 [15:21] so if it's only since 2.7, that's a problem [15:21] alecu: the original plan was for the installer to go up on friday, then we could flip the update.xml today, giving us all a little time over the weekend to do last minute playing. i think if we get the installer up today, we can probably get update.xml changed tomorrow [15:21] and lucid is an old 2.6 also [15:22] lucid doesn't have a new enough python to use dh_python2 even :( [15:22] dobey, alecu is the following pep: http://www.python.org/dev/peps/pep-3110/ [15:23] dobey, mandel: 2.6 has the "Exception as e" syntax [15:23] briancurtin, thx! [15:23] briancurtin, great. But we should be able to start "testing" right now even though the installer was not uploaded nor the update.xml was changed, right? [15:25] alecu: yes, i can get the signed installer out of the ticket and share it out for people to use (i'll do that now). i think we still want to check what gets uploaded when that happens [15:25] briancurtin, so, I was wondering if you had in mind something we should be already testing so we can flip the update.xml switch as soon as the file is uploaded. [15:25] briancurtin, right, but probably an md5 check is all we need :-) [15:26] alecu: i was just thinking a simple smoke test. download it, sign in with creds, drop a file in your u1 file, watch that it gets synced === gatox is now known as gatox_lunch [15:26] s/u1 file/u1 folder/ [15:27] mandel: so yeah, we should use the new syntax in new code, and update existing code when we change it; and maybe have a few days where we just go through and update some projects to work with py3 and py2. would be nice to start getting py3 packages of our things built in nightlies, where we can [15:28] dobey, ok, I suppose we have to start somewhere :) [15:28] although I would have liked to land this before that hehehe [15:29] mandel: better to fix it now, than to have to do it in 3 months when python2 gets bumped out of default install :) [15:33] briancurtin, that sounds awesome, since it's a quick test, and we can do that right after the l-sas say "file uploaded" and before we saying "now flip the update.xml" [15:34] briancurtin, my experience with the l-sas is that they are usually overworked, so I was hoping we can do both steps before them fully moving to some other task. [15:35] alecu: i will update the RT ticket and ask them to sign the update.xml - they haven't done that part yet [15:35] briancurtin, so, we should warn them that we will be doing the smoke test after the upload, and saying that it will take no longer than -say- 45 minutes. [15:36] ok, off to get lunch. bbiab [15:36] alecu: that'll work assuming the update.xml will be signed in time. it took a long time to get the signed installer back on friday (but maybe that was just friday) [15:37] briancurtin, right :-/ [15:37] briancurtin, so, it's a great thing moving forwards with signing the xml [15:38] briancurtin, mandel: btw: say I want to IRL test the autoupdater... are the steps documented somewhere? [15:38] alecu: mandel wrote up some steps in email, i'll forward on [15:38] alecu, let me forward you the mail I sent to QA [15:38] mandel go for it [15:38] briancurtin, I'll let you do it, you were first :) [15:39] sent [15:39] briancurtin, mandel: great, thanks! === ralsina_doctor is now known as ralsina [15:51] briancurtin: the update.xml is not signed, just deployed [15:51] oh it doesnt have to be signed? crap, i just posted to the RT about it...those people probably hate me by now haha [15:52] briancurtin, mandel, urbanape, alecu: at some point, please consider making the mac setup less time consuming (as in: here's qt, here's pyqt ;-) [15:52] briancurtin: say "sorry, I'm new here" ;-) [15:52] If we suffer, everyone has to suffer [15:55] urbanape: I am the manager. I am supposed to suffer in different ways. [15:55] For example, these grapes I am being fed by slaves? Sour. [15:57] ralsina, are you referring to number of steps or to the time that qt and pyqt take to compile? [15:58] alecu: both [15:58] alecu: ideally, we would someday have a scrip that takes exactly one step and ends in as little time as possible [15:58] alecu: in a perfect world: one command, one second. I understand we can't achieve that ;-) [15:59] but relying on a long and annoying process was one of the missteps we had on the windows port. So, let's not do it again. [16:04] looking for a volunteer for bug #983144 [16:04] Launchpad bug 983144 in ubuntuone-client (Ubuntu) "Stopping sync of a UDF causes unrelated warning" [Undecided,New] https://launchpad.net/bugs/983144 [16:04] I am betting on a bad path comparison (missing trailing /) [16:04] ralsina: getting the 3.0 installer from the site now to check it out [16:05] briancurtin: cool, thanks! [16:05] briancurtin, I'm getting the following when using the bootstrap on mac os x: paste.ubuntu.com/932699 any idea how to fix it? [16:06] mandel: hm, i have not seen that. i will take a look after i finish looking at this windows installer [16:06] briancurtin, sure, no problem === gatox_lunch is now known as gatox [16:11] I need to walk the beast, I'll me back in 20 min === salgado is now known as salgado-lunch [16:30] mandel: when you get back, I have an easy one for you: https://code.launchpad.net/~thisfred/u1db/mapping-test-improvements/+merge/102141 (no c code, even) [16:40] gotta make a quick errand. Back in a bit. [16:52] well upgrading doesn't really work because the upgrade is performed while the old u1 is running, so no files get updated... [16:52] hmm [16:55] mandel: in the current autoupdate, does u1 get stopped while the autoupdater runs? [16:56] i think people will have to manually upgrade from 2.0.3 to 3.0.0 [16:58] and we'll run into this same problem going from 3.0.0 to the next version. i think we need to quit u1 before running the autoupdate program. [17:07] stupid phone calls. Now on quick errand. [17:09] ralsina, mandel: i think ubuntuone-control-panel's ubuntuone.controlpanel.utils.windows.perform_update needs to spawn the update execution in a subprocess and then quit, allowing the update to actually update [17:10] briancurtin: +1 on that run updater separately and quit all running u1 processes [17:10] briancurtin: if the process is running the update will succeed but then not take effect until yu quit u1 [17:10] briancurtin: IIRC [17:10] briancurtin: but yes, in principle, that sounds reasonable [17:11] ralsina: let me try and make a 3.0.1 and put it on my server. the upgrade doesn't seem to work on 2.0.3 -> 3.0.0 - i've shut everything down and still have 2.0.3, but i'll try rebooting the computer to see if that may make it take effect [17:12] ralsina: except for the part where it can't replace files that are opened by another running process like say ubuntuone-syncdaemon, no? [17:12] dobey: it can [17:12] dobey: it moves the folder away somehow, then would require a reboot (normally) [17:12] ah ok [17:13] does the upgrader tell the user to reboot when it's finished? [17:13] i didn't see any notice, but i'm trying it again to see if it does, then i'll reboot and see if it works [17:17] dobey: my guess is it should, but have not tried it myself in a long time [17:19] ralsina: maybe it used to in the old code, but that got lost in the shuffle? [17:19] dobey: no idea, really [17:19] dobey: all that's supposed to be automaticly done by 3rd party code ;-) [17:19] ah [17:20] after all, syncdaemon & co could easily be running under another session [17:20] indeed [17:20] we have a similar problem on linux [17:20] BTW, the exact same thing happens in ubuntu [17:20] right [17:21] syncdaemon should autodetect it has been updated and restart itself. Put a watch in his own binary :-) [17:21] i should find some time to make the fix for that work [17:21] dobey: yeah, would be nice to have [17:22] you can't just put a watch on the script file though [17:26] so after running the 2.0.3 upgrade to 3.0.0, there's no restart notice, and it doesn't look like the new binaries it'd update to are available anywhere. rebooting now to see if it replaces them (from somewhere) [17:26] ralsina: well i see what's wrong with bug #984144 [17:26] Error: Launchpad bug 984144 could not be found [17:26] err [17:26] ralsina: well i see what's wrong with bug #983144 [17:26] Launchpad bug 983144 in ubuntuone-client (Ubuntu) "Stopping sync of a UDF causes unrelated warning" [Undecided,New] https://launchpad.net/bugs/983144 [17:26] rather [17:26] dobey: enlighten me [17:27] no trailing / [17:28] dobey: guessed as much when I called for volunteers. And because it has happened like 5 times before. [17:28] dobey: so, you volunteering? ;-) [17:29] well, i will fix it yes [17:29] dobey: awesome, thanks [17:29] ralsina, weird it used to work with no problems what so ever [17:29] briancurtin, weird it used to work with no problems what so ever [17:29] I need to change room, give me 5 min [17:29] or even less [17:30] i haven't rebooted yet to see that it worked, stupid windows updates are happening right now. i also need to pick up the lunch i just ordered before it gets cold. be back in a few minutes === mandel is now known as p [17:30] uh === p is now known as Guest16683 === Guest16683 is now known as mandel [17:31] what the heck [17:31] mandel: you don't need to review branches which already have enough people reviewing it, btw. and alecu and nessita both need to re-review gatox's branch [17:32] dobey, I spoke with alecu, he agreed I could do the re-review.. but I guess I should have asked him to change the reviewer [17:33] briancurtin, ralsina, so what is the problem with the update? it used to work and nothing has been changed [17:33] mandel: "it used to work" and the whole autoupdater code has been moved around [17:34] mandel: looks like since the binaries are in use, autoupdating may not do anything until you reboot [17:34] ralsina, move around when? [17:34] the code was not in u1cp before, was it? [17:34] ralsina, no it was not.. mierda [17:35] mandel: don't worry, this is not horribly problematic [17:35] mandel: if things update on reboot, that's as good as we had before [17:35] ralsina, but it is a PITA [17:35] mandel: there is no easy fix beyond a "reboot now" dialog [17:35] ralsina, but, does the dialog tell you you have to reboot, or it just does nothing? [17:35] mandel: even killing processes before updating is a partial fix [17:36] mandel: don't know yet [17:36] mandel: were you going to review https://code.launchpad.net/~charlesk/ubuntuone-client-gnome/lp-981270/+merge/101993 as well? [17:36] dobey, yes, I was going to give it a +1 but I wanted to talk with you first to see if there is something I don't understand [17:36] dobey, is there? [17:37] mandel: i don't know what you do or don't understand :) [17:38] mandel: aside from women of course. you clearly don't understand them. :P [17:38] dobey, he, what did i do now? [17:39] heh [17:39] dobey, I don't see why doing *path; and not *path = NULL [17:39] dobey, what is the point there? [17:41] ah, i don't know why he removed that initialization to NULL [17:41] likely because it's not there above [17:42] but i think the initialization isn't entirely necessary there [17:43] not now, but if the code ever chan [17:43] ges [17:43] .. [17:44] I realize I'm a c noob [17:44] but precisely for that reason, maybe, I prefer consistently initializing all pointers [17:44] well hopefully after precise, the code will definitely change === salgado-lunch is now known as salgado [17:47] dobey, thisfred the problems is that the docs http://developer.gnome.org/glib/2.30/glib-URI-Functions.html#g-filename-from-uri do not mention what is returned from the function if there was an error [17:47] mandel: yes it does. [17:48] Returns : a newly-allocated string holding the resulting filename, or NULL on an error. [17:48] "or NULL on an error." :) [17:48] dobey, sorry, then there is no need to init the path to NULL [17:48] dobey, I'm approving it then :) [17:52] after upgrading and rebooting, 2.0.3 is still here, and it seems to now be broken. its in an odd state where a lot of files are missing (like python libs) and it won't start [17:52] the dist folder only has about half of the files it usually has [17:53] the next thing is going to be to create a 3.0.1 installer, put it somewhere, and check that 3.0.0 can properly upgrade, but i dont think it can [17:53] briancurtin, that is after an update that cannot stop the process? [17:53] mandel: it doesn't make any attempt to, as far as i know [17:54] hmm [17:54] briancurtin: do you get *any* upgrade progress window? [17:54] when i ran the 2.0.3 and chose to upgrade, CP just stayed there in the background the whole time, the upgrade went along and did its thing, then it was just done and nothing happened. then i rebooted and now its screwed [17:54] please don't make it 3.0.1 [17:54] dobey: i just meant locally, to force an upgrade [17:55] briancurtin, we should take a look at the logs that bitrock leaves.. if any [17:55] ralsina: i do get the upgrade progress window, i see that its upgrading...but its either just silently failing to upgrade because the files in use, or something that it's not telling me about. it "looks" like its working just fine, and theres no indication that it didn't succeed (or fail), it just ends like normal [17:55] mandel: i'll see what they say. it does leave an install log [17:55] briancurtin: you should have an update log somewhere [17:57] mandel, ralsina do you have time for a Qt-related review? https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/checkwrap/+merge/102152 [17:57] gatox: sure [17:57] ralsina, thanks [17:57] gatox: although I want pictures! [17:57] shit, i might have just sent us on a wild goose chase for no reason... [17:57] ralsina, uploading picture [17:57] gatox, reviews day.. [17:58] gatox: thanks :-D [17:58] mandel: I am co-reviewer for the day [17:58] mandel: so we are supposed to both have time :-) [17:58] mandel: can you put a file up on your server for me? i think this whole thing was "upgrading" to an older version from your website [17:59] actually i can probably do it myself, hold on [18:01] ralsina, screenshot uploaded [18:01] gatox: awesome, thanks [18:03] i have a lint issue.... fixing it [18:06] gatox, super stupid question, why is that better than using a checkbox and a label that has its text set to wrap? [18:06] gatox, I mean, I don't really know how was the word-wrap implemented in Qt, but it might be more complicated than doing that.. [18:07] briancurtin, wait, you used the url in my server? [18:07] mandel, using a separated qlabel + a qcheckbox has issues about the focus style and other stuff [18:07] briancurtin, it is using an much older version than 3.0 [18:07] mandel: yeah :/ i'm changing it now... [18:07] mandel yeah i know, i screwed that up [18:08] briancurtin, mea culpa I did not tell you [18:08] * ralsina wonders if the update works with file:// urls [18:08] just to make it testable, you know [18:08] ralsina: i can check that out after straightening this out [18:09] briancurtin: just idle curiosity but maybe handy in the future :-) [18:09] ralsina: it would help my blood pressure if we had it automated [18:09] hehe [18:10] briancurtin: so let's do that then (evetually :-) [18:12] ralsina, mandel i've updated the branch..... just in case you are going to run tests, i fix a lint issue [18:12] gatox: thanks [18:12] I am running Ubuntu 11.10 fluxbox (not unity) in a Vbox VM with 1G and 2 virtual CPUs at 60%. It is causing my other guests to crash and slows the entire system (hardware) to a crawl. [18:12] at first, I thought it might be unity. But if I am running fluxbox, that wouldn't be it, right? [18:13] thereeeee we go. i just upgraded 2.0.3 to 3.0.0. one thing i did manually, though, was stop the old SSO and SD. rebooting obviously takes care of that, but if you dont want to reboot, you'll need to stop those manually [18:13] ubuntu, you crazy OS, why are you opening PDFs with the freaking gimp??? [18:13] could it be ubuntuone? [18:13] gatox, what do you mean? what is wrong with http://paste.ubuntu.com/932898/ [18:13] ralsina: why did you set it to the default? [18:13] bozonius: u1 is known to have high CPU usage on startup on systems where it syncs many thousand files. Is that the case? [18:14] dobey: I most certainly have never ever touched the defaults for PDF opening in the computer. [18:14] 5 small files + 1 2.7GB file. [18:14] bozonius: ok, it will have to read the whole 2.7GB file to calculate its hash [18:14] briancurtin, hurray! that is reall good news, a sit my pants a little when I heard it was not working [18:14] bozonius: so that may casuse high IO load [18:14] can't u1 do that quietly? [18:15] bozonius: it's supposed to :-) [18:15] why is it being such a pig [18:15] ralsina: did you remove evince? [18:15] dobey: I never liked, installed or used evince [18:15] ralsina: evince is installed by default [18:15] dobey: not when you start from kubuntu :-) [18:15] is it u1, or is it Ubuntu that pigs out on memory? [18:15] ralsina: so yes, you did it. :) [18:16] bozonius: what does 'top' say it is? [18:16] one guy suggested allocating 3G of my 4G just to run unity and u1 [18:16] mandel, it was decided not to use custom checkbox everywhere [18:16] dobey: on host or guest? [18:16] or both? [18:16] gatox, ok, code looks ok, but there are things that scare me.. [18:16] mandel, also, this function is not just for checkbox, but for every widget with the same problem [18:16] well i'm guessing the host doesn't know what processes are running in the guest [18:17] mandel, like what? [18:17] but both is probably useful if you're wondering about what is using cpu and memory [18:17] sure, dobey, but at least the vbox process driving the VM might offer some light? [18:17] (on host) [18:17] gatox, wrapping text scares me as much as utf8 [18:18] gatox, it has nothing to do with your code, so I'll approve.. [18:18] mandel, i've tried to reproduce the same behaviour of the qt wrapping for qlabels [18:19] dobey: I am deciding whether this is a u1 problem or a vbox problem. I didn't have these sorts of issues before the latest upgrade of vbox. Maybe I am bothering the wrong people? [18:19] bozonius: you could just see how much memory u1 is using [18:19] I mean, the fact that u1 or unity might be a cpu and ram pig shouldn't really make a difference to the host. [18:19] bozonius: then we would know better where the blame lies [18:19] bozonius: you need to look at the data to determine what the problem is. asking for some opinions on irc won't answer that question :) [18:20] you are right dobey and ralsina [18:20] I guess I have some homework to do first. I just hoped maybe someone else had already conquered this problem and I could get the lowdown. [18:20] Not this time I suppose. [18:21] I am tending toward the vbox upgrade, though. I don't see why Ubuntu pigging out on some virtual resources should cause ANOTHER VM to crash! [18:22] I will start there. Thanks for the help. I'll let you all know if/what I find out. [18:22] btw, I like Unity and u1! That's why I am here. [18:23] bozonius: vbox is sensitive to the host running out of resources [18:23] bozonius: at least has been in the past for me. So sometimes 2VMs going "heavy"=> both crashing [18:23] bozonius: if the vm is actually crashing, it's probably not something running inside it which causes it. probably an issue in vbox itself, or you're running too many VMs [18:31] one thing to be aware of with people upgrading: if they upgrade and do not restart and do not stop the existing SSO & SD, they will get "RemoteError 'remote_register_to_signals() takes exactly 2 arguments (3 given)'" [18:32] perhaps we should add some integrity check that SD and CP versions match, in order to give a better indication that this is the problem [18:32] in any case, i think we're ready to flip on the new update.xml [18:37] briancurtin, makes sense.. I think the update is a little cripple in that sense [18:38] mandel: well we could fix it for 3.0.0 to 3.x.y, but what's done is done for the 2.0.3 upgrade [18:39] briancurtin, indeed, is all we could squeeze this release [18:40] ralsina: should i have l-sa flip the switch and send the update.xml to be live? [18:41] briancurtin: let's wait until your EOD today. If there is no sudden influx of windows bugs, let's push it [18:42] ralsina: is there any accompanying news items or stuff we need to update to let people know it's ready? [18:42] briancurtin: no [18:42] briancurtin: not yet at least [18:42] briancurtin: now that I think about it... this is called 3.0.0 we should make some noise ;-) [18:44] ralsina: is there any official place to make that happen? like 6 people read my blog so it wouldnt do much good to post things there [18:46] briancurtin: I just notified the right people, they will do what's needed [18:46] cool, sounds good [18:46] briancurtin: I am suspecting that this being 3.0 is a total surprise to everyone in marketing [18:48] ralsina, briancurtin http://www.youtube.com/watch?v=0irL1M15DH8 [18:49] hahaha [18:55] alecu, ping? [19:00] mandel, pong [19:00] alecu, shall we mumble? [19:00] mandel, sure [19:16] ok, EOD for me, catch you all tom! [19:18] i'm unable to authorize to U1 using windows client [19:20] i'm 100% sure that i didn't mistyped mail, since i've just logged in and copied it from web interface, but 'reset password' says, it doesn't recognize my mail [19:21] mandel, ralsina: Today was a lot of catching back up and doing a little reintegrating. I'll be ready to get back in full swing tomorrow. [19:21] (not EODing or anything, just letting you know) [19:22] urbanape: cool, and expected, thanks for the update [19:42] karni, urbanape: I know very little about the mobile clients, but I think you guys should tell this guy how U1 will solve his problems: http://tonywhitmore.co.uk/blog/2012/04/16/a-hard-tablet-to-swallow/?utm_source=rss&utm_medium=rss&utm_campaign=a-hard-tablet-to-swallow :) [19:45] also tell him that half of the screen is a picture of him when i view the page [19:48] dobey: looking [19:50] urbanape: can you download a directory of files with our iOS client? [19:50] nope [19:50] aha [19:51] it will end up working much like batch downloads on the music app. [19:51] karni: but you can auto-sync the phone's (tablet's) gallery, no [19:51] same gestural interface. [19:51] ? [19:51] dobey: yes [19:51] urbanape: do UDFs auto-sync as well? [19:52] dobey: you mean, auto upload. the guy wants to get his files *onto* an iDevice, not from it. [19:52] or can they optionally if the user says "sync this directory" rather [19:52] not sure what he's talking about. Most cameras also have a USB mass storage setting. [19:52] I offloaded my DSLR pics from vacation onto my iPad. [19:52] Left my laptop at home. [19:52] urbanape: Re: gestures - right. [19:53] urbanape: ipad has a usb slot now? [19:53] dobey: no, the camera connector kit. [19:53] so, sorta [19:53] right [19:53] 30-pin on one end, and USB on the other. [19:53] the thing he mentioned not having, and not really wanting to pay $50 for [19:54] dobey: quality of rendering may indeed be influenced by the gallery app itself, may use filters to speed things up a little. I've no clue how to get them on iPad, though, apart from tapping one by one, from some U1 directory. [19:54] anyway, his use case seemed like something u1 would be perfect for [19:54] right. we just don't have downloading/syncing down a directory on iOS just yet. [19:55] we do on Android, but it was much faster for the guy to simply load up an SD card with those pics. [19:55] true [19:55] once we have the new gesture stuff in place, this will be piece of cake in both iOS and Android apps [19:55] on Android, currently it's in the context menu of a directory. [19:56] and honestly, it might be kinda nice to do up a slideshow option. [19:56] but he could pre-fill the directory on android then turn on u1 for that folder and let it 'merge' (hopefully avoiding any conflicts and downloads/uploads), and then will be much nicer for individual photo swaps probably [19:56] urbanape: :) [19:56] though, just having it on a sd card is probably faster [19:56] you could display the first, and be downloading the subsequent ones in the background, and auto transition [19:56] I was never really sure why we needed the photos app to be a separate app. [19:56] Seems a natural fit function for the files app. [19:57] urbanape: indeed [19:57] agreed [19:57] urbanape: well, nobody said it *has* to be a separate app [19:57] the mockups from patricia are just on paper at this stage [19:57] Like Robert said, we're not touching that yet. [19:57] And I agree it's a perfect fit for the files app. [19:58] I know we were/are considering if it should be separate or not, though. [19:58] and fewer apps probably will make the app review stage of releasing easier as well [19:58] dobey: +1 [20:00] hrmm, i really need a space suit like they have in modern sci-fi shows/movies [20:12] ralsina: care to review https://code.launchpad.net/~dobey/ubuntuone-client-gnome/fix-983144/+merge/102173 ? [20:16] dobey: sure! [20:19] dobey: I think I remember that you should not use a NULL to force a traling slash, but use a slash instead. At least that's the way it is for g_build_path (ad that should be g_build_path in line 15, right?) === Dr_Who is now known as tgall_foo [20:20] dobey: although reading the doc is like reading Lewis Carroll [20:20] 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. (Determination of the number of trailing copies is done without stripping leading copies, so if the separator is ABA, ABABA has 1 trailing copy.) [20:20] ralsina: that is exactly the same way we do this elsewhere. and the final NULL doesn't force a slash, it denotes the end of the arguments list [20:20] G_DIR_SEPARATOR_S is the trailing slash (or backslash on windows) [20:20] dobey: oh, right, thanks [20:21] * ralsina forgot his variable argument conventions [20:22] * dobey can't wait to throw all this code away and rewrite it in vala [20:23] dobey: wouldn't that code look exactly the same in vala? [20:24] dobey: +1 [20:30] ralsina: no. possibly close though. and i would probably not reintroduce that code in the same manner [20:31] dobey: does vala have any decent (as in not-from-c) string management ability? [20:31] dobey: that's just idle curiosity, because it's my main C gripe [20:31] ralsina: it has some better abstractions, yes. [20:32] dobey: cool [20:32] dobey: I may have to learn vala after all [20:48] eod here....... and stop fighting with wrapping and size problems...... byeeeeeee, see you tomorrow [20:54] eod for me too, I'm afraid [20:54] I will do an extra hour late tonight, ping me about review requests or whatever [20:55] ralsina: have a good EOD, i will ping l-sa in a while to get the update.xml changed [20:57] briancurtin: did you get the update install issue figured out btw? [20:59] dobey: yep, it's figured out as best it can be. upgrading from 2.0.3 to 3.0.0 works with the caveat that you either need to restart after it's done, or you need to stop CP then SSO+SD, then restart CP and it'll be the new version [21:01] ah ok. cool [21:18] * Dr_Who ponders === thisfred is now known as inspector_spacet === inspector_spacet is now known as inspctr_spacetim [21:26] bah [21:28] haha === inspctr_spacetim is now known as thisfred === salgado is now known as salgado-afk [21:56] later all. have a good evening