rogerk_I seem to have a problem with some stuff in my One-folder being synched, and some not. For example moved a couple of folder into another, those were not synched. Renamed a folder at the same location, that gets synched.07:33
rogerk_renamed one of the moved folders, that one got synched properly, but still not the other one.07:34
somethingintereshi all, just did a reinstall of my netbook. Trying to sync files from the cloud., My local Documents folder is empty, it says sync is up to date but won't let me sync08:06
somethinginteres"this folder cannot be syncronized because it contains one or more folders that are already syncronised"08:07
somethinginteressorry got disconnected. Any solutions?08:19
JamesTaitBuenos dias!08:36
somethinginterestried removing my PC, removing the password from the Passwords & Encryption Keys section. Still no luck. Says I'm already synced u1sdtool --status shows everything as fine08:39
duanedesignsomethinginteres: hello09:07
somethinginteresduanedesign: hi09:08
duanedesignsomethinginteres: can you run the command :   u1sdtool --list-folders09:14
somethinginteres duanedesign: http://paste.ubuntu.com/658537/09:17
duanedesignsomethinginteres: ok. Your Documents folder is set to sync with Ubuntu One. However Subscribed is set to False. So it will not sync09:20
duanedesignyou are wanting to make this Documents folder a cloud folder, crrect?09:21
somethinginteresduanedesign: how do I subscribe it? I've tried right click "Ubuntu One --> Sync" that doesn't fix it.09:21
somethinginteresduanedesign:: I am wanting to d/l my files from the cloud to my PC09:21
somethinginteresduanedesign: from my cloud "Documents" folder to my local09:22
duanedesigndid you copy any of the contents of your old computers Home directory to this new computer?09:23
somethinginteresduanedesign: nope, this is a fresh install09:23
duanedesignok, ust making sure09:23
duanedesignu1sdtool --subscribe-folder=24637050-d981-49f7-bdfd-00ec310c24f409:24
duanedesigncan you run that command09:25
somethinginteresduanedesign: run the cmd. No error or other output09:25
duanedesignsomethinginteres: what version of Ubuntu?09:25
somethinginteresduanedesign: 11.0409:25
somethinginteresduanedesign:  the error I spoke of before has disappeared. The folder is empty but the sync tickbox on the ribbon is now not ghosted out09:26
somethinginteresI believe a sync is taking place09:26
duanedesignwe could of also acheived that by opening the Ubuntu One control Panel, selecting Cloud Folders09:26
duanedesignand ticking the box next to Documents09:27
duanedesigncan you runt he command:   u1sdtool --waiting | wc -l09:27
somethinginteresduanedesign: Output is "1014"09:27
duanedesignok, that is how many items are now queued to sync09:28
somethinginteresduanedesign: OK may take some time. :) Any insight into why it thought the folder contained synced files?09:28
somethinginteresduanedesign: Also I need to disconect very shortly, will disrupting this proccess cause a problem?09:29
duanedesignsomethinginteres: it is a poorly worded error. It just means this folder is set to sync09:30
somethinginteresduanedesign: Right09:30
duanedesignyour are definetly not the first confused by it :)  I need to make sure their is a bug filed on that09:31
somethinginteresduanedesign: : safe to disconnect at this point?09:31
somethinginteresduanedesign: ah, I see :)09:31
duanedesignsomethinginteres: yes you should be fine now. If you have any other problems let me know09:32
somethinginteresduanedesign: Will do. Thanks very much.09:32
duanedesignrogerk_: just saw you needed help. I am going to step out for 45 minutes. I can help when I get back.09:33
rogerk_duanedesign: Oh, nice, thanks! I'm at work so might not be available at that time, but I'll try to sync up at some time after that. :-)09:38
DiegoSarmenterohi everyone10:18
faganhey DiegoSarmentero10:18
duanedesignrogerk_: no problem just ping me10:32
duanedesignhello fagan DiegoSarmentero10:32
DiegoSarmenterohi duanedesign fagan10:32
DiegoSarmenterohow are you this morning? (morning for me :P)10:33
faganhey duanedesign10:33
duanedesignDiegoSarmentero: doing well. Trying to decide on a new laptop. too many choices :/10:34
DiegoSarmenteroduanedesign, alienware! :P10:35
faganduanedesign: id suggest lenovo10:35
duanedesignfagan: i think i narrowed it down to lenovo or system7610:36
faganduanedesign: I cant buy system76 so never was an option10:36
faganI have a acer desktop, msi wind netbook and a macbook10:36
duanedesignfagan: the first lenovo model I looked at did not have usb3.0, though I do not know if i would even miss it10:36
faganduanedesign: yeah you dont really need usb 3 yet10:37
fagannearly everything is still usb 210:37
* duanedesign nods10:37
duanedesignthe model w/ out 3.0 usb is about $300-400 cheaper10:37
faganthen its an easy choice10:38
duanedesignand the more expensive model has a graphics card that uses Optimus10:38
duanedesignthough I hear lenovo has a setting in the bios to deal with this.10:38
faganduanedesign: interesting do you play games or do anything graphically difficult?10:40
* fagan has a nvidia card that cost an arm and a leg but I play lots of games so was an investment10:40
duanedesignoccasionaly...but not often.10:40
faganduanedesign: well then the optimus thing isnt really a huge priority if you dont :)10:41
duanedesignJust enough that i am interested in a graphics card. But not often enough to really make it essential10:41
duanedesignfagan: I think you are right10:41
faganduanedesign: well if it was me id be cheaping out :)10:41
duanedesignfagan: considering technology becomes old so fast it is hard to justify spending too much money10:43
duanedesignknowing you wil have to but a new computer in 3 years10:43
duanedesignfagan: do you run Ubuntu on your macbook?10:44
faganduanedesign: well im a funny person when it comes to moores law, I say get an ok computer and just replace it often it costs about the same in the long run10:44
faganduanedesign: yeah I do but im not using it at the moment10:44
faganim on Mac OS10:44
faganBut its pretty easy to dual boot10:44
faganI have to say the keyboard is bloody nice to use10:45
duanedesignthe macbooks are down right sexy10:45
faganduanedesign: have you thought about getting a desktop?10:46
faganand then getting a netbook10:46
faganso you can have something to travel with10:46
faganthat worked out so well for me10:46
faganAlthough my netbook was a little trash10:46
duanedesignfagan: yeah, not a terrible idea. Would be nice to have something more portable then a 6 pound laptop10:47
faganduanedesign: well desktops seem to keep their value longer because they have faster processors and better graphics cards10:48
faganand they are cheaper10:48
* duanedesign nods10:48
faganso I paid 500 euro for my desktop it has 4 processors and a great graphics card10:49
faganand a netbook that cost 250 and it makes up for the fact I cant move my desktop10:49
faganand 750 is cheaper than most netbooks with similar specs10:50
duanedesignfagan: ahhh, more choices :)10:51
duanedesignfagan: i will look at that...sounds reasonable10:52
faganduanedesign: well I love complicating things for other people :D10:52
* fagan errands 11:00
* fagan back 11:27
nessitahello all!11:35
nessitaralsina: ping11:35
DiegoSarmenteronessita, hello11:36
nessitahi DiegoSarmentero, how is it going? yes, I'm doing your reviews now :-)11:38
DiegoSarmenteronessita,  jejejee i'm fighting with page 18 of the installer :@.....11:40
nessitaDiegoSarmentero: question, why did you add the #pylint: disable=C0103 and the enable comment for set_title and set_subtitle?11:40
DiegoSarmenteronessita, ahhhhhhhhh my bad...... that methods were first setTitle and setSubTitle... but then I realize that setTitle and setSubTitle only were neeeded on SSOWizardPage11:41
nessitaDiegoSarmentero: thanks!11:42
DiegoSarmenteronessita, done11:46
nessitaDiegoSarmentero: another question, why do you call11:46
nessita        self.controller._title = JOIN_HEADER_LABEL % {'app_name': self.app_name}11:46
nessita        self.controller._subtitle = self.help11:46
nessitaat the beginning of the test_titles if you later call set_title and set_subtitle in the same test?11:46
nessitasame question for ErrorControllerTestCase.test_set_ui11:47
DiegoSarmenterolet me check11:48
nessita(and same question for SuccessControllerTestCase.test_set_ui)11:48
DiegoSarmenteronessita, :S yes... self.controller_title, etc is no longer needed... that was from before the second change to user a reimplementation of setTitle... removing.....11:50
nessitaDiegoSarmentero: thanks!11:50
nessitaDiegoSarmentero: the rest looks great11:50
DiegoSarmenteronessita, i removed some of them, in Error and Success they are needed because the method setupUI actually use _title and _subtitle so i have to set that values in the controller11:54
DiegoSarmenteroso i need to set that values, and call then set_title and set_subtitle to record the execution11:54
DiegoSarmenteronessita, done11:56
nessitaDiegoSarmentero: looking11:56
nessitaDiegoSarmentero: hum, is there any chance you push any extra changes? I think I'm seeing something I did not see before, but maybe I was too sleepy 5 minutes ago :-P. What I see that I don't like is the duplication of the def set_title(self, title='') and def set_subtitle(self, subtitle='')12:00
nessitaDiegoSarmentero: what we talked with ralsina was not to duplicate that code anywhere, but defining it in a single class12:00
nessitaDiegoSarmentero: I personally would like to have that as another widget, but he mentioned it may be overkill doing it that way. Anyway, the requirement is that the logic is not duplicated anywhere, so if you find yourself writing the same piece of code twice, we need to refactor12:01
DiegoSarmenteronessita, ok... i didn't in that way... because i wasn't sure if EVERY page is will have a title and subtitle... but it is a easy fix12:03
DiegoSarmenterois going to*12:03
nessitaDiegoSarmentero: well, you can make some pages inheriting this "new title'd class" and some other no, right?12:03
DiegoSarmenteronessita, yes, we could have two different SSOWizardPage... let me make the change and test if it doesn't broke anything and i will be pushing it in a few minutes12:05
DiegoSarmenteronessita, meanwhile... is syncdaemon working? i keep receiving an error when i try to sync the folders in the installer12:05
nessitaDiegoSarmentero: no two different SSOWizardPage... but 2 different base class and having mulitple inheritance?12:06
nessitaDiegoSarmentero: what error?12:06
ralsinagood morning12:06
DiegoSarmenteroralsina, good morning12:07
nessitahi ralsina12:07
DiegoSarmenteronessita, http://paste.ubuntu.com/658606/12:07
nessitaralsina: any news regarding the review I sent to you yesteday?12:07
nessitaDiegoSarmentero: is syncdaemon running in that machine?12:08
ralsinanessita: finally I followed your advice and took the night off, so starting it now12:08
DiegoSarmenteronessita, i'm running the sso-client without any problem, but when syncdaemon keeps giving me this error: http://paste.ubuntu.com/658608/12:10
nessitaDiegoSarmentero: the sso you're pointing to syncdaemon is ols?12:11
DiegoSarmenteronessita, i updated it yesterday12:11
nessitaDiegoSarmentero: try this:12:11
nessitaset PYTHONPATH=..\ubuntu-sso-client12:12
nessitafrom ubuntu_sso.main.windows import UbuntuSSOClient12:12
nessitaand let me know how it goes12:12
DiegoSarmenteronessita, ImportError: No module named ubuntu_sso.main.windows  (I'm inside ubuntuone-client)12:15
DiegoSarmenteronessita, wait!....12:17
DiegoSarmenteronessita, nevermind.......12:17
DiegoSarmenterofound the problem12:18
nessitaDiegoSarmentero: what was it?12:20
DiegoSarmenteronessita, it was pointing to another branch of sso-client :(12:20
oimonhello. i have a folder in ubuntu one folder which is showing the refreshing icon, but u1sdtool thinks it is up to date. website doesn't show the file - how can i fix?12:35
faganduanedesign: ^12:44
ralsinanessita: +1 on the epic branch!12:44
nessitaralsina: great! thanks!12:45
duanedesignhello oimon12:45
oimoni also have other folders that i've copied into u1 folder that do sync with the website but still show refreshing icon on the folder, even when it's synced12:45
ralsinaI mean +1 as soon as I finish reading the test changes, but the main stuff looks great12:45
duanedesignoimon: what do you get when you run the Terminal command: u1sdtool --waiting | wc -l12:46
nessitaralsina: is ok, I still need to wait for alecu12:46
oimonduanedesign: i'm on lucid12:46
oimonwaiting-content and metadata are both 0 lines12:46
duanedesignoimon: ok12:46
ralsinanessita: could you take a look at https://code.launchpad.net/~ralsina/ubuntu-sso-client/validate-harder/+merge/70320 ? It's very boring ;-)12:46
nessitaralsina: sure12:47
oimonduanedesign: my home dir is mounted over NFS, not sure if that is relevant at all12:48
nessitaralsina: in _validate, shouldn't you call is_correct_password?12:48
ralsinanessita: in which one?12:48
nessitathe first one12:48
duanedesignoimon: could you check if this file has anything in it: ~/.cache/ubuntuone/log/syncdaemon-exceptions.log12:48
ralsinanessita: I am not calling intentionally, because suppose someday we change the password requirements, that would mean you can't login with the old password12:49
oimonduanedesign: nope, the exceptions log files are all 0 butes12:49
ralsinaoh, wait, that one is the forgtten password. Yes, on that one I should12:50
nessitaralsina: is that validation for login or forgotten password?12:50
beunonessita, ralsina, that is already the case. My existing SSO password doesn't conform to the nre guidelines12:50
nessitaah, that12:50
beunowe did that in Android and had to revert it12:50
duanedesignoimon: ok. can you check what you get with the command:  u1sdtool -s12:50
nessitabeuno: right, for login it makes no sense, but for password change it does12:50
ralsinanessita: tke it back, the first one is "Sign In", on that one we should not call it.12:50
duanedesignoimon: and pastebin the contents at http://pastebin.ubuntu.com/12:50
ralsinanessita and that is there, in line 12412:51
nessitaralsina: ok. Another thing, suffixing number in variable/method/test cases is not recommended since when reading, is confusing what does it mean. Can you  please change CurrentUserControllerTest2 to something more explicit? probably CurrentUserControllerValidationTestCase12:52
duanedesignoimon: hmm. ok, could you please post the syncdaemon log ~/.cache/ubuntuone/log/syncdaemon.log12:54
oimonduanedesign: here's a summary of the situation. copied 1 folder (.liferea_1.6) from another pc with the same home dir mounted via NFS. this doesn't appear on website. then on my main PC , copied "building" and ".liferea_1.2" which do appear on website but still show refresh icon as per http://i.imgur.com/zkJ0c.png12:54
ralsinanessita: Now it's about validation but I want to do any new tests there instead of with mocker. I'll figure out a generic name.12:55
nessitaralsina: ideally you should have a dedicated testcase for each "grouping" of functionality being tested. Is common to group into a validation tests vase12:55
nessitatest case*12:55
nessitaralsina: so I would advice a validation test case and then add any other needed test case12:55
nessitawith a accurate name12:56
ralsinaCool, will do that then12:56
duanedesignoimon: yeah the nautilus integration in Ubuntu One is less then ideal in Lucid. So the emblems are sometimes not very accurate12:57
oimonyeah i've also had nautilus crashes with u1 too12:58
oimonduanedesign: so back to the original issue, could it be because i copied the folder while on another machine? u1 should still poll the directory though?12:59
nessitaralsina, alecu, dobey, DiegoSarmentero, mandel, fagan: standup!13:00
nessitaralsina: I know no mandel today, but we do have an alecu today, right?13:01
nessitaDiegoSarmentero: have you tried to register "gato" as your irc nickname?13:01
ralsinanessita, AFAIK yes13:02
nessitaDiegoSarmentero: you current one is too long to type it in the daily stand up!13:02
DiegoSarmenteronessita, no... i will try! :P13:02
nessitaDONE: finished and proposed epic branch13:02
nessitaTODO: hunt reviews for epic branch, do reviews for Diego and ralsina, propose old branch with controlpanel ui improvements13:02
nessitaBLOCKED: nopes13:02
nessitaNEXT: DiegoSarmentero13:02
DiegoSarmenteroSome refactoring in the use of Title and Subtitle13:02
DiegoSarmenteroFinish Pages: 18 and 19 from Wizard. Fix UI bugs from installer13:02
DiegoSarmenterodobey, go13:02
dobeyλ DONE: yet more shim work13:03
dobeyλ TODO: finish initial shim13:03
dobeyλ BLCK: None.13:03
dobeyralsina: go13:03
ralsinaDONE: half day off, windows call, reviews, proposed a couple of branches. TODO: IRL testing and bug reporting, BLOCKED: no13:04
ralsinafagan go13:04
fagan* more report for college13:04
fagan* Finish the report13:04
fagan* If anyone needs anything do that13:04
fagan* nope13:04
nessitais cooold!13:04
faganIts hot in ireland13:04
faganor maybe its the flu im coming down with13:05
=== DiegoSarmentero is now known as gato
fagangato: new nick?13:05
gatofagan, yep... shorter... and it is an usual nickname for me :P13:06
fagangato: well everything is just letter tab so it doesnt really matter about length13:06
nessitagato: congrats on the nickname! (?)13:07
gatonessita, jejeje13:07
dobeygato gordo? :)13:07
faganOk thats what she said (in honor of mandel)13:07
faganabout my last statement13:07
alecuhello #ubuntuone!13:08
gatodobey, trying not to be fat!! :P13:08
gatohello alecu13:08
alecusorry I'm late; me (no notes)13:08
* fagan break 13:08
nessitagato: so, can you please refresh  my memory regarding what we talked in the plane about setting properties to widgets?13:08
nessitaalecu: go!!!13:08
alecuhola gato! got new nick?13:08
gatoalecu, yep :P13:08
gatonessita, yes, you can do it in the designer in the property editor.... or just through code: setProperty("stringId", value)13:09
alecuDONE: worked on bug #820350, perhaps found a simple solution13:11
alecuTODO: discuss a better solution for the above bug, review new issues with bug #80367213:11
alecuBLOCKED: no13:11
ubot4Launchpad bug 820350 in ubuntuone-client "WIndows: either set_dir_readwrite and/or set_dir_readonly are not doing what they should (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/82035013:11
ubot4Launchpad bug 803672 in ubuntuone-client (Ubuntu) (and 2 other projects) "Do not start two instances of this process (affects: 2) (heat: 22)" [Undecided,Invalid] https://launchpad.net/bugs/80367213:11
alecunessita, still needing the epic review?13:11
nessitaalecu: yes please, a lot13:12
* alecu rolls up his sleeves13:12
nessitaalecu: thanks!!!13:12
=== m_conley_away is now known as m_conley
=== DiegoSarmentero is now known as gatox
* gatox sad... gato was already in use13:20
nessitagatox: chem, help. How can I set in a stylesheet something equivalent to self.setLayoutDirection(QtCore.Qt.RightToLeft)?13:22
* gatox looking...13:23
gatoxnessita, why you need to do that in qss?13:24
fagangatox: sounds sharper13:24
gatoxfagan, :P13:24
gatoxnessita, i think you can't do that with stylesheet :S13:25
nessitagatox: no problem then13:25
rogerk_duanedesign: I was dragged into a rather lengthy meeting, but now I'm available. Are you?13:34
gatoxralsina, do yu have a minute?13:35
ralsinagatox: sure13:36
gatoxralsina, is there any reason why "SuccessController" in gui.py (sso-client) doesn't extend from "BackendController" as the others?13:36
gatoxjust to know.... because it will be better for something that i'm testing to make all of them extend from BackendController13:37
ralsinagatox: probably just a mistake13:37
gatoxralsina, great...... i will modify it! :D13:38
duanedesignrogerk_: yes13:43
duanedesignrogerk_: what version of Ubuntu are you on?13:43
rogerk_duanedesign: 11.0414:02
nessitaralsina, alecu, Chipaca, gatox: shall we mumble?14:03
gatoxnessita, ack14:04
nessitaalecu: che, I can't make syncdaemon start, I'm getting a CannotListenError (http://pastebin.ubuntu.com/658660/). I don't have any other instance running, and I never installed a installer in this vm14:05
nessitaalecu: any ideas?14:05
alecunessita, trunk_14:05
alecunessita, trunk?14:05
nessitaalecu: my branch, let me confirm trunk14:06
alecunessita, this looks suspicious: self.dbus_iface.shutdown(with_restart)14:06
alecunessita, that's from the paste.14:06
alecunessita, very likely that shutdown is not stopping the ipc listener before restarting14:07
alecunessita, do you know why is syncdaemon trying to restart?14:07
nessitaalecu: the dbus_iface is code located in ubuntuone/platform/windows/__init__.py14:08
duanedesignnessita: could you take a look at this https://pastebin.canonical.com/50744/14:08
alecunessita, right. But did you tell syncdaemon to restart?14:08
duanedesignnessita: do you know what package is missing14:09
alecunessita, look at this, also from your paste: self.main.restart()14:09
nessitaalecu: not at all, I'm looking at the logs to see what happened14:09
nessitaalecu: there was a SYS_UNKNOWN_ERROR pushed to the state machine, which triggers a restart14:10
duanedesignrogerk_: can you run the command:  u1sdtool --waiting | wc -l14:10
nessitabecasue there was a transition Transition SET_CAPABILITIES --[SYS_SET_CAPABILITIES_OK]--> AUTHENTICATE, which makes no sense!14:11
nessitaalecu: I'll debug further, sorry for pinging you so early14:11
nessitaduanedesign: looking14:11
alecunessita, I don't understand that, but do you know how we can "simulate" this to reproduce this error?14:11
nessitaalecu: the restart is pretty easy to simulate in a test14:12
nessitaalecu: this particular transition makes no sense14:12
alecunessita, anyway, it's very likely that restarts are broken at the ipc level.14:12
nessitaduanedesign: what system is that?14:12
nessitaalecu: Oh. Syncdaemon uses restart when in despair14:12
nessitaalecu: in trunk, syncdaemon will not move from READY, even if the SSO service returns the credentials :-/14:14
duanedesignnessita: they are running 10.10, but i think they might be using a PPA version of U114:22
nessitaduanedesign: looks like they are missing the gir-1.<something>-unity-<something> package14:27
rogerk_duanedesign: result i 0.14:27
nessitaralsina: so, are we mumbling today?14:30
duanedesignrogerk_: have you tried restarting the syncdaemon to see if that helps. the command would be"  u1sdtool -q; u1sdtool -c14:32
nessitaralsina: ping?14:34
nessitaalecu: in trunk syncdaemon is also transitioning from [SYS_SET_CAPABILITIES_OK]--> AUTHENTICATE :-/14:34
nessitaalecu: is it working for you/14:34
ralsinasorry, was afk14:35
ralsinaI don't think we have much news since yesterday, do we/14:35
alecunessita, let me check14:35
nessitaralsina: we can skip the mumble, but I want to be sure you and Chipaca know that we're getting behind schedule14:36
nessitaralsina: and that ATM, syncdaemon is not working for me, still debugging why14:36
nessitaalecu: actually, the problem is not that transition but the SYS_UNKNOWN_ERROR taht is pushed after that14:37
ralsinanessita: I know we are behind, I will make sure Chipaca knows too14:37
ralsinaBTW: on linux: ['/home/ralsina/.config/ubuntuone', '/etc/xdg/ubuntuone'], on windows: ['/.config\\ubuntuone']  yikes14:37
nessitaralsina: one of the things that worry me most, today, is the thing I mentioned about events out of order/missing/etc in the email from eysterday14:38
ralsinaYes, that is really bad, and new :-(14:39
nessitaralsina: well, is new becasue before the fsm tests were not run because all the issues with threads that alecu and mandel fixed last week14:43
alecunessita, found the issue in trunk:14:43
ralsinanessita, the config file location problems for syncdaemon are because xdg is (as usual) broken for windows. I am going to reimplement some pieces of it like you did for xdg_cache_home14:43
alecu2011-08-04 11:38:39,907 - ubuntuone.SyncDaemon.ActionQueue - INFO - The request14:43
alecu'oauth_authenticate' failed with the error: oauth_authenticate() takes exactly 314:43
alecu arguments (4 given) and was handled with the event: SYS_UNKNOWN_ERROR14:43
nessitaalecu: yeah, I just saw that too :-/14:43
nessitaI think there was an update i storage protocol14:43
nessitathat I don't have14:43
nessitaindeed: Now on revision 136.14:44
alecunessita, yup, it looks like that.14:44
nessitaralsina: there was a new update to storage protocol, and is mandatory for u1client to work, FYI (for the installer)14:45
nessitaalecu: thanks a lot :-)14:45
alecuthis sucks14:45
nessitaalecu: how's the review going?14:45
nessitaralsina: same question ^14:45
alecunessita, almost there14:45
ralsinaSo we all need to update ubuntuone-storage-protocol?14:45
nessitaralsina: and re build and re install14:45
ralsinanessita: I gave you the +1 a while ago, forgot to mention it14:45
nessitagatox: ^14:45
nessitaralsina: thanks!14:45
alecunessita, why re-build?14:46
gatoxnessita, update storage?14:46
gatoxor something else?14:46
gatoxahhhh ok.....14:46
nessitaalecu: python setup.py build install?14:47
alecunessita, I think that just "python setup.py install" in ubuntuone-storage-protocol should be enough14:47
nessitaalecu: maybe, I never knew exactly what's build for :-D14:47
alecunessita, I think that "install" depends on "build", and does it automatically if needed.14:47
nessitaalecu: is working now!!! :-)14:50
ralsinanessita, alecu: basically every line in xdg.BaseDirectory is broken. Should I just rewrite it?14:54
alecunessita, really? got a pointer to the code?14:55
alecuI mean, ralsina.14:55
ralsinaI can't copy it into our tree because of copyright assignments, right?14:55
alecuralsina, I think we can't, right.14:55
nessitaralsina: we can't14:55
nessitaralsina: why every line is broken?14:55
ralsinaalecu: let me pastebin it14:56
nessitaralsina: syncdaemon only uses14:56
nessita     30     load_config_paths,14:56
nessita     31     save_config_path,14:56
nessita     32     xdg_data_home,14:56
nessita     33     xdg_cache_home,14:56
nessitaralsina: so we need only that14:56
ralsinaalecu, nessita: http://pastebin.ubuntu.com/658675/14:56
ralsinabecause everything assumes it's linux14:56
ralsinaand it's not that big beyond what syncdaemon uses. I would reimplement just those bits14:56
ralsinait uses the wrong separators,  home by default is "/" (literally)14:57
nessitaralsina: it does not look sooo-linux specific, it uses os.path.join14:57
nessitathis is home: os.environ.get('HOME', '/')14:57
alecunessita, it14:58
alecuit's pretty much linux specific:14:58
ralsinanessita: all the paths by default have "/" in them14:58
ralsinayes, on windows there is no $HOME14:58
ralsinait splits paths using ":" instead of os.PATHSEP14:58
alecuand also it's getting everything from the XDG_* env vars, that will not be set on a windows.14:59
nessitaalecu, ralsina: all that is already fixed in our implementation, we only need load_config_paths to work14:59
nessitaalecu: we have all that fixed already14:59
ralsinain windows I have to do like get_special_folders. So I don't think any of the parts we use is actually correct for windows. It's not big though15:00
alecunessita, ok. where is that? already in our tree?15:00
nessitaand perhaps save_config_path15:00
alecunessita, or in mandels +junk?15:00
nessitaalecu: yes, platform/xdg_base_directory15:00
ralsinaload_config_paths uses everything else15:00
ralsinanessita: so yes, I will add those two there15:01
nessitaralsina: I forgot about you review after I mentioned the name change for the testcase, restarting it now. Can you please add an ending dot to  """Tests for CurrentUserController, but without Mocker"""15:02
CardinalFangTracking Oneiric has been the most painful dogfood-eating release cycle I've seen yet.  :(15:10
ralsinanessita: pushed the "."15:11
nessitaralsina: all the '.'? I see that several tests cases have the same docstring15:11
ralsinaje, no, just the one you mentioned, will fix that in 1'15:12
nessitaralsina: I added a couple of needs fixing and a question15:13
ralsinanessita: cool, thx15:14
alecunessita, why did you merge test_deleting_open_files_is_no_cause_for_despair and test_stomp_deleted_file_is_no_cause_for_despair_either ?15:17
alecunessita, nevermind, it's the same test :P15:18
nessitaalecu: is the same test! :-)15:20
nessitaalecu: I confirmed it with facu15:20
dobeygah. found a nice bag, but it's apparently only available in australia15:21
nessitaalecu: At first I tried to make it work on windows, but I couldn't, so I just made it more readable and skipped it15:21
nessitadobey: bug or bag? :-D15:21
dobeynessita: camera/laptop bag :)15:21
nessitaah! :-P15:21
ralsinanessita: about the needsfixing, unicode and QStrings compare correctly15:25
nessitaralsina: then why are we coercing to unicode everywhere?15:26
nessitaralsina: either we do it everywhere, or nowhere, otherwise confuses the reader15:26
ralsinaQStrings are not unicodes in the sense that you can't use them interchangeably, but you can say if QString == unicode and it does the right thing15:27
ralsinaBut sure, I will add it, just mentioning it15:27
dobeyok, must get lunch. bbiba15:36
gatoxralsina, nessita i've finished with title and subtitle... i've applied several improves in the code i've  just submitted... i'm waiting to launchpad to rescan my branches to give you the url... i'll keep working on pages 18 and 19 now15:39
nessitagatox: ack!15:39
ralsinagatox: cool!15:40
ralsinagatox: I just noticed we have a SSOWizardEnhancedEditPage and a SSOWizardPage15:42
ralsinaI think we should merge them15:42
gatoxralsina, maybe... it seems that SSOWizardEnhancedEditPage is only being used by SetupAccounnt15:42
gatoxralsina, let me check.... and i will apply that change in this branch too if you want15:43
ralsinagatox: it should be used by every page that has EnhancedLineEdits, and we should use those for all line edits that need validation15:43
ralsinaI think that's for another branch not this one :-(15:43
gatoxralsina, ok! better15:44
alecunessita, a couple of very small need fixings in https://code.launchpad.net/~nataliabidart/ubuntuone-client/fix-tests/+merge/7037315:44
* alecu breathes again15:44
nessitaalecu: looking!15:47
gatoxnessita, ralsina both branches are ready for review: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/titles-on-pages/+merge/70169 --- https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/titles-on-pages/+merge/7015715:49
ralsinanessita: responded to your needfixings and fixed them, too.15:49
ralsinagatox: cool!15:50
gatoxnessita, at the end of installer changes test_gui.py it appears to have some merge problems... but that changes are not in the file if you download the branch... it might be some merge problem with trunk15:50
gatoxi'm looking if i can fix it locally and upload15:51
gatoxnessita, ralsina merge problems fixed... the branches are complete15:55
ralsinagatox: the sso branch is much nicer now15:56
gatoxralsina, :D15:57
ralsinagatox: I think we should make BaseController take the optional messagebox, title, subtitle arguments and handle them. They are always used the same, anyway15:57
gatoxralsina, ok....... i will remove it from the specialized controller and pass this arguments to the parent in the super statement15:59
nessitaalecu: I used it the or returning a lambda for consistency sake, is how facundobatista does it in other tests. I agree is aweful.16:01
nessitaalecu: not sure if preserve consistency or improve some of those only16:02
ralsinagatox: and of course if the __init__ is now just a super, remove it ;-)16:02
ralsinagatox: Then, since the controller always will have a title/subtitle, maybe the SSOWizardPage can just use it and the controller doesn't have to call set_title and whatever.16:03
gatoxdo you mean moving the initialization of title and subtitle from controller to SSOWizardPage??...... because BackendController is a controller... so it will work as usual in that way16:05
gatoxralsina, ^16:05
gatoxralsina, but some controllers are using the set_title, because they have another title based on validation or something like that16:07
=== beuno is now known as beuno-lunch
ralsinagato: ok, then how about moving it into the BaseController's __init__? If a specific controller wants to set them different, it does it after super()16:08
ralsinagatox: ^16:09
gatoxralsina, ok... it almost ready... testing...16:10
ralsinanessita: use a lambda that return a tuple instead. Slightly cleaner...16:14
nessitaralsina: we don't need two results, but an append and a defer returned. I know how to fix it, I'm not sure I want to break consistency with other tests, but I guess we have to16:15
ralsinanessita: oooook, missed that. If the other tests are the same I say leave it as is.16:16
gatoxralsina, I can't move set_title and set_subtitle from setupUI in controllers to __init__ in BackendControllers, because the header is created after the controller16:16
ralsinagatox it's ok then16:17
gatoxralsina, i could do it maybe in initializePage16:17
gatoxralsina, what do you think?16:17
gatoxralsina, ah..... no, same problem.....16:17
gatoxbecause the controllers are overwriting that method16:18
ralsinanah, we have 3 "sort-of-line-init" methods, leave it as it is16:18
ralsinathe controllers are overwriting initializePage?????16:18
gatoxSSOWizardPage is overwriting initializePage..... and calling pageInitialize in the controllers...16:20
gatoxralsina, we made this last week16:20
gatoxralsina, do you remember? to handle default buttons16:20
ralsinaoh yeah, that16:21
ralsinaI just had a code cache overflow :-)16:21
gatoxralsina, i'll try to put set_title there.... just one controller is overwriting this method... do you agree?16:21
ralsinano, let's stop fiddling with this branch, I take that objection back ;-)16:23
gatoxi will submit the first modification.... that it is really small..... and its working properly......16:24
nessitaalecu: re: lambdas with ors, would you prefer maintain consistency or change some of them?16:25
alecunessita, +1 to fix the new ones, and fix the others later.16:26
gatoxralsina, done... both branches are updated.16:28
ralsinagatox: looking...16:30
ralsinagatox: +1 with a little star on top16:31
gatoxralsina, :D16:32
nessitaalecu: everything fixed and pushed16:35
nessita(and re-scanned by LP)16:36
alecunessita, cool16:37
=== beuno-lunch is now known as beuno
ralsinanessita, alecu, gatox: I am going to change our windows-xdg thing so that it considers these as xdg_config_dirs: ['C:\\Users\\ROBERTO\\AppData\\Local', 'C:\\ProgramData']17:11
nessitaralsina: you sure about the last one?17:11
alecuralsina, cool. How are you going to build those paths?17:11
alecuralsina, are you using some win32 apis?17:12
nessitaralsina: didn't gatox mentioned something about c:\\allusers\\ for config?17:12
alecuor getting them from the registry, or something?17:12
ralsinaThat seems to me like the common practice, according to the ton of apps I have installed17:13
ralsinanessita: it seems all apps put stuff there (including ubuntuone-protocol17:16
ralsinaalecu: I got them using win32com, just like we were getting them before17:16
nessitaralsina: ok then :-)17:17
ralsinac:\users\allusers\appdata seems to be a ""'link" to c:\programdata17:17
alecuralsina, I saw that. Seven uses much saner directory names than previous versions, but it puts links around so old paths can still be used.17:18
ralsinaalecu: I am getting "CSIDL_COMMON_APPDATA" which should point to logical places on all versions17:19
ralsinaanyone has a XP handy to run 3 lines of python?17:20
alecuralsina, perhaps fagan does?17:21
ralsinaalecu: yes, but we don't have a fagan until tomorrow :-)17:24
ralsinaAnyway, it's supposed to work n Windows millenium and higher ;-)17:24
ralsinanessita, alecu, gatox: small branch for review: https://code.launchpad.net/~ralsina/ubuntuone-client/find_config_files/+merge/7047217:41
gatoxralsina, on it17:49
ralsinagatox, thanks!17:50
nessitagatox: can please refresh my memory: how can I load the qss in the designer?17:55
gatoxnessita, not load.... you press right click in the dialog/widget (whatever) and choose "change styleSheet" option17:57
gatoxnessita, and you can play around with that17:57
nessitagatox: but can I load the stylesheet from the qss in disk?17:58
gatoxnessita, honestly... i don't know... i've never done that..... but you can copy/paste all the content :P17:58
gatoxnessita, i assume this is just for testing17:59
nessitaack, thanks17:59
gatoxnessita, it's kind of ugly (from my point of view) leave qss styles in the ui17:59
nessitagatox: of course, I wanted the qss to load visually, not to modify any xml17:59
gatoxralsina, +1 (review)18:03
ralsinagatox: thanks!18:04
nhainesralsina: I can run some Python on WinXP SP3 if you need.  :)18:04
ralsinanhaines: thanks for the offer but it would probably involve a couple of hours of setup before you can run them ;-)18:05
nhainesralsina: oh, best not then... they expect me to get *some* reports out today at work.  ;)18:06
nessitagatox: have handy the doc for the button "build" guide?18:09
gatoxnessita, ?? from qt assistant for style sheet or you are talking about something else?18:10
nessitagatox: I'm talking about the spec for styling our buttons :-) (sorry for being so cryptic)18:10
gatoxnessita, jeje..... let me check18:10
faganralsina: whats up, I can give you a little time if its important I dont mind18:10
* gatox searching...18:10
ralsinafagan: cool!18:11
ralsinafagan: let me do a pastebin of the python code I need running on XP18:11
faganralsina: Oh crap ill have to do it tomorrow I broke my ubuntu install that that VM is on and forgot to fix it18:11
faganralsina: paste it and ill say if its working for when you guys get up tomorrow18:12
* fagan fixed one ubuntu but not the other 18:12
ralsinafagan: http://pastebin.ubuntu.com/658780/18:14
faganralsina: cool whats the expected output?18:14
gatoxok people... i'll take a break (i'm here since 7 :P)... i'll be back around 17/18 to try to close some installer stuff if anyone needs anything18:14
fagangatox: enjoy :)18:15
ralsinafagan: surprise me ;-) (should be a list of paths)18:15
faganralsina: cool18:15
ralsinanessita, alecu: if I could get a second review of https://code.launchpad.net/~ralsina/ubuntuone-client/find_config_files/+merge/70472 -- I checked it IRL, syncdaemon can now start without passing config files in the command line with that patch18:29
nessitaralsina: the branch needs tests...18:31
dobeytrivialish: https://code.launchpad.net/~dobey/ubuntuone-installer/launch-cp/+merge/7047918:33
ralsinanessita: I will do them. I note that I am replacing something that had no tests and was brokn with something that has no tests but works, though ;-)18:35
ralsinafagan: got the results yet?18:35
nessitaralsina: the thing that had no tests is a 3rd party lib. And I thought we agreed that every single piece of code *we* add, will have tests18:36
ralsinanessita: yes, we agreed, and I slipped.18:36
ralsinaI will add tests for the kazillion other things in that module that don't have it while I am at it18:37
nessitaralsina: *thanks*18:38
alecunessita, your branch is making tarmac unhappy. see: https://code.launchpad.net/~nataliabidart/ubuntuone-client/fix-tests/+merge/7037318:43
nessitaalecu: looking18:44
alecunessita, it looks to me that tarmac is not set to a utf8 locale18:44
nessitadobey: help? ^18:44
alecunessita, try "unset LANG; unset LANGUAGE; u1trial tests/syncdaemon/test_vm.py" and you get the same error that tarmac throws18:45
alecunessita, the thing seems like a bug in the branch18:46
dobeyare you using translated strings in tests?18:46
nessitadobey: nopes18:46
alecunessita, the branch is sending unicode to the os functions on linux18:46
nessitadobey: but I'm using non ascii chars18:46
nessitaalecu: right, and that is intended18:46
alecunessita, why?18:46
dobeywhy are you doing that?18:46
nessitaalecu: did you read the comment in the MP?18:46
alecunessita, it should be sending bytes on linux18:46
alecunessita, because if not, then we get this kind of errors18:47
nessita78+        # path is a byte sequence encoded with utf8. If we pass this to os.walk,18:47
nessita79+        # in windows, we'll get results encoded with mbcs18:47
nessitaalecu: we need unicode becasue of that ^18:47
alecunessita, on windows, it makes sense, but on linux we can't18:47
nessitaalecu: if we pass bytes to os.walk we'll get other encoded bytes as result18:48
nessitaalecu: that makes no sense... how come we can't pass unicode to os.walk?18:48
alecunessita, the native way to use os.* on linux is by passing bytes18:49
nessitaalecu: what if we have a latin1 linux FS? syncdaemon will transform that to utf, and if we pass that to os.walk...18:49
alecunessita, if we pass unicode, python will *try* to convert it to bytes before calling the operating system18:49
dobeyalecu: either should work18:49
dobeynessita: latin1 probably won't work18:49
alecudobey, no: it depends on the os encoding18:49
dobeythe problem is that you can't convert utf8 to ascii, unless you're only using ascii characters18:50
alecunessita, and when python tries to convert unicode into "ascii" bytes, we get that error.18:50
alecudobey, wrong, that is not *this* problem.18:50
alecudobey, this is "unicode" to ascii18:51
dobeyexceptions.UnicodeEncodeError: 'ascii' codec can't encode character u'\xf1' in position 161: ordinal not in range(128)18:51
dobeyalecu: that is what i just said :)18:51
nessitaalecu: so you're saying that python is trying to encode to ascii and not to utf8?18:51
alecudobey, you said "utf8"18:51
alecunessita, right. It's trying to convert unicode to the default encoding18:51
alecunessita, since LANG is not set, the default encoding is ascii18:52
alecunessita, and that is *right*18:52
nessitaalecu: then we need our defaults encoding to be utf8, included tarmac18:52
alecunessita, no18:52
nessitaalecu: why is right? :-)18:52
nessitadobey: unicode != utf818:52
dobeyyes i know that18:53
alecunessita, because we'll get the same error if the user has other encoding18:53
alecunessita, it's ok to have another encoding18:53
nessitadobey: ah, from your last sentence I thought you were mixing them, sorry18:53
nessitaalecu:  but this is a tearDown...18:53
dobeyeh, it's not my fault windows is the red headed step child that it is18:54
nessitaalecu: you propose is better to have an ifplatform there instead of us, devs, have default encoding to utf8?18:54
nessitaalecu: I will trust you on this one, of course18:54
alecunessita, right. I propose we stick to the convention18:54
alecunessita, the convention being "os.* gets unicode on windows, bytes on linux"18:55
dobeywhy would os.* get unicode on windows? is that a convention we're enforcing ourselves? i thought python2.x used bytes by default everywhere, and python3 uses unicode by default18:55
alecudobey, os.* needs unicode on windows, otherwise it starts doing funky stuff. Like os.listdir returning a list where some items are str and some are unicode.18:56
dobeywell, if you know how to fix it, i'll let you fix it then :)18:58
nessitaalecu: fixing18:58
nessitaalecu: we may consider running the whole tests suite with lang and language unset18:58
alecunessita, it seems that tarmac already does that. Or it uses a LANG with no ".UTF-8" appended.19:01
dobeyit doesn't use a lang19:01
dobeyit's run from cron, which basically has no env19:01
dobeythe problem is developers expecting everyone else's env to be exactly like theirs ;)19:04
ralsinagotta go put a contract in the mail, will be back in about one hour19:14
dobeyyou shouldn't use mail for that. it's traceable19:19
alecunessita, do you need a re-review on the "bytes on linux" change?19:32
nessitaalecu: is pushed, I was waiting for LP to rescan to approve19:32
nessitaalecu: but yes, a re-review is welcomed!19:32
dobeynessita, alecu, thisfred: quick, pretty trivial review on https://code.launchpad.net/~dobey/ubuntuone-installer/launch-cp/+merge/70479 please? :)19:35
nessitadobey: I can't say no again ;-)19:37
nessitadobey: looks good! Question, does ubuntuone.installer.gui import Window exists?19:38
nessitadobey: also, how are we handling the fact that we're moving to the -qt control panel? I see the gtk version 'hardcoded'19:39
alecunessita, tested with LANG unset, and approved again.19:40
dobeynessita: yes; define "moving to" more explicitly. is there a timeline for making those changes on Ubuntu? because the qt stuff isn't even packaged yet19:40
dobeynessita: and i was thinking we'd do some stuff to make it work with alternatives or something, so that either would work19:41
nessitadobey: right, it won't happen in ubuntu for oneriric, but the plan is to happen soonish. So, I was thinking, if this is the entry point for every ubuntu version to have the latest, are we needing a SRU to have older ubuntu running the qt version?19:42
dobeynessita: it is hardcoded right now, to make stuff work. there will be many changes in the next couple of weeks leading up to UI freeze, but for now, making it work is priority :)19:42
nessitadobey: ah, that explains it19:42
nessitadobey: ok, approving19:55
nessitaalecu: thanks!19:56
dobeynessita: thanks20:01
nhainesI can see the omgubuntu headline now.  "Ubuntu One CP moving to QT... in time for oneiric?"  And then the copy says "so far there are no plans to have this included in oneiric, but who can tell what the future will bring?"20:02
karniirk3z: osom20:08
ralsinanessita, got 2' for a quick mumble?20:28
ralsinanessita: lost you in "pongamos que el control panel"20:39
nessitadobey: seems like tarmac may be in that state were it stops landing sutff20:47
nessitadobey: my branch is approved 49 minutes ago20:47
dobeyoh, right, because it failed before. ok20:48
nessitaralsina: question re validate-harder:20:51
ralsinanessita: shoot!20:52
dobeynessita: should get picked up in ~8.5min20:52
nessitapylint complains with several lines like this:20:52
nessita    1161:  [E1103, ResetPasswordControllerValidationTest.test_invalid_confirm] Instance of 'FakeResetPasswordPage' has no 'confirm_password_line_edit' member (but some types could not be inferred)20:52
nessitaso I went to look to FakeResetPasswordPage and I see that the errors is "real" since FakeResetPasswordPage doesn't not have that attr but a:20:52
nessitaralsina: so, shouldn't the test call20:52
nessitainstead of20:53
ralsinanessita: indeed that's a bug20:53
nessitaralsina: is repeated a lot for several classes20:54
nessitaralsina: I can paste the whole output if you don't have a linux handy20:54
ralsinayes, copy& paste mistake :-)20:55
ralsinaI mean :-(20:55
ralsinaIt's ok, let me change it and push it20:55
ralsinaActually the tests work because I am using the same object for view and view.ui, anyway20:57
ralsinaso lint is actually wrong...20:57
dobeydon't look at me, i told you to use pyflakes20:58
ralsinaI get "has no 'email_edit' member (but some types could not be inferred)"20:58
ralsinanessita: ^21:00
nessitaralsina: and does it have an email_edit attr?21:03
ralsinanessita: indeed21:04
nessitaralsina: can you please ignore globally E1103 in that test file?21:05
ralsinaCheck FakeCurrentUserPage, for example. self.ui = self and then self.ui.email_edit = FakeLineEdit()21:05
ralsinaMaybe if I do self.fake_line_edit = FakeLineEdit() pylint will see it21:06
nessitaralsina: nah, just ignore globally the E1103. there are a couple of valid C0103 though21:06
ralsinanessita: ok, will look for those21:07
=== yofel_ is now known as yofel
ralsinanessita: pushed with the C0103s fixed21:14
nessitaralsina: running tests in windows and after that approving21:23
=== m_conley is now known as m_conley_away
ralsinanessita: cool, thx!21:23
nessitaralsina: are the tests in windows passing for you?21:25
nessitaI got 35 errors :-/21:25
ralsinaI get 2, about the captcha21:25
nessitaralsina: let me use a fresh branch JIC21:26
nessitaralsina: is your trunk up to date?21:27
ralsinanessita: it was a while ago, let me pull again21:27
ralsinanessita: no revisions to pull21:28
nessitaralsina: I really gotta go, I'll paste the errors and maybe you can help me see what's wrong with my env tomorrow morning?21:29
ralsinanessita: sure thing21:29
ralsinanessita: if you have any reviews left, I can do them in a bit21:29
nessitaralsina: those are the failing tests ^21:30
ralsinanessita: besides, they are in places unrelated to the changes, too21:31
nessitaralsina: I'll try to debug this tomorrow first thing. In the mean time, would you try running the tests in a clean branch of your branch, without setting any env var? just running make tests21:33
nessitarun-tests.bat, sorry :-P21:33
ralsinanessita: clean cmd, clean branch merged to trunk from launchpad, I get two or 3 failing tests, as usual. Here is a run with 3 failures: https://pastebin.canonical.com/50774/21:36
nessitaralsina: any ideas what can be run with  my env?21:38
ralsinanessita: not really, never seen that :-(21:39
nessita(FYI, this failure is not usual: ubuntu_sso.main.tests.test_windows.SSOLoginTestCase.test_emit_login_error, only the generate captcha ones)21:39
ralsinanessita: what's your PYTHONPATH?21:39
nessitaralsina: nothing21:39
nessitaralsina: the tets should take care of that21:39
nessitathe bat file set it, right?21:40
ralsinayes, sets it to "."21:40
nessitaok, I'm out of here, I'll continue tomorrow21:40
nessitabye all!21:40
ralsinabut it should not even need that21:40
ralsinajust in case, are you sure it's unset?21:40
nessitaralsina: yeap, I confirmed21:40
nessitaI'll re-check tom21:40
ralsinareboot! ;-)21:41
ralsina3 times!21:41
dobeymore triviality: https://code.launchpad.net/~dobey/ubuntuone-installer/le-appertif/+merge/7049922:08
ralsinadobey: got it!22:12
=== gato is now known as gatox

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!