[00:21] scratch that, it seems that stopping unity8 while any application is active seems to throw a spanner in the works === Guest99228 is now known as amithkk [07:05] veebers, yeah, sometimes crash, sometimes hang? [07:05] Saviq, good morning [07:05] tvoss_, yo [07:06] Saviq, hey, as you recently looked into it: do you know how to enable gdb pretty printers? [07:06] tvoss_, nope [07:08] Saviq: yeah, I added code tp that MR to handle it (make sure OSK is gone when stopping unity8, and then start it again when needed) [07:08] veebers, k thanks [07:08] Saviq: nw, do you know if anyone is working on a fix for: https://bugs.launchpad.net/mir/+bug/1238417 ? [07:08] Ubuntu bug 1238417 in Mir "Unity does not process events from evdev device created before unity is restarted (autopilot tests)" [Critical,Confirmed] [07:10] Saviq: also fyi you'll see that MR is 'needs review' [07:13] veebers, I don't think anyone is, since we have a workaround... [07:15] veebers, the clean fix would be to also fix https://bugs.launchpad.net/mir/+bug/1237784 [07:15] Ubuntu bug 1237784 in Mir "Please move input detection to libudev" [Low,Triaged] [07:16] veebers, right now, mir uses the android input stack as is, which uses inotify to get notified about device node creation [07:16] veebers, while I haven't verified myself, I'm pretty sure that switching to libudev would also fix the autopilot event injection issue [07:17] veebers, are you still "at work" to fix some issues with that MP or shall we take over? [07:17] Saviq: heh, no I'm just about to pop out to pick my partner up from work [07:18] veebers, k o/ [07:18] Saviq: ack re: the bug I mentioned. I will probabl be around a little bit later on [07:18] Saviq: also interested, what are the issues that you mentioned? [07:18] veebers, just s/unity/unity8/ in the "you need to stop unity8" [07:19] veebers, but am also wondering if we should stop it ourselves... [07:21] Saviq: right, both thomi and myself had that discussion too, I think it's safer in the long run to have the environment owner (the user sort that out) [07:21] veebers, may very well be [07:25] Good morning all [07:26] veebers, while still here, any reason to not use _patch_envirionment for QT_LOAD_TESTABILITY? [07:26] davidcalle o/ [07:30] Saviq: oh um, that's still there from the transition from having a "are we using upstart to start unity", also it's guaranteed to be cleaned up in _cleanup_launching_upstart_unity [07:30] veebers, right, will fix [07:30] veebers, if someone set it before - we shouldn't be resetting it [07:30] veebers, fyi, we need -g in cleanup, too [07:34] Saviq: hey! I guess we have still a yummi unity8 crash with latest image, mind having a look? [07:34] http://jenkins.qa.ubuntu.com/job/saucy-touch_mir-mako-smoke-unity8-autopilot/21/artifact/clientlogs/_usr_bin_unity8.32011.crash/*view*/ [07:34] didrocks, smokes truncate .crash files [07:35] didrocks, I won't get anything out of it [07:35] Saviq: let's hope not, we need to get stable at some point :) [07:35] didrocks, they do [07:35] didrocks, a unity8 crash needs to be some 12MB at least [07:35] argh [07:35] didrocks, either way [07:35] didrocks, we have fixes in queue for a few crashers [07:35] any idea why they are truncated on this arch? [07:35] ok, any ETA? [07:36] didrocks, today [07:36] (to know if we can handle + get the results today) [07:36] didrocks, utah doesn't wait for them to be completed, AFAICT [07:36] Saviq: if you can get them merged before 10 UTC (so in 2h30), we can hope getting the results [07:36] greyback, ↑ the "waitforme" branch [07:36] Saviq: ok, as soon as I have a crash on my phone, I'll fw it to you [07:36] didrocks, please apport-cli View, Keep it first [07:37] Saviq: yep, I know about it for years ;) [07:37] didrocks, sorry :) [07:37] no worry :p [07:37] didrocks, btw, smoke will also cough up only one .crash file per project [07:37] yeah, that's how apport works… [07:38] didrocks, we should collect them and move away from /crash/ after every test [07:38] agreed [07:38] well, first, they have to fix the "wait for apport to finish" [07:38] +1 [08:38] Saviq, I don't suppose you ever got an answer to this question you asked? https://bugs.launchpad.net/unity8/+bug/1150151 [08:38] Ubuntu bug 1150151 in Unity 8 "[SIDESTAGE] GU threshold for enabling sidestage should exclude 7" tablets" [Medium,Triaged] [08:38] nic-doffay, right indeed [08:38] nic-doffay, can you poke Oren about it/ [08:42] bug 1239575 [08:42] bug 1239575 in unity8 (Ubuntu) "left alignment of Music/Video tile' text looks bad" [Low,New] https://launchpad.net/bugs/1239575 [08:42] Saviq, yep np. [08:42] Cimi, ^ do you know if that was intentional ? [08:59] om26er, intentional [09:01] Cimi, looks bad doesn't it ? [09:01] om26er, nope [09:01] om26er, https://lh6.googleusercontent.com/DiFH5zJxUPhXES9dICX4ydlmQzpkKFMNeJlq3NsPhPIUAzbAH8XjX4YlZ0eii1w5csbAX2ZqnW9l6bjLSDzzd8Y1oCJf2qKzJYZ49lDeg5dPDbCiCEI-3eyi8Fs [09:01] om26er, put covers [09:02] mzanetti, can I ask you to https://code.launchpad.net/~unity-team/unity8/ap_launch_unity_with_upstart/+merge/190886 [09:02] Saviq: ofc [09:02] Cimi, doesn't look good to. In the current implementation there is still one bug. the text is overflowing a bit left [09:02] *to me [09:03] om26er, it's prob because you're missing the cover [09:03] it's grid aligned... [09:05] cover is supposed to appear inside the tile. so that should have no effect on the text alignment. [09:07] Cimi, om26er, maybe we just need a margin on the left [09:07] om26er, I think there's another bug in that screenshot [09:07] like 0.5 GU [09:08] Saviq, 1du maybe [09:08] or 2 [09:08] 0.5 is way too much [09:08] om26er, Linkin Park [09:08] Cimi, lol. But I love them [09:08] Saviq: is data/unity8.conf supposed to be our upstart service file? [09:08] Cimi, remember that UShape "softens" the edges [09:08] mzanetti, yes [09:09] the later albums were not that good.. but still [09:09] mzanetti, I need to fix, though [09:11] Saviq, tsdgeos, we want to disable the category reordering, it seems to be causing more problems than it solves, comments? [09:12] mhr3, yeah I think [09:13] pstolowski, will you mp a revert branch? [09:13] doesn't have to be revert... [09:16] mhr3, yes, i'm on it. it don't want to revert, just disable signal etc. [09:16] s/it/I/ [09:16] yea, fine with me [09:18] mhr3, Saviq, tsdgeos https://bugs.launchpad.net/unity8/+bug/1239584 [09:18] Ubuntu bug 1239584 in Unity 8 "Category reordering in Home should be disabled as it causes problems in UI" [Critical,Confirmed] [09:19] well [09:19] we ought to fix those empty categories :D [09:21] pstolowski, osk misbehaving I believe is unrelated [09:21] pstolowski, mhr3, tsdgeos so if we can fix the empty cats, that'd obviously be better [09:22] Saviq, it surely would, but do you have someone who could look at it? [09:22] mhr3, yeah, tsdgeos :) [09:23] mhr3, should've asked first, thought it was past hope ;) [09:23] :) [09:23] Saviq: well, i haven't had a look at it [09:23] not sure how do i repro tbh [09:23] pstolowski: just "search"? [09:23] mhr3, besides empty cats, there were some other glitches as well right? [09:24] yea, the "shifting" of the override model [09:24] having apps category twice there [09:24] lots and lots of issue really [09:25] tsdgeos, while i do believe you're awesome, i'm not sure if you'll be able to tackle all of that in the next couple of hours where we still can do fixes [09:26] Saviq, that's why i suggested the revert instead ^ [09:26] mhr3: sure i'm not saying we should not rever it [09:26] i'm just saying hiding them under the rug is not future proof [09:26] since i don't even have a rug around here [09:26] :D [09:26] lol [09:26] sure, i just want 13.10 to be usable :) [09:27] and you know... without being able to see bugs after two searches [09:27] should take at least dozen :P [09:29] Cimi, you're working on the music carousel overlay? [09:29] Cimi, and video carousel mount, for that matter? [09:29] -mount, layout, too [09:30] Saviq, today yes [09:30] Cimi, ok [09:30] Saviq, as well with quitting termination mode [09:30] Cimi, start with the carousels please [09:30] ok [09:31] Saviq, is this really for me? https://bugs.launchpad.net/unity8/+bug/1226221 [09:31] Ubuntu bug 1226221 in Unity 8 ""Recent searches" are not persistent and are not common between scopes" [Undecided,Triaged] [09:32] iirc might be easy to store a property across scopes [09:32] but how about the zeitgeist part? [09:35] didn't we already have a property across scopes for that? [09:37] Saviq, are ap tests meant to work as usual if executed on the device? [09:37] MacSlow, kind of, yeah [09:37] MacSlow, you need to make sure display is on [09:37] MacSlow, easiest is to see that it's on, sudo stop powerd; stop unity8 [09:38] MacSlow, from then on they should run ~fine, yes [09:38] MacSlow, but we are fixing things around that atm [09:47] Saviq, http://pastebin.ubuntu.com/6235207 [09:48] Saviq: do you know what changed in the scope backend? Apparently I'm now getting a model that doesn't support get() [09:48] which breaks the previews [09:49] MacSlow, maguro? [09:49] MacSlow, try stopping maliit-server [09:49] Saviq, yes the GX [09:49] GN [09:49] MacSlow, and make sure no app is running [09:49] MacSlow, if all else fails - reboot [09:50] reboot [09:54] tsdgeos, Saviq so ok to disable reordering? [09:54] pstolowski: sure [09:54] pstolowski, tsdgeos yeah [09:55] Saviq: i'm still having a look at the crashers on sigterm, my patch fixes the crash once everything has properly stated, but if we get a sigterm "while starting", we still crash a bit [09:55] tsdgeos, yeah btw you know you opened a pandora's box? ;) [09:56] why? [09:56] tsdgeos, now that we're destroying everything correctly [09:56] tsdgeos, we uncovered a bunch of bugs :) [09:56] tsdgeos, 3 different crashes in notidy [09:56] like? [09:57] :D [09:57] i'm sorry :D [09:58] tsdgeos, on exit, though - so it's not really bad [09:58] tsdgeos, the ones at the bottom in https://bugs.launchpad.net/unity-notifications/ [09:59] tsdgeos, only other one I found was https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1239409 [09:59] Ubuntu bug 1239409 in unity8 (Ubuntu) "unity8 crashed with SIGSEGV in __libc_do_syscall()" [Undecided,New] [09:59] Saviq: so that's unity8 crashing or notify-osd crashing? [09:59] tsdgeos, there is no notify-ods [09:59] ah [09:59] tsdgeos, it's unity8 in unity-notifications [10:00] i see [10:01] Saviq, I can't stop/kill unity8 on the device... [10:01] MacSlow, kill -9 in case [10:04] Saviq, ok... brute force seems to have worked... now let's hope the ap-test work [10:04] Cimi, https://code.launchpad.net/~om26er/unity8/add_leftMargin_tile_text/+merge/190905 [10:05] MacSlow: you need to stop unity8 from the phablet user, not from root [10:06] om26er, 0.2 gu is not processed iirc [10:06] Cimi, its working [10:06] problem with 0.2 [10:06] is that it may be a non integer in some places [10:07] which will make people unhappy [10:07] 0.2 gu i mean :D [10:08] tsdgeos, ok... but I wanted to be sure I strike with all possible might ;) [10:09] om26er, let's use 1du or 2 [10:10] om26er, and remove your code from the running tile :) [10:10] Cimi, yes. working on that [10:13] mzanetti, pushed a fix to the upstart branch... [10:13] mzanetti, will build packages for testing here [10:13] Saviq, got any new bugs I can look at while some branches of mine are reviewed? [10:13] Saviq: ok... had to reflash my device anyways. [10:14] nic-doffay, how's multi-selectors? [10:14] nic-doffay, waiting for review? [10:17] Saviq, yeah [10:18] Cimi, what provides du ? api reference only have gu and dp. [10:18] is that from qml? [10:19] nic-doffay, take https://bugs.launchpad.net/unity8/+bug/1127627 from dednick [10:19] Ubuntu bug 1127627 in Unity 8 "[Shell] Search icon in menu bar should be removed when focus is not dash" [High,Triaged] [10:19] om26er, du [10:19] om26er, was my typo :) [10:19] sorry [10:19] dp [10:19] hah [10:19] monday morning [10:20] \o/ [10:20] om26er, 1 du is 1 pixel [10:20] dp [10:20] dammit [10:20] DP [10:21] so 1 or 2 max [10:21] maybe 1 id fine [10:21] *is [10:21] Cimi, yeah, 2 looks fine here. [10:21] will look bad on desktop [10:21] nic-doffay, then, bug #1226221 [10:21] bug 1226221 in Unity 8 ""Recent searches" are not persistent and are not common between scopes" [Undecided,Triaged] https://launchpad.net/bugs/1226221 [10:24] on https://code.launchpad.net/~om26er/unity8/fix_1238837/+merge/190743 [10:24] there's a mouse area inside an item [10:25] nevermind, API helped [10:26] Cimi, so this branch is good to go now ? :) [10:26] om26er, think so [10:26] om26er, I wrote the same code friday :) [10:26] om26er, didn't realise you were doing it too [10:27] Cimi, oops, I came here to ping you to make sure you were not already working but couldn't catch you. I guess you were offline already [10:27] om26er, actually not [10:27] om26er, (I already fixed it then I went EOD) [10:27] om26er, btw [10:28] om26er, you need to write a new test or extend the current one [10:28] om26er, because you're no longer testing app termination now [10:28] Cimi, if your branch have a test, feel free to propose yours [10:28] om26er, you should now check that clicking the icon closes the app [10:28] Cimi, right, I made a comment about that https://code.launchpad.net/~om26er/unity8/fix_1238837/+merge/190743/comments/438142 [10:29] om26er, so yes, exactly [10:29] what?=¿ [10:29] ../../../../../src/modules/Unity/Application/ubuntukeyboardinfo.cpp: In member function ‘void UbuntuKeyboardInfo::tryConnectingToServer()’: [10:29] ../../../../../src/modules/Unity/Application/ubuntukeyboardinfo.cpp:61:32: error: ‘class QString’ has no member named ‘isEmtpy’ [10:29] :D [10:29] ah [10:30] om26er, in RunningApplicationTile add 'objectName: "closeIcon"' [10:30] a q_assert [10:30] om26er, then you find it in the test [10:30] Cimi, how do I run these tests? to me it says Unity.Test is not installed [10:30] om26er, ./build [10:30] …. [10:31] om26er, cd builddir [10:31] om26er, make testRunningApplicationsGrid [10:31] Cimi, are you planning on dealing with this soon or can I take it? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1226221 [10:31] Ubuntu bug 1226221 in Unity 8 ""Recent searches" are not persistent and are not common between scopes" [Undecided,Triaged] [10:31] nic-doffay, if you are doing nothing, take it [10:32] nic-doffay, should work with a shared property across scopes [10:33] nic-doffay, or if you can use zeitgeist or something else [10:33] nic-doffay, you can push and pull recent searches [10:33] nic-doffay, maybe mhr3 can help you [10:34] om26er, you want to do it or I can do it if you want [10:35] Cimi, I am working on it (trying) [10:35] mhr3, mind giving me a shout about this? [10:36] nic-doffay, you'd need to store it somewhere, but we don't have zg on the device, and it won't be for 13.10 [10:39] dbus? :D [10:39] gsettings [10:39] I meant [10:41] Saviq, you mentioned this could possibly be a regression ^ [10:42] nic-doffay, the fact that they're not common between scopes is a regression that should be relatively easy tacklable [10:42] nic-doffay, the fact that it's not persistent - not so much - we should probably store it in U1Db for syncing [10:44] Saviq, can this be done now? [10:45] nic-doffay, "this"? [10:45] nic-doffay, U1Db? http://developer.ubuntu.com/api/devel/ubuntu-13.10/qml/u1db-qt5/overview.html [10:47] Saviq, just wondering about your approach in comparison to mhr3 's [10:51] nic-doffay, "my" approach is just saying that it needs to be stored somewhere, i'm fine with somewhere being u1db, plus if it's supposed to be shared between all scopes, it can be purely qml patch afaict [10:52] mhr3, right then. [10:53] mhr3, where are the recent searches currently? [10:53] nic-doffay, read, please, read [10:53] nic-doffay, in qml :) [10:57] mhr3, Saviq I meant what source files. [10:58] nic-doffay, in qml source files [10:58] nic-doffay, you've been around it for the past month [10:59] nic-doffay, PageHeader === MacSlow is now known as MacSlow|lunch [11:04] om26er, update? [11:04] Cimi, Just pushed a test [11:04] Cimi, https://code.launchpad.net/~om26er/unity8/fix_1238837/+merge/190743 [11:06] om26er, remove empty line [11:06] before bracket [11:06] seems fine here [11:06] bravo [11:06] om26er, rename the other test [11:07] test_clickTile maybe [11:08] toTerminateApp doesn't make sense anymore since we drop the functionality [11:08] Cimi, test_clickTileNotClose sounds fine ? [11:09] since we are making sure clicking it does not close. the name of the test should also indicate that [11:09] om26er, sounds better [11:09] om26er, yeah yours is better [11:09] Saviq, is it possible to create a map for ListElement? [11:10] mhr3, .get() should work [11:10] not what i meant, a sec [11:11] Saviq, something like http://paste.ubuntu.com/6235442/ [11:11] but something that actually works :) [11:11] mhr3, ah no [11:12] :/ [11:12] mhr3, a list, yes - an object, no [11:18] Saviq, u1db? [11:19] Saviq, that way will sync across devices [11:19] Cimi, yes, that's the plan [11:19] Saviq, I would not do this [11:19] Saviq, searches depends on local content [11:19] Cimi, that's why I put ubuntu-ux on the bug [11:20] Saviq, it's not like searching google where it makes sense to share your search history [11:20] Cimi, anyway, storing in U1Db does not automagically equal syncing [11:20] Saviq, I know but it's not required [11:20] Cimi, and dconf isn't the right place to store it [11:20] Saviq, I'd go for something local [11:20] Cimi, U1Db *is* local [11:20] until you sync it === hikiko is now known as hikiko|lunch [11:35] mhr3, Saviq https://code.launchpad.net/~unity-team/unity8/disable-reordering/+merge/190918 [11:36] pstolowski, FIXME++ ? [11:36] k [11:37] eh, acked already [11:37] switched back [11:38] mhr3, good we don't have autolanding ;) [11:38] oh? [11:38] good to know [11:49] Saviq: https://code.launchpad.net/~mzanetti/unity8/fix-lazyimage-initialSize/+merge/190930 [11:50] mzanetti, thanks [11:51] tsdgeos: hi, we have a report of the proposed HUD update breaking the unity8 ap tests, I want to check I'm running the tests correctly myself [11:51] pete-woods: hi [11:51] sure [11:51] whatcha doing [11:51] ? [11:52] tsdgeos: on the device I'm stopping unity8 [11:52] then running autopilot run unity8.shell.tests.test_hud.TestHud.test_show_hud_appears [11:53] you need to be insde tests/autopilot [11:53] or tests/whatever [11:53] for that to work [11:53] tsdgeos: that's from the installed unity8-autopilot package [11:53] ah [11:53] ok [11:53] then no, you don't need to be anywhere === tvoss_ is now known as tvoss|dentist [11:53] ok, and it's not working? [11:53] the HUD gui isn't appearing [11:54] it gets to the point of showing the funny symbol in the middle of the screen [11:54] but then waits there until timeout [11:54] that is weird [11:54] i tried them here last week and it worked fine [11:55] pete-woods: what's the autopilot backtrace? [11:55] /away [11:55] tsdgeos: do you mean this part? http://pastebin.ubuntu.com/6235587/ [11:56] yeah that [11:56] have to run for lunch sorry, will try to see what's wrong later [11:56] is that ok? [11:56] jamesh, hello [11:57] tsdgeos: this bug is classified as super-urget, but yeah, you may as well get some lunch [11:57] tsdgeos: some further info, this test fails for me regardless of HUD version, so I don't think that it's the backend at fault [11:58] Saviq, I go and merge omen's branch? reviewed [11:59] *omer [12:02] * Cimi pushed === MacSlow|lunch is now known as MacSlow === _salem is now known as salem_ === dednick is now known as dednick|lunch === hikiko|lunch is now known as hikiko === jhodapp|afk is now known as jhodapp === alan_g is now known as alan_g|lunch [12:53] pete-woods: back [12:53] pete-woods: ok, let me try to see if i can repro [12:54] tsdgeos: I'm suspicious that this test might just not work under Mir? [12:54] pete-woods: does it pass on the pc but not on the device? [12:54] pete-woods: it works fine in mir, i tried [12:54] a few days ago i mean [12:54] not now [12:54] okay [12:57] pete-woods: works for me :_S [12:57] http://paste.ubuntu.com/6235761/ [12:57] tsdgeos: how are you setting up the test? are you stopping unity8 first? [12:58] what am I doing differently [12:58] sure [12:58] stop unity8 [12:58] Saviq: do you have some packages for your branch? [12:58] and then run then run the autopilot line you said [12:58] Saviq: I can't install the ones from jenkins for some reason [12:58] pete-woods: nothing i see different === tvoss|dentist is now known as tvoss [13:02] mzanetti, try this lp:~mhr3/unity8/route-overridden-previews [13:02] i'm not completely sure it works :) [13:03] mhr3: building [13:03] mhr3: hmm... just merging it into the switching-previews branch doesn't really fix it [13:04] mhr3: well, there is now an empty image [13:04] mhr3: before there wasn't even that one [13:04] mzanetti, that's better, no? :) [13:04] its one step closer, yes [13:04] not good enough tho :P [13:04] well it theoretically should work on the phone [13:04] ah ok [13:04] lemme try [13:04] you don't have click scope nor the apps on the desktop [13:05] mhr3: hmm... how come I have the suggested apps section here? [13:05] it's from software center, not clicks [13:05] ah wait... [13:05] tsdgeos: have you got time to try running those tests again with the proposed version of HUD? [13:06] mhr3: well, in the apps scope I do see the preview for the camera-app. it doesn't work with the camera-app in home [13:06] :/ [13:06] pete-woods: what do i need to do? install a few hud packages? or? === alan_g|lunch is now known as alan_g [13:08] tsdgeos: it's the newest hud build from the proposed PPA [13:08] you should only need to install the hud package itself, as nothing else has changed [13:09] mzanetti, yeah, it conflicts with the ubuntu-touch-session [13:10] Saviq: already removed that one... still some strange issue [13:10] mzanetti, like? === dednick|lunch is now known as dednick [13:11]