=== eu is now known as Guest80206
Chipacaif you restart your video lens now, you should find a couple more sources in there02:21
Chipacajust sayin'02:21
Chipacaand now, to bed02:22
=== david is now known as davidcalle
JamesTaitHappy Friday, everyone! :)09:21
mandelralsina, ping?11:14
ralsinamandel: pong11:14
mandelralsina, morning!11:14
mandelralsina, do you know a widget similart to a GtkArrow?11:15
mandelralsina, I might hate Qt depending on the answer :)11:15
ralsinamandel: I don't know gtkarrow!11:15
ralsinamandel: the icon theme has arrows, maybe using that in a label? Where do you need an arrow?11:16
mandelralsina, ah, nice, there is a way to get the arrow from the theme, how do you do that?11:16
mandelralsina, is to follow the design of the ssl dialog11:16
ralsinamandel: can I see an image?11:17
mandelralsina, I'll foward the design11:17
mandelralsina, via mail11:17
ralsinamandel: ack11:17
mandelralsina, so, the problem is that there is AFAIK a widget like the GtkExpander on Qt, so I need to do my own, which might be a problem later in Windows, but I'll work around it later11:18
gatoxgood morning!11:18
ralsinamandel: you mean for the thing that expands to show the details of the SSL cert?11:19
mandelralsina, yes, is there a widget for that already that follows the design?11:19
ralsinamandel: to get icons from the theme: QIcon::fromTheme(…)11:19
mandelralsina, that I know, I don't know the id of the arrow hehe11:19
ralsinamandel: as soon as I see the design, I will wildly guess ;-)11:19
ralsinamandel: hahaha11:19
mandelralsina, good enough :)11:20
ralsinamandel: icon names: http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html#names11:20
ralsinamandel: so, the idea is that clicking on the arrow next to "Certificate details" will show the details? A QTreeWidgetItem and a lot of detailed styling ismy best guess. Ask for gatox's  opinion now that he's up11:22
ralsinamandel: forget it,it would not look right11:22
ralsinamandel: I need to think a bit more11:22
mandelralsina, the best idea I have is to have the label and when the label is clicked make the text area y the scrollable show11:23
mandelralsina, but I need to see the exact behaviour we want.. but that can be a bug fix later11:23
gatoxback again11:24
mandelralsina, from that page => Version 0.8, 1 August 2006, Rodney Dawes.11:24
ralsinamandel: I know11:24
mandelgatox, any idea how to implement a GtkExpander in qt?11:24
ralsinaI was going to tell you to ask dobey because I knew hewrote it then I saw the time ;-)11:25
gatoxmandel, let me see what a gtkexpander is :P11:25
gatoxmandel, the window has to be resize, or you only need to expand the content of a widget without the need to resize the windows in ordeer to fit the new content?11:27
ralsinamandel: http://qt-apps.org/content/show.php/QIrExpander?content=115342 but of course we can't use it :-(11:27
mandelgatox, the dialog overall expands..11:27
mandelralsina, gatox looks like I'll have to do a custom widget for this..11:28
ralsinamandel: do a composite11:28
gatoxmandel, yes..... but i already did one when we made the db_to_u111:28
ralsinamandel: three labels, clicking on the arrow makes the bottom one larger/visible11:29
ralsinamandel: the only "bad" thing is lack of animation but who cares11:29
gatoxthen we remove it, but i did something like that, with an animatoin to look pretty and everything11:29
mandelralsina, gnome people?11:29
mandelgatox, got the code around?11:29
ralsinamandel: it's the corner of a dialog noone will see11:29
gatoxmandel, it's in the first version of db_to_u111:30
ralsinamandel: remember: we *do* have a valid certificate ;-)11:30
gatoxmaybe not the first, but, something like that11:30
mandelralsina, the button one should be a scrollable area, since we want to have a max size for the dialog11:30
gatoxmandel, let me look for it11:30
mandelgatox, thx!11:30
ralsinaok, a textview or however it's called11:30
mandelralsina, yeah, that, I uderstood what you meant, I was going to that when I though about it, only ugly part, the arrow :(11:32
ralsinamandel: and I fixed that. I win! ;-)11:32
mandelralsina, how? there is not an icon with an arrow, is there?11:33
ralsinamandel: there used to be, IIRC, let me see11:33
ralsinamandel: media-playback-start11:33
mandelralsina, ugly hack! that is going to look terrible!11:34
ralsinamandel: yes, let me find something better11:34
gatoxralsina, mandel SP_ArrowLeft11:34
gatoxin QStyle11:34
ralsinamandel: QToolButton, check arrowType11:35
ralsinamandel: gatox's is better :-)11:35
mandelyeah for gatox !!!!!11:35
gatoxyou can use it in a button, like this: QPushButton(self.style().standardIcon(QStyle.SP_ArrowLeft), 'button text')11:35
mandelalso, should be pointing right, but we are there :)11:35
ralsinamandel: better: SP_ArrowForward11:36
ralsinamandel: that works even when the app is changed for RTL languages11:36
ralsinawe don't need it but is a good habit :-011:37
mandelralsina, gatox mierda, is not the correct arrow! http://ubuntuone.com/4O1utO8zUfuwYzuXMEXQ3O11:41
gatoxmandel, that arrow is going to change depending on the os theme11:41
gatoxbut is always going to be a right arrow of some kind11:42
ralsinamandel: I think what you need is not an arrow, it's the "expand" icon/decoration?11:42
mandelralsina, yeah, something like that11:42
ralsinamandel: in a QTreeWidget, is it the right one?11:43
ralsinaright as in correct11:43
mandelralsina, let me check11:43
mandelralsina, yes that looks very very close11:45
mandelralsina, I see some pixel diffs but I'm sure that is just me11:45
ralsinabecause you could use a Qtreewidget with a single row for the arrow and the label, but it's getting complex :-(11:45
mandelralsina, maybe QStyle::PE_IndicatorArrowRight11:46
gatoxmandel, ping11:47
mandelgatox, pong!!11:47
gatoxmandel, look at db_to_u1 rev 511:47
ralsinamandel: that's not an image, I think11:47
mandelralsina, its a primitive, but is the one used for the arrows, right?11:47
gatoxmandel, central_widget file, ExplorerFrame class, methods show_login, hide_login11:47
mandelgatox, ok11:48
gatoxmandel, execute that versoin to see if that is what you want11:48
mandelgatox, I think with that and some work using the primitive it will work11:51
gatoxmandel, great11:51
* ralsina walks himself12:13
ralsinawill be back in 30'12:13
gatoxnessita, hi12:16
nessitahola gatox!12:17
nessitaand hello world!12:17
gatoxnessita, do you have time for some reviews?12:17
gatoxalecu, hi12:18
nessitagatox: yes12:18
nessitagatox: I know I have the style one12:18
gatoxnessita, https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/931452  -  https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/sso-styling  -  https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/quota-warning  -  https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/reconnect-button12:18
nessitaon those!12:19
gatoxi've recheck the cp ones this morning and resolve a conflict.....12:19
nessitagatox: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/931452/+merge/93003 approved with comment, please address before globally approving12:50
gatoxnessita, ok12:50
gatoxnessita, sorry.... i'm using that in those methods.......12:52
gatoxor you mean only that and not the other asserts?12:52
nessitagatox: let me look at it JIC12:53
nessitagatox: you're not using the helper there12:53
nessitagatox: see diff lines 412 to 42012:53
nessitagatox: my point is use assert_backend_called instead12:54
nessitaoh, and I just got a test failue :-/12:54
nessitagatox: so I will change my vote to needs fixing until the test is fixes, pasting the error in the MP12:54
gatoxnessita, ok..... i'll check the test failure now..... but i don't understand the assert_backend_called part.... i'm seeing the line in that method that call to assert_backend_called....... or am i missing something?12:55
nessitagatox: in the line of the diff 412, what do you see?12:55
gatoxnessita, self.assertIn('validate_email_and_ping', self.ui.backend._called)12:56
nessitagatox: that is not assert_backend_called12:56
nessitaassert_backend_called is in line 41912:56
gatoxnessita, yes, assert_backend_called is at the end of the method..... that why i ask if you want me to remove the others asserts12:56
nessitagatox: well, they don't test something extra, do they?12:57
gatoxnessita, ok12:57
nessitagatox: is an honest question :-) do they?12:57
nessita(perhaps I'm missing something)12:57
gatoxnessita, i don't think so12:57
nessitaok, then I think is better to remove that12:57
gatoxnessita, ok, remove that.... and the test are ok now12:59
nessitagatox: ack... you ran the whole suite?13:00
gatoxnessita, qt part..... i'm in precise and i can't run the whole suite yet13:00
nessitagatox: why? you should be able13:01
nessitagatox: you have nightlies? everything up to date?13:01
gatoxnessita, i'm looking at that right now.... i need the gtk3reactor.....13:01
nessitagatox: what does apt-cache policy python-twisted-core says?13:03
gatoxnessita, python-twisted-core:13:04
gatox  Installed: 11.1.0-113:04
gatox  Candidate: 11.1.0-1ubuntu113:04
gatox  Version table:13:04
gatox     11.1.0-1ubuntu1 013:04
gatox        500 http://archive.ubuntu.com/ubuntu/ precise/main amd64 Packages13:04
gatox *** 11.1.0-1 013:04
gatox        100 /var/lib/dpkg/status13:04
gatoxdo i need nightlies for that too?13:04
nessitagatox: yes... but the simple fact of having nightlies ppa installed, should give you that13:05
nessitagatox: what does apt-cache policy ubuntu-sso-client says?13:05
gatoxnessita, ubuntu-sso-client:13:06
gatox  Installed: 3.1+r842-30~oneiric113:06
gatox  Candidate: 3.1+r842-30~oneiric113:06
gatox  Version table:13:06
gatox *** 3.1+r842-30~oneiric1 013:06
gatox        100 /var/lib/dpkg/status13:06
gatox     2.99.4-0ubuntu1 013:06
gatox        500 http://archive.ubuntu.com/ubuntu/ precise/main amd64 Packages13:06
gatoxcould it be that everything waas reverted in the upgrade?13:06
gatoxi KNOW i had nightlies for u1-client13:06
nessitagatox: how did you upgrade? (does not look like you are in P )13:06
nessitagatox: the package says "oneiric" :-/13:06
gatoxnessita, update-manager -d13:06
nessitagatox: did you add the ppa for precise?13:07
gatoxgatox@utopia:~$ lsb_release -a13:07
gatoxNo LSB modules are available.13:07
gatoxDistributor ID:Ubuntu13:07
gatoxDescription:Ubuntu precise (development branch)13:07
nessitagatox: can you please run: sudo apt-add-repository ppa:ubuntuone/nightlies; sudo apt-get update; sudo apt-get dist-upgrade13:07
gatoxnessita, but i'm in precise, look at that ^13:08
nessitagatox: yeah, the but you're not running precise nigthlies apparently13:08
gatoxnessita, ok..... running the repo-add, update, upgrade13:09
nessitagatox: in the controlpanel quota warning branch, you added the constant FREE_ACCOUNT_SIZE and is 5368709120. But the QUOTA_THRESHOLD_ACCOUNTS has:13:12
nessita{'free': 536870912, 'pay': 3221225472}13:12
gatoxnessita, reviewing....13:13
nessitagatox: also, when cutting an if guard in multiple lines, you need to indent the second line like this:13:14
nessitaif something_really_long and\13:15
nessita    please_indent_this_part:13:15
nessitagatox: also, any reason to remove the unicode from the RED constant? everything should be an unicode13:18
ralsinaWhoa, I am finally on precise for real! I have to reboot, which in my experience will take about 10 minutes because it fscks every time. Sowill be back soonish!13:18
mandelralsina, lol13:19
gatoxnessita, no, reverting..13:19
gatoxnessita, 536870912 is ok13:21
gatoxnessita, the FREE_ACCOUNT_SIZE is 5gb and the other one: QUOTA_THRESHOLD_ACCOUNTS is 0.5gb..... to ask if the available space is less than that13:22
nessitagatox: right, but isn't it clearer to use FREE_ACCOUNT_SIZE * 0.1? (ie the 10% of it)13:22
nessitagatox: my proposal is to use instead:13:23
nessita        FREE_ACCOUNT_SIZE = (BYTES**3) * 5  # 5 gigs13:24
nessita        QUOTA_THRESHOLD_ACCOUNTS = {'free': FREE_ACCOUNT_SIZE * 0.1,  # 10% of the free account size13:24
nessita                                    'pay': (BYTES**3) * 3}  # 3 gigs13:24
nessitagatox: so, next time someone reads that, does not have to do all the math we both just did :-D13:24
mandelok, time for my lunch13:24
gatoxnessita, mmmm maybe i'm wrong, but i don't think so, because that number is not a percentage is a exact number.... yes, it gives you the same result.... but we are inn the same situation as before that we were using percentage....... maybe we could add a comment above the quota_threshold_accounts variable13:25
nessitagatox: for free account, it's the 10%13:25
gatoxnessita, done13:37
nessitagatox: yey! and reconnect-button is approved13:37
gatoxnessita, great....... still downloading nightlies here.......  :S13:38
nessitagatox: JIC, did you saw the other comment I made in the quota warning MP?13:38
nessitadid you see*13:38
gatoxnessita, the unicode red one?13:38
nessitagatox: that also, but I also added a comment in the MP, in LP13:39
nessitaabout the indentantion13:39
gatoxnessita, yes..... i fix that too as we talk in your home once, aligning the second line where the first word not if or elif start13:39
gatoxis that ok?13:40
nessitaso the second line has to be aligned with the start of whatever is next to the if, let me give you an example13:40
gatoxyes, that's what i did13:41
gatoxnessita, ^13:41
nessitagatox: an example http://pastebin.ubuntu.com/845816/plain/, let me re-check what you did13:42
gatoxyes, i did that13:42
nessitagatox: ah, ok. So I suggested another way of putting that logic using variables, but no need to change13:46
nessitahaving the proper indentantion is enough13:46
gatoxnessita, wait!13:46
gatoxnessita, ahhhhhhh that! using variables.... i miss that part13:46
gatoxi only read about the indentation13:46
gatoxnessita, i can change it if you want13:47
nessitagatox: how do you like that better?13:50
gatoxnessita, you know i'm from the tribe that doesn't tend to create variables to assign boolean results :P.... but i can change it if you think that will improve the readability13:51
gatox(boolean results to be used in ifs)13:51
nessitagatox: what I don't like if having stuff like:13:52
nessitaif a:13:52
nessita    return True13:52
nessita    return False13:52
nessitaI strongly prefer:13:52
nessita    return a13:52
nessitathis case is a bit more complex but kinda similar13:52
gatoxnessita, ahhhhhhh i understand now..... i thought you were just saying to do: var = int_bytes_total > FREE_ACCOUNT_SIZE....... and use var in the if..... i see your point now13:53
gatoxi'll change that!13:53
ralsinaOh thanks apport for showing me 152 crash dialogs for everything that has died in the last 3 days. Sigh13:57
gatoxnessita, done (the removal of that lines in sso tests)14:18
ralsinaHas anyone been running pylint on P lately? Because I am getting a ton of errors :-/14:19
gatoxralsina, me too14:19
ralsinagatox: ok, so it's not this screwed up thing14:19
gatoxralsina, i mean, i can run it.... but sometimes crash, and other times show a lot of not real issues14:19
ralsinathe old "TypeError: C++ type 'QWidget*' is not supported as  type" is back, and now the import QtGui first hack doesn fix it14:20
ralsinawhich makes it pretty much useless14:20
gatoxcof cof pyflakes cof cof14:20
dobeyralsina: in windows-installer i guess?14:21
ralsinagatox: here, have a gargaleta :-)14:21
ralsinadobey: worse, in ussoc14:21
gatoxralsina, jeje14:22
dobeyralsina: that's quite odd, as tarmac certainly runs the lint checks14:22
nessitaralsina: I'm not getting any pylint issue14:22
nessitagatox: you're getting those because you still don't have nightlies in P, no?14:22
nessitaralsina: have nightlies installed?14:22
ralsinaoh,Idon't have nightlie either14:22
ralsinagetting them14:22
nessitaralsina: ajá!14:22
gatoxnessita, mmmm maybe with that it'll work..... nighlies are still being installed14:23
nessitadobey: missing nightlies on ralsina ;-)14:23
gatoxi don't know why i lost the niightlies in the upgrade14:23
dobeygatox: because the PPA gets disabled for upgrade14:24
nessitagatox: a ppa targets an ubuntu release, so I don't think that upgrade-manager -d will change your .sources files14:24
gatoxdobey, ahhhh14:24
gatoxnessita, everything is working now14:24
nessitagatox: <will & grace "told you so" dance> told you so! told you so! </will & grace "told you so" dance>14:25
nessitagatox: you know what that dance is?14:25
gatoxno...... but i imagine you dancing and singing that and it's funny jeje :P14:26
=== m_conley_away is now known as m_conley
nessitagatox: http://www.youtube.com/watch?v=IV4sB7O97ro14:27
dobeynessita: the upgrader does disable PPA sources on upgrade, and don't re-enable them when finished. it doesn't really matter what series is targeted by the PPA itself14:27
* gatox watching---14:27
gatoxdobey, i just re-enabled them again from synaptic14:28
nessitagatox: FYI, I have the whole avis for that series14:28
gatoxnessita, jejeje was pretty much close to what i imagine :P14:28
dobeyugh, synaptic14:28
gatoxdobey, well..... for some reason i can't find "software sources"14:29
dobeygatox: software-properties-gtk14:29
gatoxyep..... that doesn't appear in the search14:30
* mandel back14:36
mandelralsina, 1-1 after the stand up?14:36
ralsinamandel: no need, I think14:36
ralsinamandel: we talked yesterday. We should re-schedule for before the team call14:36
mandelralsina, we can do one super fast, got news regarding proxy14:36
ralsinamandel: sure, now or later?14:37
mandelralsina, after the stand up?14:37
dobeygatox: software-properties-gtk is installed by default… you should be able to run that command from the terminal14:37
gatoxdobey, yes, from the terminal is ok.... but for some reason is not appearing in the dash search anymore14:37
dobeygatox: because the .desktop file has NoDisplay=true for some reason14:39
gatoxdobey, ahhhhh thanks! mistery solved!14:39
dobeygatox: i guess the way you're supposed to get to it now, is via software-center14:40
gatoxdobey, ahhhhh yes, it's there14:41
nessitaralsina: when you have a chance, would you please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/center-me/+merge/9358214:46
ralsinanessita: sure14:47
nessitagatox: you left this in the code: QUOTA_THRESHOLD_ACCOUNTS = {'free': 536870912, 'pay': 3221225472}14:48
nessita(ie we now have 2 QUOTA_THRESHOLD_ACCOUNTS)14:48
cjohnstondid anything happen overnight? i lost 35gigs of files?14:49
gatoxnessita, grrrrr.... fixiing14:49
nessitagatox: other than that, looks good!14:50
nessitacjohnston: what do you mean you lost 35G of files?14:50
gatoxnessita, done14:51
cjohnstonyesterday i had ~120 gigs of files, this morning 84gigs of stuff..14:51
nessitacjohnston: did you remove any UDF, from either the cloud or your computer?14:51
nessita(from any of your devices)14:51
nessitacjohnston: sorry, a syncrhonized "folder" outside your Ubuntu One folder14:52
nessitalike Pictures, or Videos, etc14:53
cjohnstoni still have 2..14:53
nessitacjohnston: do you see everything (every folder, etc) in the web UI?14:53
cjohnstoni see the two main folders...14:54
cjohnstonwebui is reporting it lower also14:54
nessitabeuno: ping... does that ^ ring any bell? were there any quota recalculation?14:54
ralsinanessita: about center-me, the code is fine, but isn't it rude for the app to choose it's placement?14:54
cjohnstonnessita: i have files that are missing14:54
beunonessita, nope14:54
cjohnstonie. from my desktop14:54
nessitaralsina: I don't think so... we do it with the gtk control panel14:54
ralsinanessita: ok then14:55
cjohnstona whole bunch of stuff says it was modified 2 hours ago14:55
nessitacjohnston: was/is the desktop a synchtonized folder?14:55
cjohnstonits ~/Ubuntu One/Desktop14:55
cjohnstonand i just point the desktop to that14:55
nessitacjohnston: and you sure you or someone else did not remove those?14:55
nessitaor moved to another location14:56
cjohnstonif they broke into my house14:56
nessitacjohnston: do you have the missing files in the trash?14:56
nessitarye or duanedesign: do you guys have some time to do some follow up on some files missing for cjohnston? ^14:57
cjohnstonit seems as though some of them may be14:58
gatoxmandel, dobey ?15:00
nessitaralsina: go!15:01
ralsinaDONE: team call, design call, nessita call, brian call, diego call, (notice a theme here?), reviews, scraper for Ted Talks, misc, TODO: bug #933758 and random bits and pieces, BLOCKED: no15:01
ralsinagatox: if you please15:01
gatoxProposed SSO UI style branch, fixed some other branches. Started working on captcha problem with sso-login-qt15:01
gatoxFinish with the captcha issue, enjoy looong weekend.15:01
gatoxalecu, go15:02
alecuDONE: tunnel progressing, team meeting, catched up with canonicaladmin15:02
alecuTODO: improve bugs for ffe, send them to joshua15:02
alecuBLOCKED: no15:02
alecuNEXT: nessita15:02
nessitaDONE: weekly call, releases, filed FFe, 1-1 with ralsina, started fix for bug #93363215:02
nessitaTODO: lots of reviews, finish bug #933632, have half day off15:02
nessitaBLOCKED: nopes15:02
nessitaNEXT: briancurtin15:02
briancurtinDONE: lost half of my day trying and failing to setup LXC, corrected the volume signal implementation to provide more info15:02
briancurtinTODO: see if i can get this linux box setup properly to test, otherwise complete it on windows and come back to the setup15:02
briancurtinBLOCKED: none15:02
briancurtinNEXT: mandel15:02
mandelDONE: Found some small issues with the point2point tests on proxy support. Worked on SSL dialog before landing fixes to proxy.15:02
mandelTODO: more SSL dialog work. Move back to proxy. 1-1 ralsina.15:02
mandelBLOCKED: no15:02
mandeldobey, please15:02
dobeyλ DONE: team call, twisted gireactor backport for precise, u1client upload depending on new twisted, some pyqt package splitting15:02
dobeyλ TODO: finish pyqt splitting, twisted docs, music store call15:02
dobeyλ BLCK: none.15:02
ralsinadobey: we are already doing backports for precise? How can that be if it's not even released? :-D15:02
dobeyralsina: backport from trunk, to precise15:04
ralsinadobey: ack15:04
mandelralsina, 1-1 in 10 mins?15:06
ralsinamandel: sure, ping me whenever15:06
mandelnessita, why do I get complains from pep8 regarding # -*- coding: utf-8 -*- => E261 at least two spaces before inline comment15:08
duanedesignhello cjohnston15:08
ralsinamandel: you have a space before the #?15:08
cjohnstonhey duanedesign want to take to PM?15:08
duanedesigncjohnston: you read my mind :)15:08
mandelralsina, no15:08
nessitamandel: may be misplaced in the file. It has to be first line... is it? wnat to show the code?15:08
mandelnessita, is in the first line.. weird15:09
nessitamandel: show the code!15:09
* nessita accepts the challenge15:09
mandelnessita, one sec, I'll psuh it :)15:09
nessitagatox: /931452/ approved15:11
mandelnessita, in lp:~mandel/ubuntu-sso-client/ssl-dialog file ubuntu_sso/qt/arrow.py15:11
nessitagatox: can you please trivialish review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/center-me/+merge/9358215:12
joshuahooverralsina: can you be in #ubuntu-release in 50 min (@ 16:00 utc) to discuss space for u1 on the cd?15:12
gatoxnessita, yep, on it15:12
ralsinajoshuahoover: sure15:12
joshuahooverralsina: thx15:12
nessitamandel: nessita@dali:~/canonical/ussoc/review_ssl-dialog$ ls ubuntu_sso/qt/arrrow.py15:12
nessitals: cannot access ubuntu_sso/qt/arrrow.py: No such file or directory15:12
joshuahooverralsina: as it stands now, it sounds like there won't be room, but you know more about this than i do and can chat about with waaaaay more intelligence :)15:13
mandelnessita, arrow.py not arrrow :)15:13
nessitals: cannot access ubuntu_sso/qt/arrow.py: No such file or directory15:13
nessitamandel: you did not add the file yet?15:13
ralsinajoshuahoover: the right person for knwing about this is dobey  :-)15:14
mandelnessita, bzr st returns nothing, so yes15:14
gatoxnessita, question?? you are defyning this: size = lambda *a: gui.QtCore.QSize(123456, 654321)   but at least in the diff i don't see that this variable is used.....15:14
ralsinajoshuahoover: I know much less than he does15:14
nessitagatox: is used from the "live" code, in:15:14
nessitamandel: did you push it? is not in the branch15:15
nessitagatox: 31+                    window.size(), app.desktop().availableGeometry())15:15
mandelnessita, yes, I did a bzr sync-pipeline.. bloody hell let me check15:15
gatoxnessita, ahhhhh, it's a fake15:15
nessitayes! :-)15:15
dobeynessita: did you file that bug btw?15:15
nessitadobey: no... was hoping ralsina will defend that. Let me update him15:16
gatoxnessita, now you can do the told you dance again, because only the diff didn't give me the proper context15:16
gatoxnessita, approve15:16
nessitaralsina: hola! so, this morning we had an informal conversation with some of the platform guys, regarding space in the CD15:16
mandelnessita, fixed, can you remove that branch and branch again15:17
nessitaralsina: basically they want a bug report where we justify why we want to use the Qt controlpanel instead of using the same Gtk one15:17
ralsinanessita: it's reasonable15:18
nessitaralsina: I will open the bug and put some things in there, but we certainly we'll need more info in there, that I think you have (and perhaps John as well)15:18
ralsinanessita: cool, I will put the info in it15:18
mandelralsina, mumble?15:20
ralsinamandel: sure15:21
* gatox lunch15:22
=== gatox is now known as gatox_lunch
=== gatox_lunch is now known as gatox
ralsinanessita: on ussoc, how can I add a test for timeout_func that only runs when we are using Qt?15:45
nessitaralsina: let me give that some thoughts15:45
dobeyralsina: @skipIf('qt' in sys.modules and sys.modules['qt'] is not None) ?15:46
ralsinanessita: it does need tests because it's much less trivial than the glib one15:46
ralsinadobey: it's an idea15:46
dobeyi guess @skipIf has a second argument for reason, which would be 'Not running under Qt'15:47
dobeyralsina: you can change the conditional to be more precise for the situation obviously. i was just pseudo-coding that up :)15:47
ralsinadobey: yes, I imagined that ;-)15:48
nessitaralsina, dobey: wait! I'm sure theres a better way :-D15:48
nessitalet me find it15:48
ralsinanessita: it's ok, Iam notin a hurry15:48
nessitagatox:quota warning approved15:54
gatoxnessita, thanks15:54
nessitaralsina: so, currently there is no way. We have no tests for the main/qt.py module, and eve if you use the decorator that will not work to actual run test15:55
nessitaralsina: since the test will "never" be run15:55
nessitaralsina: no, wait, I'm confused15:56
nessitaralsina: scratch all that15:56
* ralsina waits.15:56
* ralsina scratches15:56
nessitaralsina: there is a way, and a clean one :-)15:56
ralsinaneat :-)15:56
nessitain run-tests script, change the line:15:56
nessitau1trial --reactor=gi -p "$DBUS_TESTS_PATH, $QT_TESTS_PATH, $GTK_TESTS_PATH" -i "$WINDOWS_TESTS" "$MODULE"15:56
nessitato also ignore the main/tests/test_qt.py you're adding15:56
nessitaand you're set, the new tests will be run by the last line in that script (the one that reads $XVFB_CMDLINE u1trial -p "$GTK_TESTS_PATH" -i "$WINDOWS_TESTS" --reactor=qt4 --gui "$MODULE")15:57
ralsinanessita: ok, will do it after meeting and lunch15:58
dobeyi really hate all the ignore arguments there15:58
dobeyi think the skips are cleaner :)15:59
dobeynessita: you will like this: https://code.launchpad.net/~dobey/ubuntuone-installer/qt-foo/+merge/9360316:06
nessitadobey: looking!16:06
nessitadobey: no tests for that? :-/16:11
dobeynessita: to test a constant declaration?16:12
nessitadobey: and all the logic around it, spawnning something on some condition, etc16:12
ralsinajoshuahoover: I am at ubuntu-release, how does this conersation start? Should I ping someone?16:16
joshuahooverralsina: good question...maybe ping skaet there16:17
dobeyshould i be there?16:19
ralsinadobey: I think so, yes16:19
ralsinadobey: come to #ubuntu-meeting please16:19
nessitaralsina: perhpas you should hand the bug # in the meeting?16:41
ralsinanessita: good idea16:42
ralsinaargh have to call another mac guy16:51
ralsinanessita: for review: https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-933758/+merge/9361317:00
* ralsina swears it has no lint!17:00
dobeyi should probably get lunch17:02
dobeybbiab. gonna get some lunch17:09
nessitadobey: the patch I built no longer applies due to more branches landing, but I'm close to have a good now17:23
gatoxnessita, ralsina when you have a moment: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/captcha-qt-not-working/+merge/9361817:25
nessitagatox: that will have to remain for the next week17:25
gatoxnessita, i also fix a problem with the  terms checkbox....17:25
gatoxthe condition wasn't ok17:25
nessitaI'm already using my half day off17:25
gatoxnessita, ok17:26
nessitagatox: what part was buggy?17:26
gatoxnessita, when you modified that you did: if self.terms_checkbox.isVisible():..... and that should be: if not self.terms_checkbox.isVisible():....... and i didn't notice that either when i review it17:26
nessitaoh uh17:27
rmcbridedavidcalle: any ideas as to why I would not see content in the video lens?17:28
davidcallermcbride, no videos in your video folder?17:29
rmcbridedavidcalle: I have a few in the folder. wasn't sure if I needed to do something to index them first17:29
davidcallermcbride, nope, it should pick everything in there. Is it working with the remote videos (if you do a search) ?17:30
rmcbridedavidcalle: actually, it is not working with that either17:30
davidcallermcbride, hum. Have you launched it manually or just install log out log in?17:31
rmcbridedavidcalle: AFAIK, it installed with my most recent update after which I rebooted17:31
davidcallermcbride, what is the terminal output of "/usr/lib/unity-lens-video/unity-lens-video" ?17:33
rmcbridedavidcalle: strangely, "no such file or directory"17:34
rmcbrideI guess that would explain part of it... lens UI does display though17:34
rmcbridedavidcalle: Apparently I assumed from that the lens was actually there.17:35
rmcbridedavidcalle: which doesn't appear to be the case. thanks!17:36
davidcallermcbride, hum. If the lens is visible, this file should be here...17:36
rmcbridedavidcalle: the pakcage was not in fact installed.17:36
rmcbrideit is now17:36
davidcalleAnd the lens was visible... Wow. This is odd.17:37
rmcbridethough from your earlier comment I'd guess I need to relog17:37
davidcallermcbride, indeed.17:37
rmcbridedavidcalle: checking that on another precise box next. That scenario could tend towards user confusion (if it wasn't just a wierdness due to my dev environment)17:39
rmcbridenope. the video lens selection doesn't even appear on the laptop.17:39
davidcallermcbride, this shouldn't happen, I mean, if the lens is not installed, it shouldn't appear in the Dash, or maybe you installed it in some other way earlier?17:40
ralsinarmcbride: sounds like you installed it from source. Why did you do that? ;-)17:40
rmcbrideralsina: I'm not certain what happened on the dev box. I don't recall installing it from source17:41
rmcbridedavidcalle: it's possible that I did something when I was poking at the ubuntu TV stuff perhaps?17:41
rmcbridein any case the lens functions as expected after relogging17:42
davidcallermcbride, oh right! If you have installed some ubuntu TV stuff, you should try to remove the /usr/share/unity/lenses/video folder.17:43
davidcalleThen, reinstall the unity-lens-video package. It will recreate it.17:43
rmcbridedavidcalle: OK, thanks!17:44
davidcalleThat's my best guess.17:45
rmcbrideoh rock and roll. Amazon results.17:45
davidcalleWorking? :)17:45
rmcbrideit is :)17:46
davidcallermcbride, cool17:46
rmcbrideYes. Working quite well. Didn't even need to delete that folder, the package installation and relog changed the filter scope17:48
rmcbrideand everythign works identically on the dev machine and the untainted laptop17:48
davidcallermcbride, great :)17:49
davidcalleGot to go! Ping me if you have any questions.17:49
rmcbridedavidcalle: of course now I have to hook up one of my machiens to the TV :) good stuff!17:49
ralsinagrmbl run-tests from ussoc is not running pep8 even when it's installed. Fixing pep8 sins manually :-/17:56
ralsinagatox: can I get a review? https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-933758/+merge/9361318:03
gatoxralsina, yes, of course18:04
ralsinagatox: thanks!18:04
dobeynessita: it should apply to 2.99.4 though no?18:10
gatoxralsina, +118:10
nessitadobey: yes, about to upload now18:11
ralsinagatox: thanks!18:11
dobeynessita: great18:11
Chipacaok, that's the TED and Encuentro sources added18:12
Chipacaenjoy :)18:12
nessitadobey: uploaded18:14
ralsinanessita: if you have 5' later for smallish review, https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-933758/+merge/93613 (sorry if I already asked)18:20
nessitaralsina: doing it already, about to ask to fix this if possible:18:21
nessita    test_timeout_func_keeps_reference ... Traceback (most recent call last):18:21
nessita  File "/home/nessita/canonical/ussoc/review_fix-933758/ubuntu_sso/main/qt.py", line 44, in _callback18:21
nessita    TIMERS.remove(timer)18:21
nessitaKeyError: <PyQt4.QtCore.QTimer object at 0x312ec20>18:21
nessita                                 [OK]18:21
ralsinanessita: it shouldn't do that18:22
nessita it does here :-/18:22
ralsinanessita: it's a race condition. I'll take a look18:22
nessitaralsina: great news, it works! :-D18:23
dobeyi wish we could get rid of all the ignore arguments being passed to u1trial18:25
ralsinadobey: there is only one of those18:26
ralsinadobey: with two modules in it after my branch, though18:26
dobeyralsina: i mean, get rid of using them entirely, in all our projects, and just use @skipIf and such18:27
ralsinadobey: can we skip whole testcases? If yes, then that's doable18:27
dobeyralsina: hrmm. should be able to. we can with foo.SKIP = "BLAH" i think decorator will work too18:28
ralsinadobey: file a wishlist?18:28
nessitadobey: we have bigger issues than just skipping I test case... we have import errors if we don't ignore some files18:30
nessitafor example, the test_windows.py module18:30
dobeynessita: right, though i think we can fix that too18:31
nessitadobey: with try-excepts? :-/18:31
dobeyi don't know18:31
dobeyi know twisted itself doesn't have any such ignores, and they support all kinds of insanity. and tests with missing dependencies just get skipped when they should18:32
dobeythey have some extra magic as well i think, which auto-skips certain things when imports fail18:32
ralsinanessita: can you re-run the tests with revno 881?18:32
nessitaralsina: sure18:32
ralsinanessita: not really happy about the fix, but there is a chance that more than one call to the callback is queued, so the second one executes when  the timer is already removed from the set18:33
nessitaralsina: all green now, looking at the new diff18:47
ralsinanessita: cool18:47
nessitaralsina: so, since you just said that _callback could be called twice, shouldn't you call callback only when timer is in TIMERS?18:48
ralsinanessita: good idea18:50
ralsinanessita: I'll push that change in 1 sec18:52
ralsinanessita: pushed revno 882 with that fixed and a nicer usageof kw.pop()18:57
dobeyi am so not fond of the writing of wasteful tests18:59
nessitadobey: I agree. Let's not write wasteful tests :-)19:00
nessitadobey: can I help not writting hose?19:00
dobeynessita: for my installer branch? i'd rather not write tests for 2 if statements; also, pythono-coverage seems to not be very smart sometimes :(19:11
nessitadobey: yes, I personally thinkg we should not use python-coverage at all (or any coverage tool, they are misleading)19:11
nessitadobey: if you have an if if your code, it means that is important somehow do something on a given condition, no? otherwise the if will not be there. And having a test ensuring that is not a wasteful test, at least not from my POV19:12
nessitaralsina: I'm approving, though please note that docstrings for methods do not use empty line above it (Only for classes' docstring)19:27
mandelnessita, I see you19:30
mandelnessita, maybe ubot is dead :(19:30
nessitamandel: thanks!19:31
mandelnessita, np :)19:31
mandelralsina, how so you draw a circle in QPainter so that it is tanget to the widgets left side? I know it has to do something with setting the painter view port, right?19:31
ralsinamandel: excuse me while I violently context-switch to your question ;-)19:32
mandelralsina, sure :)19:32
ralsinamandel: the QPainter should have the widget as context19:32
ralsinamandel: in that case, it's just draing a circle with radius X centered at X,X19:32
mandelralsina, so, if you tell the QPainter to draw a primitive, it draws in so that is in the center..19:33
ralsinamandel: OTOH, WTF? ;-)19:33
mandelralsina, let me show you, one sec19:33
mandelralsina, http://ubuntuone.com/2Jr1UGiJA5hCwYwojfoNBh19:36
ralsinawhat's the problem?19:37
mandelralsina, the little arrow has its center in the certer of its widget but I want it to have the side against the widget so that the C gets aligned with the bullets of the lists19:37
mandelralsina, I find those 2 pixels off very very very annoying19:37
ralsinamandel: where is the code for that widget?19:38
mandelralsina, I can push it, give me a min or so19:38
ralsinamandel: is the one in your blog?19:38
mandelralsina, that would be the little arrow, yes :)19:38
mandelralsina, but let me push the entire thing19:38
mandelralsina, lp:~mandel/ubuntu-sso-client/ssl-dialog19:39
ralsinamandel: very easy. before doing the drawPrimitive, do p.translate(whatever, whatever)19:40
ralsinathe value(s) of whatever are left as an exercise, but ~ 219:40
ralsinamandel: and also, may be platform and style dependent19:41
mandelralsina, yeah.. that is very annoying19:41
mandelralsina, I mean, the size of the primitive :(19:41
ralsinamandel: sorry, no magical solutions19:43
ralsinamandel: OTOH, *maybe* the problem you are having is something else,like a padding set in the arrow widget19:44
nessitagatox: ping on PM19:44
gatoxnessita, pong19:45
nessitagatox: got my private msg?19:45
mandelralsina, there are padding issues with the label..19:45
gatoxnessita, ahh in the other server19:45
* gatox reading....19:45
ralsinamandel: that's more likely, since the painter by default should be drawing at 0,019:46
nessitagatox: you don't get all the IRC mesasages in the same IRC client?19:46
gatoxnessita, yes..... but sometimes i split them in two windows, because i have several channels , and i like to move with alt+[1-9].... sometimes i just close channels :P19:47
nessitaalecu, ralsina: can I have some review for https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-933632/+merge/93641 ?19:48
ralsinanessita: of course19:55
mandelok, EOD for me, is to later here already.. I'll fix that pixel on monday..19:58
dobeyugh, i can't raise inside a lambda:? boo.20:15
ralsinadobey: well, yes you can. Just define def _raise(exc): raise exc20:17
alecunessita, I won't be reviewing it today. I'll add it for monday.20:18
nessitaalecu: you work on Monday?20:18
alecunessita, make that "Tuesday"20:18
nessitaalecu: hum, is kinda late (nothing should land on Tuesday)20:19
alecunessita, oh, you need this to land today? didn't realize that.20:19
nessitaalecu: yeah :-/20:19
ralsinanessita: I will do one review, so maybe, if you are confident, you can land with one20:19
nessitaalecu: but I can find someone else to do another review, if you can't20:20
ralsinanessita: ask thisfred. It's early for him.20:20
thisfredwhat is the branch?20:21
nessitathisfred: welcome back!!!20:21
nessitathisfred: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-933632/+merge/9364120:21
nessitadobey: stand up time? :-/20:21
alecudobeeh: go!20:22
dobeyi really do not like python and writing tests20:22
thisfrednessita: looks great!20:30
nessitathisfred: are you on precise?20:30
thisfredand I ran the tests20:30
thisfredor no wait20:31
thisfredI am running the tests now :)20:31
thisfrednessita: reviewing two branches at the same time ;)20:31
nessitathisfred: awesome, tests will not run if not in precise with our nightlies20:31
nessitajá! been there, done that20:31
thisfrednessita: ./run-tets does nothing?20:31
nessitathisfred: does everything if you are in precise with our nightlies20:32
nessitathisfred:crashes otherwsie20:32
thisfredmmm, I'm pretty sure I have the nightlies20:32
thisfredbut it returns immediately, no warnings nothing20:32
nessitano? :-/20:33
nessitathisfred: can you please paste the output of:20:33
nessitaapt-cache policy python-twisted-core20:33
dobeynessita: twisted in precise has that now20:34
dobeynessita: but even if he had the wrong twisted, it should error out20:34
thisfrednessita: http://paste.ubuntu.com/846356/20:34
nessitadobey: that looks old, no? ^20:34
dobeyno, that's new20:34
nessitait is?20:34
nessitayou're right20:35
dobeyyes. it's the twisted with the patch20:35
dobeyhence the ubuntu1 bit :)20:35
nessitathisfred: any output at all? are you running ./run-tests20:35
nessitadobey: yes, just noticed that20:35
thisfrednessita: I am running exactly that, and no output at all, it just immediately returns20:36
dobeyand even still, run-tests has echos20:36
dobeywhich means something is really broken if no output20:36
dobeythisfred: what is $? after it exits?20:36
nessitathisfred: I smell PICNIC, but not sure how to help you confirm it20:36
thisfreddobey: huwha?20:38
ralsinanessita: +120:38
ralsinathisfred: sh -x run-tests ?20:39
thisfreddobey: nessita It does fail, because ./run-tests && echo "success" prints nothing20:39
ralsinawithout the ? of course ;-)20:39
dobeythisfred: ./run-tests; echo $?20:39
dobeyhey, my menus are drawing correctly now it seems20:40
thisfreddobey: that just printed $20:40
ralsinathisfred: are you on Ubuntu? ;-)20:41
dobeythisfred: weird20:41
dobeythisfred: apt-get install xvfb20:41
JackyAlcineI wonder if anyone'd be willing to make a Ubuntu One client in Qt. :P20:41
thisfredralsina: http://pastebin.ubuntu.com/846368/ fwiw20:41
nessitadobey: is that the issue? :-(20:41
dobeyJackyAlcine: what do you mean by client?20:41
ralsinaJackyAlcine: amazingly, we are  :-)20:41
dobeynessita: looks like it; but no idea why it's acting that way20:41
nessitaJackyAlcine: you mean besides the one we already have?20:41
JackyAlcineO.o that's done in Qt?20:41
ralsinathisfred: you are missing xvfb20:41
* JackyAlcine goes to check.20:42
ralsinaJackyAlcine: the one we released on tuesday is!20:42
JackyAlcineIt's been a while.20:42
dobeyJackyAlcine: ubuntuone-control-panel-qt ?20:42
thisfredshould have run builddeps first I guess20:42
* JackyAlcine rolls around on the floor.20:42
nessitaJackyAlcine: released yesterday to Ubuntu precise20:42
JackyAlcineLol, I didn't check it that recently!20:42
JackyAlcineAh, for precise.20:42
JackyAlcineWell, I'll have to wait then.20:42
nessitaJackyAlcine: https://launchpad.net/ubuntu/+archive/primary/+files/ubuntuone-control-panel-qt_2.99.4-0ubuntu1_all.deb20:42
ralsinaJackyAlcine: coming eventually someday to Natty and Oneiric. Maybe.20:42
JackyAlcineYou think it'd fit with Kubuntu? I've always wanted to use Ubuntu One in Kubutnu.20:42
dobeyfit how?20:42
dobeywe don't have dolphin integration or anything20:43
ralsinaJackyAlcine: Ubuntu One works just fine in Ubuntu, except for no dolphin integration20:43
JackyAlcineHmm, sounds like someone needs to implement that then.20:43
ralsinaJackyAlcine: I meant "in Kubuntu"20:43
* JackyAlcine has a new project.20:43
ralsinaJackyAlcine: or live without it. I don't miss it! ;-)20:43
JackyAlcineWell, it'd be nice to have the music I have here on Kubuntu sync'd with U1 and then onto my phone.20:44
nessitaJackyAlcine: we haven't explicitely tested it in Kubuntu, though some tweaks are very likely needed, specially in the package dependency bits (control panel depends on ubuntu-sso-client which depends on a freedesktop secrets service, and for now we list explicetly gnome-keyring)20:44
JackyAlcineWhen I used GNOME 2, I had that convenience20:44
bdmurrayOn Precise the ubuntuone plugin wasn't enabled in rhythmbox and I don't recall ever disabling it.  This is a system that has gone through multiple upgrades.20:44
JackyAlcineg-c-c, you mean? or the KDE's SystemSettings?20:44
ralsinabdmurray: is it installed and disabled, or not installed?20:45
bdmurrayralsina: installed and disabled20:45
dobeybdmurray: yep20:45
ralsinabdmurray: no idea. Maybe dobey knows20:45
thisfrednessita: all tests green20:45
nessitathisfred: YEY20:46
dobeybdmurray: https://bugs.launchpad.net/ubuntu/+source/rhythmbox-ubuntuone/+bug/93423520:46
nessitaoops sso nightlies are not building! /me fixes20:46
bdmurraydobey: thanks20:49
nessitadobey: would you trivialy-please https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-nightlies/+merge/93647 ?20:50
dobeyugh, why did unity start opening windows at +0+020:50
dobeynessita: done20:53
nessitadobey: does this bug require a FFe? https://bugs.launchpad.net/ubuntuone-client/+bug/85181020:59
nessita(I think so, but want to confirm)20:59
dobeynessita: it requires adding API?21:02
nessitadobey: a new dbus signal, yes21:03
dobeythen yeah, would require a FFE i think21:03
elopioto what package do I assign a bug with the scrollbars and the U1 gtk control panel?21:07
dobeyit's already filed21:09
dobeybut ubuntuone-control-panel21:09
elopiodobey: really? bug #915190 ?21:10
dobeyelopio: i guess so. is that the bug you're seeing?21:10
elopiodobey: yes. Let me dist-upgrade and check again.21:11
nessitaelopio: gtk control panel is no more... :-)21:13
elopionessita: it is going to be on oneiric for a while, isn't it?21:14
dobeyelopio: you get the bug on oneiric?21:15
nessitaelopio: well, according to the plan, we should backport all the qt apps that we're making work on Linux21:15
nessitaelopio: as far as I know, we have no plans to maintaining the gtk panel backwards.... but perhaps I'm wrong?21:15
ralsinano plans21:15
ralsinaunless we fail miserably at backporting21:15
elopiodobey: I haven't checked. I was asking to see if I should triage on oneiric.21:15
elopioanyway, I thought the problem was with the overlay, not with the control panel.21:16
dobeyelopio: i've only seen reports of it on 12.0421:16
dobeyalecu: it could be an issue with overlsay scrollbar. i'm not sure exactly21:16
elopionessita, ralsina, so do you want me to mark as won't fix all gtk issues?21:17
nessitaelopio: I woulod say not yet, until we definitely resolve the Qt situation21:18
nessitawhich is, atm, uncertain21:18
elopiook. I saw you tagging them. I'll do that.21:18
elopionessita: what was the tag? gtk-ui?21:19
nessitaelopio: yes!21:19
alecudobey, what "could be an issue with overlsay scrollbar" ?21:22
ralsinaok, eod for me!21:24
ralsinaI work on monday, so leaveme anything you want worked on!21:24
dobeyalecu: the scrollbar bugs in gtk control panel21:25
alecudobey, you probably didn't mean me at that point, right?21:25
dobeyalecu: no21:28
dobeyalecu: oh, sorry. ment elopio :)21:28
elopiodobey started drinking too soon.21:28
dobeyi wish21:28
elopioor maybe you should :)21:28
alecuthis looks like an EOW for me!21:35
alecubye guys and gal, see you on tue.21:36
briancurtinenjoy the weekend alecu21:36
gatoxEOW for me!21:43
nessitaand for me!21:43
gatoxbriancurtin, are you free for a review?21:44
briancurtingatox: sure, whats the link?21:44
gatoxahhhh it's a qt thingy.... maybe ralsina better if he is still around21:44
gatoxbriancurtin, thanks anyway21:44
nessitagatox: send ralsina an email, he may be able to do that later21:47
gatoxnessita, ok21:47
nessitaI already approved it21:47
dobeynessita: i added some more tests :-/21:47
nessitadobey: I'm late for my pilates class, will try to review later today (or tom morning)21:47
nessitabye all!21:48
=== m_conley is now known as m_conley_away
elopiodobey: yes, there are still crazy scroll bars.22:30

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