[00:00] <Amto_res> Adri2000: ping
[00:30] <debfx> ScottK: could you please try this kcm-touchpad debdiff: http://paste.ubuntu.com/421345/
[00:30] <debfx> it should disable the kcm instead of crashing
[00:31] <JontheEchidna> if that works, you're mine and every non-admin kubuntu user's hero :)
[00:32] <JontheEchidna> Hmm, we need to get Ark to use org.kubuntu.restrictedInstall to install unrar for maverick...
[00:32] <debfx> it doesn't actually fix the problem though
[00:33] <JontheEchidna> well, better than crashing it for everybody ;)
[00:33] <JontheEchidna> that's what I'm most worried about, since even visiting the keyboard and mouse module crashes the whole of System Settings
[00:37] <JontheEchidna> but, I think that there's no question that it needs replaced for 10.04
[00:38] <JontheEchidna> er, 10.10
[00:43] <debfx> synaptiks even has a bug tracker, that's certainly an improvement
[01:10]  * txwikinger has a feature request for konq or re-konq or whatever will be the KDE browser
[02:39] <ScottK> debfx: I have to release work to do first.  I
[02:39] <ScottK> I'll see if I can get to it tonight
[04:47] <ScottK> debfx: Looking into it now.  In the future, please use pastebins that don't require logins.
[04:51] <msv> Are testers still needed for upgrading Kubuntu from Karmic to Lucid?
[04:59] <ScottK> Yes, although at this point it's probably better to wait until Monday when we're officially testing the final stuff.
[05:00] <msv> Cool.  I have a launchpad account and am setting up a QA account.  I have a dual boot gutsy/karmic laptop
[05:00] <ScottK> debfx: No crash with the non-admin user (was just inactive) and still works with admin user, so I'd call it a win.
[05:00] <msv> so I would be upgrading just the karmic partition.
[05:01] <ScottK> Right, but I'm not the best person to ask for details.  I'm about to go pass out.
[05:01] <msv> Okay, I'll try again Monday.  Thanks =)
[06:33] <apachelogger> maco: plasmacon is written in the py... ;)
[06:47] <debfx> ScottK: does paste.ubuntu.com require to be logged in? :o
[07:47] <jussi> ScottK: they reversed the paste.ubuntu.com login thing a month or 2 ago...
[08:26] <apachelogger> well
[08:26] <apachelogger> lets hope rc1 doesnt fail me
[08:26] <apachelogger> otherwise Ill switch to gentoo
[08:26] <apachelogger> they have such nice people here
[08:26] <apachelogger> and explained me that compiling up to KDE will only take me like one weekend
[08:27] <valorie> hahaha
[08:27] <valorie> pretty much ALL of a weekend
[08:27] <valorie> and then open office.....
[08:44] <jussi> hahahahhhahahaha
[08:45] <apachelogger> valorie: except that I use unbloated software and hence would compile ko instead of ooo :P
[11:02] <Sput> apachelogger: pfff, on a modern machine, installing Gentoo including KDE shouldn't take more than half a day
[11:02] <Sput> and we do have a binary package for openoffice, valorie :P
[11:03] <Sput> (if you don't feel like waiting 4 hours on that)
[12:54] <JontheEchidna> dammit!
[12:54] <JontheEchidna> bug 534915
[12:54] <JontheEchidna> this is ridiculous
[12:55] <JontheEchidna> technically an upstream bug, though
[12:56] <JontheEchidna> but still, it pisses me off :(
[13:00] <Mamarok> JontheEchidna: Ctech translations can cause very strange things sometimes...
[13:01] <JontheEchidna> Mamarok: it's a matter of upstream handing us a busted tarball, the launchpad people using their magic powers to fix things, and then everything breaking again the next upload :(
[13:02] <Mamarok> did that fix get pushed upstream and talked about with tsdgeos?
[13:03] <Mamarok> else it will not get fixed by sheer magic upstream I would say
[13:03] <JontheEchidna> Mamarok: it was fixed in kpackagekit 0.6.0, but that depends on an API/10n-unstable version of packagekit
[13:04] <Mamarok> JontheEchidna: still, please talk to tsdgeos, he is coordinating the translations, he can help with that
[13:04] <Mamarok> just be very nice with him, he totally lacks any sort of humor
[13:05] <JontheEchidna> unfortunately it's a bit too late for kubuntu... the best we could probably do is delete the Czech translation and replace them with the uk ones again, so that it'll get fixed with the first post-release translations update
[13:07] <JontheEchidna> langpack deadline was 2 days ago :(
[13:14] <Mamarok> right, then deleting it is probably better
[13:16] <Mamarok> but please talk to tsdgeos upstream nonetheless, else it ends up again as a "bad Kubuntu" issue
[13:39] <JontheEchidna> Mamarok: would you happen to know if this is official, or just some guy packaging playround stuff? http://kde-apps.org/content/show.php/Phonon-MPlayer-Backend?content=123714
[13:40] <JontheEchidna> I'm tending towards somebody throwing tarballs of playground crap on kde-apps
[13:40] <Mamarok> not official AFAIK, that backend is not maintained since ages
[13:40] <Mamarok> JontheEchidna: but please ask sandsmark for confirmation in #phonon
[13:41] <JontheEchidna> kk
[13:41] <Mamarok> and since it is in playground anyway...
[13:41] <Mamarok> I have never seen somebody use that backend
[13:42] <JontheEchidna> well, you may soon, seeing as he's providing i386 binaries
[13:42] <Mamarok> it was promised ages ago, but that would be the very first changes I see in a long time
[13:42] <JontheEchidna> svn hasn't been touched in 8 months
[13:43] <Mamarok> well, it is stupid, because we are working hard on the vlc-backend, we better work on on really good backend instead of 4 crappy ones
[13:50]  * persia cheers on Colin Guthrie on his work in phonon backends
[13:51] <JontheEchidna> ^He also did the PulseAudio integration work for Phonon in general.
[13:51] <apachelogger> tallking about that
[13:51] <JontheEchidna> Too bad Phonon 4.4 was such a mess release-wise. (No blame to the phonon folk)
[13:51] <apachelogger> is ours still busted!
[13:51] <apachelogger> s/!/?
[13:52] <JontheEchidna> apachelogger: reports say, yes.
[13:52] <apachelogger> computer says no
[13:52] <apachelogger> :D
[13:52] <JontheEchidna> oh, neat!
[13:52] <apachelogger> anyone got psql questions
[13:52] <apachelogger> I have a psql dude walking around
[13:52] <JontheEchidna> you were brave enough to install PA?
[13:53] <apachelogger> JontheEchidna: dunno, I never know if I got PA
[13:53] <apachelogger> then at some point I reinstall and all works again
[13:53] <JontheEchidna> heh
[13:54] <JontheEchidna> I think I might try it in 4.5, to test out the fancy new kmix0r
[13:54]  * apachelogger goes looking for krake
[13:54] <Quintasan> oooh
[13:54] <Quintasan> my head @_@
[14:07] <ScottK> jussi: Not if you try to download the file.
[14:09]  * ScottK has PA installed due to this machine coming with Ubuntu and so far has found it to work.
[15:24] <ScottK> Riddell: I just uploaded debfx's kcm-touchpad fixes.  Over to you to decide if you want them in for the release or not.
[15:25]  * Riddell takes a keek
[15:36] <debfx> I don't think the bug is specific to non-admin users
[15:36] <debfx> logout and login again is enough to trigger it
[15:36] <ScottK> That's just how I could recreate it reliably
[15:37] <debfx> when I do that some synaptic properties are invalid and the desktop effects are disabled
[15:38] <debfx> not sure if that's somehow related
[15:48] <ScottK> debfx: For me this is the type of change that I don't think fully resolves the problem, but I have no doubt makes things better.
[15:50] <Riddell> it solves the problem for me
[15:50] <Riddell> so I'll let it in
[15:50] <Riddell> thanks debfx
[15:54] <Riddell> shtylman: how's that MBR ubiquity issue coming along?
[15:55] <shtylman> Riddell: I committed the fix yesterday
[15:55] <Riddell> lovely
[15:55] <shtylman> Riddell: http://bazaar.launchpad.net/~ubuntu-installer/ubiquity/trunk/revision/4111
[16:38] <shadeslayer> do i need to specify quilt in the build dep,if ive switched to 3.0 packaging?
[16:39] <ScottK> No
[16:39] <ScottK> (fsvo of no - there are circumstances where it may be required, but usually not and I don't recall the details)
[16:39] <shadeslayer> ScottK: then why do i get a lintian error? W: rekonq source: quilt-series-but-no-build-dep
[16:40] <shadeslayer> i have a patch in debian/patches/
[16:40] <ScottK> shadeslayer: First, it's a warning and not an error.  Second, the version of lintian you
[16:40] <ScottK> are using may not know about the v3 stuff
[16:40] <shadeslayer> ScottK: im on lucid...
[16:41] <shadeslayer> ah nvm
[16:41] <shadeslayer> sorted :)
[16:53] <daskreech> nixternal: ping
[17:00] <shtylman> sebas: do you guys still use KD executor for gui testing? have you ever tried frologic squish? I would like to use one of those tools to possibly do some automated testing of the installer... do they do any sort of discount for open source customers?
[17:19] <neversfelde> bulldog98: any news on amarok beta for karmic?
[17:24] <bulldog98> neversfelde: I haven’t found someone for testing yet
[17:25] <neversfelde> bulldog98: try to ask in #kubuntu(-de)
[17:29] <yuriy> hiya folks
[17:30] <ScottK> Hello yuriy
[17:32] <yuriy> leaving to Amsterdam today :D
[18:22] <Riddell> yuriy: what's happening in Amsterdam?
[18:24] <JontheEchidna> looks like I'm sharing a flight to UDS with a kernel dude.
[18:24] <JontheEchidna> Ooh, looks like dantti has a flight booked for UDS too :)
[18:24] <JontheEchidna> looks like there'll be plenty of 11:05 arrivals to share a taxi with
[18:25] <yuriy> Riddell: vacation. visiting a friend who's studying there
[18:26] <JontheEchidna> I have to leave UDS midway through friday. Oh wellz
[18:27] <yuriy> just thought i'd mention it cause i figure there is a good number of KDE folks in the NL
[18:56] <nigelbabu> nixternal: thank you (re: linkedin)
[19:04] <ryanakca> ofirk: I've asked the sysadmins to please push our RT along, I'm heading out, I'll be back tomorrow night.
[19:04] <ofirk> ryanakca: good!
[19:04] <ofirk> ryanakca: contact me when you are back
[19:04] <ryanakca> ofirk: will do
[19:15] <lex79> Riddell: can soprano 2.4.3 go into lucid-updates ?
[19:15] <lex79> http://soprano.sourceforge.net/node/46
[19:16] <ScottK> lex79: Is it just that bugfix?
[19:16] <lex79> ScottK: it seems so
[19:16] <ScottK> Then it probably qualifies
[19:16] <lex79> the package is here https://launchpad.net/~alessandro-ghersi/+archive/staging/+packages
[19:17] <ScottK> lex79: What it will most need is a bug that describes the problem and gives a proper test case for testers to verify the fix.
[19:18] <lex79> ah right
[19:43] <daskreech> !info basket lucid
[19:43] <daskreech> :-D
[19:50] <Quintasan> \o/
[19:53] <Quintasan> Riddell: hmm, can I get a branch like ~kubuntu-members/gluon/ubuntu instead of +junk? :O
[20:19] <lex79> ScottK: Trug said the crash he mentioned in the news/changelog is triggered by experimental code from telepathy-kde project
[20:19] <lex79> So I think we don't need the new release :)
[20:19] <ScottK> lex79: Probably not worth an SRU then.
[20:19] <ScottK> Agreed
[20:19] <lex79> kk
[21:08] <JontheEchidna> lex79, ScottK: funny you should mention soprano bugs... kde bug 155
[21:08] <JontheEchidna> er
[21:08] <JontheEchidna> kde bug 235280
[21:08] <JontheEchidna> bug 566116
[21:09] <lex79> oh, good !
[21:09] <lex79> but strange, trueg said it's a fix just for a bug with telepathy
[21:10] <JontheEchidna> huh, weird
[21:10] <Quintasan> JontheEchidna: do you know how to register ~kubuntu-members/gluon branch?
[21:11] <lex79> Quintasan: you should create a project in launchpad first
[21:11] <lex79> gluon project
[21:11] <lex79> and then create a branch
[21:11] <JontheEchidna> yeah
[21:12] <ScottK> Sounds like SRU material after all then.
[21:12] <ScottK> JontheEchidna: As I told lex79, we'll need an Ubuntu bug with a good test case to demonstrate the problem and that it's fixed.
[21:13] <JontheEchidna> ah, then that doesn't quite meet the requirements
[21:13] <lex79> I think the problem is found the test since it happens randomly with dolphin
[21:14] <ScottK> JontheEchidna: If it's not perfectly reproducable it might still be OK.
[21:14] <JontheEchidna> the user thinks that the bug is him not being able to install anything, so we aren't going to get much from him...
[21:16] <ScottK> JontheEchidna: Get jjesse to try it.  He can crash anything.
[21:16] <JontheEchidna> lol
[21:18] <lex79> :)
[21:23] <JontheEchidna> ScottK: is bug 569129 a bug? To me it looks like it'
[21:23] <JontheEchidna> s starting a second row of icons
[21:24] <JontheEchidna> which are smaller
[21:24] <neversfelde> there is a karmic package of amarok 2.3.1 beta in the backports beta ppa
[21:25] <neversfelde> thanks to bulldog98 for packaging it
[21:26] <ScottK> JontheEchidna: Not a bug.  See the comments
[21:26] <JontheEchidna> k, thanks
[23:13] <Quintasan> Riddell: uploading alpha release of gluon
[23:22] <Riddell> Quintasan: where to?
[23:22] <Riddell> Quintasan: you can make a branch like ~kubuntu-members/gluon/ubuntu by firstly registering a project called gluon in launchpad
[23:23] <Riddell> then just push it, assuming you're a member of ~kubuntu-members
[23:23] <Riddell> bzr push lp:~kubuntu-members/gluon/ubuntu
[23:23] <Quintasan> Riddell: I did that
[23:24] <Quintasan> Riddell: to backports, the previous version was lying there so I figured I would upload it there too
[23:24] <Quintasan> though now that you mention it, it should go to Beta Backports rather.
[23:25] <Riddell> if it's a beta yes it should
[23:26] <Quintasan> okay, reuploading
[23:28] <Quintasan> good night everyone
[23:50] <Riddell> http://www.kubuntu.org/node/143 "Amarok, KOffice, KDevelop Beta Packages Updates"


[00:39] <homebrewcider> HI there, I have setup a mythtv box with a frontend/bakend in one box role, and I keep getting the "could not connect to the master backend server" error. All the solutions I've read seem to relate to having a remote backend. Can't seem to fix this, can anyone help please?
[00:50] <patdk-lap> it doesn't matter
[00:50] <patdk-lap> make sure backend server is running
[00:50] <patdk-lap> and that the correct ip for it is in the database
[00:54] <homebrewcider> correct ip? it has <IP_ADDRESS> listed now
[01:04] <homebrewcider> think I have it now
[19:13] <solars> hey, can anyone tell me, or give me an up to date reference, how to install the lcd stuff for my antec fusion remote? I read that I have to install vdr-plugin-imon, but it isn't found in the package list..


[03:25] <bschaefer> thomi, ping
[03:25] <thomi> yo
[03:26] <bschaefer> hey I saw your awesome work using valgrind!
[03:26] <thomi> heh. It's not finished yet :)
[03:26] <bschaefer> and was trying to see what I could find but I kept getting these python 2.7 errors
[03:26] <bschaefer> and nothing relating to unity :(
[03:26] <thomi> I'm starting to think I've gotten most of the easy ones.
[03:27] <thomi> There's a bug in pango / fontconfig that leaks every time we cann pango_get_text_extents... I haven't had time to follow that up with the gtk folks yet
[03:27] <bschaefer> I keep getting this when I run valgrind
[03:27] <bschaefer> ==7515==    by 0x815749B: PyEval_EvalFrameEx (in /usr/bin/python2.7)
[03:27] <thomi> bschaefer: how are you running valgrind?
[03:28] <bschaefer>  alias valgrind-unity='G_SLICE=always-malloc G_DEBUG=gc-friendly valgrind --tool=memcheck --num-callers=50 --leak-check=full --trac    k-origins=yes --log-file=unity-valgrind.20120409T160245.txt unity --reset 2>&1 | tee ~/logs/unity-valgrind.20120409T160245.log'
[03:28] <thomi> heh, exactly the same as me
[03:28] <bschaefer> tim provided that a few months ago
[03:28] <thomi> bschaefer: and unity is in your PATH and everything?
[03:29] <thomi> actually, mine is slightly different - I make it output an XML log so I can view it with valkyrie
[03:29] <bschaefer> where I install it?
[03:29] <thomi> yeah
[03:29] <bschaefer> hmm I don't see /opt/unity there
[03:29] <bschaefer> Ill add it and see if that helps
[03:29] <thomi> bschaefer: if you do 'which unity' does it tell you the correct one?
[03:29] <thomi> and BTW: ewwww... installing to /opt is icky :)
[03:30] <bschaefer> I just take the output and use a python script to parse it :) I love parsing for some reason haha
[03:30] <bschaefer> haha, it was something I followed a while ago stuck with it!
[03:30] <thomi> does your python parsing script show you the source code for each line in the stack trace?
[03:30] <thomi> ;)
[03:31] <bschaefer> pfft, no. It just parses useless stuff from the log file haha nothing fancy yey
[03:31] <bschaefer> yet*
[03:43] <bschaefer> thomi, hmm I think it was running the wrong one. The top of the log says "/usr/local/bin/unity --reset"
[03:43] <thomi> that could be the issue
[03:43] <bschaefer> but Im still not getting anything around unity :(. Ill take a better look at this tomorrow. Thanks!
[03:44] <bschaefer> good luck with the ones you have found!
[04:35] <thomi> bschaefer: still around?
[09:14] <bluefrog> if not mistaken, bash meta key is used by the HUD and/or by the dash. any way to set another meta key in gnome-terminal?
[09:15] <bluefrog> or change the meta key used by the HUD and/or dash
[09:36] <seb128> davidcalle, hey
[09:37] <davidcalle> seb128, hey
[09:37] <seb128> davidcalle, how are you?
[09:37] <davidcalle> seb128, fine and you?
[09:38] <apw> is the dash not normally resizable ?
[09:38] <seb128> davidcalle, I'm good thanks
[09:38] <seb128> davidcalle, sorry for assigning you a music lens rhythmbox scope bug, I though you were the one who wrote the rb scope this cycle for some reason
[09:38] <seb128> davidcalle, do you know who did? ;-)
[09:40] <seb128> didrocks, mhr3: https://bugs.launchpad.net/ubuntu/+source/unity-lens-music/+bug/964347 is in the top3 duplicates from april, could you have a look (there is some dee function in there)
[09:40] <davidcalle> seb128, I did, with mhr3. We have a divergence of opinion on how to solve the bug in question. And as he has more experience than me...
[09:40] <seb128> davidcalle, so I should assign the bug to mhr3?
[09:40] <didrocks> mhr3: do not hesitate to unblock that ^
[09:40] <davidcalle> seb128, mhr3, the bug in question : album art is present, but not for everything.
[09:41] <seb128> davidcalle, well in my case album art is present for everything embedded in the mp3s and not showing
[09:41] <davidcalle> mhr3, you know, the awkward tdb database.
[09:41] <seb128> davidcalle, I did add the art to all the albums in the dir I use using itunes by then for ipod enjoyment
[09:42] <mhr3> davidcalle, i see a crash bug, no mention of album-art
[09:42] <mhr3> seb128, that was fixed in latest dee
[09:42] <davidcalle> seb128, rbox uses several ways to store album art. And there is still one place we are not looking for.
[09:42] <seb128> mhr3, good, can you close it?
[09:42] <seb128> mhr3, sorry cross discussion
[09:42] <mhr3> seb128, sure, looking for the original bug #
[09:43] <seb128> mhr3, 963991
[09:43] <seb128> ?
[09:43] <mhr3> seb128, yep
[09:43] <seb128> good
[09:43] <seb128> davidcalle, mhr3: so yeah, the second bug is bug #976067
[09:44] <seb128> davidcalle, mhr3: in my case all my mp3 have the artwork embedded in the file (done by itunes back then) so it's nothing specific to rb
[09:44] <seb128> the artwork should show
[09:44] <seb128> it does show on an ipod in rb or in banshee
[09:44] <seb128> but not in the lens
[09:44] <seb128> can I help debugging?
[09:46] <mhr3> davidcalle, so wrt to the album art bug, the issue was that there is nothing i had known of to parse that thing, last time we talked about it you said you'll check with rb devs to see what they use to parse it, correct?
[09:46] <davidcalle> seb128, the artwork would show... if we were looking for it. We don't, we rely on where the music player stores it.
[09:46] <seb128> ok, well currently the lens is a bad experience for me, all my mp3 have the artwork showing everywhere and the lens is a full grid of no artwork icons :-(
[09:47] <davidcalle> mhr3, correct, I've seen a few ml discussion of people having the same question, without answers. There is a reliable way to extract the info... But it's kind of hack-ish.
[09:48] <mhr3> davidcalle, it's either reliable or hackish, so which is it? :)
[09:49] <davidcalle> mhr3, heh, hackish because we parse the raw db file. Reliable, because it's been working for the python scope for six months.
[09:50] <mhr3> parsing raw db files cannot be reliable, db systems have locks and we're just lucky that it works most of the time
[09:52] <davidcalle> mhr3, if we parse the file, there is no lock issue. We can still duplicate it before messing with it.
[09:52] <mhr3> i'm sure we'd see issues if we unleashed that to all ubuntu users
[09:53] <mhr3> ranging from crashes to subtle incorrect string termination bugs
[09:57] <davidcalle> mhr3, crashes can be fixed, we need the data. And from what I've seen, there are no vala bindings for tdb.
[09:58] <mhr3> davidcalle, what does rb use to work with it?
[09:59] <mhr3> the crashes will be bound to the way you're getting the data, so the only fix is to use a library that can work with those dbs
[10:01] <davidcalle> mhr3, it has a dep on libtdb1. And I've just noticed tdb-tools.
[10:03] <mhr3> davidcalle, then a proper fix is to write a quick libtdb binding and use that
[10:06] <davidcalle> mhr3, you?
[10:08] <mhr3> me what? :)
[10:08] <davidcalle> mhr3, I'm obviously unable to do this kind of magic, so I'm asking if you are?
[10:09] <mhr3> there go my plans to work on libunity docs
[10:23] <bluefrog> if not mistaken, bash meta key is used by the HUD and/or by the dash. any way to set another meta key in gnome-terminal? or change the meta key used by the HUD and/or dash ?
[10:52] <krnekhelesh> JohnLea: ping
[11:12] <seb128> gord, mhr3, jaytaoko: is it known that the lens selection doesn't react well to mouseover in the dash?
[11:12] <seb128> like often the right "rectangle" doesn't get highlighted when the mouse enter it and clicks don't work
[11:12] <seb128> it gives a buggy feeling
[11:13] <mhr3> seb128, selection of what?
[11:13] <mhr3> the items in the search resultset?
[11:14] <gord> seb128, "clicks don't work", are you clicking on the rectangle or on the icon? looks like clicking on the rectangle doesn't work here, but the icon does
[11:14] <seb128> mhr3, no, the lens icons at the bottom of the dash
[11:14] <mhr3> oh
[11:14] <seb128> gord, I'm clicking on the rectangle :p
[11:14] <mhr3> yea, i can reproduce what gord says
[11:14] <seb128> well sometimes the rectangle doesn't display as it should on mouseover too
[11:14] <gord> can't reproduce that though
[11:15] <seb128> like if I do left-right with the mouse at a moderated speed I can "enter" areas that don't get highlighted
[11:16] <mhr3> yea, i see that too, if you enter mouse from the top but you go to the middle of two icons, there's no rect
[11:16] <mhr3> although it appears once you move over an icon
[11:17] <seb128> right
[11:17] <seb128> well the behaviour is not consistant, sometimes the selection change when entering the "rectangle"
[11:18] <seb128> mhr3, ok, you are right, it just change when mouseovering the icon
[11:18] <seb128> but in most my "normal" moves I overshot enough that I touch the icon it seems
[11:19] <seb128> where sometimes I stop just before
[11:19] <mhr3> nonetheless i have zero idea of that parts of the code :)
[11:19] <seb128> gord, do you know if it would be complicated to make the clicks work in the selection rectangle out of the icon as well? the current behaviour feels buggy since often you are not over the icon and the click seems to not work
[11:20] <gord> potentially, i never actually did any work with those buttons so i'm not sure, would have to take a look later in the week
[11:21] <seb128> gord, should I ping jaytaoko or somebody else rather? ;-)
[11:21] <seb128> well it's not a blocker in any way
[11:21] <seb128> one of those lack of polish that it would be nice to fix for the lts :p
[11:22] <gord> i think everyone has a bunch of priority stuff to fix today, i know jay does ;)
[11:22] <gord> he may not know it yet but he does ;)
[11:22] <seb128> hehe, indeed
[11:23] <seb128> I wil come back in a few days with all the small nitpick :p
[11:23] <gord> but yeah i agree it would be good to fix, just need to get a few scary bugs off our plate first
[11:23] <seb128> gord, SRU0 or SRU1 will do
[11:23] <Cimi> seb128, andyrock could fix this
[11:24] <andyrock> Cimi, what?
[11:24] <seb128> Cimi, that would be nice ;-)
[11:24] <Cimi> seb128, it was supposed to work as andyrock implemented by design
[11:24] <seb128> andyrock, hey
[11:24] <Cimi> seb128, I agree with you though, keep JohnLea in the loop
[11:24] <seb128> andyrock, run! work is coming :p
[11:24] <andyrock> seb128, cool :)
[11:24] <seb128> andyrock, we were discussing how selecting a lens with the mouse feel buggy because you have to click on the icons
[11:25] <seb128> andyrock, i.e you can't click in the rectangle out of the icon
[11:25] <andyrock> seb128, got it
[11:25] <seb128> andyrock, do you think it's something you could look at once you are out of blocker bugs?
[11:25] <andyrock> seb128, of course
[11:26] <seb128> andyrock, thanks, I also assigned you bug #973386 not sure if you noticed
[11:26] <seb128> Cimi, andyrock:
[11:26] <seb128> https://launchpadlibrarian.net/99959004/bug2.png
[11:26] <seb128> the bottom part of the frame is missing
[11:26] <Cimi> that is weird
[11:27] <andyrock> seb128, Cimi I've not changed anything in the overlay
[11:27] <Cimi> andyrock, maybe it's not vertically centered
[11:27] <andyrock> maybe the bug is in Nux
[11:27] <Cimi> andyrock, or does not fit in
[11:27] <krnekhelesh> andyrock, sorry to intrude but I was the one who submitted that bug...
[11:27] <Cimi> well no, the screenshot is clipped
[11:27] <andyrock> Cimi, if it doesn't fit we don't show it
[11:27] <Cimi> ok
[11:28] <krnekhelesh> andyrock, it started to happened after a compiz update
[11:28] <seb128> Cimi, andyrock: I can confirm that on my 1920x1080 screen, it's not a space issue
[11:28] <Cimi> andyrock, krnekhelesh I can reproduce
[11:28] <andyrock> I can reproduce too
[11:28] <krnekhelesh> Cimi, yeah it is not a space issue...the bottom is clipped on all resoultion monitors
[11:28] <Cimi> krnekhelesh, thanks for filing the bug
[11:29] <Cimi> krnekhelesh, I have serious difficulties in spelling your nickname :D need to teach me how at the UDS :)
[11:29] <andyrock> Cimi, seb128 ok i'll fix the lens bar bug and then the overlay one
[11:29] <krnekhelesh> Cimi, I keep changing my nickname to nik90...but it keeps changing back :P
[11:29] <seb128> andyrock, you rock, thanks ;-)
[11:29] <andyrock> np
[11:29] <andyrock> lunch time :)
[11:29] <Cimi> seb128, only didrocks really rocks
[11:29] <andyrock> ahah
[11:29] <krnekhelesh> hahaha
[11:31] <seb128> andyrock|lunch, I've opened bug #977961 for the lens clicking issue for tracking
[11:31] <didrocks> Cimi: well said! :)
[11:32] <seb128> Cimi, ^
[11:34] <krnekhelesh> seb128, can you give the bug low or medium priority?
[11:34] <seb128> krnekhelesh, which one? yours?
[11:35] <krnekhelesh> seb128, no the bug you filed just now
[11:35] <seb128> krnekhelesh, ok, put it medium, though it might be high
[11:35] <krnekhelesh>  bug #977961
[11:36] <seb128> it's very disturbing from an user perspective, it's one of those details that make you think the system isn't responsive or is buggy
[11:40] <krnekhelesh> yeah...I was a bit surprised to see it happen
[11:41] <krnekhelesh> though I am personally so happy to see so many multimonitor bugs being fixed in Unity 5.10
[12:16] <jbicha> Cimi: I redid the ubuntu-docs screenshots after all with the new default wallpaper
[12:20] <Cimi> jbicha, wohoo
[12:20] <Cimi> jbicha, awesome
[12:20] <jbicha> Cimi: I wrote a script to mostly automate taking the screenshots so it's a bit easier to re-do them or to translate them
[12:21] <Cimi> jbicha, on a different task, I don't need updated screenshots for this, but I'd like to have your +1 if possible
[12:21] <Cimi> https://bugs.launchpad.net/ayatana-design/+bug/977974
[12:21] <Cimi> oh just seen
[12:21] <Cimi> thanks jbicha
[13:41] <mhr3> seb128, could you give lp:~mhr3/unity-lens-music/rb-parse-tdb a try?
[13:41] <seb128> mhr3, sure
[13:50] <seb128> mhr3, not perfect but much better!
[13:51] <mhr3> seb128, what's missing?
[13:52] <seb128> mhr3, I think I need to play the songs once in rb for the artwork to be listed
[13:52] <seb128> mhr3, so half my songs have no artwork
[13:52] <seb128> so it seems to fix itself by playing stuff and restarting the lens
[13:53] <mhr3> seb128, sucks that restarting the lens is needed though :/
[13:53] <seb128> yes
[13:53] <seb128> mhr3, well sucks to start that I need to play stuff to have their artwork in the dash :p
[13:54] <seb128> like new install, plug your ipod, import your music -> not artwork
[13:54] <seb128> no
[13:55] <mhr3> seb128, i dont think we're trying to turn the dash into a music player that fetches artwork for everything you have, generates thumbnail etc.
[13:56] <seb128> mhr3, ok, well then consider that vcs of yours to fix the issue ;-)
[13:56] <seb128> mhr3, it's stable and works great for stuff I played
[13:56] <seb128> mhr3, you would get a bonus point if it was picking new artwork without a lens restart though ;-)
[14:04] <mhr3> seb128, that reminds me, rb is stupid and it updates its xml only on close, so your ipod use case wouldn't work properly anyway
[14:05] <mhr3> seb128, we should push for some more sane data store
[14:05] <seb128> mhr3, oh ok, well get your merge request in and I'm happy ;-)
[14:05] <mhr3> xml... ffs
[14:06] <davidcalle> mhr3, so the .vapi file is what generates the vala bindings, right?
[14:06] <mhr3> yes
[14:07] <davidcalle> Ok, I'm trying to understand what you did.
[14:07] <mhr3> also the .vapi *is* the binding
[15:14] <gotwig> mhall119: hey
[15:14] <mhall119> gotwig: hey,  congrats on membership!
[15:14] <gotwig> mhall119: thanks ^^
[16:37] <elopio> greyback: ping.
[16:38] <htorque> hi all! is there already a bug report about an invisible launcher popping up when (re)loading flash content (i think it's labeled "exe")? short clip: http://www.youtube.com/watch?v=pqQpOL61Kwk
[16:47] <andyrock> seb128, https://code.launchpad.net/~andyrock/unity/fix-977961/+merge/101397
[16:48] <htorque> andyrock: \o/ thanks
[16:48] <andyrock> yw
[16:53] <didrocks> andyrock: hey, while you are working on small issues, do you work on the overlay being cut off?
[16:53] <didrocks> andyrock: the shortcut dialog is cut of at the bottom
[16:53] <greyback> elopio: pong
[16:53] <andyrock> didrocks, i'm working on it right now but I've no idea why we have this regression
[16:54] <didrocks> ok
[16:55] <elopio> greyback: hey. I'm copying a lot of things from your tests. But testhelper.rb has no license. Can I copy it without issues?
[16:59] <greyback> elopio: you can yes, it's made of 2 chunks of ruby code I found. There are links to original sources, both of which are made available with a CC-type licence
[17:00] <greyback> a few changes were made to suit our needs
[17:01] <seb128> andyrock, nice, thanks!
[17:05] <elopio> greyback: great. Also, is there a way to use the same startup and teardown for tests in separate files? I would like all my tests to open the u1 control panel and  log in. But as a ruby newbie, I don't know how to do it.
[17:06] <elopio> greyback: in fact, I've just understood half of your code. But, now I have things running: http://ur1.ca/8z54h :)
[18:22] <greyback> elopio: hey, sorry but I had to go afk.
[18:23] <greyback> elopio: right now we use 'startup' and 'shutdown' methods, which run before and after (resp.) the whole test suite
[18:24] <greyback> elopio: to have them in separate files, hmmm, certainly possible, just need to figure out how
[18:24] <greyback> elopio: I'm no Ruby guru either
[18:45] <elopio> greyback: well, if I find something I'll let you know. In the meantime, I'll be putting everythin in the same file.
[18:45] <elopio> greyback: just another question. Is tdriver_visualizer working for you on precise? I get Aborted (core dumped)
[18:53] <greyback> elopio: curses, it's crashing on me too
[18:53] <greyback> elopio: I'll push a fix to the ppa
[18:54] <elopio> :) thanks greyback.
[18:54] <greyback> elopio: yw
[18:55] <greyback> elopio: It'll become an official package in Q-series
[19:26] <bschaefer> thomi, ping, hey! Im on now
[19:46] <thomi> bschaefer: uhhh, I forgot what it was.
[19:46] <thomi> what timezone are you in again?
[19:46] <bschaefer> umm 21 hours behind you :)
[19:46] <bschaefer> -8
[19:46] <thomi> oh ok
[19:47] <thomi> I'm still not used to Americans going to sleep while I'm eating my breakfast
[19:47] <bschaefer> It was about 11:30pm my time, which im on sometimes
[19:48] <bschaefer> It's only 12:48 pm here now, so if Im going to sleep when your eating breakfast then that would be odd
[19:50] <bschaefer> thomi, o yeah I wanted to ask you how about using valgrind.
[19:50] <bschaefer> so you just have that alias and type "valgrind-unity"
[19:50] <thomi> yup
[19:51] <bschaefer> I ran the autopilot tests after that get as many things used as possible
[19:51] <thomi> bschaefer: for me the autopilot tests don't run since unity is too slow
[19:51] <thomi> so I do things manually
[19:51] <thomi> open the dash, hud, switcher etcx
[19:51] <bschaefer> hmm nothing really slows down when Im running valgrind
[19:51] <bschaefer> that could be a problem, but I know my .log file is getting updated
[19:51] <thomi> bschaefer: ok, you're doing it wrong then :)
[19:52] <thomi> I have a blinging laptop, and it slows to a crawl for me
[19:52] <thomi> I suspect you're running valgrind over the python script, not the compiz child
[19:52] <bschaefer> o and im on a VM sooo yeah. Im guessing valgrind might be crashing
[19:52] <thomi> can you paste your valgrind alias again?
[19:52] <bschaefer> yeah
[19:52] <bschaefer> alias valgrind-unity='G_SLICE=always-malloc G_DEBUG=gc-friendly valgrind --t    ool=memcheck --num-callers=50 --leak-check=full --track-origins=yes --log-fi    le=unity-valgrind.20120409T160245.txt /opt/unity/bin/unity --reset 2>&1 | te    e ~/logs/unity-valgrind.20120409T160245.log'
[19:53] <bschaefer> thomi, that is what it seems like!
[19:53] <thomi> ahhh, I see the issue
[19:53] <thomi> You need this instead:
[19:53] <thomi> alias valgrind-unity='G_SLICE=always-malloc G_DEBUG=gc-friendly valgrind --tool=memcheck --num-callers=50 --leak-check=full --track-origins=yes --xml=yes --xml-file=unity-valgrind.`date +%Y%m%dT%H%M%S`.xml compiz --replace 2>&1 | tee /home/thomi/unity-valgrind.`date +%Y%m%dT%H%M%S`.log'
[19:53] <thomi> obviously change the paths
[19:53] <thomi> I'm running compiz, you're running unity.
[19:53] <bschaefer> ooo
[19:53] <thomi> unity is just a python script that restarts compiz.
[19:54] <bschaefer> I changed it because I had been using that command
[19:54] <thomi> There may be an option to tell memcheck to trach child processes...
[19:54] <bschaefer> duh!
[19:54] <thomi> bschaefer: at some point (maybe @ UDS) I'd love to get you to show me how to run Unity3d inside a VM. I've never been able to get it working
[19:55] <bschaefer> That would be awesome, I have to talk to thumper about that
[19:56] <bschaefer> o and Ill be getting my laptop today, sooo no more VM yay
[19:57] <thomi> sweet!
[19:57] <bschaefer> Thanks for your help! Now hopefully I can fix something :)
[19:58] <bschaefer> you seemed to have fixed a lot of things yesterday haha
[20:09] <thomi> bschaefer: yeah, there's plenty more
[20:31] <htorque> thomi: \o/ thanks for fixing so many leaks!
[20:32] <thomi> htorque: uuhhh, you're welcome.
[20:32] <thomi> there's a few more left, I hope to get them today.
[20:32] <thomi> ...unless bschaefer beats me to it ;)
[20:32] <htorque> :-)
[20:33] <thomi> I have the timezone advantage though, so I think I'm safe
[20:36] <bschaefer> thomi, haha I hope, I just had to go get some coffee
[20:36] <bschaefer> plus I have class later!
[20:37] <bschaefer> thomi, o I also stole your alias and will learn how to parse xml with valkyrie
[20:37] <thomi> bschaefer: just open valkyrie, and inside the UI get it to open the XML file.
[20:38] <thomi> it somehow automatically finds the right source code to laod as well....
[20:39] <bschaefer> awesome, thanks.
[21:29] <davidcalle> mhr3, for some unidentified reason, tonight I have rb updating the xml without restarting it.
[21:30] <mhr3> davidcalle, lol, that's nice of it :)
[22:39] <thomi> bschaefer: still around?
[22:39] <bschaefer> thomi, yeah
[22:39] <bschaefer> man there is a lot of stuff in these logs...
[22:39] <thomi> could I get you to review another valgrind fix please? https://code.launchpad.net/~thomir/unity/panelmenuview-fix-invalid-read/+merge/101452
[22:39] <bschaefer> sure!
[22:39] <thomi> bschaefer: yes, most of it is pango / fontconfig though
[22:41] <bschaefer> thomi, looks good
[22:41] <thomi> cheers
[22:42] <bschaefer> np! If you have anything else that needs review let me know
[22:42] <thomi> there's something similar happening with the hud controller, but I can't work it out
[22:42] <thomi> :-/
[22:42] <bschaefer> I think I saw that, I was working through the uninited vars
[22:43] <bschaefer> I found that there are A LOT of new icons made in LauncherController and views and they don't seem to get delete
[22:43] <bschaefer> well the model_ it self doesn't seem to get delete but I have to figure out if it gets done some how...
[22:47] <thomi> bschaefer: views are usually deleted by nux
[22:55] <bschaefer> hmm well thats good, but valgrind seems to complain about some views not getting deleted


