[02:21] <Chipaca> if you restart your video lens now, you should find a couple more sources in there
[02:21] <Chipaca> just sayin'
[02:22] <Chipaca> and now, to bed
[09:21] <JamesTait> Happy Friday, everyone! :)
[11:14] <mandel> ralsina, ping?
[11:14] <ralsina> mandel: pong
[11:14] <mandel> ralsina, morning!
[11:15] <mandel> ralsina, do you know a widget similart to a GtkArrow?
[11:15] <mandel> ralsina, I might hate Qt depending on the answer :)
[11:15] <ralsina> mandel: I don't know gtkarrow!
[11:16] <ralsina> mandel: the icon theme has arrows, maybe using that in a label? Where do you need an arrow?
[11:16] <mandel> ralsina, ah, nice, there is a way to get the arrow from the theme, how do you do that?
[11:16] <mandel> ralsina, is to follow the design of the ssl dialog
[11:17] <ralsina> mandel: can I see an image?
[11:17] <mandel> ralsina, I'll foward the design
[11:17] <mandel> ralsina, via mail
[11:17] <ralsina> mandel: ack
[11:18] <mandel> ralsina, 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 later
[11:18] <gatox> good morning!
[11:19] <ralsina> mandel: you mean for the thing that expands to show the details of the SSL cert?
[11:19] <mandel> ralsina, yes, is there a widget for that already that follows the design?
[11:19] <ralsina> mandel: to get icons from the theme: QIcon::fromTheme(…)
[11:19] <mandel> ralsina, that I know, I don't know the id of the arrow hehe
[11:19] <ralsina> mandel: as soon as I see the design, I will wildly guess ;-)
[11:19] <ralsina> mandel: hahaha
[11:20] <mandel> ralsina, good enough :)
[11:20] <ralsina> mandel: icon names: http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html#names
[11:22] <ralsina> mandel: 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 up
[11:22] <ralsina> mandel: forget it,it would not look right
[11:22] <ralsina> mandel: I need to think a bit more
[11:23] <mandel> ralsina, the best idea I have is to have the label and when the label is clicked make the text area y the scrollable show
[11:23] <mandel> ralsina, but I need to see the exact behaviour we want.. but that can be a bug fix later
[11:24] <gatox> back again
[11:24] <mandel> ralsina, from that page => Version 0.8, 1 August 2006, Rodney Dawes.
[11:24] <mandel> lol
[11:24] <ralsina> mandel: I know
[11:24] <mandel> gatox, any idea how to implement a GtkExpander in qt?
[11:25] <ralsina> I was going to tell you to ask dobey because I knew hewrote it then I saw the time ;-)
[11:25] <gatox> mandel, let me see what a gtkexpander is :P
[11:27] <gatox> mandel, 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] <ralsina> mandel: http://qt-apps.org/content/show.php/QIrExpander?content=115342 but of course we can't use it :-(
[11:27] <mandel> gatox, the dialog overall expands..
[11:28] <mandel> ralsina, gatox looks like I'll have to do a custom widget for this..
[11:28] <ralsina> mandel: do a composite
[11:28] <gatox> mandel, yes..... but i already did one when we made the db_to_u1
[11:29] <ralsina> mandel: three labels, clicking on the arrow makes the bottom one larger/visible
[11:29] <ralsina> mandel: the only "bad" thing is lack of animation but who cares
[11:29] <gatox> then we remove it, but i did something like that, with an animatoin to look pretty and everything
[11:29] <mandel> ralsina, gnome people?
[11:29] <mandel> gatox, got the code around?
[11:29] <ralsina> mandel: it's the corner of a dialog noone will see
[11:30] <gatox> mandel, it's in the first version of db_to_u1
[11:30] <ralsina> mandel: remember: we *do* have a valid certificate ;-)
[11:30] <gatox> maybe not the first, but, something like that
[11:30] <mandel> ralsina, the button one should be a scrollable area, since we want to have a max size for the dialog
[11:30] <gatox> mandel, let me look for it
[11:30] <mandel> gatox, thx!
[11:30] <ralsina> ok, a textview or however it's called
[11:32] <mandel> ralsina, yeah, that, I uderstood what you meant, I was going to that when I though about it, only ugly part, the arrow :(
[11:32] <ralsina> mandel: and I fixed that. I win! ;-)
[11:33] <mandel> ralsina, how? there is not an icon with an arrow, is there?
[11:33] <ralsina> mandel: there used to be, IIRC, let me see
[11:33] <ralsina> mandel: media-playback-start
[11:34] <ralsina> HACK!
[11:34] <mandel> ralsina, ugly hack! that is going to look terrible!
[11:34] <ralsina> mandel: yes, let me find something better
[11:34] <gatox> ralsina, mandel SP_ArrowLeft
[11:34] <gatox> in QStyle
[11:35] <gatox> StandardPixmap
[11:35] <ralsina> mandel: QToolButton, check arrowType
[11:35] <ralsina> mandel: gatox's is better :-)
[11:35] <mandel> yeah for gatox !!!!!
[11:35] <gatox> you can use it in a button, like this: QPushButton(self.style().standardIcon(QStyle.SP_ArrowLeft), 'button text')
[11:35] <mandel> also, should be pointing right, but we are there :)
[11:36] <mandel> SP_ArrowRight
[11:36] <ralsina> mandel: better: SP_ArrowForward
[11:36] <ralsina> mandel: that works even when the app is changed for RTL languages
[11:37] <ralsina> we don't need it but is a good habit :-0
[11:37] <mandel> ok
[11:41] <mandel> ralsina, gatox mierda, is not the correct arrow! http://ubuntuone.com/4O1utO8zUfuwYzuXMEXQ3O
[11:41] <gatox> mandel, that arrow is going to change depending on the os theme
[11:42] <gatox> but is always going to be a right arrow of some kind
[11:42] <ralsina> mandel: I think what you need is not an arrow, it's the "expand" icon/decoration?
[11:42] <mandel> ralsina, yeah, something like that
[11:43] <ralsina> mandel: in a QTreeWidget, is it the right one?
[11:43] <ralsina> right as in correct
[11:43] <mandel> ralsina, let me check
[11:45] <mandel> ralsina, yes that looks very very close
[11:45] <mandel> ralsina, I see some pixel diffs but I'm sure that is just me
[11:45] <ralsina> because you could use a Qtreewidget with a single row for the arrow and the label, but it's getting complex :-(
[11:46] <mandel> ralsina, maybe QStyle::PE_IndicatorArrowRight
[11:47] <gatox> mandel, ping
[11:47] <mandel> gatox, pong!!
[11:47] <gatox> mandel, look at db_to_u1 rev 5
[11:47] <ralsina> mandel: that's not an image, I think
[11:47] <mandel> ralsina, its a primitive, but is the one used for the arrows, right?
[11:47] <gatox> mandel, central_widget file, ExplorerFrame class, methods show_login, hide_login
[11:48] <mandel> gatox, ok
[11:48] <gatox> mandel, execute that versoin to see if that is what you want
[11:51] <mandel> gatox, I think with that and some work using the primitive it will work
[11:51] <gatox> mandel, great
[12:13]  * ralsina walks himself
[12:13] <ralsina> will be back in 30'
[12:16] <gatox> nessita, hi
[12:17] <nessita> hola gatox!
[12:17] <nessita> and hello world!
[12:17] <gatox> nessita, do you have time for some reviews?
[12:17] <alecu> he-llo!
[12:18] <gatox> alecu, hi
[12:18] <nessita> gatox: yes
[12:18] <nessita> gatox: I know I have the style one
[12:18] <gatox> nessita, 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-button
[12:19] <nessita> on those!
[12:19] <gatox> i've recheck the cp ones this morning and resolve a conflict.....
[12:24] <nessita> nice
[12:50] <nessita> gatox: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/931452/+merge/93003 approved with comment, please address before globally approving
[12:50] <gatox> nessita, ok
[12:52] <gatox> nessita, sorry.... i'm using that in those methods.......
[12:52] <gatox> or you mean only that and not the other asserts?
[12:53] <nessita> gatox: let me look at it JIC
[12:53] <nessita> gatox: you're not using the helper there
[12:53] <nessita> gatox: see diff lines 412 to 420
[12:54] <nessita> gatox: my point is use assert_backend_called instead
[12:54] <nessita> oh, and I just got a test failue :-/
[12:54] <nessita> gatox: so I will change my vote to needs fixing until the test is fixes, pasting the error in the MP
[12:55] <gatox> nessita, 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] <nessita> gatox: in the line of the diff 412, what do you see?
[12:56] <gatox> nessita, self.assertIn('validate_email_and_ping', self.ui.backend._called)
[12:56] <nessita> gatox: that is not assert_backend_called
[12:56] <nessita> assert_backend_called is in line 419
[12:56] <gatox> nessita, yes, assert_backend_called is at the end of the method..... that why i ask if you want me to remove the others asserts
[12:57] <nessita> gatox: well, they don't test something extra, do they?
[12:57] <gatox> nessita, ok
[12:57] <nessita> gatox: is an honest question :-) do they?
[12:57] <nessita> (perhaps I'm missing something)
[12:57] <gatox> nessita, i don't think so
[12:57] <nessita> ok, then I think is better to remove that
[12:59] <gatox> nessita, ok, remove that.... and the test are ok now
[13:00] <nessita> gatox: ack... you ran the whole suite?
[13:00] <gatox> nessita, qt part..... i'm in precise and i can't run the whole suite yet
[13:01] <nessita> gatox: why? you should be able
[13:01] <nessita> gatox: you have nightlies? everything up to date?
[13:01] <gatox> nessita, i'm looking at that right now.... i need the gtk3reactor.....
[13:03] <nessita> gatox: what does apt-cache policy python-twisted-core says?
[13:04] <gatox> nessita, python-twisted-core:
[13:04] <gatox>   Installed: 11.1.0-1
[13:04] <gatox>   Candidate: 11.1.0-1ubuntu1
[13:04] <gatox>   Version table:
[13:04] <gatox>      11.1.0-1ubuntu1 0
[13:04] <gatox>         500 http://archive.ubuntu.com/ubuntu/ precise/main amd64 Packages
[13:04] <gatox>  *** 11.1.0-1 0
[13:04] <gatox>         100 /var/lib/dpkg/status
[13:04] <gatox> mmmmm
[13:04] <gatox> do i need nightlies for that too?
[13:05] <nessita> gatox: yes... but the simple fact of having nightlies ppa installed, should give you that
[13:05] <nessita> gatox: what does apt-cache policy ubuntu-sso-client says?
[13:06] <gatox> nessita, ubuntu-sso-client:
[13:06] <gatox>   Installed: 3.1+r842-30~oneiric1
[13:06] <gatox>   Candidate: 3.1+r842-30~oneiric1
[13:06] <gatox>   Version table:
[13:06] <gatox>  *** 3.1+r842-30~oneiric1 0
[13:06] <gatox>         100 /var/lib/dpkg/status
[13:06] <gatox>      2.99.4-0ubuntu1 0
[13:06] <gatox>         500 http://archive.ubuntu.com/ubuntu/ precise/main amd64 Packages
[13:06] <gatox> could it be that everything waas reverted in the upgrade?
[13:06] <gatox> i KNOW i had nightlies for u1-client
[13:06] <nessita> gatox: how did you upgrade? (does not look like you are in P )
[13:06] <nessita> gatox: the package says "oneiric" :-/
[13:06] <gatox> nessita, update-manager -d
[13:07] <nessita> gatox: did you add the ppa for precise?
[13:07] <gatox> nessita,
[13:07] <gatox> gatox@utopia:~$ lsb_release -a
[13:07] <gatox> No LSB modules are available.
[13:07] <gatox> Distributor ID:	Ubuntu
[13:07] <gatox> Description:	Ubuntu precise (development branch)
[13:07] <gatox> Release:	12.04
[13:07] <gatox> Codename:	precise
[13:07] <gatox> gatox@utopia:~$
[13:07] <nessita> gatox: can you please run: sudo apt-add-repository ppa:ubuntuone/nightlies; sudo apt-get update; sudo apt-get dist-upgrade
[13:08] <gatox> ok
[13:08] <gatox> nessita, but i'm in precise, look at that ^
[13:08] <nessita> gatox: yeah, the but you're not running precise nigthlies apparently
[13:09] <gatox> nessita, ok..... running the repo-add, update, upgrade
[13:09] <nessita> ack!
[13:12] <nessita> gatox: 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:13] <gatox> nessita, reviewing....
[13:14] <nessita> gatox: also, when cutting an if guard in multiple lines, you need to indent the second line like this:
[13:15] <nessita> if something_really_long and\
[13:15] <nessita>     please_indent_this_part:
[13:18] <nessita> gatox: also, any reason to remove the unicode from the RED constant? everything should be an unicode
[13:18] <ralsina> Whoa, 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:19] <mandel> ralsina, lol
[13:19] <gatox> nessita, no, reverting..
[13:21] <gatox> nessita, 536870912 is ok
[13:22] <gatox> nessita, 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 that
[13:22] <nessita> gatox: right, but isn't it clearer to use FREE_ACCOUNT_SIZE * 0.1? (ie the 10% of it)
[13:23] <nessita> gatox: my proposal is to use instead:
[13:24] <nessita>         FREE_ACCOUNT_SIZE = (BYTES**3) * 5  # 5 gigs
[13:24] <nessita>         QUOTA_THRESHOLD_ACCOUNTS = {'free': FREE_ACCOUNT_SIZE * 0.1,  # 10% of the free account size
[13:24] <nessita>                                     'pay': (BYTES**3) * 3}  # 3 gigs
[13:24] <nessita> gatox: so, next time someone reads that, does not have to do all the math we both just did :-D
[13:24] <mandel> ok, time for my lunch
[13:24] <mandel> bbl
[13:25] <gatox> nessita, 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 variable
[13:25] <nessita> gatox: for free account, it's the 10%
[13:26] <gatox> ok
[13:37] <gatox> nessita, done
[13:37] <nessita> gatox: yey! and reconnect-button is approved
[13:38] <gatox> nessita, great....... still downloading nightlies here.......  :S
[13:38] <nessita> gatox: JIC, did you saw the other comment I made in the quota warning MP?
[13:38] <nessita> did you see*
[13:38] <gatox> nessita, the unicode red one?
[13:39] <nessita> gatox: that also, but I also added a comment in the MP, in LP
[13:39] <nessita> about the indentantion
[13:39] <gatox> nessita, yes..... i fix that too as we talk in your home once, aligning the second line where the first word not if or elif start
[13:40] <gatox> is that ok?
[13:40] <nessita> so the second line has to be aligned with the start of whatever is next to the if, let me give you an example
[13:41] <gatox> yes, that's what i did
[13:41] <gatox> nessita, ^
[13:42] <nessita> gatox: an example http://pastebin.ubuntu.com/845816/plain/, let me re-check what you did
[13:42] <gatox> yes, i did that
[13:46] <nessita> gatox: ah, ok. So I suggested another way of putting that logic using variables, but no need to change
[13:46] <nessita> having the proper indentantion is enough
[13:46] <gatox> nessita, wait!
[13:46] <gatox> nessita, ahhhhhhh that! using variables.... i miss that part
[13:46] <gatox> i only read about the indentation
[13:47] <gatox> nessita, i can change it if you want
[13:50] <nessita> gatox: how do you like that better?
[13:51] <gatox> nessita, 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 readability
[13:51] <gatox> (boolean results to be used in ifs)
[13:52] <nessita> hahahaha
[13:52] <nessita> gatox: what I don't like if having stuff like:
[13:52] <nessita> if a:
[13:52] <nessita>     return True
[13:52] <nessita> else:
[13:52] <nessita>     return False
[13:52] <nessita> I strongly prefer:
[13:52] <nessita>     return a
[13:52] <nessita> this case is a bit more complex but kinda similar
[13:53] <gatox> nessita, 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 now
[13:53] <gatox> i'll change that!
[13:53] <nessita> :-)
[13:57] <ralsina> Oh thanks apport for showing me 152 crash dialogs for everything that has died in the last 3 days. Sigh
[14:18] <gatox> nessita, done (the removal of that lines in sso tests)
[14:19] <ralsina> Has anyone been running pylint on P lately? Because I am getting a ton of errors :-/
[14:19] <gatox> ralsina, me too
[14:19] <ralsina> gatox: ok, so it's not this screwed up thing
[14:19] <gatox> ralsina, i mean, i can run it.... but sometimes crash, and other times show a lot of not real issues
[14:20] <ralsina> the old "TypeError: C++ type 'QWidget*' is not supported as  type" is back, and now the import QtGui first hack doesn fix it
[14:20] <ralsina> which makes it pretty much useless
[14:20] <gatox> cof cof pyflakes cof cof
[14:21] <dobey> ralsina: in windows-installer i guess?
[14:21] <ralsina> gatox: here, have a gargaleta :-)
[14:21] <ralsina> dobey: worse, in ussoc
[14:22] <gatox> ralsina, jeje
[14:22] <dobey> ralsina: that's quite odd, as tarmac certainly runs the lint checks
[14:22] <nessita> ralsina: I'm not getting any pylint issue
[14:22] <nessita> gatox: you're getting those because you still don't have nightlies in P, no?
[14:22] <nessita> ralsina: have nightlies installed?
[14:22] <ralsina> oh,Idon't have nightlie either
[14:22] <ralsina> getting them
[14:22] <nessita> ralsina: ajá!
[14:23] <nessita> :-)
[14:23] <gatox> nessita, mmmm maybe with that it'll work..... nighlies are still being installed
[14:23] <nessita> dobey: missing nightlies on ralsina ;-)
[14:23] <gatox> i don't know why i lost the niightlies in the upgrade
[14:23] <dobey> heh
[14:24] <dobey> gatox: because the PPA gets disabled for upgrade
[14:24] <nessita> gatox: a ppa targets an ubuntu release, so I don't think that upgrade-manager -d will change your .sources files
[14:24] <gatox> dobey, ahhhh
[14:24] <gatox> nessita, everything is working now
[14:25] <nessita> gatox: <will & grace "told you so" dance> told you so! told you so! </will & grace "told you so" dance>
[14:25] <nessita> :-P
[14:25] <gatox> jejejjejeee
[14:25] <nessita> gatox: you know what that dance is?
[14:26] <gatox> no...... but i imagine you dancing and singing that and it's funny jeje :P
[14:27] <nessita> gatox: http://www.youtube.com/watch?v=IV4sB7O97ro
[14:27] <dobey> nessita: 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 itself
[14:27]  * gatox watching---
[14:28] <gatox> dobey, i just re-enabled them again from synaptic
[14:28] <nessita> gatox: FYI, I have the whole avis for that series
[14:28] <gatox> nessita, jejeje was pretty much close to what i imagine :P
[14:28] <dobey> ugh, synaptic
[14:29] <gatox> dobey, well..... for some reason i can't find "software sources"
[14:29] <dobey> gatox: software-properties-gtk
[14:30] <gatox> yep..... that doesn't appear in the search
[14:36] <dobey> search?
[14:36]  * mandel back
[14:36] <mandel> ralsina, 1-1 after the stand up?
[14:36] <ralsina> mandel: no need, I think
[14:36] <ralsina> mandel: we talked yesterday. We should re-schedule for before the team call
[14:36] <mandel> ralsina, we can do one super fast, got news regarding proxy
[14:37] <ralsina> mandel: sure, now or later?
[14:37] <mandel> ralsina, after the stand up?
[14:37] <dobey> gatox: software-properties-gtk is installed by default… you should be able to run that command from the terminal
[14:37] <gatox> dobey, yes, from the terminal is ok.... but for some reason is not appearing in the dash search anymore
[14:39] <dobey> gatox: because the .desktop file has NoDisplay=true for some reason
[14:39] <gatox> dobey, ahhhhh thanks! mistery solved!
[14:40] <dobey> gatox: i guess the way you're supposed to get to it now, is via software-center
[14:41] <gatox> dobey, ahhhhh yes, it's there
[14:46] <nessita> ralsina: when you have a chance, would you please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/center-me/+merge/93582
[14:47] <ralsina> nessita: sure
[14:48] <nessita> gatox: 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:49] <cjohnston> did anything happen overnight? i lost 35gigs of files?
[14:49] <gatox> nessita, grrrrr.... fixiing
[14:50] <nessita> gatox: other than that, looks good!
[14:50] <nessita> cjohnston: what do you mean you lost 35G of files?
[14:51] <gatox> nessita, done
[14:51] <cjohnston> yesterday i had ~120 gigs of files, this morning 84gigs of stuff..
[14:51] <nessita> cjohnston: did you remove any UDF, from either the cloud or your computer?
[14:51] <nessita> (from any of your devices)
[14:52] <cjohnston> udf?
[14:52] <nessita> cjohnston: sorry, a syncrhonized "folder" outside your Ubuntu One folder
[14:53] <nessita> like Pictures, or Videos, etc
[14:53] <cjohnston> no
[14:53] <cjohnston> i still have 2..
[14:53] <nessita> cjohnston: do you see everything (every folder, etc) in the web UI?
[14:54] <cjohnston> i see the two main folders...
[14:54] <cjohnston> webui is reporting it lower also
[14:54] <nessita> beuno: ping... does that ^ ring any bell? were there any quota recalculation?
[14:54] <ralsina> nessita: about center-me, the code is fine, but isn't it rude for the app to choose it's placement?
[14:54] <cjohnston> nessita: i have files that are missing
[14:54] <beuno> nessita, nope
[14:54] <cjohnston> ie. from my desktop
[14:54] <nessita> ralsina: I don't think so... we do it with the gtk control panel
[14:55] <ralsina> nessita: ok then
[14:55] <cjohnston> a whole bunch of stuff says it was modified 2 hours ago
[14:55] <nessita> cjohnston: was/is the desktop a synchtonized folder?
[14:55] <cjohnston> its ~/Ubuntu One/Desktop
[14:55] <cjohnston> and i just point the desktop to that
[14:55] <nessita> cjohnston: and you sure you or someone else did not remove those?
[14:56] <nessita> or moved to another location
[14:56] <cjohnston> if they broke into my house
[14:56] <nessita> cjohnston: do you have the missing files in the trash?
[14:57] <nessita> rye or duanedesign: do you guys have some time to do some follow up on some files missing for cjohnston? ^
[14:58] <cjohnston> it seems as though some of them may be
[15:00] <ralsina> me
[15:00] <gatox> me
[15:00] <alecu> me
[15:00] <nessita> me
[15:00] <briancurtin> me
[15:00] <gatox> mandel, dobey ?
[15:01] <mandel> me
[15:01] <dobey> meh
[15:01] <nessita> ralsina: go!
[15:01] <ralsina> DONE: 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: no
[15:01] <ralsina> gatox: if you please
[15:01] <gatox> DONE:
[15:01] <gatox> Proposed SSO UI style branch, fixed some other branches. Started working on captcha problem with sso-login-qt
[15:01] <gatox> TODO:
[15:01] <gatox> Finish with the captcha issue, enjoy looong weekend.
[15:01] <gatox> BLOCKED:
[15:02] <gatox> No
[15:02] <gatox> alecu, go
[15:02] <alecu> DONE: tunnel progressing, team meeting, catched up with canonicaladmin
[15:02] <alecu> TODO: improve bugs for ffe, send them to joshua
[15:02] <alecu> BLOCKED: no
[15:02] <alecu> NEXT: nessita
[15:02] <nessita> DONE: weekly call, releases, filed FFe, 1-1 with ralsina, started fix for bug #933632
[15:02] <nessita> TODO: lots of reviews, finish bug #933632, have half day off
[15:02] <nessita> BLOCKED: nopes
[15:02] <nessita> NEXT: briancurtin
[15:02] <briancurtin> DONE: lost half of my day trying and failing to setup LXC, corrected the volume signal implementation to provide more info
[15:02] <briancurtin> TODO: see if i can get this linux box setup properly to test, otherwise complete it on windows and come back to the setup
[15:02] <briancurtin> BLOCKED: none
[15:02] <briancurtin> NEXT: mandel
[15:02] <mandel> DONE: Found some small issues with the point2point tests on proxy support. Worked on SSL dialog before landing fixes to proxy.
[15:02] <mandel> TODO: more SSL dialog work. Move back to proxy. 1-1 ralsina.
[15:02] <mandel> BLOCKED: no
[15:02] <mandel> dobey, please
[15:02] <dobey> λ DONE: team call, twisted gireactor backport for precise, u1client upload depending on new twisted, some pyqt package splitting
[15:02] <dobey> λ TODO: finish pyqt splitting, twisted docs, music store call
[15:02] <dobey> λ BLCK: none.
[15:02] <ralsina> dobey: we are already doing backports for precise? How can that be if it's not even released? :-D
[15:04] <dobey> ralsina: backport from trunk, to precise
[15:04] <ralsina> dobey: ack
[15:06] <mandel> ralsina, 1-1 in 10 mins?
[15:06] <ralsina> mandel: sure, ping me whenever
[15:08] <mandel> nessita, why do I get complains from pep8 regarding # -*- coding: utf-8 -*- => E261 at least two spaces before inline comment
[15:08] <duanedesign> hello cjohnston
[15:08] <ralsina> mandel: you have a space before the #?
[15:08] <cjohnston> hey duanedesign want to take to PM?
[15:08] <duanedesign> cjohnston: you read my mind :)
[15:08] <mandel> ralsina, no
[15:08] <nessita> mandel: may be misplaced in the file. It has to be first line... is it? wnat to show the code?
[15:09] <mandel> nessita, is in the first line.. weird
[15:09] <nessita> mandel: show the code!
[15:09]  * nessita accepts the challenge
[15:09] <mandel> nessita, one sec, I'll psuh it :)
[15:11] <nessita> gatox: /931452/ approved
[15:11] <mandel> nessita, in lp:~mandel/ubuntu-sso-client/ssl-dialog file ubuntu_sso/qt/arrow.py
[15:12] <nessita> gatox: can you please trivialish review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/center-me/+merge/93582
[15:12] <joshuahoover> ralsina: can you be in #ubuntu-release in 50 min (@ 16:00 utc) to discuss space for u1 on the cd?
[15:12] <gatox> nessita, yep, on it
[15:12] <ralsina> joshuahoover: sure
[15:12] <joshuahoover> ralsina: thx
[15:12] <nessita> mandel: nessita@dali:~/canonical/ussoc/review_ssl-dialog$ ls ubuntu_sso/qt/arrrow.py
[15:12] <nessita> ls: cannot access ubuntu_sso/qt/arrrow.py: No such file or directory
[15:13] <joshuahoover> ralsina: 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] <mandel> nessita, arrow.py not arrrow :)
[15:13] <nessita> ls: cannot access ubuntu_sso/qt/arrow.py: No such file or directory
[15:13] <nessita> mandel: you did not add the file yet?
[15:14] <ralsina> joshuahoover: the right person for knwing about this is dobey  :-)
[15:14] <mandel> nessita, bzr st returns nothing, so yes
[15:14] <gatox> nessita, 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] <ralsina> joshuahoover: I know much less than he does
[15:14] <nessita> gatox: is used from the "live" code, in:
[15:15] <nessita> mandel: did you push it? is not in the branch
[15:15] <nessita> gatox: 31+                    window.size(), app.desktop().availableGeometry())
[15:15] <mandel> nessita, yes, I did a bzr sync-pipeline.. bloody hell let me check
[15:15] <gatox> nessita, ahhhhh, it's a fake
[15:15] <nessita> yes! :-)
[15:15] <dobey> nessita: did you file that bug btw?
[15:16] <nessita> dobey: no... was hoping ralsina will defend that. Let me update him
[15:16] <gatox> nessita, now you can do the told you dance again, because only the diff didn't give me the proper context
[15:16] <gatox> jejeje
[15:16] <gatox> nessita, approve
[15:16] <nessita> ralsina: hola! so, this morning we had an informal conversation with some of the platform guys, regarding space in the CD
[15:17] <mandel> nessita, fixed, can you remove that branch and branch again
[15:17] <nessita> ralsina: basically they want a bug report where we justify why we want to use the Qt controlpanel instead of using the same Gtk one
[15:18] <ralsina> nessita: it's reasonable
[15:18] <nessita> ralsina: 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] <ralsina> nessita: cool, I will put the info in it
[15:20] <mandel> ralsina, mumble?
[15:21] <ralsina> mandel: sure
[15:22]  * gatox lunch
[15:45] <ralsina> nessita: on ussoc, how can I add a test for timeout_func that only runs when we are using Qt?
[15:45] <nessita> ralsina: let me give that some thoughts
[15:46] <dobey> ralsina: @skipIf('qt' in sys.modules and sys.modules['qt'] is not None) ?
[15:46] <ralsina> nessita: it does need tests because it's much less trivial than the glib one
[15:46] <ralsina> dobey: it's an idea
[15:47] <dobey> i guess @skipIf has a second argument for reason, which would be 'Not running under Qt'
[15:47] <dobey> ralsina: you can change the conditional to be more precise for the situation obviously. i was just pseudo-coding that up :)
[15:48] <ralsina> dobey: yes, I imagined that ;-)
[15:48] <nessita> ralsina, dobey: wait! I'm sure theres a better way :-D
[15:48] <nessita> let me find it
[15:48] <ralsina> nessita: it's ok, Iam notin a hurry
[15:54] <nessita> gatox:quota warning approved
[15:54] <gatox> nessita, thanks
[15:55] <nessita> ralsina: 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 test
[15:55] <nessita> ralsina: since the test will "never" be run
[15:56] <nessita> ralsina: no, wait, I'm confused
[15:56] <nessita> ralsina: scratch all that
[15:56]  * ralsina waits.
[15:56]  * ralsina scratches
[15:56] <nessita> ralsina: there is a way, and a clean one :-)
[15:56] <ralsina> neat :-)
[15:56] <nessita> in run-tests script, change the line:
[15:56] <nessita> u1trial --reactor=gi -p "$DBUS_TESTS_PATH, $QT_TESTS_PATH, $GTK_TESTS_PATH" -i "$WINDOWS_TESTS" "$MODULE"
[15:56] <nessita> to also ignore the main/tests/test_qt.py you're adding
[15:57] <nessita> and 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:58] <ralsina> nessita: ok, will do it after meeting and lunch
[15:58] <nessita> ack!
[15:58] <dobey> i really hate all the ignore arguments there
[15:59] <dobey> i think the skips are cleaner :)
[16:06] <dobey> nessita: you will like this: https://code.launchpad.net/~dobey/ubuntuone-installer/qt-foo/+merge/93603
[16:06] <nessita> dobey: looking!
[16:11] <nessita> dobey: no tests for that? :-/
[16:12] <dobey> nessita: to test a constant declaration?
[16:12] <nessita> dobey: and all the logic around it, spawnning something on some condition, etc
[16:14] <dobey> eh
[16:16] <ralsina> joshuahoover: I am at ubuntu-release, how does this conersation start? Should I ping someone?
[16:17] <joshuahoover> ralsina: good question...maybe ping skaet there
[16:19] <dobey> should i be there?
[16:19] <ralsina> dobey: I think so, yes
[16:19] <ralsina> dobey: come to #ubuntu-meeting please
[16:41] <nessita> ralsina: perhpas you should hand the bug # in the meeting?
[16:42] <ralsina> nessita: good idea
[16:51] <ralsina> argh have to call another mac guy
[17:00] <ralsina> nessita: for review: https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-933758/+merge/93613
[17:00]  * ralsina swears it has no lint!
[17:02] <dobey> i should probably get lunch
[17:09] <dobey> bbiab. gonna get some lunch
[17:23] <nessita> dobey: the patch I built no longer applies due to more branches landing, but I'm close to have a good now
[17:25] <gatox> nessita, ralsina when you have a moment: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/captcha-qt-not-working/+merge/93618
[17:25] <nessita> gatox: that will have to remain for the next week
[17:25] <gatox> nessita, i also fix a problem with the  terms checkbox....
[17:25] <gatox> the condition wasn't ok
[17:25] <nessita> I'm already using my half day off
[17:26] <gatox> nessita, ok
[17:26] <nessita> gatox: what part was buggy?
[17:26] <gatox> nessita, 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 it
[17:27] <nessita> oh uh
[17:28] <rmcbride> davidcalle: any ideas as to why I would not see content in the video lens?
[17:29] <davidcalle> rmcbride, no videos in your video folder?
[17:29] <rmcbride> davidcalle: I have a few in the folder. wasn't sure if I needed to do something to index them first
[17:30] <davidcalle> rmcbride, nope, it should pick everything in there. Is it working with the remote videos (if you do a search) ?
[17:30] <rmcbride> davidcalle: actually, it is not working with that either
[17:31] <davidcalle> rmcbride, hum. Have you launched it manually or just install log out log in?
[17:31] <rmcbride> davidcalle: AFAIK, it installed with my most recent update after which I rebooted
[17:33] <davidcalle> rmcbride, what is the terminal output of "/usr/lib/unity-lens-video/unity-lens-video" ?
[17:34] <rmcbride> davidcalle: strangely, "no such file or directory"
[17:34] <rmcbride> I guess that would explain part of it... lens UI does display though
[17:35] <rmcbride> davidcalle: Apparently I assumed from that the lens was actually there.
[17:36] <rmcbride> davidcalle: which doesn't appear to be the case. thanks!
[17:36] <davidcalle> rmcbride, hum. If the lens is visible, this file should be here...
[17:36] <rmcbride> davidcalle: the pakcage was not in fact installed.
[17:36] <rmcbride> it is now
[17:37] <davidcalle> And the lens was visible... Wow. This is odd.
[17:37] <rmcbride> though from your earlier comment I'd guess I need to relog
[17:37] <davidcalle> rmcbride, indeed.
[17:39] <rmcbride> davidcalle: 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] <rmcbride> nope. the video lens selection doesn't even appear on the laptop.
[17:40] <davidcalle> rmcbride, 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] <ralsina> rmcbride: sounds like you installed it from source. Why did you do that? ;-)
[17:41] <rmcbride> ralsina: I'm not certain what happened on the dev box. I don't recall installing it from source
[17:41] <rmcbride> davidcalle: it's possible that I did something when I was poking at the ubuntu TV stuff perhaps?
[17:42] <rmcbride> in any case the lens functions as expected after relogging
[17:43] <davidcalle> rmcbride, oh right! If you have installed some ubuntu TV stuff, you should try to remove the /usr/share/unity/lenses/video folder.
[17:43] <davidcalle> Then, reinstall the unity-lens-video package. It will recreate it.
[17:44] <rmcbride> davidcalle: OK, thanks!
[17:45] <davidcalle> That's my best guess.
[17:45] <rmcbride> oh rock and roll. Amazon results.
[17:45] <davidcalle> Working? :)
[17:46] <rmcbride> it is :)
[17:46] <davidcalle> rmcbride, cool
[17:48] <rmcbride> Yes. Working quite well. Didn't even need to delete that folder, the package installation and relog changed the filter scope
[17:48] <rmcbride> and everythign works identically on the dev machine and the untainted laptop
[17:49] <davidcalle> rmcbride, great :)
[17:49] <davidcalle> Got to go! Ping me if you have any questions.
[17:49] <rmcbride> davidcalle: of course now I have to hook up one of my machiens to the TV :) good stuff!
[17:50] <nessita> lunchtime!
[17:56] <ralsina> grmbl run-tests from ussoc is not running pep8 even when it's installed. Fixing pep8 sins manually :-/
[18:03] <ralsina> gatox: can I get a review? https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-933758/+merge/93613
[18:04] <gatox> ralsina, yes, of course
[18:04] <ralsina> gatox: thanks!
[18:10] <dobey> nessita: it should apply to 2.99.4 though no?
[18:10] <gatox> ralsina, +1
[18:11] <nessita> dobey: yes, about to upload now
[18:11] <ralsina> gatox: thanks!
[18:11] <dobey> nessita: great
[18:12] <Chipaca> ok, that's the TED and Encuentro sources added
[18:12] <Chipaca> enjoy :)
[18:14] <nessita> dobey: uploaded
[18:20] <ralsina> nessita: 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:21] <nessita> ralsina: 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 _callback
[18:21] <nessita>     TIMERS.remove(timer)
[18:21] <nessita> KeyError: <PyQt4.QtCore.QTimer object at 0x312ec20>
[18:21] <nessita>                                  [OK]
[18:22] <ralsina> nessita: it shouldn't do that
[18:22] <nessita>  it does here :-/
[18:22] <ralsina> nessita: it's a race condition. I'll take a look
[18:22] <nessita> thanks!
[18:23] <nessita> ralsina: great news, it works! :-D
[18:25] <dobey> i wish we could get rid of all the ignore arguments being passed to u1trial
[18:26] <ralsina> dobey: there is only one of those
[18:26] <ralsina> dobey: with two modules in it after my branch, though
[18:27] <dobey> ralsina: i mean, get rid of using them entirely, in all our projects, and just use @skipIf and such
[18:27] <ralsina> dobey: can we skip whole testcases? If yes, then that's doable
[18:28] <dobey> ralsina: hrmm. should be able to. we can with foo.SKIP = "BLAH" i think decorator will work too
[18:28] <ralsina> dobey: file a wishlist?
[18:30] <dobey> perhaps
[18:30] <nessita> dobey: we have bigger issues than just skipping I test case... we have import errors if we don't ignore some files
[18:30] <nessita> for example, the test_windows.py module
[18:31] <dobey> nessita: right, though i think we can fix that too
[18:31] <nessita> dobey: with try-excepts? :-/
[18:31] <dobey> i don't know
[18:32] <dobey> i 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 should
[18:32] <dobey> they have some extra magic as well i think, which auto-skips certain things when imports fail
[18:32] <ralsina> nessita: can you re-run the tests with revno 881?
[18:32] <nessita> ralsina: sure
[18:33] <ralsina> nessita: 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 set
[18:47] <nessita> ralsina: all green now, looking at the new diff
[18:47] <ralsina> nessita: cool
[18:48] <nessita> ralsina: so, since you just said that _callback could be called twice, shouldn't you call callback only when timer is in TIMERS?
[18:50] <ralsina> nessita: good idea
[18:52] <ralsina> nessita: I'll push that change in 1 sec
[18:57] <ralsina> nessita: pushed revno 882 with that fixed and a nicer usageof kw.pop()
[18:58] <nessita> ack
[18:59] <dobey> i am so not fond of the writing of wasteful tests
[19:00] <nessita> dobey: I agree. Let's not write wasteful tests :-)
[19:00] <nessita> dobey: can I help not writting hose?
[19:11] <dobey> nessita: 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] <nessita> dobey: yes, I personally thinkg we should not use python-coverage at all (or any coverage tool, they are misleading)
[19:12] <nessita> dobey: 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 POV
[19:27] <nessita> ralsina: I'm approving, though please note that docstrings for methods do not use empty line above it (Only for classes' docstring)
[19:27] <nessita> @ping
[19:30] <mandel> nessita, I see you
[19:30] <mandel> nessita, maybe ubot is dead :(
[19:31] <nessita> right
[19:31] <nessita> mandel: thanks!
[19:31] <mandel> nessita, np :)
[19:31] <mandel> ralsina, 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:32] <ralsina> mandel: excuse me while I violently context-switch to your question ;-)
[19:32] <mandel> ralsina, sure :)
[19:32] <ralsina> mandel: the QPainter should have the widget as context
[19:32] <ralsina> mandel: in that case, it's just draing a circle with radius X centered at X,X
[19:33] <mandel> ralsina, so, if you tell the QPainter to draw a primitive, it draws in so that is in the center..
[19:33] <ralsina> mandel: OTOH, WTF? ;-)
[19:33] <mandel> ralsina, let me show you, one sec
[19:36] <mandel> ralsina, http://ubuntuone.com/2Jr1UGiJA5hCwYwojfoNBh
[19:37] <ralsina> what's the problem?
[19:37] <mandel> ralsina, 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 lists
[19:37] <mandel> ralsina, I find those 2 pixels off very very very annoying
[19:38] <ralsina> mandel: where is the code for that widget?
[19:38] <mandel> ralsina, I can push it, give me a min or so
[19:38] <ralsina> mandel: is the one in your blog?
[19:38] <mandel> ralsina, that would be the little arrow, yes :)
[19:38] <mandel> ralsina, but let me push the entire thing
[19:39] <mandel> ralsina, lp:~mandel/ubuntu-sso-client/ssl-dialog
[19:40] <ralsina> mandel: very easy. before doing the drawPrimitive, do p.translate(whatever, whatever)
[19:40] <ralsina> the value(s) of whatever are left as an exercise, but ~ 2
[19:41] <ralsina> mandel: and also, may be platform and style dependent
[19:41] <mandel> ralsina, yeah.. that is very annoying
[19:41] <mandel> ralsina, I mean, the size of the primitive :(
[19:43] <ralsina> mandel: sorry, no magical solutions
[19:44] <ralsina> mandel: OTOH, *maybe* the problem you are having is something else,like a padding set in the arrow widget
[19:44] <nessita> gatox: ping on PM
[19:45] <gatox> nessita, pong
[19:45] <nessita> gatox: got my private msg?
[19:45] <mandel> ralsina, there are padding issues with the label..
[19:45] <gatox> nessita, ahh in the other server
[19:45]  * gatox reading....
[19:46] <ralsina> mandel: that's more likely, since the painter by default should be drawing at 0,0
[19:46] <nessita> gatox: you don't get all the IRC mesasages in the same IRC client?
[19:47] <gatox> nessita, 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 :P
[19:47] <nessita> heh
[19:48] <nessita> alecu, ralsina: can I have some review for https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-933632/+merge/93641 ?
[19:55] <ralsina> nessita: of course
[19:58] <mandel> ok, EOD for me, is to later here already.. I'll fix that pixel on monday..
[20:15] <dobey> ugh, i can't raise inside a lambda:? boo.
[20:17] <ralsina> dobey: well, yes you can. Just define def _raise(exc): raise exc
[20:18] <alecu> nessita, I won't be reviewing it today. I'll add it for monday.
[20:18] <nessita> alecu: you work on Monday?
[20:18] <alecu> nessita, make that "Tuesday"
[20:19] <nessita> alecu: hum, is kinda late (nothing should land on Tuesday)
[20:19] <alecu> nessita, oh, you need this to land today? didn't realize that.
[20:19] <nessita> alecu: yeah :-/
[20:19] <ralsina> nessita: I will do one review, so maybe, if you are confident, you can land with one
[20:20] <nessita> alecu: but I can find someone else to do another review, if you can't
[20:20] <ralsina> nessita: ask thisfred. It's early for him.
[20:21] <thisfred> sure
[20:21] <thisfred> what is the branch?
[20:21] <nessita> thisfred: welcome back!!!
[20:21] <dobey> meh
[20:21] <nessita> thisfred: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-933632/+merge/93641
[20:21] <thisfred> :)
[20:21] <nessita> dobey: stand up time? :-/
[20:22] <alecu> dobeeh: go!
[20:22] <dobey> i really do not like python and writing tests
[20:30] <thisfred> nessita: looks great!
[20:30] <nessita> yey!
[20:30] <nessita> thisfred: are you on precise?
[20:30] <thisfred> yes
[20:30] <thisfred> and I ran the tests
[20:31] <thisfred> or no wait
[20:31] <thisfred> I am running the tests now :)
[20:31] <nessita> hehe
[20:31] <thisfred> nessita: reviewing two branches at the same time ;)
[20:31] <nessita> thisfred: awesome, tests will not run if not in precise with our nightlies
[20:31] <nessita> já! been there, done that
[20:31] <thisfred> nessita: ./run-tets does nothing?
[20:32] <nessita> thisfred: does everything if you are in precise with our nightlies
[20:32] <nessita> thisfred:crashes otherwsie
[20:32] <thisfred> mmm, I'm pretty sure I have the nightlies
[20:32] <thisfred> but it returns immediately, no warnings nothing
[20:33] <nessita> no? :-/
[20:33] <nessita> thisfred: can you please paste the output of:
[20:33] <nessita> apt-cache policy python-twisted-core
[20:34] <dobey> nessita: twisted in precise has that now
[20:34] <dobey> nessita: but even if he had the wrong twisted, it should error out
[20:34] <thisfred> nessita: http://paste.ubuntu.com/846356/
[20:34] <nessita> dobey: that looks old, no? ^
[20:34] <dobey> no, that's new
[20:34] <nessita> it is?
[20:35] <nessita> you're right
[20:35] <dobey> yes. it's the twisted with the patch
[20:35] <dobey> hence the ubuntu1 bit :)
[20:35] <nessita> thisfred: any output at all? are you running ./run-tests
[20:35] <nessita> dobey: yes, just noticed that
[20:36] <thisfred> nessita: I am running exactly that, and no output at all, it just immediately returns
[20:36] <dobey> and even still, run-tests has echos
[20:36] <dobey> which means something is really broken if no output
[20:36] <nessita> right
[20:36] <dobey> thisfred: what is $? after it exits?
[20:36] <nessita> thisfred: I smell PICNIC, but not sure how to help you confirm it
[20:38] <thisfred> dobey: huwha?
[20:38] <ralsina> nessita: +1
[20:39] <ralsina> thisfred: sh -x run-tests ?
[20:39] <thisfred> dobey: nessita It does fail, because ./run-tests && echo "success" prints nothing
[20:39] <ralsina> without the ? of course ;-)
[20:39] <dobey> thisfred: ./run-tests; echo $?
[20:40] <thisfred> http://pastebin.ubuntu.com/846368/
[20:40] <dobey> hey, my menus are drawing correctly now it seems
[20:40] <thisfred> dobey: that just printed $
[20:41] <ralsina> thisfred: are you on Ubuntu? ;-)
[20:41] <dobey> thisfred: weird
[20:41] <dobey> thisfred: apt-get install xvfb
[20:41] <JackyAlcine> I wonder if anyone'd be willing to make a Ubuntu One client in Qt. :P
[20:41] <thisfred> ralsina: http://pastebin.ubuntu.com/846368/ fwiw
[20:41] <nessita> dobey: is that the issue? :-(
[20:41] <dobey> JackyAlcine: what do you mean by client?
[20:41] <ralsina> JackyAlcine: amazingly, we are  :-)
[20:41] <dobey> nessita: looks like it; but no idea why it's acting that way
[20:41] <nessita> JackyAlcine: you mean besides the one we already have?
[20:41] <JackyAlcine> O.o that's done in Qt?
[20:41] <ralsina> thisfred: you are missing xvfb
[20:42] <thisfred> kk
[20:42]  * JackyAlcine goes to check.
[20:42] <ralsina> JackyAlcine: the one we released on tuesday is!
[20:42] <JackyAlcine> It's been a while.
[20:42] <dobey> JackyAlcine: ubuntuone-control-panel-qt ?
[20:42] <thisfred> should have run builddeps first I guess
[20:42]  * JackyAlcine rolls around on the floor.
[20:42] <nessita> JackyAlcine: released yesterday to Ubuntu precise
[20:42] <JackyAlcine> Lol, I didn't check it that recently!
[20:42] <JackyAlcine> Ah, for precise.
[20:42] <JackyAlcine> Well, I'll have to wait then.
[20:42] <nessita> JackyAlcine: https://launchpad.net/ubuntu/+archive/primary/+files/ubuntuone-control-panel-qt_2.99.4-0ubuntu1_all.deb
[20:42] <ralsina> JackyAlcine: coming eventually someday to Natty and Oneiric. Maybe.
[20:42] <JackyAlcine> You think it'd fit with Kubuntu? I've always wanted to use Ubuntu One in Kubutnu.
[20:42] <dobey> fit how?
[20:43] <dobey> we don't have dolphin integration or anything
[20:43] <ralsina> JackyAlcine: Ubuntu One works just fine in Ubuntu, except for no dolphin integration
[20:43] <JackyAlcine> Hmm, sounds like someone needs to implement that then.
[20:43] <ralsina> JackyAlcine: I meant "in Kubuntu"
[20:43]  * JackyAlcine has a new project.
[20:43] <ralsina> JackyAlcine: or live without it. I don't miss it! ;-)
[20:44] <JackyAlcine> Well, it'd be nice to have the music I have here on Kubuntu sync'd with U1 and then onto my phone.
[20:44] <nessita> JackyAlcine: 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] <JackyAlcine> When I used GNOME 2, I had that convenience
[20:44] <bdmurray> On 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] <JackyAlcine> g-c-c, you mean? or the KDE's SystemSettings?
[20:45] <ralsina> bdmurray: is it installed and disabled, or not installed?
[20:45] <bdmurray> ralsina: installed and disabled
[20:45] <dobey> bdmurray: yep
[20:45] <ralsina> bdmurray: no idea. Maybe dobey knows
[20:45] <thisfred> nessita: all tests green
[20:46] <nessita> thisfred: YEY
[20:46] <dobey> bdmurray: https://bugs.launchpad.net/ubuntu/+source/rhythmbox-ubuntuone/+bug/934235
[20:46] <nessita> oops sso nightlies are not building! /me fixes
[20:49] <bdmurray> dobey: thanks
[20:50] <nessita> dobey: would you trivialy-please https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-nightlies/+merge/93647 ?
[20:50] <dobey> ugh, why did unity start opening windows at +0+0
[20:53] <dobey> nessita: done
[20:53] <nessita> thanks!
[20:59] <nessita> dobey: does this bug require a FFe? https://bugs.launchpad.net/ubuntuone-client/+bug/851810
[20:59] <nessita> (I think so, but want to confirm)
[21:02] <dobey> nessita: it requires adding API?
[21:03] <nessita> dobey: a new dbus signal, yes
[21:03] <dobey> then yeah, would require a FFE i think
[21:07] <elopio> to what package do I assign a bug with the scrollbars and the U1 gtk control panel?
[21:09] <dobey> it's already filed
[21:09] <dobey> but ubuntuone-control-panel
[21:10] <elopio> dobey: really? bug #915190 ?
[21:10] <dobey> elopio: i guess so. is that the bug you're seeing?
[21:11] <elopio> dobey: yes. Let me dist-upgrade and check again.
[21:13] <nessita> elopio: gtk control panel is no more... :-)
[21:14] <elopio> nessita: it is going to be on oneiric for a while, isn't it?
[21:15] <dobey> elopio: you get the bug on oneiric?
[21:15] <nessita> elopio: well, according to the plan, we should backport all the qt apps that we're making work on Linux
[21:15] <nessita> elopio: as far as I know, we have no plans to maintaining the gtk panel backwards.... but perhaps I'm wrong?
[21:15] <ralsina> no plans
[21:15] <ralsina> unless we fail miserably at backporting
[21:15] <elopio> dobey: I haven't checked. I was asking to see if I should triage on oneiric.
[21:16] <elopio> anyway, I thought the problem was with the overlay, not with the control panel.
[21:16] <dobey> elopio: i've only seen reports of it on 12.04
[21:16] <dobey> alecu: it could be an issue with overlsay scrollbar. i'm not sure exactly
[21:17] <elopio> nessita, ralsina, so do you want me to mark as won't fix all gtk issues?
[21:18] <nessita> elopio: I woulod say not yet, until we definitely resolve the Qt situation
[21:18] <nessita> which is, atm, uncertain
[21:18] <elopio> ok. I saw you tagging them. I'll do that.
[21:19] <elopio> nessita: what was the tag? gtk-ui?
[21:19] <nessita> elopio: yes!
[21:22] <alecu> dobey, what "could be an issue with overlsay scrollbar" ?
[21:24] <ralsina> ok, eod for me!
[21:24] <ralsina> I work on monday, so leaveme anything you want worked on!
[21:25] <dobey> alecu: the scrollbar bugs in gtk control panel
[21:25] <alecu> dobey, you probably didn't mean me at that point, right?
[21:28] <dobey> alecu: no
[21:28] <dobey> alecu: oh, sorry. ment elopio :)
[21:28] <elopio> dobey started drinking too soon.
[21:28] <dobey> i wish
[21:28] <elopio> or maybe you should :)
[21:29] <dobey> indeed
[21:35] <alecu> this looks like an EOW for me!
[21:36] <alecu> bye guys and gal, see you on tue.
[21:36] <briancurtin> enjoy the weekend alecu
[21:43] <gatox> EOW for me!
[21:43] <nessita> and for me!
[21:44] <gatox> briancurtin, are you free for a review?
[21:44] <briancurtin> gatox: sure, whats the link?
[21:44] <gatox> ahhhh it's a qt thingy.... maybe ralsina better if he is still around
[21:44] <gatox> briancurtin, thanks anyway
[21:47] <nessita> gatox: send ralsina an email, he may be able to do that later
[21:47] <gatox> nessita, ok
[21:47] <nessita> I already approved it
[21:47] <dobey> nessita: i added some more tests :-/
[21:47] <nessita> dobey: I'm late for my pilates class, will try to review later today (or tom morning)
[21:48] <nessita> bye all!
[21:48] <dobey> ok
[22:30] <elopio> dobey: yes, there are still crazy scroll bars.