mterrytedg, for tuesday: Icon=/usr/share/unity8/Dash/unity8-dash.png results in iconPath() of /usr/share/icons/usr/share/unity8/Dash/unity8-dash.png00:47
mterrytedg, also for tomorrow: line 189 of desktop-exec.c: should be FALSE instead of 100:53
faenilSaviq: mzanetti "unity8 can't run apps" bug fixed!09:51
faenilturns out libpam-cgm was not installed!09:52
faenilHow do we proceed? who should install it? should it be a dependency of unity8-desktop-session-mir? or should it be seeded in XeniaL?09:52
tsdgeosfaenil: do we have a bug? can you comment on it with the solution?10:09
tsdgeosso we don't forget i mean10:12
tsdgeosmzanetti is probably jet lagged and Saviq is in the US so didn't wake up yet10:12
faeniltsdgeos: I comment it10:12
ubot5Launchpad bug 1535058 in upstart (Ubuntu) "applications close instantly when launched from the launcher or dash" [Undecided,New]10:13
mzanettiI'm here... xchat crashed on me :/11:47
mzanettifaenil, hmm... seems it should be a dependency of who needs it, no?11:49
mzanettiupstart I guess11:49
faenilmzanetti: yeah, I don't know who needs it :) probably upstart, yeah...should I ping someone?11:49
faenilafter a week, it was just a missing package :D well, good to know it's nothing werse11:50
davmor2mzanetti: change over to hexchat, same interface just better and supported :)12:01
tsdgeosmzanetti: since you're one of the few people with tablets can you do https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1517830 ?12:03
ubot5Launchpad bug 1517830 in unity8 (Ubuntu) "In landscape mode the music listing jumps when touched" [High,In progress]12:03
tsdgeosi mean review https://code.launchpad.net/~aacid/unity8/playButtonFocus/+merge/28299012:03
mzanettidavmor2, notifications not working with hexchat12:34
mzanettiwell, not that gnome-xchat would be 100% reliable with that... but hexchat doesn't alert at all when you get a PM12:35
davmor2mzanetti: did you install the plugin12:35
mzanettitsdgeos, ack12:35
davmor2mzanetti: hexchat-indicator then in hexchat set the alerts you want by default I think they are all off12:37
mzanettidavmor2, ok... lets see how that goes12:51
davmor2mzanetti: \o\ \o/ /o/12:51
=== alan_g is now known as alan_g|lunch
mzanettipstolowski, hey, intentional that the reviews are in the first column?13:08
mzanettiwould've expected them in the second13:08
pstolowskimzanetti, yes, me too, but patricia insisted on 1st. 2nd should only have tabular data13:08
mzanettipstolowski, want me to send you some screenshots?13:09
pstolowskimzanetti, yeah, that would be nice, thanks!13:09
mzanettioh noes13:09
mzanettithey're still black here :D13:09
* mzanetti takes pics with the camera13:10
pstolowskimzanetti, ok, note i left the austin sprint group13:10
mzanettiyeah, I saw13:10
Saviqmzanetti, faenil, I'd venture it's ubuntu-app-launch13:22
Saviqah but I can see people have already sorted it out13:22
mzanettipstolowski, have a video in your telegram. there is an issue with installing apps13:24
pstolowskimzanetti, oh, ok, thanks for catching13:27
mzanettipstolowski, uninstalling seems ok13:27
pstolowskimzanetti, ok. prolly overlooked something..13:27
mzanettipstolowski, let me know if you need logs. but it's reproducible reliably13:28
pstolowskimzanetti, okay, let me look at the code13:28
mterrymzanetti, what would cause an app to be anchored to the top of the screen (behind the indicators)?13:37
mzanettimterry, covered by the panel?13:37
mterrymzanetti, yeah.  But not the launcher13:38
mzanettimterry, the topMostApp.fullscreen property not reaching the panel.fullscreen property13:38
mterrymzanetti, 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 it13:38
pstolowskimzanetti, ok, spotted the bug, fixing13:38
mzanettimterry, 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.qml13:39
=== alan_g|lunch is now known as alan_g
mterrymzanetti, hrm.  OK.  Could supportedOrientations screw with sizing like that too?13:40
mzanettimterry, unlikely... it was working in both orientations... but sure... depending on what your branch changes...13:41
mzanettimterry, broken with all stages?13:42
mterrymzanetti, good question, haven't tried anything else yet13:42
mterrymzanetti, I don't have unity8 changes though so I'm figuring it's just something qtmir is exporting somehow13:43
mzanettimterry, 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 case13:44
mterrymzanetti, in this case, the fullscreen property shouldn't be affected by my code.  So that might not be it13:44
mterrymzanetti, I'll keep digging13:45
mterrymzanetti, but thanks for some pointers13:45
faenilSaviq: yeah I just need to know who should pull that in, but maybe Serge already knows13:47
faenil(once he wakes up)13:47
faeniloh I think he won't be online today13:48
mterrytedg, also can you bump the debian/changelog version in your branch, so I can depend on the bumped value?14:07
mterrybregma, 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 besides14:11
bregmamterry, sweet, I'll check it out14:12
tsdgeosSaviq: any reason you did not top approve https://code.launchpad.net/~aacid/unity8/fix_shell_tests_greeter_ddas/+merge/282626 ?14:15
Saviqtsdgeos, prolly forgot14:16
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/tryCompareFunctionInsteadOfWhile/+merge/283012 too?14:16
Saviqbeen going through them all and probably waiting for Britney results on https://requests.ci-train.ubuntu.com/#/ticket/87714:17
Saviqtsdgeos, we seem to be looking good on amd64 and i386, armhf has some 50+ failures still, but that's fine14:18
Saviqwe'll get there14:18
Saviqalso https://unity8-jenkins.ubuntu.com/view/unity8/job/test-0-autopkgtest/93/ is happy so :)14:19
tedgmterry: Sorry, rebooted, forgot to turn on IRC this morning :-)14:46
tedgmterry: Sure14:46
mterrytedg, :)14:47
mterrytedg, I sent a couple notes last night too, did you get those?14:47
tedgmterry: Yes, just now though. Will do.14:47
mterrygreyback_, 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:50
greyback_mterry: is it actually drawing under the panel, or drawing too tall & being clipped? (set panel opacity 0.5 to check)14:51
mterrygreyback_, it's drawing under the panel (just checked)14:56
mterrygreyback_, 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 way14:56
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 surface14:59
mterryhumph.  I don't know how I could have changed that. But will inspect what qtmir is reporting for fullscreen14:59
greyback_nor do I. unity8 log reporting any breakages?15:01
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 know15:06
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/createQmlObjectDifferentFile/+merge/283528 would be nice15:08
tsdgeosand https://code.launchpad.net/~aacid/unity8/playButtonFocus/+merge/282990 should improve the music scope on tablets too15:09
tsdgeosboth should be relatively low risk15:10
tsdgeosSaviq: ↑↑15:10
tsdgeosjust need people to review them :D15:10
Saviqtsdgeos, I'll do those two15:11
Saviqtsdgeos, looks like there's new API in 5.5 for object creation btw? http://doc.qt.io/qt-5/qml-qtqml-component.html#createObject-method15:13
Saviqoh hmm15:13
Saviqhttp://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 that15:14
tsdgeosSaviq: yes we get a component not an object directly15:14
Saviqtsdgeos, QTBUG for the filepath, btw?15:15
tsdgeosSaviq: i didn't create one, i'll dig up a testcase and report it15:15
Saviqtsdgeos, tx15:16
tsdgeosSaviq: do you have a minute to confirm the createQmlObject testcase i have also fails on your phone too?16:10
Saviqtsdgeos, hit me16:14
tsdgeosSaviq: http://paste.ubuntu.com/14672603/16:14
tsdgeosqmlscene main.qml --desktop_file_hint=/usr/share/applications/unity8-dash.desktop16:15
Saviqtsdgeos, COMPONENT116:15
tsdgeosSaviq: read the code, what should it say?16:15
Saviqtsdgeos, COMPONENT216:16
tsdgeosonly happens on the phone though16:16
tsdgeoswhich is confusing16:16
tsdgeostried my vividchroot on the desktop16:16
tsdgeosalso get COMPNENT116:16
tsdgeosgoing to try xenial on the phone16:16
Saviqtsdgeos, is the .arg() important, could simplify without it?16:17
tsdgeosSaviq: it's not, i had it without but this way seemed more easy to prove it was the same code just changing on thing16:17
tsdgeosdo you think it hinders the readability?16:17
Saviqtsdgeos, meh, probably fine, if that had impact on anything would be a problem in itself, too16:18
tsdgeosSaviq: http://paste.ubuntu.com/14672623/ is the other option16:18
Saviqtsdgeos, yeah, maybe better with arg16:19
mterrytsdgeos, you are deep into Qt right?  Can you look at bug 1537782 real quick?  Particularly my latest comment16:37
ubot5bug 1537782 in Canonical System Image "Modifier ignored when pressing a key if TextInput has active focus" [High,Triaged] https://launchpad.net/bugs/153778216:37
tsdgeosmterry: "deep" for some definition of "deep" :D16:39
mterrytsdgeos, from the kiddy pool, it all looks deep  :)16:39
Saviqdandrader, 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 merge16:40
tsdgeosmterry: seems relatively reasonable yes16:43
mterrytsdgeos, 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
dandraderSaviq, ok16:44
mterrytsdgeos, I guess we shouldn't check for NoModifier, we want to allow shift...16:44
tsdgeosthe code is from 2011 though, weird noone complained before16:44
mterrytsdgeos, do you know if TextInput is used in other contexts, ones where they expect the modifiers to be ignored like that?16:45
tsdgeosmterry: can't think of any, no16:45
mterrytsdgeos, 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 be16:46
tsdgeosSaviq: funny on xenial+phone i get COMPONENT1 half of the times and COMPONENT2 the other half16:46
* mterry takes lunch16:47
tsdgeosmterry: patch better than bug, it has 99% more chances of being looked at16:47
Saviqmzanetti, can you base windowed-by-kbd-only off of lp:~dandrader/unity8/dontRotateDesktop-lp1515977 please16:47
tsdgeosmterry: 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 guess16:48
tsdgeosbut yeah16:48
dandraderSaviq, done16:58
mzanettiSaviq, ack17:01
Saviqdandrader, 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/console17:02
Saviq/food, biab17:02
tsdgeosSaviq: https://bugreports.qt.io/browse/QTBUG-5067717:04
dandraderSaviq, is that before or after my qtmir/removeUselessClass update?17:04
dednickdandrader: it is that branch17:06
dandraderSaviq, hmmm, seems post my update. but criss-cross by itself doesn't cause merge failures17:06
mzanettisometimes it does :)17:06
dandraderit's just a warning17:06
dandraderthe problem is that this piece of !$!$!$ is still complaining about the merge conflict I just solved *again*17:07
mzanettiSaviq, resubmitted to https://code.launchpad.net/~mzanetti/unity8/windowed-by-kbd-only/+merge/28401117:07
dednickhaving a day like mine!17:07
dednickalthough stuck in a hotel in austin...17:08
dednickthey have better bbq17:08
* dandrader tries the same steps locally17:09
mterrytsdgeos, yeah fair (about low risk)17:09
* tsdgeos eods17:10
dandraderhad my fair share of bbq for the month. went to a vegan texmex yesterday :)17:11
dandraderSaviq, ok, now it must merge smoothly17:22
=== boiko__ is now known as boiko
syehHi, 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?18:20
faenilSaviq: can you take https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1535058 ?19:07
ubot5Launchpad bug 1535058 in upstart (Ubuntu) "applications close instantly when launched from the launcher or dash" [Undecided,New]19:07
faenilas in, Serge wants info I can't provide19:07
faenilmaybe you can help19:07
faenilit's about cgroups19:07
Saviqfaenil, tedg, so should upstart itself pull in cgm, or should ual?19:10
faenilfrom what Serge says, it seems I should not even need cgroups on a laptop19:11
faenilhe said currently it's a Recommend on lxc19:11
tedgSaviq: seems like it should be unity8-desktop seed?19:11
tedgOr metapackage or whatever that is.19:11
Saviqtedg, shouldn't it be whatever puts cgroups in the upstart job config?19:12
Saviqdumping everything in the metapkg is meh19:12
tedgSaviq: So I guess the UAL, but it seems like an odd dep there.19:12
Saviqtedg, well, if it it doesn't work without them...19:13
Saviqwhich seems to be the case19:13
tedgSure, but the dependency would be on libpam-cgm, which is a long chain of deps.19:13
tedgBut that's fine. I don't care really.19:13
faenilSerge also said he will replace libpam-cgm with  ibpam-cgfs soon19:16
faenil(just fyi)19:16
tedgSaviq: So installing that package will work for user testing? Can I put the fix through with other UAL changes?19:18
Saviqtedg, not sure what you mean by "work for user testing", it's not critical, though, as it only affects unity8-desktop-session-mir it seems19:19
tedgSaviq: My understanding was that not working was blocking some user testing in design.19:20
Saviqtedg, well, yeah, but I imagine they can work off of a silo19:21
faeniltedg: I installed the package manually :P19:42
faenila silo to install a package is wasted effort :D19:43
faenilI discovered that the reason was the missing package only 10h ago, then I installed the pkg and got apps running19:44
faenilbut still, good to fix the actually bug in the cleanest possible way19:44
tedgfaenil: You see, Saviq is a manager now, he has people install packages for him now.19:44
faeniltedg: lol19:44
faenilso that's what happened!19:44
* faenil dinner, bbl19:44
mterrymzanetti, how do I link an upstream qt bug?  the "qt" project doesn't recognize https://bugreports.qt.io/browse/QTBUG-XXXX20:11
mterrySaviq, 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&xenial20:19
=== dandrader is now known as dandrader|afk
mterrySaviq, 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 xenial22:12
mterryMirv, ^ 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:22
mterrytedg, 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:38
tedgAh, no, he got eaten by the ether!22:44
tedgNo, it shouldn't work though. The dash is started by upstart.22:44
=== popey_ is now known as popey

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