[09:22] <noodles775> Hi intellectronica! The first one in the queue there for me is: https://code.edge.launchpad.net/~michael.nelson/launchpad/567922-binarypackagebuild-new-table-5/+merge/25239 when you've time.
[09:23] <noodles775> (just see the note about conflicts with db-devel on the MP... you can branch it rather than merge to run the tests).
[09:23] <intellectronica> noodles775: yes, looking at it already
[09:23] <noodles775> intellectronica: great, thanks.
[09:53] <intellectronica> noodles775: r=me
[09:54] <noodles775> Thanks intellectronica
[10:08] <jelmer_> hi intellectronica
[10:08] <jelmer_> intellectronica: Can I add another branch to the queue?
[10:08] <intellectronica> jelmer_: sure
[10:09] <jelmer_> thanks :-)
[11:08] <intellectronica> noodles775: so the next in line is 567922-binarypackagebuild-new-table-6 ?
[11:08] <noodles775> intellectronica: indeed, thanks.
[11:23] <adiroiban> henninge: hi. was this branch landed https://code.edge.launchpad.net/~adiroiban/launchpad/bug-525992/+merge/23285 ?
[11:24] <henninge> adiroiban: no, don't think so. That was probably due to ec2 problems we were (are?) having.
[11:25] <henninge> adiroiban: can you merge the current devel, please, before I try again?
[11:25] <adiroiban> henninge: yep. I will let you know when the merge was pushed.
[11:26] <henninge> thanks
[11:56] <adiroiban> henninge: merge pushed. no conflicts.
[11:57] <henninge> adiroiban: cool, I'll land it
[12:04] <intellectronica> gimme some more soyuz branches. ---- yeah!
[12:07] <jelmer_> :-)
[12:07] <jelmer_> it's more registry than soyuz I think
[12:09] <intellectronica> yes. looks fine too.
[12:14] <adiroiban> danilos, henninge, jtv : I have created a MP for the first step in having a POTemplate API. Maybe you would like to take a look https://code.edge.launchpad.net/~adiroiban/launchpad/bug-525371/+merge/25423 :)
[12:15] <jtv> adiroiban: I can take that if you like
[12:15] <adiroiban> jtv: feel free to to it :)
[12:16] <henninge> jtv: Adi and I had a pre-imp discussion about this at UDS.
[12:38] <adiroiban> adeuring: hi. Is there anything I need to do for having this branch approved and landed https://code.edge.launchpad.net/~adiroiban/launchpad/bug-402235/+merge/23973 ?
[12:40] <adeuring> adiroiban: nothing than reminding me. Sorry, I tried to get the branch in for the last release, but the ec3 test hung unfortunately. And I was on holidays last week, was conpletel offline for a few days. I'll stat the ec2 test now again.
[12:41] <adiroiban> adeuring: don't worry :) it is not a critical bug.
[14:08] <adiroiban> danilos: are there any plans for allowing project maitainers to add custom language codes? this can help with fixing bug 487137
[14:08] <mup> Bug #487137: Allow Rosetta admins to create custom language codes <Launchpad Translations:In Progress by adiroiban> <https://launchpad.net/bugs/487137>
[14:10] <danilos> adiroiban, not at this time, no... I am not sure that's a good idea yet, but it's what we should aim for in the future (exposing people to more and more unfinished features would make LP only harder to use)
[14:10] <adiroiban> danilos: ok
[14:39] <cody-somerville> Anyone interested in doing a review? :-)
[14:39] <cody-somerville> https://code.edge.launchpad.net/~cody-somerville/launchpad/create-ppa-via-api/+merge/25319
[14:53] <cody-somerville> abentley, intellectronica: ^^ :-)
[14:53] <intellectronica> cody-somerville: otp, but can look at it after that
[14:54] <cody-somerville> kk
[14:56] <danilos> bigjools, this is logging I want to add to our jobs on the build slave, any comments? https://code.edge.launchpad.net/~danilo/launchpad/bug-580345/+merge/25416
[15:00] <danilos> jtv, henninge: ^^
[15:00] <intellectronica> cody-somerville: oright, the ppa api branch?
[15:01] <cody-somerville> create-ppa-via-api, yup
[15:01] <intellectronica> cody-somerville: is it really ok to create a ppa with just a name?
[15:02] <intellectronica> i mean, is it the same in the web ui?
[15:02] <bigjools> no it's not
[15:02] <cody-somerville> displayname looks optional
[15:02] <intellectronica> cody-somerville: there you go
[15:02] <intellectronica> looks?
[15:02] <henninge> bigjools: huh?
[15:03] <bigjools> henninge: huh?
[15:03] <henninge> bigjools: what question are you replying to?
[15:03] <henninge> danilos: looks good to me.
[15:03]  * bigjools sees danilos' message and looks
[15:04] <cody-somerville> bigjools, are you says its not okay to create a PPA with just a name?
[15:04] <cody-somerville> *saying
[15:05] <bigjools> cody-somerville: ideally no
[15:05] <intellectronica> cody-somerville: i would rename the ubuntu variable to distribution. it's only a cosmetic change, but it means that if you ever want to start handling other distros it will be more natural. you can put the xxx comment near where you assign ubuntu to the distribution variable
[15:05] <bigjools> otherwise the +ppas page looks crap
[15:05] <cody-somerville> bigjools, It doesn't use the name if displayname is not set?
[15:06] <intellectronica> cody-somerville: i'm a bit concerned that this will be open to misuse. how expensive is it to create new ppas? should we maybe export the method for removing them, so that if someone creates too many they can delete them easily?
[15:07] <intellectronica> bigjools: consider this too ^^^
[15:07] <cody-somerville> There is no cost to creating a new PPA.
[15:07] <cody-somerville> and yes, I intend to export the ability to delete as well.
[15:07] <bigjools> there's always a cost :)  intellectronica, I said that this was ok since we added deletion now
[15:07] <intellectronica> cody-somerville: i'd feel more comfortable if you said that the cost is very low :)
[15:08] <intellectronica> bigjools: you mean there's already an api for deleting them, or that you can delete them using the web interface?
[15:08] <bigjools> danilos: I just added a comment to your MP
[15:08] <bigjools> intellectronica: just the web interface ATM, but as Cody said he's exporting that too
[15:09] <intellectronica> cool
[15:09] <cody-somerville> intellectronica, Sorry, there wasn't enough significant digits in my calculation to provide that level of precision :P
[15:09] <intellectronica> is there a conclusion about whether displayname and description are really optional?
[15:10] <intellectronica> cody-somerville, bigjools: ^^^^^
[15:10] <intellectronica> other than that the branch is fine
[15:10]  * bigjools looks at branch
[15:10] <danilos> bigjools, nooooo, anything but a comment!
[15:11] <danilos> bigjools, it's there because I played with different approaches to logging (ideally I'd use Launchpad logger, but didn't want to think about how I get that installed onto build slaves as well)
[15:12] <cody-somerville> intellectronica, displayname in not optional in web UI but description is
[15:13] <bigjools> cody-somerville. intellectronica: there's some refactoring needed
[15:13] <bigjools> the code is slightly duplicating the save_action() in the browser code
[15:14]  * cody-somerville nods.
[15:14] <cody-somerville> I was thinking the same thing
[15:14] <bigjools> cody-somerville: and in fact it misses a bunch of validation
[15:14] <intellectronica> cody-somerville: if you can make the web code use the same model code for doing that it would be good
[15:15] <bigjools> see also validate() in the browser code
[15:15] <intellectronica> cody-somerville, bigjools: i think i'll leave you to it, then ;)
[15:15] <bigjools> I'll comment on the MP
[15:15] <cody-somerville> bigjools, I did look at validate.
[15:16] <cody-somerville> bigjools, a lot of it isn't necessary
[15:16] <bigjools> cody-somerville: why?
[15:16] <cody-somerville> bigjools, because that view is a hack
[15:16] <cody-somerville> bigjools, its different depending if you're creating your first PPA or if you're creating a second PPA.
[15:17] <bigjools> cody-somerville: the validation on the form takes that into account
[15:17] <bigjools> but your new code does not
[15:17] <bigjools> it also doesn't check for duplicate names
[15:18] <cody-somerville> bigjools, 1. and isn't necessary since you're always supplying the name whereas the form does not
[15:18] <cody-somerville> 2. It'll error out if you try that which seems fine to me since it can't provide any more meaningful feedback via the webservices API to my understanding.
[15:18] <bigjools> cody-somerville: that doesn't matter, you still need to check that it's the first PPA and then check if the CoC is signed
[15:19] <cody-somerville> bigjools, Thats what the discussion was about with jml on Friday night.
[15:19] <bigjools> re. (2), you're not testing that so you don't know by definition ;)
[15:19] <bigjools> yes, we said we'd think about ToS-ing the Coc or something
[15:20] <cody-somerville> also, the browser class doesn't check if the CoC is signed either
[15:20] <cody-somerville> and we can't check if the PPA ToS is signed since its not actually recorded
[15:20] <bigjools> cody-somerville: it checks to see if you have accepted the ToS
[15:21] <cody-somerville> bigjools, using a hack. Its not actually recorded.
[15:21] <cody-somerville> bigjools, which is why I added a note in the function docstring that using the API constitutes accepting the PPA ToS.
[15:21] <bigjools> hack?
[15:21] <bigjools> ok that's fine I think
[15:22] <cody-somerville> bigjools, Yes. It doesn't record it. If a PPA exists, it assume the ToS is accepted. If no PPAs exist, then it'll try creating the default PPA and provide that different view that has that checkbox.
[15:22] <bigjools> but, I think you should add more tests where it adds the same name twice
[15:22] <bigjools> I suspect it'll generate an OOPS
[15:22] <bigjools> and we don't want that
[15:22] <cody-somerville> bigjools, I think thats fair feedback.
[15:23] <bigjools> I'll write on the MP
[15:23] <cody-somerville> bigjools, ty
[15:24] <cody-somerville> :-)
[15:24] <bigjools> cody-somerville: np, thanks for the change
[15:24] <bigjools> how was the hangover on Saturday? :)
[15:25] <cody-somerville> lol. didn't drink enough for one (un)fortunately.
[15:27] <cody-somerville> bigjools, how about yourself?
[15:27] <bigjools> cody-somerville: you were pretty hammered on Friday :)
[15:27] <bigjools> I was tired, slept like a baby on the train hime
[15:27] <bigjools> home
[15:29] <BjornT> cody-somerville, intellectronica, bigjools: don't forget to add tests for making sure who has permission to call the createPPA() method.
[15:30] <cody-somerville> Shouldn't there already be tests for that?
[15:30] <bigjools> cody-somerville: since you added the method, no :)
[15:30] <bigjools> you need a unit test for it
[15:30] <BjornT> cody-somerville: yes, the one who added the method should have added tests for that ;)
[15:31] <cody-somerville> createPPA is just a wrapper around new on ArchiveSet
[15:31] <bigjools> still needs a test dude
[15:32] <bigjools> you should use launchpadlib in the story test too
[15:32] <bigjools> I updated the MP
[15:33] <BjornT> cody-somerville: also think about the documentation. now you're saying that anyone can call createPPA. they shouldn't have to try to call it to find it out.
[15:35] <cody-somerville> very well.
[15:38] <bigjools> so, Ubuntu Software Centre fails under Kubuntu :/
[16:16] <EdwinGrubbs> BjornT, have you had a chance to look at the schema changes in https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-535430-needspackaging-timeout-part1/+merge/25359
[16:23] <gmb> intellectronica, abentley: I have another branch for your queue if that's okay: https://code.edge.launchpad.net/~gmb/launchpad/show-oops-ids-bug-564686/+merge/25442
[16:24] <abentley> gmb, I'll take it.
[16:24] <gmb> abentley, Thank you.
[16:24] <BjornT> EdwinGrubbs: no, i haven't. i'll look now
[16:31] <abentley> gmb, is it possible to use the same code path for generating oops URLs that other code does?
[16:33] <gmb> abentley, I looked into that, and tried to reuse the code in webapp/tales.py, but there didn't seem to be a simple way to do use it (I thought there would be a formatter, but apparently not). I'll have a dig around, though, see if I can find anything else.
[16:35] <intellectronica> adiroiban: you've got several jslint notices. what about them?
[16:37] <intellectronica> adiroiban: also, why can't you test?
[16:41] <gmb> abentley, I can't find anything reusable so far; will keep looking but I think we'd be better off having a formatter do the work for us. I might as well fix that in this branch.
[16:41] <abentley> gmb, that would be great if you don't mind.
[16:41] <gmb> abentley, Sure, no problem.
[16:57] <BjornT> EdwinGrubbs: remind me, what is section, and can it change between different series?
[16:58] <EdwinGrubbs> BjornT, section for debs are "apps", "graphics", etc.
[17:01] <BjornT> EdwinGrubbs: ok. let's say a package was in "apps" in karmic, and "graphics" in lucid. first a new release is uploaded to lucid, and then one is uploaded to karmic. what value will DistributionSourcePackage.section have?
[17:03] <BjornT> EdwinGrubbs: also, what happens if a PPA package is uploaded?
[17:04] <EdwinGrubbs> BjornT, that's an interesting point. The trigger which caches it, gets the latest one. I'm not sure what the current behavior is for the page that gets the data directly from the SPPH table.
[17:04] <BjornT> EdwinGrubbs: so, i would suggest discussing this patch with bigjools
[17:05] <EdwinGrubbs> BjornT, I forgot, I need to select only PRIMARY packages to eliminate PPAs.
[17:06] <bigjools> EdwinGrubbs: why are you using a trigger?
[17:10] <bigjools> EdwinGrubbs: BjornT also has a good point about which series' data you're caching
[17:10] <EdwinGrubbs> bigjools, since we would like to consolidate the DistributionSourcePackage class and the DistributionSourcePackageInDatabase class, the object will need to exist in the db by the time the code will use it. This could be done with a cronjob, but that actually seems like it would have more intensive queries, since it would have to query all the SPPHs for a distr+spn in order to find the one with the latest date.
[17:11] <bigjools> I suspect it should only be the one that's marked 'current' for each distro
[17:14] <bigjools> Since DSP is going to start getting published-specific data, you need to define which series that will be for, and how it gets updated when a new series is opened
[17:15] <bigjools> I think you can do that without a trigger or cron job
[17:15] <EdwinGrubbs> bigjools, where is a distroseries specified as current?
[17:16] <bigjools> its status
[17:18] <EdwinGrubbs> bigjools, when you say that I can do that without a trigger or cronjob, do you want the class to automatically create the DSP record whenever it is requested?
[17:19] <bigjools> EdwinGrubbs: I think that's a good idea, yes.  It would only happen at package upload time.
[17:19] <bigjools> (which is never a webapp transaction)
[17:20] <bigjools> so when creating a publishing record, you can check to see if it's the right series and then update/create the DSP
[17:20] <bigjools> this will also be good for Soyuz when we start caching stuff :)
[17:21] <EdwinGrubbs> bigjools, which script creates the spph record? I assume it's triggered by dput.
[17:21] <bigjools> and it's also an easy check to see if it's the PRIMARY archive
[17:21] <bigjools> EdwinGrubbs: let me check where the best place to do this is
[17:23] <adiroiban> intellectronica: back. I don't know how to fix those jslint.
[17:24] <bigjools> EdwinGrubbs: see lib/lp/soyuz/model/publishing.py
[17:25] <bigjools> EdwinGrubbs: IPublishingSet.newSourcePublication()
[17:25] <adiroiban> intellectronica: as for LEFT and RIGHT keys, it looks like Windmill does not support trigering such events
[17:26] <bigjools> EdwinGrubbs: you need to check that the archive.purpose = PRIMARY, then you can see if the DSP exists or not and create/update
[17:26] <bigjools> that's much easier to test than a trigger :)
[17:26] <adiroiban> intellectronica: here is the windmill bug http://github.com/windmill/windmill/issues#issue/33
[17:27] <bigjools> EdwinGrubbs: have you figured out which series you need to cache data for?
[17:27] <bigjools> bearing in mind development is newer than current
[17:28] <bigjools> having said that, section is not likely to change
[17:31] <intellectronica> adiroiban: ok, let me see if i can get rid of the jslint errors. we can't land with them.
[17:35] <adiroiban> intellectronica: I had those errors in other branches and Danilo said that it is ok and this is the way YUI3 works
[17:37] <intellectronica> adiroiban: i rather check first, and at least register the problem somehow. it's not a good idea to keep jslint errors in the codebase.
[17:37] <danilos> adiroiban, I don't remember saying so unless if it was a branch where you didn't touch the faulty JS which I don't want to hold you responsible for :)
[17:41] <EdwinGrubbs> bigjools, I don't know for which series the section should be cached. Maybe sinzui knows.
[17:42] <intellectronica> adiroiban: also, i think that at this point it would make sense to assign the magic numbers to CONSTANTS, but I won't ask you to do it because i see that you're just extending legacy code.
[17:42] <sinzui> series cache>
[17:42] <bigjools> EdwinGrubbs: it's an important question since this table will be useful beyond your use case I think
[17:42] <sinzui> Is someone watching my keystrokes as I addtal:content="cache:public, 1 hour"
[17:42] <sinzui> to series templates?
[17:42] <bigjools> \m/
[17:43] <adiroiban> intellectronica: I will create those contants as it should improve the code readability
[17:43] <intellectronica> adiroiban: excellent, thanks!
[17:44] <sinzui> bigjools, I think I am working on the issue you are talking about. I am working on the timeouts for distroseries and milestones that dominate oops
[17:45] <bigjools> sinzui: I am checking over Edwin's branch, I guess you're both working on this?
[17:46] <gmb> abentley-lunch, I've added a formatter for oops IDs and some tests to go with it. I've pasted a diff in the MP for your perusal.
[17:46] <sinzui> bigjools, I made the initial proposal. and I liked EdwinGrubbs suggested changes. I have not looked at the branch though
[17:48] <sinzui> bigjools, I suppose you are remarking that once a series is released, the packages should not change.
[17:48] <sinzui> is backports main?
[17:49] <intellectronica> adiroiban: that code is a bit weird, no wonder jslint complains
[17:49] <bigjools> sinzui: well a couple of things, 1. we should do it in code not in a trigger (soyuz will benefit from cache publishing data in places so I want to prepare for that), 2. yes a released series won't change
[17:50] <intellectronica> danilos: ^^^^^
[17:50] <bigjools> sinzui: backports is a pocket, not sure what you mean?
[17:50] <bigjools> you mean the primary archive?
[17:50] <sinzui> bigjools, thanks for your incite
[17:50] <sinzui> yes I mean primary archive
[17:51] <bigjools> so yes -backports is the primary archive
[17:51] <sinzui> yuck
[17:51] <bigjools> problems?
[17:52] <sinzui> um, no actually. I had a momentary lapse of reason
[17:52] <bigjools> I get that all the time :)
[17:52]  * sinzui is still thinking of series and milestones
[17:55] <EdwinGrubbs> sinzui, on which page will you be querying the cached section from the DSP table? It doesn't look like that is used on the +needs-packaging page.
[17:55] <adiroiban> intellectronica: let me rewrite it. what do you consider to be weird ? :)
[17:57] <intellectronica> adiroiban: i don't understand what is them `stem` second argument that the even handlers receive
[17:58] <intellectronica> adiroiban: oh, is it the last argument you pass to on()?
[17:58] <sinzui> EdwinGrubbs, I think the root page is the only user of memcached. I am wrapping portlets.
[17:58] <sinzui> EdwinGrubbs, I have not tested memcached with a batched listing. I do not think it will work
[18:05] <adiroiban> intellectronica: let me rewrite that code as I realise it is just stupid :)
[18:05] <intellectronica> adiroiban: i have a solution for the first batch of lint errors, but it ain't pretty :)
[18:06] <intellectronica> adiroiban: what were you thinking of doing?
[18:06] <adiroiban> intellectronica: those functions are adapters for toggleWidget()
[18:07] <adiroiban> as the YUI3 will pass the event as the first argument
[18:07] <adiroiban> I can not use them directly with Y.on
[18:09] <intellectronica> adiroiban: yeah, got it. the solution i have is to enclose the handlers in a closure. it isn't necessary in this case, since you're not capturing anything but using yui's event machinery to pass an argument, but jslint is correct to make that suggestion, and a nicer solution would require refactoring the code.
[18:09] <EdwinGrubbs> sinzui, What does memcached have to do with the DistributionSourcePackage.section column?
[18:09] <intellectronica> adiroiban: a nicer solution might be to move the body of the loop into a separate function
[18:10] <sinzui> EdwinGrubbs, memcached+tales caches page fragments. Which page/part do you want me to test?
[18:11] <adiroiban> intellectronica: i was thinking to create those adapter functions outside the loop
[18:12] <intellectronica> adiroiban: yes, that's also a possibility. whichever you prefer.
[18:21] <sinzui> EdwinGrubbs, memcached+tales does include the query string in the key. I think we can cache batches
[18:21] <EdwinGrubbs> sinzui, I'm adding the section column to the DistributionSourcePackage table to cache the data from the SPPH table. I think you told me to do that on Friday. I don't know where the query is that we are trying to speed up. This should be unrelated to memcached.
[18:23] <sinzui> EdwinGrubbs, yes, it is unrelated. To exclude language-packs, we use the section which is know in spph or in spr. I used spph in the branch I am landing today
[18:23] <sinzui> EdwinGrubbs, yes, it is unrelated. To exclude language-packs, we use the section which is know in spph or in spr. I used spph in the branch I am landing today
[19:14] <leonardr> abentley, i'd like to add https://code.edge.launchpad.net/~leonardr/lazr.restful/understand-mod-compress/+merge/25454 to the review queue
[19:59] <mars> gary_poster, ping, have a moment to review https://code.edge.launchpad.net/~mars/launchpad/disable-windmill-tests/+merge/25457 ?
[19:59] <gary_poster> mars, ok
[20:03] <gary_poster> mars, you added in TESTOPTS for theoretical correctness, yes?
[20:03] <gary_poster> that is, no one is actually using them, it seems?
[20:03] <mars> gary_poster, it was already in there, just not used properly.
[20:04] <mars> they are used by three targets lower down in the file
[20:04] <gary_poster> ah, gotcha mars
[20:04] <mars> I was going to rename VERBOSITY to TESTOPTS myself, and was surprised to find TESTOPTS was already in there
[20:05] <gary_poster> yeah, another approach would be to rip it out, but I suppose this is safer
[20:05] <mars> using the value of the VERBOSITY variable to pass "-vvv -t --layer" to the test process really rubbed me the wrong way.  Refactoring gene I guess :/
[20:05] <mars> gary_poster, safer, yes.  I don't know who else used VERBOSITY that way.
[20:06] <gary_poster> mars, have you checked to see what happens to the test runner if you say --layer='!WindmillLayer' --layer='WindmillLayer'?
[20:06] <mars> Needed a fast fix.  This is one step forward.
[20:06] <mars> My guess is that --layer=!Layer --layer=Layer *will* execute the desired test suite.
[20:06] <gary_poster> well, for a fast fix, maybe leaving TESTOPTS out of the picture would be wiser.  Otherwise I have questions about it
[20:06] <mars> But no, I have not explicitly tested that :)
[20:07] <mars> gary_poster, oh?  What questions?
[20:07] <gary_poster> for instance--if you are right (which would be my guess) then I would suggest that ``bin/test $(VERBOSITY) $(TESTOPTS) --layer=WindmillLayer`` should instead be ``bin/test $(VERBOSITY) --layer=WindmillLayer $(TESTOPTS)`` so that TESTOPTS can actually override things
[20:07] <mars> gary_poster, I treat TESTOPTS like CFLAGS.  VERBOSITY is independent of the two.
[20:08] <mars> gary_poster, ok, interesting.  That woudl have come up regardless if I used VERBOSITY or TESTOPTS.
[20:09] <gary_poster> yeah, calling it TESTOPTS is more correct, and makes the possible problem clearer.
[20:09] <gary_poster> mars, I'll approve, simply asking that you consider my question...possibly later. :-)
[20:09] <mars> +1
[20:10] <mars> gary_poster, I'll send a post to the list when this lands.
[20:10] <gary_poster> mars, ok.  approved.
[20:10] <mars> a post about the suite disablement
[20:10] <gary_poster> right
[20:10] <mars> thanks gary_poster
[20:11] <gary_poster> mars, also note that when this branch lands the linked bug will be marked fix committed I think.  If so, I'm guessing (?) you'll want to manually change it back, since this is a bandaid not a fix.
[20:12] <mars> gary_poster, yep!  Had to do that a few times now.  The QA scripts also do that.
[20:12] <mars> I'm working against the grain!
[20:12] <gary_poster> :-)
[21:16] <leonardr> abentley, i don't know if you're still reviewing but i've also got a launchpad follow-up to my earlier lazr.restful branch
[21:16] <leonardr> https://code.edge.launchpad.net/~leonardr/launchpad/understand-mod-compress
[21:19] <abentley> leonardr, did you mean to target your branch at db-devel, rather than devel?
[21:20] <leonardr> abentley: i don't think so, i generally target devel--is devel closed atm or something?
[21:21] <abentley> leonardr, devel is not closed.  You have proposed for landing against db-devel.  This means that some other changes from devel are showing in your diff.
[21:21] <leonardr> abentley: i misunderstood your question
[21:21] <leonardr> i thought you were saying "you targeted your branch at devel, but you probably meant db-devel" rather than vice versa
[21:22] <abentley> leonardr, ah, thought so.
[21:22] <leonardr> abentley, i'm redoing the proposal
[21:22] <abentley> leonardr, thanks.
[21:23] <leonardr> done
[21:39] <abentley> leonardr, there should be two blank lines before a new header in a docstring.
[21:41] <abentley> leonardr, also, the differences between '"foo-gzip"' and '"foo"' info '"foo"-gzip' are pretty subtle.  It would be nice to mention them both at the same time.
[21:42] <abentley> e.g.:  mod_compress may turn '"foo"' into '"foo"-gzip' or '"foo-gzip"'.  We treat both forms as the same.
[21:43] <abentley> also, you say "info" where you mean "into".
[22:03] <leonardr> gary: abentley reviewed my launchpad branch but not the lazr.restful branch it uses. would you take a look?
[22:03] <leonardr> https://code.edge.launchpad.net/~leonardr/lazr.restful/understand-mod-compress/+merge/25454
[22:03] <gary_poster> leonardr: ack, looking
[22:13] <adiroiban> intellectronica: i have updated the JS code adding constanst for key codes and fixing jslint warnings.
[22:13] <gary_poster> leonardr: approved, no comments.  thank you
[22:13] <leonardr> gary, great


