| dpm | hey all, good morning! | 08:13 |
|---|---|---|
| dpm | I bought a bunch of songs on the U1 store in Rhythmbox last night. Some of them were downloaded with a size of 0 bytes, and are obviously unplayable. Is there a way to retrigger the downloads? Or else, how can I fix this? | 08:13 |
| mandel | dpm, hm.. I've heard about a similar problem.. | 09:02 |
| dpm | mandel, do you know whether the downloads can be retriggered, or the best person I should talk to? | 09:04 |
| mandel | dpm, how big is you u1account? | 09:04 |
| mandel | dpm, one possible way, although is brute force, would be to stop syncdaemon, then remove the metadata and de 0byte files and then start it again | 09:05 |
| mandel | dpm, but wait and we can talk with rye | 09:05 |
| mandel | dpm, he is the persona that is good at this type of weird problems | 09:05 |
| dpm | mandel, thanks. Yeah, I've got about 30GB on my account, and I'm on a relatively slow connection, so I'd rather prefer avoiding re-syncing everything for just 4 or 5 songs :) | 09:07 |
| mandel | dpm, if you just delete the bad songs and not the rest it wont have to download it, it will check that you have files in the machine compare the hashes and not download them | 09:08 |
| mandel | dpm, but before that deleting the metadata | 09:08 |
| mandel | dpm, let me see if there is a more obvious way to do it.. | 09:08 |
| dpm | ok, thanks mandel! | 09:10 |
| mandel | dpm, no problem, I might take a little to get back to you, I just got back from holidays and I'm dealing with a crazy inbox :-/ | 09:11 |
| dpm | mandel, no worries, did you have a good holiday? | 09:13 |
| mandel | dpm, yes, going to mallorca is always great :) | 09:14 |
| dpm | ah, back home, right? Nice :) | 09:14 |
| mandel | dpm, yes, it was great, maybe too short hehe | 09:14 |
| dpm | :) | 09:14 |
| rye | dpm: i joined a bit late.. | 09:15 |
| dpm | rye, no worries, there's no rush | 09:15 |
| rye | dpm: so, can you PM me your e-mail so that I look your account up and ask the vendor to process the refund for these broken songs | 09:16 |
| dpm | rye, sure, but is there no way to re-download the songs? If possible, I'd prefer that over a refund | 09:16 |
| brendand | anyone know why the files client is crashing on iOS 5? | 09:24 |
| mandel | brendand, the developers that know about the ios app are in the usa, you will have to wait a little longer to get their help | 09:43 |
| === __chip__ is now known as Chipaca | ||
| === yofel_ is now known as yofel | ||
| gatox | good morning! | 11:14 |
| mandel | gatox, morning! | 11:14 |
| gatox | mandel, hi! | 11:14 |
| mandel | gatox, have I missed anything interesting these days? | 11:14 |
| gatox | mandel, yes, you were fired..... nobody told you? | 11:15 |
| gatox | jeje | 11:15 |
| gatox | mandel, mmmmm let me think | 11:16 |
| mandel | gatox, I still get the email.. so I guess I still have the bad part of the work ;) | 11:16 |
| gatox | mandel, things are moving into the python3 direction..... and we have feature freeze on the 24th | 11:16 |
| mandel | gatox, ok, and mac, any news? | 11:17 |
| gatox | mandel, mmm nothing new as far as i remember | 11:18 |
| gatox | mandel, i have the feeling that you want to do a review :P please: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-status-actions/+merge/119170 | 11:30 |
| mandel | gatox, sure! | 11:30 |
| gatox | alecu, PING.... (when you are here please!!) | 11:30 |
| mandel | gatox, in line 103 of the diff, what happens if the backed does not have the backend methods, should you be adding None as the default result to avoid the attr error? | 11:49 |
| gatox | mandel, later i do: if self._backend_method is not None: | 11:50 |
| gatox | mandel, or what do you mean? | 11:52 |
| mandel | gatox, no, that is not the problem, if you do self._backend_method = getattr(self.backend, data['backend_method']) and data['backend_method'] is 'paquito' and there is not method no attr named paquito you will get a AttributeError | 11:52 |
| gatox | ah yes | 11:52 |
| mandel | gatox, you should do getattr(self.backend, data['backend_method'], None) so that if the method is not there you get None :) | 11:52 |
| gatox | ahhhhh roger that | 11:52 |
| mandel | gatox, adding it to a comment so that we have history of the review :) | 11:53 |
| gatox | mandel, fixed | 11:55 |
| mandel | ack | 11:56 |
| mandel | gatox, when running the tests, do you get lots of tray icons? | 11:57 |
| gatox | nop | 11:57 |
| gatox | linux, windows? | 11:58 |
| gatox | ahhhhh yes | 11:58 |
| gatox | didn't notice that before | 11:58 |
| gatox | but they get close at the end | 11:59 |
| mandel | gatox, yes, is no a major pita | 11:59 |
| mandel | gatox, probably means that the code that does the x11 buffering does not do it for the tray icon | 11:59 |
| mandel | gatox, also I added a comment, I'm getting a lint issue in P | 11:59 |
| gatox | mandel, no....... the lint issue is because you didnn't read the description :P | 12:00 |
| mandel | gatox, uh, there is code in u1client too.. ok | 12:01 |
| mandel | gatox, other question, what is happening here: http://paste.ubuntu.com/1144725/ | 12:05 |
| mandel | gatox, icon is never going to be none unless QtGui.QIcon() returns None, is that possible? | 12:05 |
| gatox | ahh right.... that is something that ended that way because of some previous code i was doing | 12:07 |
| gatox | the right way is: if icon_name is not None | 12:08 |
| gatox | mandel, pushed | 12:08 |
| mandel | gatox, awesome, one last question, is you do QLabel.setText(None) what happens, it works ok, right? | 12:09 |
| gatox | yes | 12:10 |
| mandel | ok | 12:11 |
| gatox | because it takes the none as the parent arg | 12:11 |
| gatox | but we are assigning the parent later always anyway | 12:11 |
| mandel | ok | 12:13 |
| mandel | approving then | 12:13 |
| mandel | gatox, I'm off to go for lunch, will take a little longer since I have not food at home after the holidays.. | 12:33 |
| gatox | mandel, hehe ok | 12:33 |
| gatox | mandel, don't go to eat sushi please :P | 12:33 |
| ralsina_ | good morning! | 12:44 |
| gatox | ralsina_, hi | 12:46 |
| thisfred | ralsina_, buenos! I have added the itemClicked handler and pushed, that works great | 12:53 |
| ralsina_ | thisfred: cool, I'll take a look ASAP | 12:54 |
| thisfred | thx! | 12:54 |
| ralsina_ | thisfred: +1 | 13:04 |
| thisfred | gracias | 13:04 |
| dobey | hmm | 13:09 |
| gatox | ralsina_, already file the swap from october 8 to december 28 correctly... | 13:11 |
| ralsina_ | gatox: awesome | 13:12 |
| === m_conley_away is now known as m_conley | ||
| * mandel back | 13:23 | |
| mandel | ralsina_, 1-1 when ever you please, AFAIK we have it today :) | 13:24 |
| mandel | dobey, may I have a re-review for: https://code.launchpad.net/~mandel/ubuntuone-fsevents-daemon/long-domain-socket/+merge/116269 | 13:24 |
| dobey | mandel: you didn't fix my issue :) | 13:27 |
| dobey | oh | 13:27 |
| mandel | dobey, there is not just constant present in darwin | 13:28 |
| dobey | i meant the path == NULL | 13:28 |
| dobey | overlooked that comment, sorry :) | 13:28 |
| mandel | dobey, no problem, I do that all the time :) | 13:28 |
| mandel | dobey, I'm not the one that should ever complain about not reading comments or typos :P | 13:28 |
| dobey | heh | 13:30 |
| dobey | anyway, gave it +1 now | 13:30 |
| mandel | thx! | 13:30 |
| mandel | alecu, gatox, ralsina_, briancurtin, you guys noticed that the windows port is broken on jenkins as from friday? | 13:31 |
| gatox | mandel, nop | 13:32 |
| ralsina_ | mandel: noticed very late on friday | 13:32 |
| mandel | :( | 13:32 |
| mandel | I wonder what happened.. | 13:32 |
| brendand | has anyone had problems with the iOS files application recently? | 13:42 |
| brendand | for me i open it and it crashes after a few seconds | 13:42 |
| mandel | urbanape, ^ | 13:43 |
| mandel | brendand, urbanape can give you a hand with it | 13:44 |
| urbanape | brendand: hi, here to help | 13:44 |
| urbanape | Do you have the latest version from the App Store? | 13:45 |
| brendand | urbanape, yep. first thing i did was to uninstall and reinstall | 13:45 |
| dobey | doh; have to remember to call and arrange shipping for this thing today | 13:47 |
| urbanape | brendand: do you have a Mac or Windows machine running iTunes? We have yet to add some in-app diagnostics so we can pull crash logs directly. | 13:48 |
| brendand | urbanape, yeah give me a few minutes. i'll ping back when i'm ready | 13:48 |
| urbanape | thanks | 13:48 |
| alecu | hello, all! | 13:49 |
| dobey | holas alecu | 13:51 |
| gatox | alecu, hi!!!! i was waiting for you!!!! | 13:51 |
| mandel | alecu, buenas! | 13:51 |
| gatox | alecu, could you please re-review this branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ipcmenu | 13:51 |
| gatox | alecu, this one is avoiding my u1-cp branches to land cause of lint issues | 13:52 |
| gatox | that depends on that branch | 13:52 |
| ralsina_ | alecu: you missed a call. Don't worry too much though. | 13:52 |
| alecu | ralsina_: ouch. I had to run some errands downtown, and it took a loong while to get there and back. :-( | 13:53 |
| alecu | ralsina_: sorry all | 13:53 |
| ralsina_ | alecu: like I said, no problem, really | 13:54 |
| === ralsina_ is now known as ralsina | ||
| alecu | gatox: small needs fixing | 14:13 |
| gatox | alecu, ack | 14:13 |
| brendand | urbanape, i'm ready to go. sorry it took a while | 14:41 |
| ralsina | alecu: so, bad news about the windows build testing from elopio, it looks like we still have some bits that don't use the proxy | 14:43 |
| urbanape | brendand: that's okay. Do you normally sync your phone to this iTunes? | 14:43 |
| alecu | ralsina: ugh | 14:43 |
| ralsina | alecu: my exact guttural noise | 14:44 |
| alecu | ralsina: is there a mail about that? | 14:44 |
| ralsina | alecu: yes, you are probably not on it, let me forward | 14:44 |
| ralsina | alecu: sent | 14:45 |
| alecu | thanks | 14:45 |
| brendand | urbanape, yeah | 14:46 |
| urbanape | okay, if you have time, go ahead and sync it with iTunes now (unless it did automatically when you connected it) | 14:46 |
| ralsina | alecu, dobey, gatox, mmcc, thisfred, briancurtin, mandel: standup in 7' | 14:54 |
| mandel | ack | 14:54 |
| gatox | ack | 14:54 |
| thisfred | yak | 14:54 |
| briancurtin | i'll be there | 14:54 |
| gatox | me | 15:00 |
| dobey | me | 15:00 |
| thisfred | me | 15:00 |
| briancurtin | me | 15:00 |
| ralsina | me | 15:01 |
| dobey | mandel, alecu: | 15:02 |
| mandel | me | 15:02 |
| alecu | me | 15:02 |
| mandel | mmcc seems not to be here | 15:02 |
| dobey | ralsina: you mentioned mmcc, but he's not here (on holiday today) | 15:02 |
| dobey | mandel: right, he's on holiday today | 15:02 |
| dobey | gatox: go? | 15:03 |
| gatox | DONE: | 15:03 |
| gatox | Fixed u1-client menu branch. Propose a couple of branches for u1-cp menu feature. Finishing another one. | 15:03 |
| gatox | TODO: | 15:03 |
| gatox | Finish with u1-cp menu branches. Propose a branch for macfsevents in github. | 15:03 |
| gatox | BLOCKED: | 15:03 |
| gatox | No | 15:03 |
| gatox | dobey, go | 15:03 |
| dobey | DONE: reviews, proposed u1db in universe | 15:03 |
| dobey | TODO: poke about u1db package, rush to get everything done for feature freeze, reorg to drop ubuntuone-installer for good, SRUs for lucid/natty/oneiric | 15:03 |
| dobey | BLCK: None. | 15:03 |
| dobey | thisfred: go | 15:03 |
| ralsina | forgot about mmcc's day off | 15:03 |
| ralsina | THISFRED! ;-) | 15:04 |
| thisfred | DONE: conflict resolution in cosas TODO: u1db documentation. Lots of it. BLOCKED: no NEXT: dobey | 15:04 |
| briancurtin | DONE: holiday | 15:04 |
| briancurtin | TODO: catch up with a few comments on thursday's last Python 3 merge proposals, start actually running SSO on Python 3 | 15:04 |
| briancurtin | NEXT: ralsina | 15:04 |
| ralsina | DONE: 1-1s, canonicaladmin, reviews, helped around, some bug triaging, extra dash call, misc. TODO: tech leads, reviews, fix a bug hopefully BLOCKED: no NEXT mandel | 15:04 |
| mandel | DONE: Holidays. Catched up with email after days off. Reviews. Upgraded to montain lion. Cry because Xocde 4.4 set all the project targets to 10.8 (stupid apple!!) | 15:05 |
| mandel | TODO: Fix the state of the daemon project after update. Go after people for reviews. 1-1 with ralsina. Is my review day. | 15:05 |
| mandel | BLOCKED: no (although considering no to use getoptlong for testability reasons). | 15:05 |
| mandel | alecu, please | 15:05 |
| alecu | DONE: small vacation at Gualeguaychú | 15:05 |
| alecu | TODO: catch up with mail, help briancurtin get sso running on py3 | 15:05 |
| alecu | BLOCKED: no | 15:05 |
| mandel | ralsina, 1-1 now before you have other meetings? | 15:05 |
| ralsina | alecu: tech leads is now | 15:05 |
| ralsina | mandel: I am busy for about 1 hour | 15:06 |
| mandel | ralsina, ping me when done I should still be here :) | 15:06 |
| ralsina | mandel: thanks | 15:06 |
| mandel | ralsina, no problem :) | 15:06 |
| brendand | urbanape, it's all synced | 15:13 |
| urbanape | okay, is this Mac or Windows? | 15:13 |
| briancurtin | alecu: for that py3-callable branch, you're totally right. as for the branch, should i reject the MP and then set the branch status to abandoned...or just delete the branch all together? | 15:14 |
| brendand | urbanape, to be exact a windows xp vm | 15:14 |
| urbanape | brendand: You can use this document to find the pertinent crash logs: http://aplus.rs/apple/how-to-find-crash-logs-for-iphone-applications-on-mac-vista-and-xp/ | 15:14 |
| urbanape | if you can zip those up and send them to me (zachery.bir@canonical.com), I'll take a look, thanks! | 15:15 |
| alecu | briancurtin: rejected+abandoned for this sounds right, since it serves as a reminder of what bits we should be changing (or not) next cycle when we port the other u1 projects. | 15:23 |
| briancurtin | cool, done | 15:24 |
| gatox | alecu, this branch is fixed: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ipcmenu/+merge/118621 | 15:25 |
| gatox | alecu, and i also have 2 more branches for review..... i don't know if you want to review them, or do you prefer that i ask someone else... | 15:25 |
| alecu | gatox: I'm otp right now... you don't mind if I review them later, right? | 15:26 |
| alecu | gatox: try forcing mandel to do them, since he's back from vacations, and he loves doing reviews! | 15:26 |
| gatox | alecu, no..... if you want i can ask someone else for the control panel branches... the important one for me is the u1-client that is blocking the others | 15:27 |
| ralsina | alecu, gatox: and it's his review day ;-) | 15:27 |
| mandel | and is my review day.. although I don't love them alecu | 15:27 |
| gatox | ralsina, if you have a moment (mandel already review that one): https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-status-actions/+merge/119170 | 15:27 |
| alecu | oh, this reminds me I should find some way to send money to mandel, so he can get me the "game of thrones board game" when he comes for pycon!!!!! | 15:27 |
| ralsina | gatox: not immediately, but in a while | 15:27 |
| mandel | alecu, haha I need to ask for a place to stay :P | 15:27 |
| gatox | ralsina, yes, no problem! thankx | 15:27 |
| alecu | mandel: but we love you! | 15:27 |
| gatox | thanks | 15:28 |
| urbanape | brendand: received. I'll let you know | 15:28 |
| mandel | gatox, any review do send them my way :) | 15:28 |
| ralsina | mandel: I would offer you my couch, but I live right across the universe from pycon | 15:28 |
| alecu | mandel: I have rented the flat with cold water you used last time... I'll try to find some other place, though! | 15:28 |
| urbanape | brendand: you said it crashes immediately on launch? | 15:28 |
| gatox | mandel, i have this one: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-desktop-services-actions/+merge/119370 but i was expecting for launchpad to end updating the diff to review it myself first | 15:28 |
| brendand | urbanape, no | 15:29 |
| mandel | ralsina, hehe worst case scenario I can take the sofa :) | 15:29 |
| urbanape | pauses for about 10 seconds and then crashes? | 15:29 |
| mandel | alecu, don't worry too much hehe | 15:29 |
| ralsina | mandel: mi casa es tu casa | 15:29 |
| dobey | mandel: i hope he has insurance | 15:29 |
| brendand | urbanape, yes | 15:29 |
| gatox | mandel, you can stay here..... but you will need to take a flight everyday :P | 15:29 |
| mandel | dobey, I'm not that badly behaved! | 15:30 |
| mandel | gatox, thx for the offer.. but I pass :P | 15:30 |
| urbanape | brendand: I'm assuming you enabled auto-upload, yeah? | 15:30 |
| gatox | mandel, jeje | 15:30 |
| brendand | urbanape, yes | 15:30 |
| urbanape | it's probably consuming too much time trying to walk your camera roll (guessing you have over 1,000 items in the camera roll?) | 15:30 |
| brendand | 520 | 15:31 |
| urbanape | what device? | 15:31 |
| brendand | 4s | 15:31 |
| urbanape | that's really weird. | 15:31 |
| urbanape | 520 is nothing. I have 5,000 items in my camera roll, and don't get the 10s crash. Lemme think on it. | 15:32 |
| ralsina | mandel: there's a bunch of cheap places in airbnb | 15:32 |
| mandel | ralsina, it should not be a huge problem, is a matter of looking :) | 15:36 |
| dobey | alecu: for bug #1032336 i'm wondering how much of the code i need to write tests for. for example, having tests for the OptionParser class, I wonder if the derived classes which simply define options to use, actually need a test for exery single option? and i'm kind of curious as to what the best way to test the test runners whil running the tests with the test runners, would be exactly :) | 15:45 |
| ubot5` | Launchpad bug 1032336 in ubuntuone-dev-tools "New options parsing needs more test coverage" [Undecided,Triaged] https://launchpad.net/bugs/1032336 | 15:45 |
| gatox | mandel, just in case, i've made some changes in the docstrings of this brnach: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-desktop-services-actions/+merge/119370 | 15:45 |
| gatox | now is up to date | 15:45 |
| mandel | ok | 15:46 |
| * gatox lunch/ | 15:46 | |
| === gatox is now known as gatox_lunch | ||
| mandel | gatox_lunch, how do you feel about this: http://paste.ubuntu.com/1145069/ | 15:58 |
| mandel | I'm just very lazy | 15:58 |
| gatox_lunch | mandel, mmmm i can be wrong... but for me... that adds more confusion than clarity to the code | 16:01 |
| mandel | gatox_lunch, I prefer it, but is a personal thing, I don't like lots of line that are exactly the same is more error prone for me | 16:05 |
| mandel | gatox_lunch, with the loop way I know I have to make sure the couples are right, in the other I have to read all the call to do it, but again, is a personal thing | 16:06 |
| ralsina | mandel: mumble? | 16:11 |
| mandel | ralsina, sure | 16:12 |
| rye | mandel: bug #1036270 goes to?.. | 16:12 |
| ubot5` | Launchpad bug 1036270 in Ubuntu Single Sign On Client "[nightlies] SSL errors crash ubuntu-sso-client" [Undecided,New] https://launchpad.net/bugs/1036270 | 16:12 |
| mandel | rye, no idea, desktop+ and we will sort it out later | 16:13 |
| dobey | ok, i'm off to get lunch. bbiab | 16:29 |
| mandel | gatox_lunch, review done, some needs fixing there | 16:30 |
| ralsina | gatox_lunch: +1 on menu-status-actions | 16:30 |
| mandel | gatox_lunch, looks like there is some code duplication, because I already mentioned those errors, right? | 16:31 |
| czajkowski | mandel: boo :) | 16:32 |
| mandel | czajkowski, lol | 16:32 |
| mandel | gatox_lunch, if it is the same code I think a base class sounds like a logical thing to do | 16:40 |
| ralsina | lunch time for me. | 16:51 |
| === gatox_lunch is now known as gatox | ||
| gatox | mandel, checking.... | 16:56 |
| gatox | mandel, it wasn't code duplication..... i forgot to merge with that branch after the changes | 17:06 |
| gatox | done | 17:06 |
| gatox | checking the other comments | 17:06 |
| mandel | gatox, the removal of the inlineCallbacks makes sense, right? | 17:08 |
| gatox | mandel, i thought that i remove that tests.... (i need to learn bzr pipeline)..... that test shouldn't be there | 17:10 |
| gatox | is not doing anything useful | 17:11 |
| mandel | gatox, is that an older pipe? | 17:11 |
| chaselivingston | Hi folks, just wanted to introduce myself. I'm the new support person for U1, getting acclimated and introduced to the community as much as I can | 17:11 |
| mandel | gatox, did you do a pump so that changes from earlier branches are push to later ones? | 17:11 |
| gatox | mandel, the problem is that is not a pipe..... i'm doing independent branches | 17:11 |
| mandel | gatox, ah, then I cannot help you :) | 17:12 |
| mandel | chaselivingston, hola, bienvenido! (that is welcome to the land of the spanish speakers :P ) | 17:12 |
| chaselivingston | mandel: Thanks! | 17:12 |
| gatox | mandel, ok.... everything is fixed now..... except the "personal thing".... maybe we can ask alecu for his opinion.... but i personally prefer to leave it as it is... i think is more clear and easier to check looking at the code that everything is properly connected, etc | 17:13 |
| mandel | gatox, so we need a third vote :) | 17:15 |
| gatox | mandel, or maybe ralsina if alecu is busy | 17:15 |
| gatox | i don't want to get block for that | 17:15 |
| gatox | the rest of the changes are already pushed | 17:16 |
| mandel | gatox, or briancurtin2 I'm sure he writes more pythonic code than we do :) | 17:16 |
| gatox | mandel, yes, but i was thinking from the ui eye too | 17:16 |
| gatox | to see the code and identify stuff | 17:16 |
| mandel | briancurtin2, can you look at the last part of my needs fixing: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-desktop-services-actions/+merge/119370 and vote :) | 17:16 |
| briancurtin2 | mandel: sure | 17:17 |
| mandel | gatox, well, the problem I have with that is since I'm a little dislexic I wont see an error at all when there are some many char repeated in the lines :P | 17:17 |
| alecu | gatox: what is the "personal thing" you mentioned? I'm lacking context! | 17:26 |
| alecu | chaselivingston: hey, welcome to the team! | 17:27 |
| gatox | alecu, here: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-desktop-services-actions/+merge/119370 | 17:27 |
| mandel | alecu, last comment from me here: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-desktop-services-actions/+merge/119370 | 17:27 |
| mandel | alecu, I prefer the loop because my eyes go nuts when I see that much text and I assume that everything is ok | 17:28 |
| gatox | alecu, sorry for being annoying..... but when you have a moment, please review this branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/ipcmenu/+merge/118621 is blocking me 3 branches | 17:31 |
| alecu | gatox, mandel: Both ways look readable. I like the first one more, because I like "data as code"; but it's more verbose, so I really don't have an opinion. | 17:31 |
| alecu | gatox: yes, I reviewed that other branch... three hours ago! I'll get to re-review it soon, have some patience!!!! :-) | 17:32 |
| mandel | alecu, gatox I won't block for that, so +1. did you check why is there duplicated code gatox ? | 17:32 |
| gatox | alecu, jeje ok, sorry | 17:32 |
| gatox | mandel, it wasn't duplicated..... i forgot to merge with the previous branch..... already done | 17:32 |
| mandel | gatox, ok :) | 17:33 |
| mandel | then, EOD here, is to hot and madrid is empty | 17:33 |
| gatox | mandel, ok..... bye mandel! | 17:33 |
| mandel | ralsina, have fun with the spaniard, he seems weak :P | 17:33 |
| === briancurtin2 is now known as briancurtin | ||
| chaselivingston | alecu: Thanks! | 17:42 |
| dobey | alecu: hey. did you see my question earlier re: dev-tools tests? | 17:52 |
| alecu | dobey: oh, I'm looking now, sorry. | 17:54 |
| dobey | alecu: no worries. thanks | 17:55 |
| * briancurtin lunch | 18:01 | |
| ralsina | alecu SMSd me that his internet is down | 18:23 |
| ralsina | unless he's back of course ;-) | 18:23 |
| dobey | i guess that's why he hasn't said anything else to me :P | 18:24 |
| dobey | hrmm | 18:33 |
| ralsina | briancurtin: I am really undecided about whether we should release 3.0.2b or not | 18:37 |
| ralsina | briancurtin: it doesn't fix syncing but it does make u1cp work behind a proxy :-/ | 18:37 |
| briancurtin | ralsina: i don't know nearly enough about proxies to know if that's good enough for what people want out of it. sounds like it's probably not good | 18:38 |
| ralsina | briancurtin: yep, and it triggers a big download for everyone | 18:38 |
| ralsina | briancurtin: please kill the RT :-( | 18:38 |
| ralsina | joshuahoover: we are not going to release 3.0.2b will try to do a real bugfix and do a real release then | 18:39 |
| joshuahoover | ralsina: ok, can you update the rt ticket then? | 18:39 |
| briancurtin | i'm commenting now | 18:40 |
| ralsina | joshuahoover: asked briancurtin to do it | 18:40 |
| dobey | briancurtin, ralsina: what are the remaining issues that we need to get in for doing a 3.0.3 release? next monday is 3.99.x release day btw, as feature freeze is next week | 18:40 |
| joshuahoover | ralsina: does he have someone he get ask to do it so we can get a giant chain of "delegation" going? ;) | 18:40 |
| ralsina | joshuahoover: I think he has a dog :-) | 18:40 |
| joshuahoover | ralsina: perfect! | 18:41 |
| ralsina | dobey, joshuahoover, gatox: we are going to need a FFE for the "publish files" tab. | 18:41 |
| ralsina | and perhaps one for the sync menu stuff in some form or another | 18:42 |
| dobey | we have a publish files tab? | 18:42 |
| ralsina | dobey: we have a design for one, and it's easy-ish to do | 18:42 |
| dobey | sweet | 18:42 |
| ralsina | dobey: and solves the classic "what files am I sharing?" issue | 18:42 |
| dobey | then we can just remove the nautilus extension | 18:42 |
| ralsina | dobey: that's the secret plan | 18:42 |
| dobey | and nautilus | 18:42 |
| ralsina | dobey: tell noone | 18:43 |
| dobey | ralsina: i've been trying to get rid of the file 'manager' for years :) | 18:43 |
| ralsina | dobey: I like the new ideas for Nautilus in Gnome 3 quite a bit | 18:44 |
| ralsina | dobey: specially concepts like "make it useful by default" | 18:44 |
| dobey | also, i am sad that there has been no time to deal with that stuff :-/ | 18:44 |
| ralsina | yeah | 18:44 |
| ralsina | but then again, we declared it a rewrite candidate for two cycles already :-( | 18:45 |
| dobey | ralsina: eh, i think maybe you misunderstand the designs for 'files' in gnome 3. 'files' is not nautilus, though it might start out based on the nautilus code | 18:45 |
| ralsina | dobey: trust you on that | 18:45 |
| dobey | yeah; and i'm quickly wanting to move it to 'DIAF candidate' | 18:45 |
| ralsina | dobey: anyway, maybe we can try to set aside some time for that rewrite *early* in next cycle, but considering the fluidity of the nautilus-in-ubuntu situation... | 18:46 |
| dobey | well, we need to rewrite the libsyncdaemon part, even if we just end up dropping the nautilus extension. we *do* need some stuff for music store and other things | 18:47 |
| ralsina | yes | 18:47 |
| dobey | maybe i should try to move the assets today | 18:48 |
| alecu | hello, I'm back. | 19:45 |
| alecu | dobey: sorry I didn't reply; | 19:45 |
| dobey | alecu: welcome back to the land of the living! | 19:46 |
| dobey | or... zombie! | 19:46 |
| alecu | dobey: so, I see your point now. Most of the code that would make a lot of sense to be tested was in TestRunner and got moved to BaseTestRunner | 19:46 |
| alecu | lol | 19:47 |
| alecu | dobey: did you end up watching some of the "Plaga Zombie" films? :P | 19:47 |
| dobey | i haven't actually watched it yet. was having trouble with totem/handbrake picking the right language | 19:48 |
| dobey | it kept wanting to play the commentary track for some reason :( | 19:48 |
| alecu | dobey: so, most of the code that could use some tests was just moved (say, _collect_tests, or get_suite). | 19:50 |
| alecu | dobey: I think we should retitle the bug to affect those functions needing tests, while lowering the bug priority, since it's code that afaik works fine. | 19:52 |
| dobey | alecu: well, i do agree the OptionsParser code needs tested. and i have tests for that in a branch | 19:55 |
| alecu | dobey: awesome then. Should I invalidate this bug then? | 19:56 |
| dobey | it is new code, and i've already found/fixed one issue in it last week | 19:56 |
| alecu | gatox: approved. | 19:56 |
| dobey | alecu: well, this branch fixes that bug, so i don't think so | 19:56 |
| gatox | alecu, awesome!!!! | 19:56 |
| * gatox starts landing branches | 19:56 | |
| gatox | briancurtin, do you have time for a review? | 19:57 |
| briancurtin | gatox: sure, where at? | 19:57 |
| dobey | alecu: should i just propose that, and not try to write a complex series of tunnels to test the old code that we've been using for years any more than we are testing it by actually running thet tests? | 19:57 |
| gatox | briancurtin, thx..... here: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/menu-desktop-services-actions/+merge/119370 | 19:57 |
| gatox | eod here!! see you tomorrow people! | 20:13 |
| dobey | alecu: lose the internets again? :) | 20:14 |
| alecu | @ping | 20:25 |
| briancurtin | pong | 20:25 |
| alecu | dobey: yes, my isp is acting strangely, yet again. | 20:25 |
| alecu | briancurtin: @thanks! | 20:25 |
| alecu | dobey: again: "test by running" is a non-starter to me :-) | 20:26 |
| alecu | dobey: I agree we have more interesting bits to build than tests for existing code, so yes: propose what you have and don't worry about complex series of twisty passages. | 20:28 |
| dobey | ok, yeah | 21:05 |
| dobey | trying to load a test suite, within a test, is not working out so well :( | 21:05 |
| dobey | alecu, ralsina: if you could, https://code.launchpad.net/~dobey/ubuntuone-dev-tools/runner-tests/+merge/119436 | 21:10 |
| ralsina | dobey: looking | 21:10 |
| ralsina | dobey: I'll have it tomorrow, I am EODing very soon | 21:11 |
| === m_conley is now known as m_conley_away | ||
| dobey | alright, i'm off as well. later all | 21:56 |
| === zyga is now known as zyga-afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!