| mmcc | yes. although i didn't see the same issue with the ctime (I misspoke, it wasn't atime) that I saw with mtime, moving backwards | 00:02 |
|---|---|---|
| mandel | mmcc, fuuuu the test_multiple_files_added is wrong.. with you chages we should be getting 4 but we do not and the test passes.. | 00:04 |
| mmcc | ugh | 00:05 |
| mandel | mmcc, but has nothing to do with the new changes, the logs do not show any event being added (I added logs at least for that part) | 00:06 |
| mandel | mmcc, so it probably does patches something or god knows what.. | 00:06 |
| mandel | mmcc, is already 2 am and this code is depressing.. I'm off to sleep and face this with more energies | 00:07 |
| mmcc | ok mandel, sleep well - remember what ralsina said - don't pull 18 hour days :) | 00:11 |
| mandel | mmcc, it was not 18 hours, maybe 14 I had that rugby break :) | 00:12 |
| mmcc | heh, right - talk to you tomorrow | 00:13 |
| mandel | mmcc, laters! | 00:13 |
| mmcc | ok, i have to go too for a bit now. back for more tonight | 00:15 |
| dobey | hmm | 00:44 |
| === mthaddon` is now known as mthaddon | ||
| JamesTait | Good morning all! :) | 08:18 |
| rye | mandel: hi, what does this mean - Deleting metadata, because of node_id=None, of 'C:\\Users...' ? | 10:18 |
| rye | mandel: i.e. how node_id can be None? alecu ^ too | 10:20 |
| rye | anybody has a windows 7 vm with non-metered network connection i can use? | 10:36 |
| === yofel_ is now known as yofel | ||
| gatox | good morning | 11:21 |
| === eu is now known as Guest76304 | ||
| gatox | need to restart.... brb | 12:21 |
| ralsina | morning! | 12:48 |
| gatox | ralsina, hi | 12:52 |
| ralsina | hi gatox, anything needing reviews? | 13:00 |
| gatox | ralsina, not yet | 13:00 |
| ralsina | gatox: ok | 13:00 |
| alecu | hello, all! | 13:01 |
| ralsina | mandel: +1 on sbjson-link | 13:02 |
| ralsina | hello alecu! | 13:02 |
| gatox | alecu, hi | 13:06 |
| mandel | ralsina, awesome thx! | 13:23 |
| mandel | oh, and morning/afternoon! | 13:23 |
| mandel | I was really tired to start early today | 13:23 |
| ralsina | mandel: youa re not supposed to! | 13:26 |
| * ralsina starts drawing a diagram where mandel's work hours (his time, so to speak) is *shifted* | 13:26 | |
| mandel | hehe | 13:26 |
| ralsina | mandel: so, how's the sprint going? | 13:26 |
| ralsina | mandel: will you guys have something to show QA today? tomorrow? | 13:27 |
| ralsina | mandel: monday? ;-) | 13:27 |
| mandel | ralsina, good, we are focusing in two diff things we are kinda worry about, we have the blessing, which is hard and mmcc is worried about (for the root daemon) and the bugs we have found in fsevents which I'm fixing. The idea is to get the fsevents ready before because it looks more plausible and have the daemon as a second target | 13:28 |
| ralsina | mandel: ok | 13:28 |
| mandel | ralsina, we should have a working version with fsevents today | 13:28 |
| ralsina | mandel: awesome | 13:28 |
| ralsina | mandel: I will try to schedule QA for tomorrow | 13:28 |
| ralsina | mandel: it doesn't matter if it still needs some manual tweaking, for example | 13:29 |
| mandel | ralsina, at least uploading and downloading and be ready to be tested.. maybe we have problems with the signing though and they will have to execute it by right click + open | 13:29 |
| ralsina | mandel: that's ok for this week | 13:29 |
| mandel | ralsina, we also have a very very annoying Qt bug regarding the scrolling in the control panel | 13:29 |
| mandel | I hope to have time to fix that today | 13:29 |
| ralsina | mandel: prioritize, don't jump around | 13:30 |
| mandel | ralsina, no no, fsevents goes first | 13:30 |
| ralsina | mandel: ok, we may even get gatox to check on that scrolling bug | 13:30 |
| ralsina | gatox: what day are you leaving or NZ? | 13:30 |
| mandel | ralsina, that would be very nice | 13:30 |
| gatox | ralsina, monday is my last day here | 13:30 |
| ralsina | gatox: and you come back in 2 weeks? | 13:31 |
| gatox | ralsina, yep | 13:31 |
| ralsina | gatox: ok, how are you doing with the new tab? | 13:31 |
| gatox | ralsina, i think i can finish it (tests+code) by tomorrow | 13:32 |
| ralsina | gatox: awesome, that would leave you a little time maybe friday or monday to check on that mac bug mandel mentiones | 13:32 |
| ralsina | mentioned* | 13:32 |
| ralsina | gatox: and when you come back, you can do the sync menu integration :-) | 13:33 |
| gatox | ralsina, yes, i already see that bug, mandel show it to me once | 13:33 |
| gatox | ralsina, cool | 13:33 |
| ralsina | gatox: or alecu, or me, depending on urgency | 13:33 |
| ralsina | so we are good there | 13:33 |
| ralsina | this cycle is looking good :-) | 13:33 |
| gatox | \o/ | 13:34 |
| alecu | :-) | 13:34 |
| gatox | ralsina, that happens when the team is awesome :P jeje | 13:35 |
| alecu | gatox: btw: I've take a look at the current syncmenu branches, and they are looking good. | 13:35 |
| alecu | *taken | 13:35 |
| alecu | gatox: they already have the progressbar running | 13:35 |
| * ralsina is using u1cp --with-icon all the time now | 13:35 | |
| alecu | gatox: it is not yet packaged, and it does not yet have python bindings, but stil.. | 13:36 |
| alecu | and it is not yet landed... :P | 13:36 |
| gatox | :P | 13:37 |
| ralsina | alecu: come on, it's only a +15000 branch :-) | 13:38 |
| mandel | alecu, ralsina, you that are wiser, is there any reason to be importing os in every test case rather than on the top of the test module? | 14:00 |
| alecu | mandel: url? | 14:01 |
| ralsina | mandel: unless it's being patched, I am guessing no, but I would like to see the code | 14:02 |
| mandel | alecu, here, look at the tests: https://github.com/malthe/macfsevents/blob/master/tests.py | 14:02 |
| mandel | ralsina, ^ | 14:02 |
| ralsina | looking | 14:02 |
| mandel | ralsina, alecu, time and os are imported in nearly every test case, I wonder if I'm missing something | 14:02 |
| ralsina | mandel: naïve first impression, that just looks weird | 14:02 |
| ralsina | mandel: same about time | 14:03 |
| mandel | I have the same opinion | 14:03 |
| ralsina | mandel: it's like he's avoiding any top-level imports | 14:03 |
| ralsina | well, it's one way to make sure things are very self-contained | 14:03 |
| alecu | an ugly way, but yes. | 14:04 |
| mandel | ralsina, alecu, AFAIK the imports are cached so it does not make much sense.. | 14:05 |
| ralsina | mandel: I would say it's quirky style, but since this is an upstream, you should follow it instead of doing patches we don't really care about | 14:05 |
| rye | mandel: is u1 working on windows 8? I am interested in syncdaemon only | 14:05 |
| ralsina | rye: should be | 14:05 |
| rye | ralsina: okay, that's the only windows in a vm I have on this machine :( | 14:05 |
| mandel | ralsina, that is what I'm doing, I'm strongly fighting against the urge of changing everything | 14:05 |
| ralsina | mandel: stay strong! | 14:06 |
| ralsina | rye: it's not very tested, but it should just work | 14:06 |
| alecu | mandel: it's not as if the modules are unloaded when the function is exited... so... just weird style. | 14:06 |
| mandel | alecu, exactly, just what I though and is done everywhere.. | 14:08 |
| ralsina | alecu: I think it's a misread of instructions like this http://moztrap.readthedocs.org/en/latest/standards.html | 14:08 |
| ralsina | "Avoid importing the code under test at module level in the test file; instead, import it in helper methods that are called by the tests that use it. This ensures that even broken imports cause only the affected tests to fail, rather than the entire test module." | 14:08 |
| ralsina | of course, if "import os" fails, your python is already broken beyond recognizion | 14:09 |
| ralsina | cion* | 14:09 |
| ralsina | or maybe tion* | 14:09 |
| alecu | shion! | 14:10 |
| ralsina | so, it's *very* defensive programming :) | 14:12 |
| alecu | yup, that's how they program in the DoD! | 14:13 |
| mmcc | hiya | 14:14 |
| alecu | ralsina: so, that looks like a decent coding standard | 14:15 |
| ralsina | alecu: yes, now that he mentions, it, importing the testable code in teh tests (or maybe in setup and assign to a member?) is a good idea | 14:16 |
| ralsina | we could even delete the import of teardown or something | 14:16 |
| briancurtin | rye: if it doesnt work on win8 that would be interesting to know. as far as i know, it should just work. i setup a win8 machine the other day for a python build slave, but havent checked u1 yet | 14:19 |
| rye | briancurtin: unfortunatelly it's now too slow to be usable here (in virtual box on my laptop) so i won't test today, searching for different windows machines' | 14:19 |
| alecu | mmcc: hi there! | 14:21 |
| ralsina | hi mmcc! | 14:22 |
| mmcc | hi lag! ;) | 14:22 |
| alecu | mmcc: you mentioned last week that you wanted to test on a different 10.7 machine... and I said that we should ask ralsina to set up a vnc or similar on his macmini. | 14:22 |
| alecu | (since both mandel and I are on 10.8 now) | 14:23 |
| ralsina | mine is 10.7 still | 14:23 |
| ralsina | but I am not at home today :-/ | 14:23 |
| mmcc | alecu: oh right, about the u1cp test suite issue. | 14:23 |
| ralsina | maybe gatox's? | 14:23 |
| alecu | mmcc: right! I've forgotten what we needed it for :-) | 14:23 |
| gatox | anyone need my mac? | 14:24 |
| ralsina | gatox: is it still 10.7? | 14:24 |
| gatox | ralsina, let me check | 14:24 |
| mandel | gatox, that is a yes, unless you went to the devel center and downloaded the new version or paid for it | 14:25 |
| mandel | mmcc, I'm nearly done adding correct support for move_from_not_watched_path to watched_path to generate create and modify and will ensure that the opposite raises a delete | 14:25 |
| gatox | 10.7.4 | 14:26 |
| mmcc | mandel, remind me what you're changing to add that support? just fsevents.py, right? | 14:27 |
| mandel | mmcc, yes, just fsevents.py | 14:27 |
| mmcc | you shouldn't have had to change anything for the delete, right, just my print was wrong - the code detects it right... | 14:28 |
| mandel | mmcc, ok, great news then, but I'll add the test anyway | 14:28 |
| mmcc | mandel: I just tested the fix I was going to make yesterday and it works - do you want me to push it up or have you already done the same thing? I just made it issue a MODIFY when it sees a create with either a modified flag *or* a renamed flag… before, it was ignoring the renamed flag | 14:31 |
| mandel | mmcc, push the fix and I'll take a look, I have s diff approach and want to decide what is better | 14:34 |
| mmcc | mandel: ok. I'm curious what you did too - can you paste the file somewhere or something? | 14:34 |
| mandel | mmcc, sure, give me a couple of mins to finish some tests | 14:35 |
| mmcc | mandel: you bet | 14:36 |
| rye | ralsina: is Ubuntu One ok about running under Administrator windows account now? | 14:39 |
| ralsina | rye: yes | 14:39 |
| rye | ralsina: sweeet | 14:39 |
| mmcc | mandel: ok my changes are pushed now to lp:~mikemc/+junk/python-macfsevents/ | 14:40 |
| mandel | ack | 14:40 |
| rye | weird, 8bpp rdp connection is slower than 15bpp | 14:41 |
| alecu | rye: do you have a background image, or gradients on windows borders? | 14:44 |
| alecu | rye: the image dithering in those case may compress worst than 15bpp. | 14:45 |
| rye | erm | 14:50 |
| rye | alecu: well, you may be right, the patterns may have used more bandwidth | 14:51 |
| dobey | rye: can you test that upgrading python-ubuntuone-storageprotocol fixes that valicert issue? | 14:51 |
| rye | ralsina: found a weird bug - disconnect ubuntuone in windows, close control panel. Open control panel via tray - ubuntuone gets connected and synchronizes files | 14:51 |
| mandel | mmcc, you can find the changes here with the tests: https://github.com/mandel-macaque/macfsevents | 14:53 |
| mandel | mmcc, it has the changes you made plus gatox but making them as extra keywords so that it is easier that they are accepted by the upstream | 14:53 |
| mandel | mmcc, if you give me a few mins I'll change u1-client to use that api and we can test the app with it | 14:54 |
| rye | YES | 14:54 |
| rye | no | 14:54 |
| dobey | maybe so? | 14:55 |
| mmcc | mandel: sure. I see you're always adding the IN_MODIFY after the create - that'll give us an extra modify event in the case of copying something large in | 14:55 |
| mandel | mmcc, yes, and that is not a problem at all, the hash queue will notice that and will do the right thing which is what happens on linux | 14:56 |
| mandel | mmcc, I had a version where it minimized the number of modifies but I find this way safer | 14:56 |
| ralsina | rye: interesting | 14:58 |
| ralsina | rye: the tray *is* control panel | 14:58 |
| ralsina | rye: you probably have autconnect enabled, so when the u1cp opens, sd starts | 14:58 |
| mmcc | mandel: ok. If the rest of syncdaemon handles that ok, then that's fine. And since we're not using IPC here, I'm not as concerned about performance impact of extra events… | 14:59 |
| mmcc | not yet, anyway | 14:59 |
| mandel | mmcc, yes, I had the same idea, better safe than sorry, we can be smarter in a second iteration | 14:59 |
| gatox | me | 15:00 |
| rye | ralsina: well, yes, autoconnect is enabled but i am not quitting control panel and SD, both remain running but restoring the CP window causes syncdaemon to connect again | 15:00 |
| thisfred | me | 15:00 |
| dobey | me | 15:00 |
| mandel | me | 15:00 |
| ralsina | mandel, mmcc, dobey, thisfred, gatox, alecu, briancurtin: remember that today is a the new 2-minute standup. If you are not accounted for at 12:02 don't say me, just report | 15:00 |
| ralsina | me | 15:00 |
| mmcc | me | 15:01 |
| * alecu hurries to write notes | 15:01 | |
| briancurtin | me | 15:01 |
| gatox | DONE: | 15:01 |
| gatox | Working on Share links tab for control panel. | 15:01 |
| gatox | TODO: | 15:01 |
| gatox | Finish (probably tomorrow) with this branch for the share links tab and propose. | 15:01 |
| gatox | BLOCKED: | 15:01 |
| gatox | No | 15:01 |
| gatox | thisfred, go | 15:01 |
| thisfred | DONE: documentation6 | cdocs TODO: last documentation fixes + db explorer app? + ? BLOCKED: no NEXT: dobey | 15:01 |
| dobey | DONE: releases, discussed cert issue with security | 15:01 |
| dobey | TODO: test cert issue fix, finish releases, icon generating magic | 15:01 |
| dobey | BLCK: None. | 15:01 |
| dobey | mandel: go | 15:01 |
| briancurtin | wait what | 15:01 |
| mandel | DONE: Lots of bug fixing in fsevents lib. Merged chages from mmcc and gatox in the history of upstream. Changed u1-client to use the changes (latency && process_events_asap). | 15:01 |
| mandel | TODO: Test changes in bundle. Found a couple of bugs that would be nice to fix. | 15:01 |
| mandel | BLOCKED: no | 15:01 |
| mandel | ralsina, please | 15:01 |
| ralsina | DONE: mgmt call, other calls, 1-1s, canonicaladmin, ton of reviews, helped around TODO: music store call, reviews, etc. BLOCKED: no, NEXT: mmcc | 15:01 |
| mmcc | DONE: more SMJobBless, more bug hunt in fsevents.py - looking good | 15:02 |
| mmcc | TODO: more SMJobBless & code signing | 15:02 |
| mmcc | BLCK: no | 15:02 |
| mmcc | next: briancurtin | 15:02 |
| briancurtin | DONE: a few more devtools branches | 15:02 |
| briancurtin | TODO: devtools branch to make u1lint work | 15:02 |
| briancurtin | NEXT: | 15:02 |
| briancurtin | i guess alecu? | 15:02 |
| alecu | DONE: playing with vala and oauth, some reviews for brian | 15:03 |
| alecu | TODO: more on the dash | 15:03 |
| alecu | BLOCKED: no | 15:03 |
| ralsina | and alecu is the winner o the first "I will say something bad about him on twitter because he was late for standup award" (aka the IWSBAHOTBHWLFSies) | 15:03 |
| ralsina | good standup team! :-) | 15:04 |
| rye | alecu: playing with oauth and vala? tell me moar | 15:04 |
| mandel | mmcc, would it be possible for you to build a .app with lp:~mandel/ubuntuone-client/use-new-fsevents-api and the code in github I showed you? | 15:04 |
| mandel | mmcc, it should get us closer to the goal, or so I hope | 15:04 |
| mmcc | mandel: sure | 15:05 |
| dobey | rye: did you see my earlier query re: protocol nightlies? | 15:06 |
| mandel | mmcc, if that works I'd like to fix the UI bug and one extra things, it seems that if I do right click -> quit in the app icon on the dock it does not quit, have you seen that? | 15:06 |
| alecu | rye: I'm still learning vala, and I've been looking at an oauth library, but it seems to have little tests, so I don't trust it too much yet. | 15:07 |
| mmcc | mandel: I haven't seen that, no, let me try it (i need to quit it anyway) | 15:07 |
| mmcc | mandel: worked fine for me | 15:07 |
| mandel | mmcc, hmm weird, well I try with the new version you are going to send me :) | 15:08 |
| rye | alecu: if you need signing only with no postdata the vapi is simple - http://bazaar.launchpad.net/~rye/indicator-ubuntuone/trunk/view/head:/vapi/oauth.vapi | 15:08 |
| rye | alecu: but that's with liboauth | 15:08 |
| alecu | rye: great! | 15:09 |
| rye | alecu: the generated calls from that vapi are working but if i really want to use postdata (i don't know why, though) then there appears an extra argument | 15:10 |
| rye | which reminds me i wanted to file a bug about that | 15:10 |
| rye | half a year ago | 15:10 |
| ralsina | alecu: https://twitter.com/ralsina/status/238293067386458112 | 15:14 |
| gatox | ownd! | 15:17 |
| mmcc | sorry mandel, I fat fingered a merge and screwed up my branches, have to grab them again. will have a build for you asap | 15:20 |
| mandel | mmcc, no problem, it happens :) | 15:20 |
| ralsina | may that be a cautionary tale for the team in future standups ;-) | 15:20 |
| * gatox lunch! | 15:24 | |
| === gatox is now known as gatox_lunch | ||
| dobey | lunch, bbiab | 15:32 |
| mmcc | mandel, it's working for me - I'll just go put a .zip of the new .app into the folder I'm sharing with you | 15:36 |
| mandel | mmcc, awesomeness!! | 15:36 |
| ralsina | mmcc, mandel: I just added you to the "test u1 on mac" event in calendar tomorrow. Be there ;-) | 15:36 |
| ralsina | mmcc, mandel: which mostly means "be here, and have a binary" | 15:36 |
| mandel | ralsina, you need a post-it to be a little more passive aggressive hehehehe | 15:37 |
| mmcc | mandel I'd make one small change though - the log.warn about create with no create flag should just be a debug - that happens on every move, and it's not really a problem | 15:37 |
| mmcc | ralsina: ack. | 15:37 |
| mandel | mmcc, sure, I can do that :) | 15:38 |
| ralsina | mandel: I can also do agressive agressive :-) | 15:38 |
| mandel | ralsina, in a post-it? hehe | 15:38 |
| mmcc | an angrily crumpled post-it | 15:40 |
| mandel | mmcc, pushed code to github with all the log.warn as debug | 15:41 |
| mmcc | mandel: ok, great | 15:42 |
| mandel | mmcc, let me know when you have the .app uploaded, if it works I can work in some cosmetic fixes so that I don't bother you will you work on the blessing | 15:42 |
| mandel | mmcc, if we can have a bundle with both better but if not we should not worry | 15:43 |
| === salgado is now known as salgado-lunch | ||
| mmcc | mandel: sounds great, it's uploading now | 15:44 |
| ralsina | mmcc: yes, no icon now is ok. | 15:44 |
| mmcc | ralsina: ack. thx | 15:45 |
| mandel | mmcc, where are you uploading the new .app? | 15:48 |
| mandel | or better said, new .zip | 15:48 |
| mmcc | mandel: it's in the "UbuntuOne Mac Client Builds" shared folder… | 15:50 |
| mandel | mmcc, ok, just appeared in the web ui, lets see how it goes, I'll test it a little and will propose the branch with the changes | 15:51 |
| * mmcc crosses fingers | 15:52 | |
| mandel | mmcc, did you test the drag & drop? | 16:02 |
| mmcc | mandel: yes, I tested moving a file into the watched folder via drag and drop. that's how that .zip got there, for instance | 16:03 |
| mandel | mmcc, I just did one from a not watched path to a watch path and it did not work.. or at least the file is not uploading | 16:03 |
| mmcc | I also just tested copying via drag and drop and that worked too… | 16:03 |
| mandel | mmcc, weird, I'll see with the test script what happens | 16:04 |
| mmcc | mandel: is the client connecting to an old syncdaemon process? | 16:05 |
| mmcc | I mean is control-panel connecting to an old syncdaemon? I've run into problems from that while debugging before | 16:05 |
| mandel | mmcc, hm.. bloody good point, maybe, I'll kill everything | 16:05 |
| mandel | mmcc, what is the name of the process? | 16:06 |
| mmcc | mandel: it's probably 'ubuntuone-syncdaemon' | 16:07 |
| mmcc | But thanks for reminding me that that's a bug - it should be something different and I haven't looked into it yet :) | 16:07 |
| mandel | mmcc, he, not a huge bug | 16:07 |
| mmcc | mandel: yeah, probably not. but it made me nervous in that "you're not running what you think you're running" kind of way | 16:08 |
| mandel | mmcc, now it worked like a charm | 16:08 |
| mandel | mmcc, we should do some kind of version checking, specially for whenever we have updates | 16:09 |
| mmcc | mandel: true, how do we handle that on other systems? the installer kills syncdaemon? | 16:09 |
| mandel | mmcc, I'm testing slow operations by using a samba share | 16:09 |
| mandel | mmcc, yes, on windows it gets killed on ubuntu we trust the .debs | 16:10 |
| mmcc | mandel: I figured | 16:10 |
| mandel | mmcc, so I'm testing a 250mg move from a samba share to u1, if uploads correctly I cannot thing of any other possible problem | 16:11 |
| mmcc | mandel: I'm curious what happens with the slow op with your extra modify event. I think it'll upload a small file first then the full file after it's done and it gets the second (real) modify event | 16:11 |
| mandel | mmcc, probably we will have a small file there, similar to stopping the upload in the middle | 16:12 |
| mandel | mmcc, you get a 0bytes file, just checked | 16:12 |
| mmcc | aha. that's what I expected… | 16:12 |
| mandel | mmcc, which is not a problem because 0bytes are useless, once the move from the samba to the folder finished the sd noticed and started uploading | 16:13 |
| mandel | mmcc, I think we have an alpha :) | 16:13 |
| mmcc | if we tested the flag and only sent the modify, we wouldn't get that 0 byte file… | 16:14 |
| mandel | mmcc, we have the same problem on linux, so not to worry just right now | 16:14 |
| mandel | mmcc, lets try to wrap this thing, fix some small issues and let people play with it | 16:15 |
| mmcc | mandel: great. I'm sure we'll end up tweaking it some more as people test it anyway | 16:16 |
| mandel | mmcc, exactly, for an alpha the quality of the result is much better than the one we had for windows | 16:16 |
| mmcc | thanks to all the work on the windows client making it easier to port to mac… | 16:18 |
| mandel | mmcc, yes, that was a very good thing, the about of work that had to be done for windows was crazy | 16:19 |
| mmcc | ok, quick coffee break here… | 16:19 |
| mandel | ack | 16:19 |
| === gatox_lunch is now known as gatox | ||
| mandel | gatox, ralsina, super lame review: https://code.launchpad.net/~mandel/ubuntuone-control-panel/fix-1019224/+merge/120823 | 16:29 |
| gatox | on it | 16:29 |
| ralsina | mandel: got it | 16:30 |
| ralsina | mandel: the daemon is perfect? Why are you tweaking qss???? | 16:31 |
| mandel | ralsina, is perfect with fsevents, now I have to wait for mmcc to get the blessing working and we move to the daemon | 16:32 |
| ralsina | mandel: whoohoo | 16:32 |
| mandel | ralsina, that branch should have added QTabWidget::tab-bar{aligment:left;} but the launchpad is miss behaving.. | 16:33 |
| ralsina | mandel: pictures? | 16:33 |
| mmcc | .3. | 16:33 |
| mandel | ein? | 16:33 |
| mandel | .3. | 16:34 |
| mmcc | toddler typing, sorry | 16:34 |
| ralsina | jajaja | 16:34 |
| mmcc | that's Hadley saying hi | 16:34 |
| ralsina | hi Hadley! | 16:34 |
| mmcc | :) | 16:34 |
| mandel | hahahaha | 16:34 |
| ralsina | wtf launchpad? a 1-line diff takes over 10 minutes to update now? | 16:39 |
| mandel | ralsina, is broken, do a bzr diff --old lp:ubuntuone-control-panel | 16:39 |
| mandel | ralsina, it also was saying that the branch was empty | 16:39 |
| ralsina | mandel: that assumes I can do things like branch out of launchpad | 16:40 |
| ralsina | oh, there it goes | 16:40 |
| mandel | ralsina, you can, can't you? | 16:40 |
| ralsina | mandel: took a long time to start, but yes I can | 16:41 |
| gatox | mandel, +1 | 16:41 |
| mandel | mmcc, I think we can share the last bundle with alecu and gatox for them to try it, the more the better | 16:41 |
| mandel | gatox, thx! | 16:41 |
| ralsina | mandel: merge it gatox's +1 | 16:41 |
| ralsina | mandel: the change looks harmless enough | 16:41 |
| ralsina | although on mac, tabs are *supposed* to be centered ;-) | 16:42 |
| mmcc | mandel: ok, sure - gatox, alecu, ralsina, tell me what email to send a share to | 16:42 |
| mmcc | ralsina: on mac, tabs center you | 16:42 |
| mandel | ralsina, yes, but design filed the bug :) | 16:42 |
| gatox | mmcc, diego.sarmentero (at) gmail.com | 16:42 |
| gatox | ralsina, yes, all the tabs always are centered in the middle by default in mac | 16:43 |
| gatox | on macc | 16:43 |
| ralsina | yes, so go ahead and merge | 16:43 |
| * alecu will have lunch+errands now. | 16:45 | |
| ralsina | same here | 16:46 |
| mandel | gatox, any idea where I should start looking regarding the scrolling bug? | 16:48 |
| mandel | gatox, I'd like to have that fix before we let many others test this | 16:48 |
| gatox | mandel, i would start looking how those buttons are added and who is the parent | 16:48 |
| mandel | gatox, ack | 16:48 |
| mandel | alecu, mmcc, gatox, may I have a review for https://code.launchpad.net/~mandel/ubuntuone-client/use-new-fsevents-api/+merge/120833 | 17:06 |
| gatox | mandel, looking | 17:06 |
| mmcc | mandel: looking too. | 17:07 |
| mandel | gatox, mmcc, awesome, it looks like launcpad is not working so for a decent diff use the bzr diff --old lp:ubuntuone-client --using meld | 17:09 |
| mandel | or which ever diff tool you like :) | 17:09 |
| dobey | elopio: you are done testing the client stuff from yesterday i presume? | 17:09 |
| elopio | dobey: yes, I sent a mail to ubunet-discuss like an hour ago. | 17:10 |
| mandel | ralsina, can we move the alpha testing a little earlier, I'd like to go to rugby training, if it not possible don't worry | 17:10 |
| dobey | ah, ok :) | 17:10 |
| mmcc | so mandel, it's just a one-line diff in fsevents_client? | 17:11 |
| mandel | mmcc, yes, super lame, but we have to pass those keyword args :) | 17:11 |
| gatox | mandel, is just a one line change? | 17:14 |
| === salgado-lunch is now known as salgado | ||
| mandel | gatox, yes, because all the other fixes are in that github project | 17:15 |
| gatox | mandel, +1 | 17:15 |
| mmcc | mandel +1, from me too | 17:16 |
| mandel | awesome! | 17:16 |
| mandel | mmcc, we should be able to build a working bundle from trunk plus the github code | 17:16 |
| mmcc | mandel: that should be identical to the bundle I already posted, are you thinking we should test with trunk once it merges just to be sure? | 17:18 |
| mandel | mmcc, should be identical so not to worry just yet, I'll be doing some UI fixed while you are at the blessing stuff and if those land we can build a new UI with less annoying bugs :) | 17:19 |
| mmcc | mandel: ok yeah. Once you're done with those ui fixes it might be nice to have you try to build the .app. I'm still not sure anyone but me has done it | 17:20 |
| mandel | mmcc, yes, like with briancurtin he is the poor soul that know how to do it on windows | 17:21 |
| mmcc | job security! | 17:21 |
| briancurtin | ha. once you get the installer building stuff under control, build_installer.py could be expanded to build the mac bundle (or whatever its called) | 17:23 |
| mandel | we should automate the thing, that is true | 17:24 |
| mmcc | briancurtin: well, there is no installer, really, so it basically is automated now. setup-mac doesn't know about either fsevents daemon, but There's a Bug for That™ | 17:24 |
| mmcc | and py2app knows how to create a .dmg if we want to do that instead of a .zip | 17:25 |
| === dduffey_afk is now known as dduffey | ||
| === urbanape` is now known as urbanape | ||
| mandel | mmcc, seems that launchpad is busted.. | 17:41 |
| briancurtin | surprising | 17:41 |
| mmcc | mandel: here's another quick thing we should discuss before releasing an alpha: see line 39 of __init__.py | 17:47 |
| mmcc | sorry, ubuntuone/platform/__init__.py | 17:47 |
| mandel | mmcc, ok, looking | 17:47 |
| mmcc | I looked into it a couple weeks ago because it looks awful, we shouldn't be redefining a separate platform string based on sys.platform… but it's historical, and turns out the string is only used to send to the servers as the platform ID | 17:48 |
| mmcc | so we might want to change it before we start using the client to test, so that the server-side has the right client platform | 17:49 |
| mandel | mmcc, yes, we will be messing up the stats otherwise, but should not be a terrible problem, lets file a bug for that | 17:50 |
| mmcc | there's already a bug | 17:51 |
| mmcc | well, sort of: https://bugs.launchpad.net/ubuntuone-client/+bug/1037435 | 17:52 |
| ubot5 | Ubuntu bug 1037435 in Ubuntu One Client "TestPlatform.test_import fails on darwin" [High,New] | 17:52 |
| mmcc | it's the same issue but the bug is about the test | 17:52 |
| briancurtin | anyone have a few minutes for a simple review? import renaming for py3, print function - https://code.launchpad.net/~brian.curtin/ubuntuone-dev-tools/py3-u1lint/+merge/120819 | 17:53 |
| mandel | mmcc, ok, byut I think these are bugs we should not worry for the friday deadline | 17:53 |
| mmcc | mandel: ok. it's fast enough to fix anyway if someone cares after we've started testing | 17:54 |
| mandel | seriously what the frack is going on with lp, do you guys have problems too? | 17:55 |
| briancurtin | works alright for me | 17:55 |
| mmcc | mandel: yeah, your one-line branch still doesn't have a diff showing for me, and now the autopilot commented twice about additional revisions that aren't there… | 17:57 |
| mmcc | I guess that's the same thing as tarmac? "otto-pilot"? | 17:58 |
| ralsina | mmcc: that's because it has never scanned the revision that is there | 17:58 |
| mmcc | ralsina: aha | 17:58 |
| ralsina | briancurtin: I can do that review | 17:59 |
| mandel | is plain weird | 17:59 |
| ralsina | mandel: could you re-propose? | 17:59 |
| ralsina | mandel: it could be just your branch | 17:59 |
| mandel | ralsina, smae problem with all the branch I proposed.. | 18:00 |
| ralsina | mandel: briancurtin proposed this one 2 hours ago and it looks ok | 18:00 |
| mandel | ralsina, mine are very recent, I'll try again | 18:00 |
| ralsina | mandel: ok | 18:00 |
| ralsina | briancurtin: global aprove | 18:01 |
| briancurtin | ralsina: have time for another one? it has one review but since its unicode related it's a bit more involved than most of my small trivial branches: https://code.launchpad.net/~brian.curtin/ubuntuone-dev-tools/py3-unicode-literals/+merge/120662 | 18:03 |
| ralsina | briancurtin: sure | 18:04 |
| mandel | ralsina, look, just pushed to a diff location and it states is empty: https://code.launchpad.net/ubuntuone-control-panel | 18:05 |
| ralsina | mandel: grmp | 18:06 |
| ralsina | mandel: ask in #launchpad maybe they can figure it out | 18:06 |
| mandel | ralsina, I was just asking there | 18:07 |
| ralsina | mandel: ok | 18:07 |
| mandel | ralsina, can you try and fork and push? maybe it works for you | 18:07 |
| mmcc | yeah, bazaar.launchpad.net seems to be not responding for me… I can't browse mandel's branch | 18:07 |
| ralsina | mandel: sure, in 1' | 18:07 |
| ralsina | mandel: pushed to https://code.launchpad.net/~ralsina/ubuntuone-control-panel/fix-1019224 still appears empty there, too | 18:09 |
| mandel | mmcc, he, that means we are getting close to the deadline, greblims have started doing their work | 18:09 |
| ralsina | mandel: you can still probably merge manually | 18:09 |
| mandel | ralsina, yes, but to trunk? | 18:10 |
| * mmcc doesn't believe in greblims | 18:10 | |
| ralsina | mandel: well... I know, scary | 18:10 |
| ralsina | mandel, mmcc: you guys could continue in your own branch and merge there | 18:10 |
| ralsina | mandel, mmcc: so this doesn't disrupt your sprint | 18:10 |
| ralsina | mandel, mmcc: bzr serve and all that | 18:11 |
| mandel | ralsina, yes, should not be a HUGE problem, I'm talking at #launchpad to see what can be done | 18:11 |
| ralsina | mandel: ok | 18:11 |
| mandel | ralsina, I might just mean and early dinner break for me and done :) | 18:12 |
| ralsina | mandel: definitely doesn't update for me either, so it's either somethingin u1cp or something in that branch's changes, or all of launchpad | 18:13 |
| ralsina | mandel: and "all of launchpad" would make more noise :-) | 18:13 |
| mandel | ralsina, in canonical server at #launchpad-ops | 18:13 |
| dobey | it says it's updating | 18:14 |
| dobey | it's definitely not 'all of lp' | 18:15 |
| dobey | at least i haen't noticed any such problems in my work today | 18:16 |
| === zyga is now known as zyga-afk | ||
| dobey | mandel: tried to delete the branch on lp, and then just push it again? | 18:17 |
| mandel | dobey, from launchpad ops: 'Yes that is excessive. We do not have any script failures at this moment, so slowness is largely backlog' | 18:17 |
| mandel | dobey, so that would not work | 18:17 |
| ralsina | mandel: I pushed it too and it never finishes scanning | 18:17 |
| dobey | mandel: when did you push to that branch? | 18:18 |
| ralsina | I'll delete my branch to make that backlog slightly smaller ;) | 18:18 |
| dobey | err, i guess that's ralsina's branch | 18:18 |
| mandel | dobey, 2 hours ago, and a copy of it a few mins ago | 18:18 |
| dobey | well, my branches are getting scanned just fine | 18:19 |
| dobey | even the one for ubuntuone-client which has like 1300 toplevel revisions | 18:19 |
| mandel | dobey, can you branch lp:~mandel/ubuntuone-control-panel/left-tabs and push it to a location of yours? just for testing | 18:19 |
| ralsina | mandel: I did that already | 18:20 |
| mandel | then we are just stuck like that | 18:20 |
| dobey | speaking of branches. | 18:23 |
| dobey | alecu, ralsina: https://code.launchpad.net/~dobey/ubuntuone-client/update-4-0/+merge/120847 please :) | 18:23 |
| ralsina | dobey: ack | 18:23 |
| dobey | mandel: it's just your branch which is the problem | 18:28 |
| mandel | dobey, but has happened with several of my branches | 18:29 |
| dobey | mandel: the stacking is broken | 18:29 |
| ralsina | briancurtin: +1 on py3-unicode-literals | 18:29 |
| ralsina | dobey: +1 on ubuntuone-client/update-4-0 | 18:29 |
| dobey | mandel: i just pulled trunk and pushed it to a new branch, and it was rescanned within 5 minutes | 18:30 |
| dobey | actually, less than that. | 18:30 |
| mandel | dobey, but using mine or a diff branch? | 18:30 |
| dobey | less than a minute perhaps | 18:30 |
| dobey | mandel: lp:ubuntuone-control-panel | 18:31 |
| dobey | mandel: copy of your branch still isn't rescanned | 18:31 |
| mandel | bummer | 18:31 |
| dobey | mandel: which means your branch is busted | 18:31 |
| mandel | dobey, I'll create it from scratch and will simple merge the diff with patch | 18:31 |
| briancurtin | ralsina: thanks for the reviews | 18:32 |
| dobey | mandel: and just merged your branch into my test, committed and pushed, and it was rescanned | 18:32 |
| mandel | can you propose it, is a simple qss change :) | 18:33 |
| * briancurtin lunch | 18:34 | |
| dobey | i am doing not so simple releases which are a much higher priority :) | 18:34 |
| mandel | dobey, sure, no problem | 18:34 |
| mandel | I'm off to have dinner | 18:54 |
| * mandel dinner | 18:54 | |
| dobey | wtf | 18:59 |
| dobey | dh_install: usr/share/ubuntu-sso-client/com.ubuntu.sso.service.in exists in debian/tmp but is not installed to anywhere | 18:59 |
| dobey | why is that file installed to the system?! | 18:59 |
| ralsina | school run, I may have to babysit, so I will be working late today | 19:07 |
| ralsina | ping me for reviews and such | 19:07 |
| dobey | hrmm | 19:18 |
| dobey | for the qt sso and control panel, aren't all the resource data files 'compiled in' to the app? | 19:19 |
| dobey | anyone? mcfly? | 19:22 |
| mmcc | dobey: on what platform? are you just talking about the qt ui files or other stuff too? | 19:22 |
| dobey | i guess it's the same in all platforms. but the ui files, qss files, png files, and font | 19:23 |
| dobey | oh fml | 19:24 |
| dobey | why are the tests using so much memory merging to stable-4-0 for ubuntuone-client, but not when merging to trunk?! | 19:25 |
| dobey | guess i'll have to merge by hand :-/ | 19:25 |
| dobey | (again) | 19:25 |
| mmcc | dobey, so yeah, those resources should all be compiled in for those projects. I thought I remembered other stuff that wasn't, but it was for u1client and storage protocol (conf files and .pems) | 19:31 |
| dobey | hmm, guess i should try | 19:38 |
| ralsina | dobey: yes, everything is compiled into .py files | 19:53 |
| * mandel back | 19:59 | |
| mmcc | welcome back mandel | 20:05 |
| mandel | mmcc, need dinner, lets see if I can merge those branches to trunk | 20:05 |
| mmcc | I thought you just had dinner? | 20:05 |
| dobey | dh --fail-missing hates me | 20:08 |
| mmcc | boy, an old part of me sure rebels when I think about all the layers of code this stuff is going through, just to be portable… thank god computers are fast | 20:09 |
| mmcc | (not just our stuff, but Qt, PyQt, python <-> OSX C API, ctypes translation, etc etc etc | 20:10 |
| mandel | mmcc, sorry should have been past tense | 20:10 |
| === zyga-afk is now known as zyga | ||
| mmcc | mandel: I'm glad you're sorry | 20:10 |
| mandel | mmcc, plus the extra layers we added for the ipc and platform | 20:10 |
| mandel | mmcc, I'm not in my best typing :) | 20:10 |
| mmcc | mandel: for real. I'm crashing python over here with ctypes, and some of these stack traces are impressive - and that's just the bottom layer before twisted and ipc and junk. still it could be worse | 20:11 |
| mandel | mmcc, yes, is complicated code, I guess is one of the reasons why it does not get boring, yet a little exasperating | 20:12 |
| mmcc | definitely not boring. | 20:13 |
| mmcc | nothing is boring if it's broken :) | 20:13 |
| alecu | mmcc: let's forget layers! let's go bare skin on metal and code assembler! | 20:22 |
| alecu | woohoo! | 20:23 |
| * alecu looks around and starts putting his clothes back on. | 20:24 | |
| mmcc | just saw that --> :D | 20:28 |
| briancurtin | be back in a bit, need to get away from the computer for a bit | 20:37 |
| gatox | alecu, philosophical question | 20:46 |
| gatox | alecu, is there any reason that everytime the control panel loses focus when it gains focus again it changes automatically to the folders tab? | 20:46 |
| gatox | at least for me is really annoying | 20:47 |
| gatox | well..... actually is not when it loses focus..... but when i change from one desktop to another | 20:47 |
| gatox | it is happening in the showEvent | 20:48 |
| alecu | gatox: I can't seem to duplicate that behaviour on Precise. | 20:48 |
| gatox | it happens on minimize and restore | 20:48 |
| gatox | alecu, minimize and restore the u1-cp | 20:48 |
| alecu | gatox: I can't see that changing desktops nor restoring the window. | 20:48 |
| * gatox is once again the corner case guy | 20:49 | |
| alecu | gatox: do you have the line of code where this happens? | 20:49 |
| gatox | alecu, let me check | 20:50 |
| alecu | gatox: I just reproduced it on Quantal | 20:50 |
| gatox | :S | 20:50 |
| gatox | alecu, i'm on precise with unity2d | 20:51 |
| gatox | alecu, i didn't find it yet, but my money is on switch_to being called from someplace on show | 20:52 |
| alecu | gatox: this probably happens because I've got loads of videomem on my Precise machine, so the window is still on the backing store. | 20:52 |
| gatox | alecu, look at this (not related to the issue i mentioned): http://ubuntuone.com/4QUXWKwvJPEIIRTTz74BoM - http://ubuntuone.com/4Zr1b5ot1wGxJvlkAdAnSg | 20:54 |
| * gatox is excited about that | 20:54 | |
| mmcc | gatox, very nice! | 20:55 |
| alecu | nice! | 20:55 |
| gatox | mmcc, thx! :D | 20:55 |
| gatox | i hope to finish it by tomorrow | 20:55 |
| alecu | gatox: so, this is a cross platform way to publish files, right? | 20:55 |
| alecu | gatox: awesome | 20:55 |
| gatox | alecu, yes | 20:55 |
| * briancurtin back | 20:55 | |
| gatox | wowwwwwwww 5:55...... | 20:55 |
| gatox | ok...... eod for me! | 20:55 |
| gatox | time flies when you do ui :P | 20:56 |
| gatox | see you tomorrow people!! :D | 20:57 |
| === salgado is now known as salgado-brb | ||
| mandel | mmcc, I'm going to call it a day, my productivity is -100 | 21:15 |
| mandel | all, see you tom! | 21:15 |
| mmcc | ok mandel, rest up. I've got my python version of the job-bless example working, hopefully by tomorrow I'll have it in the control panel code and we can test it out | 21:16 |
| mmcc | btw, I know how to solve the version issue too, without designing a new IPC version interface, since we can query the info.plist of the installed daemon for a version number and kill it if it's too old (or something) | 21:17 |
| mmcc | lunchtime for me here… | 21:25 |
| dobey | alecu, ralsina: https://code.launchpad.net/~dobey/ubuntuone-control-panel/update-4-0/+merge/120874 please | 21:30 |
| alecu | dobey: +1 | 21:36 |
| alecu | (trivial) | 21:36 |
| alecu | and EOD for me. | 21:36 |
| === salgado-brb is now known as salgado | ||
| dobey | sigh, i guess i won't get this finished tonight. am seeing some weird test failures :( | 22:22 |
| dobey | when running tests from the tarball; but the tests ran fine from the tree | 22:22 |
| dobey | le sigh | 22:22 |
| dobey | but no new files in the diff really, so not like it's due to something not getting disted in the tarball | 22:25 |
| dobey | :( | 22:25 |
| dobey | oh well, need to get away for a while. later all | 22:29 |
| mmcc | bye dobey, better luck tomorrow | 22:30 |
| mmcc | ok, I have to run for a bit, will be back tonight to finish this up. I'm optimistic here - my SMJobBless code seems to be working fine, the only hurdle now is code signing, and I think that might be OK… | 23:21 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!