=== m_conley is now known as m_conley_away | ||
=== trijntje__ is now known as trijntje | ||
mmcc | OK, going to sleep now - I've deposited a reasonably big merge proposal with a lot of awful ctypes boilerplate for CoreFoundation functions over here: https://code.launchpad.net/~mikemc/ubuntuone-control-panel/launchdaemon/+merge/124847 | 07:12 |
---|---|---|
mmcc | As a special reward for anyone who wants to review it, it has to be code signed to test IRL, so you'll need my build. | 07:25 |
mmcc | …and it's up - here is a build that suffices to test the proposed branch above: http://ubuntuone.com/4KKHgfhAK46ABkOuvrdZci | 07:47 |
mmcc | and I'm out! | 07:48 |
mandel | morning all! | 07:57 |
ramon | morning | 08:01 |
ramon | ^^ | 08:01 |
JamesTait | Happy Tuesday, folks! :-D | 08:17 |
j0nr | morning | 08:50 |
j0nr | funny problem | 08:50 |
j0nr | web player, playing a song, keeps stopping midway. | 08:50 |
j0nr | rest of songs is there if I drag the timeline along and plays other parts of song, just if I play it from the beginning it stops midway | 08:50 |
rye | j0nr: what browser and how reproducible is that? | 10:01 |
gatox | good morning! | 11:17 |
mandel | gatox, morning! | 11:28 |
gatox | mandel, hi | 11:38 |
ralsina | good morning! | 11:47 |
gatox | ralsina, hi | 11:47 |
=== eu is now known as Guest45327 | ||
dobey | hmm | 12:32 |
ralsina | dobey: did you do the u1-client release yet? | 12:33 |
dobey | no | 12:33 |
ralsina | dobey: because trunk is broken on ubuntu. I'll have a fix in about 15' though | 12:33 |
dobey | broken? | 12:33 |
ralsina | dobey: yes, syncdaemon doesn't start because of a mac-related config change | 12:34 |
dobey | ah | 12:34 |
ralsina | revno 1311 to be exact | 12:34 |
dobey | and gatox's changes haven't landed yet have they? | 12:35 |
gatox | dobey, no | 12:36 |
gatox | dobey, did you create the package yesterday? | 12:36 |
dobey | no | 12:39 |
ralsina | dobey, gatox: easy review https://code.launchpad.net/~ralsina/ubuntuone-client/exorcize-daemon/+merge/124906 | 12:42 |
gatox | dobey, so, if i propose the sync menu today and it lands..... it will be included in the package? | 12:42 |
dobey | gatox: not necessarily, no. the FFe isn't approved afaict | 12:44 |
gatox | ack | 12:44 |
ralsina | gatox: is the other half of it (the unity side) already released? | 12:44 |
gatox | ralsina, yes | 12:44 |
ralsina | gatox: ok | 12:45 |
dobey | see https://bugs.launchpad.net/ubuntuone-client/+bug/1042343/comments/3 | 12:45 |
ubot5 | Ubuntu bug 1042343 in Ubuntu One Client "[FFE] Ubuntu One integration with Q sync indicator" [High,Triaged] | 12:45 |
dobey | joshuahoover: ^^ | 12:45 |
alecu | hola, all! | 12:45 |
ralsina | mmcc: so, the right way to change what fsm is the default on mac is, I think, to rename them, and have "daemon" called "default" and "default" be called "userspace" or something | 12:46 |
ralsina | dobey, gatox, joshuahoover: I have spoken with jfunk to schedule testing for sync menu for this thursday. | 12:47 |
gatox | ack | 12:47 |
dobey | i'm not sure we actually need all that. we certainly do need the link to the mail in the docs/translations teams archives | 12:50 |
ralsina | dobey: we can use what we did before about "we have internal QA"etc | 12:50 |
mandel | ralsina, you are right, my fault for reviewing the branch :-/ | 12:50 |
ralsina | mandel: so did I | 12:50 |
ralsina | mandel: and so did mmcc I suppose ;-) | 12:51 |
mandel | ralsina, good news are, we are close to get the preview done as per design request: http://ubuntuone.com/7Xy0W9QCu4SMeDB2UH2s8G | 12:51 |
ralsina | mandel: and we have no tests for "configuration is sane" | 12:51 |
dobey | ralsina: we still need the notification of docs/translations teams, regardless of testing; this does introduce new strings (although I suppose docs will be a quick 'yeah we don't have u1 docs' ack) | 12:51 |
mandel | ralsina, I hope to be done soon, then help alecu and ask for something to do :) | 12:51 |
ralsina | mandel: awesome | 12:51 |
ralsina | I wonder what a "sane configuration" test would be. Start sd and wait 30 seconds, see if it's still there? | 12:52 |
mandel | lunch time here | 12:53 |
* mandel -> lunch | 12:53 | |
dobey | ralsina: i suppose it might help if the current syncdaemon integration tests actually worked :) | 12:53 |
ralsina | dobey: hehe | 12:53 |
ralsina | dobey: we are not running those routinely, and we should put them somewhere to catch this kind of thing | 12:54 |
dobey | jenkins is running them routinely | 12:54 |
dobey | but the server does not work on precise, so it always fails | 12:54 |
ralsina | ack | 12:55 |
ralsina | I thought the server was running on P nowdays | 12:55 |
dobey | not according to jenkins :) | 12:58 |
=== yofel_ is now known as yofel | ||
ralsina | If bug #1052470 doesn't convince devs that UX is hard, nothing will. | 13:09 |
ubot5 | Launchpad bug 1052470 in ubuntuone-control-panel (Ubuntu) "ubuntu one logs out when removing a device" [Undecided,New] https://launchpad.net/bugs/1052470 | 13:09 |
dobey | UX isn't hard; and this isn't the first time that issue has come up | 13:11 |
ralsina | dobey: UX is hard, and yes, it's a duplicate. | 13:12 |
ralsina | dobey: at least GOOD UX is hard | 13:12 |
alecu | ralsina: we should make the list of devices "non-selectable". And move the "delete" button to the left, just right of the current device name. (and perhaps make it a link) | 13:20 |
alecu | ralsina: I think we used to have a "remove" button for each of the devices.... that's why that button was right aligned. | 13:20 |
ralsina | alecu: yes to all of that except the link probably. Button feels natural. | 13:20 |
ralsina | alecu: yes, we removed those because "it was confusing" IIRC | 13:21 |
alecu | ralsina: but button would look weird left aligned | 13:21 |
dobey | alecu: if we're going to not have a way to remove the devices, we should remove the list of devices | 13:21 |
dobey | alecu: and the button should say "Log Out" | 13:21 |
ralsina | dobey: actually, I have talked on and off with design about having a real logout story | 13:21 |
ralsina | dobey: which is one of the reasons why this specific bug has never been tackled seriously | 13:22 |
ralsina | and when I say "tackled seriously" I mean "I never assigned any resources to it" | 13:23 |
nessita | ralsina: I approved the branch at https://code.launchpad.net/~ralsina/ubuntuone-client/exorcize-daemon/+merge/124906, but was wondering if you could add a test to confirm the default does not change (or to confirm the default monitor is "findable" in the platform being used) | 13:31 |
ralsina | nessita: that's covered by the integration tests | 13:32 |
ralsina | nessita: thing is, they were not being run, but now they are | 13:32 |
nessita | ack | 13:32 |
* nessita approves gloablly | 13:32 | |
nessita | (the branch, not the whole world) | 13:32 |
dobey | actually, a unit test for that probably makes sense | 13:32 |
dobey | rather than relying on external integrationt ests | 13:33 |
ralsina | dobey: every item in the config can stop sd from starting | 13:34 |
ralsina | dobey: so the test would be a copy of the config and a call to cmp :-) | 13:34 |
dobey | ralsina: see tests/syncdaemon/test_config.py | 13:35 |
dobey | if we have a config value that isn't tested there, especially one which references a python module name, it's a bug :) | 13:37 |
=== m_conley_away is now known as m_conley | ||
=== slank` is now known as slank | ||
=== slank is now known as Guest61577 | ||
dobey | ugh, ohloh | 13:53 |
* mandel back | 14:00 | |
mandel | alecu, ping | 14:00 |
ralsina | dobey: marcus is answering in #canonical, please pick it up I am otp :-) | 14:00 |
alecu | mandel: pong | 14:02 |
mandel | alecu, mumble to catch up with the payment stuff? | 14:02 |
alecu | mandel: can we do it in 15'? | 14:02 |
mandel | alecu, sure :) | 14:02 |
mandel | alecu, I'm at mumble, just pop in when ever you can | 14:11 |
mandel | alecu, http://ubuntuone.com/7Xy0W9QCu4SMeDB2UH2s8G | 14:17 |
mmcc | hi folks. | 14:20 |
mmcc | ralsina: thanks for that quick fix. in retrospect I really should've tested that branch more. :| | 14:21 |
=== mbarnett` is now known as mbarnett | ||
mandel | alecu, video: http://ubuntuone.com/3HZBt7XP4gE9jCU3GxVDsd | 14:23 |
ralsina | mmcc: np, neither reviewer saw it. It happens. | 14:24 |
ralsina | mmcc: did you see my suggested "real fix"? Renaming the mac fsm implementations? | 14:24 |
dobey | thisfred, mmcc: https://code.launchpad.net/~dobey/ubuntuone-client/update-linguas/+merge/124934 | 14:26 |
mmcc | ralsina: yes, I saw that. I'll do that today | 14:28 |
* thisfred reviews | 14:28 | |
ralsina | mmcc: awesome | 14:28 |
ralsina | mmcc: get a second opinion first ;-) | 14:29 |
thisfred | dobey, +1. Didn't know the ca@valencia form. Is that valid? or is it schemaless anyway? | 14:29 |
thisfred | probably not for us to care about | 14:30 |
thisfred | if it comes out of LP | 14:30 |
dobey | thisfred: it's valid, yes. | 14:30 |
dobey | just like en_US@ebonics would also be valid :) | 14:32 |
* ralsina will start typing in en_US@valleygirl | 14:32 | |
ralsina | in any case, I am off to find a power outlet | 14:32 |
ralsina | probably lunch too, do standup without me if I am not here yet by that time | 14:33 |
mandel | ralsina, one sec in mumble with alecu and I? | 14:33 |
ralsina | mandel: like literally | 14:33 |
mandel | ralsina, sure | 14:33 |
mmcc | dobey: I'm happy to review that but I'm not sure how, exactly… | 14:33 |
dobey | for i in `cat LINGUAS`; do if [ ! -e $i ]; then die 'OMG YOU LIE'; fi; done ? | 14:34 |
dobey | oh, sorry; -e "$i.po" | 14:35 |
dobey | or something like that anyway | 14:35 |
dobey | maybe i should add a check to intltool's check rule to verify the LINGUAS file is correct | 14:36 |
dobey | or just rubber stamp it | 14:37 |
dobey | it doesn't really need 2 reviews | 14:38 |
alecu | 4 mins to standup | 14:57 |
* alecu writes notes | 14:57 | |
thisfred | notes schmotes, I'll just wing it | 14:59 |
gatox | me | 15:00 |
alecu | me | 15:00 |
briancurtin | me | 15:00 |
thisfred | me | 15:00 |
mandel | me | 15:01 |
mmcc | me | 15:01 |
dobey | me | 15:02 |
gatox | DONE: | 15:02 |
gatox | Sync Menu branch almost complete, start fixing some tests related to shares, and sync status. | 15:02 |
gatox | TODO: | 15:02 |
gatox | Finish fixing the tests and propose. | 15:02 |
gatox | BLOCKED: | 15:02 |
gatox | No | 15:02 |
gatox | alecu, go | 15:02 |
alecu | NEXT: tech leads call, fixed a broken Q vm, refactor in lens code | 15:02 |
alecu | TODO: more cleanups, find a suitable PPA for dash & lens work | 15:02 |
alecu | BLOCKED: no | 15:02 |
alecu | NEXT: briancurtin | 15:02 |
briancurtin | DONE: fiddle around with oauth related webclient tests | 15:02 |
briancurtin | TODO: going to step back and work on other tests besides test_webclient to get a small break from that since it hasn't been overly productive the last few days | 15:02 |
briancurtin | NEXT: thisfred | 15:02 |
thisfred | DONE: ubuntuone-servers u1db playlists TODO: finish <- together with webm0nk3y BLOCKED: no NEXT: mandel | 15:02 |
mandel | DONE: Update payment preview following the final design => http://ubuntuone.com/7Xy0W9QCu4SMeDB2UH2s8G Fixed some issues in the branch I proposed to unity to fix global search + preview. | 15:02 |
mandel | TODO: Finish new style (spacing). Fix PreviewStateMachine to have a decent transition. | 15:02 |
mandel | BLOCKED:no | 15:02 |
mandel | mmcc, please | 15:02 |
mmcc | DONE: daemon upgrade done, discussed yes/no dock icon with lisette | 15:02 |
mmcc | TODO: rename daemon fs_monitor, daemon event processing bug with move | 15:03 |
mmcc | BLOK: | 15:03 |
mmcc | NEXT: dobey | 15:03 |
dobey | DONE: releases, merged in available new old icons, bug #979130 x 3, bug #1052131 | 15:03 |
dobey | TODO: finish releases, finish new old icons?, webapp packaging | 15:03 |
ubot5 | Launchpad bug 979130 in Ubuntu One Client stable-4-0 "Update *.po files before make a new release" [Undecided,Triaged] https://launchpad.net/bugs/979130 | 15:03 |
ubot5 | Launchpad bug 1052131 in Ubuntu One Client stable-4-0 "distcheck broken by vala support" [Undecided,Triaged] https://launchpad.net/bugs/1052131 | 15:03 |
dobey | BLCK: None. | 15:03 |
dobey | ralsina: despair | 15:03 |
gatox | fixed broken q vm counts as a task?? add several of those in my done :P | 15:03 |
dobey | thisfred or mmcc: if one of you could quickly poke at https://code.launchpad.net/~dobey/ubuntu-sso-client/update-linguas/+merge/124950 that would be awesome. thanks :) | 15:04 |
alecu | gatox: it does :P | 15:04 |
mmcc | dobey: I'll get that one | 15:04 |
thisfred | dobey, +1 | 15:05 |
thisfred | it looked familiar | 15:05 |
* thisfred checks turnitin.com | 15:05 | |
mmcc | dobey: approved | 15:08 |
dobey | thanks | 15:08 |
mmcc | So, anyone with a mac want to take a look at this branch: https://code.launchpad.net/~mikemc/ubuntuone-control-panel/launchdaemon/+merge/124847 | 15:21 |
dobey | lunch time, bbiab | 15:30 |
mmcc | huh. what's going on with UI string formatting in ubuntu_sso? | 15:42 |
mmcc | I saw a link that said 'Register with {app_name}' in the UI just now | 15:43 |
mmcc | the tests pass, and they use a mix of "".format() and "" % dict… I can't tell why they're used where they are | 15:44 |
mmcc | specifically test_current_user_sign_in_page.py, line 84 vs 86 | 15:44 |
mmcc | here's a bug for it: https://bugs.launchpad.net/ubuntu-sso-client/+bug/1052555 -- ralsina, I assigned you so you can decide who to poke | 15:53 |
ubot5 | Ubuntu bug 1052555 in Ubuntu Single Sign On Client "UI shows string with format placeholder" [High,New] | 15:53 |
mandel | time for me to go, laters! | 16:13 |
gatox | mandel, bye | 16:13 |
chaselivingston | mmcc: how's that bug squashing going? | 16:14 |
mmcc | chaselivingston: not squashed yet, working on it now. I can reproduce it now, and I'm pretty sure it's the same issue we had with the other backend. | 16:16 |
chaselivingston | mmcc: cool, good luck | 16:16 |
gatox | alecu, here is the SyncMenu branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-syncc/+merge/124968 who else should i ask for a review? anyone in particular? | 16:30 |
gatox | ralsina, ^ | 16:33 |
alecu | gatox: whoever is available, yes :-) | 16:37 |
gatox | ok | 16:37 |
gatox | dobey, ralsina one of you can take a look at that branch ^ | 16:37 |
gatox | please :P | 16:38 |
gatox | and now...... lunch for me! | 16:38 |
=== gatox is now known as gatox_lunch | ||
dobey | hmm | 16:55 |
briancurtin | lunch + pharmacy run = brb | 17:02 |
ralsina | FINALLY! Hello again. | 17:06 |
ralsina | gatox_lunch: looking... | 17:06 |
ralsina | gatox_lunch: when you are fed and nurtured, please shoot a couple of chosen screenshots of sync menu my way | 17:13 |
ralsina | gatox_lunch: work-environment-appropiate backgrounds only please ;-) | 17:13 |
gatox_lunch | ralsina, jeje ok, i'll try to avoid spiderman wallpapers jeje | 17:15 |
ralsina | gatox_lunch: I was thinking about the yoda one :-) | 17:15 |
gatox_lunch | ralsina, ahhh right..... don't worry, this is in the q vm.... i have the default in ubuntu jeje | 17:16 |
ralsina | gatox_lunch: the instructions in your branch build indicator-sync from source, I thought you said it was shipped already? | 17:18 |
alecu | ralsina: that's to run the menu inside the indicator testing tool | 17:19 |
gatox_lunch | ralsina, yes, is shipped.... also i need to add some instructions to install gir for syncmenu..... but that's the way to execcute the menu for testing | 17:19 |
alecu | ralsina: the sync indicator may be shipped already, but I don't know if it's started on boot, or how to start it, so the testing tool was needed. | 17:20 |
alecu | gatox_lunch: that's something we should ask charles: if it's installed by defautl and if not, how to start it. | 17:20 |
* alecu goes get some lunch. | 17:20 | |
gatox_lunch | ralsina, i just added this comment: | 17:21 |
gatox_lunch | It's necessary also to install: | 17:21 |
gatox_lunch | sudo apt-get install gir1.2-syncmenu-0.1 | 17:21 |
gatox_lunch | alecu, at least in the q-beta1 vm..... is not there by default | 17:21 |
dobey | hmm | 17:21 |
ralsina | alecu, gatox_lunch: ok! | 17:22 |
ralsina | mmcc: saw your mail | 17:35 |
ralsina | mmcc: basically, we *could* rip the tray stuff from u1cp and put it on a separate script which, when "activated" launches u1cp instead of showing the window | 17:35 |
dobey | ? | 17:36 |
ralsina | mmcc: that would also work on windows and ubuntu (when no sync menu is available) | 17:36 |
=== gatox_lunch is now known as gatox | ||
ralsina | mmcc: however, it's quite a bit of work at this stage ... | 17:36 |
mmcc | dobey: you interested? I left you off because it was mac stuff, sorry… | 17:36 |
dobey | is it not possible to only show a dock icon when a window is shown, rather than when the process is running? | 17:37 |
ralsina | dobey: nope | 17:38 |
ralsina | dobey: you need a process for it | 17:38 |
ralsina | dobey: sorry got you backwards there | 17:38 |
mmcc | right. an app is either background or not, and you can't dynamically change it | 17:38 |
ralsina | dobey: no, on mac, the icons are per-app not per-window | 17:38 |
mmcc | dobey: just sent you the email | 17:38 |
mmcc | where background = shows a dock icon and menu bar | 17:39 |
ralsina | I'm araid all this menu/tray/icon/taskbar thing is very platform-specific | 17:39 |
mmcc | duh, I meant doesn't show | 17:39 |
ralsina | afraid | 17:39 |
dobey | and we can't just put the menu in ubuntuone-syncdaemon, because qt? | 17:40 |
ralsina | dobey: exactly | 17:40 |
dobey | but it's using qt on windows/mac anyway, so that's not really an issue, right? | 17:40 |
ralsina | sd is not using qt on windows nor mac | 17:41 |
* briancurtin back | 17:41 | |
ralsina | at least it's not starting a qt main loop AFAIK | 17:41 |
dobey | it's using the default twisted reactor? | 17:42 |
ralsina | dobey: think so | 17:42 |
dobey | bummer | 17:43 |
ralsina | dobey: the only qt bits are in the proxy process | 17:43 |
ralsina | dobey: well, that was intentional :-) | 17:43 |
ralsina | so, it's mostly taking one class from u1cp and stitching it to a boilerplate main() | 17:43 |
ralsina | but then packaging for windows + mac, plus installer, plus autostart, plus crap I can't recall but surely is there :-( | 17:44 |
dobey | yes, but then we have yet another .exe/.app/whatever, and annoying. | 17:44 |
dobey | plus that menu really would be better off in sd :( | 17:44 |
mmcc | well, if the binary is only for mac, this isn't so bad… packaging one more app is quick, and I haven't done autostart for anything yet | 17:45 |
ralsina | mmcc: we can do it for mac experimentally | 17:46 |
dobey | mmcc: well if that's the case, then we have the same thing running in 3 different ways; which also sucks :) | 17:46 |
mmcc | dobey: yes, agreed | 17:46 |
ralsina | dobey: we can unify mac+windows at least | 17:46 |
ralsina | dobey: and ubuntu is different because of the sync menu which has no equivalente anywhere | 17:47 |
dobey | what about having it so 2 copies of u1cp can run on windows? | 17:47 |
dobey | err, on win/mac | 17:47 |
mmcc | ok, I will put this on my list after: 1. fix sync bug in daemon 2. fix default monitor name 3. fix startup wizard hang 4. fix hang on quit… | 17:48 |
dobey | ralsina: well, if the win/mac menu were in sd, all 3 could be unified in how they handle events from inside sd/etc… even if there are 2 slightly different ways of displaying the menu itself | 17:48 |
ralsina | dobey: yes | 17:48 |
ralsina | dobey: but are you interested on getting qtreactor into sd? | 17:48 |
ralsina | dobey: the potential for subtle bugs is scary as hell | 17:48 |
mmcc | dobey: that's basically what we're doing, right? one of those copies needs to not do all the web api calls, and on mac at least it really does need to be a separate .app so we can actually launch two copes | 17:48 |
dobey | you can't launch the same .app twice on mac? | 17:49 |
gatox | ralsina, screenshots added here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ubuntuone-client-syncc/+merge/124968 | 17:49 |
ralsina | dobey: the menu itself is generated in sd, the other things are small wrappers that call that and display the UI | 17:49 |
ralsina | gatox: awe----some | 17:49 |
mmcc | dobey: honestly I think maybe you can, but we do need them to be different .apps - if they're the same .app wrapper then they'll have the same background setting, and we're back to square one | 17:50 |
mmcc | actually, maybe you can't. I think it checks the bundle id | 17:51 |
mmcc | you definitely can't double-click to open it twice. this really never came up :) | 17:51 |
mmcc | but the .app wrapper is trivial, it doesn't imply any duplicated code or (much) duplicated resources | 17:52 |
dobey | well cp has internal uniqueness stuff | 17:53 |
dobey | it was a more general question | 17:53 |
=== m_conley is now known as m_conley_away | ||
dobey | but true about the .app vs icon status issue i guess :( | 17:53 |
mmcc | yeah, I forgot about the CP internal stuff… I was answering the general question (ok, not answering) - it really hasn't come up before… | 17:53 |
ralsina | mmcc: OTOH if these are the kind of things which are our major problems now? We should be happy campers. | 17:54 |
mmcc | ralsina: see my list above, my tent is leaking | 17:55 |
ralsina | mmcc: your tent is 99% ok ;-) | 17:56 |
=== m_conley_away is now known as m_conley | ||
* mmcc will refrain from torturing a tent metaphor | 17:56 | |
ralsina | mmcc: I would not mind having these minor interaction issues on the beta | 17:58 |
ralsina | mmcc: or even on a .0 release | 17:58 |
mmcc | ralsina: well, I'll be a happier camper when the daemon syncs a dragged-in file correctly | 17:59 |
ralsina | mmcc: yes, that one I would not ship with | 18:01 |
dobey | thisfred or mmcc: simple update branch for stable: https://code.launchpad.net/~dobey/ubuntuone-control-panel/update-4-0/+merge/125007 | 18:12 |
ralsina | gatox: I gave you a +1 on code review only because Q is in one of those days. | 18:13 |
thisfred | dobey, on it | 18:13 |
gatox | ralsina, cool..... and yes, i've been there :P | 18:14 |
thisfred | dobey, +1 | 18:14 |
dobey | thanks | 18:15 |
ralsina | dobey: did you get any response from marcus? | 18:23 |
dobey | ralsina: no | 18:26 |
ralsina | dobey: grmbl | 18:26 |
dobey | brb | 18:35 |
rye | gatox: files with longer names will transfer longer :) | 18:39 |
gatox | rye, sorry? | 18:40 |
ralsina | gatox: your screenshot | 18:40 |
ralsina | gatox, rye: that's actually a sync indicator bug | 18:40 |
ralsina | but it looks *bad* | 18:40 |
gatox | mmmmmm yes..... maybe we can implement some kind of ellide text to make everything of the same size.... but maybe that should be in the SyncMenu itself | 18:41 |
ralsina | gatox: just add spaces to the name before displaying? ;-) | 18:42 |
ralsina | gatox: yes, the scrollbars have no reason to be the same length as the text, that's a syncmenu thing | 18:42 |
gatox | ralsina, it was a joke or for real the "add spaces"? | 18:43 |
ralsina | they should be the full width of the menu | 18:43 |
ralsina | gatox: joke | 18:44 |
ralsina | gatox: unless we were using a monospaced font, in which case for real. | 18:44 |
gatox | ralsina, ahhhhh yes.... i thought it was a joke, because we are not going to have a monospace font there | 18:45 |
gatox | also.... the size of the menu is different depending of the biggest item being displayed..... that's why i suggest the elide thing..... but i think that is the responsability of how shows that.... what do you think? | 18:46 |
gatox | ralsina, ^ | 18:46 |
gatox | s/of how shows/of who shows | 18:46 |
ralsina | gatox: agreed | 18:47 |
gatox | ralsina, do you want me to contact charles to suggest that? | 18:48 |
ralsina | gatox: yes please | 18:49 |
gatox | ralsina, done https://bugs.launchpad.net/indicator-sync/+bug/1052620 | 18:54 |
ubot5 | Ubuntu bug 1052620 in The Sync Menu "The items in the transfers menu can have multiples sizes" [Undecided,New] | 18:54 |
ralsina | gatox: cool | 18:56 |
ralsina | gatox, briancurtin, alecu, mmcc, dobey, thisfred (and mandel in absentia): just in case I don't say this often enough, you guys kick ass. | 18:56 |
gatox | ralsina, :'D | 18:56 |
thisfred | :D | 18:56 |
briancurtin | its a good team to be kicking ass with | 18:56 |
mmcc | snif | 18:57 |
thisfred | all you have to do is tell us whose ass ;) | 18:57 |
ralsina | thisfred: at least you are big :-) | 18:57 |
ralsina | I said that because sometimes it seems all I do all day is throw broken stuff at you all. | 18:58 |
thisfred | yeah, making us do work. What are you thinking!? ;) | 18:59 |
briancurtin | dealing with broken stuff, windows, and broken stuff on windows...it's "what i do" | 19:00 |
thisfred | dept. of broken windows | 19:01 |
ralsina | hehe | 19:01 |
alecu | ralsina: we've already stablished that I can't fit in kick ass suit. I'm thinking of "big daddy" for halloween. | 19:01 |
gatox | ralsina, really trivial one: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/wrong-string-formatting/+merge/125023 we were using the wrong formatting option for that string formatting notation..... and the test wasn't efective, because was doing the same..... so the string in both cases was equally wrong | 19:01 |
ralsina | alecu: too bad Shane is not with us anymore, he would be a great hitgirl :-) | 19:01 |
ralsina | gatox: got it | 19:01 |
alecu | ralsina: I thought gatox claimed that costume! | 19:02 |
gatox | alecu, hey!!! you assign that to me withoout asking | 19:02 |
gatox | jeje | 19:02 |
ralsina | dibs on the flying machine-gun thingie! | 19:02 |
* thisfred will go as The Human Stain. Or is that not a superhero. | 19:03 | |
alecu | ok, it's time for this superhero to put away his costume and take Amelia to swimming lessons. | 19:03 |
thisfred | shark girl! | 19:03 |
alecu | I will be back later, to do gatox' review | 19:03 |
gatox | alecu, good good :P | 19:04 |
ralsina | thisfred: you remind me, I need to know how sharkboy became a werewolf in his teens | 19:04 |
thisfred | ralsina, wasn't that teen wolf? ;) | 19:04 |
ralsina | thisfred: let me get you pictures | 19:05 |
thisfred | no need! | 19:05 |
=== eu is now known as adorilson | ||
ralsina | thisfred: hahaha | 19:05 |
thisfred | also, can we ban movie sequels with 'too' instead of 2/two in the title? | 19:05 |
ralsina | thisfred: I prefer the ones that just pluralize the 1st | 19:06 |
thisfred | alienses | 19:06 |
ralsina | For example, having the sequel to "Rocky" be called "Rockies" would have been awesome | 19:06 |
thisfred | die hards | 19:06 |
ralsina | Titanic -> Titanics | 19:06 |
ralsina | thisfred: 'Die harder' was a great title | 19:07 |
ralsina | Also "Live free or Die Hard" | 19:07 |
thisfred | Last one I saw was 3 I think | 19:08 |
thisfred | maybe that was the last one | 19:08 |
thisfred | Bruce Willis is great in the new Wes Anderson, btw | 19:09 |
thisfred | Moonrise Kingdom | 19:09 |
thisfred | highly recommended | 19:10 |
thisfred | if you like his movies at all | 19:10 |
ralsina | thisfred: no, 4 is *nice*. | 19:11 |
ralsina | Bruce Willis is severely underrated as an actor | 19:11 |
dobey | alecu: haha; you in the big daddy outfit walking around holding amelia's hand would be awesome! | 19:11 |
dobey | Bruce Willis is great in that lawsuit against Apple | 19:12 |
thisfred | ralsina, well, he's not very discriminating in the roles he accepts, so it's not that surprising. But yeah, I have a fondness for him. I even like Hudson Hawke, which is considered one of the worst movies of all time :) | 19:12 |
ralsina | Hudson Hawk???? I still time my heists by singing! | 19:13 |
dobey | Hudson Hawk was awesome | 19:13 |
thisfred | and he directed that | 19:13 |
thisfred | and co-wrote it, I think | 19:13 |
dobey | also, the CSI Miami guy in that was pretty funny | 19:13 |
mmcc | gatox, ralsina - there are other strings with the same formatting style in ui.py -- also can you mark it as fixing the bug I filed for that issue, too? bug #1052555 | 19:13 |
ubot5 | Launchpad bug 1052555 in Ubuntu Single Sign On Client "UI shows string with format placeholder" [High,New] https://launchpad.net/bugs/1052555 | 19:13 |
ralsina | dobey: who, the eczema guy? | 19:14 |
dobey | ralsina: the ginger who always wears sunglasses? | 19:14 |
gatox | mmcc, checking the other strings | 19:14 |
ralsina | dobey: well, he seems to always be TAKING OFF the sunglasses but I never saw a whole CSI Miami | 19:14 |
dobey | heh | 19:15 |
gatox | mmcc, the problem is not that they said: {app_name}... but how are they used later..... | 19:15 |
dobey | ralsina: he was 'Snickers' in Hudson Hawk | 19:15 |
mmcc | gatox: ok, just wanted to make sure we looked at all of them, since there was a mix of formatting styles | 19:15 |
gatox | mmcc, yes..... actually i personally believe we should avoid the mixing | 19:16 |
gatox | mmcc, linked | 19:17 |
mmcc | gatox, thanks. | 19:17 |
ralsina | gatox: +1 | 19:18 |
gatox | mmcc, could you do the other review? | 19:18 |
mmcc | gatox: sure | 19:19 |
gatox | mmcc, it's really trivial | 19:19 |
mmcc | gatox: ok, I checked all the other strings, looks like that was the only wrong one. approved :) | 19:28 |
gatox | mmcc, cool, thx! | 19:28 |
dobey | tests? :P | 19:28 |
mmcc | dobey: hmm. | 19:31 |
thisfred | wow, better get cracking on that ark, it's coming down | 19:51 |
ralsina | thisfred: here too. That must be heck of a storm! | 19:52 |
dobey | it's the skrillex storm | 19:53 |
thisfred | hehe | 19:53 |
chaselivingston | ralsina, thisfred: coming down like cats and dogs here as well, quite a ways away | 19:53 |
* ralsina is ignorant of that skrillex you mention. Should he fix that? | 19:53 | |
ralsina | if mmcc says it's raining there too, I *am* thinking of that ark | 19:54 |
thisfred | global weather! That'll save a lot on meteorology expenses | 19:54 |
dobey | ralsina: http://what-if.xkcd.com/12/ | 19:54 |
mmcc | dry here. | 19:56 |
mmcc | we also have a bay that can accommodate very large arks indeed… | 19:56 |
* mmcc is curious about ralsina's reaction to dubstep | 19:57 | |
dobey | haha | 19:58 |
ralsina | mmcc: having recently had my thousandth birthday, I suspect not good. | 19:58 |
ralsina | thisfred: also, haha | 19:58 |
thisfred | ralsina, your only possible answer is: TURN THAT RACKET DOWN! | 19:58 |
* ralsina gets an onion for his belt | 19:58 | |
mmcc | ralsina: just keep an open mind, and don't try to think of it as "music" | 19:58 |
thisfred | hehe | 19:59 |
mmcc | btw ralsina, belated happy birthday, sorry I missed that | 19:59 |
ralsina | mmcc: thanks :-) | 19:59 |
* thisfred suspects he probably owns the most dubstep of anyone on the team. | 19:59 | |
ralsina | thisfred: and you have it nicely tagged in order of emetic power | 19:59 |
mmcc | more than zero dubstep? thisfred might be right | 19:59 |
thisfred | yep | 19:59 |
thisfred | I could start arguing that there's dubstep and dubstep but... | 20:00 |
ralsina | from "like sea water on full stomach" to "kicked i the gut by a robot mule" | 20:00 |
thisfred | http://www.blameitonthevoices.com/2011/10/dubstep-intervention.html | 20:00 |
mmcc | you know some of those bass drops get low enough that they're almost subsonic vomit weapons | 20:00 |
thisfred | brown note! | 20:01 |
thisfred | fwiw I detest Skrillex | 20:01 |
ralsina | "I call this note the t-rex-in-jurassic-park-b-flat" | 20:01 |
mmcc | thisfred, I also didn't have a big positive reaction to him. I just heard this track from bassnectar last night and dug it: http://snd.sc/GYtfHW | 20:02 |
thisfred | if you want to check out dubstep that I think is not garbage, I'd recommend artists like Burial, 2526, Amon Tobin, Four Tet. One could also argue that they're not really dubstep, and that that's the reason they're not garbage ;) | 20:03 |
dobey | man alt+tab in unity is really not usable with multiple terminal windows :-/ | 20:03 |
mmcc | the sound cloud visualizations of these tracks are pretty funny | 20:03 |
thisfred | Four Tet is definitely an edge case | 20:03 |
* thisfred listens to bassnectar | 20:03 | |
mmcc | ugh, "do not read the comments" extends to soundcloud track comments. what is wrong with people | 20:04 |
thisfred | dobey, I keep saying this | 20:04 |
thisfred | and filing bugs | 20:05 |
thisfred | that keep expiring | 20:05 |
dobey | heh | 20:05 |
ralsina | well, musical taste is usually fixed by what you listened as a teen. Which means I am closer to liking doo wop than dubstep | 20:05 |
thisfred | ralsina, I'd have thought madrigals | 20:05 |
ralsina | thisfred: that too, but the pun doesn't work | 20:05 |
thisfred | oh wasn't a pun, I was just trying to imply you're from the middle ages :P | 20:06 |
dobey | heh | 20:06 |
dobey | ralsina: we can add some bass drops to The Chordettes, then maybe you'll enjoy it more | 20:06 |
thisfred | I'm glad I kept an open mind. I almost exclusively listened to hip-hop in my teens, and that would leave *very* little of interest today | 20:06 |
* ralsina avoids jokes about going medieval on thisfred and all that (hey, Bruce Willis again!) | 20:06 | |
thisfred | hehe | 20:07 |
dobey | bruce willis has a pretty ok blues band | 20:07 |
thisfred | blues willis? | 20:07 |
* ralsina imagines gangsta thisfred | 20:07 | |
* ralsina rewinds his brain | 20:07 | |
thisfred | I never liked gangsta, that's where hip-hop and I went our separate ways | 20:08 |
thisfred | unless you consider Wu-Tang gangsta. Which it is, I guess | 20:08 |
ralsina | thisfred: you more the mercedes benz sign necklace style? | 20:08 |
thisfred | but if you add kung fu samples to anything it becomes good. scientific fact | 20:08 |
briancurtin | +1, approved | 20:08 |
thisfred | ralsina, Beastie Boys + Run-D.M.C. was my first concert | 20:09 |
ralsina | hey, doo wop (which I had never heard and just googled with "music styles from the 50s") is actually cool! | 20:10 |
ralsina | learn something every day! | 20:10 |
thisfred | which I went to with my Mom, just so you guys don't start thinking I was ever cool | 20:10 |
dobey | there is some pretty good bubble gum rock | 20:10 |
ralsina | thisfred: going to beastie boys with your mum is one of those things that need to be in your CV | 20:10 |
chaselivingston | mmcc: can't seem to sync my purchased music folder w/ the mac app, any idea why? | 20:11 |
ralsina | chaselivingston: you may need to subscribe it? | 20:11 |
chaselivingston | ralsina: not quite sure what you mean... | 20:11 |
dobey | thisfred: apparently your mom was pretty cool | 20:12 |
mmcc | chaselivingston: not off the top of my head, can you file a bug with notes on what you're doing? I haven't tried that yet… | 20:12 |
ralsina | chaselivingston: in the control panel, you should see an item for your purchased music folder | 20:12 |
ralsina | chaselivingston: you have to tick it so it subscribes and downloads | 20:12 |
chaselivingston | ralsina: right, but when i click "sync locally" it puts the folder underneath a hidden folder, .ubuntuone in my home directory and doesn't download anything | 20:13 |
thisfred | dobey, she sat in the rafters while I was up front, but she didn't complain. This was the tour where they had a 20ft inflatable penis coming out of the stage halfway through the show. I don't think that was ever mentioned. | 20:13 |
mmcc | neat! | 20:13 |
thisfred | She might have been reading a book. | 20:13 |
dobey | thisfred: i was just taking the opportunity to make the obvious "your mom" joke | 20:13 |
thisfred | hehe | 20:14 |
ralsina | chaselivingston: nothing comes to mind then | 20:14 |
chaselivingston | cool, i'll file a bug | 20:14 |
chaselivingston | ralsina: should I tag the bug u1-darwin? | 20:17 |
chaselivingston | mmcc: ^^ | 20:17 |
mmcc | call trail for that bug: MUSIC_REAL_PATH is defined in gui/__init__.py as u'.ubuntuone/Purchased from Ubuntu One' -- used in folders.py's _process_name which is called in add_folder… | 20:17 |
mmcc | chaselivingston: sure | 20:17 |
chaselivingston | mmcc, ralsina: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/1052649 | 20:18 |
ubot5 | Ubuntu bug 1052649 in Ubuntu One Control Panel "Purchased music folder doesn't sync correctly" [Undecided,New] | 20:18 |
mmcc | thanks chaselivingston ! | 20:18 |
chaselivingston | mmcc: np, hope to see a fix soon :) | 20:18 |
mmcc | chaselivingston: if you can attach your controlpanel.log to that, it'd be helpful | 20:19 |
chaselivingston | mmcc: sure | 20:19 |
gatox | eod here!!! see you tomorrow people!! byeeee | 20:20 |
mmcc | there goes the last person on the team young enough to grok dubstep | 20:21 |
chaselivingston | mmcc: attached that file | 20:22 |
mmcc | ack, thanks chaselivingston | 20:22 |
chaselivingston | np | 20:22 |
dobey | mmcc, thisfred: https://code.launchpad.net/~dobey/ubuntu-sso-client/update-4-0/+merge/125042 another update branch, please. :) | 20:25 |
ralsina | dobey: I got it | 20:26 |
thisfred | on it | 20:27 |
thisfred | dobey, +1 | 20:28 |
thisfred | I'm a fast reader, and I know all languages, what?! | 20:28 |
chaselivingston | mmcc: hm, well now it appears to be working, but still in that odd location | 20:29 |
mmcc | dobey, that diff doesn't look like it contains all the changes in the commit message | 20:29 |
ralsina | chaselivingston: there is a symlink inside ~/Ubuntu One | 20:30 |
dobey | mmcc: how so? | 20:30 |
ralsina | chaselivingston: finder has this stupid idea about dereferencing symlinks though :( | 20:30 |
chaselivingston | ralsina: hmmm | 20:30 |
* briancurtin brb, need to get some fresh air | 20:30 | |
dobey | mmcc: you'll have to look at an actual copy, lp truncates very long diffs | 20:30 |
mmcc | dobey: I can't find manuel's change about the twisted runner, and I don't think diego's change is in there either | 20:30 |
ralsina | mmcc: truncated | 20:31 |
dobey | mmcc: they're under ubuntu_sso/ which comes after po/ | 20:31 |
mmcc | I was checking for diego's change because the branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/wrong-string-formatting/+merge/125023 | 20:32 |
mmcc | … changed a string "register with" - that all the translations have | 20:32 |
mmcc | and the translations have the wrong format specifier now | 20:32 |
mmcc | it might be worth backing out diego's fix and just fixing the calling code instead… | 20:32 |
dobey | oh crap, right, the translations | 20:32 |
dobey | totally forgot about those | 20:32 |
ralsina | faaaaaaark | 20:32 |
dobey | but easy fix either way | 20:33 |
ralsina | so did I | 20:33 |
ralsina | dobey: let's ask everyone to retranslate that ;-) | 20:33 |
mmcc | ok, I missed the truncation thing, whoops | 20:33 |
dobey | ralsina: perl -pi -e 's/old/new/g' *.po :) | 20:33 |
dobey | though it might still cause an issue, for the langpacks | 20:34 |
dobey | so we should probably back out the change and fix the code that uses that instead | 20:34 |
mmcc | it's only called in one place, iirc, | 20:34 |
ralsina | yes, better to keep the old string and change the other side | 20:34 |
ralsina | yep | 20:34 |
ralsina | volunteers? I am leaving in about 16 minutes | 20:34 |
=== Guest61577 is now known as slank | ||
mmcc | I'll do it | 20:35 |
=== slank is now known as Guest70664 | ||
dobey | meh the battery life on the dell duo is so awful :-/ | 20:36 |
* ralsina puts the volunteering cattle prod back in the case. | 20:36 | |
ralsina | you may wonder how that works. I ask "who volunteers?" then prod someone. He will say "I". | 20:38 |
ralsina | works better on spanish-peaking volunteers. English speakers have a bad tendency to say "ouch". In that case, repeat. | 20:38 |
dobey | will have to do that release in the morning i guess | 20:44 |
dobey | along with u1-client | 20:44 |
dobey | and u1-client i am a bit worried abut | 20:44 |
dobey | about | 20:44 |
* briancurtin back | 20:45 | |
mmcc | for your consideration: https://code.launchpad.net/~mikemc/ubuntu-sso-client/re-re-format-register-link/+merge/125045 | 20:46 |
ralsina | dobey: anything I can do besides promising to be around early for reviews? | 20:47 |
mmcc | I also added tests that the format string contains the format argument, which would've caught the original bug | 20:47 |
ralsina | mmcc: cool | 20:47 |
dobey | mmcc: can you commit --fixes the same bug # that diego's branch did please? | 20:47 |
mmcc | dobey: sure | 20:48 |
* thisfred reviews | 20:49 | |
mmcc | done | 20:50 |
mmcc | btw, I only tested that on darwin | 20:51 |
ralsina | EOD for me. If my ISP has mercy on my soul I may be back later, but don't bet on it. | 20:52 |
thisfred | +1 | 20:53 |
briancurtin | +1 as well | 20:54 |
mmcc | hooray, my highest rated branch yet | 20:55 |
mmcc | ok, on that high note I'm going for a much needed late lunch | 20:55 |
dobey | have a good evening all. battery about to die on this laptop. might be around later to try and get a couple things done to prep for releaseing in the morning | 21:16 |
=== m_conley is now known as m_conley_away | ||
chaselivingston | mmcc: any chance of a new build today? :) | 23:05 |
mmcc | chase, yes. I just fixed the sync bug, I am testing it now, will put up a build shortly | 23:06 |
chaselivingston | mmcc: awesome, putting it in that shared folder? | 23:06 |
mmcc | chaselivingston: yes | 23:07 |
mmcc | I'll ping you here when it's up | 23:07 |
chaselivingston | mmcc: sweet, thanks man. you rock | 23:08 |
mmcc | heh. thanks. I do get paid to do this, you know ;) | 23:14 |
mmcc | uploading now | 23:15 |
chaselivingston | mmcc: yeah, so do i :) doesn't mean I don't enjoy it | 23:15 |
mmcc | that's the spirit :) | 23:16 |
chaselivingston | mmcc: i'm getting a message that it's damaged and i should move it to the trash... | 23:23 |
mmcc | huh. ok, let me poke at it a bit | 23:24 |
chaselivingston | mmcc: i may be afk for a little while but i'll leave irc up so ping me w/ any news | 23:26 |
mmcc | chaselivingston: not sure what's going on there, I'm curious what size it is | 23:26 |
chaselivingston | mmcc: the zip was 32.3MB | 23:26 |
mmcc | chaselivingston and when you get back if you could try downloading it from http://ubuntuone.com/0FNiarEDBStjRVpiUhW5xO instead? | 23:27 |
chaselivingston | mmcc: trying now, still have a few mins | 23:28 |
chaselivingston | mmcc: still getting the error, the .app is 92MB and some change | 23:28 |
mmcc | hrm. ok, I'll see what I can figure out | 23:29 |
chaselivingston | mmcc: cool thanks | 23:29 |
mmcc | OK, here's the merge proposal for my daemon sync fix: https://code.launchpad.net/~mikemc/ubuntuone-client/fix-daemon-rename-event/+merge/125074 | 23:50 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!