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:47 |
---|---|---|
mterry | tedg, also for tomorrow: line 189 of desktop-exec.c: should be FALSE instead of 1 | 00:53 |
faenil | Saviq: mzanetti "unity8 can't run apps" bug fixed! | 09:51 |
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? | 09:52 |
tsdgeos | faenil: do we have a bug? can you comment on it with the solution? | 10:09 |
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:12 |
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] | 10:13 |
mzanetti | I'm here... xchat crashed on me :/ | 11:47 |
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:49 |
faenil | after a week, it was just a missing package :D well, good to know it's nothing werse | 11:50 |
faenil | worse* | 11:50 |
davmor2 | mzanetti: change over to hexchat, same interface just better and supported :) | 12:01 |
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:03 |
mzanetti | davmor2, notifications not working with hexchat | 12:34 |
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:35 |
davmor2 | mzanetti: hexchat-indicator then in hexchat set the alerts you want by default I think they are all off | 12:37 |
mzanetti | davmor2, ok... lets see how that goes | 12:51 |
davmor2 | mzanetti: \o\ \o/ /o/ | 12:51 |
=== alan_g is now known as alan_g|lunch | ||
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:08 |
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:09 |
* 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:10 |
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:22 |
mzanetti | pstolowski, have a video in your telegram. there is an issue with installing apps | 13:24 |
pstolowski | mzanetti, oh, ok, thanks for catching | 13:27 |
mzanetti | pstolowski, uninstalling seems ok | 13:27 |
pstolowski | mzanetti, ok. prolly overlooked something.. | 13:27 |
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:28 |
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:37 |
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:38 |
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 | 13:39 |
=== alan_g|lunch is now known as alan_g | ||
mterry | mzanetti, hrm. OK. Could supportedOrientations screw with sizing like that too? | 13:40 |
mzanetti | mterry, unlikely... it was working in both orientations... but sure... depending on what your branch changes... | 13:41 |
mzanetti | mterry, broken with all stages? | 13:42 |
mterry | mzanetti, good question, haven't tried anything else yet | 13:42 |
mterry | mzanetti, I don't have unity8 changes though so I'm figuring it's just something qtmir is exporting somehow | 13:43 |
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:44 |
mterry | mzanetti, I'll keep digging | 13:45 |
mterry | mzanetti, but thanks for some pointers | 13:45 |
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:47 |
faenil | oh I think he won't be online today | 13:48 |
mterry | tedg, also can you bump the debian/changelog version in your branch, so I can depend on the bumped value? | 14:07 |
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:11 |
bregma | mterry, sweet, I'll check it out | 14:12 |
tsdgeos | Saviq: any reason you did not top approve https://code.launchpad.net/~aacid/unity8/fix_shell_tests_greeter_ddas/+merge/282626 ? | 14:15 |
Saviq | tsdgeos, prolly forgot | 14:16 |
tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/tryCompareFunctionInsteadOfWhile/+merge/283012 too? | 14:16 |
Saviq | yup | 14:16 |
Saviq | been going through them all and probably waiting for Britney results on https://requests.ci-train.ubuntu.com/#/ticket/877 | 14:17 |
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:18 |
Saviq | also https://unity8-jenkins.ubuntu.com/view/unity8/job/test-0-autopkgtest/93/ is happy so :) | 14:19 |
tsdgeos | :) | 14:20 |
tedg | mterry: Sorry, rebooted, forgot to turn on IRC this morning :-) | 14:46 |
tedg | mterry: Sure | 14:46 |
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:47 |
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: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 |
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: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 surface | 14:59 |
mterry | humph. I don't know how I could have changed that. But will inspect what qtmir is reporting for fullscreen | 14: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 know | 15:06 |
tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/createQmlObjectDifferentFile/+merge/283528 would be nice | 15:08 |
Saviq | ack | 15:09 |
tsdgeos | and https://code.launchpad.net/~aacid/unity8/playButtonFocus/+merge/282990 should improve the music scope on tablets too | 15:09 |
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:10 |
Saviq | tsdgeos, I'll do those two | 15:11 |
tsdgeos | k | 15:11 |
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:13 |
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:14 |
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:15 |
Saviq | tsdgeos, tx | 15:16 |
tsdgeos | Saviq: do you have a minute to confirm the createQmlObject testcase i have also fails on your phone too? | 16:10 |
Saviq | tsdgeos, hit me | 16:14 |
tsdgeos | Saviq: http://paste.ubuntu.com/14672603/ | 16:14 |
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:15 |
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:16 |
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:17 |
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:18 |
Saviq | tsdgeos, yeah, maybe better with arg | 16:19 |
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:37 |
tsdgeos | mterry: "deep" for some definition of "deep" :D | 16:39 |
mterry | tsdgeos, from the kiddy pool, it all looks deep :) | 16:39 |
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:40 |
tsdgeos | mterry: seems relatively reasonable yes | 16:43 |
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:44 |
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:45 |
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:46 |
* 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:47 |
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:48 |
dandrader | Saviq, done | 16:58 |
mzanetti | Saviq, ack | 17:01 |
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:02 |
tsdgeos | Saviq: https://bugreports.qt.io/browse/QTBUG-50677 | 17:04 |
dandrader | Saviq, is that before or after my qtmir/removeUselessClass update? | 17:04 |
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:06 |
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:07 |
dednick | although stuck in a hotel in austin... | 17:08 |
dednick | they have better bbq | 17:08 |
* dandrader tries the same steps locally | 17:09 | |
mterry | tsdgeos, yeah fair (about low risk) | 17:09 |
* tsdgeos eods | 17:10 | |
dandrader | had my fair share of bbq for the month. went to a vegan texmex yesterday :) | 17:11 |
dandrader | Saviq, ok, now it must merge smoothly | 17:22 |
=== boiko__ is now known as boiko | ||
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? | 18:20 |
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:07 |
Saviq | faenil, tedg, so should upstart itself pull in cgm, or should ual? | 19:10 |
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:11 |
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:12 |
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:13 |
faenil | Serge also said he will replace libpam-cgm with ibpam-cgfs soon | 19:16 |
faenil | (just fyi) | 19:16 |
tedg | Saviq: So installing that package will work for user testing? Can I put the fix through with other UAL changes? | 19:18 |
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:19 |
tedg | Saviq: My understanding was that not working was blocking some user testing in design. | 19:20 |
Saviq | tedg, well, yeah, but I imagine they can work off of a silo | 19:21 |
faenil | tedg: I installed the package manually :P | 19:42 |
faenil | a silo to install a package is wasted effort :D | 19:43 |
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 | 19:44 | |
mterry | mzanetti, how do I link an upstream qt bug? the "qt" project doesn't recognize https://bugreports.qt.io/browse/QTBUG-XXXX | 20:11 |
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 | 20:19 |
=== dandrader is now known as dandrader|afk | ||
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:12 |
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:22 |
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:38 |
tedg | Ah, no, he got eaten by the ether! | 22:44 |
tedg | No, 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!