/srv/irclogs.ubuntu.com/2016/05/25/#ubuntu-unity.txt

mikodoI am hearing of much activity happening with development of unity8. Rightly or wrongly, that is what I heard. How can I restate my desire to see this become a priority in unity8  https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/140058003:22
ubot5Launchpad bug 1400580 in unity8 (Ubuntu) " Color Inverse on display. Toggle Negative Image" [Wishlist,Triaged]03:22
mikodoEven hearing that it looks like unity8 session has gone from being a pre-view to being the intended Xserver Ubuntu replacement. Is is time to build in replacement shading for Unity8 to replace X like my bug reads?03:22
duflumikodo: Yeah I implemented that in Mir over a year ago. I would like Unity8 to get access to that implementation too (via unity-system-compositor), rather than waiting indefinitely for Unity8 to reimplement the same feature03:25
mikododuflu, Yes I saw you in the channel and was thinking of you03:26
dufluEssentially we don't even need agreement from Unity8. Just Mir team to agree on some key combo to access system-compositor features (like Ctrl+Alt+Shift+...)03:27
dufluSame goes for video mode switching etc. But it would be better if Unity8 was involved so we could have a GUI to control it03:28
mikodoWell, who should I talk too :)03:28
duflumikodo: Best to comment on the bug: "Hey where's the feature?"03:28
dufluAnd the Unity8 team (at least some of them) will see it03:29
mikodoCool. I was thinking of that, but didn't  want to sound to forward respective of your work03:29
dufluWe can do high contrast and red-shift at the same time03:29
duflumikodo: Speaking as a Canonical employee, I recommend you do beat us up. Ubuntu is yours and you should be hitting us over the head with it to make it better03:30
mikodoThanks!03:31
dufluSometimes change only happens when we get bad publicity. It's better if we change before then03:31
mikodoSomewhere, somehow I have an advocate. Let me think on what you have said.03:32
mikodoOkay. Thanks. camako (Cemil) , Greyback (Gerry), SABDFL (Mark) are going to here about this. I think it is too important for them to not see the stonewalling on this important user accommodation.04:15
duflumikodo: Again, please keep it formal by commenting in the bug05:20
mikododuflu, Okay. No worries.05:24
mikodoThank you. Though not legally blind, I do have vision problems. More importantly I do have a a brother who is,  and has much less vision. I want to think on what I am going to include in the bug as, it is not just we brothers who I am asking this for. That leaves me with some trepidation,  as I will speaking for others needing these features from around the globe. So, I will comment on this in the bug, and take it further then (as needed)  when I am c05:42
mikodoomfortable with my words. I need to think on this a while.05:42
tsdgeosgreyback: yep you need a new laptop/wifi driver :D09:48
greybackyeah :(09:49
* Saviq doesn't trust wifi, still on ethernet wherever possible09:50
pstolowskiSaviq, i thought that was fixed, no? https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-yakkety-ci-train-ppa-service-landing-047/yakkety/i386/u/unity8/20160524_215652@/log.gz11:35
pstolowskiSaviq, or is it Y-specific now?11:35
pstolowskiSaviq, in any case, i think silo 47 is good to go11:36
=== JanC is now known as Guest85398
=== JanC_ is now known as JanC
mterrySaviq, thanks for managing silo 5912:21
mterryI'll help test today, hopefully we can pass to qa12:22
dandradertsdgeos, in https://code.launchpad.net/~aacid/qtmir/compile_with_ubsan/+merge/29567712:51
dandradertsdgeos, what's "ubsan'?12:51
dandradertsdgeos, in the diff I only see you adding a header file12:52
tsdgeosdandrader: sorry12:52
tsdgeosdandrader: undefined behaviour sanitizer12:52
tsdgeos-fsanitize=undefined12:52
tsdgeosdandrader: http://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html12:52
dandradertsdgeos, ok, thanks12:52
tsdgeosdandrader: if you also use https://code.launchpad.net/~aacid/qtmir/ecm_optional_santizers/+merge/295673 you can enable it by using -DECM_ENABLE_SANITIZERS='undefined' if you have ecm installed12:53
tsdgeosdandrader: i'll add it to the description12:54
Saviqmterry, nw, we should ask if there's any more code we wanna add since12:55
Saviqand make sure things are reviewed (https://code.launchpad.net/~saviq/unity8/build-arm64/+merge/295573 hint hint)12:55
mterrySaviq, I was hoping to cut things off and do another silo after with everything new -- neverending cycle of silos.  I figured the silo was already quite large12:56
mterrySaviq, should use qml-module-ubuntu-web instead of qtdec*  :)12:57
mterryotherwise fine12:57
Saviqmterry, I think there's a branch from tsdgeos for that12:57
mterryThough I see you snuck a fullscreen fix in12:57
Saviqdid I?12:57
mterrySaviq, hah, then ltinkl did...12:58
Saviqah for fullscreen notifs, yeah we did12:58
tsdgeosSaviq: mterry: i'd also appreciate if we can land this one asap instead of keeping adding things, the handle_termination_signals branch is needed for the dash performance metrings since otherwise we use random debs that will break at some point12:58
mterrySaviq, yeah but you moved the declaration to a different file, could have fixed.  But fine, we'll let tsdgeos do it  :)12:58
Saviqmterry, ok, cut off is fine with me12:58
tsdgeosSaviq: mterry: yeah but conflcits as hell, so we shelved it for next time12:58
ltinklmterry, I'm innocent :)12:58
Saviqmterry, it needed a rebuild anyway, so that's when I snuck in12:59
mterrySilos need bzr blame12:59
tsdgeoshe je12:59
mterrytsdgeos, when you rebase your branch, make sure to look at build.sh too, it has build-deps there12:59
tsdgeosmterry: can you mark that as needs fixing on the branch?13:00
tsdgeosotherwise i'll forget :D13:00
mterrytsdgeos, oh actually use the process?!  ok  :)13:00
Saviqmterry, "show audit logs"13:01
mterrySaviq, whoa13:01
Saviqmterry, kicked https://unity8-jenkins.ubuntu.com/job/test-ppa-autopkgtest/171/ off13:08
mterrySaviq, oh thanks -- is the entry point for doing so at unity8-jenkins.ubuntu.com?13:08
mterryNever manually started that job13:09
Saviqmterry, yeah https://unity8-jenkins.ubuntu.com/job/test-ppa-autopkgtest/build?delay=0sec simply13:09
Saviqmterry, it's a slightly different job to the test-0-autopkgtest one - not "public" yet, need to clear a thing or two up in there13:10
mterryltinkl, I can't get dragPanelDownToRestoreWindow to work for me -- is there a trick?13:13
mterryltinkl, oh haha!13:14
mterryltinkl, user error, nm13:14
mterryltinkl, hrm.  no actually, my question stands...13:22
mterrytsdgeos, you approved that branch ^ (https://code.launchpad.net/~lukas-kde/unity8/dragPanelDownToRestoreWindow/+merge/290327)  -- is there a trick to get it to work?13:34
tsdgeosmterry: it did work back when i tried it13:35
tsdgeosmterry: you can't drag from the indicators i think13:35
mterrytsdgeos, just drag with a mouse on the non-notification area panel?13:35
tsdgeosyeah drag down with the maximized window13:35
tsdgeosdoesn't work?13:35
mterrydidn't for me13:35
mterrymight be being dumb though13:35
tsdgeoswhich silo, want me to try?13:36
tsdgeosjust in case you're really doing it wrong?13:36
mterrytsdgeos, silo 5913:36
=== dandrader is now known as dandrader|bbl
tsdgeosmterry: it seems it has indeed broken13:55
mterrytsdgeos, oh good (sorta)13:55
mterrypaging ltinkl!13:55
tsdgeoslet me try the branch standaalone again13:55
tsdgeosmterry: something has defenitely "conflicted"14:08
tsdgeosif i go back to current unity8+ltinkl's branch it works again14:08
tsdgeosbtu with silo59 it does not14:08
tsdgeosso even if it doesn't code conflict14:08
tsdgeosthere's a "this doesn't work anymore" conflict14:09
mterrytsdgeos, hmph.  Easy enough to drop from silo, but I'd prefer ltinkl's guess on whether it can be quickly fixed14:09
tsdgeos+114:09
mterrytsdgeos, thanks for testing14:09
ltinklmterry, let me test it in the silo (and without)14:10
ltinklmterry, how quickly you want to hand it over to QA?14:11
mterryltinkl, I'm still testing it manually, but I hoped to do so today14:11
ltinklmterry, I wonder what could be the cause... some botched merge probably14:12
* ltinkl installs the silo14:12
mterryltinkl, merges are the worst  :)14:12
mterryltinkl, well not quite a botched merge -- the branch alone works14:12
mterryltinkl, just some interaction with something else in the silo14:12
ltinklmterry, yeah, let me diff my branch and the silo14:13
ltinklmterry, looks like some other MouseArea in Panel.qml14:15
mterryltinkl, hrm.  Also with the fullscreen notificatino branch, the PUK dialog drops down to reveal the panel when I do vol up/ vol down14:20
ltinklmterry, yeah, that'đ a feature14:20
ltinklmterry, odd I know14:20
mterryhuh14:20
mterryltinkl, why?  the vol dialog is obscuring the panel in that case.  And you can't drag it down...14:21
ltinklmterry, the "confirmation notification" (like volume) get by design inserted unconditionally to the top of the displayed queue14:21
mterryltinkl, that's fine...  I don't mind displaying it on top of the PUK14:21
mterryltinkl, but the PUK gets moved, which seems wrong.  And the vol dialog is placed oddly (overlapping panel)14:22
ltinklmterry, yeah, again - hope this gets solved by having dialogs for those things... we can't display on notification over another, the container is a listview14:23
ltinklone notification *14:23
mterryltinkl, humph.  ok.  I guess it's not worse than before14:23
mterrythanks14:23
ltinklmterry, branch unbroken :) can you pls rebuild?14:32
mterryltinkl, awesome.  I also noticed a test failure that looks related -- test_dragPanelToRestoreMaximizedWindow -- would this fix that then?14:32
ltinklmterry, yeah, this was a proof it was not working :)14:33
mterrypresumably the test failure was capturing the incompatibility and will now pass14:33
mterrycool14:33
ltinklmterry, exactly14:33
ltinklmterry, hmm, this failure https://unity8-jenkins.ubuntu.com/job/test-ppa-autopkgtest/171/label=amd64,package=unity8,release=xenial+overlay,testname=qmluitests.sh/testReport/(root)/ListViewWithPageHeaderTest/testMaximizeVisibleAreaAndShowHeader/ could be caused by this branch https://code.launchpad.net/~aacid/unity8/optimize_LVWPH_layout/+merge/290021 in the new silo14:41
mterryltinkl, tsdgeos: that failure only showed up in xenial, so might be flaky.  Which isn't a good thing either, but still14:42
tsdgeoslet me see14:43
tsdgeosdo we have two lvpwh branches?14:43
tsdgeosif so it may be a problem14:43
tsdgeossince i'm renaming some names in that branch14:43
tsdgeosso the other branch may cleanly merge but still fail14:43
tsdgeosbut otoh didn't fail in vivid14:43
tsdgeosi'll download the code and run the tests here to see if i can repro14:44
tsdgeosis there a way to know if an app was killer by the oom ?14:44
Saviqtsdgeos, syslog14:46
tsdgeosyep, i'm getting oom killed :/14:55
tsdgeoswill talk to sergio tomorrow see if i can split this in smaller runs instead of one big one14:56
=== charles_ is now known as charles
tsdgeosmterry: fwiw i've been running the 4 tests that failed on xenial in a loop for 20 minutes15:21
tsdgeosno failure yet15:21
mterrytsdgeos, oh good thanks.  That's odd that all 4 failed though15:21
mterrybut good to know they aren't completely bogus15:21
mterrytsdgeos, since only one test failed in all 3 releases, I think the others must be flaky.  Maybe more flaky in jenkins than on our machines though...15:22
ltinkltsdgeos, tried adding some "stress"?15:22
tsdgeosyeah15:22
tsdgeosltinkl: well running the 4 of them at the same time already uses like 70% of the cpu15:23
* tsdgeos updates the huge kde l10n svn tree to create some hd noise15:24
ltinkltsdgeos, that's not enough I'm afraid... the flaky wizard test I fixed earlier wasn't failing either untill I added "stress" to the mix15:24
ltinkltsdgeos, "stress --cpu 8 --io 2"15:24
tsdgeosltinkl: i'm just saying "the tests are not wrong per se"15:24
tsdgeosi'm not saying they are good :D15:24
* mterry starts silo 59 official autopkg run, goes to gym15:42
Saviqhrm why does this jenkins job take an hour... when a normal CI run takes 20mins15:51
=== dandrader|bbl is now known as dandrader
robert_ancelljosharenson, hi, regarding bug 1583624, have you dug into Mir as to why it's trying to get the VT?21:37
ubot5bug 1583624 in Light Display Manager "Mir cannot open a tty when started by lightdm" [Undecided,Incomplete] https://launchpad.net/bugs/158362421:37
josharensonrobert_ancell: no further than knowing a mir server needs a VT to run in...22:06
salty-horsenoticed something strange. if my executable is named a.out, the icon it sets on startup isn't respected. if I rename it, I see the icon. any idea why that happens?22:06
robert_ancelljosharenson, when it's running as non-root? Why does it need a VT?22:07
josharensonrobert_ancell: I'd have to ask a mir person :-/22:07
robert_ancellI'll ask RAOF when he's on later22:07
robert_ancelljosharenson, Can I reproduce the issue here? Is this on Yakkety?22:08
josharensonrobert_ancell: I haven't tried it on yakkety yet (just upgrade this morning) but it should be pretty easy to reproduce locally. I can point you to a branch or build you debs..22:09
robert_ancelljosharenson, a branch is good22:09
josharensonrobert_ancell: ok, 1 min. I need to push 1 small change then22:10
robert_ancellno rush22:10
* josharenson is also resyncing w/ trunk as its been a while22:11
josharensonrobert_ancell: https://code.launchpad.net/~josharenson/unity8/session-chooser-gui should do it. Just be mindful of https://bugs.launchpad.net/unity-system-compositor/+bug/1536662 (which looks fixed now)22:14
ubot5Launchpad bug 1536662 in Mir 0.21 "[regression] Black screen: Mir hangs and then crashes on startup/login due to reading from /dev/random" [Critical,Triaged]22:14
* josharenson goes afk for 10 min to walk his dog22:18
josharensonrobert_ancell: I'm going eod, but if you have any questions, ill be back on for a bit in a few hours23:11
robert_ancelljosharenson, thanks23:17

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