| === urbanape_ is now known as urbanape | ||
| JamesTait | Good morning all! | 08:06 |
|---|---|---|
| mandel | morning all! | 08:56 |
| mandel | fagan, buenos dias! | 08:56 |
| === mpt_ is now known as mpt | ||
| karni | Good morning! | 10:24 |
| gatox | mandel, hi | 10:29 |
| mandel | gatox, hello! | 10:32 |
| mandel | gatox, did you get my number? | 10:32 |
| gatox | mandel, hi!! you send me a ping | 10:32 |
| gatox | mandel, yes yes THANKS! | 10:32 |
| mandel | gatox, it was just for that ;) | 10:32 |
| gatox | mandel, jeje ok!! thanks!!! you will be receiving an awesome phone now! :P | 10:33 |
| mandel | gatox, cool, then if they ever give you a tracking number let me know :) | 10:33 |
| facundobatista | Holaaaa | 10:33 |
| gatox | mandel, yes | 10:33 |
| gatox | facundobatista, buenas | 10:33 |
| duanedesign | morning all | 10:35 |
| mandel | duanedesign, morning! | 10:44 |
| gatox | hi nessita! | 11:04 |
| nessita | hello everyone! | 11:08 |
| gatox | nessita, let me know when you are free..... i have a branch already approved by ralsina | 11:09 |
| gatox | pending for review | 11:09 |
| nessita | gatox: shoot! | 11:11 |
| gatox | nessita, https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/uninstall/+merge/72402 | 11:12 |
| facundobatista | Hola nessita | 11:14 |
| nessita | hola facundobatista | 11:14 |
| nessita | gatox: inside the windows implementation of uninstall, is it needed to ask if sys.platform == "win32" and ...? | 11:16 |
| gatox | nessita, sorry win32 part shouldn't be there | 11:17 |
| nessita | gatox: also, I've noticed you tend to write a long lines of code in multiple lines, instead of defining variables. That approach is not wrong at all, but usually makes code a bit harder to read. For example, lines 343-346 in your diff are very hard to parse for new eyes. Examples of how to improve that are http://pastebin.ubuntu.com/673732/ | 11:23 |
| nessita | gatox: so, of course you don't need to change anything already done in our code base, but could you improve a bit the uninstall_application? | 11:24 |
| gatox | nessita, ok, on it... anyway i did it that way to avoid unnecesary assignments | 11:25 |
| nessita | gatox: there is no recommended "official" guideline about this, as far as I know, but I personally think that is better to do assignments that have those long lines, which are hard to follow and match parenthesis | 11:26 |
| nessita | facundobatista: do you know if there is an official guideline? ^ | 11:26 |
| nessita | hum, I'm finding something in pep-8 | 11:26 |
| gatox | nessita, what it says? | 11:27 |
| facundobatista | nessita, the official guideline is to let the coder use his/her own style if there's no official guideline :) | 11:27 |
| facundobatista | nessita, that said, which code is it? | 11:27 |
| facundobatista | ah, the mp | 11:27 |
| nessita | facundobatista: specifically, lines 343-351 https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/uninstall/+merge/72402 | 11:28 |
| facundobatista | that *is* ugly :) | 11:28 |
| nessita | gatox: well, it talks about indenting, but in the examples always uses variable names, not more calls. Another issue is that any call fail, is harder to debug which one, since they are all together in the same "sentence" | 11:29 |
| nessita | facundobatista: yes, I agree, but I've noticed is ralsina and gatox's style | 11:29 |
| nessita | maybe is a QT thingy? :-P | 11:29 |
| gatox | nessita, :P | 11:29 |
| gatox | nessita, don't worry... i'm changing uninstall implementation | 11:29 |
| nessita | gatox: you follow me about the hardness of reading that (specially matching parenthesis) and the complex of debugging if there is an exception in that line? I mean, thanks for changing it, but I would like to know if you may see the point to it or you just want me to shut up :-D | 11:30 |
| gatox | nessita, jejeje yes, i see the point, it's easier to read, no doubt there... anyway i prefer always to avoid unnecessary assignments... but not problem | 11:32 |
| mandel | I think is just to make you shut up.... | 11:32 |
| * mandel hides | 11:32 | |
| gatox | mandel, jeje hey! | 11:32 |
| * mandel is not here | 11:32 | |
| mandel | :P | 11:32 |
| nessita | gatox: well, if you think you're using the variable in another call is not unnecessary :-D | 11:33 |
| fagan | mandel: you cant hide from nessita | 11:33 |
| nessita | mandel: just the men I was looking for! | 11:33 |
| facundobatista | gatox, also, you're repeating a lot of code! | 11:33 |
| nessita | mandel: so, how is it going with rick's bug? | 11:33 |
| facundobatista | gatox, I'd do this: http://pastebin.ubuntu.com/673740/ | 11:33 |
| facundobatista | nessita, ^ | 11:33 |
| gatox | facundobatista, let me see | 11:33 |
| nessita | facundobatista: right, minus a syntax issue there, I agree | 11:34 |
| mandel | nessita, I've fixed the quota bug and running the test atm on linux so I can propose and will start looking into that right now, I was going to send that in the report as soon as tests pass :) | 11:34 |
| gatox | facundobatista, yes, you are right | 11:34 |
| facundobatista | nessita, ah, yes, I edited it on the fly | 11:34 |
| nessita | mandel: the quota bug? was still there? | 11:34 |
| nessita | gatox: once that's done, we'll talk about the test. Big applause for adding them, but we need to make some adjustments :-) | 11:35 |
| mandel | nessita, yes, this one: https://bugs.launchpad.net/ubuntuone-client/+bug/727396 | 11:35 |
| ubot4 | Launchpad bug 727396 in ubuntuone-client (Ubuntu Natty) (and 2 other projects) "AttributeError: 'Root' object has no attribute 'subscribed' (affects: 1) (heat: 8)" [Medium,In progress] | 11:35 |
| mandel | oi!!! hal is back!!!! \o/ | 11:36 |
| gatox | nessita, ok!! no problem... i'm applying facundobatista changes right now... | 11:36 |
| nessita | mandel: was it a windows-only bug? | 11:36 |
| nessita | gatox: great! | 11:36 |
| mandel | nessita, nop, linux too | 11:36 |
| mandel | nessita, Root (as in volume_manager.Root) does not have that attribute so I added tests to ensure that the get_share_dict is not called and that we use get_udf_dict, also ensure that it works with root | 11:37 |
| nessita | mandel: nice! | 11:37 |
| mandel | nessita, will be pinging chicharra for a review of that | 11:37 |
| nessita | mandel: please, in particular guillo | 11:41 |
| gatox | nessita, changes done: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/uninstall/+merge/72402 | 11:46 |
| gatox | nessita, let's talk about tests! :P | 11:46 |
| nessita | yes!!! | 11:47 |
| nessita | gatox: so, you added a test_windows.py that tests the uninstall, which is great, but there is some coupling in that module between the code in utils itself and the gui. So, we should have the utils/tests/test_windows.py only tetsing uninstall_application and nothing else, no calls to GUI whatsoever | 11:48 |
| nessita | neither imports to gui, nor to any module outside utils that is not standard lib or 3rd party, of course | 11:48 |
| nessita | so, for utils, you should have a couple of tests asserting: | 11:49 |
| nessita | * sys.frozen is defined -> call uninstall app having the win32api.ShellExecute patches and assert over params | 11:49 |
| nessita | * sys.frozen not defined -> call uninstall app having the win32api.ShellExecute patches and assert over params | 11:49 |
| gatox | nessita, ok, let me change that..... | 11:50 |
| nessita | gatox: wait!!! | 11:50 |
| gatox | nessita, yes? | 11:50 |
| nessita | gatox: that's for the utils tests. Then, since we're adding these lines to gui.py | 11:51 |
| nessita | 61+ elif self.currentId() == self.LICENSE_PAGE_ID: | 11:51 |
| nessita | 62+ if not AreYouSure(self).exec_(): | 11:51 |
| nessita | 63+ utils.uninstall_application() | 11:51 |
| nessita | we need to also write tests inside test_gui for this adding | 11:51 |
| nessita | gatox: so, in those tests, you will patch utils.uninstall_application and assert it was called in the proper scenario | 11:51 |
| nessita | you also need to test that *it was not called* in the other scenario, since this is sensitive | 11:51 |
| mandel | nessita, ping | 11:52 |
| nessita | gatox: makes sense? | 11:52 |
| nessita | mandel: pong | 11:52 |
| gatox | nessita, yep | 11:52 |
| nessita | great! | 11:52 |
| nessita | gatox: I'll add this to the MP | 11:52 |
| mandel | nessita, do you know which version of sd was rick using? | 11:53 |
| nessita | mandel: no, but is pretty newish | 11:53 |
| nessita | mandel: Thursday installer afaik | 11:53 |
| mandel | nessita, hmm we seem to have fs notification very broken, get_by_path fails when the \\?\ is not present... | 11:54 |
| mandel | nessita, which looks like the fs are adding events with it.. has alecu finished with the fs_notifications code? | 11:54 |
| nessita | mandel: pretty much, yes. Where are you seeing that get_by_path fails when the \\?\ is not present? | 11:56 |
| nessita | mandel: I would guess that is only True for old metadata | 11:56 |
| mandel | nessita, https://launchpadlibrarian.net/77877394/syncdaemon-exceptions.log | 11:56 |
| mandel | nessita, the first exceptions, which are also very wrong | 11:56 |
| mandel | nessita, since, it should not fail and that path should never be requested... | 11:57 |
| nessita | mandel: this exceptions.KeyError: 'C:\\Users\\Rick McBride\\AppData\\Local\\xdg\\cache\\ubuntuone\\log' does not mean the \\?\ must to be there! | 11:57 |
| nessita | mandel: that means we're injecting events in the system that are not synced by syncdaemon | 11:57 |
| mandel | nessita, yes, you are right | 11:57 |
| mandel | nessita, but it means that the fs_notifications he was using are very very broken.... | 11:57 |
| nessita | mandel: so this is a dup of what Chipaca was saying yesterady about the watches being crazy | 11:57 |
| nessita | mandel: seems like that is the case in the Thursday installer. Can you please try to reproduce? | 11:58 |
| mandel | nessita, yes :) | 11:58 |
| nessita | mandel: I guess you need a fresh account and run that installer | 11:58 |
| nessita | mandel: be sure to have no metadata present | 11:58 |
| mandel | nessita, okis | 11:58 |
| ralsina | morning! | 11:58 |
| nessita | mandel: and use a dummy account, but with an UDF located next to the Ubuntu One folder | 11:58 |
| mandel | nessita, ok | 12:00 |
| ralsina | I did a new installer last night, you could ask rick to try again. He may have to remove metadata to start cleanly, right? | 12:00 |
| mandel | ralsina, yes, that would be the best thing to do | 12:01 |
| nessita | ralsina: anyway, we have a bigger issue here | 12:01 |
| nessita | mandel: can you try the same to reproduce? | 12:01 |
| mandel | yes | 12:01 |
| ralsina | yes, I see you mentioned extra events? | 12:01 |
| mandel | nessita, ralsina what extra events? | 12:02 |
| nessita | ralsina: we're getting watches in the user home which triggers bazilliong of events | 12:02 |
| ralsina | oh | 12:02 |
| ralsina | let me see my logs... | 12:02 |
| nessita | ralsina: which most of them fail, since they are not "valid events" for syncdaemon | 12:02 |
| ralsina | yep, and for extra fun, that means we watch our log folder :-( | 12:02 |
| nessita | ralsina: you need to have a subscribed UDF next to Ubuntu One folder | 12:03 |
| mandel | oh, that yes... very very bad... the add_ancestors_wathces seems to be broken | 12:03 |
| nessita | mandel: maybe. I would love if you can reproduce and reduce the source of the issue | 12:03 |
| mandel | nessita, on it | 12:03 |
| nessita | mandel: get_udf_dict also requests the subscribed member | 12:07 |
| nessita | mandel: why would that fix the error when using Root? | 12:07 |
| nessita | (referring to https://code.launchpad.net/~mandel/ubuntuone-client/missing-subscribed/+merge/72704) | 12:07 |
| mandel | nessita, give me 1 min and I'll get back to you | 12:08 |
| nessita | sure | 12:09 |
| mandel | nessita, it only looks at it if it is present in the __dict__ but it does not use share.subscribed. Look at line 38 of interaction interfaces and line 58 | 12:11 |
| mandel | nessita, I think that we should use get_udf_dict for root since it is not a share... also look how the tests use get_udf_dict and do not fail | 12:12 |
| nessita | mandel: Root *is* a share from syncdaemon point of view | 12:12 |
| nessita | mandel: and using get_udf_dict is confusing and sematically wrong, since Root is not and UDF but a Share inside syncdaemon | 12:13 |
| nessita | mandel: we may create a new get_root_dict and re-use that inside the other dicts | 12:14 |
| nessita | or we can change this: | 12:14 |
| nessita | 38 if 'subscribed' not in share_dict: | 12:14 |
| nessita | 39 share_dict['subscribed'] = share.subscribed | 12:14 |
| nessita | to be | 12:14 |
| mandel | nessita, sure, that seems a reasonable change | 12:14 |
| nessita | 38 if 'subscribed' not in share_dict and getattr(share, 'subscribed', None) is not None: | 12:14 |
| nessita | 39 share_dict['subscribed'] = share.subscribed | 12:14 |
| mandel | nessita, what about getattr(share, 'subscribed', True) since Root is always subscrined? | 12:15 |
| mandel | nessita, or check in the type of the instance | 12:15 |
| nessita | mandel: the first one will not work, since we can have (old) shares without the attr and yet the subscribed should not be True | 12:17 |
| nessita | mandel: and we should not send the subscribed attr for the Rott object | 12:18 |
| nessita | Root* | 12:18 |
| nessita | mandel: so, if you don't like the getattr+None, let's define a new method for share, but the change will be bigger (we need to modify all the places where get_share_dict is used) | 12:18 |
| mandel | nessita, ok, I can change the get_share_dict method and add a comment about root | 12:18 |
| mandel | nessita, I don't mind the getattr I was thinking about the results I've seen when running get_udf_dict on a root, that is all | 12:19 |
| nessita | mandel: root is not an udf! :-) | 12:20 |
| mandel | nessita, line 95 in tests/syncdaemon/interaction_interfaces.py | 12:20 |
| nessita | 95 def current_status(self): | 12:20 |
| nessita | ??? | 12:21 |
| nessita | ah, tests | 12:21 |
| mandel | nessita, :) | 12:21 |
| nessita | mandel: what's your point? | 12:21 |
| mandel | nessita, that I though we have to provide it because its in the __dict__ that is all | 12:21 |
| nessita | that test is wrong, Root should not get a subscribed attr | 12:22 |
| nessita | much less a suggested_path, and is not getting one by a implementation detail | 12:22 |
| nessita | that if it gets changed, we'll end up with Root instances with suggested_path attrs | 12:22 |
| mandel | nessita, ok, then I'll change the get_share_dict implementation and will change the test a little | 12:23 |
| nessita | mandel: so, do you follow me that Root is treated like a share, inside syncdaemon? | 12:23 |
| mandel | nessita, I trust that you are right on that, I don't know the exact semantics of those realtions | 12:24 |
| mandel | you know more about that area, so I'll trust you :) | 12:24 |
| ralsina | nessita, mandel: trivial review -- https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-setup/+merge/72707 | 12:27 |
| * mandel looks | 12:28 | |
| mandel | ralsina, ubuntuone_log_dir is not lonver used? | 12:30 |
| gatox | nessita, all the changes are ready for review: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/uninstall/+merge/72402 | 12:30 |
| ralsina | mandel: it will stay in u1-client where everyone else expects it to be | 12:30 |
| ralsina | mandel: a previous branch moved the xdg stuff into sso, and that one went long by mistake | 12:31 |
| ralsina | s/long/along/ | 12:31 |
| nessita | gatox: ack | 12:31 |
| mandel | nessita, susbcribed will be present in Root look at the definition od f Volume, line 121 in volume_manager.py | 12:33 |
| mandel | ralsina, ok | 12:33 |
| nessita | mandel: looking | 12:35 |
| === m_conley_away is now known as m_conley | ||
| nessita | mandel: right, and subscribed will always be True | 12:36 |
| nessita | mandel: then I don't see the bug in the current code | 12:36 |
| mandel | let me check that | 12:37 |
| mandel | nessita, indeed doint a Root().subscribed states that is present... we can reject the merge and set the bug as invalid, or update the tests to use the correct get_share_dict and have an excuse to add more tests | 12:39 |
| mandel | nessita, as you wish.... | 12:39 |
| nessita | mandel: I'm not sure I understood the second option | 12:39 |
| mandel | nessita, more tests where added for a module that has none, lets reject the MP, set bug as invalid and push the tests in a new branch | 12:40 |
| mandel | so that we do not loos the work | 12:40 |
| mandel | ralsina, I dont understand one thing ubuntu_sso.xdg_base_directory.py already existed but was not being added in the package? | 12:40 |
| ralsina | mandel: yes | 12:40 |
| nessita | mandel: ok | 12:41 |
| mandel | ralsina, weird... and how did the thing work? | 12:42 |
| ralsina | mandel: when was the lst time you actually *installed* sso using setup.py? :-D | 12:42 |
| mandel | ralsina, I never use setup.py I was copying the code to my path :P | 12:44 |
| mandel | ralsina, does uninstall remove the metadata? it would be nice if it did | 12:44 |
| ralsina | mandel: doesn't and can't | 12:46 |
| ralsina | because we are uninstalling from the system, so we don't know which users have metadata | 12:46 |
| mandel | ralsina, oh... I wonder what happens when you try to uninstall and is running... does it brake? | 12:47 |
| ralsina | mandel: it probably stops working | 12:48 |
| ralsina | not the same as breaking ;-) | 12:48 |
| mandel | ralsina, but who, sd or the uninstaller? because if it tries to remove a file that is being opened it wont be able, and a running process is using the process file, right? | 12:50 |
| mandel | solo estoy tocando las bolas ;) | 12:50 |
| ralsina | mandel: I *could* add checks to see if the files are open and then ask for a reboot or something like that. | 12:50 |
| ralsina | mandel: I am not sure | 12:50 |
| mandel | ralsina, I'm going to guess that the app we use to create that is smart enought to mark it as remove on reboot, but I dont know... is something to consider... | 12:51 |
| ralsina | mandel: it probably is, but yes, someone should test it | 12:51 |
| mandel | rmcbride, ^ :P | 12:52 |
| mandel | I'm learning to delegate hehehe | 12:52 |
| fagan | me | 12:54 |
| fagan | crap | 12:54 |
| fagan | bad enter button | 12:55 |
| mandel | ? | 12:55 |
| ralsina | fagan alecu mandel nessita dobey gatox: standup in 5' | 12:56 |
| gatox | ralsina, ack | 12:56 |
| fagan | ralsina: ack ack :) | 12:56 |
| mandel | ralsina, gatox, nessita has anyone created a UDF from control panel and seen it in the UI? | 12:59 |
| ralsina | I have created one from the wizard, which is the same widget | 12:59 |
| ralsina | and I have seen it on the u1cp UI but had to kill syncdaemon first | 12:59 |
| fagan | me | 13:00 |
| mandel | ralsina, in the last one I'm tryng to create one and I dont get it :( | 13:00 |
| gatox | me | 13:00 |
| nessita | me | 13:00 |
| fagan | ralsina dobey mandel | 13:00 |
| fagan | standup | 13:00 |
| mandel | me | 13:01 |
| ralsina | me | 13:01 |
| fagan | ill go | 13:01 |
| fagan | DONE | 13:01 |
| fagan | * Read a book | 13:01 |
| fagan | TODO | 13:01 |
| fagan | * what ever anyone needs | 13:01 |
| fagan | BLOCKED | 13:01 |
| fagan | * No | 13:01 |
| fagan | gatox: go | 13:01 |
| gatox | DONE: | 13:01 |
| gatox | Branch for Bug #823414 and Bug #824653 should be ready to merge. | 13:01 |
| gatox | TODO: | 13:01 |
| gatox | Keep working on: Bug #829652, Bug #829365 | 13:01 |
| gatox | BLOCKED: | 13:01 |
| gatox | No. | 13:01 |
| ubot4 | gatox: Bug 823414 on http://launchpad.net/bugs/823414 is private | 13:01 |
| ubot4 | Launchpad bug 824653 in ubuntuone-windows-installer "QT UI no graphic UI for the uninstaller (affects: 1) (heat: 8)" [High,In progress] https://launchpad.net/bugs/824653 | 13:01 |
| ubot4 | gatox: Bug 829652 on http://launchpad.net/bugs/829652 is private | 13:01 |
| gatox | nessita, go | 13:01 |
| ubot4 | gatox: Bug 829365 on http://launchpad.net/bugs/829365 is private | 13:01 |
| nessita | DONE: bug #831043, bug #818190 | 13:02 |
| nessita | TODO: finish aforementioned bugs, windows side is crashing | 13:02 |
| nessita | BLOCKED: a little | 13:02 |
| nessita | NEXT: mandel | 13:02 |
| ubot4 | Launchpad bug 831043 in ubuntuone-client (and 1 other project) "Provide a linux implementation for login_email_password (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/831043 | 13:02 |
| ubot4 | Launchpad bug 818190 in ubuntuone-client "The Qt implementation is not filtering by appname correctly (affects: 1) (heat: 26)" [High,Triaged] https://launchpad.net/bugs/818190 | 13:02 |
| mandel | DONE: looked at bug 727396 set as in valid per talk with nessita yet will create a branch with the added tests so work is not lost.Started looking t bug 831634 | 13:02 |
| mandel | TODO: 831634 + others. Look at flights tickets for Pycon AR. | 13:02 |
| mandel | BLOCKED: no | 13:02 |
| mandel | ralsina, go! | 13:02 |
| ubot4 | Launchpad bug 727396 in ubuntuone-client (Ubuntu Natty) (and 2 other projects) "AttributeError: 'Root' object has no attribute 'subscribed' (affects: 1) (heat: 8)" [Medium,Invalid] https://launchpad.net/bugs/727396 | 13:02 |
| ralsina | DONE: started moving xdg to the right places, started fixing the stderr mess on windows, minor installer fixes, calls, administrivia. TODO: finish xdg-moving, finish stderr, fix the opening of folders in u1cp. BLOCKED: no | 13:02 |
| ubot4 | Launchpad bug 831634 in ubuntuone-client "ActionQueue - Upload Error running the command: Path u'\\\\?\\c:\\users\\rickmc~1\\appdata\\local\\temp\\tmpkxbjks' should be a bytes sequence (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/831634 | 13:02 |
| mandel | EOM? | 13:03 |
| ralsina | eom! | 13:03 |
| nessita | mandel: probably | 13:03 |
| mandel | I need to go for lunch or I'll kill myself driving trying to get to the mumble call :) | 13:03 |
| ralsina | anyone seen alecu? | 13:03 |
| mandel | and I drive like a spaniard :P | 13:03 |
| * mandel lunch | 13:03 | |
| ralsina | nessita: you claimed this one, right? https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-setup/+merge/72707 | 13:06 |
| nessita | yes, queued up for review | 13:06 |
| ralsina | nessita: ok | 13:06 |
| dobey | me | 13:06 |
| dobey | λ DONE: jenkins call, nearly finished bug #828195 | 13:07 |
| dobey | λ TODO: finish #828195, installer release/upload | 13:07 |
| dobey | λ BLCK: None. | 13:07 |
| ubot4 | Launchpad bug 828195 in ubuntuone-installer "Wrong text content in installer (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/828195 | 13:07 |
| nessita | ralsina: needs fixing'd | 13:07 |
| ralsina | nessita: could you set it to approved so I can fix the lint and push it? trunk is broken without this | 13:07 |
| alecu | hello! | 13:08 |
| nessita | ralsina: ah, I need to review the code still | 13:08 |
| nessita | hi alecu | 13:08 |
| ralsina | nessita: ok. The good thing is, it only removes code ;-) | 13:08 |
| fagan | wow big storm in ireland | 13:10 |
| fagan | thunder and lightning going on | 13:10 |
| fagan | lots of rain | 13:11 |
| gatox | nessita, let me know when you have the chance to see the uninstaller branch :P | 13:11 |
| nessita | gatox: before the mumble ideally :-) | 13:11 |
| gatox | nessita, ok | 13:12 |
| rmcbride | OK so yea, testing uninstaller on windows, observing whhether or not it's marking as reboot required | 13:15 |
| rmcbride | ralsina: regarding the "does it flag for reboot" question above that mandel referred to me, it does indeed prompt for reboot on windows when uninstalling | 13:17 |
| ralsina | rmcbride: cool, less things to do | 13:17 |
| nessita | alecu: can we mumble in 5 minutes? | 13:22 |
| alecu | nessita, sure | 13:22 |
| nessita | alecu: I'm getting an abrigo first, I'm freezing | 13:27 |
| nessita | alecu: I'm there | 13:30 |
| gatox | lisette, ping | 13:32 |
| ralsina | nessita: I fixed the lint problem in https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-setup/+merge/72707 | 13:33 |
| === m_conley is now known as m_conley_away | ||
| lisette1 | gatox: in a meeting for an hour or so, can i ping you afterwards? | 13:46 |
| gatox | lisette1, yes, no problem! | 13:47 |
| nessita | alecu: the proposed fix works GREAT! I'm so happy! I've spent hours debugging this | 13:51 |
| alecu | nessita, congrats!!!! } | 13:51 |
| nessita | ralsina: looks good. I thought mandel reviewed it too? | 13:53 |
| nessita | (I don't see his vote nor review claim) | 13:53 |
| nessita | gatox: going back to yours now | 13:54 |
| gatox | nessita, awesome | 13:54 |
| ralsina | nessita: he asked a few things but commented nothing | 13:55 |
| nessita | gatox: was a design decision to have this Agree && continue with a lowercase continue? | 13:55 |
| nessita | I would expect: Agree && Continue | 13:55 |
| ralsina | nessita: the text came from design, yes | 13:56 |
| gatox | nessita, yes | 13:56 |
| ralsina | nessita: "Agree && Continue" is capitalization for titles, this is not a title | 13:56 |
| * ralsina is guessing quite a bit there | 13:56 | |
| gatox | nessita, it's specified in the bug... do you want the link? | 13:56 |
| nessita | gatox: no need, thanks! | 13:57 |
| nessita | gatox: so, the BaseTestCase you're inheriting in test_gui already provides a method called "_set_called" | 14:00 |
| nessita | gatox: that you can use to patch stuff like the uninstall thingy | 14:01 |
| nessita | gatox: and then you can assert over a self._called attribute that the same basetestcase provides | 14:01 |
| nessita | gatox: so there is no need to create yet another fake class. We usually create fakes when we need to track something more than just a call | 14:01 |
| nessita | gatox: so this call | 14:03 |
| nessita | self.patch(utils, "uninstall_application", | 14:03 |
| nessita | fake_uninstaller.call_uninstaller) | 14:03 |
| nessita | would be | 14:03 |
| nessita | self.patch(utils, "uninstall_application", self._set_called) | 14:03 |
| nessita | and then you can assert over the self._called attr | 14:03 |
| gatox | nessita, yes..... i'll change that now | 14:04 |
| nessita | gatox: thanks! | 14:04 |
| nessita | gatox, ralsina, Chipaca: mumble? | 14:05 |
| ralsina | nessita: ack | 14:05 |
| ralsina | chipaca: windows mumble ping? | 14:08 |
| Chipaca | ralsina: voy | 14:09 |
| === m_conley_away is now known as m_conley | ||
| karni | I connected via phone's 3G. Man it's striking thunders here! | 14:36 |
| === karni_ is now known as karni | ||
| mandel | nessita, ralsina, gatox, Chipaca, alecu super shitty script that will listen to changes to the given path, just pass the path through the command line | 14:51 |
| mandel | you will not be able to rename or delete the watch dir | 14:51 |
| ralsina | nessita: if you are talking we can't hear you :-) | 14:52 |
| mandel | Chipaca, the only issue I see with that watcher code is how to hook that to twited, atm we use deferToThread | 14:54 |
| Chipaca | mandel: you or roberto can ask the guy at his interview | 14:55 |
| mandel | Chipaca, we could take a look of how to fix that.. but it uses IOCP and that is complicated code... | 14:56 |
| mandel | Chipaca, which interview? | 14:56 |
| Chipaca | mandel: to see if we hire him | 14:56 |
| * nessita is back! | 14:59 | |
| mandel | Chipaca, ah.. ok I'll ping ralsina about this :P | 14:59 |
| ralsina | mandel: I am currently setting things up in the HR system. Once that is done, we'll contact the fellow ASAP | 15:00 |
| mandel | ralsina, ah.. ok :) | 15:00 |
| mandel | ralsina, by the way, I cannot get the UDFs to work on IRL, can you try that? | 15:01 |
| mandel | ralsina, I'm also not getting the wathers in the parent dir, but ofcourse that is due to the UDFs not being created... | 15:02 |
| ralsina | mandel: not for a while, sorry. But apparently Chipaca couldn't either, so it's broken. What you could try is setting up the account on Linux and then just syncing a UDF? | 15:02 |
| nessita | mandel: you can create a UDF in the web | 15:02 |
| nessita | or no? | 15:02 |
| * nessita checks | 15:02 | |
| Chipaca | ralsina: what udf did you try? | 15:03 |
| nessita | ah no | 15:03 |
| nessita | bummer | 15:03 |
| nessita | mandel: forget the last thing I said :-) | 15:03 |
| ralsina | Chipaca: I had one for my mobile pictures | 15:03 |
| * mandel forgets.. | 15:03 | |
| Chipaca | I tried "My Documents", but I'm not sure if that's a real folder or not any more | 15:04 |
| nessita | alecu: this one looks approved! https://code.launchpad.net/~alecu/ubuntuone-client/fix-vbox-weird-clock/+merge/71965 | 15:04 |
| nessita | mandel: can you please review? is trivial, but it helped me debug the signals issue https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/better-logging/+merge/72728 | 15:04 |
| mandel | nessita, sure! | 15:06 |
| mandel | nessita, you have my plus 1 | 15:12 |
| nessita | thanks! | 15:13 |
| nessita | gatox: out of curioisty, any reason to have this cut off in two lines? | 15:14 |
| nessita | self.patch(utils, "uninstall_application", | 15:14 |
| nessita | self._set_called) | 15:14 |
| gatox | nessita, sorry... that was from the previous implementation, when that line was too long... | 15:15 |
| * gatox fixing..... | 15:15 | |
| nessita | gatox: another question, why are you adding the skip decorator to a window specific test? | 15:15 |
| nessita | @skipIfOS('linux2', 'Windows-specific tests.') | 15:15 |
| nessita | gatox: test_windows.py should be ignored in linux (we may need to tweak the test runner) | 15:15 |
| nessita | gatox: look at the ussoc test runner, let me grab the line number | 15:15 |
| nessita | gatox: see run-tests in ussoc trunk, we should have something very similar in installer | 15:16 |
| gatox | nessita, ok....... i will change that | 15:17 |
| nessita | alecu, mandel: big one, but final result should make us happier https://code.launchpad.net/~nataliabidart/ubuntuone-client/login-email-password-for-everyone/+merge/72735 :-) | 15:18 |
| nessita | it fixes bugs and improves uglyness | 15:18 |
| dobey | ok, lunch time. bbiab | 15:25 |
| gatox | nessita, done | 15:25 |
| nessita | gatox: relooking! | 15:33 |
| mandel | nessita, bye bye mocker I suppose :P | 15:38 |
| nessita | mandel: YEAH (at least for that module) | 15:39 |
| mandel | :D | 15:39 |
| nessita | gatox: are tests passing for you in linux for installer? | 15:41 |
| nessita | I've got FAILED (failures=3, successes=117) | 15:41 |
| gatox | nessita, let me check again | 15:42 |
| nessita | http://pastebin.ubuntu.com/673913/ | 15:42 |
| nessita | not related to your code though | 15:42 |
| ralsina | nessita: when you have a moment, I re-added ubuntuone_log_dir here https://code.launchpad.net/~ralsina/ubuntuone-client/log-betterer/+merge/72597 | 15:52 |
| nessita | ralsina: sure! | 15:53 |
| ralsina | And I go have a lunch break | 15:53 |
| nessita | ralsina: question | 15:53 |
| nessita | ralsina: why you removed and re-added tests/platform/test_xdg_base_directory.py? can't we just remove from inside the file what's no longer needed? | 15:54 |
| nessita | we lost bzr history in the file that way | 15:54 |
| ralsina | nessita: I had removed it first, | 15:54 |
| ralsina | I can do a revert and re-edit, is that the right way? | 15:54 |
| nessita | ralsina: yes sir, you may have an issue with the new file you added, so maybe you need to either remove that (not sure if that will work) or start a new branch as a last resource | 15:55 |
| ralsina | ok, I'll untangle that don't worry | 15:55 |
| ralsina | grmbl, looks like a new branch is the easiest way | 16:02 |
| === beuno is now known as beuno-lunch | ||
| ralsina | nessita: pushed a new branch, it's the same as the old one but without breaing history: https://code.launchpad.net/~ralsina/ubuntuone-client/log-betterer/+merge/72747 | 16:30 |
| nessita | ralsina: great! | 16:30 |
| alecu | nessita, ralsina: we've been discussing a bit on the download nanny with facundo, and we have been doing some tests with mandel, and here are some conclusions we have arrived to: http://piratepad.net/aBJrwCsZOO | 16:50 |
| nessita | looking | 16:50 |
| nessita | gatox: this swicth is not valid for pep8 | 16:51 |
| nessita | -i "test_linux.py" . | 16:51 |
| nessita | alecu: read. The nanny is the only one depending on the file_close_{no}write? | 16:52 |
| alecu | nessita, apparently yes. I've not digged further, but it makes sense for it to be the one needing the close_nowrite | 16:53 |
| alecu | nessita, the close_write is useful for sync though | 16:53 |
| nessita | right | 16:54 |
| alecu | nessita, but that can be in fact simulated with the "updated" event we receive on windows plus a small timeout to see that no more "update" are received immediately. | 16:55 |
| alecu | nessita, so that's the high level overview, to be able to assess the risk of not having this working properly. | 16:56 |
| nessita | ack | 16:56 |
| alecu | nessita, I think it's a lower risk than we thought, but it's a risk nonetheless | 16:56 |
| alecu | I will dig further into this points, but I wanted to request some comments or ideas on this. | 16:57 |
| nessita | kinda agree, though I would like seen this fixed before the release to the public/press | 16:57 |
| nessita | alecu: what app do we plan to target to test? | 16:59 |
| nessita | ralsina: I filed bug #833190, found those tests failures while reviewing a branch from gatox | 16:59 |
| ubot4 | Launchpad bug 833190 in ubuntuone-windows-installer "Failing tests (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/833190 | 16:59 |
| nessita | gatox: did you see the issue with the -i and pep8? | 17:00 |
| nessita | ralsina: you removed the old proposal for log-betterer? | 17:01 |
| alecu | nessita, *the* app would be "office" | 17:01 |
| nessita | alecu: can we also test notepad, and some image editor (ideally the default one)? | 17:02 |
| gatox | nessita, i'm running it now with the -i included in run-tests or do you mean something else? | 17:02 |
| nessita | gatox: you added a -i to pep8, which is not supported | 17:02 |
| nessita | gatox: pep8 should also check the test_linux | 17:02 |
| gatox | nessita, yes... i'm fixing that... i added in the wrong place for run-tests.bat | 17:03 |
| alecu | nessita, can you please re-review or change the vote on https://code.launchpad.net/~alecu/ubuntuone-client/fix-vbox-weird-clock/+merge/71965 | 17:11 |
| alecu | ? | 17:11 |
| nessita | alecu: does tarmac complains for a needs information? :-/ | 17:11 |
| dobey | nessita: yes | 17:13 |
| nessita | dobey: why? | 17:13 |
| dobey | nessita: because it signifies that you need more information to be able to complete the review | 17:13 |
| nessita | I got my information :-) | 17:13 |
| dobey | nessita: if said information has been provided, you need to change your vote | 17:13 |
| nessita | anyways, let's vote | 17:13 |
| nessita | ralsina: ping | 17:13 |
| dobey | either approve or obstain or needs fixing or something :) | 17:13 |
| dobey | abstain even | 17:14 |
| nessita | approved! | 17:14 |
| alecu | thanks! | 17:16 |
| nessita | :-) | 17:16 |
| nessita | alecu: did you notice my review request? no to much hurries, but just wanted to be sure is in your radar | 17:17 |
| alecu | nessita, will get some lunch and do your review afterwards. | 17:18 |
| nessita | alecu: great! thanks | 17:18 |
| === beuno-lunch is now known as beuno | ||
| dobey | can i have a review please? https://code.launchpad.net/~dobey/ubuntuone-installer/new-style/+merge/72764 | 17:36 |
| gatox | nessita, is there any problem with the tests in sso?? i have 33 tests failing and i want to know if it is just me | 17:41 |
| nessita | gatox: remember to STOP the real sso service :-) | 17:41 |
| gatox | nessita, i'm going to check if it is running in background | 17:42 |
| gatox | nessita, yep... sso was in background | 17:43 |
| nessita | oh | 17:43 |
| gatox | nessita, so now i have a very little branch to propose while i'm finishing the other | 17:44 |
| nessita | ralsina: ping? | 17:45 |
| nessita | dobey: I'm leaving in a while, may I ask you a favor? | 17:45 |
| * beuno whispers "it's a trap" | 17:46 | |
| nessita | lol | 17:46 |
| dobey | it's always a trap | 17:47 |
| nessita | dobey: I don't want to forget to mention to ralsina that we should not delete merge proposal with votes in it, but reject them. This happened with https://code.launchpad.net/~ralsina/ubuntuone-client/log-betterer/+merge/72597 | 17:48 |
| nessita | gatox: APPROVE!@:-) | 17:48 |
| dobey | oh i can yell at ralsina. that's easy :) | 17:48 |
| gatox | nessita, nice | 17:48 |
| nessita | dobey: no yell needed, I think. As far as I know if the first time we let him know that :-) | 17:49 |
| dobey | oh that reminds me | 17:52 |
| gatox | nessita, ralsina a quick review? https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/ui-form/+merge/72768 | 17:56 |
| nessita | gatox: sure1 | 17:57 |
| dobey | thisfred: can you review https://code.launchpad.net/~dobey/ubuntuone-installer/new-style/+merge/72764 ? should be quick and easy :) | 17:58 |
| nessita | gatox: indentation for + def __init__(self, line_edit, valid_cb=lambda x: False, is not pep-8'd | 17:58 |
| nessita | gatox: see http://www.python.org/dev/peps/pep-0008/, Code lay-out section | 17:58 |
| dobey | or anyone else ^ | 17:59 |
| nessita | see the Yes and No | 17:59 |
| nessita | dobey: I'll review | 17:59 |
| dobey | nessita: thanks | 17:59 |
| ralsina | gatox: sure! | 17:59 |
| nessita | dobey: why do we have attr starting with dunder? | 18:00 |
| nessita | attrs* | 18:00 |
| ralsina | ok, won't delete MPs with votes. Why? | 18:00 |
| dobey | nessita: because they're private | 18:00 |
| nessita | dobey: in that case they should have one underscore | 18:00 |
| dobey | no | 18:00 |
| ralsina | __ is private, mangled | 18:00 |
| dobey | yes | 18:01 |
| nessita | hum, let me confirm | 18:01 |
| dobey | _ is "you really shouldn't touch this, but you can" | 18:01 |
| thisfred | dobey: on it | 18:02 |
| dobey | also, it took me a minute to realize what you meant by "dunder" | 18:02 |
| nessita | dobey: you're right | 18:02 |
| thisfred | I don't like underunders either, but if it's a pattern within gtk, I won't object | 18:03 |
| nessita | dobey: I've heard it a lot in python talks, I think I got that from Raymond Hettinger | 18:03 |
| gatox | nessita, that rule applies too for??: | 18:04 |
| gatox | + self.clear_label.setPixmap(QApplication.style().standardIcon( | 18:04 |
| gatox | 22+ QStyle.SP_MessageBoxWarning).pixmap(16, 16)) | 18:04 |
| nessita | and I loved it | 18:04 |
| thisfred | dobey: __ you can still touch, if you really want to. I usually use _ to signal intent and then if someone poops on the welcome mat, so to speak, they deserve what they get, which is breakage | 18:04 |
| ralsina | nessita: got the needsfixing, I have no idea how that test passes here. I'll fix it right away. | 18:04 |
| nessita | gatox: please use variables in that sentence!!! | 18:04 |
| nessita | ralsina: you run tests in linux? | 18:04 |
| ralsina | nessita: I have no working linux at the moment :-( | 18:04 |
| dobey | thisfred: yes, but you have to mangle your code to do it | 18:04 |
| nessita | ralsina: got the no-delete-proposal-with-votes-please? | 18:05 |
| ralsina | nessita: got it | 18:05 |
| dobey | thisfred: i'd rather it give them the AttributeError | 18:05 |
| dobey | than say "here you go!" | 18:05 |
| nessita | dobey: so, where are all the tests for that code? :-) | 18:05 |
| thisfred | dobey: I seem to remember __ come with their own set of problems, which is why their use is not encouraged. | 18:06 |
| dobey | nessita: in gtk+? | 18:06 |
| nessita | dobey: no, the tests for your code, the one you're writting | 18:06 |
| ralsina | Ok, I now see why it didn't fail. If I pass two test modules to u1trial, it runs only the first one :-( | 18:06 |
| nessita | dobey: we should be testing all those lines of code | 18:06 |
| dobey | sigh | 18:07 |
| dobey | ralsina: hi | 18:07 |
| dobey | ralsina: stop deleting merge proposals! :) | 18:08 |
| ralsina | dobey: got it, got it! | 18:08 |
| ralsina | nessita: pushed with the fix for that and test instructions that actually run the tests | 18:08 |
| dobey | ralsina: also, file a bug against devtools to allow passing multiple modules to u1trial | 18:09 |
| ralsina | dobey: filing! | 18:09 |
| nessita | ralsina: instructions are run 'run-tests.bat' in windows :-) | 18:09 |
| nessita | and make check in linux | 18:09 |
| nessita | no? | 18:09 |
| ralsina | nessita: no, because it's u1-client and that has lots of failures? | 18:09 |
| dobey | for u1client? | 18:09 |
| dobey | yes | 18:09 |
| ralsina | on linux, maybe, but this code has to be tested on both sides | 18:10 |
| nessita | ralsina: but they are known and we should check no new ones are added | 18:10 |
| ralsina | nessita: known by whom exactly ? | 18:10 |
| nessita | ralsina: how can we ensure we're not breaking anything new then? | 18:10 |
| nessita | ralsina: by anyone that run the suite in trunk\ | 18:10 |
| ralsina | nessita: well, that's why we have been fixing tests for the last 6 weeks, I would say :-) | 18:10 |
| dobey | nessita: anyway i have no idea how any unit tests on ubuntuone-installer code would be useful, and actually be unit tests, right now | 18:10 |
| nessita | dobey: just like we're building unit tests for the gtk control panel | 18:11 |
| nessita | dobey: I can help you tomorrow | 18:11 |
| dobey | nessita: this needs to land asap | 18:11 |
| ralsina | nessita: are you sure I see the same test failures you see? | 18:11 |
| ralsina | nessita: known would mean documented to me, in this case. | 18:11 |
| dobey | nessita: UI freeze is in < 6 hrs | 18:11 |
| nessita | ralsina: 95% sure they should be the same. We're not having timing errors, but real failures | 18:12 |
| ralsina | nessita: ok, I will try to document them myself then | 18:12 |
| nessita | ralsina: anyways, if you run the whole suite in trunk and in your branch, you can ensure you're not breaking anything new | 18:12 |
| nessita | I can share a test run if you'd like | 18:12 |
| ralsina | nessita: don't worry, I will do it | 18:13 |
| nessita | dobey: I will abstain and email the bosses, I disagree with having this code in the CD without backing up tests | 18:13 |
| nessita | dobey: but I will not delay this today | 18:14 |
| dobey | it's not on the CD yet (or even in main) | 18:14 |
| nessita | well, we're hoping it to be, no? | 18:14 |
| dobey | yes, but probably won't be today | 18:14 |
| dobey | oh | 18:15 |
| dobey | i take that back | 18:15 |
| dobey | it is in main now :) | 18:15 |
| gatox | ralsina, nessita please review this branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/line-edit/+merge/72772 | 18:15 |
| dobey | as of 50 minutes ago anyway :P | 18:15 |
| nessita | gatox: did you fixed the one you linked before that? | 18:16 |
| gatox | nessita, i had some problems and launchpad get blocked for that branch, so i delete it and resubmit in another one | 18:16 |
| nessita | dobey: is there any test written in that project? | 18:16 |
| ralsina | gatox: shouldn't you hide the label if it's not meant to be seen? Otherwise it will still be there (but maybe with size 0) | 18:16 |
| nessita | gatox: don't delete! :-) resubmit | 18:16 |
| dobey | nessita: yes | 18:16 |
| gatox | nessita, yes, but it was stuck.... some locking problem (and not in my local branch.... i've tried bzr break-lock) | 18:17 |
| nessita | gatox: and what happened? | 18:18 |
| gatox | nessita, i delete it and create another branch | 18:18 |
| gatox | it's this one | 18:18 |
| nessita | gatox: what happened with the bzr break-lock I mean :-) | 18:18 |
| dobey | nessita: i had to change _find_data_file() in that branch, in order to keep them from failing, even :) | 18:18 |
| gatox | nessita, ahhh...... nothing... i could push code anyway | 18:19 |
| gatox | couldn't | 18:19 |
| ralsina | gatox: ^^ | 18:19 |
| nessita | gatox: ok, next time feel free to ask for help to unblock that | 18:19 |
| gatox | nessita, ok | 18:19 |
| gatox | ralsina, sorry... i can't find the label..... which line? | 18:20 |
| gatox | ralsina, do you mean setVisible(False)? | 18:20 |
| ralsina | gatox: around line 20 | 18:20 |
| ralsina | gatox: yes | 18:20 |
| ralsina | oh, wait. You are setting it to false. When is it set to true? | 18:21 |
| nessita | dobey: since we have 6 hours to freeze, can't you add tests in the mean time? | 18:21 |
| ralsina | gatox: let's start again ;-) | 18:21 |
| nessita | gatox: what's the new MP link (after the removed one)? | 18:22 |
| gatox | nessita, https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/line-edit/+merge/72772 | 18:22 |
| ralsina | gatox: I meant: if warning_sign is False, it should never be set to visible at all. And I don't see that in the MP diff | 18:22 |
| dobey | nessita: i'd rather not have to work those full 6 hours, no. i do also have personal life things i need to accomplish today after i finish working | 18:22 |
| nessita | dobey: but you have still 2.5 hours left, if I do the math properly | 18:23 |
| gatox | ralsina, well..... it is set to true when the validation functino says so... but as it is always empty...... i doesn't show anything and keeps being invisible | 18:23 |
| ralsina | gatox: you are counting on the label being width=0 | 18:23 |
| gatox | ralsina, or we can validate if the label doesn't contain a pixmap | 18:23 |
| ralsina | gatox: that can stop being true by (for example) changing the stylesheet | 18:23 |
| gatox | ralsina, no | 18:23 |
| gatox | ralsina, the label is over the line edit | 18:24 |
| ralsina | gatox: yes. And it may cover the text in the lineeedit if it's set to visible and has any width | 18:24 |
| dobey | nessita: yes, and i also need to put icons in, make the release, update the packaging, get someone to sponsor the upload, etc | 18:24 |
| dobey | nessita: and i'm not trying to avoid tests. i really don't see how any tests could possibly be useful here | 18:25 |
| dobey | (beyond the tests that are already in place) | 18:26 |
| nessita | dobey: every line of code we add is error-prone | 18:27 |
| nessita | and is very easy to break when adding new code, causing regressions | 18:27 |
| nessita | anyways, I gotta run to the uni | 18:28 |
| dobey | as are the tests. anywya | 18:28 |
| nessita | ok, I'm gone | 18:30 |
| nessita | gatox: I'll try to finish your reviews today, otherwise I'll do them tomorrow morning | 18:30 |
| nessita | bye all! | 18:30 |
| ralsina | bye! | 18:30 |
| dobey | there is like 0 interactive UI in this app | 18:30 |
| ralsina | dobey: what's the branch in question? | 18:31 |
| dobey | the next and cancel buttons are pretty much it (and the learn more thing, which opens a browser) | 18:31 |
| dobey | ralsina: https://code.launchpad.net/~dobey/ubuntuone-installer/new-style/+merge/72764 | 18:31 |
| * ralsina looks | 18:31 | |
| dobey | it's all advertising | 18:31 |
| dobey | text and images, and layout; the cancel/install buttons already get tested, ensuring that the page gets switched to the progress bar page | 18:32 |
| ralsina | dobey: considering the time constraints and that you have been all alone doing this stuff, +1 on your own recongnizance (sp?) | 18:32 |
| dobey | i don't know what you mean by recognizance :) | 18:33 |
| dobey | though i do recognize you didn't mean to put an n before the g in it | 18:34 |
| * ralsina heard that on Law & Order | 18:34 | |
| ralsina | http://en.wikipedia.org/wiki/Recognizance | 18:34 |
| dobey | a bond or obligation of record entered into before a court of record or a magistrate, binding a person to do a particular act. | 18:34 |
| dobey | i don't think i've had to deal with any bail bondsmen on this branch :) | 18:35 |
| ralsina | So, I trust your word that this doesn't break stuff, and if it does, it's your faul ;-) | 18:35 |
| ralsina | fault | 18:35 |
| dobey | well we all know my code is perfect anyway :) | 18:35 |
| gatox | ralsina, ready!! branch updated... i'll submit my other branch later... have to go! | 18:39 |
| gatox | byeeeeee | 18:39 |
| ralsina | bye! | 18:39 |
| dobey | ah, manuel | 18:41 |
| dobey | ralsina: is manuel coming to UDS too? | 18:42 |
| ralsina | dobey: AFAIK no | 18:42 |
| dobey | aww | 18:42 |
| ralsina | dobey: UI freeze means you can still make uploads until the next freeze, right? | 18:43 |
| ralsina | dobey: just no new strings/UI? | 18:43 |
| dobey | ralsina: it means you can't change UI without an exception | 18:43 |
| ralsina | ok | 18:43 |
| dobey | ralsina: strings can change. but ui can't | 18:43 |
| nhaines | Well, except it's also beta freeze in 6 hours. | 18:43 |
| dobey | string freeze is in a couple weeks | 18:44 |
| dobey | nhaines: 5.25 | 18:44 |
| dobey | ralsina: is there other stuff we really need to get in ubuntu today? | 18:45 |
| nhaines | dobey: I'm in California, I can't handle UTC math during Daylight Saving time. :) | 18:45 |
| ralsina | dobey: not that I know of | 18:45 |
| dobey | nhaines: just use UTC for everything. be a force of change :) | 18:45 |
| nhaines | haha! | 18:45 |
| dobey | "There is only Zulu." | 18:45 |
| ralsina | dobey: answered email about those tests, check response on your own time | 18:48 |
| dobey | ok | 18:53 |
| nhaines | dobey: :) | 18:53 |
| ralsina | And I am gone to pick up my kid from school. Will put more hours later for today's release. | 19:03 |
| dobey | ralsina: haha. "linux team" :) | 19:21 |
| ralsina | dobey: I knew you would appreciate it ;-) | 19:37 |
| dobey | ralsina: https://code.launchpad.net/~dobey/ubuntuone-installer/add-icons/+merge/72780 <- added some icons. review please? :) | 19:38 |
| ralsina | So much for the idea that all tests that fail on u1-client fail all the time | 19:42 |
| dobey | haha | 19:43 |
| ralsina | dobey: +1 trivial, just looked at it, so pinky-swear that it works | 19:45 |
| dobey | heh | 19:46 |
| karni | duanedesign: yo my man, still around? (you are crazy if you are! ;) | 19:52 |
| dobey | karni: why is that crazy? he's like -0200 from me :) | 19:53 |
| karni | dobey: because he starts at 6-7 AM to get more time with european co-workers | 19:54 |
| duanedesign | karni: :) | 19:54 |
| karni | wait.. | 19:54 |
| karni | still it's like -7h from here probably :D | 19:54 |
| dobey | rye is probably still around :P | 19:54 |
| karni | dobey: he's always around | 19:54 |
| dobey | exactly :) | 19:55 |
| karni | :) | 19:56 |
| karni | duanedesign: sticky please! :) http://ubuntuforums.org/showthread.php?p=11183473#post11183473 | 19:56 |
| karni | nhaines: http://ubuntuforums.org/showthread.php?t=1832288 | 20:12 |
| ralsina | dobey: it's just me or comparing full test suite runs is pretty hard? http://diffchecker.com/Jo7wN68 | 20:13 |
| duanedesign | karni: done | 20:14 |
| karni | duanedesign: thanks! :) | 20:14 |
| dobey | ralsina: it is very hard :) | 20:14 |
| ralsina | good to know I am not just tired | 20:15 |
| dobey | ralsina: adding or removing tests makes it significantly harder, because ordering, etc… can change | 20:16 |
| ralsina | I love how the number of failures is the same but they are not the same tests | 20:16 |
| dobey | heh | 20:17 |
| nhaines | karni: great! The QR code is useful of course. :D | 20:40 |
| karni | nhaines: =) | 20:41 |
| nhaines | karni: okay, I'll play with it... didn't find any problems with the last one. :) | 20:42 |
| karni | nhaines: Coolio! Much thanks :) (IIRC you already had 1.0.3*, this is hopefully last rc before the Market) | 20:43 |
| nhaines | karni: yes, and SSO worked great (except one thing) but I had no pretty progress meters. :) | 20:44 |
| karni | nhaines: What was the problem? | 20:45 |
| nhaines | Oh, now that I think of it, that one thing is that my Android notification bar always shows the "syncing" icon... I think it might be Ubuntu One. | 20:45 |
| karni | nhaines: Ah right! Not only you have progress bars, you can download stuff while uploading as well (and vice versa) | 20:45 |
| karni | nhaines: yes, it has been fixed | 20:45 |
| nhaines | yay! | 20:45 |
| karni | nhaines: could you please uninstall U1F | 20:45 |
| karni | nhaines: and make sure there is no Ubuntu One in Account Manager | 20:45 |
| nhaines | Sure. I did install the update already though. | 20:46 |
| karni | nhaines: once installed, you will no longer see the sync icon. sorry about that. | 20:46 |
| * karni thinks | 20:46 | |
| karni | let me see | 20:46 |
| nhaines | Okay, uninstalling now. | 20:46 |
| nhaines | Ubuntu One is no longer listed in Account Manager. | 20:47 |
| nhaines | Haha, I like the "New Storage Detected" notice. It's so cheerful. :D | 20:48 |
| karni | nhaines: yes, the sync is disabled during _creation_ of the account, so it's best to uninstal. and since 1.0.3 never made it to the market, only people who used any RC candidate are affected | 20:48 |
| karni | nhaines: :) | 20:48 |
| nhaines | Ah yes, now sync is off for the U1 account. | 20:49 |
| karni | nhaines: you got me thinking. I should let other testes know. | 20:50 |
| nhaines | Yeah, probably. I'm kicking myself for not filing a bug though... it just slipped my mind. | 20:54 |
| karni | nhaines: That's OK. It's probably just few people. Thanks for testing! | 20:54 |
| nhaines | karni: well, I'm just glad you caught it as well. :) Maybe some time the sync feature will be useful. | 20:55 |
| karni | nhaines: My manager caught it ;) And yes. It's one of the features I'm looking forward to the most. However, not only it has not been given high priority, it's not well supported via REST API either. | 20:56 |
| karni | nhaines: If we had it, I think it would be a killer. No dropbox, no sugarsync, just u1f. | 20:56 |
| * karni just gave examples | 20:56 | |
| nhaines | The app is definitely great! | 20:57 |
| karni | Actually, dropbox is a bad example. They have no sync at all, but there are 3rd party apps that provide this feature (in a dumb way, anyway..) | 20:57 |
| nhaines | karni: It's compelling! "WD 2go" offers a personal cloud with a My Book Live drive (1 - 3TB) at your home, but you can't really save the files and work with them later. (Disclaimer: I'm a WD employee) | 20:59 |
| nhaines | So U1 has less storage but more utility for certain things. :) | 20:59 |
| karni | nhaines: You're WD employee! How awesome is that :)! I got my parents a WD 2TB (RAID 1, 2x1TB) MyStudio book (or something) as a safe storage solution. | 21:00 |
| karni | nhaines: What do you mean you cant save files? | 21:01 |
| nhaines | karni: yup, I'm a technical support auditor, haha. | 21:01 |
| karni | nhaines: Hahah I know who to contact if my drive fails ;D | 21:02 |
| nhaines | With WD 2go's mobile app, you can access any file on your drive but you have to have an app that will work with it first. | 21:02 |
| nhaines | So for example if I tapped on a video file I'd saved nothing happened. then when I installed RockViewer, it would download and stream... but I can't save the file permanently to my phone. | 21:02 |
| karni | nhaines: Isn't it the same with U1F? U1F uses Android/3rd party apps to open files. | 21:03 |
| karni | oh | 21:03 |
| nhaines | karni: sure, but at least the files stay there in the /sdcard/U1 folder | 21:03 |
| karni | nhaines: Right, for off-line access. | 21:03 |
| === m_conley is now known as m_conley_away | ||
| nhaines | karni: or to not destroy your data quota. :) | 21:04 |
| karni | nhaines: I'm looking for a day/opportunity to take on sync. It won't be soon, but man I wish for it. | 21:09 |
| nhaines | karni: it'll definitely be awesome. :) | 21:09 |
| facundobatista | dobey, ping | 21:56 |
| snap-l | Hmm, having some trouble with a machine and Ubuntu One | 22:07 |
| snap-l | 2011-08-24 18:06:44,449 - ubuntuone.SyncDaemon.ActionQueue - WARNING - Connection lost: Connection was closed cleanly. | 22:07 |
| snap-l | This is on a wirless network | 22:07 |
| snap-l | and even on a wired network, it's not working | 22:11 |
| facundobatista | snap-l, we're having some connectivity issues | 22:17 |
| snap-l | That would explain it. | 22:19 |
| facundobatista | snap-l, what about now? it should be solved | 23:13 |
| snap-l | facundobatista: I believe it's fixed now. Thank you. | 23:42 |
| === yofel_ is now known as yofel | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!