[08:41] -queuebot:#ubuntu-release- Unapproved: accepted neutron [source] (bionic-proposed) [2:12.0.5-0ubuntu5]
[08:50] -queuebot:#ubuntu-release- New: accepted linux-signed-azure [amd64] (bionic-proposed) [4.18.0-1019.19~18.04.1]
[08:50] -queuebot:#ubuntu-release- New: accepted linux-signed-azure [amd64] (cosmic-proposed) [4.18.0-1019.19]
[08:56] <rbalint> sil2100, could you please release livecd-rootfs sru-s? it is early but it is a special package
[09:00] <sil2100> rbalint: let me look at those in some moments
[09:05] -queuebot:#ubuntu-release- New binary: linux-signed-azure [amd64] (disco-proposed/main) [5.0.0-1007.7] (core, kernel)
[09:09] -queuebot:#ubuntu-release- New: accepted linux-signed-azure [amd64] (disco-proposed) [5.0.0-1007.7]
[11:14] -queuebot:#ubuntu-release- Unapproved: accepted apt-setup [source] (bionic-proposed) [1:0.104ubuntu5.1]
[11:57] -queuebot:#ubuntu-release- New binary: linux-signed-gcp [amd64] (cosmic-proposed/main) [4.18.0-1012.13] (kernel)
[12:12] -queuebot:#ubuntu-release- New: accepted linux-signed-gcp [amd64] (cosmic-proposed) [4.18.0-1012.13]
[12:14] -queuebot:#ubuntu-release- New binary: linux-signed [amd64] (cosmic-proposed/main) [4.18.0-21.22] (core, kernel)
[12:14] -queuebot:#ubuntu-release- New binary: linux-signed [ppc64el] (cosmic-proposed/main) [4.18.0-21.22] (core, kernel)
[12:15] -queuebot:#ubuntu-release- New: accepted linux-signed [amd64] (cosmic-proposed) [4.18.0-21.22]
[12:15] -queuebot:#ubuntu-release- New: accepted linux-signed [ppc64el] (cosmic-proposed) [4.18.0-21.22]
[13:08] -queuebot:#ubuntu-release- New binary: linux-signed-azure [amd64] (xenial-proposed/main) [4.15.0-1046.50] (kernel)
[13:34] -queuebot:#ubuntu-release- New: accepted linux-signed-azure [amd64] (xenial-proposed) [4.15.0-1046.50]
[13:35] -queuebot:#ubuntu-release- New binary: linux-signed-hwe [amd64] (xenial-proposed/main) [4.15.0-51.55~16.04.1] (kernel)
[13:36] -queuebot:#ubuntu-release- New binary: linux-signed-hwe [ppc64el] (xenial-proposed/main) [4.15.0-51.55~16.04.1] (kernel)
[13:42] -queuebot:#ubuntu-release- New: accepted linux-signed-hwe [amd64] (xenial-proposed) [4.15.0-51.55~16.04.1]
[13:42] -queuebot:#ubuntu-release- New: accepted linux-signed-hwe [ppc64el] (xenial-proposed) [4.15.0-51.55~16.04.1]
[14:13] <coreycb> infinity: sil2100: hello, if you have some cycles we have a nova SRU in the bionic unapproved queue that is feeling some heat
[14:20] <sil2100> coreycb: ACK
[14:21] -queuebot:#ubuntu-release- Unapproved: accepted nautilus [source] (disco-proposed) [1:3.32.1-0ubuntu<IP_ADDRESS>]
[14:22] -queuebot:#ubuntu-release- New binary: linux-signed-gcp [amd64] (bionic-proposed/main) [4.15.0-1033.35] (kernel)
[14:32] -queuebot:#ubuntu-release- New: accepted linux-signed-gcp [amd64] (bionic-proposed) [4.15.0-1033.35]
[14:47] -queuebot:#ubuntu-release- Unapproved: accepted gnome-software [source] (disco-proposed) [3.30.6-2ubuntu<IP_ADDRESS>]
[15:05] -queuebot:#ubuntu-release- Unapproved: accepted nova [source] (bionic-proposed) [2:17.0.9-0ubuntu3]
[15:09] -queuebot:#ubuntu-release- New binary: linux-signed-gcp [amd64] (xenial-proposed/main) [4.15.0-1033.35~16.04.1] (kernel)
[16:36] -queuebot:#ubuntu-release- New: accepted linux-signed-gcp [amd64] (xenial-proposed) [4.15.0-1033.35~16.04.1]
[17:16] -queuebot:#ubuntu-release- New binary: linux-signed-gcp-edge [amd64] (bionic-proposed/main) [4.18.0-1012.13~18.04.1] (kernel)
[23:40] -queuebot:#ubuntu-release- New binary: swift-im [s390x] (eoan-proposed/universe) [3.0.4-1.1] (no packageset)
[23:46] -queuebot:#ubuntu-release- New binary: swift-im [amd64] (eoan-proposed/universe) [3.0.4-1.1] (no packageset)
[23:49] -queuebot:#ubuntu-release- New binary: swift-im [i386] (eoan-proposed/universe) [3.0.4-1.1] (no packageset)
[23:51] -queuebot:#ubuntu-release- New binary: swift-im [ppc64el] (eoan-proposed/universe) [3.0.4-1.1] (no packageset)