=== Guest42341 is now known as O
=== O is now known as Guest78101
tsdgeosMirv: not sure if we told you or you did realize but the support for the audio roles landed in unity808:49
Mirvtsdgeos: I did realize it, and I've rebuilt it in 5.5.1 PPA and I would test it if xenial images were not broken (not sure if they would now be fixed already)08:49
Mirvtsdgeos: anyway, it's excellent! a big landing you had.08:50
tsdgeosyep :D08:50
tsdgeoswe roll like that08:50
Mirvs/I did realize it/I did publish it/ ;)08:50
tsdgeosdednick: did you see daniel's comment in https://code.launchpad.net/~nick-dedekind/qtubuntu/1493530-mimeData/+merge/272630 ?09:34
tsdgeosmzanetti: what's the fate of https://code.launchpad.net/~mzanetti/ubuntu-clock-app/detect-qtmm-version/+merge/275177 ?09:34
mzanettitsdgeos, I don't know...  I'll ping popey to get one of them merged09:35
mzanettipopey, ^ :)09:35
* popey looks09:36
dednicktsdgeos: no i havent09:37
dednickQ_ASSERT? not so sure. it seems to call it with null every startup.09:37
dednicktsdgeos: or maybe not. i do keep getting a log message about null clipboard data09:40
dednickcant remember where from though...09:40
dednickprobably server side09:40
tsdgeosdednick: not saying he's right or nt, just saying "let's move it forward, he needs an answer" :D09:41
mzanettidednick, hey, any progress on the freezes Saviq reported?09:43
mzanettiI had them quite often during the weekend09:43
dednickmzanetti: no. i managed to get it to freeze a few times, but didnt have debug symbols. After i installed, i spent about an hour trying to reproduce it with no luck. I'll try again today.09:45
dednickmzanetti: when i did get it to freeze, it seemed to be stuck in a loop. was getting loads of log messages from the card creator re "binding loop for height"09:46
dednickbut there's also another freeze which is happening (in trunk as well i think) where it will freeze for 5-10 seconds and come right again.09:47
dednickmzanetti: does yours come right or keep locked?09:47
mzanettidednick, afaict it keeps locked until I open the right edge09:48
mzanettihowever, if it freezes for multiple seconds, there's the chance that I've just not been patient enough09:49
dednickmzanetti: next time it happens can you try wait for a bit?09:49
mzanettidednick, happens like 5 times a day or so... at least over the weekend09:50
popeymzanetti, I don't quite understand which of the two merges should land. there's another linked from that one, and loads of abstentions!09:50
mzanettipopey, you decide :)09:50
Guest78101mzanetti, same here on arale, constant freezing09:51
mzanettiyep, rc-proposed09:51
=== Guest78101 is now known as QUESTION
QUESTIONcan i ask something??09:53
ubot5Please don't ask to ask a question, simply ask the question (all on ONE line and in the channel, so that others can read and follow it easily). If anyone knows the answer they will most likely reply. :-) See also !patience09:56
=== QUESTION is now known as Guest78101
davmor2Guest78101: did you ask the question yet?10:02
dednicktsdgeos: i've commented and pushed a change. I don't think we should be clearing the clipboard.10:14
tsdgeosoki :)10:14
tsdgeosi don't know enough about it tbh10:14
mzanettitsdgeos, ok, all branches merged10:29
tsdgeosmzanetti: i think you missed https://code.launchpad.net/~mzanetti/unity8/ubuntuanimations/+merge/276511 ?10:30
=== marcusto_ is now known as marcustomlinson
dandradermzanetti, made so that https://code.launchpad.net/~dandrader/unity8/noStretchOnResize/+merge/274752 now does not depend on that qtmir branch anymore12:10
dandradermzanetti, so it can get merged independently12:10
mzanettidandrader, nic12:10
dandradermzanetti, it will take forever for the qtmir branch to get landed because of other outstanding branches that also change the same code.12:11
dandradermzanetti, and I fear leaving that untiy8 branch hanging aroung as that part of the code (desktop window management) is under heavy development, so it can bit rot quite fast12:12
mzanettifair enough, yes12:12
=== alan_g is now known as alan_g|lunch
mzanettitsdgeos, hey, https://code.launchpad.net/~aacid/unity8/dash_reset_instead_of_fatal/+merge/274363 links to an abandoned silo13:22
mzanettiwhat's the status? should unity-scopes=8 be landed by us then?13:22
tsdgeosmzanetti: pstolowski what happened with that silo?13:30
mzanettitsdgeos, the dash seems to freeze for a couple of seconds frequently since the last landing. any thoughts?13:31
tsdgeosnot really let me check the log13:32
=== dandrader is now known as dandrader|afk
mzanettitsdgeos, seems to happen quite often when it is focused13:50
mzanettitsdgeos, only happening with the dash tho afaict13:50
tsdgeosmzanetti: i can't find anything obvious13:51
tsdgeoseven though with the move to 1.3 there's lots of ubuntu shape changes13:51
pstolowskimzanetti, tsdgeos i had to free that silo for now cause we were short on silos. will recreate it once aggregators are ready and i can land the feature13:51
pstolowskimzanetti, tsdgeos feel free to land your branch with another silo13:52
tsdgeospstolowski: can't since you made me increase the dependencies :D13:53
pstolowskitsdgeos, ah, indeed13:53
tsdgeospstolowski: so can i have the names of the branches that it depends on so i can list them in our MR?13:54
tsdgeosmzanetti: do you do anything in special? or?13:55
mzanettitsdgeos, this is reproducible quite easily. just focus some other app, after a few secs go back to the dash and continuosly flick it up and down. after some few seconds it will freeze for like 10 secs or so13:56
mzanettitsdgeos, nothing in the logs :/13:56
tsdgeos10 seconds?13:56
mzanettisomething like that. not always the same amount13:56
tsdgeosi thought you meant something like 0.2 seconds :D13:56
mzanettisometies like 3 secs, but also have seen 20 too13:56
tsdgeosok, let me try to reproduce it here and i'll try to figure out what's wrong13:57
mzanetticool, thanks13:57
pstolowskitsdgeos, https://code.launchpad.net/~stolowski/unity-scopes-shell/diff-updates/+merge/27355413:58
pstolowskitsdgeos, and https://code.launchpad.net/~stolowski/unity-api/scope-activate-action/+merge/27355713:58
tsdgeosok, tx, mzanetti i've updated the branches13:59
=== dandrader|afk is now known as dandrader
=== alan_g|lunch is now known as alan_g
=== balloons is now known as Guest42098
Guest42341tsdgeos, same thing as mzanetti here with arale on rc-proposed, the dash freezes for 2-10 sec14:41
tsdgeosgreyback_: what can cause this? ↓14:41
tsdgeosqtmir.surfaces: MirSurfaceItem::dropPendingBuffer() surface = qtmir::MirSurface(0x2e1ce10) buffer dropped. 0 left.14:41
tsdgeosmzanetti: ↑ i get that as unity8 log when i see the freeze, can you confirm?14:41
greyback_tsdgeos: if a surface is set visible=false, and app keeps drawing. We drop those new frames from the app to avoid blocking it14:41
tsdgeosgreyback_: how can i know who 0x2e1ce10 belongs to?14:42
mzanettiI think I saw this message even though the surface was visible14:42
tsdgeosgreyback_: is it possible there's a bug in there? i'm seeing it in conjunction with unity8-dash rendering "freezing" when on the front14:42
greyback_tsdgeos: should be printed earlier in the log14:42
greyback_tsdgeos: possible, sure.14:43
mzanettidednick, so looks like being occlusion related after all ^14:43
greyback_dednick: you were trying to repro that last week ^^14:43
tsdgeosgreyback_: http://paste.ubuntu.com/13208991/ :/14:43
greyback_mzanetti: occlusion is the act of setting surfaces visible=true/false14:43
mzanettiyes. I know14:43
tsdgeosgreyback_: maybe i need to enable more logging?14:43
greyback_tsdgeos: from high level, would indicate to me the MirSurfaceItem has visible=false, but app is still drawing somehow.14:44
greyback_tsdgeos: there is no more to enable, we should improve that14:44
mzanettigreyback_, yep. but the surface *is* visible. so seems like the occlusion branches tell MirSurface it would be not even though it is14:45
greyback_mzanetti: a likely theory14:45
greyback_I'm assuming this just happens randomly?14:46
mzanettigreyback_, quite reproducible14:46
tsdgeosit's randomly quite reproducible :D14:46
greyback_dednick wasn't able to on Friday, playing around for a good hour14:47
tsdgeosi mean you can't make it happen 100% but you can make it happen quite often14:47
mzanettigreyback_, open some random app, keep it focused for a few secs, then switch to dash and swipe it up/down frequently. the dash will freeze14:47
mzanettidefinitely a good 90% hit rate here14:47
tsdgeosi've a much lower hit rate14:47
tsdgeosmaybe 30%14:47
tsdgeosbut still "reproducible enough"14:48
mzanettikrillin here14:48
tsdgeosMX4 for me14:48
tsdgeosmaybe the speedness of the hw "helps"14:48
greyback_mzanetti: please add that to the bug, would help14:48
greyback_also what device? I suspect a race condition14:48
greyback_yeah, my thinking14:48
=== Guest42098 is now known as balloons_
mzanettigreyback_, is there a bug report for it?14:48
greyback_dednick ^ ?14:48
dednickreading backlog14:49
dednicki was getting those messages when the surface was visible in u814:50
greyback_dednick: which would make me suspicious. We shouldn't need to drop frames from a visible surface14:51
greyback_only case that would happen is if client not respecting vsync and swapping faster than 60htz14:51
greyback_and dash doesn't do that14:52
mzanettioh dear... OTA-8 final freeze is tomorrow14:53
tsdgeosgreyback_: mzanetti: dednick: so i leave the investigation in your hands? you probably can find a cause/cure faster than me in that area15:01
mzanettitsdgeos, yes, thanks for confirming tho15:01
dednicki thought occlusion has landde?15:01
greyback_tsdgeos: yep15:01
mzanettidednick, yes, it has, which is the issue with the OTA-8 freeze15:01
mzanettidednick, need to ship a fix by tonight I guess, or we'll have to revert15:01
dednickmzanetti: ok15:02
=== balloons_ is now known as balloons
=== dandrader is now known as dandrader|bbl
ltinklcimi, if you want to review the new deco visuals, be my guest: https://code.launchpad.net/~lukas-kde/unity8/newWindowDecosAndPanel/+merge/27681016:31
tsdgeoscimi: so https://trello.com/c/dNjca4ie/229-range-input-filter-widget is all good and ready for review?16:41
mterryltinkl, on your desktopFileActions branch, is there any concern about letting Touch apps define such extra actions and running code that way?17:32
ltinklmterry, right, this is intended more for existing desktop apps (chrome, libreoffice, ...)17:33
mterryltinkl, I know, but there's nothing in this branch restricting it to legacy apps17:33
ltinklmterry, not aware of any touch app that would use this currently17:33
ltinklmterry, it's restricted to the same binary currently (because of the security model)17:33
mterryltinkl, oh I don't see that restricting...17:34
ltinklmterry, but I think apps like the touch webbrowser might want to add a "New tab..." shortcut to its desktop file and handle that within the same instance/window17:35
mterryltinkl, yeah... I'm not worried about legitimate uses.  I'm thinking of malicious apps here17:35
ltinklmterry, in the MP, lines 387-39417:35
mterryltinkl, ah...  it uses app_id there, not the result of exec()17:36
ltinklmterry, I extract the args and pass it to the same binary17:36
ltinklmterry, yeah, it should be safe although the args splitting might be a bit naive17:36
mterryltinkl, my eyes glossed over that and assumed that first argument was the first exec token17:36
ltinklmterry, anyways, the MP could use a review ;)17:37
mterryltinkl, yeah I was about to comment on it, but wanted to talk with higher bandwidth first  :)17:38
ltinklmterry, worth noting I got rid of QSettings there... it was failing badly17:38
mterryltinkl, so now I'm less concerned about malicious apps.  But for a legitimate app, feels odd that I can just put garbage in the first token of the Exec line...17:39
ltinklmterry, yea... our sandboxing wouldn't allow launching another app anyway I bet17:39
mterryltinkl, well this is unity8 launching it.  I could imagine app "malicious.mterry" might try to launch another app like "good.ltinkl" if it was allowed to17:40
ltinklmterry, still I can think of good usecases: file manager, right click on a ZIP file, select "Extract to...", launches some archiver program17:40
mterryltinkl, so sandboxing isn't enough, we definitely should restrict it to same appId17:41
mterryltinkl, like your MP does17:41
ltinklmterry, yea17:41
ltinklmterry, but a I said, I could also think of valid use cases when one might want to launch another app17:41
mterryltinkl, but it just feels weird that "good.ltinkl" if it wants to use these actions has to put in some ignored token on the exec line (presumably its own executable, so maybe it doesn't matter).  But if they put in something else, they don't get any warnings, they just get a different executable than they expected running17:41
ltinklmterry, yeah, perhaps at least a warning would be in place?17:42
mterryltinkl, but that sort of cross-app dependency is not something our app ecosystem is ready for  :)  (unless they are core apps maybe?)17:42
ltinklmterry, dunno really, such cross deps should probably be handled by the content hub17:43
mterryltinkl, yeah.  I was just reiterating that it's good we don't allow launching other appIds right now  :)17:43
mterryltinkl, ok will really review your branch now  :)17:44
mterrymzanetti, regarding QSettings vs gsettings for desktop parsing, I remember you looking into that in the past.  What was the story there? <- ltinkl17:45
ltinklmterry, it just failed to parse anything else than the [Desktop Entry] group for me17:46
mzanettimterry, so, QSettings can do it for simple things. If you need more complicated things, QSettings won't do17:46
ltinklmterry, also omitting some regular translated entries17:46
mterryltinkl, silly QSettings17:46
ltinklah right, I remember17:47
mterrymzanetti, yeah we had converted to gsettings already somewhere in unity8 right?17:47
ltinklit fell over when it encountered17:47
ltinklright to left text17:47
mterryer... not gsettings.  But glib17:47
mzanettimterry, well, QSettings does not implement the .desktop file spec. It implements the .ini file spec, which happens to be a subset of the .desktop one17:47
mterrymzanetti, right17:47
ltinkland QSettings is gone :)17:48
ltinkland doesn't parse anything past this point17:48
mterryltinkl, well looks like it's still used in ./plugins/Unity/Indicators/indicatorsmanager.cpp17:48
mterryltinkl, but that's for actual ini files I think17:49
mterrySo probably ok17:49
ltinkldunno that code17:49
ltinklmterry, qtmir having similar code is something to worry more17:49
mzanettimterry, ah, did the question arise when reviewing lukas' branch?17:49
mterrymzanetti, yeah17:49
mterrymzanetti, I was surprised that we still had QSettings code in there actually17:50
mzanettimterry, it was enough for what the launcher needed so far17:50
mzanettibut not any more...17:50
* mterry is caught up :)17:50
=== dandrader|bbl is now known as dandrader

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