| ottadini | hello, just joined to ask for some help. | 04:49 |
|---|---|---|
| ottadini | It seems I may have lost an entire dir tree due to a failed sync with U1. | 04:50 |
| ottadini | The folder structure is there, but no files. | 04:50 |
| ottadini | u1sdtool status reports it is idle. | 04:51 |
| ottadini | How it happened: I dragged a folder that was already a UDF and being synced with U1 into the 'Ubuntu One' folder in Nautilus. | 04:54 |
| ottadini | Minutes later I reversed this, and copied the directory back to its original location. | 04:55 |
| ottadini | End result: no files, just the directory tree. | 04:55 |
| ottadini | I have checked online at my account on one.ubuntu.com, and same result. | 04:57 |
| === zyga-afk is now known as zyga | ||
| === teknico_ is now known as teknico | ||
| JamesTait | Good morning all! Happy Friday the 13th! >:-) | 08:49 |
| popey | :( my quantal machine has been "uploading" some files for about a month now | 08:59 |
| popey | so i removed all the files from the folder in nautilus just now.. | 09:00 |
| popey | syncdaemon still seems to be chatting away about the files in its log | 09:00 |
| popey | and sync is "in progress" | 09:00 |
| popey | I stop and start and it still things it's uploading files which I moved out of that synced folder | 09:01 |
| mandel | popey, our staff is very very broken in Q atm | 10:42 |
| mandel | popey, sorry :( | 10:43 |
| mandel | popey, with the move to python3, the mac port etc.. we are overwhelmed atm but afaik ralsina is going to look into the problems | 10:43 |
| popey | mandel, ok :) | 11:02 |
| gatox | good morning! | 11:09 |
| gatox | mandel, hi | 11:09 |
| mandel | gatox, morning! feeling any better? | 11:09 |
| gatox | mandel, MUCH, i slept all the day yesterday........ did you do anything with my branches yesterday? | 11:10 |
| mandel | gatox, I approved darwin4 I'm going to propose one with the daemon, I have move notify processor to its own package which removes circular importans and I have added a darwin only command line arg to choose if the deamon is used :) | 11:11 |
| mandel | gatox, my I have reviews: https://code.launchpad.net/~mandel/ubuntuone-client/fsevents-daemon/+merge/114836 :) | 11:12 |
| gatox | yes | 11:12 |
| gatox | mandel, 2163 lines?? and you complain about me...... shame on you! :P | 11:13 |
| mandel | gatox, wait, missing dep! | 11:14 |
| mandel | give me a sec | 11:14 |
| mandel | gatox, oh no, looks like a move is not considered a move.. | 11:15 |
| mandel | bad bzr.. | 11:15 |
| gatox | mandel, did you do bzr mv? | 11:16 |
| mandel | gatox, afaik yes, but in one of the pump when wrong and I have to resolve a conflict.. maybe that screwed up the diff | 11:16 |
| mandel | gatox, you can see there is a large delete and then the same code added later.. | 11:16 |
| gatox | ok | 11:17 |
| mandel | gatox, sorry for the PITA | 11:17 |
| gatox | :P i'm kidding | 11:18 |
| mandel | gatox, regarding the NotifyProcessor, in a later branch I have moved it to filesystem_notifications/notify_processor/ that was we can remove a circular import issue we had and make things cleaner | 11:19 |
| gatox | mandel, where are we having a circular import? | 11:20 |
| mandel | gatox, in you look in notify_processor you will see there is a XXX: circular import comment, I have fixed that | 11:20 |
| mandel | gatox, mainly because later I was having a circular import when trying to reuse the notify processor you added, the fsevents and the fsevents_daemon can share it | 11:21 |
| gatox | ahhhhhhh yes, the one in the __init__ i remember | 11:23 |
| mandel | gatox, exactly, so that is out, which I think is a good move :) | 11:24 |
| mandel | gatox, I'm starting to wonder (although won't change it) what is the need of having platform infront of everything, but that is a minor detail | 11:24 |
| mandel | gatox, the code is looking a lot better know, we might be able to port it to the sp3 ;) | 11:24 |
| gatox | sp3? ps3? | 11:25 |
| mandel | gatox, sorry ps3 | 11:26 |
| mandel | gatox, one hand is faster than the other.. :P | 11:26 |
| mandel | or was I thinking about xp sp3 | 11:26 |
| mandel | .. | 11:26 |
| gatox | jejej | 11:26 |
| gatox | you are such a windows lover..... maybe was that | 11:26 |
| mandel | maybe.. | 11:27 |
| mandel | gatox, early lunch for me, ok? | 11:40 |
| * mandel lunch | 11:41 | |
| gatox | mandel, ack | 11:44 |
| * gatox realize that mandel's branch is going to cause conflicts in the refactor one.... fun not | 11:56 | |
| gatox | mandel, mmmmm something don't look right..... shouldn't this be depending on darwin4?? i see some changes that are already in trunk | 11:58 |
| gatox | mmm i think there is some kind of problem merging darwin4.... first time i see this on launchpad: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/darwin4-fsevents/+merge/114405 | 12:04 |
| ralsina | good morning! | 12:19 |
| ralsina | how are you today gatox? | 12:19 |
| gatox | ralsina, fine..... today was 100% rest :P | 12:19 |
| gatox | ralsina, do you know what it means "This proposal supersedes" here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/darwin4-fsevents/+merge/114405 | 12:20 |
| ralsina | gatox: looking... | 12:20 |
| gatox | i'm trying to look for some doc or something, but i can't find anything clear enough | 12:20 |
| gatox | or how to fix it | 12:20 |
| ralsina | gatox: that this branch was proposed twice and this is the newer one | 12:20 |
| ralsina | that's not bad, AFAIK | 12:20 |
| gatox | ralsina, aja..... but why is not being merged? | 12:20 |
| gatox | ahhhhh | 12:21 |
| gatox | commit message | 12:21 |
| gatox | it seems i forget to include that when i repropose | 12:22 |
| === zyga is now known as zyga-food | ||
| === zyga-food is now known as zyga | ||
| alecu | hello! | 12:47 |
| alecu | gatox: I see that darwin4 has landed. Awesome! | 12:51 |
| alecu | gatox: I've created a bug for the deferred stop and the thread join fun. I'll probably be working on that. | 12:51 |
| gatox | alecu, yes, i saw it | 12:53 |
| gatox | alecu, ahhhh.... i thought it was for me :P | 12:53 |
| dobey | hrmm | 13:04 |
| dobey | no reviews on https://code.launchpad.net/~dobey/dirspec/python3/+merge/114713 :-/ | 13:05 |
| === yofel_ is now known as yofel | ||
| gatox | alecu, ping | 13:19 |
| gatox | alecu, when you are back...... is this the bug you were talking about? https://bugs.launchpad.net/ubuntuone-client/+bug/1024102 i see that is assigned to me, and i was going to add it to the tests-refactoring branch | 13:23 |
| ubot5 | Ubuntu bug 1024102 in Ubuntu One Client "Deferred should be fired when a watch is stopped" [Undecided,New] | 13:23 |
| gatox | mandel, ping when you are back.... i've some questions about your branch | 13:24 |
| dobey | gatox: maybe you should review https://code.launchpad.net/~mikemc/ubuntuone-client/use-dirspec-get-cmdline/+merge/114720 | 13:27 |
| gatox | dobey, ack | 13:27 |
| dobey | mandel: 2163 lines?! | 13:35 |
| dobey | briancurtin: any more work on the py3-unicode branch? it's still needs fixing from alecu | 13:37 |
| * alecu is back | 13:38 | |
| gatox | alecu, when you are back...... is this the bug you were talking about? https://bugs.launchpad.net/ubuntuone-client/+bug/1024102 i see that is assigned to me, and i was going to add it to the tests-refactoring | 13:39 |
| ubot5 | Ubuntu bug 1024102 in Ubuntu One Client "Deferred should be fired when a watch is stopped" [Undecided,New] | 13:39 |
| alecu | gatox: that's the bug, yes. I assigned it to you, but ralsina told us yesterday that you may start working on a different project | 13:39 |
| gatox | alecu, ahhhhh ok | 13:39 |
| alecu | gatox: that's why I said I might be working on fixing that. | 13:39 |
| gatox | ohhh i miss the meeting..... | 13:40 |
| briancurtin | dobey: i haven't gotten to that one. i'll take a look today (but i'll be gone for a while in the morning through late afternoon) | 13:40 |
| ralsina | gatox: yes, I am planning on switching you to the menu/indicator thing starting next week | 13:40 |
| gatox | anyone know what i'm going to be doing? :P | 13:40 |
| briancurtin | s/late/mid | 13:40 |
| alecu | gatox: you are being moved to work on the u1 port to VMS. | 13:40 |
| gatox | ralsina, ohhhhh great!! \o/ | 13:40 |
| gatox | alecu, haah | 13:40 |
| alecu | gatox: or was it OS/2? | 13:40 |
| gatox | ralsina, cool! | 13:40 |
| ralsina | gatox: see ⟰ | 13:40 |
| dobey | python 4 | 13:40 |
| ralsina | dobey: writing it! | 13:41 |
| * ralsina liked VMS. Sniff. | 13:41 | |
| dobey | it's the version of python where guido hits senility, and it's more like brainfuck than python | 13:41 |
| gatox | alecu, ok..... so..... if you are going to do that..... i think this is ready for review: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/darwin-tests-refactoring/+merge/113289 (about the comments: we already talk about the assert_logs removal, and i removed the unnecesary setUp for TestNotifyProcessor) | 13:42 |
| alecu | gatox: ack | 13:43 |
| alecu | dobey: I'm running the tests for ~dobey/dirspec/python3, and I'm getting an error when running the python2 tests on Precise. | 13:49 |
| alecu | dobey: should I be running them on quantal? | 13:49 |
| dobey | alecu: python2 or python3? | 13:50 |
| dobey | alecu: the tests are passing for me on precise | 13:52 |
| === teknico_ is now known as teknico | ||
| alecu | dobey: I'm getting these errors when running on precise: http://pastebin.ubuntu.com/1089886/ | 14:02 |
| alecu | dobey: I'm updating to see if I had something really old. | 14:02 |
| dobey | alecu: there is a testools in the nightlies PPA, that is needed | 14:06 |
| dobey | alecu: and you'll need python3-setuptools and python3-testtools | 14:08 |
| dobey | updated the description about that | 14:11 |
| mandel | gatox, shoot | 14:14 |
| mandel | dobey, yes, there was some code moves and turns out to be bigger than expected | 14:15 |
| mandel | dobey, is mainly a copy paste to a diff module to be honest | 14:15 |
| gatox | mandel, i'm seeing some lines that shouldn't be there because are changes from trunk..... or am i missing something? | 14:15 |
| dobey | bzr mv didn't work? :) | 14:15 |
| gatox | mandel, for example: class TestWatchManager(BaseTwistedTestCase): | 14:16 |
| gatox | 77+ """Test the watch manager.""" | 14:16 |
| gatox | and the tests in there | 14:16 |
| mandel | dobey, no because it was not the entiry module :( | 14:16 |
| mandel | gatox, which line of the diff? | 14:16 |
| ralsina | dobey: how do you feel about uploading dirspec to pypi.python.org eventually | 14:16 |
| dobey | ralsina: "eventually" sounds fine. perhaps with the release on monday | 14:17 |
| gatox | mandel, 76.... also..... test_stop_multiple has no longer that implementation, we remove the time.sleep | 14:17 |
| mandel | gatox, hm.. there was probably a miss merge somewhere.. | 14:17 |
| * mandel hates having so many branches.. | 14:18 | |
| ralsina | dobey: yes, after release is ok | 14:18 |
| mandel | gatox, let me remerge and see what can be fixed | 14:18 |
| alecu | dobey: I've updated everything, installed python3-testtools, but I still get that error in python 2, and now the same error with python 3 | 14:18 |
| gatox | mandel, ack | 14:18 |
| dobey | alecu: weird | 14:19 |
| dobey | alecu: what does "ls -lhd ~/.config/x" show? | 14:24 |
| briancurtin | going to do my apartment thing, back hopefully soon | 14:24 |
| dobey | alecu: seems like you have a file or directory at ~/.config/x | 14:25 |
| dobey | alecu: indeed, if i touch ~/.config/x, the test then fails | 14:26 |
| dobey | wonder how you got that file :) | 14:26 |
| dobey | anyway, need to run an errand. brb | 14:26 |
| mandel | gatox, branch updated | 14:41 |
| gatox | mandel, ack | 14:41 |
| mandel | gatox, he, the diff lost 500 lines :) | 14:44 |
| gatox | :P | 14:44 |
| mandel | gatox, nad if the bzr mv had worked the diff would be smaller.. | 14:45 |
| gatox | mmcc, ping | 14:57 |
| mmcc | gatox pong | 14:57 |
| gatox | mmcc, i put a need fixing here: https://code.launchpad.net/~mikemc/ubuntuone-client/use-dirspec-get-cmdline/+merge/114720 because that lint error is causing problems to run the tests on linux | 14:58 |
| mmcc | gatox, ok thanks. wonder why that didn't show up when I ran them :\ | 14:59 |
| mandel | me | 15:00 |
| gatox | me | 15:00 |
| mmcc | me | 15:00 |
| dobey | meh | 15:00 |
| ralsina | me | 15:01 |
| ralsina | alecu, thisfred: standup | 15:03 |
| thisfred | ne | 15:03 |
| ralsina | brian is away for a bit | 15:03 |
| thisfred | me | 15:03 |
| alecu | me | 15:03 |
| ralsina | so alecu is last, go mandel | 15:03 |
| mandel | DONE: Proposed fsevents daemon code for u1-client. Added a new branch that removes cicle imports from the NotifyProcessor code. Added a new command line arg just for darwin. | 15:03 |
| mandel | TODO: Split file system monitor for better import. Propose the branches. Be done with fsevents, although if gatox is moved there are a number of bugs that should be assigned to me next week regarding fsevents :( | 15:03 |
| mandel | BLOCKED: no | 15:03 |
| mandel | gatox, please | 15:03 |
| gatox | DONE: | 15:04 |
| gatox | Take the day off yesterday to reload the health battery. Reviews, small fix in tests-refactoring (NEED REVIEWS FOR THIS). Sent email to ralsina, alecu and lisette about the file publisher in the indicator menu, start reading the syncmenu wiki and X platform desktop indicator menus document. | 15:04 |
| gatox | TODO: | 15:04 |
| gatox | Finish with those documents. Keep reviewing branches and fixing fsevents related bugs. | 15:04 |
| gatox | BLOCKED: | 15:04 |
| gatox | No | 15:04 |
| gatox | mmcc, go | 15:04 |
| mmcc | DONE: finished path finding, hacked on setup-mac | 15:04 |
| mmcc | TODO: cleanup u1client branch, more setup-mac, reduce package size | 15:04 |
| mmcc | BLCK: | 15:04 |
| mmcc | NEXT: dobeh | 15:04 |
| dobey | DONE: team call, bug #1024003, bug #1016224, backported quantal software-center to nightlies to unbreak packages from sso-gtk removal | 15:04 |
| ubot5 | Launchpad bug 1024003 in dirspec (Ubuntu Quantal) "Unnecessarily requires twisted trial for running tests" [Undecided,Triaged] https://launchpad.net/bugs/1024003 | 15:04 |
| dobey | TODO: reviews, investigate twisted-less dev-tools, work on some reorg to drop ubuntuone-installer for good, SRU verification poking | 15:04 |
| ubot5 | Launchpad bug 1016224 in dirspec trunk "Does not work with Python 3.x" [Undecided,In progress] https://launchpad.net/bugs/1016224 | 15:04 |
| dobey | BLCK: None. | 15:04 |
| dobey | ralsina | 15:04 |
| ralsina | DONE: team call, reviews, VM haggling, talked a lot with lisette, contractor wrangling, some reviews, little coding TODO: more of the same, try to debug lists on quantal BLOCKED: coughing a whole lot. NOTE: mandel, we can delay gatox's switch a couple of days more if needed NEXT thisfred | 15:04 |
| thisfred | DONE: worry about txids in u1db TODO: write better integration tests that do the worrying for me BLOCKED: no, but I could use a few clones or a time machine NEXT: alecu | 15:04 |
| gatox | mandel, i've right now 2 issues related to non-root fsevents, do you have any other in mind? | 15:06 |
| * alecu is writting notes | 15:06 | |
| alecu | DONE: many reviews, 1-1 and team meetings, rebuilt my broken thunderbird setup | 15:08 |
| alecu | TODO: catch up with briancurtin and py3k | 15:08 |
| alecu | BLOCKED: no | 15:08 |
| dobey | EOM? | 15:10 |
| alecu | NEXT: briancurtin (when he returns) | 15:10 |
| dobey | alecu: did you see my replies about my branch and the failing test? | 15:11 |
| alecu | dobey: yes, I've just ran the tests after deleting .config/x, and all tests passed. | 15:11 |
| dobey | ah ok | 15:11 |
| alecu | dobey: are we going to use "from __future__ import unicode_literals, print_function" in a given module even though we don't use print there? | 15:13 |
| alecu | dobey: I think it's a good convention, so +1 to i. | 15:13 |
| alecu | it | 15:13 |
| dobey | alecu: i think we should, in case anyone adds a print it will complain even on python2 | 15:14 |
| dobey | it's good to keep the code from regressing in that syntax | 15:14 |
| alecu | yup | 15:15 |
| mmcc | gatox, mandel - darwin4-fsevents doesn't fix the tests, right? just checking if I should be expecting passes | 15:15 |
| dobey | mmcc: i guess according to jenkins, it doesn't | 15:16 |
| gatox | mmcc, all the tests inside filesystem_notifications | 15:16 |
| mandel | dobey, that is windows work that was broken by someone else. I have it in my radar to fix it asap | 15:17 |
| mandel | dobey, or pass the ball ;) | 15:17 |
| dobey | ok, well fix it :) | 15:17 |
| mmcc | alright. | 15:18 |
| mandel | dobey, yes yes I will but I need longer days hehe | 15:18 |
| dobey | heh | 15:18 |
| mmcc | dobey, regarding your comment about adding ubuntuone/utils.py - I'm glad you noticed. I was going to ask for specific feedback on that in the MP but forgot. I really didn't know where to put it, but I guess syncdaemon/utils.py would work, as long as there aren't any circular imports created... | 15:24 |
| dobey | well, the circular imports would exist regardless of where it lives, if there are any. that's a separate problem to where the module itself lives | 15:25 |
| alecu | dobey: in assert_utf8_bytes the value.decode(...) was changed to value.encode(...). I can't understand why that change makes sense. | 15:26 |
| dobey | alecu: python3 doesn't have decode on the data type that value is; only encode | 15:26 |
| mmcc | dobey, I meant because utils is used in platform/tools and platform/ipc, which are imported by syncdaemon, so if I move utils into syncdaemon, then platform <---imports--> syncdaemon, but i think it'll work out anyway | 15:27 |
| alecu | dobey: in assert_utf8_bytes, value should be a sequence of bytes, so it only has "decode", not encode. | 15:27 |
| dobey | mmcc: ah, but syncdaemon.utils itself doesn't import those, so it's not circular. also i think we should move all the other code to be under syncdaemon at some point as well | 15:28 |
| alecu | dobey: also unicode_path() has the same issue. | 15:29 |
| dobey | alecu: i think that was broken before then | 15:29 |
| mmcc | dobey: righto - it's ok as long as syncdaemon.__init__ doesn't import them. which it doesn't. and I agree, someone ought to do that. maybe when manuel gets his longer days or borrows eric's time machine | 15:30 |
| thisfred | MINE | 15:30 |
| mandel | this one I have to share because is just funny, the spanish government just defined funerals as 'entertainment' to increase the VAT on them.. hehehe | 15:32 |
| dobey | alecu: happy to fix it to be the other way around, but not sure exactly how to do that at the moment. | 15:33 |
| dobey | and i need to get lunch | 15:33 |
| alecu | mandel: that's a killer tax! | 15:34 |
| mandel | alecu, lol | 15:34 |
| mandel | dobey, which all other code? | 15:36 |
| dobey | killer party man! | 15:37 |
| dobey | mandel: everything under ubuntuone/ in ubuntuone-client, which is not already under ubuntuone/syncdaemon/ | 15:37 |
| dobey | mandel: don't worry about it though | 15:38 |
| mandel | dobey, you mean getting out of platform and moving it to syncdaemon? | 15:38 |
| mandel | dobey, it does make sense because it is a lot cleaner know, is a matter of getting time like mmcc said :) | 15:38 |
| mandel | dobey, I'm had that in my mind for some time | 15:38 |
| dobey | mandel: i mean, i really haven't thought about how to do it exactly yet, but it seems like everything should be under there, that is in ubuntuone-client | 15:38 |
| dobey | anywya | 15:39 |
| dobey | need to get lunch | 15:39 |
| dobey | bbiab :) | 15:39 |
| gatox | lunch for me! | 15:52 |
| === gatox is now known as gatox_lunch | ||
| mandel | gatox_lunch, ralsina ping! | 16:10 |
| ralsina | mandel: pong | 16:10 |
| mandel | ralsina, can you run the windows tests from trunk and let me know if all but one pass? | 16:11 |
| ralsina | mandel: not right now | 16:11 |
| ralsina | mandel: that was the case last week | 16:11 |
| mandel | ralsina, i have some timeout failures but it might well be that I have a slow machine.. | 16:11 |
| ralsina | mandel: jenkins seems to say that's still true | 16:11 |
| mandel | ok | 16:12 |
| mmcc | mandel, which windows tests? u1-client? | 16:13 |
| mandel | mmcc, yes | 16:14 |
| mmcc | mandel, I am about to run those, I'll let you know. it was the case yesterday - only one failure, having to do with unicode paths | 16:14 |
| mandel | mmcc, awesome, please let me know | 16:15 |
| mmcc | really would it have killed them to make 'cd E:\' work like it does everywhere else? | 16:17 |
| mandel | mmcc, probably the did it so support some old dos program.. | 16:18 |
| mmcc | well, I get the *reason* - it's that E: is a 'drive' not a 'directory', and they're really different things, but seriously. | 16:18 |
| mmcc | and the fact that the right way is just to type the drive name with no command? who came up with that | 16:19 |
| mmcc | </rant> | 16:19 |
| mmcc | and for some reason python.exe wasn't on my path anymore? winDOOOWSS! | 16:25 |
| * mmcc shakes fist | 16:25 | |
| briancurtin | ugh, 2 hours and only saw one apartment. at least it was a decent place | 16:27 |
| === gatox_lunch is now known as gatox | ||
| gatox | mandel, pong | 16:31 |
| mandel | gatox, I was going to ask you to run the windows tests but mmcc is on that :) | 16:31 |
| gatox | ok! | 16:31 |
| mandel | gatox, I have an other branch for review, is refactoring and some tiny changes but since there is no bzr mv --line 22-50 the diff looks bigger than the number of changes | 16:32 |
| mandel | gatox, is https://code.launchpad.net/~mandel/ubuntuone-client/unify-processors/+merge/114906 | 16:32 |
| gatox | mandel, ok..... i'll finish with the reviews no | 16:33 |
| gatox | now | 16:33 |
| mandel | gatox, haha I though you said no :P | 16:35 |
| mandel | gatox, I imagined you as joda hahaha | 16:35 |
| gatox | jaajajaj | 16:36 |
| gatox | LOL | 16:36 |
| gatox | mandel, also.... i need your review here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/darwin-tests-refactoring/+merge/113289 | 16:39 |
| mandel | gatox, we are the masters of the large reviews lol | 16:40 |
| dobey | hmm | 16:40 |
| gatox | mandel, jejeje it's a valuable skill jeje | 16:41 |
| mandel | gatox, so that branch, just changes the darwin tests, is that right? | 16:41 |
| gatox | mandel, no.... windows too...... move all the common code to common.py | 16:41 |
| gatox | doesn't add or remove anything.... just move things | 16:42 |
| mmcc | gatox, dobey reviews requested for this guy: https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/setup-mac-add-sd/+merge/114749 | 16:42 |
| gatox | well...... it removes repeated code, but you know what i mean | 16:42 |
| gatox | mmcc, ack | 16:42 |
| gatox | mmcc, let me know when you fix the other one | 16:42 |
| mmcc | gatox, ok, a couple minutes. made larger changes due to dobey's comments, testing on win/linux now | 16:43 |
| gatox | mmcc, ack | 16:43 |
| mandel | gatox, ack | 16:43 |
| mandel | gatox, it will have conflicts with mine.. but I'll fix those later :) | 16:44 |
| mmcc | does everyone just always set TRIAL_TEMP_DIR to something short on windows, or am I the only unlucky one with long paths to branches? should we just set it in env.bat? | 16:44 |
| gatox | mandel, yes, your turn to fix now! | 16:44 |
| mandel | mmcc, we all set trial_temp_dir, that was the reason we added it :) | 16:45 |
| mmcc | mandel, right but do you set it manually or what | 16:45 |
| mandel | mmcc, you can set it as env var for windows | 16:46 |
| mandel | mmcc, look for environment variable in the control center on windows 7 and will show you where to add it | 16:46 |
| mmcc | mandel, ok that's what I was looking for, thanks. | 16:47 |
| mandel | np | 16:47 |
| mandel | gatox, needs fixing, lint issue! | 16:52 |
| gatox | bloody hell | 16:52 |
| mandel | gatox, lol | 16:53 |
| dobey | heh | 16:55 |
| mandel | fu**, my income taxes just went up 140%.. hijos de la gran pu.. | 17:01 |
| * mandel looks for a shot-gun to go to the demonstration | 17:02 | |
| mmcc | mandel, ouch | 17:03 |
| mmcc | mandel, before you go to the revolution, I'm sure you'll want to know that u1client trunk still has just that one windows failing test. | 17:03 |
| mandel | mmcc, yes, in a single day.. also VAT went up 160% | 17:03 |
| mmcc | mandel, also http://paste.ubuntu.com/1090200/ | 17:03 |
| mandel | mmcc, awesome, so is my slow vm :) | 17:03 |
| dobey | ugh | 17:04 |
| mmcc | mandel, how slow? mine's awfully slow too - note the 617 seconds those tests take | 17:04 |
| mandel | mmcc, slower.. I need to see what is wrong with the setup | 17:04 |
| mmcc | mandel, press the turbo button on the front of your minitower | 17:05 |
| dobey | python standard lib is so… conflicted | 17:05 |
| mandel | hahs and the social security tax went up a 250%!!! | 17:07 |
| mandel | mmcc, lol I loved those turbo buttons :) | 17:07 |
| * ralsina suspects mandel needs a refresher in percentages | 17:11 | |
| ralsina | as in: going from 10 to 14 is not a 140% raise | 17:11 |
| mandel | ralsina, some things have gone in the VAT from 8% to 21% which is a 160% and all the other changes are the same (smaller diff) | 17:15 |
| ralsina | mandel: he, here's 21% for everything. Ecept phones, there's 37.5% | 17:16 |
| mandel | ralsina, then I'm move from a 18% to a 21% plus an extra 21% on top of that one + social secury has gone from around 100 eur to 350 | 17:16 |
| mandel | ralsina, but did they increase it in a single day? oh, while they have 30.000 million eurs to the banks... | 17:17 |
| mandel | s/have/gave | 17:17 |
| ralsina | see? plenty of money there! ;-) | 17:18 |
| dobey | welcome to america | 17:18 |
| dobey | only 350eur for social security? Man, our republican conservatives would love to live in Spain I bet! | 17:19 |
| mandel | dobey, well, most of madrid is on strikes round now blocking streets etc.. | 17:20 |
| mmcc | dobey, I bet they wouldn't: http://en.wikipedia.org/wiki/Spanish_National_Health_System | 17:20 |
| dobey | mandel: good thing i didn't take a holiday to madrid then | 17:21 |
| mandel | mmcc, yes, we are waaaay to the left | 17:22 |
| mandel | dobey, never know, might be fun, did you see the video of the miners? | 17:22 |
| ralsina | dobey: we need to package pyasn1 for python3 for Q sometime in the next few weeks, to upload to main | 17:23 |
| mmcc | ok folks, lunchtime for me, u1 client path branch tests are running now, I'll check in and ping for a re review in a few mintues | 17:23 |
| mmcc | or minweds, minthurs , you know, whenever | 17:23 |
| dobey | ralsina: huh? | 17:23 |
| ralsina | dobey: missing dependency for twisted pythn3 | 17:24 |
| dobey | oh, twisted.conch uses it apparently | 17:24 |
| ralsina | dobey: yep | 17:25 |
| ralsina | dobey: forget about it, barry will do it | 17:25 |
| dobey | ok | 17:25 |
| mmcc | uh, linux u1-client tests failed with this error I don't recognize, in filesystem_notifications: http://paste.ubuntu.com/1090236 anyone recognize? | 17:27 |
| * mmcc lunchtime for real | 17:27 | |
| === salgado is now known as salgado-lunch | ||
| dobey | os.path methods do not return bytes :( | 17:31 |
| ralsina | dobey: they should if they get bytes | 17:31 |
| gatox | dobey, python3? | 17:31 |
| dobey | gatox: both 2 and 3 | 17:40 |
| gatox | dobey, for python3 i don't know..... but for 2 it should be as ralsina says | 17:40 |
| ralsina | for python3 it's the same thing | 17:41 |
| ralsina | if you give it bytes you get bytes | 17:41 |
| ralsina | dobey: what specific function? | 17:41 |
| mandel | ok, eod for me | 17:51 |
| mandel | see you all on monday! | 17:51 |
| gatox | mandel, bye | 17:54 |
| gatox | mandel, also.... lint issue fixed here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/darwin-tests-refactoring/+merge/113289 | 17:54 |
| gatox | and +1 to your branch... the one with the 1500 lines diff | 17:54 |
| dobey | ralsina: some methods only accept strings (like os.path.join) | 17:57 |
| ralsina | dobey: nope | 17:57 |
| gatox | dobey, | 17:58 |
| gatox | >>> os.path.join(u'asd', 'foo') | 17:58 |
| gatox | u'asd/foo' | 17:58 |
| ralsina | dobey: https://pastebin.canonical.com/70055/ | 17:58 |
| gatox | ralsina, we are really creative with the examples :P | 17:58 |
| briancurtin | os.path.join(b"C:\\lol", "rofl".encode("utf8")) | 17:59 |
| ralsina | dobey, gatox: however, it's a bad idea to pass one unicode and one bytes because there will be an implicit str.decode(ascii) there somewhere | 17:59 |
| ralsina | briancurtin: "rofl".encode("utf8") is just evil :-) | 17:59 |
| ralsina | unless it's python 3 of course | 17:59 |
| briancurtin | but you can do it! | 17:59 |
| briancurtin | just to add to the examples | 18:00 |
| ralsina | you can also put ketchup on fries. Doesn't mean it's right ;-) | 18:00 |
| briancurtin | sriracha goes on fries | 18:00 |
| gatox | you have my attention | 18:00 |
| ralsina | +1 on sriracha on fries -1 on encoding bytes ;-) | 18:00 |
| gatox | what sriracha is? | 18:00 |
| ralsina | gatox: hot sauce | 18:00 |
| gatox | ahhh | 18:00 |
| briancurtin | it's like a hot ketchup. more of a paste-like thing than liquidy hot sauce | 18:01 |
| gatox | also...... cheddar (melt) with fries..... it's awesome! | 18:01 |
| dobey | 'r'.encode().decode().encode().encode().decode() | 18:01 |
| gatox | cheddar cheese | 18:01 |
| dobey | python2 is fun! | 18:01 |
| dobey | i guess dirspec should enforce bytes everywhere though | 18:01 |
| dobey | being a file path thing | 18:01 |
| dobey | alecu, ralsina, briancurtin: does that sound right? | 18:02 |
| === salgado-lunch is now known as salgado | ||
| alecu | dobey: "should enforce bytes" <- I agree for python 2. I'm still not sure it's the proper api for python 3 libraries, but since we'll be using it mostly from u1, bytes on python 3 sounds right. | 18:08 |
| ralsina | dobey, alecu: whatever, as long as it's consistent | 18:09 |
| dobey | alecu: well, filesystem paths are bytes. and it's basically impossible to enforce bytes on one version and strings on another. either we enforce unicode strings everywhere, or bytes everywhere | 18:11 |
| ralsina | on py3 where there's no automatic conversion, ad thus no risk of mixing them up, it doesn't matter as much | 18:11 |
| dobey | previously, the code was doing a mix, which is why this diff is a bit large | 18:11 |
| alecu | dobey: "filesystem paths are bytes" only on linux. | 18:11 |
| alecu | but yes, and bytes everywhere sounds good for our usecase. | 18:12 |
| dobey | alecu: they're bytes everywhere. on windows they're just a much smaller subset of possible bytes, and it's easier to get a reliable encoding for the bytes they're in :) | 18:12 |
| dobey | ok | 18:13 |
| dobey | i don't care either way, but we have to enforce either bytes or unicode strings, and not dilly dally somewhere in the middle :) | 18:14 |
| dobey | alecu: https://code.launchpad.net/~dobey/dirspec/python3/+merge/114713 is updated and changes stuff to be bytes everywhere | 18:18 |
| dobey | gatox: ^^ if you could please review that too | 18:20 |
| gatox | dobey, ack | 18:20 |
| * briancurtin lunch + doctor run | 18:31 | |
| ralsina | oh, lunch sounds like a cool concept | 18:36 |
| * ralsina investigates | 18:36 | |
| dobey | ralsina: given the temperature there, maybe you should eat a hot lunch | 18:37 |
| gatox | dobey, i'm getting this running the tests for dirspec-python3 http://paste.ubuntu.com/1090342/ | 18:40 |
| gatox | do you know what i'm missing? | 18:41 |
| gatox | i have python3-(testtools|setuptools) | 18:41 |
| mmcc | so did anyone recognize this: http://paste.ubuntu.com/1090236 test_zip_release_lock_compression_error failing on linux? | 18:43 |
| dobey | gatox: do you have python-testtools installed? that's python 2.7 that's failing there | 18:46 |
| dobey | mmcc: i've seen it a couple times now, but not sure why it happens | 18:47 |
| dobey | mmcc: i'm tempted to add a skip for that test and file a bug if it keeps happening (and a bug isn't already filed) | 18:47 |
| mmcc | oh, it's intermittent? I"ll rerun and see if I see it again... | 18:47 |
| gatox | dobey, i have it for python3 but not for 2 :P | 18:48 |
| dobey | mmcc: yeah; it might be system load related too, not sure | 18:49 |
| mmcc | dobey: interesting. that'd make sense. this was the first time I'd tried running the tests in both VMs at once on this poor old mac mini | 18:50 |
| dobey | ah | 18:50 |
| gatox | dobey, +1 https://code.launchpad.net/~dobey/dirspec/python3/+merge/114713 | 18:50 |
| dobey | gatox: thanks. though i just realized, i bet the tests are broken on mac/win now :) | 18:51 |
| dobey | actually, no. windows is still using trial | 18:52 |
| dobey | forgot about that | 18:52 |
| mmcc | ok, tests finally finished. dobey, gatox: can you revisit your reviews here: https://code.launchpad.net/~mikemc/ubuntuone-client/use-dirspec-get-cmdline/+merge/114720 | 18:59 |
| dobey | gatox, alecu: can you also run the python3 branch tests on osx and/or win now? i've now updated the run-tests and run-tests.bat | 19:00 |
| gatox | mmcc, i'm on that right now | 19:01 |
| gatox | dobey, ok.... but it might take a while... i need to install python3 and stuff con win/mac | 19:02 |
| dobey | gatox: no you don't need to install python3 there | 19:02 |
| gatox | ah just run the tests.... | 19:02 |
| dobey | gatox: we aren't using python3 on win/mac. i've updated the run-tests to only run the python3 tests when it's available now | 19:02 |
| dobey | gatox: but need to make sure the tests aren't broken on those platforms under python2 | 19:03 |
| dobey | gatox: and i'm not sure if the way i changed the run-tests.bat for windows is correct to run the setup.py build test clean | 19:03 |
| ralsina | python3 u1 for mac/win is a post ROFLing Rhino release thing | 19:03 |
| dobey | you might need to install setuptools/testtools though if they aren't already there | 19:04 |
| dobey | and i guess we will need to update the buildout to include them if they aren't being included already | 19:04 |
| gatox | dobey, yep.... i need to install that | 19:05 |
| mmcc | dobey or gatox, trivial merge: https://code.launchpad.net/~mikemc/ubuntuone-windows-installer/fix-1024518/+merge/114924 | 19:06 |
| gatox | mmcc, looking after dobey 's | 19:06 |
| dobey | mmcc: +1 | 19:08 |
| mmcc | thx | 19:08 |
| gatox | dobey, i have 3 failures on windows for your branch | 19:10 |
| gatox | dobey, should i change it to need fixing? | 19:10 |
| dobey | gatox: i guess so :P | 19:10 |
| gatox | mmcc, +1 and globally approve | 19:11 |
| gatox | dobey, i'll copy the trace there | 19:11 |
| gatox | dobey, there: https://code.launchpad.net/~dobey/dirspec/python3/+merge/114713 | 19:12 |
| alecu | kindertime! | 19:12 |
| dobey | gatox: ah, thanks. i wonder if these are broken in trunk already too :) | 19:13 |
| gatox | alecu, have fun playing with the other kids! :P | 19:14 |
| dobey | gatox: those tests should be fixed on windows now in r14 | 19:20 |
| gatox | dobey, ack.... checking | 19:21 |
| gatox | dobey everything is fine on windows, but i'm trying to run the tests on mac and it seems that is trying to use distutils instead of testtools.... so i'm taking a look at that | 19:38 |
| dobey | instead of setuptools? | 19:40 |
| dobey | hmm | 19:41 |
| gatox | when it does: python setup.py build test....... it can't recognize "test" command | 19:42 |
| dobey | hmm | 19:43 |
| dobey | sounds like you're missing setuptools | 19:43 |
| dobey | gatox: is it running system python instead of buildout python? or did you install setuptools into the wrong place? | 19:44 |
| gatox | i've setuptools installed in the system, and i was trying to run the tests with the system python....... should i use the buildout one? | 19:46 |
| dobey | i don't know. i presume you should use the buildout one | 19:47 |
| dobey | but i have to run right now | 19:47 |
| dobey | brb | 19:47 |
| gatox | dobey, yes, using the python from the buildout works, approving branch | 19:52 |
| ralsina | I think I just coughed a small piece of lung | 20:02 |
| gatox | :S | 20:02 |
| * ralsina calls the cat to perform a quick field analysis | 20:02 | |
| ralsina | I better go lay down for a couple of days. Mail me if you need me. | 20:03 |
| ralsina | no, not lung. \o/ | 20:03 |
| gatox | ralsina, roger that! get better! | 20:03 |
| ralsina | See you all on monday! | 20:03 |
| gatox | ralsina, bye | 20:04 |
| mmcc | bye ralsina, feel better | 20:04 |
| alecu | that sounds disgusting! get well, boss! | 20:05 |
| gatox | and EOD for me!! see you on monday people.... | 20:08 |
| * gatox goes to play with QML | 20:08 | |
| mmcc | oh gatox, did I miss you? you had a really minor needs-fixing on https://code.launchpad.net/~mikemc/ubuntuone-client/use-dirspec-get-cmdline/+merge/114720 that I've fixed… | 20:13 |
| mmcc | ah, nevermind - enjoy your weekend :) | 20:15 |
| gatox | mmcc, do you need it right now?? or can i do it later today or tomorrow morning? | 20:15 |
| mmcc | gatox, do it on monday, have a great weekend! | 20:15 |
| gatox | mmcc, ok then! see you | 20:15 |
| dobey | hrmm | 20:30 |
| dobey | need another review | 20:31 |
| dobey | mmcc: want to do a review? :) | 20:37 |
| mmcc | dobey, you bet your ass I do | 20:40 |
| mmcc | I mean, yes | 20:40 |
| dobey | mmcc: https://code.launchpad.net/~dobey/dirspec/python3/+merge/114713 | 20:40 |
| mmcc | dobey, +1. and do you have time to revisit your needs-info on https://code.launchpad.net/~mikemc/ubuntu-sso-client/use-dirspec-get-program-path/+merge/114690 ? I agreed and did a little constant cleanup. | 21:09 |
| mmcc | but I could use a second opinion on something I noticed while doing that. in ubuntu_sso/__init__.py, we add '.exe' to two of the four exe names we might launch... I'm not sure why it's only those two, and I think the changes I've made might now result in .exe.exe in those cases | 21:10 |
| dobey | hrmm | 21:11 |
| mmcc | I'm going to try to build the exe on windows and test it... | 21:11 |
| mmcc | is briancurtin still around? I'm wondering how to get debug logging set and where to look for logs when I run the packaged thing on windows | 21:12 |
| dobey | mmcc: i guess that probably needs to be removed, and the '.exe' needs to be added in the get_bin_cmd() wrapper? | 21:12 |
| mmcc | dobey, it's there in the wrapper already | 21:12 |
| mmcc | er, it's in dirspec I mean | 21:12 |
| dobey | mmcc: ah, then i'd remove it from the __init__.py there | 21:13 |
| dobey | it also looks weird and out of place in that location anyway | 21:13 |
| mmcc | _get_exe_path_frozen_win32 does it now | 21:13 |
| dobey | ah right | 21:13 |
| dobey | yeah, remove that from sso then | 21:13 |
| mmcc | wow so much rain this week. poor grass thinks it can start growing again | 21:18 |
| dobey | yeah, i need to cut my weeds | 21:19 |
| dobey | and i need to go now | 21:19 |
| dobey | mmcc: remove that + '.exe' bit from sso, and i'll re-review later | 21:19 |
| dobey | have a good weekend :) | 21:20 |
| briancurtin | mmcc: to get debug logging you set U1_DEBUG=True as an environment variable. at least for syncdaemon you need to additionally run it with the --debug flag | 21:21 |
| briancurtin | mmcc: i'll get the log path | 21:21 |
| briancurtin | mmcc: C:\Users\brian\AppData\Local\xdg\cache\ubuntuone\log is where my logs end up | 21:21 |
| mmcc | thanks briancurtin | 21:21 |
| mmcc | looks like I need to install intltool? briancurtin is that supposed to be in the buildout , or do I install it somewhere else/ | 21:36 |
| mmcc | ? | 21:36 |
| briancurtin | mmcc: you don't need that. i guess at some point we should remove whatever causes that error message to come up. ive never had it | 21:36 |
| mmcc | briancurtin: so you just ignore it and continue? | 21:48 |
| briancurtin | yep | 21:48 |
| mmcc | (sorry, stepped away for a minute to rescue a lawn ornament from the deluge. that's right, a metal rooster) | 21:48 |
| briancurtin | i forget where it even comes up, but yeah, not needed | 21:48 |
| mmcc | it tries to run intltool-update as part of setup.py prepare | 21:49 |
| briancurtin | ah yes | 21:49 |
| mmcc | unless you're talking about the rooster, which is for real absolutely needed. | 21:49 |
| mmcc | hrm, seems like it died in py2exe - http://paste.ubuntu.com/1090611 - I say seems because the error text is "The operation completed successfully" ?! | 21:53 |
| briancurtin | ah crap. i have no idea why that happens...i lost a lot of time to that, and i still dont get it. what i had to do was remove py2exe from the buildout setup, install py2exe for the system python in C:\Python27, then it worked | 21:54 |
| briancurtin | that didn't used to happen, and i have no explaination for why it started happening | 21:55 |
| mmcc | ok, I'll go do that. thanks. should I add a note about this to the wiki page, or somewhere else? | 21:56 |
| mmcc | also, should I get the latest from the website or install the same version as the egg in the buildout? | 21:59 |
| mmcc | nevermind, same version | 21:59 |
| briancurtin | mmcc: maybe add a note, but i'd like to actually figure it out at some point, before anyone else has to setup another environment | 22:03 |
| briancurtin | mmcc: there's also a patch you need to apply to py2exe if you're going to be building the binaries and potentially the installers. well, there's no patch, but i can tell you what exactly to change | 22:04 |
| mmcc | ok, shoot | 22:04 |
| briancurtin | in C:\Python27\Lib\site-packages\py2exe\boot_common.py | 22:04 |
| briancurtin | inside that sys.frozen block starting at ~line 45, remove whatever that big class is near the top and just make it look like this https://pastebin.canonical.com/70082/ | 22:05 |
| mmcc | ok, will do. | 22:06 |
| briancurtin | sys.stderr should be set to a Blackhole, which renders whatever that other class is irrelevant | 22:06 |
| mmcc | briancurtin: if you're still around, I've got built .exe's but should I be able to just run them from dist? I'm getting an importerror for pkg_resources trying to run control-panel | 22:34 |
| briancurtin | yeah you should just be able to run from dist. what's the exact ImportError you're getting? | 22:35 |
| mmcc | no module named pkg_resources | 22:35 |
| mmcc | I'm just running ubuntuone-control-panel-qt.exe with no args | 22:36 |
| briancurtin | hmm, that's a part of distribute | 22:37 |
| briancurtin | >>> pkg_resources | 22:37 |
| briancurtin | <module 'pkg_resources' from 'c:\Python27\lib\site-packages\distribute-0.6.25-py2.7.egg\pkg_resources.pyc'> | 22:37 |
| briancurtin | when you just open a python prompt and do the same, do you have pkg_resources, and if so where? | 22:37 |
| mmcc | briancurtin: http://paste.ubuntu.com/1090675 | 22:40 |
| mmcc | ah, shoot, I have to run for dinner - if that makes sense, let me know and I'll give another try later. have a great weekend! | 22:41 |
| briancurtin | mmcc: maybe try installing distribute for your site-wide python, then remove the egg path from bin/python-script.py so it uses the system one, then regenerate the binaries | 22:41 |
| briancurtin | i think its a similar thing to the py2exe problem | 22:41 |
| briancurtin | have a good weekend man | 22:41 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!