[00:02] <jono> what is the best of getting a list of apps in the archive written in pygtk that use gtkstatusicon?
[00:05] <RAOF> Hm.  I _think_ you'll have to grab the sources of all the packages that depend on python-gtk2 and grep for gtk.StatusIcon.
[00:08] <RAOF> I'd be interested to hear any better solutions, though :)
[00:20] <slangasek> ScottK: yay, removals are my favorite!
[00:21] <johanbr> jono, for packages in debian, you can use http://source.debian.net/source/
[00:21] <jono> thanks johanbr
[00:22] <johanbr> you're welcome :)
[00:31] <tweakt_> should I be able to preseed language, country and keyboard settings?
[00:32] <cjwatson> tweakt_: yes but only using boot parameters
[00:32] <tweakt_> ahh, that explains it
[00:33] <cjwatson> the programs that handle file/url preseeding run after language/country/keyboard have been asked
[00:38] <robert_ancell> jamesh, is there a repository for launchpad-integration? I have a patch for C# support
[00:43] <cjwatson> johanbr: oh my, I hadn't seen that before.  that's fantastic.
[00:44] <johanbr> saw that yesterday on a Fedora mailing list, of all places
[00:44] <cjwatson> this is going to save me a ridiculous amount of bandwidth over time.
[00:44] <johanbr> :)
[00:46] <micahg> robert_ancell: https://edge.launchpad.net/lpx
[00:49] <robert_ancell> micahg, i don't see it there - I'm talking about the package "launchpad-integration" which adds menu items into GTK+ apps help menu
[00:51] <persia> robert_ancell: launchpad-integration is a native package, so lp:ubuntu/lucid/launchpad-integration might be a reasonable thing to branch
[00:51] <persia> robert_ancell: Also, apt-cache show launchpad-integration shows an Original Maintainer who tends to be around, you might also contact that individual.
[00:52] <micahg> robert_ancell: ah, sorry
[00:53]  * micahg missed the -
[00:54] <robert_ancell> persia, yes, as far as I can tell jamesh is the original author and seb128 packaged it.  I'm guessing the answer is no (there is not an official repository) but I can bully him into putting it somewhere oficially. (and getting mvo to link it to lpx - thanks michag)
[00:54] <jamesh> robert_ancell: you should be able to find what you're after here: https://launchpad.net/launchpad-integration
[00:55] <jamesh> I haven't been involved with it much since the first few versions though.
[00:55] <robert_ancell> jamesh, is it stored in version control somewhere?  Could I encourage you to make a lp:launchpad-integration branch :)
[00:55] <robert_ancell> jamesh, ok
[00:55] <jamesh> robert_ancell: lp:~ubuntu-core-dev/launchpad-integration/ubuntu looks like the trunk
[00:56] <jamesh> looks like mvo is the one who'd have permissions to make that branch accessible as lp:launchpad-integration
[00:57] <robert_ancell> jamesh, thanks, that's what I was looking for.  I'll bug mvo to update the LP page
[01:02] <persia> I'm not convinced we want to encourage the practice of enabling lp:foo for native packages where there isn't a distinct upstream.
[01:02] <persia> More likely, it would make sense to integrate that into lp:ubuntu/* and update the tags appropriately.
[01:03] <persia> (unless the content is useful outside the context of the distribution, in which case I question whether it should be a native package).
[01:03] <jamesh> persia: but if people (like robert_ancell) want to make contributions, they'll want to base them on that branch
[01:03] <jamesh> that seems like enough reason to set it as the development focus
[01:04] <persia> (for more thoughts on the matter, read backscroll from about 11 hours ago, talking about ubiq_uity and joc_key
[01:04] <persia> jamesh: Except I think that branch ought to go away for the new model of distributed development.
[01:05] <persia> Because either the software is a separate upstream (in which case it doesn't make sense for ~ubuntu-core-dev to be the trunk branch owner), or it isn't (in which case there's little value to the project external to Ubuntu and that branch only exists as a historical workaround until lp:/ubuntu/* worked)
[01:06] <jamesh> persia: I don't really see what difference it makes though.  If the branch owned by ~ubuntu-core-dev is where the development occurs, then it is the development focus
[01:07] <jamesh> if development occurs elsewhere, then it is not.
[01:07] <jamesh> the development focus might also be the branch where the packaging is managed, but that is separate.
[01:07] <cjwatson> the difference is the same as the difference between a "real" upstream package with independent upstreamish existence, and a native package that exists just for the distribution
[01:07] <persia> Yes, but my argument is that the current branch is a temporary workaround, and it makes sense to fix that rather than setting focus.
[01:08] <persia> The two ways to fix it being 1) set an upstream team and branch or 2) use the DistributedDevelopment branch
[01:08] <cjwatson> which are definitely two different things in distro people's heads - in particular native distro packages aren't particularly expected to be used/usable by other distros
[01:09] <jamesh> if some other distro shows an interest, then it might make sense to separate things out
[01:09] <jamesh> but right now, it seems pretty clear that the ~ubuntu-core-dev branch is the development focus in this specific case
[01:09] <jamesh> that might not always be true, but it is true now.
[01:10] <cjwatson> what persia is I think arguing is that it does not make sense for there to be a project for it at all
[01:10] <persia> Or if there should be a project, the trunk branch should be defined differently and the packaging done differently.
[07:30] <didrocks> good morning
[07:30] <dholbach> good morning
[07:35] <didrocks> superm1: about bug #507121, I've track it down and it's /usr/lib/user-setup/user-setup-apply which screw up custom.conf in both casper and ubiquity phases. So, I'll push a fix today and ping you so that you can remove your workaround
[07:36] <persia> If I want to ask for non-free (unable to modify) data be moved from multiverse to restricted, is that just an MIR, or does it first require a request to the TB?
[10:47] <coz_> hey guys.. out of curiosity...for xplash the frames set are 50  I believe... where do I change that setting?
[10:47] <coz_> also is there a file to change the location of the animated xsplash on the screen?
[10:56] <arand> coz_: afaik, it must be done in the source before compiling, and it's rather untrivial.
[10:56] <coz_> arand,  mm  ok    so  essentially 50 frames   I can deal with that :)
[10:57] <coz_> arand,  thanks
[11:13] <woozy_> coz_: the framerate can be controlled by giving xsplash a parameter on startup, xsplash --help should tell you which one
[11:14] <coz_> woozy_,  ah   that may help quite a bit  thanks :)
[11:22] <dholbach> james_w, kenvandine, sommer, crimsun: can anybody of you please block dav23 from the !ubuntu identi.ca group?
[11:34] <didrocks> cjwatson: if you have any time to sponsor bug #507121 to fix all that session stuff in persistent usb key and after ubiquity install, thanks :)
[11:42] <cjwatson> didrocks: in progress
[11:42] <didrocks> cjwatson: thanks
[11:57] <jussi01> Hey all, are there any ~ubuntu-archive members about atm?
[12:13] <seb128> jussi01, you should ask your question
[12:23] <geser> that reminds me: can an archive admin please check why libxcb in lucid is still 1.4-1 and the both syncs of 1.5 failed in the end? (see bug #492247 and bug #503290)
[12:28] <seb128> geser, looking
[12:28] <seb128> geser, 2010-01-15 12:28:19 ERROR   Exception while processing upload /home/lp_queue/sync-queue/incoming/seb128-20100115-122812 (OOPS-1476FTPMASTER1)
[12:29] <seb128> geser, I'm pinging soyuz guys to know if that's a known bug
[12:47] <cjwatson> didrocks: merged the user-setup one.  For future reference you don't really need ubiquity tasks when it's just a standard rebuild, but I've merged that branch anyway
[12:48] <didrocks> cjwatson: oh, ok. I note it down. Thanks :)
[12:52] <LordMetroid> Alright, I am going to go ahead and break my system
[12:52] <LordMetroid> See you one the other side of alpha2
[12:52] <persia> Could someone press the retry button on https://launchpad.net/ubuntu/+source/metacity/1:2.28.0-2ubuntu2/+build/1430446
[12:53] <cjwatson> persia: done
[12:53] <persia> cjwatson: Thanks.
[14:01] <mdeslaur> slangasek: do you know of anyone who can test a libthai security update?
[14:01] <jdstrand> mdeslaur: actually, ArneGoetje may be able to
[14:01] <mdeslaur> ah! cool, thanks jdstrand
[14:02] <jdstrand> mdeslaur: I'm not sure, but he is our lang guy
[14:03] <mdeslaur> jdstrand: thanks, I'll ask him
[14:03] <mdeslaur> ArneGoetje: ping
[14:03] <persia> Seems odd that we don't have an #ubuntu-th
[14:05] <mdeslaur> persia: we don't? or are you being ironic? :)
[14:06] <persia> mdeslaur: It's not listed as an official channel
[14:06] <persia> But you could ask there, because it's *supposed* to be the Thai channel.
[14:06] <persia> (where official channels are defined at https://help.ubuntu.com/community/InternetRelayChat#Channels , although some might argue that a fairly open wiki is a strange way to define "official")
[14:06] <mdeslaur> persia: oh, ok. thanks.
[14:29] <RainCT> pitti: Hey. Seems like you forgot to push your changes to ubuntu-dev-tools to the branch (and you also skipped the 0.88 release :)).
[14:32] <persia> RainCT: So, based on some discussion about 24 hours ago, there's apparently some procedure that can be done to switch from the current branch to just using lp:ubuntu/${release}/ubuntu-dev-tools .  Do you happen to know who most owns that package?
[14:33] <RainCT> persia: Oh. No, I haven't looked into any of that VCS stuff yet.
[14:35] <apw> pitti, does your new burndown data include the kernel wiki page?
[14:38] <RainCT> persia: It looks like that's handled transparently, though. lp:ubuntu/karmic/ubuntu-dev-tools already exists, and "bzr info" shows lp:~ubuntu-branches/ubuntu/karmic/ubuntu-dev-tools/karmic/
[14:38] <cjwatson> RainCT: that's probably an automatic import rather than the existing branch, though.  I suspect you'll find it isn't nearly so detailed.
[14:39] <cjwatson> RainCT: the procedure for switching the branch references is to file a bug on the 'udd' project.
[14:43] <RainCT> cjwatson: Oh, OK. So it's not ready for mass consumption yet :)
[14:44] <cjwatson> RainCT: I didn't say that?
[14:44] <apw> pitti, does your new burndown data include the kernel wiki page?
[14:44] <cjwatson> RainCT: it's generally fine, but it needs tweaks when people were already using revision control to manage their package
[14:45] <persia> But it has the excellent super-cool feature of automatically dealing with uploads, so we don't have to pester the forgetful uploader to push to the branch.
[14:46] <persia> (or at least that's the impression I have)
[14:46] <RainCT> cjwatson: So you aren't supposed to be able to just pull and push, but need to ask for access to the branch? Or is that just for now (which is what my comment referred to)
[14:46] <persia> RainCT: It's that the real history of the development branch needs to be merged into the import branch.
[14:47] <RainCT> OK. So if I take some other package which was't maintained in bzr before I can push there?
[14:49] <persia> RainCT: https://wiki.ubuntu.com/DistributedDevelopment might be a good primer
[14:50] <RainCT> persia: Yeah, I was trying to avoid reading all that :)
[14:51] <persia> heh :)
[14:51] <persia> But as long as we're managing ubuntu-dev-tools in bzr anyway, it seems reasonable we should use the branches made available in that system.
[14:52] <cjwatson> RainCT: I believe you've misunderstood me somehow.
[14:52] <persia> Then again, I don't know the system well enough to drive that (but I hope I've already completed all my ubuntu-dev-tools changes for lucid)
[14:52] <cjwatson> RainCT: ~ubuntu-branches is magic.  Anyone who can upload the package can edit the branch.
[14:53] <RainCT> cjwatson: So your comment was only about merging in the old history?
[14:53] <cjwatson> RainCT: but, as I said above, if you already have a branch with more detailed history, it would be more appropriate to get the lp:ubuntu/... pointers updated to point to that.
[14:53] <cjwatson> RainCT: my comment was about changing what lp:ubuntu/... points to
[14:53] <RainCT> cjwatson: OK, I see. Thanks for the clarification.
[14:54] <cjwatson> I would not recommend merging the old history into the automatically imported branches in the sense of 'bzr merge' - the result is not likely to be pretty.  It is usually more appropriate to ignore the automatically imported branches if you already have better ones, but it *is* still useful in that case to get lp:ubuntu/... updated.
[14:55] <persia> cjwatson: But if we redirect lp:ubuntu/... will we lose the magic access control and automatic import of uploads?
[14:56] <cjwatson> persia: probably.
[14:56] <cjwatson> I'm not sure about access control.
[14:56] <cjwatson> alternatively you could push --overwrite over the top of the ~ubuntu-branches branch - but ask james_w first, and I'd recommend keeping a copy elsewhere
[14:57] <persia> Hrm.  I think I'll have to learn more about this before promoting it further.
[15:51] <kees> cjwatson: oh, btw, can I help at all with debian bug 562048 ?  it's a workitem for the security team, so I'm hoping you might have time to review it.  Do you want me to s/ReportExtraversion/DebianBanner/ on the proposed patch and resubmit?
[15:52] <cjwatson> kees: more hours in day pls; I'm finishing up a task this week, so ask me at the start of next week?
[15:53] <kees> cjwatson: okay, thanks.  :)
[15:55] <hwilde> hours++
[16:09] <LordMetroid> Breakage of my system complete, I am now in Alpha2, just a restart away(*shudders*)
[16:16] <Incubuss> Can anyone confirm/deny for me that the Interface tab has been removed from Appearance Preferences in gnome?
[16:17] <seb128> Incubuss, it has
[16:19] <Incubuss> any idea why or if the options being reimplemented anywhere else?
[16:19] <seb128> they are gconf keys
[16:20] <seb128> I think they decided that's something useful in the ui
[16:34] <directhex> seb128, so there's now no way to revert the ass-backwards "no icons in menus please" behaviour without gconf-editor?
[16:34] <seb128> no
[16:35] <Incubuss> Ubuntu-tweak makes it possible to get them back but it still took me a good 5 minutes to find the option
[16:46] <Sweb> hi need help with ubuntu
[16:46] <Sweb> any user here?
[16:48] <LordMetroid> Ohh noes, alpha2 can not even start, it totally locks during bootup
[16:48] <LordMetroid> Anyone else have this problem?
[17:52] <slangasek> asac: so, what specs are missing from your report?
[17:53] <slangasek> pitti: could we get some kind of final run of the alpha-2 burndown charts, to verify that all the unfinished workitems have been postponed?
[18:29] <asac> slangasek: so on: http://macaroni.ubuntu.com/~pitti/workitems/canonical-mobile.html
[18:29] <asac> 1st. we have like 30% work item loss over what we had on the old one ..
[18:29] <asac> but i think that was due to the new parser being pickier about the syntax
[18:29] <slangasek> ok
[18:29] <asac> i am going through the itesm now to verify everything is fine
[18:29] <asac> then we have to wait and see if thats good
[18:30] <asac> 2nd. there are specs that dont belong there ;)
[18:30] <asac> desktop-lucid-xorg-halsectomy
[18:30] <asac> foundations-lucid-dropping-sun-java6
[18:30] <asac> 3rd. similarly i suspect that we lost one or two specs ... will check that after doing 1st ;)
[18:30] <slangasek> asac: well, they belong there because the work items are assigne to members of your team
[18:30] <asac> ah ;)
[18:30] <asac> ok
[18:30] <slangasek> that's by design
[18:30] <asac> thats good then
[18:31] <asac> ok. then maybe 3rd is not the case and the loss of items is due to the syntax
[18:31] <asac> lets check how it looks on monday then.
[18:31] <slangasek> ok
[18:32] <asac> and firefox-support spec obviously too ... ok
[18:32] <asac> ;)
[18:32] <slangasek> assign them all to ccheney? :)
[18:33] <asac> i plan to do that next time pitti is about to deliver a all 100% report ... just to nag him :-P
[18:33] <asac> ok enjoy weekend
[18:33] <slangasek> might be more productive if ccheney has more warning that he's responsible for them ;)
[18:33] <slangasek> you too!
[18:49] <Nafallo> kirkland: hey. take a look at bug #505909 when you have a moment.
[18:50] <Nafallo> meh. that wasn't a message. but at least the bot got something to do...
[20:34] <niktaris> hi,  I am remastering the latest version of ubuntu but can't seem to dist-upgrade it. It keeps failing on kernel and rsyslog. Any Ideas ?
[20:36] <niktaris> a yes, by dist-upgrade I mean via chroot
[20:52] <BBHoss> Does anyone remember a bug a while back with tickless kernels? that made the cpu usage go up for things that used timers heavily?
[20:52] <BBHoss> specifically FreeSWITCH
[21:18] <mathiaz> kees: hi - I'm writting a blog post to announce the demotion of nis to universe
[21:18] <mathiaz> kees: what are the main reasons for such a demotion?
[21:18] <mathiaz> jdstrand: insecure? old? unmaintained?
[21:18] <mathiaz> jdstrand: ^^
[21:19] <slangasek> insecure-by-design protocol?
[21:19] <jdstrand> it's both old and a bad design with better alternatives
[21:19] <jdstrand> well, a bad design in the modern age, I guess it was the cat's pajamas back in the day
[21:20] <jdstrand> (always insecure though)
[21:20] <slangasek> I have never known a cat that would wear NIS pajamas
[21:20] <jdstrand> you need an old cat
[21:22] <siretart> slangasek: in combination with kerberos, nis turns out to be pretty useful
[21:22] <slangasek> siretart: no, it's still insecure.
[21:22] <slangasek> unless you mean you've found a kerberos-enabled implementation of NIS itself?  which the one we ship isn't
[21:22] <slangasek> (and then it's pretty much not protocol-compatible)
[21:23] <siretart> no, at my department, we removed password hashes from our nis database and use kerberos for authenticating
[21:23]  * kees agrees with jdstrand and slangasek
[21:24] <slangasek> siretart: that's not secure.  Anyone with access to your network can spoof the maps.
[21:24] <kees> "my uid is 0!"
[21:25] <siretart> slangasek: well, true, we have additional measures in our switches. which we would want even if we would switch to ldap anyway
[21:26] <mathiaz> kees: http://paste.ubuntu.com/357257/ <- does this look ok?
[21:27] <kees> mathiaz: ieee line wrap ;)  reading...
[21:27] <mathiaz> kees: sorry :/
[21:27] <kees> mathiaz: yeah, sounds about right.
[21:27] <mathiaz> kees: great - thanks for the quick review!
[21:28] <kees> np
[21:59] <lamont> I wonder - how long is openclipart supposed to take to build...
[21:59] <lamont> thallium has been working on it for a while
[22:03] <sistpoty> create random set of pixels, check if it looks like a clipart, if not -> begin again :)


[11:57] <hazmat> fwiw finishing followup gary_poster|away
[13:08]  * benji reboots after installing updates.
[13:31] <frankban> good morning gary_poster, do you have a minute for a quick call?
[13:31] <gary_poster> frankban, on call, ping you after?
[13:31] <frankban> gary_poster: sure thanks
[13:31] <gary_poster> frankban, when's your lunch?
[13:32] <frankban> gary_poster: done
[13:32] <gary_poster> ok
[14:34] <gary_poster> frankban, juju-ui?
[14:34] <frankban> gary_poster: ok
[15:15] <therve> hazmat, around?
[15:15] <hazmat> therve, yes
[15:15] <hazmat> therve, what's up
[15:15] <therve> hazmat, do you have some time for questions? there is still one piece of the puzzle I'm missing
[15:20] <hazmat> therve, in 1m for 10m yes
[15:20] <therve> ok :)
[15:22] <hazmat> therve, https://plus.google.com/hangouts/_/578ca1fef07eca8375038c223c90d5cdb2667c70?authuser=0&hl=en
[15:27] <gary_poster> Makyo, on call but could you try tests on trunk, make test-debug?  I think there is a test failure that you could identify quickly
[15:27] <Makyo> gary_poster, sure thing.
[15:27] <gary_poster> ty Makyo
[15:30] <Makyo> gary_poster, Weird.  Alright, looking into it.
[15:34] <bac> gary_poster: hanging out?
[15:34] <gary_poster> bac, y
[15:44] <bac> gary_poster: are you attempting to join?
[15:44] <hazmat> gary_poster, ping hangout
[15:44] <gary_poster> bac, hazmat, coming
[15:53] <bac> teknico: i've gotten complaints lately that my audio is not understandable due to choppiness so i thought i'd try the low bandwidth for video.  hope it helped.
[15:54] <teknico> bac, I noticed no problems with your audio
[15:54] <hazmat> bac it did
[15:54] <bac> perhaps a static picture might be less annoying than the line drawing.
[15:54] <hazmat> bac but i normally don't have issues with your audio
[15:54] <hazmat> bac, for me without the low bandwidth, i wouldn't frequently get freeze ups and drops
[15:54] <hazmat> er.. would
[15:54] <bac> hazmat: how do you get service?  dsl?
[15:55] <teknico> and I'm always in favor of anything that helps correcting the gross imbalance between video and audio in contemporary media culture!
[15:55] <teknico> I don't find the solarized/line drawing video annoying at all anyway
[15:56] <teknico> bac, did you happen to try the low bandwidth option (the one without "ultra") too?
[15:56] <bac> teknico: not yet
[15:56] <teknico> and hazmat too ^^
[15:57] <hazmat> teknico, not yet
[18:17] <teknico> does -req lbox option work? on two attempts out of two I got:
[18:17] <teknico> error: Failed to send patch set to codereview: diff is empty
[18:17] <teknico> does *the -req...
[18:19] <teknico> and, can the merge proposals for two branches be linked to the same bug, in lbox?
[18:27] <teknico> resubmitted without prerequisite, unfortunately the diff will be much bigger than needed, until the first branch lands
[19:22] <bac> bcsaller: it looks like your latest merge broke one test.  have you seen that?
[19:23] <bcsaller> bac: no, what are you seeing?
[19:23] <bac> gary_poster, benji: could one of you review my branch in secondary?
[19:23] <benji> bac: sure
[19:24] <bac> bcsaller: must be able to add a relation from the control panel ‣
[19:24] <bac> expected '-58' to not equal '-58'
[19:24] <Makyo> bac, https://codereview.appspot.com/7102055/
[19:24] <bac> bcsaller: i haven't investigated, just noted that reverting to r313 does not show the problem
[19:25] <bac> Makyo: is that a review request?  i'd be glad to.
[19:25] <bac> Makyo: oh, that's the fix for the problem i just annoyed bcsaller about?
[19:25] <Makyo> bac, A branch that fixes that test.  If you'd be willing to see if that fixes i--yes :)
[19:25] <bcsaller> nice
[20:59] <gary_poster> bac benji and I approved thank you
[20:59] <bac> gary_poster: i saw.  thank you and benji
[20:59] <gary_poster> cool bac
[20:59] <bac> lbox submitting...
[21:00] <bac> followed closely by lbox dog -walk
[21:05] <bac> gary_poster: i just updated config-prod.js on uistage to have the user/passwd
[21:05] <bac> so in ten minutes the login should be disabled
[21:05] <gary_poster> thanks bac on call
[21:05] <bac> np
[21:10] <bac> cool, uistaging no longer prompts for password
[21:44] <benji> gary_poster: (no rush) so, what is the most kanban-y thing to do with the 1090716 card?
[21:44] <gary_poster> benji sorry on call till 5
[21:47] <hazmat> therve, env uuid branch just about ready for review (though finding a reviewer might be tough)
[21:48] <hazmat> twill need a follow up to also make it available off the ws
[21:56] <hazmat> therve, if your up for a review its at https://codereview.appspot.com/7092055
[22:14] <gary_poster> benji now off call, and rushing to next thing.  um. I think the most accurate/historically informative representation will be to keep it active somehow.  Back to coding would be fine.
[22:36] <benji> gary_poster: will do


[09:14] <didrocks> tsdgeos: hey, did you see the email on the unity8 maguro tests failing (they are multiple of them starting on image 181)
[09:15] <didrocks> that can be due to latest Mir
[09:15] <tsdgeos> didrocks: i did see it, and in my evilness i decided to unsee it :D
[09:16] <tsdgeos> didrocks: can't we just deprecate maguro already and be happy?
[09:17] <mhr3> tsdgeos, it seems we're not displaying the activity indicator when waiting for a video thumbnail
[09:18] <mhr3> expected?
[09:18] <mhr3> Cimi, ^?
[09:18] <mhr3> that's for video widget
[09:18] <Cimi> mhr3, I missed that
[09:19] <mhr3> Cimi, isn't it part of lazy image itself?
[09:19] <mhr3> Cimi, if not, pls add
[09:19] <tsdgeos> yeah that's what i though
[09:19] <tsdgeos> by using a LazyImage
[09:19] <Cimi> dunno then
[09:19] <tsdgeos> that we use
[09:19] <tsdgeos> we should get it
[09:19] <tsdgeos> Cimi: please investigate why we don't :D
[09:19] <Cimi> mhr3, test the conditions pls
[09:19] <tsdgeos> maybe
[09:19] <tsdgeos>         visible: state [09:19] <tsdgeos> is breaking it
[09:20] <tsdgeos> probably we need to move that to the image and mousearea, but not to the lazy image
[09:20] <tsdgeos> so we get the spinning wheel
[09:21] <didrocks> asac: happy if we discare the unity8 failures on maguro and promote 181 if dogfooding is good?
[09:21] <didrocks> asac: seems people want to not take it into account even if I multiple times said it would be even worse with the emulator, so better to check for issues
[09:22] <Cimi> tsdgeos, https://code.launchpad.net/~cimi/unity8/units8.previews_RatingStars/+merge/206239
[09:23] <tsdgeos> didrocks: asac: about maguro, yes, there is/was at least a test that i could reproduce all the time back on monday/tuesday and i know it could be worse on the emulator *but* what i was seeing may be caused by a lot of causes, one of them Qt having a bug on slower stuff (we know qt5.0 scene graph loop is bad) so before investing more time in getting it fixed i want to have Qt 5.2 first, and then if it still happens we can devote time to properly
[09:23] <tsdgeos> investigate it. My problem is doing it *now*, not investigating it
[09:23] <tsdgeos> that was a big long ^_^
[09:23] <tsdgeos> didrocks: asac: did i made sense in that sentence?
[09:23] <didrocks> let me reread… it's… long :p
[09:23] <tsdgeos> Cimi: saw it, will do in a bit
[09:24] <didrocks> tsdgeos: if you just take a look at the output and can sense "this is a test only issue, no behavioral broken itself", I'm happy
[09:26] <tsdgeos> didrocks: i can't say that
[09:27] <didrocks> sad we even don't have results on 182 to know if some are reliably failing
[09:27] <tsdgeos> as said, the test failure on maguro (at least the one i debugged a bit at beginning of the week) is stuff that should not be happening
[09:27] <tsdgeos> but as trivial as "something that should be index 1 of a list is index 2"
[09:27] <tsdgeos> so i don't want to debug that in an ancient Qt
[09:28] <didrocks> tsdgeos: maguro isn't officially deprecated though, but beh…
[09:29] <tsdgeos> didrocks: i know, i'm not saying that it should not be investigated, i just don't want to investigate it now
[09:29] <tsdgeos> after we update to qt 5.2
[09:29] <tsdgeos> i'm happy to do
[09:29] <didrocks> tsdgeos: that would mean we block promotion until we get 5.2?
[09:30] <tsdgeos> didrocks: that's for you guys to decide :D
[09:31] <tsdgeos> didrocks: i do not think that this bugs are new
[09:31] <tsdgeos> s/bugs/test failures
[09:31] <tsdgeos> but the data says otherwise
[09:31] <didrocks> they didn't show as failures
[09:31] <didrocks> but ok, let's see
[09:31] <tsdgeos> so i can be just speaking crap
[09:32] <tsdgeos> mhr3: this https://code.launchpad.net/~aacid/unity8/misc_previewlist_fixes/+merge/206365 should fix the previewlist opening/closing problems you were having yesterday, can you check
[09:32] <asac> so the problem is that you guys decay our stuff
[09:32] <asac> shoudl we try removing MIR again?
[09:32] <asac> etc.?
[09:32] <asac> should we ignore?
[09:33] <asac> tsdgeos: you already said the same last week
[09:33] <asac> e.g. its not new
[09:33] <asac> but there is no indication of that being right
[09:33] <tsdgeos> asac: i know i'm repeating myself
[09:33] <asac> we made an exception last week to give you more time to investigate
[09:34] <asac> now you say you cant make it now
[09:34] <asac> there needs to be a STOP at some point
[09:34] <asac> or REVERT
[09:34] <tsdgeos> asac: sure
[09:34] <asac> tsdgeos: please check these failures out... if you can proof them to be SoC/maguro specific
[09:34] <asac> then we can ignore
[09:34] <tsdgeos> asac: i can't do that and MWC at the same time
[09:34] <asac> but htey are most likely genuine
[09:34] <asac> races etc.
[09:35] <asac> tsdgeos: i am not sure what you do for MWC
[09:35] <tsdgeos> asac: let me repeat again "qt 5.0 qml is racy as hell and i don't want to debug it because it has been fixed already"
[09:35] <asac> tsdgeos: we mght not get 5.2
[09:35] <asac> you dont know that yet
[09:35] <tsdgeos> so i am not going to spend my time on that unless my manager tells me
[09:35] <asac> (most likely we will get it)
[09:36] <tsdgeos> speak to my manager
[09:36] <asac> kgunn: ^
[09:36] <asac> :)
[09:36] <tsdgeos> thanks
[09:36] <seb128> asac, not getting 5.2 seems like that would be a mistake
[09:36] <asac> seb128: sure it would be
[09:36] <asac> just sayin gthat whatever isnt reality might never happen (even if unlikely and brutal)
[09:36] <tsdgeos> asac: or speak to saviq when he's back on monday, i'm just a regular guy that shouldn't and don't want to take this decisions
[09:37] <tsdgeos> greyback: morning, seen the mail by kevin about the sdk failure on demo-stuff ppa?
[09:37] <asac> tsdgeos: my understanding was that you are saviq's placeholder
[09:37] <asac> while he is gone
[09:37] <tsdgeos> asac: i am saviq's placeholder
[09:37] <greyback> tsdgeos: yeah just now
[09:37] <asac> tsdgeos: so you can make the decisions as he could
[09:38] <tsdgeos> asac: that's a bit of a overstatement :D I don't have all the knowledge about all the other components he has, i've just been raised temporarily because we needed someone
[09:38] <asac> tsdgeos: so what do you propose? as a leader you must have a clear suggestion
[09:39] <tsdgeos> asac: i do have a clear suggestion, ignore test failures that happen in maguro but not on mako until 5.2 lands
[09:39] <seb128> asac, imho it doesn't make sense spend time to workaround toolkit issues in our code, especially if getting a workaround is not trivial, when we know those issues are bugs fixed in the new version
[09:39] <tsdgeos> asac: i think i've been pretty clear in that
[09:39] <asac> noone has proven that
[09:39] <asac> its qt
[09:39] <asac> its a guess
[09:39] <asac> and hope
[09:39] <tsdgeos> asac: noone has proven that it is not
[09:39] <asac> right,k hence investigation is needed
[09:40] <asac> we had many many flaky things fixed
[09:40] <seb128> or moving on with qt 5.2 is needed
[09:40] <asac> on maguro
[09:40] <tsdgeos> asac: and we have proven that there are billions of problems in qt
[09:40] <asac> never was the toolkit cited as the reason
[09:40] <seb128> it's creating an increasing number of resources wastes
[09:40] <asac> try how the APs succeed with qt 5.2 then
[09:40] <asac> do they?
[09:40] <asac> that could easily make us rest
[09:40] <tsdgeos> asac: i don't know, do they?
[09:59] <tsdgeos> Cimi: you wokring on the video stuff?
[10:03] <tsdgeos> Cimi: the RatingStars.qml we weren't using it yet?
[10:03] <Cimi> tsdgeos, we were using it
[10:03] <Cimi> tsdgeos, thus I added the compatibility layer
[10:03] <tsdgeos> ah righ
[10:03] <Cimi> tsdgeos, mainly old previews
[10:04] <tsdgeos> just saw it now
[10:05] <tsdgeos> Cimi: should then tst_RatingStars.qml still be there and not rename/Rework it to proof the old way still works?
[10:05] <tsdgeos> and add a tst_Rating.qml with the new code
[10:06] <Cimi> tsdgeos, the tst_Rating is tested
[10:06] <tsdgeos> Cimi: yes
[10:06] <tsdgeos> Cimi: but you deleted the tst_RatingStars.qml
[10:07] <tsdgeos> can we leave it there while we don't delete the compatibility stuff with the old scopes?
[10:08] <Cimi> tsdgeos, how do I revert the previous file and add the new one?
[10:09] <tsdgeos> good question :D
[10:09] <Cimi> tsdgeos, I'll simply add the file
[10:09] <tsdgeos> ok
[10:09] <Cimi> and who cares
[10:10] <Cimi> tsdgeos, at this point I can remove the compatibility layer and have double widget
[10:11] <tsdgeos> ?
[10:11] <Cimi> tsdgeos, copy back RatingStars
[10:11] <tsdgeos> ah
[10:11] <tsdgeos> sure
[10:11] <tsdgeos> that is even better :D
[10:11] <tsdgeos> be sure not to break old-scopes
[10:11] <Cimi> tsdgeos, actually
[10:11] <Cimi> tsdgeos, my new rating stars provides new assets
[10:11] <Cimi> boring
[10:11] <Cimi> :-\
[10:15] <Cimi> tsdgeos, readied old stuff
[10:15] <Cimi> *readded
[10:17] <tsdgeos> Cimi: the test too?
[10:17] <Cimi> nope
[10:18] <Cimi> tsdgeos, forgot to run bzr add
[10:18] <Cimi> tsdgeos, done
[10:26] <greyback> anyone familar with LP builds? Getting weirdest error with: https://launchpad.net/~unity-team/+archive/demo-stuff/+recipebuild/652566
[10:27] <tsdgeos> nope :/
[10:27] <greyback> ah, seems to be a more general fail. #ubuntu-devel discussing it now
[10:29] <greyback> @unity: don't update your machine right now, there seems to be a broken libgcc1 package in the repo atm
[10:29] <tsdgeos> \o/
[10:29] <tsdgeos> not
[10:29] <tsdgeos> let's revert everything!
[11:09] <tsdgeos> Cimi: the ratings widget has the same problem vidoe had with the height
[11:10] <Cimi> tsdgeos, ah sorry dude
[11:10] <Cimi> tsdgeos, will fix
[11:11] <tsdgeos> Cimi: no worries, that's why we have people reviewing :)
[11:11] <Cimi> tsdgeos, pushed
[11:12] <tsdgeos> Cimi: it ain't working
[11:13] <tsdgeos> Cimi: apply the patch i said and see the difference between clip enabled and not enabled
[11:13] <tsdgeos> unless it's on purpose that the line edit and button are not part of the height
[11:13] <tsdgeos> which i think is a bit weird :D
[11:14] <Cimi> what?
[11:14] <Cimi> why is not working with implicitHeight?
[11:20] <Cimi> tsdgeos,
[11:20] <tsdgeos> Cimi: it's not that implicitHeight is not working, i'd say that your childrenRect is unhappy
[11:21] <tsdgeos> let me have a look
[11:21] <Cimi> tsdgeos, how do I test it?
[11:21] <tsdgeos> Cimi: see the comment on the review
[11:21] <tsdgeos> Cimi: i'd say your reviewContainer is missing a implicitheight too
[11:22] <Cimi> I'm looking
[11:22] <Cimi> tsdgeos, works for you?
[11:22] <tsdgeos> nope :D
[11:29] <tsdgeos> Cimi: http://paste.ubuntu.com/6930678/ ?
[11:30] <Cimi> tsdgeos, I ant it to stretch
[11:30] <Cimi> *want
[11:30] <Cimi> mmm
[11:31] <tsdgeos> Cimi: but it is not stretching :D
[11:31] <tsdgeos> Cimi: see your own tryPreviewRating without any change
[11:31] <tsdgeos> you have it set to anchors.fill: parent
[11:31] <Cimi> tsdgeos, I understand why its not stretching
[11:32] <tsdgeos> and it's just not height enough
[11:32] <Cimi> but I'd like it to stretch
[11:32] <tsdgeos> besides this is the implicitheight
[11:32] <Cimi> tsdgeos, in case you want two line of comment
[11:32] <tsdgeos> doesn't mean it can't adapt to more height
[11:32] <tsdgeos> it just gives a default one
[11:32] <tsdgeos> so if it should not prevent you from stretching
[11:34] <mzanetti> do we need some unreleased uitk version for the new-scopes stuff?
[11:34] <mzanetti> getting this: nativeOrientationAngle: 0 // stop contents being re-rotated when shell in landscape
[11:35] <mzanetti> but qtdeclarative5-ubuntu-ui-toolkit-plugin seems to be up to date
[11:35] <mzanetti> err. this is the error: Cannot assign to non-existent property "nativeOrientationAngle"
[11:36] <mzanetti> greyback: you might know this? ^
[11:36] <greyback> mzanetti: yeah, I'm trying to get it built into the PPA
[11:38] <mzanetti> greyback: and it seems to be rotated on the desktop now
[11:38] <greyback> mzanetti: oh, that's not supposed to happen
[11:39] <tsdgeos> mzanetti: greyback: maybe because of the missing uitk stuff?
[11:40] <Cimi> tsdgeos, but if I disable the anchors, yes
[11:40] <tsdgeos> Cimi: diff?
[11:40] <Cimi> tsdgeos, your
[11:41] <greyback> tsdgeos: mzanetti: possibly, checking now
[11:43] <Cimi> tsdgeos, even mine doesn't stretch though
[11:43] <tsdgeos> Cimi: that's what i was saying :D
[11:43] <Cimi> hah
[11:45] <mzanetti> ok... going for lunch then and will try again. greyback, please let me know when the uitk package is ready
[11:45] <greyback> mzanetti: will do. LP giving me grief
[11:45] <mzanetti> *try again later
[11:46] <tsdgeos> Cimi: http://paste.ubuntu.com/6930736/
[11:46] <tsdgeos> Cimi: this works here with both
[11:46] <tsdgeos> anchors.fill: parent and no width defined
[11:46] <tsdgeos> and with width: parent.width no anchors defined and no height defined
[11:50] <Cimi> tsdgeos, nope
[11:50] <Cimi> tsdgeos, maybe sdk bug
[11:50] <tsdgeos> nope what?
[11:51] <Cimi> tsdgeos, try typing in the entry when it's filled
[11:51] <Cimi> stretched
[11:51] <tsdgeos> lol
[11:52]  * greyback bbiab
[11:52] <tsdgeos> Cimi: so maybe just have my first patch and consider stretching for later?
[11:52] <Cimi> tsdgeos, I want to fix the stretching if possible
[11:52] <Cimi> tsdgeos, let me see
[11:52] <tsdgeos> ok, work on that
[11:53] <Cimi> tsdgeos, review the tests
[11:53] <Cimi> tsdgeos, not sure they are done in the correct way
[11:53] <Cimi> tsdgeos, like, logically correct to put logic inside tests
[11:53] <tsdgeos> ok
[11:53] <Cimi> tsdgeos, for me is fine, but I'd like an opinion
[11:53] <tsdgeos> will do
[11:54] <Cimi> mhr3, btw, I received nexus 4 yesterday
[11:54] <Cimi> mhr3, guess which was the first thing I did?
[11:55] <mhr3> Cimi, i don't want to know! :P poor n4, it's naked now!
[11:55] <Cimi> ehehe
[11:56] <Cimi> mhr3, thinking of getting a prepaid
[11:56] <Cimi> mhr3, and try ubuntu on it outside dev
[11:56] <Cimi> mhr3, is there a way to disable 3d data?
[11:56] <Cimi> *3g
[11:56] <mhr3> Cimi, you think it would work out-of-the-box? :)
[11:56] <tsdgeos> mhr3: when you have some time i'd appreciate a quick test to see if https://code.launchpad.net/~aacid/unity8/misc_previewlist_fixes/+merge/206365 fixes your preview opening/closing problems
[11:57] <mhr3> Cimi, i mean it might... but well :)
[11:57] <Cimi> mhr3, still buggered?
[11:57] <mhr3> tsdgeos, am just about to try it
[11:57] <mhr3> tsdgeos, i need https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-replacing-preview/+merge/206354 :)
[11:59] <tsdgeos> mhr3: you want me to review it or you want me to make unity8 work with it?
[11:59] <mhr3> tsdgeos, review
[11:59] <mhr3> tsdgeos, it doesn't change much of the shell-facing api
[12:00] <tsdgeos> mhr3: but does it work? :D
[12:00] <mhr3> tsdgeos, ie nothing breaks
[12:00] <mhr3> yes :)
[12:00] <tsdgeos> amazing
[12:00] <mhr3> tsdgeos, your branch does breaks the header though
[12:00] <mhr3> after closing preview it's not there
[12:00] <tsdgeos> sure, it's because of the scrolling of the view
[12:01] <tsdgeos> you can scroll it back
[12:01] <tsdgeos> but you're right it doesn't make sense
[12:01] <tsdgeos> since there's enough space
[12:01] <tsdgeos> it should be visible
[12:02] <mhr3> yep
[12:03] <mhr3> tsdgeos, is there a way to make the animation super-slow?
[12:03] <tsdgeos> mhr3: the opening one?
[12:03] <mhr3> yea
[12:03] <mhr3> when opening preview for a scope it's super weird
[12:04] <mhr3> i want to see what's going on
[12:04] <tsdgeos> i'd say openeffect
[12:04] <tsdgeos> but let me check
[12:04] <tsdgeos> i don't see anything weird though with this branch :S
[12:04] <mhr3> tsdgeos, i meant specifically scopes scope is odd (probably because of no-art)
[12:04] <tsdgeos> sure i've been using scopes scope
[12:09] <tsdgeos> mhr3: yes qml/Dash/DashContentOpenEffect.qml
[12:09] <tsdgeos> -            duration: 200
[12:09] <tsdgeos> +            duration: 2000
[12:10] <mhr3> yea, the first frame make everything jump
[12:10] <mhr3> makes
[12:10] <tsdgeos> yep
[12:10] <tsdgeos> going to see why that happens
[12:10] <tsdgeos> that'll fix the closing too most probably
[12:10] <mhr3> it works fine for the first row of results
[12:10] <mhr3> interesting
[12:11] <tsdgeos> yeah because there's some calculation in there going wrong
[12:11] <tsdgeos> positionListView in PreviewListView.qml is a bit of too much calculation :D
[12:20] <tsdgeos> mhr3: pushed
[12:20] <mhr3> same branch?
[12:22] <gatox> mhr3, hi, do you know if it is possible to return a preview with the new scopes api when an action is triggered in a preview and the perform_action method is called?? (for example, in the click scope move from the Install preview to Progress preview)
[12:23] <mhr3> gatox, it is, but scope-tool doesn't support it yet
[12:23] <gatox> mhr3, we have been trying to figure it out how to do that yesterday, but no luck
[12:23] <gatox> mhr3, ahhhhh ok
[12:23] <mhr3> pstolowski, can you explain pls ^ trying to prepare all the mps now
[12:24] <pstolowski> gatox, you need to respond with ActivationResponse with status=ShowPreview
[12:24] <pstolowski> gatox, and then you get a call to preview()
[12:24] <gatox> pstolowski, but that is not working yet, right?
[12:25] <pstolowski> gatox, so it's an extra round-trip (unlike the old API, where you could pass serialized preview right away)
[12:25] <pstolowski> gatox, not supported by scope-tool, right
[12:25] <gatox> pstolowski, ack, thanks
[12:25] <tsdgeos> mhr3: yes
[12:27] <Cimi> tsdgeos, maybe because textfield is supposed to be a single line
[12:28] <tsdgeos> Cimi: maybe, there's a text area?
[12:28] <Cimi> tsdgeos, it's another option
[12:28] <Cimi> tsdgeos, not sure how it is themed
[12:30] <tsdgeos> mhr3: i don't think that i can honestly review https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-replacing-preview/+merge/206354
[12:30] <tsdgeos> mhr3: i can say you're missing a const :D
[12:30] <tsdgeos> in some functions
[12:31] <tsdgeos> but i have almost no knowledge of how that thing works
[12:31] <tsdgeos> i don't see any obvious coding mistake
[12:31] <mhr3> tsdgeos, just run it :)
[12:31] <tsdgeos> but given that tests pass, that was to be expected
[12:32] <tsdgeos> mhr3: and do anything special? i don't know how to get an install/progress widget, none of the scopes i have seem to do it
[12:32] <tsdgeos> mhr3: or just do a test run of stuff?
[12:33] <mhr3> tsdgeos, yea, replacing previews working is guaranteed by the test
[12:33] <mhr3> famous last words :)
[12:34] <mhr3> tsdgeos, or do what saviq does, get coverage report and see if the change lines are covered :)
[12:35] <kgunn> tsdgeos: mhr3 mzanetti ...i am so sorry to leave broken ppa's for your morning, did we recover ?
[12:36] <mhr3> kgunn, no, lp is borked :/
[12:36] <tsdgeos> kgunn: greyback was working on it
[12:36] <tsdgeos> but seems we have a broken gcc or soemthing that makes it hard to recover
[12:36] <greyback> kgunn: LP having problems, and a bad libgcc1 broke everything for a while, but things seem to be moving slowly again
[12:36] <kgunn> thanks...felt bad going to bed leaving it that way
[12:37] <greyback> kgunn: sleep more important
[12:40] <mhr3> gatox, we have a branch with which scope-tool can do it, but lp is borked, can't rebuild the demo ppa
[12:41] <gatox> mhr3, ack..... any estimate for that??
[12:41] <mhr3> gatox, scratch that, lp works, it's building
[12:42] <gatox> mhr3, great
[12:42] <mhr3> soon in demo ppa near you
[12:43] <tsdgeos> mhr3: approved, want me to top approve too?
[12:43] <mhr3> tsdgeos, if it works for you, sure
[12:43] <tsdgeos> mhr3: did you have time to look at the second fix for the previewlist preview open/close thing?
[12:43] <mhr3> not yet
[12:55] <mhr3> tsdgeos, yea, now it seems to actually work :)
[12:55] <kgunn> mhr3 demo-stuff works ?
[12:56] <mhr3> kgunn, no, tsdgeos's branch
[12:56] <mhr3> kgunn, what's broken there anyway?
[12:56] <kgunn> mhr3: :) thot i was crazy...
[12:56] <kgunn> ubuntu-ui-toolkit
[12:56] <kgunn> i just cleaned and rekicked...let's see
[12:56] <mhr3> oh :/
[13:05] <mhr3> gatox, built for desktop > https://code.launchpad.net/~unity-team/+archive/demo-stuff/+sourcepub/3913988/+listing-archive-extra
[13:09] <gatox> mhr3, thanks
[13:21] <kgunn> alrigth ubutnu-ui-toolkit seems to have built!...soon in the demo-stuff
[13:21] <kgunn> greyback_: ^
[13:21] <greyback_> \o/
[13:22] <mzanetti> kgunn: nope, not recovered yet. the merge conflicts between new scopes and right edge are quite bad... still trying to fix them
[13:47] <kgunn> mzanetti: thanks...(and actually ui-toolkit is only part of the demo-stuff...for n7 rotation)
[13:47] <kgunn> starting to need a crib sheet :)
[13:47] <mzanetti> kgunn: well, the new scopes stuff depends on it
[13:47] <mzanetti> kgunn: which means I need to get it merged with my branches too
[13:48] <kgunn> mzanetti: oh crap...so we have to pull it in
[13:48] <kgunn> it==ui-toolkit
[13:48] <mzanetti> kgunn: right now I tied to run the new scopes stuff on my desktop and it doesn't even start without the unreleased uitk stuff
[13:48] <kgunn> really!?!
[13:48] <mzanetti> kgunn: commenting away the bits that make it fail, makes the shell start again, but rotated on the desktop
[13:49] <mzanetti> which in turn tries to load the unfinished sidestage stuff from my branch
[13:49] <kgunn> mzanetti: well...true we didn't test the effect
[13:49] <kgunn> i blame greyback_
[13:49] <mzanetti> :D
[13:49] <mzanetti> I do so too :P
[13:49] <kgunn> he's an easy target
[13:49] <mzanetti> nah... we'll get it fixed
[13:50] <mzanetti> but yeah, need to the get the uitk stuff into the ppa I believe
[13:50] <mzanetti> greyback_: right? ^
[13:50] <greyback_> what did I do now?
[13:51] <mzanetti> greyback_: just kidding. you didn't do anyhting
[13:51] <greyback_> mzanetti: http://pastebin.ubuntu.com/6931260/ will at least un-rotate on desktop
[13:56] <mzanetti> kgunn: what's the plan actually? to have the new scopes + right edge on all phones?
[13:56] <mzanetti> kgunn: or will there be different phone images?
[13:56] <kgunn> mzanetti: yes...that's the plan
[13:56] <kgunn> mzanetti: check my thinking but demo-stuff will be for n10/n7
[13:56] <kgunn> and phone-right-edge will be for n4
[13:57] <mzanetti> ok... so the right edge branch needs still some fixing to work with the new scopes...
[13:57]  * mzanetti is on it
[13:57] <kgunn> mzanetti: thanks sir
[13:57] <kgunn> no pressure but mark will be reviewing early next week
[13:58] <mzanetti> :D
[13:58] <mzanetti> kgunn: what's the ETA on new scopes landing into trunk?
[14:00] <tsdgeos> pete-woods: ping
[14:00] <pete-woods> tsdgeos: hi
[14:00] <tsdgeos> pete-woods: hud-service is at 100%
[14:00] <tsdgeos> pete-woods: what do i do?
[14:00] <mzanetti> kgunn: anyways, this branch rebases right-edge-2 on new-scopes: lp:~mzanetti/unity8/right-edge-based-on-new-scopes
[14:01] <mzanetti> kgunn: should make it start up again with that, still some broken animations tho
[14:01] <pete-woods> tsdgeos: can you attach gdb to it?
[14:01] <tsdgeos> i guess
[14:01] <tsdgeos> let me
[14:02] <pete-woods> is it gdb -p PID ?
[14:02] <kgunn> mzanetti: ta...
[14:02] <gatox> mhr3, do i need any particular unity branch to use previes activation to show a new preview? or upgrading the system is enough (on desktop)
[14:02] <kgunn> i was gonna test dandrader's copy/paste from the landing silo....then i'll turn my eyes to that
[14:03] <tsdgeos> pete-woods: http://paste.ubuntu.com/6931318/ http://paste.ubuntu.com/6931320/
[14:03] <tsdgeos> pete-woods: let me see if i can get debug symbols for libdbusmenu-qt5.so.2 and for hud-service
[14:03] <mhr3> gatox, new-scopes is enough
[14:03] <pete-woods> tsdgeos: thanks
[14:06] <tsdgeos> pete-woods: http://paste.ubuntu.com/6931329/ http://paste.ubuntu.com/6931331/ http://paste.ubuntu.com/6931338/
[14:06] <tsdgeos> pete-woods: the DBusMenuImporter::slotMenuAboutToShow and waitForWatcher seem to be the offenders
[14:06] <tsdgeos> sil2100: ↑↑↑ is that you?
[14:08] <tsdgeos> oh
[14:08] <tsdgeos> now that i see
[14:08] <tsdgeos> i also have a few
[14:09] <tsdgeos> QVariantMap DBusMenuExporterDBus::getProperties(int, const QStringList&) const: Condition failed: action
[14:09] <tsdgeos> bool DBusMenuExporterDBus::AboutToShow(int): Condition failed: menu
[14:09] <tsdgeos> in my console
[14:09] <tsdgeos> no clue where they come from
[14:09] <tsdgeos> but seem related
[14:09] <pete-woods> tsdgeos: thanks for this debugging info - very helpful!
[14:10] <tsdgeos> pete-woods: can i kill it now?
[14:10] <pete-woods> sure
[14:10] <pete-woods> tsdgeos: if they are in your console, it makes me think it's an app that you started from there that is the culprit
[14:10] <pete-woods> obviously HUD should be resilient to it
[14:11] <pete-woods> but it would still be very handy if we could find the exact app
[14:11] <tsdgeos> fg says it may be kate
[14:12] <tsdgeos> which is the only app i ran from there and is still running now
[14:12] <Debolaz> I suppose you get this question a lot, but when is Unity 8 planned to be ready for desktop use?
[14:13] <tsdgeos> Cimi: you there?
[14:15] <tsdgeos> mhr3: ping
[14:23] <kgunn> Debolaz: it'll be some time before unity8 is full fledged on desktop..however there are efforts under way to create a desktop preview this cycle...so look for the 14.04 rel
[14:26] <sil2100> tsdgeos: wazzup?
[14:27] <tsdgeos> sil2100: hud-service spinin 100% cpu on libdbusmenu-qt5, read a bit up
[14:27] <tsdgeos> sil2100: afair where you involved in that? or not anymore?
[14:28] <sil2100> tsdgeos: yes, but hm, this code didn't change since like a year
[14:28] <tsdgeos> ok
[14:35] <Cimi> tsdgeos, sorry, irc clinet crashed and didn't realise
[14:42] <mzanetti> tsdgeos: hey, I think there are 2 regressions in the new-scopes stuff in regard to the shell background.
[14:42] <mzanetti> tsdgeos: 1st, there isn't a background any more at all which makes running apps shine through the dash
[14:43] <tsdgeos> mzanetti: what do you mean there isn't a background?
[14:43] <tsdgeos> mzanetti: there's the white thing
[14:43] <Cimi> tsdgeos, also
[14:43] <mzanetti> tsdgeos: but its not opaque
[14:43] <Cimi> tsdgeos, with textarea
[14:43] <Cimi> tsdgeos, shall it make 3 lines tall?
[14:43] <Cimi> I make it
[14:44] <mzanetti> tsdgeos: 2nd, the dummy image for checking the background settings is gone which will break the greeter wallpaper in certain circumstances
[14:44] <tsdgeos> Cimi: works for me
[14:44] <tsdgeos> mzanetti: ok, i'll have a look unless you beat me to it :D
[14:45] <mzanetti> tsdgeos: this is what I mean: http://i.imgur.com/00KTKsq.png with 2st
[14:45] <mzanetti> 1st...
[14:45] <Cimi> tsdgeos, lineHeight * 3 is bad?
[14:45] <Cimi> or paintedHeight
[14:45] <Cimi> whatever
[14:45] <Cimi> not sure what works
[14:45] <tsdgeos> Cimi: try if it works first and then we can discuss how bad it looks :D
[14:49] <Cimi> tsdgeos, nevermind
[14:49] <Cimi> tsdgeos, textArea doesn't have lineHeight
[14:49] <Cimi> tsdgeos, so it's not possible
[14:49] <Cimi> tsdgeos, tried contentHeight / lineCount but it can't be done or it loops
[14:50] <kgunn> dandrader: ok, i got your qml file...but how do i run on the phone? (e.g. particularly directory? command line incantation?)
[14:52] <Cimi> tsdgeos, ok, pushed
[14:53] <Cimi> @unity: for the nexus 4 wifi, it doesn't work yet if you flash ubuntu from android 4.4. All you have to do it is either flash ubuntu when you're on android 4.3, or flash the radio from android 4.3 https://www.copy.com/s/JniBu/4.3%20files%20%28mako%29/radio-mako-m9615a-cefwmazm-2.0.1700.84.zip
[14:57] <tsdgeos> Cimi: cool, so what i said (or was trying to say at least :D)
[15:00] <dandrader> kgunn, qmlscene [SOME_QML_FILE].qml --desktop_file_hint=[SOME_EXISTING_DESKTOP_FILE_TO_FOOL_UNITY8]
[15:00] <tsdgeos> greyback_: how's that fixed uitk coming?
[15:01] <greyback_> tsdgeos: should be in
[15:01] <tsdgeos> greyback_: where?
[15:01] <tsdgeos> in demo-stuff?
[15:01] <dandrader> kgunn, thus: qmlscene copy-paste.qml --desktop_file_hint=/usr/share/click/preinstalled/com.ubuntu.calendar/current/calendar-app.desktop
[15:01] <greyback_> tsdgeos: yeah
[15:01] <tsdgeos> not getting it
[15:01] <tsdgeos> greyback_: kgunn: you need to rebuild, trusty has a newer version, so it wins
[15:02] <greyback_> tsdgeos: seems archive has newer version entirely, do apt-cache showpkg to see what version and maybe switch
[15:02] <tsdgeos> greyback_: that may help me, but not anyone else installing the ppa, we need to get that working
[15:02] <greyback_> tsdgeos: yep
[15:03] <kgunn> dandrader: uh...doesn't seem to work for me
[15:03] <kgunn> at least...doesn't do what i expected
[15:04] <greyback_> kgunn: what you think? Can we replace the date string in the package name with "demo-stuff" to avoid this?
[15:04] <kgunn> greyback_: yes...its only for demo
[15:04] <kgunn> should be fine
[15:04] <dandrader> kgunn, what do you get?
[15:05] <dandrader> kgunn, it's a pretty ugly test app, I warn you :)
[15:05] <kgunn> dandrader: so i hit "copy all" words highlight...but then no matter what "paste" doesn't result in anything
[15:05] <dandrader> kgunn, hmmm, then it's not working. are you sure you have both MPs there?
[15:06] <dandrader> kgunn, the one for unity-mir and the other for platform-api?
[15:06] <kgunn> in fact dandrader if i hit "paste" while the text is highlighted...it just erases all highlighted text (like a delete)
[15:06] <kgunn> yep
[15:06] <dandrader> kgunn, a better way is to copy. close the app. run it again and then paste
[15:06] <Cimi> tsdgeos, replied
[15:06] <kgunn> dandrader: well....let me do a full dist upgrade just in case
[15:06] <kgunn> dandrader: i'll try your trick first
[15:07] <dandrader> kgunn, but from what you told me, I can already tell you the clipboard is not working
[15:07] <kgunn> ...mmm, your trick didn't work
[15:07] <kgunn> let me dist-upgrade
[15:08] <dandrader> kgunn, so your image must be missing one of my patches...
[15:08] <kgunn> dandrader: i installed one by one...so just in case
[15:08] <kgunn> btw, i'm using the ppa from here
[15:08] <tsdgeos> Cimi: tx
[15:08] <kgunn> https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=sharing#gid=24
[15:09] <kgunn> in the landing-006 sheet
[15:11] <tsdgeos> Cimi: did you fix the height thing too?
[15:11] <Cimi> yep
[15:13]  * tsdgeos reviews again
[15:16] <tsdgeos> Cimi: still not working
[15:16] <Cimi> ouch
[15:16] <Cimi> tsdgeos, I used your patch
[15:16] <tsdgeos> Cimi: set clip to true and see how the text field is cut instead of shrinked
[15:17] <Cimi> ok
[15:17] <tsdgeos> Cimi: my patch worked when it was not a textarea i guess :D
[15:17] <Cimi> tsdgeos, I'll quickly go out for errads
[15:17] <tsdgeos> or maybe didn't work and i didn't realize
[15:17] <Cimi> tsdgeos, there is a break with the rain
[15:17] <tsdgeos> run!
[15:17] <Cimi> I'll do this soom
[15:17] <Cimi> ahah
[15:17] <Cimi> yeah
[15:17] <Cimi> pretty shitty today
[15:23] <kgunn> Cimi: i'm gonna break your heart after weeks of 0 C, we've got sunshine & 23 C
[15:24] <mzanetti> kgunn: Cimi: complaining about weather? this is happening in the area I come from: http://www.youtube.com/watch?v=f5waSw2mMfY
[15:25] <mzanetti> kgunn: right-edge + new-scopes working on my phone now
[15:25] <mzanetti> kgunn: should I upload a new package to the ppa or will that conflict with what you're doing?
[15:25] <kgunn> mzanetti: @weather...deadly but beautiful
[15:26] <tsdgeos> mzanetti: i don't know how to solve that background problem without having the image two times :/
[15:26] <kgunn> mzanetti: if you want to push to
[15:26] <tsdgeos> mzanetti: or maybe i can add the image also to the scope tool
[15:26] <tsdgeos> that'd work too
[15:26] <mzanetti> tsdgeos: which image do you mean? the background (white) one?
[15:26] <tsdgeos> mzanetti: yes
[15:27] <kgunn> mzanetti: if you could merge to lp:~unity-team/unity8/merge-new-scopes-right-edge-2
[15:27] <tsdgeos> mzanetti: the problem is not that the white thing is transparent, the problem is that we make the dash transparent while "bringing it up", which in turns makes the image transparent because it is inside the dash
[15:27] <kgunn> then we can trigger another build
[15:27] <mzanetti> kgunn: ack
[15:27] <tsdgeos> mzanetti: so i'd need to move it out of the dashh, but then that'd make the scope-tool not have a background, then i guess i can add the backgrond to the scope tool too
[15:27] <kgunn> its not interfering with me...i'm on copy/paste landing effort
[15:28] <kgunn> mzanetti: just wanted to note...i created a seperate branch to push to (for merging right-edge-2 & new-scopes)
[15:28] <kgunn> i should've told you in the mail...but i was getting goofy last night and forgot
[15:29] <mzanetti> kgunn: ah ok. no prob. will push to there
[15:29] <kgunn> mzanetti: did that in order to keep automerging/building with new-scopes updates
[15:29] <kgunn> or somethign like that
[15:29] <kgunn> ...i can't rememebr...but i had a good reason last night :D
[15:29] <mzanetti> sure, makes sense... I just didn't know and pushed to yet another branch. but no probl, really
[15:30] <kgunn> i think it was conflicts maybe...
[15:30] <mzanetti> tsdgeos: also, this one breaks the greeter if the settings have a bad image path; http://paste.ubuntu.com/6931738
[15:31] <tsdgeos> mzanetti: propose a merge :)
[15:32] <mzanetti> tsdgeos: so the background image in the shell is gone for good, right?
[15:32] <tsdgeos> mzanetti: the background image for the shell is the white paper
[15:32] <kgunn> dandrader|afk: bad news my friend...double checked, using ppa's from silo build i referenced earlier, it still doesn't paste
[15:33] <kgunn> dandrader: can you try using ppa:ci-train-ppa-service/landing-006 on top of latest image
[15:33] <kgunn> (or image greater than 181)
[15:33] <kgunn> 181 has mir0.1.5
[15:34] <dandrader> kgunn, hmmm, ok, will try that out
[15:34] <kgunn> dandrader: oh shit...
[15:34] <kgunn> nvmd
[15:34] <kgunn> i just checked...the ppa is empty!
[15:34] <dandrader> kgunn, what?
[15:34] <kgunn> wtf
[15:34] <dandrader> hahahaha
[15:35] <dandrader> what a relief :)
[15:35] <kgunn> hmmmm....
[15:44] <tsdgeos> mzanetti: https://code.launchpad.net/~aacid/unity8/new-scopes-dash-background-opacity/+merge/206516
[15:46] <tsdgeos> kgunn: greyback_: dandrader: i forced the "good" uitk and now i can't unlock the greeter anymore
[15:46] <tsdgeos> kgunn: greyback_: dandrader: do you mind if we revert the change the new-scopes branch until it works?
[15:48] <dandrader> tsdgeos, don't know what you're talking about
[15:48] <greyback_> tsdgeos: what device?
[15:48] <tsdgeos> greyback_: my computer
[15:48] <greyback_> tsdgeos: and you've the demo-stuff ppa installed on it?
[15:48] <tsdgeos> dandrader: i i move the greeter left and it keeps rebounding
[15:48] <tsdgeos> greyback_: yes
[15:49] <dandrader> tsdgeos, and that's with what unity8?
[15:49] <greyback_> tsdgeos: ok, I just want to reproduce before reverting
[15:49] <tsdgeos> dandrader: the one that is in demo-stuff
[15:49] <tsdgeos> new-scopes branch
[15:50] <tsdgeos> ah wait
[15:50] <tsdgeos> maybe i forgot to recompiel?
[15:50] <tsdgeos> ah yes
[15:50] <dandrader> tsdgeos, well, uitk is unrelated to borked DragHandle/DDA (greeter unlock)
[15:50] <tsdgeos> if i recompile i get it in vertical :D
[15:51] <tsdgeos> so yes, can we please get new-scopes to the working place it was before?
[15:51] <greyback_> tsdgeos: yep, my bad, need to do a new build
[15:51] <tsdgeos> people is going to give me shit because of it
[15:51] <tsdgeos> and honestly i've had enough :D
[15:52] <greyback_> tsdgeos: what branch of unity8?
[15:52] <tsdgeos> greyback_: new-scopes
[15:52] <dandrader> tsdgeos, I'm innocent! never touched that new-scopes branch
[15:52] <greyback_> tsdgeos: checking...
[15:52] <tsdgeos> dandrader: well, but kgunn merged your stuff there
[15:53] <greyback_> tsdgeos: would you please merge latest lp:~unity-team/unity8/rotate-n7 just to check?
[15:54] <tsdgeos> greyback_: that brings some sanity in :)
[15:54] <mzanetti> tsdgeos: ScopeTool.qml is not used in unity8, right? only if you run the standalone scopetool
[15:54] <tsdgeos> mzanetti: right
[15:54] <tsdgeos> greyback_: want me to push that?
[15:54] <greyback_> tsdgeos: yeah please
[15:55] <mzanetti> tsdgeos: approved
[15:57] <tsdgeos> tx
[15:57] <kgunn> greyback_: do we need to change the unity8 recipe to merge lp:~unity-team/unity8/rotate-n7
[15:57] <kgunn> into the unity8 package builds
[15:58] <kgunn> thot i did that....
[15:58] <greyback_> kgunn: IMO let's use all new-scopes from now on, else we'll end up with more conflicts/confusion
[15:58] <kgunn> greyback_: sure....
[15:59] <kgunn> and good...i didn't do it
[16:23] <kgunn> dandrader: it workie!
[16:23] <kgunn> going to find a browser person to bother
[16:23] <dandrader> \o/
[16:24] <kgunn> hey hey... dandrader it even pastes into notepad
[16:28] <tsdgeos> greyback_: how do we get the demo-stuff uitk version number to be bigger than the released one?
[16:28] <greyback_> tsdgeos: already done, have kicked off a build
[16:28] <tsdgeos> cool!
[16:29] <tsdgeos> mzanetti: are you proposing a merge with that "invalid background thing"?
[16:29] <tsdgeos> karni: there?
[16:29] <dandrader> kgunn, nice. I recall i had problems trying to make that "paste" pop-up to show itself
[16:29] <kgunn> hey dandrader so i was able to copy text from cnet into the messaging app
[16:29] <kgunn> so... voodoo fix ?
[16:30] <tsdgeos> pete-woods: actually, now that i think, it can't be kate since that's not qt5 based, so it has to be something else
[16:30] <dandrader> kgunn, who knows, maybe I had a borked ubuntu-ui-toolkit installation or something....
[16:30] <kgunn> dandrader: great work
[16:31] <tsdgeos> pete-woods: so either unity8 or assistant, those are basically the only qt5 apps i run afaik
[16:31] <kgunn> dandrader: i did find one thing you can't do...paste into the google search box
[16:31] <pete-woods> tsdgeos: it's got to be some qt application (4 or 5) or firefox, they are the only things still using dbusmenu
[16:31] <tsdgeos> pete-woods: but the backtrace said libdbusmenu-qt5, no?
[16:31] <pete-woods> it's probably not firefox, as I'd have noticed that by now
[16:31] <pete-woods> tsdgeos: that's just on the HUD end, not the client
[16:32] <tsdgeos> ahhhhh
[16:32] <tsdgeos> trueth
[16:34] <pete-woods> tsdgeos: I've found libdbusmenu-qt to be "somewhat twitchy" so far, not really looking forward to trying to figure this out
[16:34] <pete-woods> I have the feeling that the library has only really been extensively tested for exporting menus, not importing
[16:37] <mzanetti> tsdgeos: I was looking into it and it seems we've a copy of that error recovery inside the Greeter now and load the background image for the greeter from AccountsService
[16:38] <tsdgeos> mzanetti: may be, Saviq did some stuff in there
[16:38] <mzanetti> tsdgeos: so the question is actually why we still load the background from gsettings and not just set the jpg files to greeter.defaultBackground
[16:38] <tsdgeos> mzanetti: no idea tbh, let's just leave it for monday when Saviq is back :D
[16:40] <tsdgeos> Cimi: https://code.launchpad.net/~aacid/unity8/show_video_playback_screenshot_loading_indicator/+merge/206525
[16:41] <Cimi> tsdgeos, k
[16:41] <Cimi> tsdgeos, I fixed the bug
[16:41] <Cimi> tsdgeos, not sure about the mouse are enabled
[16:41] <tsdgeos> Cimi: well, we don't show the play button, it'd be weird to actually play
[16:42] <Cimi> tsdgeos, we maybe still want to eat events?
[16:42] <Cimi> tsdgeos, ah right
[16:42] <Cimi> tsdgeos, maybe we want to change onclicked?
[16:42] <tsdgeos> Cimi: i can do that if you prefer
[16:42] <Cimi> tsdgeos, what do you think?
[16:43] <tsdgeos> Cimi: i am not sure, i think that ideally what i have should work, but not sure tbh
[16:43] <Cimi> tsdgeos, it works, but events are going through or not?
[16:43] <tsdgeos> yours is probably a bit safer even it is "less declarative"
[16:44] <greyback_> damn, now why didn't you upload.... https://code.launchpad.net/~unity-team/+recipe/ubuntu-ui-toolkit-daily
[16:44] <tsdgeos> Cimi: they are
[16:44] <tsdgeos> Cimi: but they are also going thorugh in the header or text for example
[16:45] <tsdgeos> Cimi: so it should not be a problem
[16:45] <tsdgeos> no?
[16:46] <Cimi> dunno :D
[16:48] <mzanetti> tsdgeos: ack
[16:48] <tsdgeos> Cimi: well you didn't add a big mousearea catching clicks and doing nothing to PreviewTextSummary, no?
[16:49] <tsdgeos> so they go thoguth
[16:49] <tsdgeos> so i don't se why we should make them not go in this case
[16:49] <Cimi> k
[16:57] <mzanetti> kgunn: Just flashed a fresh image, added the ppa, upgraded and rebooted. Flicking through an app spread now!
[16:58] <kgunn> mzanetti: \o/
[16:58] <kgunn> can't thank you guys enough
[16:58] <cwayne> mzanetti, with new-scopes too?
[16:58] <mzanetti> cwayne: yip yip
[16:59] <cwayne> :D
[16:59] <cwayne> mzanetti, what's the ppa?
[16:59] <mzanetti> its secret :P
[16:59] <mzanetti> cwayne: here you go: https://launchpad.net/~unity-team/+archive/phone-right-edge/+packages
[17:00] <cwayne> mzanetti, :P thanks
[17:05] <cwayne> mzanetti, would i need this and the demo-stuff ppa?
[17:06] <mzanetti> cwayne: no, only this
[17:07] <cwayne> mzanetti, tried, got this: E: Unable to locate package unity-api
[17:07] <cwayne> E: Unable to locate package unity-plugin-scope
[17:08] <mzanetti> cwayne: hmm... not sure where that comes from. I tested on a fresh image and it worked fine
[17:08] <cwayne> mzanetti, did you just do a dist-upgrade? i took the package list from the ppa description
[17:09] <Cimi> tsdgeos, was thinking
[17:09] <Cimi> tsdgeos, we can setup a wiki page
[17:09] <Cimi> tsdgeos, like https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8
[17:09] <Cimi> tsdgeos, already with the answers
[17:09] <mzanetti> cwayne: I just did apt-get upgrade
[17:09] <Cimi> tsdgeos, so we can easily copy paste
[17:09] <Cimi> :P
[17:09] <cwayne> mzanetti, ack, ill try that thanks
[17:10] <mzanetti> cwayne: and made sure I have this version of unity8: http://paste.ubuntu.com/6932240
[17:10] <kgunn> cwayne: phone-right-edge is only for phone/n4....and demo-stuff will be for tablet/n10,n7
[17:11] <cwayne> right
[17:12]  * tsdgeos EODs
[17:12] <tsdgeos> tty on monday!
[17:14] <cwayne> mzanetti, hm, seems i still have the old unity-scope-click
[17:15] <cwayne> looks like it may need to be copied over from demo-stuff, not seeing it in phone-right-edge
[17:20] <mzanetti> kgunn: hmm.. looks like cwayne is right... the new scopes stuff in unity8 is loaded, but no backends seem to be loaded.
[17:21] <mzanetti> do we need some more packages in the ppa?
[17:45] <pete-woods> tsdgeos: I've got an experimental libdbusmenu-qt branch you could try here: https://code.launchpad.net/~pete-woods/libdbusmenu-qt/remove-busy-watcher/+merge/206538
[17:46] <pete-woods> ah, EOD
[17:50] <karni> tsdgeos: I guews I'm late to reply to your ping
[17:51] <cwayne> mzanetti, so far it works with just adding the unity-scope-click from the demo-stuff ppa
[17:51] <cwayne> looks nice too :) good job
[17:51] <karni> hrm... I did change my nick to karni-holiday, but only on canonical irc
[18:18] <tsdgeos> karni: so i did suggest a different way to fix the resolution thing, did you see it?
[18:21]  * tsdgeos has to go
[18:33]  * karni looks
[18:37] <kgunn> mhr3: you around ?
[18:43] <karni-holiday> Cimi: Do you plan to look for better fix than this? https://code.launchpad.net/~unity-team/unity8/new-scopes-carousel-bounds-patch/+merge/206307
[18:44] <karni-holiday> Cimi: If not, blocking on an awkward carousel behavior fix on tablet for MWC will not help :(
[18:46] <karni-holiday> Cimi: I'm leaving this with you guys. cwayne, please note that either my fix should go in to avoid that weird behavior on tablet for MWC demoing or Cimi is the only person that can help (he marked my MP as "Needs fixing", and I don't have a better fix).
[19:04] <ssweeny> mhr3, ping, do you know if it's possible to disable previews entirely for a new-style scope? i tried just returning null but that opens a blank preview
[19:05] <ssweeny> *returning null from the preview() function that is
[19:05] <karni-holiday> ssweeny: searched through a number of source files, can't find the binding glue, sorry
[19:07] <cwayne> ssweeny, worst case scenario couldn't we just have some text in the previews?
[19:08] <ssweeny> cwayne, probably, but for some of them it makes no sense to have anything
[19:28] <Cimi> karni-holiday, we have to increase the number of items
[19:28] <Cimi> karni-holiday, this is a regression
[19:28] <Cimi> we have 0 regressions policy
[19:29] <Cimi> we need to switch back to grid for less than 7 items
[19:29] <Cimi> and provide a fix for 7-10
[19:30] <Cimi> kgunn, who is management here? https://code.launchpad.net/~unity-team/unity8/new-scopes-carousel-bounds-patch/+merge/206307
[19:31] <Cimi> solution is
[19:32] <Cimi> first we revert the patch that reverts to grid when there are 7 or less items
[19:32] <Cimi> second we work on improving drag over bounds
[19:34] <karni-holiday> Cimi: please follow up with cwayne.
[19:34] <karni-holiday> you dont revert the patch to item count. you'll break more than the carousel.
[19:35] <karni-holiday> anyway, not here. sorry.
[19:38] <Cimi> karni-holiday, we have to!
[19:38] <Cimi> we don't add regressions
[19:39] <Cimi> the merge should have been blocked at review
[19:39] <karni-holiday> you want to revert to <=6 limit in new-scopes?
[19:40] <Cimi> I want to have grid till 6 items
[19:40] <karni-holiday> ok, plewse raise this with cwayne and achiang asap
[19:40] <Cimi> as it was before
[19:40] <Cimi> if we manage to fix the carousel with 5 items, we can reapply the item count
[19:40] <karni-holiday> That is not possie without consulting with ^
[19:40] <Cimi> it should
[19:40] <Cimi> we can just change code in unity
[19:40] <Cimi> isn't it?
[19:41] <karni-holiday> Im on my pho e, please talk to cwayne.
[19:41] <karni-holiday> *phone
[19:41] <Cimi> ok
[19:41] <Cimi> cwayne, ping me when you're back
[19:41] <karni-holiday> thank you, Cimi
[19:41] <Cimi> karni-holiday, I can work on this next week, but I doubt I can make the carousel work with 1-5 elements
[19:42] <Cimi> karni-holiday, that's why we have to have grid for few elements...
[19:42] <Cimi> and UX-wise, is also way better
[19:42] <Cimi> a carousel with 1-5 elements does not make sense
[19:42] <Cimi> makes the interface slower to operate
[19:43] <Cimi> what I can possibly fix is the drag over bounds
[19:43] <Cimi> but I have to dig it properly on monday
[20:40] <ssweeny> Cimi, are we talking about <7 elements on tablet profile or in all cases falling back to grid?
[20:41] <Cimi> ssweeny, on tablet should be 7 or more
[20:41] <Cimi> ssweeny, phone maybe 5
[20:41] <kgunn> Cimi: you need anything from me?
[20:42] <ssweeny> Cimi, in that case i agree. 5 looks pretty reasonable to me on a phone but on a 10-inch tablet in landscape it would be weird
[20:43] <Cimi> kgunn, yep
[20:43] <Cimi> kgunn, I'm trying to understand what's going on, what happened
[20:43] <kgunn> heh
[20:43] <Cimi> kgunn, with the carousel minimum shown items
[20:44] <Cimi> kgunn, with oren we agreed we wanted filter grid for 6 or less
[20:44] <kgunn> Cimi: so big picture...everything is ok atm, building, running from the demo-stuff & phone-right-edge ppa
[20:44] <kgunn> we need to focus on the tweaks/bug fixes now
[20:45] <kgunn> landing is somewhat secondary....primary is getting fixes into branches feeding those ppas
[20:45] <kgunn> ...so
[20:45] <kgunn> for this carousel issue...we should just use our brains and pick the right limit
[20:46] <kgunn> Cimi: can you determine screen size from w/in carousel ?
[20:46] <kgunn> then set your limit based on that ?
[20:47] <kgunn> so you could keep your limit 6 on the phone...8 on the tablet ?
[20:47]  * kgunn wonders if i'm even tracking with the issue properly
[20:49] <cwayne> Cimi, i think we'd be okay with 5 on the phone, 7 on the tablet
[20:49] <cwayne> if that works for you :)
[20:49] <ssweeny> kgunn, we've been working with 5-item carousels on the phone and that's been going well for us
[20:49] <ssweeny> yeah what cwayne said :)
[20:50] <kgunn> makes sense to me
[20:50] <kgunn> Cimi: possible ?
[20:50] <kgunn> realizing its past your beer time Cimi
[20:50] <Cimi> kgunn, nope
[20:50] <Cimi> kgunn, I was on the loo :)
[20:50] <Cimi> hah
[20:51] <Cimi> anyway
[20:51] <Cimi> I'll have a look next week
[20:51] <kgunn> Cimi: ok, please do...surely from the sdk somewhere you can get dpi and pixel width of the parent...
[20:51] <Cimi> on monday morning I'll try to see if there's a clever way to detect
[20:51] <cwayne> don't we just check if units.gu > 80
[20:52] <Cimi> i depends on rotation... I'll ask next week
[20:52] <Cimi> best to know with the rest of the team which is the proper way of doing it
[20:52] <cwayne> oh, i didn't realize we were gonna have shell rotation
[20:52] <cwayne> cool!
[20:52] <Cimi> cwayne, not sure if it works yet :)
[20:53] <Cimi> but at least I now have the steps
[20:53] <Cimi> glad we agree on having the grid with less items
[20:54] <Cimi> I have to go, girlfriend is upset
[20:54] <Cimi> enjoy weekend guys!
[20:54] <cwayne> you too Cimi!