JamesTaitGood morning all! :-D08:16
Nicekiwiis there an Ubuntu-One extension/Script/Plug-in for Thunar?09:59
dpmhi all. I was looking at bug 1063170 and I can't figure out where the "Purchased from Ubuntu One" string comes from in the plugin code. Could someone give me a hand?10:40
ubot5Launchpad bug 1063170 in Ubuntu Translations "Not translatable string "Purchased from Ubuntu One" in Rhythmbox" [High,Triaged] https://launchpad.net/bugs/106317010:40
mandeldpm, hm.. let me take a look for you10:47
mandeldpm, AFAIK that is in that name because the folder name of the music that has been purchased from ubuntuone is named like that11:05
gatoxgood morning11:06
mandeldpm, so, ideally you will want us to make the folder path translatable11:06
dpmmandel, bummer, not translatable, then11:06
mandelgatox, buenas o/11:06
dpmmandel, yes11:06
gatoxmandel, hi11:06
mandeldpm, I guess it can be translated, yes, because is a constant and we should be able to name it ~/.ubuntuone/Paquito\ el \ chocolatero if we wanted11:06
dpmmandel, exactly, that was also the song I had just in mind11:07
dpmI was doing the dance too11:07
mandeldpm, problem is, if you use a diff folder name in one machine and then you move to a machine that is in eng it brakes11:07
mandeldpm, as in, first time the udf is created using the spanish name, then you go to a machine in eng and then you have the spanish name synced and shows in spanish11:08
mandeldpm, so it does not work very well, I think the less damaging thing we could do is remove adj, that is, remove 'purchased' and leave it with the name which is 'international'11:08
mandeldpm, do I make sense?11:09
dpmmandel, gotcha. What about leaving the folder name as it is and make what is shown on Rhythmbox translatable?11:09
mandeldpm, that means patching rhythmbox because is part of the code using by it to monitor folders11:09
dpmhm, not really a reusable plugin architecture, then11:10
dpmok, let me add this info to the bug for now, thanks for the help!11:10
mandeldpm, no problem :)11:10
mandeldpm, do explain that changing the folder name is a no no due to sync between machines with diff names11:11
mandeldpm, I wonder what would happen if we use a symlink which can be translated..11:11
mandeldpm, udf folder is always the same, we do a ln -s to a translated version11:12
dpmI don't know, I wonder too if translatable XDG directory names could be used11:12
mandeldpm, you can try looking at how to add diff sources: http://library.gnome.org/users/rhythmbox/stable/customize.html.en11:12
mandeldpm, if we can add a level of indirection between the udf and the player (symlink for example) we are fine11:13
mandeldpm, as long as we don't change the actual udf name we are fine11:13
gatoxbrb.... machine die11:16
alecuhello, all!12:13
gatoxalecu, hi12:14
gatoxalecu, mandel any of you want to start the morning with a review? (not much morning for you mandel) https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/12831212:15
mandelgatox, I can start the afternoon :)12:15
gatoxmandel, sounds good :P12:16
ralsinagood morning!12:17
gatoxralsina, hi12:17
ralsinahola gatox12:18
mandelgatox, looking at your branch, I think you might abusing lambdas a little to much, I mean, you are creating a new lambda each time you add a new handler when you could simply appending to the collection, right?12:18
mandelgatox, is the lambda really needed or could be have a class private method that does it?12:21
gatoxmandel, if you use a private method, you will need a lambda anyway, because you will need to do something as: lambda *args, **kw: _method(signal_name)12:26
gatoxmandel, if you use a private method, you will need a lambda anyway, because you will need to do something as: lambda *args, **kw: _method(signal_name, *args, **kw)12:26
=== muffinresearch_ is now known as muffinresearch
mandelgatox, lets look at it step by step: first improvement I see => http://paste.ubuntu.com/1269167/12:31
mandelgatox, there is no need to create a new lambda each time you add a handler, right?12:31
gatoxmandel, we will find out when we run the tests..... i would say no..... but i'm not sure if maybe that binds to the value of the list at that moment12:32
gatoxmandel, ok........... it was just a precaution..... this works tooooo12:34
mandelgatox, next improvement => http://paste.ubuntu.com/1269176/12:35
mandelgatox, that will at least allow us to test that all handlers are called, improves testability, which I think is an improvement..12:36
gatoxmandel, we can test that already12:36
gatoxmandel, actually, we are doing it in the tests12:38
gatoxmandel, i can change that if it's necessary..... maybe we can ask someone else, because i actually prefer the lambda solution12:42
mandelgatox, do think about memory etc.. creating a new lambda object for every new handler that has to be gc is an overhead that is not really needed12:43
* ralsina looks around12:43
ralsinamandel, gatox: you guys asked for a 3rd opinion?12:43
gatoxmandel, mmmmm.... i see12:44
gatoxmandel, now i'm more convinced12:44
gatoxralsina, i think that is not necessary anymore12:44
gatoxralsina, mandel has presented a new argument that is more valid to me12:44
* ralsina points out that 500 lambdas are less than 20KB or RAM and take a microsecond to gc12:44
* ralsina is great at making up numbers12:44
ralsinaI would prefer "pretty, readable code" but I leave it to you gatox12:45
gatoxralsina, i'll change it to the private function12:45
ralsinagatox: cool12:46
gatoxi only needed a good reason :P12:46
mandelgatox, ralsina, just so that I can add a new diff way to do it, you can use something like this: http://paste.ubuntu.com/1269200/12:50
mandelgatox, ralsina, add a new handler list if not present, and append the callbacks, then the list can be called with the *args and **kwargs and will do the right thing..12:50
mandelbut might be too evil :)12:50
mandelgatox, ralsina, you reduce the need of a lambda, you use the same amount of memory as a single list and you simple add a new type definition12:51
mandelbut I think I have been doing too much c/c++/golan recently and I have started thinking about memory too much :P12:52
ralsinamandel: that's a list of lambdas disguised as a class12:52
ralsinamandel: disguised as a functiokn12:52
mandelralsina, no, is not, in the other you a list with with callbacks and per callback that you added to the list you were creating a new lambda12:52
mandelralsina, so, in the second handler added, you had the old lambda (first handle) the 2 handlers (2 lambdas) then create the new lambda (2 handler lambda) and hope that the first lambda was gc at some point12:53
ralsinamandel: I really really hate "for cb in self"  :-)12:53
mandelralsina, in this case you have a list with the callbacks and there is no need for the generic lambda or the new creation of a lambda12:53
mandelralsina, as I said, a little evil :)12:54
ralsinamandel: I would just keep a list of functions and iterate. Explicit.12:54
ralsinabut anyway, it's not worth discussing this much, I think. Clear code is best.12:54
mandelralsina, yes, I was just having fun by stretching the subclassing model and wondering how little code I could write to do the job and use the less amount of objects hehehe12:55
mandelralsina, lets call it brainexercise12:55
mandelok, lunch time here :)12:55
* gatox is not convinced again.....12:55
mandelgatox, I'll do a second review when i'm back :)12:55
ralsinagatox: try to write it as dumb as possible12:56
ralsinagatox: remember I may have to read it later!12:56
gatoxralsina, for mandel?12:56
mandelwrite for the common denominator :)12:56
gatoxi'll do it with a private method.......12:56
* gatox sacrifice his love for list comprehensions12:56
ralsinagatox: if it needs a \ it's a bad list comprehension :-)12:57
gatoxralsina, it didn't need that....... because i was splitting the line inn the right places jejejeje12:57
ralsinagatox: oh, but then it had a \n ;-)12:57
* ralsina goes track when we started having a double spinner12:58
gatoxmandel, branch updated12:58
gatoxralsina, do you have any special task for me?? or i should try to start killing u1-cp bugs?.... i might found a weird bug in u1-client too12:59
gatoxmight not....... i found12:59
ralsinagatox: yes, check https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AiE68jdtyfBcdG1RTkpEYl8yNV9yaEtIRkdUbF9OZGc#gid=0 and make it work :-)13:00
gatoxbut i don't know if it is a priority13:00
gatoxralsina, ack..... really clear list of tasks13:00
gatoxralsina, mmmmm..... some weird stuff has been reported there..... i'll need to track each person and ask13:05
ralsinagatox: I think I fixed one of those at least13:05
ralsinagatox: but not on stable-4 :-(13:05
gatoxralsina, also.... maybe something is not clear..... because it seems they expect the things being transfered from the server to the client to appear there too13:06
ralsinagatox: then you need to clarify the leftmost column :-)13:07
=== m_conley_away is now known as m_conley
mandelgatox, got a small needs fixing14:13
gatoxmandel, ack14:13
gatoxmandel, fixed14:17
gatoxmandel, it was there for when i was testing stuff14:17
ralsinagatox: this bug is ours, right? https://bugs.launchpad.net/ubuntu/+source/indicator-sync/+bug/106395014:23
ubot5Ubuntu bug 1063950 in indicator-sync (Ubuntu) "Menu item "Get more space" opens "one.u.c/services/#storage_panel" instead of "one.u.c/services/add-storage/"" [Undecided,New]14:23
* gatox checking...14:23
ralsinagatox: as in, that's really a bug in u1-client not in the sync menu14:23
gatoxralsina, i'm fixing that right now14:23
ralsinagatox: awesome, change the affects in the bug14:23
gatoxralsina, ack14:23
mandelgatox, do you think that adding a test where you add a handler to one signal and an other handler to an other signal and assert the correct handler is called is a good idea?14:31
gatoxmandel, will do in a few miins14:32
gatoxin the meanwhile..... mandel ralsina  if you can: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/add-storage/+merge/12872914:34
mandelgatox, sure14:37
gatoxmandel, new test added: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/12831214:44
mandelralsina, gatox, alecu the ISP eng is here to install fiber in my place, I'll me off line a little until he is done14:44
ralsinamandel: ack14:46
gatoxmandel, ack14:46
ralsinaalecu and I are on a call we'll miss standup, you guys go ahead without us14:50
alecumandel: fiber == awesome!14:50
ralsinamandel: you getting whole bran?14:52
ralsinamandel: if that doesn't work, try activia!14:52
ralsinagood morning mmcc!14:52
mmcchi ralsina14:52
gatoxmmcc, hi14:53
gatoxmmcc, i have a present for you: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/12831214:53
mmccit's a fine day for reviews14:54
ralsinammcc: it's QA day!14:54
mmccralsina: tell me more14:55
ralsinammcc: today QA tests our mac build to see if we do the beta release on the 11th14:55
ralsinammcc: so... let's do a build and give it to QA :-)14:55
ralsinaelopio: it says "windows testing" on the schedule, but it should be mac testing14:57
elopioralsina: you said you wanted a windows testing session this week.14:58
gatoxmmcc, first review my branches that fix some of the problems we were having :P14:58
elopiodon't you want it anymore? :)14:58
gatoxmmcc, so you can include them in the build \o/14:58
ralsinaelopio: yes, that too :-)14:59
ralsinaelopio: I am confusing sometiimes, I know, sorry14:59
mmccralsina: aha, ok. are they expecting anything to have changed since the last build? gatox' change shouldn't affect the final behavior…14:59
mmccbecause the last qa build had a workaround14:59
ralsinammcc: I don't know what the last build was, really14:59
elopioralsina: so, are we having a windows installer too?14:59
ralsinammcc: I would like to get a fix I did yesterday14:59
ralsinaelopio: there are daily windows builds in jenkins again now14:59
mmccralsina: aha, ok. great15:00
ralsinaelopio: but I am not sure it even does something on mac15:00
briancurtinralsina, elopio: jenkins is down again...15:00
ralsinabriancurtin: oops15:00
briancurtinwill take a look shortly15:00
ralsinabriancurtin: jenkins is down or builds are failing?15:00
briancurtinralsina: the windows jenkins machine isn't connected. everything is N/A15:01
ralsinathat one about "I am not sure..." was for mmcc sorry15:01
ralsinabriancurtin: ok, it's tuesday, which is "break jenkins" day15:01
gatoxneed to go downstairs for a sec..... so here is my standup:15:03
gatoxPropose a branch to start fixing some u1-client sync menu related stuff, improve some other branches already proposed.15:03
gatoxKeep fixing u1-client sync menu and u1-cp bugs.15:03
thisfredDONE: processed email/colonial imperialist day TODO: playlists BLOCKED: no NEXT: gatox15:04
briancurtinDONE: u1c and u1cp tests running fine (well, some failures on client, but they run to completion)15:04
briancurtinTODO: CredentialsError still coming from SSO, working out how its finding the binaries. i found one place that is wrong but then it seemed to try again with the right commdand line and failed.15:04
briancurtinTODO (on the side): i noticed we're sending post2008server in a URL post while logging in to SSO (can't remember the exact string, will dig it up). this is from the stdlib platform module not identifying Windows 8. i dont know what we actually use that for on the server side, probably stats, but it's an easy change in the stdlib. i did the change for win7/2008server15:04
briancurtinNEXT: mmcc15:04
mmccDONE: long weekend, recent transfers menu15:04
mmccTODO: reviews, set up .DMG, qa build15:04
mmccBLOCK: no15:04
mmccwho are we missing, ralsina mandel alecu? or are more people off today15:06
* mmcc needs to check the logs15:06
alecummcc: both ralsina and I are on a hangout15:06
mmccalecu: ack15:07
* ralsina declares this "day columbus noticed this was not japan" day.15:08
briancurtinbrb, need to let the heating/furnace dude in. it's cold in here.15:23
mmccgatox, quick needsfixing on that client branch15:24
mmccgatox: I'm also still not 100% clear why we need that change. when were multiple functions being connected to the same signal?15:25
gatoxmmcc, i already explain that and show you some code from u1-client on friday..... and you answered that you understand15:26
gatoxmmcc, also.... i explain it on the comment i added15:26
mmccgatox: sorry if I'm just forgetting - I'll look in the log on friday.15:27
gatoxmmcc, or read the comment15:27
mmccgatox: ok, I had to read the comment again, sorry - can you point me to the place in syncdaemon core that was also connecting to the signal? not urgent though, I believe you - this is just for my understanding15:29
gatoxmmcc, could you check your logs for that?? because i don't remember and i'll need to debug to find it..... i already send that piece of code to you on friday..... if you don't have it let me know..... and i'll check15:30
mmccok, I'll look gatox. don't worry about it15:30
gatoxmmcc, thx........ because it was kind of tricky to find it15:32
mmccgatox: yeah, I definitely know how that is15:33
mmccthat's a good argument for keeping notes on this stuff - whenever I figure out one of these long IPC chains, I write down what calls what - it's already helped a couple times when I had to go back and check something15:34
alecuhello Hello!15:34
gatoxmmcc, docstring updated15:35
mmccgreat gatox, thanks - now to try testing it on macos15:40
mmccgoodbye Hello15:55
mmccoh I mean HELLO15:55
* gatox lunch16:01
=== gatox is now known as gatox_lunch
ralsinaapparently mandel's ISP guy is still trying to connect the optic fiber using duct tape or something, will take a while longer to show up16:07
briancurtinha. fiber to the house, duct tape the rest of the way16:08
ralsinabriancurtin: the last mile is a sticky problem16:08
mmcchrmm. so this branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/128312 with control-panel *trunk* fixes the share links tab for me IRL on osx16:28
* mmcc waves16:29
gatox_lunchmmcc, but you will need this cp branch too: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/u1-cp-publishapi/+merge/128316 or you will only be able to share a file once16:31
mmccgatox_lunch: aha! thanks116:32
* mmcc was getting very confused16:32
mmccralsina: how do I check that your change from yesterday works on os x?16:45
ralsinammcc: hmmm...16:46
ralsinammcc: run ubuntuone-control-panel-qt --switch-to=devices?16:46
mmccralsina: with or without credentials?16:47
ralsinammcc: with16:47
mmccok, works for me16:48
ralsinammcc: cool16:48
ralsinammcc: OTOH, if you have a "share a file" link in the menu icon's popup, does it open in the right tab?16:49
ralsinammcc: if it doesn't that's a separate bug16:49
mmccralsina: share a file link in the menu icon? I don't know what that is16:50
ralsinammcc: in the icon in the mac global menu, do you have a "share a file" item?16:50
mmccralsina: no, and looking, that's a # TODO in systray.py16:51
ralsinammcc: ack16:52
ralsinagatox_lunch: row 8 in the sync indicator testing sheet needs to clarify that's not implemented and a bug link16:53
gatox_lunchralsina, ack16:53
gatox_lunchralsina, mmcc that was because de sync menu was implemented before de shares tab16:54
mmccralsina, gatox_lunch: did you see this bug from last friday: https://bugs.launchpad.net/ubuntuone-client/+bug/106272916:55
ubot5Ubuntu bug 1062729 in Ubuntu One Client "sync menu uses wrong upload size for percentages" [Undecided,Confirmed]16:55
mmccwondering if that's on the testing sheet - you probably won't notice unless you have lots of highly-compressible large files :)16:56
ralsinammcc: saw it, did nothing about it16:56
ralsinammcc: shhhh ;-)16:56
mmccI noticed it when uploading a ~200MB text .sql file16:56
ralsinammcc: yeah, those files are uncommon16:56
ralsinaplus, it will just show "slow slow slow done!"16:56
mmccheh :) ok, like I said, it's mostly a minor issue but it made debugging other stuff very confusing :)16:57
mmccyeah, it was showing "9%" forever and I started looking for string truncation or something :)16:57
ralsinagatox_lunch: so, after that one is marked as unimplemented we are down to ~2 issues with sync menu. Not horrible.16:59
=== gatox_lunch is now known as gatox
gatoxralsina, yap17:00
mmccgatox: +1 on https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/128312 -- you still need to get mandel to approve that though17:05
gatoxmmcc, yes..... he said he will review it when he is back17:07
mmccgatox cool. also +1 on the other branch17:07
gatoxmmcc, cool.... thx17:07
mmccthanks for being patient with me while I figured out what was going on there17:07
gatoxmmcc, no problem :D17:08
mmccgatox: it would've been easier if your comment here during lunch about "you will only be able to share a file once" was in the MP description under "HOW TO TEST" or something :) I can look at the code and understand what it's doing, but sometimes it's hard to understand what it's fixing at the top level17:09
mmccok, done bugging gatox for the day :)17:09
gatoxmmcc, jeje ok.... will add that next time.... promise :P17:10
gatoxralsina, so....... i've marked the share a file option in the menu as not implemented, and added the bug link..... now we have 2 reds..... and i already propose a branch for one..... and the remaining one, is the crazy thing that is happening with the shares that i mentioned17:23
gatoxralsina, so i'm going to start working on that one if you agree..... probably i'll need to contact people from the server team, because i'm not sure if they are sending that data now17:24
gatoxralsina, here is the branch for the other bug..... mandel already approve it..... is one line change: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/add-storage/+merge/12872917:25
=== eu is now known as Guest69817
ralsinagatox: agreed17:38
ralsinagatox: looking at the branch17:39
ralsinagatox: global +117:39
gatoxralsina, thx17:39
ralsinagatox: well, there's also the "sync menu doesn't show the cloud icon" but that's not us17:42
gatoxralsina, mmmmm i think that maybe i can prioritize this one: "Status in indicator is not updated when the status is changed from U1 control panel"17:43
gatoxand then the new shares one17:43
ralsinagatox: happy either way17:43
gatoxralsina, ok17:43
mandelI'm back!!!17:44
mandelralsina, sorry, the guy from movistar/telefonica had some serious problem getting the connection17:45
ralsinamandel: ack17:45
ralsinamandel: np, it happens17:45
mandelralsina, at least now I have fiber :)17:46
mandelwhich makes me very very happy17:46
ralsinamandel: yes, slow transit is bad17:51
mandelralsina, in nearly everything :)17:53
mmccgatox, what is the bug # for that is that indicator not updated bug?17:54
gatoxmmcc, sorry, what?17:55
gatoxmandel, please: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/12831217:55
mmccgatox - this one from ten minutes ago: "ralsina, mmmmm i think that maybe i can prioritize this one: "Status in indicator is not updated when the status is changed from U1 control panel" "17:55
mandelgatox, sorted17:56
gatoxmmcc, that one is only for the sync menu in ubuntu17:56
gatoxmandel, ack, thx17:56
mmccgatox: ok, cool17:56
mmccyay, nice looking .dmg with background image and everything, all set up18:10
gatoxmmcc, cool18:11
mmccralsina: for sending to QA, is that just ASAP or when?18:11
ralsinammcc: asap please18:12
mmccralsina: okboss18:12
mmccgatox: is https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-publishapi/+merge/128312 waiting on anything? mandel approved but didn't flip the switch18:18
mandelmmcc, I did not 'cause I though we agreed to the owner to do it.. or am I outdated?18:18
gatoxmmcc, i'll do it....... also..... this branch includes all the changes..... and there was the other one with the workaround that didn't landed..... and i think i can delete it18:18
mmccmandel, you're not outdated - I was poking the owner :)18:19
mmccgatox: you're talking about the control-panel branch where you used the return value instead of the signal right?18:19
mmccwas there a workaround branch for u1-client too? I remember one that changed a comma that did land, but it was just a comment change once it landed18:20
gatoxmmcc, ahhh no....... i was confuse..... just for u1-cp18:21
mmccok cool18:23
chaselivingstonmmcc: will the current and recent transfers menu item work in the public beta?18:27
mmccchaselivingston: no18:27
mmccin fact I had a note to myself, maybe we should just not show it. ralsina?18:28
chaselivingstonmmcc: ok, good to know, that might not be a bad idea18:28
ralsinaif it's broken, yes, disable it18:28
ralsinawe can re-enable it a week or two later18:28
mmccralsina: ack. will do18:28
gatoxralsina, do you know why is broken? it has anything to do with the changes that you talk about in u1-client files progress?18:29
mmccralsina: what's the best way to do that, just have a side branch that comments it out and build with that?18:29
mmccgatox: I know why it's broken, I was working on it on friday18:29
gatoxmmcc, can you tell me why?18:29
mmccI don't know why it doesn't work on linux, but on osx, the timer is not called to populate the menu18:29
ralsinammcc: yes18:30
ralsinaI didn't know it was broken :-/18:30
mmccgatox, in the TransfersMenu __init__ it only starts the timer if sys.platform isn't 'win32' or 'darwin', and I'm not sure why… but I have some code to fix that and try to get live-updating working on osx too, so it doesn't matter too much18:31
gatoxmmcc, ack18:32
mandelsorry guys, but I have to call it a day, is close to dinner time and have to walk the dog, I'll work a couple of extra hours tom18:36
gatoxmandel, bye18:38
=== slank` is now known as slank
IpsilonDoes ubuntu one lock files? I misread and thought it supported webdav.19:28
ralsinaIpsilon: no, to both19:28
IpsilonOk, thanks. That's too bad, would have been perfect for zotero.19:29
ralsinaIpsilon: if the app does locking through a lock file, the lock file itself will be synced but that's not reliable at all19:32
ralsinaIpsilon: I don't know of any sync solution that reliably locks files19:32
IpsilonWell, webdav works. I believe through locking files...although maybe it's through a different method.19:33
IpsilonWorks for zotero, that is.19:34
mmccok, finally have current build uploading for QA. ralsina who do I ping when it's there?19:46
mmccOK, new build for QA here: http://ubuntuone.com/1Ny7EqmY50jLxRaufZLnrX19:51
mmccI guess ping elopio --^19:52
ralsinammcc: elopio19:55
ralsinammcc: and jfunk19:55
ralsinammcc: and joshuahoover19:55
joshuahooverchaselivingston: ^^19:56
chaselivingstonsaw all this, thanks, working on installing it now19:56
ralsinachaselivingston: sorry I forgot about you :-)19:57
chaselivingstonralsina: no problem ;)19:57
gatoxok..... i need to fix some tests and i'll have the sync menu and syncdaemon always in sync regarding the connection status20:02
gatoxi'll propose a branch for this tomorrow morning20:02
gatoxeod here!20:02
gatoxbye people!20:02
chaselivingstonmmcc: neat choice on the black menubar icon lol20:06
mmccchaselivingston: is that a good lol or a bad lol?20:07
chaselivingstonmmcc: i think it's a good, we'll see after i use it for a while20:07
chaselivingstonmmcc: just can't wait to be able to get rid of the icon in the dock ;)20:08
mmccchaselivingston: understood. for now, how about just getting rid of the dock? I hide that sucker, makes me happier :)20:09
chaselivingstonmmcc: haha, i've done that in the past, but like having it shown most of the time20:09
mmccit takes all kinds I guess ;_20:09
mmccer, ;P20:09
ralsinachaselivingston: we do have a way to do that, but we needed to fix the more critical stuff first20:10
chaselivingstonralsina: sure, i understand. i know it'll happen eventually20:10
mmcctrivial review for setup-mac anyone?20:24
briancurtinmmcc: i can't test, but my eyes say approve20:28
mmccthanks briancurtin. I assert that it works :)20:30
chaselivingstonnice work on the mac app guys, i think we're in a very good place for release on thursday20:31
mmccthanks chaselivingston , that's good to hear20:31
chaselivingstonmmcc: can't wait to get some user feedback on it20:31
ralsinachaselivingston: yay!20:33
mmcca couple other really small packaging branches for review: https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/bump-version/+merge/128813 and https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/embiggen-log/+merge/12881420:33
mmccralsina: can you in particular check the version # one?20:34
ralsinammcc: sure20:34
ralsinammcc: +1 cromulent in embiggen20:35
ralsinammcc: also +1 on the bump20:36
ralsinawe really need to automate that, we have like 5 version number-like things in -installer20:36
mmccralsina: now for the trifecta, shall I globally approve the https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/make-cp-fg/+merge/12881120:36
ralsinammcc: looking...20:36
ralsinammcc: I have no idea what the flag does :-)20:37
mmccralsina: yes definitely - also currently syncdaemon's version number is always 0.1 on mac/win20:37
mmccralsina: hmm, let me go look and see what it does… "_20:37
mmccit's is_background, and True means you have no menu bar or dock icon. False means you do20:38
ralsinammcc: we change that in the logs manually on windows in the last release20:38
ralsinammcc: we should do that for the real release on mac too20:38
ralsinammcc: for which process is this setting?20:38
mmccthe last few builds have been built with is_background=False, giving us an icon. this is for controlpanel20:39
mmccthis setting change is old, I just haven't gotten around to pushing the change20:39
chaselivingstonmmcc, ralsina: what is the process going to be for public testers as far as updates to the app go?20:39
ralsinammcc: ok, yes global +120:40
mmccchaselivingston: what do you mean exactly?20:40
mmccralsina: great, thanks20:40
chaselivingstonmmcc: how will updates be pushed to users?20:40
ralsinachaselivingston: we don't have an autoupdater for mac yet20:41
mmccchaselivingston: yeah, TODO20:41
ralsinachaselivingston: we need to work on one. at least something that pops a "get the new version" message20:41
chaselivingstonralsina: right. so will users have to come back to the download page to download updates?20:41
ralsinachaselivingston: yeah, let's keep it simple20:41
chaselivingstonralsina: ok, so for now they're going to have to either watch that page, or perhaps social media to know when to grab updates?20:42
ralsinammcc: maybe we should timebomb the beta :-)20:42
ralsinachaselivingston: yes, social media && eyeballs20:42
ralsinachaselivingston: the next one should have a simple updater20:42
chaselivingstonralsina: ok, we probably need to make sure that's clear in the beginning, i'll need to make sure roberta knows about that20:43
ralsinachaselivingston: yes, I'll tell her to add something in the communication20:43
chaselivingstonralsina: ok cool20:43
chaselivingstonralsina: i'll leave a note in the gdoc for her20:44
ralsinachaselivingston: awesome20:44
mmcchrm. timebomb is appealing. what would be nice is if we had a web api call to ask if there's any info to show the user… like GET /message?clientversion=foo , which can optionally give us some HTML to show to the user, a place for "HEY YOU ARE USING AN OLD VERSION" or whatever20:45
mmccI've seen that pattern in other apps, looks handy20:45
ralsinammcc: we have that for windows20:45
mmccoh hey, great!20:45
ralsinammcc: but there it's done using a huge binary installbuildre creates and a xml file20:46
mmccoh hey, great?20:46
ralsinammcc: I think we can do a much better thing using 10 lines of code and a file somewhere that says 1234 (or correct number)20:46
* briancurtin reboot20:49
mmccralsina: probably, yeah. it can be useful to set it up to display a message downloaded from a known place too20:50
mmccgood for release notes or explanations of why it's important to update20:50
=== m_conley is now known as m_conley_away
mmcclaaaate lunch time here21:57
ralsinaEOD for me22:01
ralsinabye people!22:01
mmccok, lunch was late enough that I have to call it a day. I'll be back a bit later tonight too22:40

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