/srv/irclogs.ubuntu.com/2012/04/16/#ubuntuone.txt

jo-erlend__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 Activit06:11
jo-erlend__y Journal that should be fixed?06:11
=== OutOfControl is now known as benonsoftware
JamesTaitMorning all! :D08:58
andtorghi09:04
andtorgjust asking before doing damage: is it possible to move Ubuntu One folder elsewhere and replace it with a symlink in my /home?09:06
vdsandtorg, I would check with mandel but I guess it is possible, stop the sync daemon first09:17
vdsthat's timing! :)09:17
vdsmaybe rye ? :)09:18
andtorglol, thanks vds09:18
=== gord_ is now known as gord
andtorgvds: just to let you know, it worked10:00
vdsandtorg, cool10:00
=== yofel_ is now known as yofel
mandelvds, 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 :)10:59
gatoxgood morning!11:08
vdsmandel, yep11:10
mandelgatox, morning!11:11
gatoxmandel, hi11:11
mandelgatox, I'm in my review day, can you please update the following mp: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/10098411:11
* gatox looking....11:11
mandelgatox, so, about the mp, will you update it?11:26
gatoxmandel, yes..... i'm changing that.....11:26
alecuhello!12:11
mandelalecu, morning!12:13
mandelalecu, I have updated the webserver branch here: https://code.launchpad.net/~mandel/ubuntuone-dev-tools/mocked-webserver/+merge/10122412:13
alecumandel, awesome, looking.12:13
mandelalecu, 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
gatoxalecu, hi12:14
alecumandel, got links to those branches?12:15
mandelalecu, 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/8275412:15
mandelalecu, 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 dobey12:15
mandelalecu, let me grab a list12:16
mandelalecu, I think this is the only one: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 and gatox is making the required changes12:17
alecumandel, yes, that branch only needs a fix that nessita requested. After that you may do the re-review.12:19
mandelalecu, ok, great!12:19
gatoxmandel, please review this: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/10098412:22
gatoxlet me know if that is ok12:22
gatox(about the implementation... the code is working)12:22
mandelalecu, FYI I've talk with nessita, I'll push people around to update their branches and will let her know when we can re-review12:22
mandelgatox, I'll do it after lunch, is that ok?12:27
gatoxmandel, no problme12:27
mandelgatox, ok!12:27
* mandel lunch12:27
gatoxalecu, mandel, it seems i don't have review day12:28
mandelgatox, monday!12:28
mandel:P12:28
gatoxmandel, that's yours :P12:28
gatoxalecu, ?12:28
mandelgatox, I know hehehe12:28
duanedesign6/1412:31
ralsinagood morning!12:39
ralsinagatox: you have to claim your review day on the calendar, I think12:40
gatoxralsina, morning..... ok..... but monday to friday is already taken recursively.... that's why i was asking :P12:44
ralsinagatox: pick a day, and that day there's 212:44
gatoxralsina, ok!12:44
ralsinagatox: probably friday is a good idea since it's release-freeze day12:45
gatoxralsina, good12:45
ralsinagatox: and since I am the one that does second reviews all days, I spend fridays on other stuff12:45
ralsinagatox, mandel, alecu: I have a doctor's appointment i 112:48
ralsinaargh12:48
ralsinain 1:45 so I will miss standup12:48
gatoxralsina, ack12:48
ralsinaI will stay late to compensate12:48
ralsinaIn other news, 117.0 and still losing weight as planned :-)12:50
ralsinaalthough half the time I could eat a horse12:50
gatoxralsina, good!! I'm trying to do the same.... not success yet12:50
ralsinagatox: did you go see a doctor?12:50
ralsinagatox: it does make a difference12:51
Chipacaralsina: guttenmorgenflerfen!12:51
ralsinaHello Chipaca!12:51
gatoxralsina, no, i was just eating healthy and doing a lot of exercise12:51
Chipacaralsina: ping me when you've finished booting :)12:51
ralsinagatox: eating healthy pigs is not enough12:51
ralsinaChipaca: well, about now is good :-)12:51
urbanapemorning, all!13:09
gatoxurbanape, hi13:09
ralsinamorning urbanape!13:18
ralsinaurbanape: how was vacation?13:18
ralsinaurbanape: while you were gone, we finished the macport and put you in charge of the Amiga port.13:19
ralsinaurbanape: your Amiga 500 + external floppy will be arriving shortly13:19
urbanaperestauraciĆ³n13:19
urbanapewoohoo!13:20
ralsinaurbanape: on real news, brian is going back mostly to windows and mandel is going to help you more13:20
urbanapeand alecu?13:21
ralsinaurbanape: alecu as well13:21
ralsinaurbanape: and gatox can do reviews every now and then13:22
urbanapeexcellent13:22
ralsinaurbanape: but alecu is new tech lead while natalia is on rotation, so he's not going to do aything fulltime13:22
urbanaperight right13:22
ralsinaurbanape: also, we instated review days13:24
urbanapeI saw the calendar email13:24
ralsinaurbanape: right, pick a day, you will have to share though13:24
ralsinaurbanape: if I may suggest tuesday, it's my busiest day, and would free me some13:25
urbanapesure, that sounds good13:25
dobeymandel: hmm?13:27
alecuralsina, 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 #33475713:28
ubot5Launchpad bug 334757 in KDE Bindings "update-notifier-kde.py crashed with SIGSEGV in QSocketNotifier::setEnabled()" [High,Confirmed] https://launchpad.net/bugs/33475713:28
ralsinaalecu: saw the emails on PyQt, congrats :-)13:28
dobeyalecu: i was just about to ping you about that13:28
alecuIt was a *very* old bug.13:28
dobeyalecu: can you test the fix that's in precise-proposed?13:29
alecudobey, sure. How should I install the package in precise-proposed?13:29
alecudobey, is it a ppa?13:30
dobeyalecu: no. you can enable it in the "Updates" tab of the software sources config dialog13:30
dobeysoftware-properties-gtk13:30
alecuawesome, thanks!13:30
dobeyalecu: be careful though as there may be other updates in there, which might break stuff :)13:31
alecuright :-)13:31
ralsinaalecu: there is another fix that's coming in PyQt soon about disconnect, but we have workarounds in place for that one13:35
alecuralsina, yup, it's usually harder to find workarounds for SEGFAULTS in python :-)13:35
ralsinaalecu: hehe13:36
ralsinaalecu: do we still want to put faulthandler in main? I can ask, you know.13:36
alecuralsina, I'm checking the proposed pyqt packages, and if they solve it we may not need it.13:37
ralsinaalecu: cool13:37
* mandel back13:41
mandeldobey, you need me?13:41
ralsinaAnyone needs any reviews? And if you don't whathave you all been doing????? ;-)13:43
mandeldobey, 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:43
mandelralsina, che, todays is my review day, don't steal my job!13:44
ralsinamandel: I am on review day every day (except tue and fri)13:44
* ralsina is the goalkeeper of reviews13:44
dobeymandel: ignore them for now13:46
dobeyralsina: i've been trying to enjoy the weekend, thank you very much :P13:47
mandeldobey, 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-client13:47
ralsinadobey: good :-)13:47
dobeymandel: right, i saw those and was going to look at them today :)13:47
mandeldobey, sweet, I did one of them, will move to the second one in a few mins13:48
mandeldobey, I also updated the ubuntuone-dev-tools one (mocked-webserver) removing the code duplication that you mentioned13:49
mandeland talk with nessita about the comments you added for an MP to ubuntuone-dev-tools, dobey she told me she will chat with you13:49
dobeyyes13:52
Dr_WhoHi urbanape and ping13:59
urbanapepong14:06
mandelDr_Who, ^14:06
Dr_WhoHi 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 it14:08
urbanapeDr_Who: yup. I'm the primary (now) developer on both14:08
Dr_Whoah cool !14:09
Dr_WhoI'm just working on things in my spare time but happy to help out14:10
Dr_Whoon the music app, I've got an initial implementation to move the settings into the iOS settings app14:10
urbanapeWe started that way, but there's no way using the normal settings bundles to do conditional disclosure of settings.14:10
Dr_Whoas 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 one14:11
urbanapeApple obviously can, but 3rd party devs are limited to a single plist describing the settings.14:11
urbanapeNot sure what you mean14:11
urbanapethe policy is this: Cache everything as it's played, up to being within 10% of the device's storage remaining. Purge LRU.14:12
ralsinaI'm off to the doctor, so he can tell me I am still fat. Will be back in about 90 minutes14:12
=== ralsina is now known as ralsina_doctor
urbanapetracks explicitly downloaded (pinned via the buttons on album view) are exempt from both storage limits and LRU purge.14:12
Dr_Whoso 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 resumed14:12
Dr_Whoah ok14:13
Dr_Whoso 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 in14:13
Dr_Whowell it used to ? I modified that slightly :-)14:14
urbanapeWhat's the use of it not talking to U1?14:14
urbanapeI mean, if you've got tracks downloaded, it works just fine in offline mode (airplane mode)14:14
Dr_Whoif 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 play14:14
Dr_Whoright14:15
urbanape(or in No Service areas)14:15
Dr_Whoexactly14:15
urbanapeit does that. You can't navigate to music that doesn't exist locally if you're offline14:15
Dr_Whoright and not suggesting that you should be able to either14:15
Dr_Whotho I am suggesting that signing out of u1 == airplane mode or lack of wifi or whatever without having to turn on airplane mode for instance14:16
Dr_WhoI think we're on the same page14:16
urbanapeMaybe. Is the point just to keep the app from talking to the server?14:17
Dr_Whoeffectively14:17
urbanapeusing network? I guess if you've got a really, really horrible data plan, that could save you a couple HTTP requests.14:17
Dr_Whoand being signed out would keep your selection of music confined to what you have locally14:18
urbanapeyeah, except we don't purge the Core Data entities just because we can't reach the server.14:19
urbanapeSo, there's still gonna be all the greyed out entries in Artists, Albums, and Songs14:19
Dr_Whoyou 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 list14:21
Dr_WhoI'd suggest if you're signed out or without network, there's not much sense in displaying them14:21
Dr_Whotho I suppose that could have a downside if you're on an bad network14:21
Dr_Whoanyway all that said, is there some areas that would be quite helpful to you ?14:22
urbanapeI've got one outstanding bug about resuming the auto-download queue when reacquiring network connectivity14:23
urbanapeso, probably need to have something triggered on the Reachability events that starts it back up.14:23
Dr_Whoo that one?14:24
urbanapehttps://bugs.launchpad.net/ubuntuone-ios-music/+bug/97257414:24
ubot5Ubuntu bug 972574 in Ubuntu One Music App for iOS "Problem with Caching Songs - moving from online to offline mode" [High,Triaged]14:24
Dr_Whothere's one bit of detail there,  I added some code to address it14:24
Dr_Whoo nm  that's not the one I'm thinking14:24
Dr_Whoah ok ?  sure I'll see what i can find14:25
Dr_Whothe 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 functioning14:26
Dr_Whoanyway I added a bit of logic and a UIAlert or two to instruction users how of the situation and how to correct14:26
Dr_Whothat's a different topic however14:26
mandelgatox, +1 for https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/10098414:28
urbanapebleah. Yeah, I hate that whole aspect.14:28
urbanapeit would be better that if you disallowed it, you'd just get those stripped out of the EXIF data.14:28
urbanapebut that's not up to me14:28
Dr_Whoyeah agreed,  it's a bummer of a design14:31
mandelbriancurtin, ping14:32
gatoxmandel, yeyyyyy14:32
briancurtinmandel pong14:32
mandelbriancurtin, 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
briancurtinmandel yeah i'll see if i can figure that out14:33
mandelbriancurtin, I think that ensuring that the argparse gets the correct params should be ok, but I might be wrong14:34
mandelbriancurtin, let me know when done and I'll get you the second review you need14:35
briancurtinmandel: the change in tunnel_runner shouldn't be there, i probably pushed it there by accident. i'll look to fix that typo part though14:36
dobeymandel: what he heck is "iri" ?14:36
gatoxmandel, ahhhhh but i still need alecu and nessita's approval14:37
mandeldobey, http://en.wikipedia.org/wiki/Internationalized_Resource_Identifier14:37
mandelalecu, can you abstain in gatox branch?14:38
dobeymandel: but why do we need a "get_iri()" here? and why a get_uri() that just returns the same thing?14:38
mandeldobey, one returns a unicode based url, the reason is that the webclient from ubuntu_sso expects all its urls to be unicode14:40
dobeythey both return a unicode object14:40
joshuahooverbriancurtin: 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/release14:40
dobeyoh, no i see you do bytes()14:41
dobeyugh14:41
briancurtinjoshuahoover: its not. waiting on the people who have access to do that to carry out the upload14:41
mandeldobey, yeah: bytes(iri2uri(iri))14:41
joshuahooverbriancurtin: k, but it's all ready to be uploaded to the site, correct?14:41
briancurtinyep, we're in a queue for an admin to execute that14:42
briancurtinjoshuahoover: 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.xml14:43
joshuahooverbriancurtin: right, got ya...and this version is 3.0.0, correct?14:43
briancurtinjoshuahoover: correct14:43
joshuahoovercool, thanks!14:43
dobeymandel: 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 ascii14:44
mandeldobey, iri2uri will return unicode, how would you prefer to pass it to bytes?14:46
dobeymandel: we shouldn't pass it to bytes14:46
dobeymandel: think about py3 for example :)14:46
mandeldobey,  urllib2 likes bytes, as well as twisted :(14:47
dobeymandel: even on python 3?14:47
mandeldobey, if you do an HTTPClientFactory with something that is not bytes you get an error, I think is the same with libsoup14:48
mandelqt is fine because it uses QStrings14:48
dobeymandel: 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
alecudobey, I've installed pyqt from proposed, and my script is not segfaulting anymore.14:49
dobeyalecu: hooray!14:49
mandeldobey, so you propose that I go though every instance of get_uri used in the tests and call the iri2uri there?14:50
dobeymandel: no14:50
dobeymandel: 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:51
mandeldobey, when we pass to py3 we would brake, how would you do it?14:52
dobeymandel: i would start by never returning bytes(somethingelse)14:54
mandeldobey, so using decode, is that what you mean?14:57
dobeyno14:57
dobeywell, 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
dobeyi think that's the right way to do that in python14:58
dobeyi think we need to expose and understand all the cases where unicode doesn't work first, though14:59
dobeyand i don't think that is well known14:59
mandeldobey, well, I know twisted does not like it, urllib2 and libsoup15:00
briancurtinme15:00
urbanapeme15:01
mandelme15:01
mandeldobey, 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-client15:02
mandeldobey, 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 twisted15:02
mandelralsina_doctor, gatox, alecu, dobey  stand up?15:03
gatoxme15:03
alecume15:03
dobeymeh15:05
alecubriancurtin, go15:05
briancurtinDONE: signing stuff, hacking on the build_installer.py script to work around some buildout oddities15:05
briancurtinTODO: fixing the upoad-spellcheck branch15:05
briancurtinBLOCKED: return None15:05
briancurtinNEXT: urbanape15:05
urbanapeDONE: vacation TODO: Catch up, merge some of mandel's branches, push the proposal, BLOCK: NO LONGER ON VACATIOn15:05
urbanapemandel: please15:06
mandelReviews Day:15:06
mandelDONE: 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 upda15:06
mandelte from nessita due to comments form dobey,15:06
mandelhttps://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/100984 - is blocked due to needed reviews form alecu and nessita15:06
mandelTODO: Reviews as needed, finish fixed for mocked-webserver (talking with dobey about it). Review urbanapes mac branch.15:06
mandelBLOCKED: no15:06
mandel3. Branches that I cannot review (they are mine):15:06
mandel    https://code.launchpad.net/~mandel/ubuntu-sso-client/remove-duplication/+merge/10172415:06
mandel   15:06
mandel    15:06
mandelgatox, go15:06
mandelstupid tomboy paste..15:06
gatoxDONE:15:06
gatoxSwap 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
gatoxTODO:15:06
gatoxFinish with wrapping tests and propose, keep fixing ui stuff.15:06
gatoxBLOCKED:15:06
ubot5Launchpad bug 824574 in Ubuntu One Control Panel "QT UI scroll bar overlaps list area bottom border" [Low,In progress] https://launchpad.net/bugs/82457415:06
gatoxNo15:06
gatoxalecu, go15:06
alecuDONE: a script to reproduce the pyqt segfault which got fixed upstream, some reviews, making plans for UDS15:06
alecuTODO: get back to bugfixing15:06
alecuBLOCKED: no15:06
alecuNEXT: gatox15:06
gatoxalecu, no.... i already did my standup15:07
dobeyĪ» DONE: reviews, bug #682954, bug #95640715:07
dobeyĪ» TODO: SRUs, finish reviewing mandel's branch, couple small reviews, u1db packaging15:07
ubot5Launchpad 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/68295415:07
dobeyĪ» BLCK: none.15:07
ubot5Launchpad bug 956407 in Ubuntu One Client stable-1-6 "Apport recipe attaches old/useless log files" [Low,Triaged] https://launchpad.net/bugs/95640715:07
alecusorry15:07
alecuNEXT: dobey15:07
dobeythisfred: ?15:07
thisfredme15:07
thisfredDONE: test improvements u1db TODO: same + start work on u1db example app? BLOCKED: no15:08
briancurtinis Harry around?15:10
alecuhe's meeting Sally.15:10
alecubriancurtin, was the installer finally signed by the l-sas?15:10
briancurtinalecu: it's signed, just waiting on them to do the actual upload. apparently they're a bit backed up this morning15:11
alecubriancurtin, awesome anyway.15:12
briancurtinmandel: 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&ltyp=AllFixedPriceItemTypes&var=sbar15:12
alecubriancurtin, is that the installer with the fixed autoupdate?15:12
mandeldobey, so, what do we do with that bytes, also you whould know that a similar code is present under ubuntu-sso-client webclient15:12
mandelbriancurtin, haha great idea!15:13
mandelalecu, you mean the re-added autoupdate?15:13
gatoxbrb....... reboot15:13
alecumandel, that, right :-)15:13
mandelalecu, yes, should have been merged in trunk for 3.0.0 AFAIK15:13
alecubriancurtin, and I thought the Chuck Norris plugin for jenkins was cool. Not anymore!15:14
briancurtinalecu: yep, that's in there. it's stable 3.0 plus one hacky branch to allow the Cloud To Computer dialog on Windows to work15:14
dobeymandel: i'm not entirely sure. maybe barry has some idea about how to handle that compatibly for both py2 and py315:14
dobeymandel: i also needs fixing'ed your branch again, for string format stuff. we should use .format() notation for all new strings now15:15
mandeldobey, ein, are we already moving all the way to python 3?15:15
alecudobey, briancurtin, all: do we need anything else from the l-sas these days?15:15
alecuralsina_doctor, too15:15
mandelalecu, not from me15:15
alecucheepaca was asking us about that.15:16
briancurtinalecu: 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
mandelalecu, ein? is that about my python 3 comment or losas?15:16
dobeyalecu: i don't know of anything other than the windows stuff.15:18
mandeldobey, 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
alecubriancurtin, right.15:19
* gatox lunch!15:19
dobeymandel: we need to try and avoid creating more work for us in the future when we do make python3 the default15:19
mandeldobey, same with exceptions then, Exception as e ?15:19
dobeymandel: did something change for exceptions for py3?15:20
alecubriancurtin, do we have any checks planned for the time between the file goes up, and the time update.xml is changed?15:20
alecumandel, we can already start using "Exception as e" in 2.7 afaik15:21
mandeldobey, yes: http://docs.python.org/release/3.0.1/whatsnew/3.0.html#changed-syntax15:21
alecumandel, I think we can already use it in lucid, but lemme check.15:21
dobeyalecu: no lucid is python 2.615:21
dobeyso if it's only since 2.7, that's a problem15:21
briancurtinalecu: 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 tomorrow15:21
dobeyand lucid is an old 2.6 also15:21
dobeylucid doesn't have a new enough python to use dh_python2 even :(15:22
mandeldobey, alecu is the following pep: http://www.python.org/dev/peps/pep-3110/15:22
briancurtindobey, mandel: 2.6 has the "Exception as e" syntax15:23
mandelbriancurtin, thx!15:23
alecubriancurtin, 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:23
briancurtinalecu: 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 happens15:25
alecubriancurtin, 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
alecubriancurtin, right, but probably an md5 check is all we need :-)15:25
briancurtinalecu: 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 synced15:26
=== gatox is now known as gatox_lunch
briancurtins/u1 file/u1 folder/15:26
dobeymandel: 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 can15:27
mandeldobey, ok, I suppose we have to start somewhere :)15:28
mandelalthough I would have liked to land this before that hehehe15:28
dobeymandel: better to fix it now, than to have to do it in 3 months when python2 gets bumped out of default install :)15:29
alecubriancurtin, 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:33
alecubriancurtin, 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:34
briancurtinalecu: i will update the RT ticket and ask them to sign the update.xml - they haven't done that part yet15:35
alecubriancurtin, 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:35
dobeyok, off to get lunch. bbiab15:36
briancurtinalecu: 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:36
alecubriancurtin, right :-/15:37
alecubriancurtin, so, it's a great thing moving forwards with signing the xml15:37
alecubriancurtin, mandel: btw: say I want to IRL test the autoupdater... are the steps documented somewhere?15:38
briancurtinalecu: mandel wrote up some steps in email, i'll forward on15:38
mandelalecu, let me forward you the mail I sent to QA15:38
briancurtinmandel go for it15:38
mandelbriancurtin, I'll let you do it, you were first :)15:38
briancurtinsent15:39
alecubriancurtin, mandel: great, thanks!15:39
=== ralsina_doctor is now known as ralsina
ralsinabriancurtin: the update.xml is not signed, just deployed15:51
briancurtinoh it doesnt have to be signed? crap, i just posted to the RT about it...those people probably hate me by now haha15:51
ralsinabriancurtin, 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
ralsinabriancurtin: say "sorry, I'm new here" ;-)15:52
urbanapeIf we suffer, everyone has to suffer15:52
ralsinaurbanape: I am the manager. I am supposed to suffer in different ways.15:55
ralsinaFor example, these grapes I am being fed by slaves? Sour.15:55
alecuralsina, are you referring to number of steps or to the time that qt and pyqt take to compile?15:57
ralsinaalecu: both15:58
ralsinaalecu: ideally, we would someday have a scrip that takes exactly one step and ends in as little time as possible15:58
ralsinaalecu: in a perfect world: one command, one second. I understand we can't achieve that ;-)15:58
ralsinabut 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.15:59
ralsinalooking for a volunteer for bug #98314416:04
ubot5Launchpad bug 983144 in ubuntuone-client (Ubuntu) "Stopping sync of a UDF causes unrelated warning" [Undecided,New] https://launchpad.net/bugs/98314416:04
ralsinaI am betting on a bad path comparison (missing trailing /)16:04
briancurtinralsina: getting the 3.0 installer from the site now to check it out16:04
ralsinabriancurtin: cool, thanks!16:05
mandelbriancurtin, I'm getting the following when using the bootstrap on mac os x: paste.ubuntu.com/932699 any idea how to fix it?16:05
briancurtinmandel: hm, i have not seen that. i will take a look after i finish looking at this windows installer16:06
mandelbriancurtin, sure, no problem16:06
=== gatox_lunch is now known as gatox
mandelI need to walk the beast, I'll me back in 20 min16:11
=== salgado is now known as salgado-lunch
thisfredmandel: 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:30
urbanapegotta make a quick errand. Back in a bit.16:40
briancurtinwell upgrading doesn't really work because the upgrade is performed while the old u1 is running, so no files get updated...16:52
dobeyhmm16:52
briancurtinmandel: in the current autoupdate, does u1 get stopped while the autoupdater runs?16:55
briancurtini think people will have to manually upgrade from 2.0.3 to 3.0.016:56
briancurtinand 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.16:58
urbanapestupid phone calls. Now on quick errand.17:07
briancurtinralsina, 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 update17:09
dobeybriancurtin: +1 on that run updater separately and quit all running u1 processes17:10
ralsinabriancurtin: if the process is running the update will succeed but then not take effect until yu quit u117:10
ralsinabriancurtin: IIRC17:10
ralsinabriancurtin: but yes, in principle, that sounds reasonable17:10
briancurtinralsina: 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 effect17:11
dobeyralsina: except for the part where it can't replace files that are opened by another running process like say ubuntuone-syncdaemon, no?17:12
ralsinadobey: it can17:12
ralsinadobey: it moves the folder away somehow, then would require a reboot (normally)17:12
dobeyah ok17:12
dobeydoes the upgrader tell the user to reboot when it's finished?17:13
briancurtini didn't see any notice, but i'm trying it again to see if it does, then i'll reboot and see if it works17:13
ralsinadobey: my guess is it should, but have not tried it myself in a long time17:17
dobeyralsina: maybe it used to in the old code, but that got lost in the shuffle?17:19
ralsinadobey: no idea, really17:19
ralsinadobey: all that's supposed to be automaticly done by 3rd party code ;-)17:19
dobeyah17:19
ralsinaafter all, syncdaemon & co could easily be running under another session17:20
dobeyindeed17:20
dobeywe have a similar problem on linux17:20
ralsinaBTW, the exact same thing happens in ubuntu17:20
ralsinaright17:20
ralsinasyncdaemon should autodetect it has been updated and restart itself. Put a watch in his own binary :-)17:21
dobeyi should find some time to make the fix for that work17:21
ralsinadobey: yeah, would be nice to have17:21
dobeyyou can't just put a watch on the script file though17:22
briancurtinso 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
dobeyralsina: well i see what's wrong with bug #98414417:26
ubot5Error: Launchpad bug 984144 could not be found17:26
dobeyerr17:26
dobeyralsina: well i see what's wrong with bug #98314417:26
ubot5Launchpad bug 983144 in ubuntuone-client (Ubuntu) "Stopping sync of a UDF causes unrelated warning" [Undecided,New] https://launchpad.net/bugs/98314417:26
dobeyrather17:26
ralsinadobey: enlighten me17:26
dobeyno trailing /17:27
ralsinadobey: guessed as much when I called for volunteers. And because it has happened like 5 times before.17:28
ralsinadobey: so, you volunteering? ;-)17:28
dobeywell, i will fix it yes17:29
ralsinadobey: awesome, thanks17:29
mandelralsina, weird it used to work with no problems what so ever17:29
mandelbriancurtin,  weird it used to work with no problems what so ever17:29
mandelI need to change room, give me 5 min17:29
mandelor even less17:29
briancurtini 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 minutes17:30
=== mandel is now known as p
dobeyuh17:30
=== p is now known as Guest16683
=== Guest16683 is now known as mandel
dobeywhat the heck17:31
dobeymandel: 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 branch17:31
mandeldobey, I spoke with alecu, he agreed I could do the re-review.. but I guess I should have asked him to change the reviewer17:32
mandelbriancurtin, ralsina, so what is the problem with the update? it used to work and nothing has been changed17:33
ralsinamandel: "it used to work" and the whole autoupdater code has been moved around17:33
ralsinamandel: looks like since the binaries are in use, autoupdating may not do anything until you reboot17:34
mandelralsina, move around when?17:34
ralsinathe code was not in u1cp before, was it?17:34
mandelralsina, no it was not.. mierda17:34
ralsinamandel: don't worry, this is not horribly problematic17:35
ralsinamandel: if things update on reboot, that's as good as we had before17:35
mandelralsina, but it is a PITA17:35
ralsinamandel: there is no easy fix beyond a "reboot now" dialog17:35
mandelralsina, but, does the dialog tell you you have to reboot, or it just does nothing?17:35
ralsinamandel: even killing processes before updating is a partial fix17:35
ralsinamandel: don't know yet17:36
dobeymandel: were you going to review https://code.launchpad.net/~charlesk/ubuntuone-client-gnome/lp-981270/+merge/101993 as well?17:36
mandeldobey, 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 understand17:36
mandeldobey, is there?17:36
dobeymandel: i don't know what you do or don't understand :)17:37
dobeymandel: aside from women of course. you clearly don't understand them. :P17:38
mandeldobey, he, what did i do now?17:38
dobeyheh17:39
mandeldobey, I don't see why doing *path; and not *path = NULL17:39
mandeldobey, what is the point there?17:39
dobeyah, i don't know why he removed that initialization to NULL17:41
thisfredlikely because it's not there above17:41
dobeybut i think the initialization isn't entirely necessary there17:42
thisfrednot now, but if the code ever chan17:43
thisfredges17:43
thisfred..17:43
thisfredI realize I'm a c noob17:44
thisfredbut precisely for that reason, maybe, I prefer consistently initializing all pointers17:44
dobeywell hopefully after precise, the code will definitely change17:44
=== salgado-lunch is now known as salgado
mandeldobey, 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 error17:47
dobeymandel: yes it does.17:47
dobeyReturns : a newly-allocated string holding the resulting filename, or NULL on an error.17:48
dobey"or NULL on an error." :)17:48
mandeldobey, sorry, then there is no need to init the path to NULL17:48
mandeldobey, I'm approving it then :)17:48
briancurtinafter 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 start17:52
briancurtinthe dist folder only has about half of the files it usually has17:52
briancurtinthe 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 can17:53
mandelbriancurtin, that is after an update that cannot stop the process?17:53
briancurtinmandel: it doesn't make any attempt to, as far as i know17:53
dobeyhmm17:54
ralsinabriancurtin: do you get *any* upgrade progress window?17:54
briancurtinwhen 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 screwed17:54
dobeyplease don't make it 3.0.117:54
briancurtindobey: i just meant locally, to force an upgrade17:54
mandelbriancurtin, we should take a look at the logs that bitrock leaves.. if any17:55
briancurtinralsina: 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 normal17:55
briancurtinmandel: i'll see what they say. it does leave an install log17:55
ralsinabriancurtin: you should have an update log somewhere17:55
gatoxmandel, ralsina do you have time for a Qt-related review? https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/checkwrap/+merge/10215217:57
ralsinagatox: sure17:57
gatoxralsina, thanks17:57
ralsinagatox: although I want pictures!17:57
briancurtinshit, i might have just sent us on a wild goose chase for no reason...17:57
gatoxralsina, uploading picture17:57
mandelgatox, reviews day..17:57
ralsinagatox: thanks :-D17:58
ralsinamandel: I am co-reviewer for the day17:58
ralsinamandel: so we are supposed to both have time :-)17:58
briancurtinmandel: can you put a file up on your server for me? i think this whole thing was "upgrading" to an older version from your website17:58
briancurtinactually i can probably do it myself, hold on17:59
gatoxralsina, screenshot uploaded18:01
ralsinagatox: awesome, thanks18:01
gatoxi have a lint issue.... fixing it18:03
mandelgatox, super stupid question, why is that better than using a checkbox and a label that has its text set to wrap?18:06
mandelgatox, 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:06
mandelbriancurtin, wait, you used the url in my server?18:07
gatoxmandel, using a separated qlabel + a qcheckbox has issues about the focus style and other stuff18:07
mandelbriancurtin, it is using an much older version than 3.018:07
briancurtinmandel: yeah :/ i'm changing it now...18:07
briancurtinmandel yeah i know, i screwed that up18:07
mandelbriancurtin, mea culpa I did not tell you18:08
* ralsina wonders if the update works with file:// urls18:08
ralsinajust to make it testable, you know18:08
briancurtinralsina: i can check that out after straightening this out18:08
ralsinabriancurtin: just idle curiosity but maybe handy in the future :-)18:09
briancurtinralsina: it would help my blood pressure if we had it automated18:09
ralsinahehe18:09
ralsinabriancurtin: so let's do that then (evetually :-)18:10
gatoxralsina, mandel i've updated the branch..... just in case you are going to run tests, i fix a lint issue18:12
ralsinagatox: thanks18:12
bozoniusI 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
bozoniusat first, I thought it might be unity.  But if I am running fluxbox, that wouldn't be it, right?18:12
briancurtinthereeeee 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 manually18:13
ralsinaubuntu, you crazy OS, why are you opening PDFs with the freaking gimp???18:13
bozoniuscould it be ubuntuone?18:13
mandelgatox, what do you mean? what is wrong with http://paste.ubuntu.com/932898/18:13
dobeyralsina: why did you set it to the default?18:13
ralsinabozonius: u1 is known to have high CPU usage on startup on systems where it syncs many thousand files. Is that the case?18:13
ralsinadobey: I most certainly have never ever touched the defaults for PDF opening in the computer.18:14
bozonius5 small files + 1 2.7GB file.18:14
ralsinabozonius: ok, it will have to read the whole 2.7GB file to calculate its hash18:14
mandelbriancurtin, hurray! that is reall good news, a sit my pants a little when I heard it was not working18:14
ralsinabozonius: so that may casuse high IO load18:14
bozoniuscan't u1 do that quietly?18:14
ralsinabozonius: it's supposed to :-)18:15
bozoniuswhy is it being such a pig18:15
dobeyralsina: did you remove evince?18:15
ralsinadobey: I never liked, installed or used evince18:15
dobeyralsina: evince is installed by default18:15
ralsinadobey: not when you start from kubuntu :-)18:15
bozoniusis it u1, or is it Ubuntu that pigs out on memory?18:15
dobeyralsina: so yes, you did it. :)18:15
dobeybozonius: what does 'top' say it is?18:16
bozoniusone guy suggested allocating 3G of my 4G just to run unity and u118:16
gatoxmandel, it was decided not to use custom checkbox everywhere18:16
bozoniusdobey:  on host or guest?18:16
bozoniusor both?18:16
mandelgatox, ok, code looks ok, but there are things that scare me..18:16
gatoxmandel, also, this function is not just for checkbox, but for every widget with the same problem18:16
dobeywell i'm guessing the host doesn't know what processes are running in the guest18:16
gatoxmandel, like what?18:17
dobeybut both is probably useful if you're wondering about what is using cpu and memory18:17
bozoniussure, dobey, but at least the vbox process driving the VM might offer some light?18:17
bozonius(on host)18:17
mandelgatox, wrapping text scares me as much as utf818:17
mandelgatox, it has nothing to do with your code, so I'll approve..18:18
gatoxmandel, i've tried to reproduce the same behaviour of the qt wrapping for qlabels18:18
bozoniusdobey:  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
ralsinabozonius: you could just see how much memory u1 is using18:19
bozoniusI 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
ralsinabozonius: then we would know better where the blame lies18:19
dobeybozonius: 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:19
bozoniusyou are right dobey and ralsina18:20
bozoniusI 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
bozoniusNot this time I suppose.18:20
bozoniusI 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:21
bozoniusI will start there.  Thanks for the help.  I'll let you all know if/what I find out.18:22
bozoniusbtw, I like Unity and u1!  That's why I am here.18:22
ralsinabozonius: vbox is sensitive to the host running out of resources18:23
ralsinabozonius: at least has been in the past for me. So sometimes 2VMs going "heavy"=> both crashing18:23
dobeybozonius: 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 VMs18:23
briancurtinone 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:31
briancurtinperhaps we should add some integrity check that SD and CP versions match, in order to give a better indication that this is the problem18:32
briancurtinin any case, i think we're ready to flip on the new update.xml18:32
mandelbriancurtin, makes sense.. I think the update is a little cripple in that sense18:37
briancurtinmandel: well we could fix it for 3.0.0 to 3.x.y, but what's done is done for the 2.0.3 upgrade18:38
mandelbriancurtin, indeed, is all we could squeeze this release18:39
briancurtinralsina: should i have l-sa flip the switch and send the update.xml to be live?18:40
ralsinabriancurtin: let's wait until your EOD today. If there is no sudden influx of windows bugs, let's push it18:41
briancurtinralsina: is there any accompanying news items or stuff we need to update to let people know it's ready?18:42
ralsinabriancurtin: no18:42
ralsinabriancurtin: not yet at least18:42
ralsinabriancurtin: now that I think about it... this is called 3.0.0 we should make some noise ;-)18:42
briancurtinralsina: is there any official place to make that happen? like 6 people read my blog so it wouldnt do much good to post things there18:44
ralsinabriancurtin: I just notified the right people, they will do what's needed18:46
briancurtincool, sounds good18:46
ralsinabriancurtin: I am suspecting that this being 3.0 is a total surprise to everyone in marketing18:46
mandelralsina, briancurtin http://www.youtube.com/watch?v=0irL1M15DH818:48
briancurtinhahaha18:49
mandelalecu, ping?18:55
alecumandel, pong19:00
mandelalecu, shall we mumble?19:00
alecumandel, sure19:00
mandelok, EOD for me, catch you all tom!19:16
BlessJahi'm unable to authorize to U1 using windows client19:18
BlessJahi'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 mail19:20
urbanapemandel, 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
urbanape(not EODing or anything, just letting you know)19:21
ralsinaurbanape: cool, and expected, thanks for the update19:22
dobeykarni, 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:42
briancurtinalso tell him that half of the screen is a picture of him when i view the page19:45
karnidobey: looking19:48
karniurbanape: can you download a directory of files with our iOS client?19:50
urbanapenope19:50
karniaha19:50
urbanapeit will end up working much like batch downloads on the music app.19:51
dobeykarni: but you can auto-sync the phone's (tablet's) gallery, no19:51
urbanapesame gestural interface.19:51
dobey?19:51
urbanapedobey: yes19:51
dobeyurbanape: do UDFs auto-sync as well?19:51
karnidobey: you mean, auto upload. the guy wants to get his files *onto* an iDevice, not from it.19:52
dobeyor can they optionally if the user says "sync this directory" rather19:52
urbanapenot sure what he's talking about. Most cameras also have a USB mass storage setting.19:52
urbanapeI offloaded my DSLR pics from vacation onto my iPad.19:52
urbanapeLeft my laptop at home.19:52
karniurbanape: Re: gestures - right.19:52
dobeyurbanape: ipad has a usb slot now?19:53
urbanapedobey: no, the camera connector kit.19:53
urbanapeso, sorta19:53
dobeyright19:53
urbanape30-pin on one end, and USB on the other.19:53
dobeythe thing he mentioned not having, and not really wanting to pay $50 for19:53
karnidobey: 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
dobeyanyway, his use case seemed like something u1 would be perfect for19:54
karniright. we just don't have downloading/syncing down a directory on iOS just yet.19:54
karniwe do on Android, but it was much faster for the guy to simply load up an SD card with those pics.19:55
dobeytrue19:55
karnionce we have the new gesture stuff in place, this will be piece of cake in both iOS and Android apps19:55
karnion Android, currently it's in the context menu of a directory.19:55
urbanapeand honestly, it might be kinda nice to do up a slideshow option.19:56
dobeybut 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 probably19:56
karniurbanape: :)19:56
dobeythough, just having it on a sd card is probably faster19:56
urbanapeyou could display the first, and be downloading the subsequent ones in the background, and auto transition19:56
urbanapeI was never really sure why we needed the photos app to be a separate app.19:56
urbanapeSeems a natural fit function for the files app.19:56
karniurbanape: indeed19:57
dobeyagreed19:57
karniurbanape: well, nobody said it *has* to be a separate app19:57
karnithe mockups from patricia are just on paper at this stage19:57
karniLike Robert said, we're not touching that yet.19:57
karniAnd I agree it's a perfect fit for the files app.19:57
karniI know we were/are considering if it should be separate or not, though.19:58
dobeyand fewer apps probably will make the app review stage of releasing easier as well19:58
karnidobey: +119:58
dobeyhrmm, i really need a space suit like they have in modern sci-fi shows/movies20:00
dobeyralsina: care to review https://code.launchpad.net/~dobey/ubuntuone-client-gnome/fix-983144/+merge/102173 ?20:12
ralsinadobey: sure!20:16
ralsinadobey: 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?)20:19
=== Dr_Who is now known as tgall_foo
ralsinadobey: although reading the doc is like reading Lewis Carroll20:20
ralsinaThe 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
dobeyralsina: 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 list20:20
dobeyG_DIR_SEPARATOR_S is the trailing slash (or backslash on windows)20:20
ralsinadobey: oh, right, thanks20:20
* ralsina forgot his variable argument conventions20:21
* dobey can't wait to throw all this code away and rewrite it in vala20:22
ralsinadobey: wouldn't that code look exactly the same in vala?20:23
ralsinadobey: +120:24
dobeyralsina: no. possibly close though. and i would probably not reintroduce that code in the same manner20:30
ralsinadobey: does vala have any decent (as in not-from-c) string management ability?20:31
ralsinadobey: that's just idle curiosity, because it's my main C gripe20:31
dobeyralsina: it has some better abstractions, yes.20:31
ralsinadobey: cool20:32
ralsinadobey: I may have to learn vala after all20:32
gatoxeod here....... and stop fighting with wrapping and size problems...... byeeeeeee, see you tomorrow20:48
ralsinaeod for me too, I'm afraid20:54
ralsinaI will do an extra hour late tonight, ping me about review requests or whatever20:54
briancurtinralsina: have a good EOD, i will ping l-sa in a while to get the update.xml changed20:55
dobeybriancurtin: did you get the update install issue figured out btw?20:57
briancurtindobey: 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 version20:59
dobeyah ok. cool21:01
* Dr_Who ponders21:18
=== thisfred is now known as inspector_spacet
=== inspector_spacet is now known as inspctr_spacetim
inspctr_spacetimbah21:26
dobeyhaha21:28
=== inspctr_spacetim is now known as thisfred
=== salgado is now known as salgado-afk
dobeylater all. have a good evening21:56

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