[07:21] <rion> Hi. I have an app which has tray icon with context menu. When I choose something from menu, windows open in background. It's expected to open them in foreground by user input. I can't find any api for that
[07:22] <rion> It qt app. With sni-qt in Ubuntu 14.04.
[07:26] <rion> I guess I have to call some X11 functions directly but I believe there should be better way
[07:31] <rion> Hm I knew it's useless to ask something in irc.. Gotta review unity sources..
[09:06] <tsdgeos> Cimi: mzanetti: what's the status of https://code.launchpad.net/~mzanetti/unity8/fix-left-edge-on-spread/+merge/243400 ?
[09:32] <tsdgeos> Cimi: is this what we use https://code.launchpad.net/~cimi/unity8/fix-1363400/+merge/247129 in rtm or there's a corresponding rtm branch for systemsettings?
[09:33] <tsdgeos> or is the bug for vivid only atm?
[09:40] <tsdgeos> dednick: you seem the most indicate to review this one liner https://code.launchpad.net/~seb128/unity8/set-inline-reply-hint/+merge/240766
[09:40] <tsdgeos> dednick: i looks good to me, but i'd appreciate you having a quick look
[09:40] <tsdgeos> dednick: morning btw :D
[09:41] <dednick> tsdgeos: :) good morning to you as well
[09:41] <dednick> tsdgeos: i'll take a look
[09:41] <tsdgeos> cool :)
[09:46] <dednick> tsdgeos, seb128: nearly there
[09:47] <dednick> just update pot file.
[09:47] <tsdgeos> trueth
[09:47] <seb128> dednick, you do that in individual mps? don't that make difficult to avoid merge conflicts when trying to land several branches?
[09:47] <seb128> tsdgeos, ^
[09:48] <dednick> i dont actually know. maybe not
[09:48] <tsdgeos> seb128: we actually do, don't have that many strings (new or changes) so it's not really an issue
[09:48] <seb128> k
[09:48] <seb128> we stopped doing that in settings because we kept having conflicts
[09:49] <dednick> yeah, i guess all of our strings come from third party
[09:49] <tsdgeos> you probably have much more changes there than us
[09:51] <seb128> right
[09:51] <seb128> anyway, going to update the pot
[09:51] <seb128> thanks for the review :-)
[10:11] <seb128> dednick, tsdgeos, updated with the pot update (rebased on trunk as well)
[10:13] <tsdgeos> tx
[10:13] <tsdgeos> dednick: do you do the honors?
[10:14] <dednick> tsdgeos: seb128: approved
[10:14] <tsdgeos> one more down!
[10:15] <seb128> dednick, thanks
[10:15] <tsdgeos> Cimi: how busy are you. Since https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1411748 is quite critical, want me to take it?
[10:15] <Cimi> tsdgeos, I started 5 mins ago
[10:18] <tsdgeos> Cimi: ok
[10:18] <tsdgeos> Cimi: is this what we use https://code.launchpad.net/~cimi/unity8/fix-1363400/+merge/247129 in rtm or there's a corresponding rtm branch for systemsettings?
[10:19] <Cimi> tsdgeos, there is another one, but is for ubuntu system settings
[10:19] <Cimi> tsdgeos, I am trying to see if it works with the binding
[10:19] <tsdgeos> Cimi: can you cross link them?
[10:21] <Cimi> tsdgeos, I tried with the binding before, It didn't work, trying again
[10:21] <tsdgeos> since they're basically "the same" review probably, no?
[10:22] <tsdgeos> so once we get one don't the other should be good too?
[10:23] <Cimi> tsdgeos, yes
[10:23] <Cimi> tsdgeos, I didn't cross link them
[10:24] <Cimi> tsdgeos, I added both to the bugreport
[10:24] <tsdgeos> oki, so some links in the description or something would be useful
[10:24] <tsdgeos> ah that makes sense, but can you do the link in the description for lazy people as me ? :D
[10:26] <Cimi> tsdgeos, it doesn't work with the binding
[10:27] <Cimi> tsdgeos, because it needs onNetworkNameChanged
[10:27] <Cimi> tsdgeos, which isn't a property
[10:27] <Cimi> tsdgeos, you get it through a binding
[10:27] <tsdgeos> it isn't?
[10:27] <Cimi> binding/methos
[10:28] <Cimi> tsdgeos, http://paste.ubuntu.com/9817316/
[10:28] <Cimi> tsdgeos, currentNetworkMode is the only property, but it doesn't change when you switch wifi
[10:29] <tsdgeos> that's weird
[10:29] <tsdgeos> and contradicts http://doc-snapshot.qt-project.org/qt-mobility/qml-networkinfo.html
[10:32] <tsdgeos> Cimi: where you got that paste from?
[10:36] <dandrader> mzanetti, ping
[10:39] <mzanetti> dandrader: pong
[10:39] <dandrader> mzanetti, did you go running yesterday?
[10:39] <mzanetti> dandrader: nope... weather doesn't play nice these days
[10:40] <tsdgeos> Cimi: but yeah you're right it's not a property
[10:40] <tsdgeos> Cimi: ok then, can you explain so in the MR?
[10:42] <mzanetti> dandrader: fyi, yesterday I saw the issue with a vivid device
[10:42] <mzanetti> dandrader: just for having it in my pocket for a minute or two
[10:42] <mzanetti> obviously not logging
[10:44] <dandrader> mzanetti, this is actually good news
[10:44] <dandrader> (that is also happens in devel-proposed)
[10:44] <dandrader> it
[10:45] <dandrader> mzanetti, I think the problem is your pocket :D
[10:45] <mzanetti> probably :D
[10:52] <Cimi> tsdgeos, doing
[10:52] <tsdgeos> Cimi: i did
[10:52] <Cimi> ok
[10:52] <Cimi> tsdgeos, was replying to phablet :)
[11:02] <tsdgeos> seb128: since https://code.launchpad.net/~cimi/ubuntu-system-settings/fix-1363400/+merge/247131 has both a u.s.s and a unity8 MR i guess only one of us should review it? You guys do or we do?
[11:55] <greyback> seb128: hey, the gtk fix is in silo 12, can I ask you to test?
[11:59] <dandrader> mzanetti, why it didn't build for arm? https://code.launchpad.net/~dandrader/+recipe/unity8-debug-edges
[12:00] <tsdgeos> i think only some special ppa's build on arm
[12:00] <tsdgeos> that's why we use the right-edge ppa
[12:00] <tsdgeos> even it's not about the right-edge anymore
[12:00] <tsdgeos> or that's my understanding
[12:00] <tsdgeos> could be wrong
[12:01] <dandrader> dammit
[12:02] <mzanetti> dandrader: you have to ping someone to enable arm builds, however you still only get a qemu builder then
[12:03] <mzanetti> which won't work because of a bug that it crashes when running tests
[12:09] <Cimi> tsdgeos, https://code.launchpad.net/~cimi/unity8/fix-1411748/+merge/247278
[12:10]  * tsdgeos clicks
[12:10]  * tsdgeos reviews
[12:37] <tsdgeos> Cimi: i'd change the commit message
[12:37] <tsdgeos> to "Background is no longer visible for horizontal cards with art and summary that don't specify a background"
[12:37] <tsdgeos> or something like that
[12:38] <tsdgeos> i mean it'll be visible if specified, no?
[14:12] <seb128> greyback_, hey, sure can, doing that in a bit
[14:12] <seb128> tsdgeos, whoever has free slots first, if you want to do it please do
[14:13] <dandrader> mterry, hey, all fixed in https://code.launchpad.net/~dandrader/unity8/unifyLightDMMocks/+merge/244593
[14:14] <mterry> dandrader, yar ok
[14:20] <tsdgeos> Cimi: i changed the commit message of https://code.launchpad.net/~cimi/unity8/fix-1411748/+merge/247278
[14:20] <tsdgeos> what do you think?
[14:22] <mterry> dandrader, the unity8.application_lifecycle.tests.test_application_lifecycle.ApplicationLifecycleTests.test_greeter_hides_on_app_focus test fails...
[14:22] <mterry> dandrader, it doesn't on trunk for me
[14:39] <tsdgeos> Cimi: also added some comments
[14:41] <paulliu> hi. https://code.launchpad.net/~canonical/unity8/pair_with_paul/+merge/246330
[14:41] <paulliu> Can anyone please review this for me? I think I'm not good for review because I wrote some code.
[14:50] <Cimi> tsdgeos, ok pushing now
[14:50] <tsdgeos> paulliu: who wrote it? you and allan?
[14:51] <tsdgeos> you and leo?
[14:51] <paulliu> tsdgeos: yes. Leo and me.
[14:56] <dandrader> mterry, it passed here
[14:56] <dandrader> mterry, did you run it more than once?
[14:56] <dandrader> mterry, maybe this test is unstable
[14:57] <mterry> dandrader, it failed in jenkins
[14:59] <tsdgeos> Cimi: https://code.launchpad.net/~cimi/unity8/fix-1411748/+merge/247278 looks good, let's wait for jenkins to finish
[15:01] <dandrader> mterry, many other AP tests failed there. And the very same set also failed for this other, unrelated, MP: https://code.launchpad.net/~cimi/unity8/fix-1363400/+merge/247129
[15:01] <dandrader> mterry, so to me there AP failures seem unrelated to the MP changes
[15:01] <mterry> dandrader, I just see one failure there?
[15:01] <mterry> and it's not the same one
[15:02] <dandrader> mterry, click on "Test Result"
[15:02] <dandrader> mterry, and you will get the full set
[15:02] <mterry> dandrader, still see one?
[15:02] <mterry> https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/793/testReport/?
[15:03] <dandrader> mterry, ah, I misread the page. it has "All failed tests" and "all tests"
[15:04] <dandrader> mterry, so the common failure is unity8.indicators.tests.test_indicators.IndicatorPageTitleMatchesWidgetTestCase.test_indicator_page_title_matches_widget(Messaging,Native Device)
[15:04] <mterry> dandrader, right.  I don't think that's related to your changes
[15:04]  * dandrader clicks on "rebuild" and hopes for the best
[15:04] <mterry> dandrader, but there is a greeter failure that might be
[15:05] <dandrader> oh, my VPN seems to be broken :/
[15:09] <dandrader> the underlying cause of the failure there is a dbus connection error in the code introspection
[15:15] <tsdgeos> what
[15:15] <tsdgeos> we have 36 qmltests failing?¿?¿?¿?¿
[15:15] <tsdgeos> how did that happen :D
[15:21] <mzanetti> err
[15:21] <mzanetti> really?
[15:21]  * mzanetti hides
[15:23] <tsdgeos> paulliu's branch has that many and has autopilot only changes so it would seem
[15:23] <tsdgeos> i'm building mster to test atm
[15:24] <seb128> greyback_, silo 012 fixes gtk apps for me (well, gedit doesn't run but that's probably another issue, gnome-calculator and a small test program I had to test click events work)
[15:24] <greyback_> seb128: yay
[15:24] <greyback_> seb128: tho strange, gedit worked for me >1 week ago.
[15:24] <greyback_> seb128: did you test phome/emulator by any chance?
[15:25] <seb128> greyback_, not yet, looking at gedit
[15:25] <greyback_> ok
[15:28] <kgunn> bug 1411748
[15:34] <mzanetti> mterry: hey, in testShell, the greeter shows up in tablet mode
[15:34] <mzanetti> any idea what could have happened?
[15:34] <mterry> mzanetti, huh...
[15:34] <mterry> mzanetti, that's based on width/height I think
[15:34] <mterry> mzanetti, in short no, I'm not sure why that would have changed
[15:34] <mzanetti> ack
[15:34] <mzanetti> looking
[15:35] <mzanetti> mterry: seems "multiUser" is true
[15:35] <mterry> ah that would do it too
[15:36] <mzanetti> mterry: so probably the wrong LightDM mock is loaded?
[15:36] <mterry> mzanetti, sounds like it
[15:53] <Cimi> https://code.launchpad.net/~cimi/unity8/fix-1363400/+merge/247129 is failing
[15:53] <Cimi> maybe I need qml-module-qtsysteminfo for build?
[15:54] <Cimi> in debian/control ?
[15:54] <Cimi> mterry, ^
[15:55] <Cimi> because it is used in tests I believe, since we load qml components that need qml-module-qtsysteminfo
[15:56] <mterry> Cimi, yeah, I'm not sure how jenkins runs the qmluitests, whether it installs the packages or just the build-deps
[15:56] <mterry> Cimi, but you are clearly getting "module "QtSystemInfo" is not installed" errors
[15:56] <mterry> Cimi, so I'd guess it just installs build-deps and you should add it
[16:00] <Cimi> mterry, thanks, pushed
[16:02] <seb128> greyback_, reading the upstart log, it seems gedit/gtk-mir calls regularly gdk_mir_display_request_selection_notification, that's not the case when running under the mir test server (where gedit runs without issue)
[16:02] <seb128> not sure if that's where it goes wrong though
[16:04] <greyback_> seb128: I've no clue why that might be. Sounds like separate issue to the rendering bug though. Think it worth a separate bug
[16:05] <seb128> yeah
[16:05] <seb128> the rendering issue is fixed
[16:07] <josharenson> Saviq: can you review this if you have time? It should resolve the issue you found in my screenshot branch. https://code.launchpad.net/~josharenson/unity-system-compositor/handle_screenshot_button
[16:07] <greyback_> josharenson: Saviq on hols this week
[16:07] <josharenson> greyback_: explains why he isn't on any other channel :-)
[16:08] <greyback_> josharenson: only time he is marked away from IRC is if he's half-way up a mountain
[16:08] <josharenson> greyback_: haha ok
[16:09] <rion4ik> Can someone tell me what api I should use to force activate/bring to front window in unity?
[16:09] <larsu> rion4ik: depends on the toolkit you're using
[16:09] <larsu> rion4ik: in gtk, it's gtk_window_present()
[16:10] <rion4ik> I use qt
[16:10] <rion4ik> But it's not a problem to link with anything
[16:11] <larsu> don't link with gtk if you're writing a qt app. There's probably some api on QWindow that does this
[16:13] <rion4ik> Actual problem is that my app can't still focus from another focused app.
[16:13] <larsu> that is a window manager feature - your app shouldn't do that
[16:14] <greyback_> rion4ik: that's something the window manager is designed to prevent you from doing
[16:15] <rion4ik> My app has context menu in tray. When user wants to open options for example he chooses corresponding item from context menu but sees nothing since options were opened in background
[16:16] <rion4ik> Quite annoying to click again to activate window
[16:17] <larsu> that's probably a bug in compiz :/
[16:18] <rion4ik> Probably when user clicks on tray compiz should mark current focused window as to be able to transfer focus to another app
[16:19] <rion4ik> Sorry for my English. Not native)
[16:20] <rion4ik> Should I report it?
[16:21] <greyback_> rion4ik: yes please do
[16:21] <greyback_> rion4ik: this is a workaround, but use it with care: http://bazaar.launchpad.net/~unity-2d-team/unity-2d/trunk/view/head:/libunity-2d-private/src/unity2ddeclarativeview.cpp#L249
[16:21] <rion4ik> Thanks
[16:22] <mzanetti> paulliu: hey, you guys forgot the checklist here: https://code.launchpad.net/~canonical/unity8/pair_with_paul/+merge/246330
[16:22] <paulliu> mzanetti: yeah. indeed. Let me paste the checklist.
[16:27] <mzanetti> paulliu: thanks
[17:51] <mterry> How can I tell if a qml object has been deleted out from under me?
[17:52] <mterry> er, scratch that, I have to go afk, but am interested in answers sitting on irc in meanwhile...
[18:59] <rion> greyback_: the workaround works perfect! 😃
[19:00] <greyback_> rion: happy to hear it. Use with care though, bringing random windows to the front will annoy users! :)
[19:10] <rion> greyback_: sure. Thanks 😃
[20:01] <Wellark> hey, who is the current unity7 guru ?
[20:02] <Wellark> I have very weird graphical glitches on vivid desktop with fglrx driver which makes working on the machine almost impossible or highly irritating at least
[20:03] <Wellark> basically the gl textures for different windows are wrong
[20:03] <Wellark> if one window updates, part of it's contents are filling the window on top of it
[20:03] <Wellark> and now I have qmlscene running as the bottom window
[20:04] <Wellark> and on top of it there are multiple terminals and what not
[20:04] <kgunn> Wellark: best bet is prolly mlankhorst for graphical stuff on unity7
[20:04] <Wellark> but the surface of the qmlscene window keeps popping up on top of the different windows constantly when something in it changes
[20:05] <Wellark> kgunn: is he on irc?
[20:05] <kgunn> unless bregma has an opinion
[20:05] <Wellark> it's like the z-ordering is totally off
[20:05] <kgunn> Wellark: yeah...might check ubuntu-devel
[20:05] <kgunn> or ubuntu-desktop
[20:06] <kgunn> he's somewhere in europe iirc
[20:06] <Wellark> I know sam was constantly battling these issues on proprietary drivers
[20:06] <Wellark> but it's the fglrx.. so I would kinda hope that unity7 on 15.04 would still be usable :)
[20:09] <Wellark> kgunn: or you could switch on your slave driver hat and get unity8 for the desktop ready by 15.04 so that I could switch over ;)
[20:11] <kgunn> :)
[20:11]  * kgunn cracks whip
[20:11] <kgunn> & sings Rawhide
[20:14] <bregma> Wellark, can you possibly revert to a previous version of Compiz?
[20:16] <bregma> sounds like a buffer problem, either fglrx (likely) or a recent change to Compiz to fix a problem seen with nVidia drivers
[20:21]  * davmor2 pictures kgunn like this now https://www.youtube.com/watch?v=RdR6MN2jKYs
[20:36] <Wellark> bregma: I can.. if you tell me how :)
[20:37] <Wellark> bregma: I so do not envy the fact that you guys have to battle these issues
[20:37] <Wellark> so hat off and free pints for the whole team! :)
[20:40] <Wellark> bregma: I now manually got a new driver from AMD
[20:40] <Wellark> I will first try it
[20:41] <Wellark> fglrx-14.301.1010
[20:47] <kgunn> bug 1390393
[20:47] <kgunn> josharenson:  wrt ^
[20:47] <kgunn> so i got your usc branch reviewed
[20:48] <josharenson> kgunn: cool
[20:48] <kgunn> is that going to need to be landed along with the unity8 branch?
[20:48] <josharenson> kgunn: no the USC branch shouldn't affect anything
[20:48] <josharenson> until the unity branch is mergec
[20:48] <josharenson> merged*
[20:48] <kgunn> josharenson: getting a little pressure on trying to land that one for rtm too :-/
[20:49] <kgunn> josharenson: is there any benefit to landing them seperately ?
[20:49] <josharenson> kgunn: It should be good to go, I ported mzanetti's earpiece changes into it already
[20:49] <josharenson> kgunn: no, the USC branch is less risky
[20:49] <kgunn> josharenson: ok, i'm gonna put a vivid silo together then for those 2
[20:50] <kgunn> josharenson: i think tsdgeos indicated a merge conflict tho...if you don't mind addressing
[20:50] <josharenson> kgunn: I haven't tested the most recent changes thoroughly yet (to the unity one)
[20:50] <josharenson> kgunn: yeah that was expected, working on it now
[21:08] <mzanetti> josharenson1: ping
[21:08] <josharenson1> mzanetti: pong
[21:09] <mzanetti> josharenson1: hey, just had a look at your input branch. one small thing:
[21:09] <josharenson1> sure
[21:09] <Wellark> bregma: if you could give me the instructions how to downgrade the compiz version, that would be great :)
[21:09] <Wellark> the new fglrx from amd hosed X.org
[21:09] <Wellark> so I reverted back to whatever is in the restricted repo
[21:10] <mzanetti> josharenson1: could you please change the PhysicalKeyMapper.onKeyPressed() to return a bool if the key has been handled or not
[21:10] <Wellark> reboot seems to have reduced the problem
[21:10] <mzanetti> josharenson1: instead if then re-comparing the keys in shell.qml
[21:10] <Wellark> but I suspect it gets worse over time
[21:10] <mzanetti> josharenson1: otherwise that'll drift over time and we return event.accepted = false while instead it has been handled or vice versa
[21:11] <mzanetti> so that shell only has a event.accepted = physicalKeysMapper.onKeyPressed(event.key);
[21:11] <josharenson1> mzanetti: someone told me a while back to do the exact opposite... it makes more sense the way you are saying
[21:11] <mzanetti> oh
[21:11] <mzanetti> really?
[21:11] <mzanetti> oh well then
[21:11] <josharenson1> mzanetti: I could even move the media keys into PhysicalKeyMapper
[21:11] <josharenson1> if PhysicalKeyMapper returned a bool
[21:11] <mzanetti> yeah, I'd say that would make sense
[21:11] <josharenson1> mzanetti: I'm for doing it your way
[21:11] <mzanetti> who told you to do it the other way round?
[21:12] <mzanetti> if it was Saviq, it takes precedence :)
[21:12] <josharenson1> mzanetti: it was a while ago, so I'm not sure...
[21:12]  * mzanetti wouldn't think saviq would do that...
[21:12] <mzanetti> if you agree too, let's change it...
[21:13] <josharenson1> mzanetti: no it was either dandrader or tsdgeos
[21:13] <josharenson1> mzanetti: ok I'm changing it.. Do you think I should move the media keys too? Have to get the CallManager into the PhysicalKeysMapper
[21:14] <mzanetti> if that opens a can of worms, no, if its simple to do, works and looks cleaner, yes
[21:14] <josharenson1> mzanetti: ack
[21:15] <mzanetti> sorry... I should have reviewed this earlier... but LP somehow doesn't show your commits between the comments, so I thought it was still WIP
[21:15] <mzanetti> i.e. not responded to the last comment yet
[21:22] <bregma> Wellark, try checking to see if you have an older version of *compiz*.deb in /var/cache/apt/archives -- you might get lucky
[21:23] <bregma> Wellark, are you on vivid?
[21:34] <Wellark> bregma: yes, on vivid
[21:34] <Wellark> bregma: I should be able to install older version by forcing it with apt-get
[21:34] <Wellark> the older version is still available in the archives, right?
[21:34] <Wellark> just tell me what version number I nee
[21:35] <Wellark> *need
[21:35] <Wellark> although I'm not seeing much of problems right now after a reboot, but they tend to manifest after the system has been running for a while
[21:35] <Wellark> bregma: btw, totally unrelated question
[21:36] <Wellark> if I grab unity7 source code
[21:36] <Wellark> am I able to run the local build inside a window for testing purposes
[21:36] <Wellark> I have one crazy prototype I want to try
[21:36] <Wellark> if it's not possible, then I guess I can set up a virtual machine where I can actually install and restart unity7 when needed
[22:31] <rion> Wellark: I have the same issue on VirtualBox.
[22:34] <rion> Not sure if gl related. Start screen of qt creator partially overlaps windows on top of it