/srv/irclogs.ubuntu.com/2012/07/18/#ubuntuone.txt

mmccUbuntuOne.app down to a slim 113 MB00:08
mmccnext up for issues: ubuntuone/storageprotocol/context.py", line 92, in get_certificates : exceptions.IOError: [Errno 2] No such file or directory: '/etc/ssl/certs/UbuntuOne-Go_Daddy_Class_2_CA.pem'00:32
mmccsaved as bug 1025950, time to go play with the baby00:51
ubot5Launchpad bug 1025950 in Ubuntu One storage protocol "storageprotocol.context has paths for ssl cert locations for windows and linux but not darwin" [Critical,In progress] https://launchpad.net/bugs/102595000:51
mmcca quick edit in storage-protocol and I have an almost-working UbuntuOne.app!  It pulls from the server great, but doesn't seem to want to upload.05:52
mmccand I've now seen the following error from IPC twice, in different spots each time: http://paste.ubuntu.com/1097819/05:54
mmccand the sync status isn't updated correctly...05:56
mmccaha - I created a test file in ~/Ubuntu One/ and got a similar error: http://paste.ubuntu.com/1097821/05:56
mmcc(but the file *was* uploaded)05:57
mmccnow it's uploading its 113MB self, at apparently ~50KB/sec. So I'm going to sleep.06:01
mmccnote: as it's uploading, the sync status is correct.06:02
mmccso what seems to be broken now: the upload/download finished notifications, and noticing that a new folder (the .app bundle in this case) was copied in.06:03
mmccotherwise: victory!06:03
mmccissues to discuss tomorrow - where config files and ssl certificates belong on OS X.06:07
mmccI guess I'm going to sleep now06:07
mmccnope, that .app uploaded fast so I'm sending an email with info on how to try it06:24
JamesTaitMorning all! :)08:19
mandelmorning all!08:39
=== zyga is now known as zyga-afk
=== zyga-afk is now known as zyga
gatoxgood morning11:17
mandelgatox, morning! how is the tests branch going?11:22
gatoxmandel, it's ready since yesterday, i forgot to sent you an email about it11:23
gatoxmandel, but you can re-review it now11:23
mandelgatox, oh, ok, let me finish some code and I'll take a look11:24
mandelgatox, I'm off 10- min to give water to the dog :)11:26
gatoxmandel, wow..... that dog drinks a lot!11:26
gatoxjeje11:26
mandelgatox, haha terrible joke, is just that I have to go home (I'm working in an office now :) )11:26
mandeland is 5 min way, :P11:26
gatoxahhhhh11:27
gatoxjjje11:27
* mandel back11:37
mandelgatox, while I do your review, can you do one for me here: https://code.launchpad.net/~mandel/ubuntuone-client/daemon-options/+merge/11535411:47
mandelgatox, I like mmcc comment about the imports but I prefer to fix that in a diff branch to keep things simple11:48
* gatox looking....11:48
mandelgatox, thx! now I only have to go after alecu to review the mother branch of all this branches.. and start landing them like crazy :)11:50
gatoxmandel, the sauron ring of all the other branches? :P11:51
mandelgatox, yes hehe11:51
gatoxmandel, yes, i agree with the imports, they look messy..... it's not possible to have that defined in ubuntuone.platform.filesystem_notifications.monitor, where i assume you are doing something similar to if sys.platform blah blah.... and use it directly from there?11:55
mandelgatox, I had some issue with it due to the namespace, I want to be careful and do it in a diff branch with a bug number11:55
mandelgatox, you know, land it working so we get something ready for usage by people then clean the code11:56
* gatox read that..... and want to kill someone....11:56
gatoxok..... let me run the tests11:57
mandelgatox, ein?11:58
mandelgatox, read what? kill who?11:58
gatoxnothing nothing..... anger managment12:00
gatoxjejee12:00
mandelgatox, tell me :)12:03
gatoxmandel, need fixing: https://code.launchpad.net/~mandel/ubuntuone-client/daemon-options/+merge/11535412:21
mandelgatox, oh, goot catch, fixing12:22
mandelgatox, fix pushed12:43
gatoxmandel, ack12:43
dobeyhrmm12:49
alecuhola mandel!13:12
* alecu hides13:13
mandelalecu, tu! ven aca, review!13:13
mandelalecu, hehehe13:13
dobeybrb, gotta go to dealer real quick13:13
mandeldobey, get me some dope, please :)13:14
gatoxalecu, hi13:15
gatoxalecu, let me know when you have a moment please13:19
mandelok, lunch time here :)13:25
* mandel lunch13:25
ralsinappl I am on vacation but I have an eye here, so feel free to ping me if you need me, I may be slow to respond though13:26
gatoxralsina, ack.......and enjoy! :D13:27
gatoxalecu, i think i have pretty clear the big picture..... i just need to have a quick mumble with you when you have some time to finish to understand one thing, let me know! :D13:28
alecugatox: sure13:28
alecugatox: 5 mins13:29
gatoxalecu, in 5 mins or 5 mins from now? :P13:29
dobeyheh13:36
dobeybrb again. need to go get some water/snack before i pass out of dehydration and starvation13:40
alecugatox: mumble?13:44
gatoxalecu, ack13:46
mattgriffin_1aquarius: congrats on the release of the java library!13:48
aquariusheya mattgriffin_1 :) All credit belongs to karni :)13:48
mattgriffin_1aquarius: not sure if it was intended in the blog post but <code></code> blocks don't show up with different formatting - http://voices.canonical.com/ubuntuone/2012/07/18/ubuntu-one-files-java-library/13:49
mattgriffin_1karni: w00t!13:49
karnimattgriffin_1: Hi Matt! Good to see you, thank you!! :)13:49
mattgriffin_1:)13:49
karniGood catch on the <code> tags13:50
aquariusheh, I hadn't seen that we'd released that yet :)13:52
karniaquarius: We went live just now :)13:53
dobeyoi13:55
alecugatox: https://wiki.ubuntu.com/SingleSignOn/UbuntuSsoClient14:16
alecugatox: it seems to be up to date, but has lost some of the original formatting.14:17
gatoxalecu, ok, thx, i'll take a look at that14:17
alecugatox: also, it has very simple dictionaries, not nested ones. For example: DICT OF {STRING, STRING} EXTRA_PARAMS14:17
gatoxalecu, ok, i'll add the spec there, and then create an example in dbus to check how it behaves with nested data types14:18
alecugatox: also https://one.ubuntu.com/developer/files/store_files/ubuntu14:19
alecugatox: this second doc seems to be generated from the dbus docstrings14:19
* mandel back14:20
alecugatox: but it's very ugly :-)14:20
mandelalecu, do you think you can do the reviews, if not you can pass and I'll ask dobey or mmcc14:21
dobeyeh?14:21
mandeldobey, nothing just yet, begging for a review :)14:22
alecumandel: I'll force myself into doing it right away.14:22
alecumandel: meanwhile you shoot anybody trying to interrupt me.14:22
mandelalecu, thx! is just that is blocking other 3 branches from landing14:22
mandelalecu, ok, let me know and I'll take care :)14:22
alecumandel: it's this one, right? https://code.launchpad.net/~mandel/ubuntuone-client/fsevents-daemon/+merge/11483614:22
mandelalecu, yes14:23
mandelmmcc, this has been updated: https://code.launchpad.net/~mandel/ubuntuone-client/daemon-options/+merge/11535414:27
briancurtindoes anyone else use thunderbird, and if so, do you have any good filtering setup so canonical email isn't like drinking from a firehose?14:38
briancurtinright now my filter has become "look through, see if i recognize any names looking for merge proposals. if not, delete all"14:38
ralsinabriancurtin: I don't look at merge mail, really14:39
ralsinabriancurtin: I wait for people to ask for reviews14:39
briancurtinmaybe i could just delete all merge mail as long as it's not my review day :)14:39
briancurtinnah i guess i do like peeking through the actual MP, but dont need to see every comment, every approve, etc...especially from people not on our team14:40
mandelbriancurtin, there are a number of launchpad header you can use for that, I look for X-launchpad-bug containns product=dirspec for bugs14:40
mandelbriancurtin, and for reviews: X-launchpad-notification-type = code-review14:41
alecubriancurtin: I've been using thunderbird up till a week ago14:41
mandelbriancurtin, but I 'painfully' use evolution14:41
alecubriancurtin: if you still want to use thunderbird, I suggest auto... something. Let me find it.14:41
mandelalecu, hey, you, don't get distracted!14:42
* mandel shoots at briancurtin 14:42
mandelsorry, following orders..14:42
alecubriancurtin: https://wiki.canonical.com/KamalMostafa/Autofolder14:43
briancurtinalecu: ahhh there we go, thats what i was thinking of14:44
alecubriancurtin: I was thinking of moving to that, but I got really tired of imap as a protocol.14:45
briancurtinim just tired of email as an anything14:45
alecu:-)14:45
mmcchowdy folks14:47
mmccmandel, looking at your daemon-options merge now. saw it earlier but toddler and code reviews = frustration :)14:48
mandelmmcc, no problem, I'll be fixing the __init__ and imports in a later branch so that the addition of the daemon is not late due to that if you done mind :)14:51
mmccmandel, I don't mind. that sounds like a good plan.14:51
gatoxmandel, i don't see that you review my branch again....... DO IT NOW!! jeje14:52
mandelgatox, I'm looking and trying to see how bad will it merge with mine :)14:52
gatoxmandel, don't worry about yours..... now is your time to fix conflicts14:53
gatoxjust deal with it14:53
mandelgatox, that is why I worry :)14:53
mmccmandel, sorry, I noticed a couple more places for s/active/available/ - should be trivial to fix up, and I'll re review super fast14:55
mmccmandel: hope it's not too annoying, just want the naming to be consistent14:55
mandelmmcc, not at all, just add a comment :)14:56
mmccmandel, I did…14:56
* mmcc goes to try setting up autofolder14:57
briancurtinme15:02
gatoxme15:03
dobeymeh15:03
mmccme15:04
dobeymandel, ralsina, alecu, thisfred: ?15:05
thisfredme15:05
mandelme15:05
mandelralsina is on holidays AFAIK15:05
ralsinaDONE: vacation TODO: vacation BLOCKED: vacation15:05
dobeyoh15:05
ralsinabut as you can see, I'm intermittently around15:05
* dobey really needs to schedule some of that15:05
ralsinaso no parties15:06
* dobey puts away the rum15:06
mmccralsina, when are you back?15:07
alecume15:07
alecuralsina, all: I'll be taking half day tomorrow after pm, to babysit amelia, who's on winter holidays.15:07
gatoxbriancurtin, go15:08
briancurtinDONE: dirspec buildout branch (ended up being really trivial...ugh)15:09
briancurtinTODO: py3 remaining branches, have some filter/comp changes to consider. want to do that before unicode so i actually feel like i'm writing code rather than just reading all the time!15:09
briancurtinNEXT: gatox15:09
gatoxDONE:15:09
gatoxReviews, investigate some special widgets for the menu, read about status aggregator, writting the spec for the ipc menu request.15:09
gatoxTODO:15:09
gatoxPublish the spec in the wiki, start coding the ipc menu functions.15:09
gatoxBLOCKED:15:09
gatoxNo15:09
gatoxdobey, go15:09
thisfredmmcc, you15:10
mmccDONE: reviews, built .app, synced some files15:10
mmccTODO: figure out syncdaemon status IPC bug15:10
mmccBLCK: none15:10
mmccNEXT: thisfred15:10
mmccREMINDER: I'm out tomorrow through monday, moving out of Texas.15:10
thisfredDONE: refactored query parser in python TODO: refactor query parser in C | add combine() mapping | release u1db on pypi | *party like it's 1999* BLOCKED: no NEXT: mandel15:10
mandelDONE: Reviews, update MO begged for reviews.15:10
mandelTODO: file bug about imports and fix it. Beg for reviews land daemon code.15:10
mandelBLOCKED: no15:10
mandelalecu, go15:11
* alecu is writting notes15:11
thisfredmmcc, where are you moving?15:11
dobeyDONE: releases, prepare some backport branches, make tarmac run tests in C.UTF-8u locale15:12
thisfredor just anywhere but Texas? :)15:12
dobeyTODO: finish releases/uploads, investigate twisted-less dev-tools, work on some reorg to drop ubuntuone-installer for good, finish SRU verification poking15:12
dobeyBLCK: None.15:12
dobeysorry. got distracted15:12
thisfreddamn butterflies15:12
mmccthisfred: Back to San Diego, CA. (But I'll be waking up much earlier to stay ~ in sync with you guys)15:12
thisfredmmcc, nice!15:12
mandelmmcc, not really to worry I'm the only eu guy, right?15:13
thisfredI'm moving to Portland, OR in september, so very much in the same boat time wise15:13
mmccthisfred: yeah, we're looking forward to it. we'll be much closer to family.15:13
mmccthisfred: cool! I love Portland - we might go there after San Diego (when the money runs out :)15:14
thisfredhehe15:14
ralsinammcc: I will be back late next week15:14
mandelthisfred, does that mean you are going to ride a fixy and be a hipster?15:14
mandelthisfred, you already have the music, the interesting dog and the t-shirts ;)15:14
mmccralsina: cool! have a great vacation :)15:14
ralsinathisfred: WAT?15:14
thisfredmandel, I am totally a hipster. King of all hipsters!15:15
ralsinaeveryone on the team who's not changing timezones raise his hand please! ;-)15:15
thisfredralsina, oh damn, you're still here15:15
mandelo/15:15
thisfredralsina, I meant parse line 199915:15
alecuDONE: fixes for the twisted bug upstream, mumbled re status aggregator with gatox15:15
alecuTODO: reviews, more reviews15:15
alecuBLOCKED: no15:15
mandelralsina, but if country goes down the toilet I'll have to15:15
* ralsina schedules mandel as a mover15:15
thisfredralsina, I told you about the move, though that was approximately a year ago :)15:16
gatoxmandel, i just looking for excuses to move to argentina....15:16
thisfredyou said it was ok, as long as I was willing to get up at 6 :)15:16
briancurtinralsina: i'm not changing timezones and it looks like i wont even have to move. apartment situation figured out! (at least for now...)15:17
mandelmmcc, branch updated :)15:18
mmccmandel, looking now15:18
mandelthx!15:18
mmccmandel: approved15:19
mandeltoma! mmcc thx15:19
mandelI'll file a bug for the imports15:20
mmcccool15:20
ralsinabriancurtin: awesome!15:20
mandelalecu, you are the last reviewer I need :)15:20
ralsinabriancurtin: you had me worried15:20
mmccso, did anyone get my email from last night and try out the .app I posted?15:20
ralsinammcc: got it, not tried it15:20
mandelmmcc, nop, but I can try it in a few mins or later after EOD infront of the tv :)15:20
briancurtini was pretty worried myself. the way this whole thing was going i was pretty sure they were just going to kick everyone in the building out right away15:20
mmccralsina: ok, well you're on vacation, so take your time :)15:20
mmccmandel, if you're just waiting now, can you peek at these errors I'm getting from IPC between syncdaemon and control-panel on darwin: http://paste.ubuntu.com/1097819/ http://paste.ubuntu.com/1097821/15:22
gatoxmmcc, did you send it to me too?? i don't see it15:22
mmccmandel: just to see if anything pops out. I am pretty sure I'm up to date with the packages I put in there15:22
mandelmmcc, we have that in all platforms, we never got to fix it15:22
briancurtinspeaking of apartments, since im out of ink i'm heading down the street to print out some docs i need to sign for the lease. will be back shortly15:22
mmccgatox: I meant to.. let me check15:22
mmccmandel: really? it seems to cause real problems. ..15:23
mandelmmcc, I've seen it on windows for sure and some linux.. gatox, alecu, ralsina you have seen that, right?15:23
ralsinaon windows it seemed to be harmless15:24
mmcchmm. ok, maybe it is harmless. there is certainly room for something else to be causing the problems I saw15:24
mmccI tried to figure out what was causing it but that RPC code is a little convoluted for 2AM15:25
* gatox lunch15:27
=== gatox is now known as gatox_lunch
alecummcc: it's more than a little convoluted, no matter how much awake you are15:29
mmccalecu: :)15:30
* mmcc was being polite :)15:30
mandelmmcc, alecu, is great, because it works and it can be used to share secrets within the code and no one will ever know15:33
mandelis that a real method, is the decorator needed?15:34
mmccmandel, were you asking me?15:39
mandelmmcc, bad joke :)15:39
mandelmmcc, once you get in that ipc stuff.. you will understand the decorator comment15:40
mmccmandel: heh, ok.15:40
alecumandel: ping15:43
mandelalecu, mmcc, which reminds me to http://www.youtube.com/watch?v=2EnkSshoEkk&feature=youtu.be&t=19m30s15:43
mandelalecu, pong15:43
alecumandel: why is this static? DAEMON_SOCKET = '/var/run/ubuntuone_fsevents_daemon'15:43
mandelalecu, because at the moment is always the same one, will fix that in a later branch to be passed15:44
* briancurtin back15:47
mmccbriancurtin: did you get a chance to try that autofolder thing? I too was drowning in bug mail… tried it but the instructions don't work for me :\15:49
alecumandel: does this mean that right now every logged in user can spy on the events of every other user?15:49
briancurtinmmcc: i didn't try it yet. i guess you need to do an RT and get access to run it all. i may start down that path later this afternoon15:50
alecumandel: so, you don't regret spaghetti ipc either??? :-)15:50
briancurtinbut i might also just forget about it and just delete all of my email15:50
mandelalecu, no, the sever will request the user id from the connected client and will only send events for those paths owned by the same user id15:51
=== salgado is now known as salgado-lunch
mandelalecu, I regret it, and I inherited the design, the 3/4 layers in sso are not mine :)15:51
mandelalecu, but is in my list of 'pecados' that I want to fix in u1client15:51
mmccbriancurtin: ah, ok. yeah I can log into mail.canonical.com but it kicks me right away. I'll let you know if I learn anything valuable15:52
mandelalecu, the idea is that when the user connects we get the uid from the connecting process so you cannot request events of a path you don't own15:52
alecumandel: right. But what if two users are logged in at once? Can you do that in mac at all?15:52
mmccalecu: two users can log in, sure. but the server won't send the events for a path owned by user 501 to user 50215:53
mmccand once it's integrated, the hardcoded path to a socket goes away in favor of a launchd 'checkin'15:53
mandelwhich is the next step15:54
mandelbut right now is hardcoded to get a bundle working15:54
alecummcc, mandel: great then!15:54
mandelalecu, ideally we are in the state where we have an alpha doing syncing etc.. and yet not perfectly integrated with launchd, step by step :)15:55
mmccmandel: yes - today I'll try getting a bundled app to connect with the root daemon15:56
mandelmmcc, awesome, I think it will nice for us to talk about the next steps, but maybe after your holiday/move15:57
alecumandel: "from ubuntuone.darwin import fsevents" <- where does this come from?15:57
mandelalecu, is the python code in the lp:ubuntuone-fsevents-daemon so that all protocol info/code resides in the same place15:57
mmccmandel: sure, I have a few smaller things to clean up today. we can talk next week15:58
mandelalecu, ideally if there are changes in the protocol u1-client should not care15:58
mandelmmcc, same here :)15:58
mmccassuming I make it out of Texas alive15:58
dobeyugh. "ubuntuone.darwin" ?16:00
alecudobey: ditto16:00
mandelmeh, I had to add a bloody namespace and I did not want to do ubuntuone/platform/darwin16:00
alecumandel: would you mind making a bug to move that to a different namespace?16:00
dobeyubuntuone.fsevents should be the namespace of ubuntuone-fsevents-daemon16:01
mandelalecu, not at all16:01
alecumandel: great.16:01
mandeldobey, sure, I'm bad with names and is not too much work16:01
mmcchrm. ubuntuone.fsevents doesn't tell me that it's darwin-only, or that it's the root daemon (as opposed to the other module we have that accesses fsevents)16:02
mmccnot that I have a much better suggestion right now…16:02
alecuubuntuone.fseventsd ?16:03
mandeldobey, alecu, mmcc bug 102620916:03
ubot5Launchpad bug 1026209 in Ubuntu One FsEvents daemon "Change python namespace" [Medium,Triaged] https://launchpad.net/bugs/102620916:03
mandelcomments for names are welcome :)16:03
alecumandel: your name looks fine. Let me know if you have kids or a new dog.16:04
alecumandel: I can do a review in that case :-)16:04
mandelalecu, hehe ok :)16:04
alecumandel: also, do you have a bug for filling in the empty function events_dropper() ?16:04
mandelalecu, adding, I have to talk with _facundobatista about what to do then16:05
mandelmmcc, FYI bug 102621216:06
ubot5Launchpad bug 1026212 in Ubuntu One Client "File system monitor imports are messy" [Medium,Triaged] https://launchpad.net/bugs/102621216:06
dobeyfile bugs as bugs, not TODOs :)16:06
mandeldobey, agg I rephrase things..16:07
alecumandel: here's a different concern: "any(path.startswith(watched_path) for watched_path in self.watched_paths)"16:08
mandelalecu, shoot, what is the issue16:09
alecumandel: those lookups have a tendency to degrade awfully. But it's probably just me thinking of premature optimization.16:09
alecumandel: the thing is that for every watched path (or ignored, in the following line), we end up doing a whole byte-per-byte comparison.16:10
mandelalecu, the max num of watched paths is the number of udfs, it will take a long time to be a problem, on the other hand the ignored paths could be a problem if there are lots of add remove dir operations and we might look at a way to clean that16:11
mandelalecu, having said that, we should fix that on windows and the user fsevents implementation because a similar approach is taken16:11
alecumandel: oh, right, with the root daemon we have a watch per udf, I was thinking of a watch per folder.16:12
mandelalecu, exactly16:12
mandelalecu,  we can look at optimizations about this for the ignored paths16:12
alecumandel: there are a few typos all around, but this one is really hard to understand: "lets generate to" -> "let's generate two"16:15
mandelalecu, uh.. sorry can you add them in comments and I'll fix all of the in one go?16:16
alecusure16:16
mandelthx16:16
alecumandel: in fact, I'll only correct that one, I find it an excercise in futility, since you have not yet installed a spellchecker in your editor :-P16:17
mandelalecu, I'll take the thing for a spell check :)16:18
alecucan you explain why this happens?16:18
alecu1031 + # FIXME: event deduces the pathname wrong and we need to manually16:18
alecu1032 + # set it16:18
mandelalecu, is an issue for the pyinotify code we have, I need to look at it to understand the bug16:18
=== gatox_lunch is now known as gatox
mandelalecu, under the pyinotify agnostic code16:19
mandelalecu, I can add a bug and fix it in all the impl that use it, windows, user fsevents and daemon fsevents have the same problem16:20
alecumandel: that would be great.16:23
dobeyok, need to get lunch. bbiab16:23
alecumandel: process_IN_MOVED_TO has too many levels of nested ifs16:23
alecumandel: perhaps you can move the outermost if else, so the else is on top, and returns from the function after logging the error...16:25
alecumandel: and why is the second level else not logging something?16:25
alecumandel: the second level else has the same comment as the first: # we should never get to this point on darwin, never ever!16:26
alecumandel: but the second level else logs no warning...16:26
* alecu wears a puzzled look16:26
mandelalecu, yes, in a following branch the notify processor is shared between windows,darwin-user and darwin-daemon and that should be fixed there16:26
mandelalecu, look here: https://code.launchpad.net/~mandel/ubuntuone-client/unify-processors/+merge/11490616:27
mandelalecu, this is just the tip of a huge 6000 lines branch split in diff smaller working branches16:27
mandelalecu, I think we have to fix that in the later branches, also, linux has the same amount of nested ifs which we might as well fix: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/filesystem_notifications/linux.py#L25416:28
mandelis not a nice method, you are right16:28
mandelis also interesting to note that I dont think that is really needed because is dealing with lost events which AFAIK does not happen in any platform but linux16:29
mandelso we can clean up that method a lot more16:29
mmccmandel - lost events? is that the same as dropped events? those do happen on osx16:30
mandelmmcc, lost not dropped, moved_to with no moved_from16:30
mandelmmcc, when in linux you moved from a watched folder to a not watched one16:30
alecumandel: you've mixed the examples!16:31
alecumandel: "moved from a watched folder to a not watched one" is "moved_from without moved_to"16:32
mandelalecu, sorry yes16:32
mmccah, ok - and IIRC, the other platforms fake those events and generate them together or something right? so there is always both events on win/darwin?16:32
mandelwhile on windows that does not happen and on fsevents user AFAIK it does not (is translated to a delete, right gatox)16:33
mandelmmcc, yes, in windows you always get them one after the other, on fsevents user is translated to a delete on fsevents daemon you do get both events16:33
mmccaha, ok16:33
gatoxmandel, right16:33
mmccso I need to write some code to do part of the 'install' step on first run, since we won't have an installer for macos16:35
mmccneed to copy .conf files and ssl certs to the right place.16:35
mandelmmcc, and add the daemon if possible16:36
mandelmmcc,  which means adding it to the bundle too, right?16:36
mmccneed to figure out where to put stuff: ssl certs on OSX are traditionally stored in the keychain, but since we're not using OSX api, I think we can just put them in the "~/Library/Application Support/UbuntuOne/" directory (which will be xdg_data_home)16:36
mmccmandel: yes, that'll be the same code that adds the daemon.16:37
mmccmandel: and yes, it needs to go in the bundle16:37
mmccmandel, that'll be bug 102623516:39
ubot5Launchpad bug 1026235 in Ubuntu One Windows Installer "setup-mac does not package the root fsevents daemon" [Critical,Confirmed] https://launchpad.net/bugs/102623516:39
mandelmmcc, awesome16:40
mandelall, I'm EOD laters!17:10
mmccUbuntuOne.app now ~10 MB smaller, because all dependencies are in the .zip17:19
mandelmmcc, current total size?17:20
mmccmandel, 98 MB.17:20
mandelmmcc, getting to a reasonable size17:21
mandelby the way I have been thinking about this: https://github.com/kfdm/gntp/17:21
mandelmmcc, anything against growl integration for the notifications?17:21
mmccmandel: yep. there are probably some things we can prune too. All the Qt libs are included and we may not use them all…17:22
mandelanyway, I'm off to have a milkshake, laters :)17:23
mmccmandel: no, I like growl. need to read up on how they're changing WRT 10.8 though17:23
mmccbecause 10.8 has its own notification center that behaves differently17:23
mmccand IIRC sandboxed apps can't use growl17:23
mandelmmcc, we can do growl integration and 10.8 thingy integration, also, there is a growl for windows17:23
mmccenjoy your milkshake, I'll talk to you Tuesday17:24
mandelmmcc, so, we can have 2 for 1 :P17:24
=== salgado-lunch is now known as salgado
mmccsounds good. just as long as we avoid ever showing two notifications for the same event… that'll get some angry twitters I bet17:24
mmccfwiw - I was wrong, a recent update to growl has a 'lite' version that works from within sandboxed apps: http://growl.info/notetodevelopers17:27
gatoxalecu, it's working with dbus :D17:49
mmccalecu, may I request a review for setup-mac changes: https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/package-everything/+merge/11559617:57
dobeyis anyone helping alecu do reviews today?18:01
mmccalecu: and (this one is much shorter) https://code.launchpad.net/~mikemc/ubuntuone-storage-protocol/darwin-certs/+merge/11559918:02
briancurtini wouldnt mind doing a review. mmcc - is that package one anything super mac specific?18:03
mmccbriancurtin: yes! it's as mac-specific as it gets. you won't be able to test it, I guess, but a code review couldn't hurt18:03
briancurtinmmcc: i'll take a look and see what i can contribute18:04
mmccbriancurtin thanks! also the other one is a lot less mac specific.18:04
dobeyi need easy reviews of backport branches that are just merging stuff from trunk to stable-4-018:05
mmccdobey, maybe you could take a look at this one too? I'm not 100% sure it's doing the right thing, it's telling storageprotocol to look for the SSL certs in XDG_DATA_HOME, which is similar to how we do it in windows... https://code.launchpad.net/~mikemc/ubuntuone-storage-protocol/darwin-certs/+merge/11559918:06
dobeyhttps://code.launchpad.net/~dobey/dirspec/update-4-0/+merge/11559718:06
mmccdobey: I'll help with some of those18:06
dobeymmcc: i don't think that's write; the _HOME variables would be paths into the user's directory18:06
dobeymmcc: also, you can just import "xdg_data_home" and use it as the variable18:07
mmccdobey: yeah I wanted to hash out where the certs should go. there isn't a standard path to put them in OSX - the usual thing is to store them in the keychain, using some C API we could use if we wanted to18:08
dobeywell, we should probably switch to using the system certs DB on all the platforms, rather than shipping our own copies of a few certs18:10
dobeybut either way, we wouldn't be installing anything to the user's home18:10
mmccbriancurtin: what is the Common AppData reg key on your system? I don't seem to have it in my VM's registry. That's where storageprotocol looks for the certs…18:12
mmccthe other AppData keys are in my home dir.18:12
mmccI wonder if we add the cert to the system keychain, will Qt use it?18:13
briancurtinmmcc: do you have a full path? i'm familiar with AppData by itself, "Common AppData" looks weird as a key18:13
dobeymmcc: the certs are probably already in the system keychain18:13
mmccbriancurtin: here's the code: http://paste.ubuntu.com/1098734/18:14
briancurtinohhh C:\ProgramData18:15
briancurtinand there's a ubuntuone-storageprotocol folder in there with the certs18:15
briancurtinmmcc: what version of win are you running in this vm?18:16
mmccbriancurtin: it's win 718:17
mmccdobey, the valicert one and the godaddy class 2 one are indeed in my keychain already. The godaddy intermediate one is not.18:21
briancurtinmmcc: even though you dont have that Common AppData, do you have C:\ProgramData with any ubuntuone sub-folders?18:22
dobeymmcc: i guess on osx we'd have to pull them out of the bundle somehow to load them18:23
dobeymmcc: so maybe use the get_program_path() from dirspec utils to get them?18:24
dobeyalthough it will fail when not in the bundle18:24
mmccbriancurtin: yes, I have C:\ProgramData\ubuntuone-storageprotocol with the certs in there... weird that I didn't see that path in regedit18:24
dobeymaybe we need some more API there18:25
briancurtini got there from the full "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\Shell Folders\Common AppData"18:25
mmccbriancurtin: aha, in retrospect I was looking at HKEY_CURRENT_USER\blah de blah - even though I thought I checked that :\18:27
mmccthx18:27
mmccdobey: yes, dirspec needs some more tweaking for osx. these config paths should really be under ~/Library/18:30
mmccas far as finding the certs to install them, they'll be at a fixed location in the app package, so the same code that installs the conf files can do the cert install, and I don't think we need to use dirspec for that. It'll only be used in one place18:31
dobeymmcc: well, dirspec already has the code to look in the sub-app and whatnot. i don't think we'll need to only look in (from) one place to find extra data files18:33
mmccdobey, the data files from the sub-apps (eg, syncdaemon's conf file and storageprotocol's certs) are all installed into the top-level app's Resources directory. All the sub apps just link to it.*18:37
mmcc* - that's a lie: for now, they all have a copy of every conf file and .pem, but that's a TODO18:37
mmccok, time for lunch here. back in a bit18:39
dobeyalecu, briancurtin: either of you care to review https://code.launchpad.net/~dobey/dirspec/update-4-0/+merge/115597 ?18:44
alecudobey: sure. I'm doing mmcc's reviews, and then I'll get to yours.18:44
briancurtindobey: will do18:45
dobeyalso have https://code.launchpad.net/~dobey/ubuntu-sso-client/update-4-0/+merge/115606 which needs review18:54
dobeyand https://code.launchpad.net/~dobey/ubuntuone-control-panel/update-4-0/+merge/11560719:10
=== Daviey_ is now known as Daviey
gatoxalecu, ping19:49
alecugatox: pong19:49
gatoxalecu, i already have a new item in the api, where it's returning a lot of stuff via dbus that i can use in the menu, such as the files being uploaded and the progress of each one, etc.....19:50
gatoxalecu, buttttt19:50
gatoxi have some doubts regardings some of the actions19:50
alecugatox: do you have the spec for the dict?19:51
alecugatox: do you want to mumble regarding the actions?19:51
gatoxalecu, ok, better19:51
gatoxalecu, in mumble19:51
=== urbanape` is now known as urbanape
alecugatox: http://pastebin.ubuntu.com/1098954/20:01
gatoxalecu, http://developer.ubuntu.com/api/ubuntu-11.10/python/Dbusmenu-0.4.html20:02
alecugatox: https://wiki.edubuntu.org/Unity/LauncherAPI20:04
dobeyare you adding quicklists to the launcher, or are you working on the sync menu?20:04
gatoxalecu, bssssasaa{si}as20:19
dobeygatox: what the heck? you're adding new dbus api with a signature like that?20:23
alecudobey: I'm talking him out of that idea, right now.20:24
dobeygood20:25
dobeywe probably already have all the api he needs (aside from the fact that one set of signals is currently broken)20:25
alecu{"file1_percentage": 30, "file1_name": "pr0n.png"}20:25
alecugatox: ^20:25
dobeyuh20:25
dobeyjust fix the UploadProgress and DownloadProgress signals to work :)20:26
alecudobey: I remember that verterok disabled some of those dbus signals because they were clogging the cpu20:32
mmccabout halfway through that u1client update-4-0 merge.... just so we don't get too many people reading all 4000 lines of it20:34
dobeyalecu: right, i know the Event one would have done that. but if we're going to be sending progress over dbus every X ms or anything, we should use the signals we already have20:35
alecudobey: we are only going to query progress when the sync menu is opened. And perhaps query it every 1000ms while it's opened.20:37
alecudobey: I think it makes no sense to send those signals all the time if nobody is using them for nothing.20:38
mmccno way to only send signals if someone's listening?20:38
* mmcc is not familiar with dbus20:38
dobeyalecu: we had one person wanting to use them last week, in their ubuntu app showdown entry; but the signals don't work so they couldn't use them20:39
dobeymmcc: the bus daemon (the server) is always listening. it's basically a proxy for the messages20:39
dobeymmcc: so we'd always send them regardless of any apps listening or not20:40
dobeybut we could add API to "register" that something is watching, and only send those signals out when that dict isn't empty or something20:41
mmccdobey: ok. I guess I could imagine a more complicated protocol that'd have the server let you know when to start & stop sending messages based on who'd registered for them... but it sounds like you'd have to build that yourself20:41
mmccayup20:41
mmccdobey, ping about osx-specific basedirs in dirspec20:56
mmccfor the paths vars, since the linux paths (/etc/xdg/) *could* exist on osx, I was going to just insert the osx-specific version in front.20:57
mmccfor example, default_config_path = '/Library/Preferences:/etc/xdg'20:57
dobeymmcc: that's probably doable. i have no idea what real OSX apps expect.20:58
alecugatox: I can't hear you anymore20:58
gatoxah20:58
gatoxalecu, well, i was just saying:20:58
alecucrap, it must be my isp again20:59
gatoxalecu, that i'm going to ask mandel if this structure is possible in windows, and if he says yes, go with it, or in the other case, start thinking about another possible structure20:59
alecusince they've opened a customer care center round the block, the internet connection has been crapier.20:59
alecugatox: awesome20:59
mmccdobey: an app written just for osx will almost certainly only ever write files to your home, and it'll put everything under ~/Library - Caches, Preferences, and "Application Support"  for misc other data21:00
mmccand that's what u1 should do, but since dirspec is supposed to be more broadly usable I thought it couldn't hurt to allow it to return search paths that have fallbacks to linux style paths21:01
dobeyright21:01
mmccalthough I am not sure I have a good use case for that, really. maybe if I'm using dirspec to read a file written by a linux utility that doesn't use dirspec, and also doesn't follow osx conventions…21:02
dobeywell the freedesktop base dir spec doesn't say anything about osx, i think. but it probably should21:02
dobeyit also doesn't say anything about windows, but we support it in dirspec21:02
mmccyeah. I'm looking at the spec again now… IIRC it doesn't mention macs21:03
gatoxbye all!! see you tomorrow people! :D21:04
mmccbye gatox21:06
mmccdobey, now that I read the spec again I think the right thing is to define the osx-specific defaults but also include the xdg defined versions as fallback, *including* putting the default ~/.config at the front of default-config-path, so the search goes like this: osx standard home path, XDG default homepath, osx standard global path, xdg default global path(s)21:09
dobeymmcc: we can't have both the osx and xdg home paths though, because they aren't path (separated by ":") variables. they're just single paths21:11
mmccdobey: I know - I was suggesting adding the xdg default value for e.g. config_home to the beginning of the config_path21:19
mmccbecause the spec says you search in config_home, then you search through config_path. so we'd be searching osx-home,xdg-home,osx-global,xdg-global(s)21:20
mmcceffectively21:20
dobeymmcc: no, there would be no xdg-home there21:21
dobeyor well, xdg_*_home would be the same as whatever osx would expect it to be21:22
mmccright - this is what I meant: http://paste.ubuntu.com/1099076/21:23
mmccthe config_path variable starts with the default value for config_home from the xdg spec21:24
dobeyright, that would be wrong21:24
mmcccan you expand on that? :) The spec doesn't say not to put things under $HOME in config_path…21:25
dobeythe spec says you should look in $XDG_CONFIG_HOME first, then in $XDG_CONFIG_DIRS. that doesn't mean that $XDG_CONFIG_DIRS should start with a different value for CONFIG_HOME than CONFIG_HOME is defined as21:26
mmcchm. ok, fair enough.21:28
mmccso then i think it should use the osx conventions only - since only providing the xdg default fallback for the global location doesn't seem right…21:30
dobeywell, i think we do need to provide the fallbacks for the global locations, in the library21:35
dobeyotherwise it's useless to developers who expect it to behave with the POSIX paths21:36
dobeybut on the other hand, some of these directories also just don't exist on OSX21:36
dobeyso i think we need to see what does exist in those situations as well, and use what makes sense for covering the most cases with the least code21:36
mmccwtf21:39
dobey?21:39
mmcccan't seem to get it to send my response, but it sends 'wtf'21:39
mmccsorry21:39
dobeyheh21:39
mmccso, /usr/share and /usr/local/share both exist and have lots of crap in them on both my macs... /etc exists (symlink to /private/etc) but /etc/xdg does not21:39
mmccif we provide the global fallbacks, that's fine - the spec says just ignore nonexistent entries in the *path21:41
dobeyright21:41
dobeyand that is done already, automatically by the APIs in dirspec21:41
mmccok, I'm convinced. osx specific HOME locations, and global search paths that start with the global osx location but include fallbacks for global xdg defaults21:41
alecummcc: I think "ubuntu-sso-proxy-creds-qt" is still used; it's "ubuntu-sso-ssl-certificate-qt" the one that should be commented.21:46
alecummcc: I'm talking about https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/package-everything/+merge/11559621:47
mmccalecu, indeed - thanks! I need a vacation.21:47
mmccok, not really. maybe just a nap21:47
mmccalecu, ok pushed a fixed version. I did briancurtin's suggested style change too. Thanks!21:51
alecummcc: great!21:51
mmccjust saw a note that mac app store submissions now require an icon at 1024x1024 resolution21:56
alecuisn't it already time for vector pictures?21:58
alecummcc: what's the best way to test this new setup-mac.py?21:58
dobeymmcc: i should send apple a note requiring them to give me a 300dpi display for submitting an app to their store21:59
mmccalecu, i suggest trying to run it - the change description has some notes, but I'm sure I probably missed some thing22:00
mmccdobey - heh. they'll be glad to, for only $220022:01
mmccalecu, the quirk is that setup-mac currently assumes it's in a separate branch under the buildout's parts/ directory22:02
mmccso if you have an existing buildout, go to parts and branch lp:ubuntuone-windows-installer22:02
alecummcc, oh, ok. That's what I need to know.22:02
dobeyok, well, time for me to go. will have to finish up these releases in the morning i hope22:02
dobeyhave a good evening all22:02
mmccbye dobey, thanks for hashing out that dirspec stuff with me (again :) )22:03
mmccalecu: then grab branches of the stuff I mention in the description, and when you run it, I guess temporarily symlink /etc/ssl/ to ubuntuone-syncdaemon/data22:04
alecummcc: ln -s /etc/ssl ubuntuone-syncdaemon/data ? or viceversa?22:05
mmccvice versa. make /etc/ssl a link to ubuntuone-storage-protocol/data22:06
mmccI meant storage-protocol the first time too22:07
alecuack22:08
mmccalecu wait there's another bit - you need current trunk of py2app also22:08
* mmcc needs to consolidate these notes, sorry22:08
alecummcc: where should I install py2app?22:08
alecuor should I use brew for that?22:09
alecummcc: also: when running "PYTHONPATH=$PYTHONPATH:. python setup-mac.py py2app" I get this error:22:09
alecuerror: /Users/alecu/canonical/ubuntuone-windows-installer/scripts/devsetup/parts/package-everything/scripts/bin/ubuntu-sso-login-qt.py: No such file or directory22:10
mmccno, don't use brew, you need to get it from bitbucket…22:10
mmccand you will need to run setup-mac.py prepare first (you can run it as setup-mac.py prepare py2app)22:10
mmccsorry this is rough - thanks for being the first to wade through it22:10
mmccI'm looking for the full notes. I think I even needed to get dev versions of py2app's dependencies22:11
alecuDONE!22:12
mmccreally?22:13
alecuI've just clicked on the control pannel .app22:13
alecummcc: great work, congrats!22:13
mmccthanks, but I don't believe you :)22:13
alecummcc: The control panel is totally broken, but it's not your fault :-)22:14
mmccwhat's broken for you? it should be almost totally working, depending on how recent your source branches are22:14
mmccalso - did you have to get py2app from bitbucket? and did it install OK with its deps?22:15
mmccand finally, what os x version are you using?22:16
alecummcc: I've not even installed the latest py2app yet, so this is probably it.22:18
alecummcc: lion22:18
mmccok, interesting.22:19
alecummcc: also: probably not all trunks are up to date. And I've not started sso, u1-client, etc.22:19
mmccso how far does it get?22:19
alecummcc: I just wanted to test your branch, and it seems to be building everything as expected :-)22:19
mmccheh, fair enough - no need to debug everything. yes it does seem to be doing what it should. can you maybe do a quick ls of UbuntuOne.app/Contents/Resources/Ubuntu SSO Helper.app/Contents and check that it's created symlinks?22:21
mmccsymlinks that aren't broken? that'd be the best test of the current change22:21
alecummcc: as soon as I open u1cp, it opens a dialog that says "Sorry, an error has occurred and Ubuntu One needs to close." But the control panel is shown beneath it, and the web links at the botton show up.22:21
mmccalecu: ok, sounds like out of date branches.22:22
mmccor the same issue I found on Lion. that's a todo- for now it may not build right on Lion22:22
alecummcc: I see that both Fraeworks and PkgInfo are symlinks to ../../../x22:23
mmccsounds good22:23
=== karni_ is now known as karni
alecummcc: I've approved it.22:26
mmccthx alecu.22:26
mmccanyone still around for a really trivial review of a typo fix in dirspec? https://code.launchpad.net/~mikemc/dirspec/fix-1026369-typo/+merge/11563322:50
alecummcc: done22:53
alecuand EOD!22:53
=== wgrant_ is now known as wgrant
mmccand I'm out - see everyone on Tuesday23:29

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