| Cimi | morning guys | 07:30 |
|---|---|---|
| Cimi | Saviq, dzien dobry - I finished friday if you want to check out | 07:40 |
| Cimi | Saviq, you on holiday today? | 08:00 |
| Saviq | Cimi, no, just late... sorry | 08:40 |
| Cimi | Saviq, no problem :) | 08:42 |
| Saviq | Cimi, I'll try and have a look, looks like today's gonna be a review day anyway | 08:42 |
| mhr3 | Saviq, a super quick one - https://code.launchpad.net/~unity-team/unity-api/scope-customizations/+merge/223893 | 08:43 |
| Saviq | mhr3, bump API version, and is there no test that checks for props? | 08:43 |
| mhr3 | Saviq, the version isn't fine-grained enough, so did just deb bump | 08:44 |
| Saviq | mhr3, why not "fine-grained enough"? | 08:45 |
| Saviq | mhr3, I don't mean the one in .h | 08:45 |
| Saviq | mhr3, but the one in CMakeLists.txt, that gets baked into the .pc file | 08:46 |
| Saviq | mhr3, that one's per-interface, meant exactly for this? | 08:46 |
| seb128 | Saviq, mhr3: hey, do you know who decided to use X-Ubuntu-Touch=true and why? | 08:46 |
| Saviq | seb128, ~me, because we were looking at all .desktop files and wanted to only list those that explicitly said that they support touch | 08:47 |
| Saviq | seb128, it should probably be rethought, or dropped altogether | 08:47 |
| Saviq | mhr3, deb-bump is needed, of course, for anyone that wants to implement that interface | 08:47 |
| mhr3 | Saviq, yea, exactly there's just set(VERSION 1) | 08:47 |
| mhr3 | sucks | 08:47 |
| Saviq | mhr3, yeah, just bump to 2 | 08:47 |
| Saviq | mhr3, it's just an interface version, what do you care ;) | 08:48 |
| Saviq | mhr3, but then the "other" version is for consumers, we actually should make it so Provides: gets automagically bumped | 08:48 |
| Saviq | mhr3, it's kinda like SONAME | 08:48 |
| seb128 | Saviq, right, because once we get Gtk/MIR ideally you should be able to run any GTK app on desktop, and we don't want to have to patch every single .desktop entry for that | 08:48 |
| Saviq | mhr3, just an int there as well | 08:48 |
| mhr3 | Saviq, it's completely backwards compatible, with minor addition, you don't do major version bumps in those cases | 08:48 |
| mzanetti | Saviq: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1302761 :/ | 08:48 |
| ubot5 | Ubuntu bug 1302761 in unity8 (Ubuntu) "Wrong icon when dragging items in the launcher" [Undecided,Confirmed] | 08:48 |
| Saviq | mhr3, that is true, only problem is that I will have to check whether that property is there if I'm not sure, or we'll be logging warnings all over the place ;) | 08:49 |
| Saviq | mzanetti, that should be fixed with 5.3 now? | 08:49 |
| mhr3 | Saviq, meh.. warnings if system is assembled with wrong components... who cares :P | 08:49 |
| mzanetti | Saviq: hmm... its not fixed with the latest stable image... currently flashing the development device. will check | 08:50 |
| Saviq | mhr3, well, the problem is, if you don't bump, you say those are *right* components | 08:50 |
| Saviq | mhr3, because scopes-shell provides unity-shell-impl-1, and that does not include the customizations prop | 08:50 |
| Saviq | mhr3, and if shell starts expecting that prop | 08:51 |
| mhr3 | Saviq, ultimately i don't want to have version 35 after two months | 08:51 |
| Saviq | mhr3, then stop changing the API! ;) | 08:51 |
| Saviq | mhr3, and why not? it's just a number :D | 08:51 |
| mhr3 | Saviq, tell design to stop changing things :) | 08:51 |
| mhr3 | or rather... to stop designing :) | 08:52 |
| mzanetti | Saviq: no, still broken | 08:52 |
| Saviq | mzanetti, bump the bug, then, Loïc wrote that it should be fixed with 5.3 | 08:52 |
| mhr3 | Saviq, imo we should do those bump max twice per cycle, it doesn't matter for us, and others implementing our interfaces will be basing them off released ubuntu | 08:53 |
| Saviq | mzanetti, https://bugs.launchpad.net/unity8/+bug/1296777/comments/16 | 08:53 |
| ubot5 | Ubuntu bug 1296777 in Unity 8 "avatar displayed incorrectly in notification" [High,In progress] | 08:53 |
| mzanetti | Saviq: well, unless we reverted 5.3 from the latest devel-proposed, its still broken | 08:54 |
| Saviq | mhr3, I don't understand the resistance, the whole purpose of this version is to bump it whenever the API changes (which should hopefully stabilize at some point) | 08:54 |
| Saviq | mzanetti, that's why I'm saying bump the bug (well, you might make it dupe of that last one I linked, since the problem is the same) | 08:55 |
| mhr3 | Saviq, it's a change which sole purpose is to force more changes without any real benefit right now | 08:55 |
| Saviq | MacSlow, silo 4 doesn't seem to include anything re: boot anim, or am I mistaken? | 08:55 |
| Saviq | mhr3, the benefit is that upgrading unity8 will require the upgraded plugin | 08:55 |
| mzanetti | Saviq: I bumped the bug, but I don't think anyone will notice as it doesn't have an assigneed in SDK | 08:55 |
| MacSlow | Saviq, no... that's why I'm guessing kgunn saw this problem. | 08:55 |
| mhr3 | Saviq, like we didn't do system images | 08:56 |
| MacSlow | Saviq, once my phone is charged I wanted to take a closer look, if that's really the reason... sofar I'm only guessing because mterry told me about that branch before the weekend | 08:56 |
| Saviq | mhr3, which break if stuff migrates from proposed partially | 08:56 |
| Saviq | mhr3, which would not be the case if unity8 was held in proposed until scopes-shell came up with the new Provides | 08:57 |
| Saviq | MacSlow, don't understand, why would the spinner come up anyway? what enabled it all of a sudden? | 08:57 |
| mhr3 | Saviq, which still wouldn't affect anything in this case... but fine you win | 08:57 |
| MacSlow | Saviq, that I don't know | 08:57 |
| Cimi | paulliu, tags! | 08:58 |
| Cimi | https://code.launchpad.net/~paulliu/unity8/lp1330957/+merge/223779 | 08:58 |
| Saviq | mhr3, if you come up with a better way (like versioned imports), I'm fine with that | 08:58 |
| Saviq | mhr3, but we still need to denote API bumps somewhere in debian/control | 08:58 |
| seb128 | Saviq, robert_ancell filed https://bugs.launchpad.net/unity8/+bug/1333029 on the topic btw, would be nice if you could comment | 08:59 |
| ubot5 | Ubuntu bug 1333029 in Unity 8 "Only shows applications with X-Ubuntu-Touch=true" [Undecided,New] | 08:59 |
| Saviq | seb128, unity8 does not filter them, probably unity-scope-click by now? it was originally unity-scope-applications | 08:59 |
| seb128 | is -click handling non clicks? | 09:00 |
| Cimi | paulliu, I don't understand though why it should not have any card or background in those tests | 09:00 |
| seb128 | mhr3, ^ | 09:00 |
| Saviq | seb128, yeah, there's no other scope | 09:00 |
| mhr3 | seb128, yes, both have checks for those | 09:00 |
| seb128 | Saviq, mhr3: can you just reassign/still comment on the intent/what we plan to do? ;-) | 09:01 |
| mhr3 | seb128, honestly i don't know... guess it's now kindof a "SupportsMir=true" thing | 09:02 |
| seb128 | mhr3, we are still not going to patch every single .desktop of the archive to add that | 09:02 |
| seb128 | that makes no sense | 09:03 |
| mhr3 | yet you don't want to show apps that you can't launch | 09:03 |
| seb128 | well, once you can start any gtk and qt apps, you don't have much you can't launch | 09:03 |
| Saviq | MacSlow, ok looks like the spinner is enabled in the current proposed image already | 09:03 |
| mhr3 | seb128, java, wxwidgets? | 09:03 |
| mhr3 | seb128, or really anything not written in gtk nor qt | 09:04 |
| Saviq | MacSlow, and it works fine, so unrelated | 09:04 |
| seb128 | mhr3, well, my point was most our apps are GTK or Qt ones | 09:04 |
| * Saviq kicks rebuild on the silo in any case | 09:04 | |
| MacSlow | Saviq, ah ok | 09:04 |
| seb128 | mhr3, I said "most" not "all" | 09:04 |
| mhr3 | seb128, so you'd still have apps that you can't launch.. sucks | 09:04 |
| seb128 | mhr3, either you have too much list of not enough | 09:05 |
| seb128 | I would prefer be able to run my apps that are working | 09:05 |
| seb128 | even if a few icons don't work | 09:05 |
| seb128 | than not being able to run things that work | 09:05 |
| Saviq | mhr3, seb128, maybe we need a blacklist shipped with the scope or something? | 09:05 |
| Saviq | mhr3, seb128, I agree maintaining it per-package is dumb | 09:05 |
| Saviq | but maybe a single list? | 09:06 |
| mhr3 | seb128, i'm just saying both sucks | 09:06 |
| Saviq | or well, isn't XMir supposed to solve all that? ;P | 09:06 |
| seb128 | yeah, it should ;-) | 09:06 |
| Saviq | crap | 09:06 |
| seb128 | mhr3, well, once we have XMir we should be able to run everything | 09:06 |
| mhr3 | that sounds like good enough solution then :) | 09:07 |
| seb128 | Saviq, can we land the unity8 logout? ;-) | 09:09 |
| mhr3 | Saviq, pushed | 09:09 |
| Saviq | seb128, trying! ;) | 09:09 |
| seb128 | Saviq, thanks | 09:10 |
| Saviq | mhr3, kthxbai, and sorry for pushing, but that's the system, the system might be flawed, but until we have a new one, I'd like to stick to it | 09:10 |
| Saviq | mhr3, it's not like I thought of it myself, Didier thought of it! | 09:10 |
| mhr3 | i knew it was didrocks' fault! | 09:11 |
| mhr3 | he's the evil man behind everything | 09:11 |
| mhr3 | and not even here | 09:11 |
| paulliu | Cimi: if we don't provide those in cardData, they will be null. | 09:21 |
| paulliu | Cimi: what tags? | 09:21 |
| Saviq | MacSlow, well... TBH https://launchpad.net/~ci-train-ppa-service/+archive/landing-004 | 09:26 |
| Saviq | MacSlow, that silo has no packages | 09:26 |
| Saviq | MacSlow, and I don't see how it could, there was a conflict between two branches | 09:27 |
| Saviq | MacSlow, so, in any case, I'm on it | 09:27 |
| MacSlow | Saviq, :) ok ... I'm ironing out the last bit on the unity8 combo-button branch | 09:27 |
| Saviq | Cimi, please grab pete-woods today to see what's up with new infog :) | 09:30 |
| Cimi | paulliu, why we don't provide them? | 09:32 |
| Cimi | paulliu, cimi@draco:~/Development/unity8/lp1330957$ bzr tags | 09:33 |
| Cimi | 0.1.16 ? | 09:33 |
| Cimi | pete-woods, ping indeed :) | 09:33 |
| pete-woods | Cimi: hi | 09:38 |
| Cimi | pete-woods, welcome back | 09:38 |
| pete-woods | thanks :) | 09:38 |
| Cimi | pete-woods, so we got the infographics running, but camera app doesn't populate! | 09:41 |
| Cimi | pete-woods, also, can we have a backport compativility with older infographics? | 09:41 |
| pete-woods | Cimi: you need to install the camera app from click | 09:42 |
| pete-woods | otherwise we don't get the hooks set up | 09:42 |
| pete-woods | I don't think that our current PPA mechanism actually allows us to test this | 09:42 |
| === vrruiz_ is now known as rvr | ||
| Cimi | pete-woods, how? | 09:44 |
| pete-woods | Cimi: well there's a click-build.sh script in the camera app source tree | 09:45 |
| Cimi | pete-woods, we have a silo with it | 09:45 |
| pete-woods | silos only contain debs though, right | 09:45 |
| pete-woods | not click packages | 09:45 |
| Cimi | ok | 09:45 |
| Cimi | pete-woods, so we build it in the phone? | 09:46 |
| pete-woods | Cimi: actually I have an idea | 09:46 |
| pete-woods | Cimi: I should make the camera app work in both deb mode and click mode | 09:46 |
| pete-woods | Cimi: give me a little bit and I'll update the branch to support both modes | 09:47 |
| Saviq | mzanetti, can you have another look at bug #1302761, it looks to me like the underlying issue is fixed actually, the behaviour in the launcher is now different - before it always displayed the same icon (the first one that was ever dragged) | 09:57 |
| ubot5 | bug 1302761 in Ubuntu UI Toolkit "Wrong icon when dragging items in the launcher" [Undecided,New] https://launchpad.net/bugs/1302761 | 09:57 |
| Saviq | mzanetti, now it always shows the *previous* icon that you dragged | 09:57 |
| Saviq | mzanetti, and the other instance of this bug (wrong avatar in notification) looks fine now | 09:57 |
| mzanetti | mhm... will have alook. still the code in the launcher didn't change | 09:58 |
| Saviq | mzanetti, so it looks like the icon is updated now, but uses the previous one somehow | 09:58 |
| Saviq | mzanetti, and please mention on the bug that this was supposed to be fixed with 5.3 and is not | 09:58 |
| Saviq | mzanetti, FWIW your test case works, too https://launchpadlibrarian.net/172053666/shapetest.qml | 10:00 |
| karni | Saviq / mhr3: Do you guys know whether Unity preview art is right-angle shaped by design? (no longer contained within Ubuntu shape, that is) - Joe likes that, I just wanted to check with you guys whether that was accidental or design team decision :) | 10:00 |
| Saviq | karni, no, first I've heard (we did decide on an option to disable, but didn't implement yet) | 10:01 |
| Saviq | karni, and I can't repro, can you? | 10:02 |
| dednick | anyone noticed that the wider the shell window is, the darker the background? not sure if it's only on desktop app though... | 10:26 |
| Saviq | dednick, which background? dash or greeter? | 10:32 |
| dednick | Saviq: dash | 10:32 |
| Saviq | dednick, right, ouch | 10:33 |
| dednick | Saviq: although not really sure how it's doing it. probably just an svg thing. increasing size changes gradient | 10:34 |
| Saviq | dednick, looks like the notification overlay | 10:34 |
| Saviq | dednick, no, you can't interact with the dash any more either :| | 10:34 |
| dednick | Saviq: ah. didnt notice that! | 10:34 |
| dednick | woops | 10:34 |
| Saviq | it only happens on resize, though | 10:35 |
| Saviq | something weird | 10:35 |
| dednick | Saviq: ya, it works if you resize before switching from greeter | 10:35 |
| Saviq | MacSlow, it looks like the notification input-block overlay behaves weirdly when you resize the window, could you please have a look | 10:35 |
| Saviq | MacSlow, ./run.sh; unlock, resize → overlay activates | 10:36 |
| Cimi | pete-woods, any luck? | 10:48 |
| pete-woods | Cimi: yep, you'll need to rebuild both libusermetrics and the camera-app though | 10:49 |
| === TheMuso` is now known as TheMuso | ||
| Cimi | pete-woods, Saviq can you update silo? ^ | 10:50 |
| * Saviq does | 10:53 | |
| Saviq | I rebuilt the whole thing, unity8 changed, too | 10:53 |
| Saviq | in the mean time | 10:53 |
| MacSlow | Saviq, ok... I will. | 10:57 |
| Saviq | MacSlow, shout if you want me to file a bug | 10:57 |
| MacSlow | Saviq, sure | 10:58 |
| === MacSlow is now known as MacSlow|lunch | ||
| pete-woods | Saviq: could you add this MR to the infographics silo? https://code.launchpad.net/~unity-api-team/telephony-service/file-based-infographics/+merge/224105 | 11:02 |
| pete-woods | just realised that we need to update the telephony service, too | 11:03 |
| Saviq | pete-woods, doing | 11:04 |
| pete-woods | Saviq: thanks! | 11:04 |
| Saviq | pete-woods, btw, shouldn't the compat layer have taken care of telephony, though? | 11:31 |
| Saviq | pete-woods, or does the compat layer require a bit adaptations in any case? | 11:31 |
| pete-woods | Saviq: yes, but you still need to add the infographic hook, the API is exactly the same | 11:32 |
| Saviq | pete-woods, ah ok, that | 11:33 |
| pete-woods | otherwise I'd have to go globbing the cache directory for stuff | 11:33 |
| * pete-woods doesn't want to do that | 11:33 | |
| pete-woods | the other alternative would be to add an extra permission for apps to write in a common dir | 11:34 |
| Saviq | got it | 11:34 |
| seb128 | Saviq, is anything blocking the landing of the logout silo? did you want to give it an extra round of testing? | 11:36 |
| Saviq | seb128, I don't think unity8's testplan was executed against it | 11:37 |
| Saviq | seb128, I'm in the process of landing another unity8 silo now | 11:38 |
| seb128 | k :/ | 11:38 |
| Saviq | seb128, will get onto the logout one asap | 11:38 |
| seb128 | which means that one is going to need another rebuild/test round? | 11:38 |
| Saviq | seb128, yeah | 11:39 |
| Saviq | seb128, but just unity8-wise | 11:39 |
| seb128 | right | 11:39 |
| Saviq | seb128, I'll take care of that today | 11:39 |
| seb128 | Saviq, thanks | 11:40 |
| mzanetti | Saviq: ok. found the issue with the launcher icons | 11:54 |
| Saviq | mzanetti, local isn't it? | 11:54 |
| mzanetti | Saviq: so the thing is, that the UbuntuShape now seems to do stuff slower | 11:55 |
| mzanetti | Saviq: i.e. the Image gives me statusChanged to Ready | 11:55 |
| mzanetti | Saviq: but the Shape still paints the old one | 11:55 |
| mzanetti | then I update the ShaderEffectSource | 11:55 |
| mzanetti | and only at some point later the Shape would update | 11:55 |
| mzanetti | but my ShaderEffectSource is not updating any more | 11:55 |
| mzanetti | so not sure... | 11:56 |
| mzanetti | feels like a bug in UbuntuShape still | 11:56 |
| Saviq | mzanetti, hmm hmm | 11:56 |
| Saviq | mzanetti, we'd need a status on UbuntuShape then, wouldn't we | 11:56 |
| Saviq | mzanetti, you've a non-live Source there, right? | 11:56 |
| mzanetti | yep | 11:56 |
| mzanetti | statusChanged on the shape would help I guess | 11:56 |
| mzanetti | or - making it somewhat sync again, to be done when the Image emits statusChanged | 11:57 |
| Saviq | mzanetti, so I think it's fine that Image gives up Ready, because well... it is, but we'd need Shape to have a status as well | 11:57 |
| paulliu | Cimi: I don't know. Maybe the code will be smaller? | 11:57 |
| Saviq | mzanetti, or well, just make it live? | 11:57 |
| mzanetti | Saviq: hmm... should I? | 11:57 |
| mzanetti | Saviq: its all launcher icons | 11:58 |
| mzanetti | well, I could make it live while dragging for sure | 11:58 |
| mzanetti | will do that | 11:58 |
| Saviq | mzanetti, other than Shape having a status prop, not sure what else :| | 11:58 |
| Saviq | yeah, that ↑ should work fine | 11:58 |
| mzanetti | well, I know my way around | 11:59 |
| mzanetti | Saviq: just wanted to let you know and see if your opinion matches mine | 11:59 |
| Saviq | mzanetti, I think it should be fine live in any case | 12:00 |
| Saviq | mzanetti, since according to http://qt-project.org/doc/qt-5/qml-qtquick-shadereffectsource.html#live-prop | 12:00 |
| Saviq | mzanetti, it will only update whenever the actual item changes | 12:00 |
| mzanetti | Saviq: yeah... sounds about right... | 12:00 |
| mzanetti | ok. will drop the non-live stuff | 12:00 |
| mzanetti | cheers | 12:01 |
| === MacSlow|lunch is now known as MacSlow | ||
| Saviq | mhr3, just thought of one more appearance prop that we might need to expose | 12:40 |
| Saviq | mhr3, icon key color for colourizing | 12:40 |
| Saviq | mhr3, since we default to #808080, if someone supplies an icon using that colour, it would be colourized | 12:42 |
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| === dandrader is now known as dandrader|afk | ||
| MacSlow | Saviq, did you file the bug regarding the input-blocking MouseArea of notifications yet? | 13:05 |
| Saviq | MacSlow, you didn't shout! ;) | 13:05 |
| MacSlow | Saviq, damn :) | 13:06 |
| MacSlow | Saviq, in turn I will then file a bug against xchat's new voice-amplification plugin ;) | 13:07 |
| Saviq | MacSlow, can still file one if you'd like, but not sure we really need it, feels like an unnecessary overhead | 13:17 |
| MacSlow | Saviq, organization-wise your call... I'll look into it anyway. | 13:18 |
| Saviq | MacSlow, yup, no bug then | 13:18 |
| === dandrader|afk is now known as dandrader | ||
| === _salem is now known as salem_ | ||
| Cimi | paulliu, ping | 13:39 |
| paulliu | Cimi: hi | 13:39 |
| paulliu | Cimi: I just removed the tags. | 13:40 |
| Cimi | paulliu, cool | 13:40 |
| paulliu | Cimi: I'm wondering where the tags comes from. | 13:40 |
| Cimi | paulliu, could you also add the margin when there is just the title? | 13:40 |
| Cimi | paulliu, or better | 13:40 |
| Cimi | paulliu, when there is no art or mascot | 13:40 |
| Cimi | paulliu, basically we always want the margin | 13:41 |
| paulliu | Cimi: isn't the current patch already done that? | 13:42 |
| paulliu | Cimi: I mean, if there's art or mascot, it generates the margin already on the other part. | 13:42 |
| paulliu | Cimi: And subtitle is following title's anchor. | 13:43 |
| mhr3 | Saviq, colorizing what where? | 13:45 |
| Saviq | mhr3, foreground colour | 13:46 |
| Saviq | mhr3, affects emblem and attributes as well | 13:46 |
| Saviq | mhr3, but if someone supplies an icon that is actually meant to be #808080 | 13:47 |
| mhr3 | Saviq, so if i set it to red, what will be red? | 13:47 |
| Saviq | mhr3, that's the key color | 13:47 |
| Saviq | mhr3, if you set the key color to red, all the red in the image will be replaced with the foreground color | 13:47 |
| mhr3 | ok, i see now | 13:48 |
| mhr3 | why isn't that just using normal transparency? | 13:48 |
| mhr3 | oh or is that like symbolic icons? | 13:49 |
| Saviq | mhr3, yes | 13:52 |
| Saviq | mhr3, it's for theme icons | 13:53 |
| Saviq | mhr3, basically flat monochrome icons are meant to have the same colour as the text | 13:53 |
| mhr3 | right, don't think it will be used much, but yea, just add it to the json spec | 13:54 |
| === salem_ is now known as _salem | ||
| pete-woods1 | Saviq: hey, I pushed what I think is the right fix for the telephony-service break | 14:09 |
| pete-woods1 | looks like it was caused by qt5.3 | 14:10 |
| === pete-woods1 is now known as pete-woods | ||
| pete-woods | it added some UID checks which were confused by their LD_PRELOADed system library | 14:11 |
| Saviq | pete-woods, same MP? | 14:11 |
| pete-woods | yep | 14:11 |
| pete-woods | Saviq: can split if you want | 14:11 |
| Saviq | pete-woods, not my project ;) | 14:11 |
| Saviq | pete-woods, build kicked | 14:12 |
| pete-woods | Saviq: thanks :) | 14:12 |
| === _salem is now known as salem_ | ||
| mhr3 | Saviq, known that left swipe from an app to dash flickers? | 14:16 |
| dednick | Saviq: can i use url dispatcher in qml? | 14:16 |
| mhr3 | Saviq, i mean, there isn't an animation anymore after the switch to 5.3 | 14:17 |
| Saviq | dednick, Qt.openUrlExternally | 14:17 |
| Saviq | mhr3, I just filed a bug | 14:17 |
| dednick | Saviq: cool. thanks | 14:17 |
| Saviq | mhr3, bug #1333261 | 14:17 |
| ubot5 | bug 1333261 in Unity 8 "Left-edge long-swipe dash fade-in is not smooth" [Critical,Triaged] https://launchpad.net/bugs/1333261 | 14:17 |
| mhr3 | thx | 14:18 |
| Saviq | mhr3, just dumped its importance to Medium, as this goes away with dash-as-app anyway.. | 14:20 |
| Saviq | we'll probably just be bringing the dash from the left is all | 14:20 |
| mhr3 | sounds reasonable | 14:21 |
| Saviq | not like we've design for it :| | 14:22 |
| MacSlow | Saviq, I made the regex-suggestion work and tests sofar look good... http://macslow.org/regex-icon-label-snap-decision.png | 14:26 |
| Saviq | MacSlow, cool, this will get us a bit more robustness for sure | 14:27 |
| MacSlow | Saviq, there are two examples skipping the icon... and one using a colon as part of the normal text. | 14:27 |
| Saviq | MacSlow, I can help you with the regex if you need | 14:27 |
| Saviq | but gotta run now, biab | 14:28 |
| MacSlow | Saviq, regarding the "gap" for the omitted icon... I'm not what the opinion of Design is... iirc they always want an icon for the comboList-entries. | 14:28 |
| MacSlow | Saviq, no the regex is fine. | 14:28 |
| paulliu | Cimi: seems the tags are coming from trunk. | 14:30 |
| Cimi | paulliu, remove them anyway | 14:30 |
| MacSlow | greyback, standup | 14:33 |
| dednick | Saviq: hmm Qt.openUrlExternally doesnt seem to work with url-dispatcher | 14:34 |
| dednick | Saviq: looks like we haven't got support in unity8. I'll add it to utils | 14:37 |
| === salem_ is now known as _salem | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| Saviq | dednick, it's from QPA, no idea why it wouldn't work (and it should!) | 16:03 |
| Saviq | dednick, aren't we using it when activating stuff in indicators for example? | 16:03 |
| dednick | Saviq: it's done through indicator backend | 16:04 |
| Saviq | dednick, right, in any case, it should work, if it doesn't - it's a bug | 16:04 |
| Saviq | seb128, unity8 still didn't migrate, so logout is still blocked, I'll come back later tonight and will try and land it | 16:05 |
| dednick | Saviq: from qpa? as in qtubuntu? | 16:05 |
| seb128 | Saviq, right, see -touch for unity8 not migrating | 16:05 |
| seb128 | Saviq, needs some click revert | 16:05 |
| Saviq | dednick, yes | 16:05 |
| seb128 | Saviq, don't bother about coming back, tomorrow is fine | 16:05 |
| seb128 | Saviq, enjoy your evening ;-) | 16:06 |
| dednick | Saviq: openUrlExternally does not reside there (on tree search) | 16:06 |
| Saviq | dednick, https://code.launchpad.net/~aacid/qtubuntu/qtubunturl/+merge/181752 | 16:06 |
| Saviq | dednick, it's Qt API exposed from QDesktopServices | 16:06 |
| === gatox is now known as gatox_lunch | ||
| dednick | Saviq: i c. hum. maybe doesnt work on desktop then | 16:08 |
| dednick | need to set qpa | 16:08 |
| mhr3 | it falls back to regular xdg-open on desktop | 16:09 |
| Saviq | dednick, it should work even more on desktop ;) | 16:10 |
| Saviq | dednick, try a simple .qml with onCompleted: Qt.openUrlExternally("http://google.com") | 16:10 |
| Saviq | dednick, if that works, it should work in u8 just as well - if doesn't → bug! ;) | 16:10 |
| mhr3 | Saviq, it will work, but won't go over ual | 16:11 |
| Cimi | Saviq, let me push without the padding fix | 16:11 |
| Cimi | Saviq, I'll fix when we get emblems in | 16:11 |
| Cimi | Saviq, and paul branch | 16:11 |
| Cimi | Saviq, or the worlds will collide | 16:11 |
| Cimi | it's an aeasy fix, I'll make the padding static possibly | 16:11 |
| Saviq | mhr3, sure | 16:12 |
| Saviq | Cimi, k | 16:12 |
| dednick | Saviq: http works | 16:13 |
| dednick | but custom url does now | 16:13 |
| dednick | *not | 16:13 |
| dednick | gvfs-open: dialer:///?view=liveCall: error opening location: The specified location is not supported | 16:13 |
| Saviq | dednick, ah, nothing handles those outside of the phone | 16:13 |
| dednick | Saviq: ah. but it'll work on phone? | 16:14 |
| Saviq | dednick, that will only work on the phone, as url-dispatcher is the one that knows these url schemes, xdg doesn't | 16:14 |
| dednick | Saviq: but i have url dispatcher installed on my desktop.. | 16:14 |
| Saviq | dednick, still, you don't use the QPA which uses url dispatcher | 16:14 |
| Saviq | dednick, so it goes straight to xdg-open | 16:14 |
| dednick | Saviq: ah. i c | 16:14 |
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| dednick | Saviq: hm. should i just "assume" it works on phone (test obviously) and go through Qt.openUrlExternally, or directly on the url-dispatcher? | 16:16 |
| dednick | since i've written a utils plugin component for it. | 16:17 |
| dednick | screw it. i'll use the qt function | 16:18 |
| === dandrader_ is now known as dandrader|lunch | ||
| === _salem is now known as salem_ | ||
| kgunn | dednick: thanks for adding unit test to unity-mir trusted session | 16:50 |
| kgunn | unrelated topic, did the battery indicator drop the slider for for brightness ? | 16:50 |
| kgunn | charles: ^ ? | 16:52 |
| dednick | kgunn: http://bazaar.launchpad.net/~indicator-applet-developers/indicator-power/trunk.14.10/revision/244 | 16:58 |
| kgunn | ta | 16:58 |
| === alan_g is now known as alan_g|EOD | ||
| dednick | kgunn: although it looks like that decision has been reverted | 17:00 |
| === gatox_lunch is now known as gatox | ||
| charles | kgunn, yes but no. last week mpt and seb128 among others dicussed it and decided to reinstate the slider in indicator-power | 17:01 |
| dednick | kgunn, charles: https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1289470/comments/4 | 17:01 |
| ubot5 | Ubuntu bug 1289470 in indicator-power (Ubuntu) "The phone has three brightness sliders" [High,Confirmed] | 17:01 |
| charles | kgunn, with the rationale being that it was something that needed quicker access than navigating through settings | 17:01 |
| === dandrader|lunch is now known as dandrader | ||
| greyback | dednick: hey I added some comments/questions to https://code.launchpad.net/~nick-dedekind/unity-mir/trusted-sessions/+merge/223432 | 17:06 |
| dednick | greyback: oh my, that's a long one. might have to wait for tomorrow morning ;) | 17:06 |
| greyback | dednick: still digesting it though, there's more to come | 17:07 |
| greyback | dednick: sure. I'm mostly worried about you creating unitymir::Session - I'm not sure why you need that. Anyway, there's no rush, we can take up that topic tomorrow | 17:07 |
| dednick | greyback: hm. it's juts a wrapper really. not much use. it used to do more, but i wasn't sure about holding a list of mir sessions in the app manager. | 17:11 |
| greyback | dednick: yeah, I'm not a fan of that either. | 17:11 |
| greyback | anyway, tomorrow! | 17:12 |
| dednick | greyback: huh. actually doens't look like i use the tracked sessions much anymore. Might be able to get rid of the lits alltogether. talk about it tomorrow. | 17:15 |
| dednick | s/lits/list | 17:15 |
| greyback | ack. Enjoy evening! | 17:15 |
| dednick | you too | 17:16 |
| === salem_ is now known as _salem | ||
| kgunn | mterry: hey...you fixed the spinner eating cpu prob right ? | 19:16 |
| mterry | kgunn, yeah | 19:16 |
| mterry | kgunn, you asking that question makes me nervous :) | 19:17 |
| kgunn | mterry: lemme look at one thing.... | 19:18 |
| kgunn | mterry: so i was testing AlbertA's ppa of powerd-to-usc display blank changes....and saw spinner taking like 8% | 19:24 |
| kgunn | i just reverted the ppa & its still there | 19:24 |
| mterry | kgunn, the spinner shouldn't even be running | 19:24 |
| kgunn | i retested n4 with the ppa and it is fine...so i'm thinking this is an n10 specific issue | 19:24 |
| mterry | kgunn, and your unity-system-compositor is all upgraded? | 19:24 |
| mterry | kgunn, huh... I haven't played with n10 for a while | 19:25 |
| mterry | That being a difference doesn't make sense to me though... | 19:25 |
| kgunn | mterry: yeah these are all devel-proposd images from today | 19:25 |
| kgunn | mterry: hang on... | 19:25 |
| kgunn | mterry: so weird...n10, if screen is off u-s-c is ~1.2% cpu | 19:26 |
| kgunn | with screen on it hovers around 7.5 to 9% | 19:26 |
| mterry | kgunn, oh wait, USC itself? | 19:27 |
| mterry | kgunn, sounds different from the spinner issue -- that's a separate process | 19:27 |
| mterry | Which is killed by USC when it's done | 19:27 |
| kgunn | 1459 root 20 0 182344 23716 19516 S 9.8 2.1 0:13.06 unity-system-compositor --spinner=/usr/bin/unity-system-c | 19:27 |
| mterry | kgunn, and separately, that CPU usage doesn't sound good, agreed | 19:27 |
| mterry | kgunn, yeah "spinner" will show up on the command line, but it's just passing a command for USC to run and then kill | 19:27 |
| kgunn | mterry: right...i did know that.... | 19:28 |
| kgunn | mterry: which is why there is 1 usc... | 19:28 |
| kgunn | that takes arg spinner... | 19:28 |
| kgunn | sorry, just coming back to you only cause we had the problem last | 19:28 |
| kgunn | but maybe this is AlbertA's prob ? | 19:28 |
| kgunn | or who knows....maybe its been there | 19:28 |
| kgunn | and we've never noticed | 19:28 |
| kgunn | which is more likely, n10 power has always super-sucked | 19:29 |
| mterry | kgunn, on n10, I wouldn't know when it showed up. I rarely boot up my n10 | 19:29 |
| AlbertA | kgunn: can't think of anything that would cause it right now | 19:29 |
| mterry | Meaning, I wouldn't have noticed when it first started | 19:29 |
| kgunn | AlbertA: i'm willing to continue landing, log a bug...and assume this was a pre-existing condition | 19:30 |
| alberto_ | kgunn: I see usc using 8-9% on a virigin image | 19:41 |
| alberto_ | kgunn: I don't see spinner | 19:41 |
| alberto_ | kgunn: just with adb shell top | 19:41 |
| mterry | alberto_, yeah that's the problem -- spinner is a red herring. Looks like USC usage is too high | 19:44 |
| kgunn | kdub_: is there some reason usc might just churn on n10 ? | 19:46 |
| kdub_ | I can't think of one | 19:47 |
| kdub_ | might be interesting to turn on some of the mir logs (maybe rpc-report), and see if anything is talking to it | 19:48 |
| kdub_ | or compositor-report if its suspected that we're drawing idly | 19:48 |
| alberto_ | kgunn: is this with the screen off? | 19:54 |
| kgunn | alberto_: so with screen off, its like ~1.2%...screen on its ~7.5-9% | 19:56 |
| alberto_ | kgunn: yeah, it's all the compositing thread | 19:56 |
| alberto_ | kgunn: and the mali render thread | 19:56 |
| alberto_ | kgunn: and unity8 is around 30% with screen on | 19:56 |
| alberto_ | mostly in the QSGRenderThread | 19:56 |
| kgunn | alberto_: but even so...1.2% during screen off ?...what's it doing ? | 19:56 |
| alberto_ | kgunn: I see 18% cpu on unity8 with screen off | 19:58 |
| alberto_ | kgunn: so in usc I see most of it is in the main usc thread and mir/ipc handling thread | 19:58 |
| alberto_ | kgunn: so mainloop + Ipc probably from being called by unity8 | 19:59 |
| kgunn | alberto_: huh...my n10, screen off w/ ppa from silo20 on today's devel-proposed image consistently 1.2% with screen off | 19:59 |
| kgunn | alberto_: oh sorry...you said unity8 | 19:59 |
| kgunn | yes...i see that too... | 19:59 |
| alberto_ | yeah usc 1% unity8 18% | 19:59 |
| kgunn | alberto_: that's just crazy | 20:00 |
| alberto_ | if you do top -H pid <pid> you can see which threads | 20:00 |
| mhall119 | kgunn: ping | 20:00 |
| kgunn | mhall119: what's up? | 20:01 |
| kgunn | alberto_: that's still nuts... | 20:01 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
