=== chihchun_afk is now known as chihchun
=== hikiko is now known as hikiko|ln
[10:51] <alan_g> greyback: have you looked at this? (I'm tempted to update, but won't if a review is already in progress) https://code.launchpad.net/~alan-griffiths/miral/spike-miral-qt-integration/+merge/303165
[10:52] <greyback> alan_g: I had a brief look this morning over coffee, was going to review proper after lunch.
[10:52] <greyback> welcome to update it
[10:52] <greyback> I mostly was thinking of plan for action to test it out
[10:55] <alan_g> greyback: updated, now you plan can test the combined changes
[10:56] <greyback> ack
=== chihchun is now known as chihchun_afk
[11:42] <alan_g> greyback: did you ever look into the crash I see when apps close?
[11:43] <greyback> alan_g: not yet, but the current work I'm doing touches that code so I hope to fix it that way
[11:44] <alan_g> OK, I was thinking of poking around, but it clearly isn't the right time just now.
=== hikiko|ln is now known as hikiko
[13:36] <greyback> alan_g: could you do me a favour and check if this compiles ok for you (with MIRAL_ENABLE_TESTS=1): lp:~gerboland/miral/use-cmake-extras
[13:37] <alan_g> GRE
[13:37] <alan_g> greyback: sure. 2 min
[13:38] <greyback> ta
[13:39] <greyback> I'm getting http://pastebin.ubuntu.com/23067425/ which I'm presuming is due to my custom mir build (with test LTO disabled)
[13:40] <alan_g> greyback: that's bug 1603080
[13:40] <ubot5> bug 1603080 in mir (Ubuntu) "mirtest-dev provides an incorrect .pc file" [Undecided,New] https://launchpad.net/bugs/1603080
[13:40] <greyback> aha
[13:41] <alan_g> It's your own fault for not reporting mirtest-dev bugs when you got it.
[13:41] <greyback> I only got that now!
[13:41] <greyback> instead I was getting fail on gtest.a not existing
[13:46] <alan_g> Hm cmake not happy: "cannot create target "gtest" because another target of the same name exists"
[13:46]  * alan_g suspects a conflict with miral/test
[13:49] <alan_g> greyback: it works on its own, but conflicts with miral/test's gtest stuff.
[13:50] <greyback> alan_g: you're on yakkety, right?
[13:50] <alan_g> yes
[13:51] <alan_g> At least for that trial
[13:51] <greyback> it works well for me on xenial+overlay, no conflict. I'll  try yakkety
[13:53] <alan_g> Oops I mean you can't have both -DMIRAL_ENABLE_TESTS=on and -DMIRAL_ENABLE_QT=ON
[13:54] <greyback> alan_g: ah, I see what you mean
[13:54] <greyback> yeah, that's step 2
[13:54] <greyback> for me, I just hit the fact that Ninja doesn't like the GMock/test script shipped in miral
[13:55] <alan_g> Anyway, you just need to hack your mirtest.pc file to build for yourself
[13:57]  * alan_g switches back to xenial (without overlay)
[14:00] <alan_g> greyback: are you planning to fix the conflict with miral-qt?
[14:01] <greyback> alan_g: I'm working on it now
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== JanC is now known as Guest7844
=== JanC_ is now known as JanC