/srv/irclogs.ubuntu.com/2015/09/29/#ubuntu-unity.txt

=== JanC_ is now known as JanC
tsdgeosgreyback_: https://code.launchpad.net/~aacid/qtmir/no_double_search/+merge/272707 ?08:46
tsdgeosmzanetti: https://code.launchpad.net/~aacid/unity8/stabilize-launcher-test-more/+merge/27270808:48
mzanettitsdgeos, thanks08:48
mzanettiwould've never found this without being able to repro :/08:48
tsdgeosyeah, repro is nice in these cases08:51
tsdgeoswell mostly always :D08:52
greyback_tsdgeos: nice find, thanks08:53
mzanettitsdgeos, did you intentionally set my branch as target?08:56
tsdgeosmzanetti: i did intentionally set your branch as prerequisite, and failed :D08:58
tsdgeosfixing08:58
mzanettiok :)08:58
tsdgeosmzanetti: now https://code.launchpad.net/~aacid/unity8/stabilize-launcher-test-more/+merge/27270909:01
=== marcusto_ is now known as marcustomlinson
Saviqtsdgeos, looks like your clazy run MP reduced our memory usage significantly enough that Victor could not get webbrowser to get OOM-killed on arale ;)09:39
tsdgeoslol09:40
tsdgeosreally?09:40
mzanettiwat09:40
Saviqhe struggled to get it killed indeed09:40
Saviqand nothing else in the list looks to me like could have any impact https://requests.ci-train.ubuntu.com/#/ticket/41009:40
Saviqmaybe all the QStringLiterals?09:41
mzanettiwell, arale is also not the best example to try to repro that09:41
Saviqthat'd be dumb seeing what QSL is09:41
tsdgeoswell09:42
Saviqmzanetti, yeah, but he went: silo → can't get it killed; no silo → killed; silo → can't get it killed09:42
tsdgeosthe nice thing about it09:42
tsdgeosis not that it saves memory09:42
tsdgeosit's that it doesn't fragment it09:42
tsdgeosso it saves much more memory than what it really saves09:42
tsdgeossince doesn't need to allocate/deallocate it09:42
tsdgeosall the time09:42
Saviqright09:43
mzanettiyeah, I would have expected it to have a bigger impact on perf than on actual usage09:43
mzanettibut glad to hear it also gets us better there09:43
=== marcusto_ is now known as marcustomlinson
pstolowskiSaviq, hey, I got a svg icon for the dash from design (to be used in unity8-dash.desktop), do you have any recommendations where to put it? qml/graphics/applicationIcons?09:54
Saviqpstolowski, data/10:14
Saviqhmm can't access launchpad and other Ubuntu servers...10:19
Saviqanyone else have problems?10:20
pstolowskiSaviq, work here10:21
tsdgeosworks here too10:21
Saviqtx10:21
tsdgeosi have problems accessing the yahoo servers though :D10:21
tsdgeosso maybe the internet is breaking little by little10:22
Saviqmight be10:22
Saviqyeah it's back again for me10:22
tsdgeosand yahoo is back here too :D10:22
tsdgeosweird10:22
mzanettiI am struggling too10:22
greybackme too10:23
mzanettilaunchpad, IRC...10:23
mzanettigoogle stuff seems ok10:23
Saviqltinkl, hey, greyback pointed me at https://launchpad.net/libqtdbustest that could be useful for the dbus mocking (although in the case of logind it's probably easier to just use the template provided by dbusmock)10:38
ltinklSaviq, hi, thanks for the ref, I can have a look if it provides something better10:43
popeykgunn: dpm tells me you had fun with xorg recently - a bad update? (I'm getting xorg exploding randomly on my intel laptop) wondered if there was a bug to follow or some other detail?10:51
* tsdgeos got a xorg update in vivid overlay10:52
tsdgeosnow i'm scared to reboot :D10:52
popeytsdgeos: I'm on wily :)11:12
tsdgeosok11:12
Mirvunity8 requires webbrowser nowadays?11:25
SaviqMirv, we need the web view11:44
SaviqMirv, to display the HERE terms11:45
=== alan_g is now known as alan_g|lunch
MirvSaviq: ok. I just noticed it since a) Oxide just (re)gained dependency on private Qt ABI, needing a rebuild, b) Oxide fails to rebuild against Qt 5.5, c) webbrowser needs oxide d) unity8 needs webbrowser12:11
Saviqsorries :)12:11
Mirvthis after I was finally able to hack UITK far enough to get packages out and I was rejoicing at the possibility of rebuilding webbrowser + unity8 to be able to run Qt 5.5 on the phone for the first time since July :)12:11
Mirvor was it June12:12
MirvSaviq: no problem, life is just complicated12:12
SaviqMirv, did you see mterry's questions yesterday about an upstream LP project for Qt?12:12
Saviqto be able to assign upstream bugs to it12:12
MirvSaviq: yes, I replied to his e-mail about this another patch and said to do whatever he thinks would help. I wasn't aware of anything that could be done aside from coding Qt bug report support into LP12:13
Saviqack12:13
Saviq@unity please merge trunk in your branches, most of the MPs had conflicts with new trunk12:14
mzanettiack12:14
seb128Saviq, I didn't see that, but it's basically bug #157488?12:15
ubot5`bug 157488 in Launchpad itself "Add bugwatch support for the JIRA bugtracker" [High,Triaged] https://launchpad.net/bugs/15748812:15
seb128now that launchpad has some more hackers maybe they can do that? ;-)12:15
=== alan_g|lunch is now known as alan_g
seb128Saviq, which is indicator-bluetooth not working on wily? (just saw that in the unity8 changes)12:32
Saviqseb128, you can't enable it12:35
Saviqseb128, I thought it was a known issue that we've not ported to BlueZ 5 yet...12:35
seb128Saviq, it should work, it got ported to bluez5 and works on unity7, do we have a bug?12:35
seb128Saviq, you though we would release ubuntu desktop without working bluetooth? ;-)12:36
Saviqseb128, not desktop, but wily phone is quite ignored12:36
seb128oh, that commit doesn't state phone12:36
seb128what about unity8 desktop session?12:36
seb128though in theory it should work on the phone, I'm unsure it got tested and about the kernel side though...12:37
Saviqseb128, indeed we didn't pay enough attention to desktop, sorry about that, I just thought this is an expected state at this point in time, let me verify and file a bug12:39
seb128Saviq, thanks12:39
seb128I'm going to update my test machine and try there as well12:39
seb128it should work, but if it doesn't it's good to know and we should fix it ;-)12:39
tsdgeosSaviq: so this landing means the tests passed in that new test-y thing we added right?12:50
tsdgeosautopkg tests i mean12:50
Saviqtsdgeos, at the moment they're "always failed" because of the missing logind mock12:51
tsdgeosoh right12:51
Saviqand armhf looks broken in terms of dependencies, so need to look into that too12:51
Saviqseb128, can't get a unity8 desktop session, just get a black screen and broken vtswitching :/12:53
tsdgeosSaviq: arhmf on CI or when building?12:55
Saviqtsdgeos, autopkgtest12:55
tsdgeosah12:55
Saviqwhich is != than CI, because it installs packages from -proposed, doesn't build the whole thing12:55
tsdgeosright12:56
Saviqseb128, bug #150085512:57
ubot5`bug 1500855 in indicator-bluetooth (Ubuntu) "Bluetooth can't be enabled on devel-proposed phone" [Undecided,New] https://launchpad.net/bugs/150085512:57
Saviqtsdgeos, but we should be getting green CI now12:57
tsdgeoslet's see12:58
tsdgeosi proposed another clazy run with 3 small fixes12:58
tsdgeosshould be green12:58
seb128Saviq, thanks12:58
seb128Saviq, is the desktop session not working an unity8 regression?12:59
Saviqseb128, doubt it (the cursor disappears, too, which suggests unity-system-compositor), will need investigation13:00
SaviqOTOH I can see an ibus and unity8 crash13:00
seb128let me see if it does that here13:00
seb128unity8 segfault doesn't sound too good13:01
* Saviq tries again with a clean /var/crash13:01
Saviqseb128, no unity8 crash this time, but ibus yes, stacktracetop mentions mirclient13:08
Saviqseb128, here's the ibus oops https://errors.ubuntu.com/oops/79161cec-66ab-11e5-8bc8-fa163e22e46713:11
seb128Saviq, thanks13:12
Saviqseb128, quick glance suggests it's similar to bug #143920213:24
ubot5`bug 1439202 in ibus (Ubuntu) "/usr/lib/ibus/ibus-ui-gtk3:11:XKeysymToKeycode:keybinding_manager_bind:panel_keybinding_manager_bind:panel_bind_switch_shortcut:panel_construct" [High,Confirmed] https://launchpad.net/bugs/143920213:24
seb128Saviq, yeah, not likely the unity8 session issue13:25
Saviqseb128, indeed13:25
=== pat_ is now known as Guest41182
tsdgeosdandrader: greyback_: since the clazy fixes for unity8 seem to have been so good i've done one for qtubuntu too https://code.launchpad.net/~aacid/qtubuntu/clazy_fixes/+merge/27274713:37
dandraderok13:38
Saviqltinkl, dandrader's mousePointer branch already bumps application api to 9, you might wanna rebase on that13:59
ltinklSaviq, yikes, noted; I feared that :)14:00
=== davidcalle_ is now known as davidcalle
* ltinkl on it14:03
seb128Saviq, on my wily test laptop unity8 desktop session works and indicator-bluetooth loads and shows devices and has working items14:04
Saviqseb128, glad to hear14:08
greyback_tsdgeos: thank you14:13
biothey, seb128 sounds familiar14:19
biotseb128: I think you reclassified some bug I looked at, gvfs breaking magnet links14:20
Saviqdandrader, did you merge qtmir as well?14:21
dandraderSaviq, qtmir/mousePointer?14:21
Saviqdandrader, yeah14:21
Saviqdebian/control has a conflict there too14:21
dandraderSaviq, not today14:21
Saviqplease do14:21
dandraderSaviq, done14:23
Saviqtx14:23
* guest42315 ubuntuonair in 20 min http://ubuntuonair.com/14:38
seb128biot, yes, I did14:38
=== dandrader is now known as dandrader|afk
tsdgeosouch15:15
tsdgeoswe had a function in qmenumodel not returning anything15:15
tsdgeoswhen it should be returning a bool15:15
tsdgeosdednick: https://code.launchpad.net/~aacid/qmenumodel/clazy_run/+merge/27278815:20
mterrygreyback, heyo -- we talked about X-Ubuntu-Touch the other day -- assuming that's the road we go down, is this MP roughly acceptable?  https://code.launchpad.net/~mterry/qtmir/no-touch-no-lifecycle/+merge/27279115:25
mterrytsdgeos, how is that not a compile error?  :)  stupid lenient compile modes15:26
greybackmterry: kinda. First thing I see if you're implementing policy in qtmir - I'd rather you export the "isTouchApp" as a bool to the shell, so it can decide what to do15:26
tsdgeosmterry: yaeh :/15:26
greybackmterry: there is no second thing :)15:27
mterrygreyback, it looked like that policy decision was already in qtmir?15:27
dednicktsdgeos: looks fine. suppose i should test it... :)15:27
mterrygreyback, but I can do the round trip way -- that's something that mzanetti needs eventually anyway to show a different close button15:27
dednicktsdgeos: what's with the reserve?15:28
mterrygreyback, if I'm doing that though, I feel like we ought to pull the whitelist out of qtmir15:28
greybackmterry: well it is the shell which decides if an app should be suspended or not.15:28
dednicktsdgeos: just dynamic alloc?15:28
greybackmterry: yep, that I agree with15:28
tsdgeosdednick: so makes sure the list is the szie we need from start instead of growing as it "fills" and needing to realloc and stuff15:29
tsdgeosdednick: it's minor optimization15:29
mterrygreyback, right -- that's why I stuck it in qtmir.  Because it was doing the policy via whitelist already.  But sure, I'll move all out15:29
tsdgeossince we know the size it doesn't harm to reserve15:29
greybackmterry: cool, thanks15:29
dednicktsdgeos: mkay15:29
dednickfigured15:30
greybackdednick: "replaceme" in a debian .symbols file does what I think it does?15:31
dednickgreyback: i think the ci bot comes along and changes it when released15:32
greybackcool15:32
dednickgreyback: looked at history and seems to be what happens :|15:32
greybackI didn't know that15:32
dednickgreyback: i've requested a review from ricmm, who should know the details15:33
greybackack. he's a busy bunny tho15:34
greybackdednick: this Holder thingy is weird15:34
dednickgreyback: yeah. doesn't do anything that i can see15:34
greybackwas it supposed to be a QScopedPointer/unique_ptr15:35
greybackbut it's got no deleter afaics15:35
dednickgreyback: the other sensors are all handled by android registration keeping shared pointers. so they aren't destroyed till the sensor controller is. the haptic is a special case.15:36
tsdgeoslarsu: yeah it's ugly, but saves lots of allocs15:36
tsdgeosthat's why it's cool having a tool that does the replacement for you15:36
tsdgeoswe can code and then just run the tool :D15:36
greybackdednick: I see15:36
dednickgreyback: i'm thinking it probably should be shared, but it isn't here.15:36
larsutsdgeos: ya I just read about it, pretty neat15:36
dednickdont see a point of 2 things asking for it creating 2 dbus connections.15:37
greybackdednick: to be absolutely correct, maybe yeah. Let's see what a papi maintainer says15:37
dednickI'm hoping every button doesnt do that :)15:38
dednicki think the Qt sensor may be shared15:38
greybackyeah, but qtsensors does that itself15:38
greybackIMO papi should be the one dedupicating such resources15:39
greyback+l15:39
=== dandrader|afk is now known as dandrader
dandraderdednick, about your polit-close branch: ApplicationManager test is still failing15:56
dednickdandrader: hm weird. give me a minute15:56
dednickdandrader: passes for me :/15:57
dandraderdednick, and item 2 here still applies https://code.launchpad.net/~nick-dedekind/qtmir/polite-close/+merge/262188/comments/68301115:58
dednickdandrader: ok. fixing in a minute16:01
dandradergreyback,  could you check if you get the test failure I describe here? https://code.launchpad.net/~nick-dedekind/qtmir/polite-close/+merge/262188/comments/68301116:01
dednickdandrader: you on mir 0.1616:03
dednick?16:03
dandraderdednick, yes16:04
dandraderdednick, but it's a test, shoulnd't make a difference16:05
dednickwell i just upgraded things have just gone shitty16:05
dednickApplication test wont run either.16:06
dednickdandrader: ^16:06
dandraderdednick, great. so you're getting the same test failure now16:07
dednickdandrader: no. Application, not ApplicationManager16:07
dednickdandrader: ApplicationTests.checkResumeAcquiresWakeLock16:08
dandraderdednick, that one passes here :)16:08
dednickthose wake lock tests have always been flaky for me16:09
dednickdandrader: tests passed on jenkins wily a couple of days ago.16:12
dednickdandrader: i'll kick off another run16:12
dednickdandrader: deleted all my build files and rebuilt. all tests pass again16:15
dandraderSaviq, could you check if you get the same test failure I describe here? https://code.launchpad.net/~nick-dedekind/qtmir/polite-close/+merge/262188/comments/68301116:35
dandraderSaviq, I'm getting it but dednick is not16:36
=== alan_g is now known as alan_g|EOD
Saviqdandrader, wouldn't ci show a test failure?16:58
dandraderSaviq, right, and it's passing there. damn it17:02
Saviqdandrader, in any case, all passed here as well17:02
SaviqI'm on wily, mind you17:03
dandraderme too17:07
dandraderI do out of source builds. But I would be really surpised if that's the cause17:07
=== dandrader is now known as dandrader|afk
mterrymzanetti, I assume you're not around, but if you are, I updated my no-touch-no-lifecycle branch to have an isTouchApp bool you can consume in unity818:29
mterryhttps://code.launchpad.net/~mterry/unity-api/no-touch-no-lifecycle/+merge/27282918:29
mterryhttps://code.launchpad.net/~mterry/qtmir/no-touch-no-lifecycle/+merge/27279118:29
mterrymzanetti, I'm working on the unity8 side, but it may already be useful to you18:29
mzanettimterry, cool! thanks. not working on it any more today, but will check it out tomorrow morning18:30
mterrymzanetti, good answer  :)18:30
=== dandrader|afk is now known as dandrader
dandradergreyback, still around?20:01
greybackdandrader: for brief questions, yes20:02
dandradergreyback, why not having polite-close as a pre-requisite of https://code.launchpad.net/~gerboland/qtmir/dont-delete-qml-cache-on-good-stop/+merge/272761 ?20:02
dandradergreyback, as you said you took a bunch of code from it20:02
greybackdandrader: I will tomorrow. I didn't realise polite-close was going to be approved so fast20:02
greybackrebase should be easy20:03
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
mzanettijosharenson, https://plus.google.com/105839534016416729197/posts/BSnrzs65r7i20:50
mzanettithere's some feedback in the comments20:50
josharensonmzanetti: ah good to know... I'll merge trunk into the ppa branch and try it on wily (just finished upgrading to wily about 10 min ago)20:51
=== Cimi_ is now known as cimi

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