[00:47] <mterry> tedg, for tuesday: Icon=/usr/share/unity8/Dash/unity8-dash.png results in iconPath() of /usr/share/icons/usr/share/unity8/Dash/unity8-dash.png
[00:53] <mterry> tedg, also for tomorrow: line 189 of desktop-exec.c: should be FALSE instead of 1
[09:51] <faenil> Saviq: mzanetti "unity8 can't run apps" bug fixed!
[09:52] <faenil> turns out libpam-cgm was not installed!
[09:52] <faenil> How do we proceed? who should install it? should it be a dependency of unity8-desktop-session-mir? or should it be seeded in XeniaL?
[10:09] <tsdgeos> faenil: do we have a bug? can you comment on it with the solution?
[10:12] <tsdgeos> so we don't forget i mean
[10:12] <tsdgeos> mzanetti is probably jet lagged and Saviq is in the US so didn't wake up yet
[10:12] <faenil> tsdgeos: I comment it
[10:12] <faenil> commented*
[10:12] <tsdgeos> oki
[10:13] <faenil> https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1535058
[10:13] <ubot5> Launchpad bug 1535058 in upstart (Ubuntu) "applications close instantly when launched from the launcher or dash" [Undecided,New]
[11:47] <mzanetti> I'm here... xchat crashed on me :/
[11:49] <mzanetti> faenil, hmm... seems it should be a dependency of who needs it, no?
[11:49] <mzanetti> upstart I guess
[11:49] <faenil> mzanetti: yeah, I don't know who needs it :) probably upstart, yeah...should I ping someone?
[11:50] <faenil> after a week, it was just a missing package :D well, good to know it's nothing werse
[11:50] <faenil> worse*
[12:01] <davmor2> mzanetti: change over to hexchat, same interface just better and supported :)
[12:03] <tsdgeos> mzanetti: since you're one of the few people with tablets can you do https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1517830 ?
[12:03] <ubot5> Launchpad bug 1517830 in unity8 (Ubuntu) "In landscape mode the music listing jumps when touched" [High,In progress]
[12:03] <tsdgeos> i mean review https://code.launchpad.net/~aacid/unity8/playButtonFocus/+merge/282990
[12:34] <mzanetti> davmor2, notifications not working with hexchat
[12:35] <mzanetti> well, not that gnome-xchat would be 100% reliable with that... but hexchat doesn't alert at all when you get a PM
[12:35] <davmor2> mzanetti: did you install the plugin
[12:35] <mzanetti> plugin?
[12:35] <mzanetti> no
[12:35] <mzanetti> tsdgeos, ack
[12:37] <davmor2> mzanetti: hexchat-indicator then in hexchat set the alerts you want by default I think they are all off
[12:51] <mzanetti> davmor2, ok... lets see how that goes
[12:51] <davmor2> mzanetti: \o\ \o/ /o/
=== alan_g is now known as alan_g|lunch
[13:08] <mzanetti> pstolowski, hey, intentional that the reviews are in the first column?
[13:08] <mzanetti> would've expected them in the second
[13:08] <pstolowski> mzanetti, yes, me too, but patricia insisted on 1st. 2nd should only have tabular data
[13:09] <mzanetti> ok
[13:09] <mzanetti> pstolowski, want me to send you some screenshots?
[13:09] <pstolowski> mzanetti, yeah, that would be nice, thanks!
[13:09] <mzanetti> kk
[13:09] <mzanetti> telegram
[13:09] <mzanetti> oh noes
[13:09] <mzanetti> they're still black here :D
[13:10]  * mzanetti takes pics with the camera
[13:10] <pstolowski> mzanetti, ok, note i left the austin sprint group
[13:10] <mzanetti> yeah, I saw
[13:22] <Saviq> mzanetti, faenil, I'd venture it's ubuntu-app-launch
[13:22] <Saviq> ah but I can see people have already sorted it out
[13:24] <mzanetti> pstolowski, have a video in your telegram. there is an issue with installing apps
[13:27] <pstolowski> mzanetti, oh, ok, thanks for catching
[13:27] <mzanetti> pstolowski, uninstalling seems ok
[13:27] <pstolowski> mzanetti, ok. prolly overlooked something..
[13:28] <mzanetti> pstolowski, let me know if you need logs. but it's reproducible reliably
[13:28] <pstolowski> mzanetti, okay, let me look at the code
[13:37] <mterry> mzanetti, what would cause an app to be anchored to the top of the screen (behind the indicators)?
[13:37] <mzanetti> mterry, covered by the panel?
[13:38] <mterry> mzanetti, yeah.  But not the launcher
[13:38] <mzanetti> mterry, the topMostApp.fullscreen property not reaching the panel.fullscreen property
[13:38] <mterry> mzanetti, I'm testing a qtmir branch that uses Ubuntu App Launch for various bits of app metadata and I'm seeing this behavior.  Wondered how I got it
[13:38] <pstolowski> mzanetti, ok, spotted the bug, fixing
[13:39] <mzanetti> mterry, so the stage exports a "topMostApp" or so. each app has a "fullscreen" property. that needs to be wired up to panel.fullscreen in Shell.qml
=== alan_g|lunch is now known as alan_g
[13:40] <mterry> mzanetti, hrm.  OK.  Could supportedOrientations screw with sizing like that too?
[13:41] <mzanetti> mterry, unlikely... it was working in both orientations... but sure... depending on what your branch changes...
[13:42] <mzanetti> mterry, broken with all stages?
[13:42] <mterry> mzanetti, good question, haven't tried anything else yet
[13:43] <mterry> mzanetti, I don't have unity8 changes though so I'm figuring it's just something qtmir is exporting somehow
[13:44] <mzanetti> mterry, well, it could also be that qtmir doesn't properly export the fullscreen property any more... although I figure the stage would not make it fullscreen either in that case
[13:44] <mterry> mzanetti, in this case, the fullscreen property shouldn't be affected by my code.  So that might not be it
[13:45] <mterry> mzanetti, I'll keep digging
[13:45] <mterry> mzanetti, but thanks for some pointers
[13:47] <faenil> Saviq: yeah I just need to know who should pull that in, but maybe Serge already knows
[13:47] <faenil> (once he wakes up)
[13:48] <faenil> oh I think he won't be online today
[14:07] <mterry> tedg, also can you bump the debian/changelog version in your branch, so I can depend on the bumped value?
[14:11] <mterry> bregma, I believe my lp:~mterry/qtmir/use-ual-info branch should mostly work for your use case.  Any oddities are possibly now due to info that u-a-l itself exposes.  There is an odd bug with app content appearing underneath the top panel.  Working on that, but it's usable besides
[14:12] <bregma> mterry, sweet, I'll check it out
[14:15] <tsdgeos> Saviq: any reason you did not top approve https://code.launchpad.net/~aacid/unity8/fix_shell_tests_greeter_ddas/+merge/282626 ?
[14:16] <Saviq> tsdgeos, prolly forgot
[14:16] <tsdgeos> Saviq: https://code.launchpad.net/~aacid/unity8/tryCompareFunctionInsteadOfWhile/+merge/283012 too?
[14:16] <Saviq> yup
[14:17] <Saviq> been going through them all and probably waiting for Britney results on https://requests.ci-train.ubuntu.com/#/ticket/877
[14:18] <Saviq> tsdgeos, we seem to be looking good on amd64 and i386, armhf has some 50+ failures still, but that's fine
[14:18] <Saviq> we'll get there
[14:19] <Saviq> also https://unity8-jenkins.ubuntu.com/view/unity8/job/test-0-autopkgtest/93/ is happy so :)
[14:20] <tsdgeos> :)
[14:46] <tedg> mterry: Sorry, rebooted, forgot to turn on IRC this morning :-)
[14:46] <tedg> mterry: Sure
[14:47] <mterry> tedg, :)
[14:47] <mterry> tedg, I sent a couple notes last night too, did you get those?
[14:47] <tedg> mterry: Yes, just now though. Will do.
[14:50] <mterry> greyback_, I somehow have changed qtmir such that apps render below the top panel.  Do you know what bits I may have managed to toggle to do that?
[14:51] <greyback_> mterry: is it actually drawing under the panel, or drawing too tall & being clipped? (set panel opacity 0.5 to check)
[14:56] <mterry> greyback_, it's drawing under the panel (just checked)
[14:56] <mterry> greyback_, I made some changes to qtmir to pull app info from UAL instead of desktop files.  But I didn't expect I could break things this way
[14:59] <greyback_> mterry: sounds like fullscreen property broke somehow. it is unity8 which decides the size of the surface, and should only size one fullscreen, if the app asked for a fullscreen surface
[14:59] <mterry> humph.  I don't know how I could have changed that. But will inspect what qtmir is reporting for fullscreen
[15:01] <greyback_> nor do I. unity8 log reporting any breakages?
[15:06] <Saviq> @unity I'm prepping OTA9.5 silo for unity8+friends, if there's something beyond this list https://launchpad.net/canonical-devices-system-image/+milestone/ww04-2016 and not in this req https://requests.ci-train.ubuntu.com/#/ticket/877 that you think should be in there, please let me know
[15:08] <tsdgeos> Saviq: https://code.launchpad.net/~aacid/unity8/createQmlObjectDifferentFile/+merge/283528 would be nice
[15:09] <Saviq> ack
[15:09] <tsdgeos> and https://code.launchpad.net/~aacid/unity8/playButtonFocus/+merge/282990 should improve the music scope on tablets too
[15:10] <tsdgeos> both should be relatively low risk
[15:10] <tsdgeos> Saviq: ↑↑
[15:10] <Saviq> ack
[15:10] <tsdgeos> just need people to review them :D
[15:11] <Saviq> tsdgeos, I'll do those two
[15:11] <tsdgeos> k
[15:13] <Saviq> tsdgeos, looks like there's new API in 5.5 for object creation btw? http://doc.qt.io/qt-5/qml-qtqml-component.html#createObject-method
[15:13] <Saviq> oh hmm
[15:14] <Saviq> http://doc.qt.io/qt-5/qml-qtqml-qt.html#createQmlObject-method is there, too, we're creating the object to get a component out of it or something?
[15:14]  * Saviq recalls doing something weird like that
[15:14] <tsdgeos> Saviq: yes we get a component not an object directly
[15:15] <Saviq> tsdgeos, QTBUG for the filepath, btw?
[15:15] <tsdgeos> Saviq: i didn't create one, i'll dig up a testcase and report it
[15:16] <Saviq> tsdgeos, tx
[16:10] <tsdgeos> Saviq: do you have a minute to confirm the createQmlObject testcase i have also fails on your phone too?
[16:14] <Saviq> tsdgeos, hit me
[16:14] <tsdgeos> Saviq: http://paste.ubuntu.com/14672603/
[16:15] <tsdgeos> qmlscene main.qml --desktop_file_hint=/usr/share/applications/unity8-dash.desktop
[16:15] <Saviq> tsdgeos, COMPONENT1
[16:15] <tsdgeos> Saviq: read the code, what should it say?
[16:16] <Saviq> tsdgeos, COMPONENT2
[16:16] <tsdgeos> correct
[16:16] <tsdgeos> only happens on the phone though
[16:16] <tsdgeos> which is confusing
[16:16] <tsdgeos> tried my vividchroot on the desktop
[16:16] <tsdgeos> also get COMPNENT1
[16:16] <tsdgeos> going to try xenial on the phone
[16:17] <Saviq> tsdgeos, is the .arg() important, could simplify without it?
[16:17] <tsdgeos> Saviq: it's not, i had it without but this way seemed more easy to prove it was the same code just changing on thing
[16:17] <tsdgeos> do you think it hinders the readability?
[16:18] <Saviq> tsdgeos, meh, probably fine, if that had impact on anything would be a problem in itself, too
[16:18] <tsdgeos> Saviq: http://paste.ubuntu.com/14672623/ is the other option
[16:19] <Saviq> tsdgeos, yeah, maybe better with arg
[16:37] <mterry> tsdgeos, you are deep into Qt right?  Can you look at bug 1537782 real quick?  Particularly my latest comment
[16:37] <ubot5> bug 1537782 in Canonical System Image "Modifier ignored when pressing a key if TextInput has active focus" [High,Triaged] https://launchpad.net/bugs/1537782
[16:39] <tsdgeos> mterry: "deep" for some definition of "deep" :D
[16:39] <mterry> tsdgeos, from the kiddy pool, it all looks deep  :)
[16:40] <Saviq> dandrader, dednick, https://ci-train.ubuntu.com/job/ubuntu-landing-019-1-build/16/console looks like either useless class or test refactor got conflicts, can you please merge
[16:43] <tsdgeos> mterry: seems relatively reasonable yes
[16:44] <mterry> tsdgeos, I can't think of a case where you're holding down alt or ctrl and want a key press to hit the text input ...
[16:44] <dandrader> Saviq, ok
[16:44] <mterry> tsdgeos, I guess we shouldn't check for NoModifier, we want to allow shift...
[16:44] <tsdgeos> the code is from 2011 though, weird noone complained before
[16:45] <mterry> tsdgeos, do you know if TextInput is used in other contexts, ones where they expect the modifiers to be ignored like that?
[16:45] <tsdgeos> mterry: can't think of any, no
[16:46] <mterry> tsdgeos, I'm thinking my sequence should be: file an upstream bug/patch and see what they say.  But this bug is targetted for ota 9.5.  I wonder how risky cowboying the patch could be
[16:46] <tsdgeos> Saviq: funny on xenial+phone i get COMPONENT1 half of the times and COMPONENT2 the other half
[16:47]  * mterry takes lunch
[16:47] <tsdgeos> mterry: patch better than bug, it has 99% more chances of being looked at
[16:47] <Saviq> mzanetti, can you base windowed-by-kbd-only off of lp:~dandrader/unity8/dontRotateDesktop-lp1515977 please
[16:48] <tsdgeos> mterry: i'd say that the risk is "low" since we don't "care" much for keyboard on the phone so if it broke it wouldnt' be terrible i guess
[16:48] <tsdgeos> but yeah
[16:58] <dandrader> Saviq, done
[17:01] <mzanetti> Saviq, ack
[17:02] <Saviq> dandrader, dednick, ugh, it's criss-cross, you might need to replay some of the history :/ https://ci-train.ubuntu.com/job/ubuntu-landing-019-1-build/18/console
[17:02] <Saviq> /food, biab
[17:04] <tsdgeos> Saviq: https://bugreports.qt.io/browse/QTBUG-50677
[17:04] <dandrader> Saviq, is that before or after my qtmir/removeUselessClass update?
[17:06] <dednick> dandrader: it is that branch
[17:06] <dandrader> Saviq, hmmm, seems post my update. but criss-cross by itself doesn't cause merge failures
[17:06] <mzanetti> sometimes it does :)
[17:06] <dandrader> it's just a warning
[17:07] <dandrader> the problem is that this piece of !$!$!$ is still complaining about the merge conflict I just solved *again*
[17:07] <dednick> lol
[17:07] <mzanetti> Saviq, resubmitted to https://code.launchpad.net/~mzanetti/unity8/windowed-by-kbd-only/+merge/284011
[17:07] <dednick> having a day like mine!
[17:08] <dednick> although stuck in a hotel in austin...
[17:08] <dednick> they have better bbq
[17:09]  * dandrader tries the same steps locally
[17:09] <mterry> tsdgeos, yeah fair (about low risk)
[17:10]  * tsdgeos eods
[17:11] <dandrader> had my fair share of bbq for the month. went to a vegan texmex yesterday :)
[17:22] <dandrader> Saviq, ok, now it must merge smoothly
=== boiko__ is now known as boiko
[18:20] <syeh> Hi, I'm debugging an issue where Ubuntu 12.04 doesn't resize the background image properly.  Does any one know how Unity sets the background image, e.g with XsetRoot?
[19:07] <faenil> Saviq: can you take https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1535058 ?
[19:07] <ubot5> Launchpad bug 1535058 in upstart (Ubuntu) "applications close instantly when launched from the launcher or dash" [Undecided,New]
[19:07] <faenil> as in, Serge wants info I can't provide
[19:07] <faenil> maybe you can help
[19:07] <faenil> it's about cgroups
[19:10] <Saviq> faenil, tedg, so should upstart itself pull in cgm, or should ual?
[19:11] <faenil> from what Serge says, it seems I should not even need cgroups on a laptop
[19:11] <faenil> he said currently it's a Recommend on lxc
[19:11] <tedg> Saviq: seems like it should be unity8-desktop seed?
[19:11] <tedg> Or metapackage or whatever that is.
[19:12] <Saviq> tedg, shouldn't it be whatever puts cgroups in the upstart job config?
[19:12] <Saviq> dumping everything in the metapkg is meh
[19:12] <tedg> Saviq: So I guess the UAL, but it seems like an odd dep there.
[19:13] <Saviq> tedg, well, if it it doesn't work without them...
[19:13] <Saviq> which seems to be the case
[19:13] <tedg> Sure, but the dependency would be on libpam-cgm, which is a long chain of deps.
[19:13] <tedg> But that's fine. I don't care really.
[19:16] <faenil> Serge also said he will replace libpam-cgm with  ibpam-cgfs soon
[19:16] <faenil> (just fyi)
[19:18] <tedg> Saviq: So installing that package will work for user testing? Can I put the fix through with other UAL changes?
[19:19] <Saviq> tedg, not sure what you mean by "work for user testing", it's not critical, though, as it only affects unity8-desktop-session-mir it seems
[19:20] <tedg> Saviq: My understanding was that not working was blocking some user testing in design.
[19:21] <Saviq> tedg, well, yeah, but I imagine they can work off of a silo
[19:42] <faenil> tedg: I installed the package manually :P
[19:43] <faenil> a silo to install a package is wasted effort :D
[19:44] <faenil> I discovered that the reason was the missing package only 10h ago, then I installed the pkg and got apps running
[19:44] <faenil> but still, good to fix the actually bug in the cleanest possible way
[19:44] <tedg> faenil: You see, Saviq is a manager now, he has people install packages for him now.
[19:44] <faenil> tedg: lol
[19:44] <faenil> so that's what happened!
[19:44]  * faenil dinner, bbl
[20:11] <mterry> mzanetti, how do I link an upstream qt bug?  the "qt" project doesn't recognize https://bugreports.qt.io/browse/QTBUG-XXXX
[20:19] <mterry> Saviq, so I've got a patch for that TextInput bug -- but it's on qtdeclarative.  Once I've finished testing on phone, do you want to try to squeeze it into the 9.5 silo?  I can upload to that silo directly for vivid&xenial
=== dandrader is now known as dandrader|afk
[22:12] <mterry> Saviq, OK.  After testing and assuming that you want to squeeze it in (since you set the milestone), I've uploaded qtdeclarative(-gles) to the landing-019 PPA for vivid and xenial
[22:22] <mterry> Mirv, ^ I have a patch for qtdeclarative in landing-019.  I did the -gles dance, so I think I did it right.  But do you have any conflicting updates in flight?
[22:38] <mterry> tedg, btw "ubuntu-app-launch unity8-dash" doesn't work because it sets NoDisplay.  But I feel like it probably should still launch in that case?
[22:44] <tedg> Ah, no, he got eaten by the ether!
[22:44] <tedg> No, it shouldn't work though. The dash is started by upstart.
=== popey_ is now known as popey