[04:14] Urgh. Have I broken CI again? [04:14] The test-build worked, damnit! [04:14] ON CI! === LockeAnarchist- is now known as LockeAnarchist [05:25] OOOOooooooooh! [05:37] I have become the destroyer of CI! [05:45] * RAOF should not work in branches where the default push is to the default CI runner. [06:14] Alright! [06:14] CI should really, really work now you guys. [06:14] FOR REALZ! [06:14] * RAOF goes to pick up his car. [07:27] Ok. Now CI's just messing with me. === tvoss is now known as tvoss|test === tvoss|test is now known as tvoss === yofel_ is now known as yofel [09:24] yay renaming functions with the c-preprocessor is great [09:27] It is a powerful technique - and should be used responsibly (i.e. rarely) [09:28] here it was used to link to archives together that use a lot of copied code [09:28] *two [09:28] hence one is built with a header full of #definef func old_func .. === davmor2_ is now known as davmor2 [11:22] alan_g: Hi! Could you please elaborate on "Indeed wouldn't placing a cache implementation directly in the configuration object be sufficient?". Do you mean that the configuration object should have something like: std::shared_ptr the_platform_library() ? [11:24] alf_: I was thinking of a data member SharedLibraryCache with a load_library member. [11:26] But I guess we only ever load the one, so a std::shared_ptr platform_library member would do. [11:28] alan_g: The SharedLibraryCache needs to be shared/survive across configurations and MirConnections, so I am not sure how a SharedLibraryCache as data member would work. [11:28] alf_: OK, that answers my "needs info". === alan_g is now known as alan_g|lunch === Saviq_ is now known as Saviq [12:19] hi [12:22] Does someone has metrics for 2D/3D throuput of mir? [12:23] throughput [12:24] there's a glmark built for mir [12:24] Does the performance are good compared to legacy X? [12:26] sure [12:26] In what proportion? [12:34] too vague questions [12:36] I'm wanting to write a 2D game, and wanting to know what is the blitter speed ? [12:38] with the in-game stuff, thats up to you... for mir at least [12:38] we bypass fullscreen on mesa [12:38] and use the overlay path on android [12:41] could everyone take a look at https://code.launchpad.net/~kdub/mir/frontend-tracking/+merge/232123 [12:41] and indicate which of the two exchange_buffer messages are preferable? === alan_g|lunch is now known as alan_g [13:08] is there some vsync for smooth animation? [13:09] kdub: I've looked and don't have a preference [13:10] thanks alan_g [13:42] have to leave you, have a nive day/evening. [14:01] kdub: fine with both, although I wonder if allowing the client to indicate surf_id could be used maliciously. Of course we already trust the client library to give us correct info in general, so perhaps this is a non-issue. [14:02] alf_, yeah, agree [14:03] at any rate, i'll be on later than normal tonight, so I'll just chat it over with RAOF then [15:09] a gmock question: is using MOCK_METHODX(....) any good when I want to check a complex parameter of a method? [15:09] I have a parameter like this "const QList &points" [15:09] so I wanna iterate the list and check the values of the TouchPoint structure, etc [15:11] I wonder if using the gmock magic stuff (like declaring the method with MOCK_METHOD() and using things like EXPECT_CALL()... in the test) will be of any use in this case [15:11] or if I'm better of doing the mock implementation manually [15:11] * dandrader was told mir developers were the local gmock experts [15:11] dandrader, you might want to look into MATCHER_P [15:12] dandrader, so yes, gmock is quite useful in your scenario, and you can just describe your matching logic in MATCHER_P [15:13] * dandrader googles for MATCHER_P [15:13] there are also some container matchers [15:13] https://code.google.com/p/googlemock/wiki/CheatSheet#Container_Matchers [15:13] that might be helpful [15:14] tvoss, kdub, ok, thanks for the tips. will look into those === chihchun is now known as chihchun_afk [16:40] greyback__ when I build qtmir, it keeps failing on a mir header file, even though the file exists... [16:40] josharenson: can I see the build output please? How are you building? [16:41] josharenson: if you're building manually on the phone, you need this: qmake "QMAKE_CXX=arm-linux-gnueabihf-g++-4.9" "QMAKE_LINK=arm-linux-gnueabihf-g++-4.9" "QMAKE_LINK_SHLIB=arm-linux-gnueabihf-g++-4.9" [16:41] as per the readme.. mk-build-deps, qmake, make (getting a pastebin of the output) [16:41] ah, I better update the readme - the 4.9 thing is annoying [16:41] i was going to update it when I got it working :-p [16:42] http://pastebin.ubuntu.com/8151394/ [16:44] josharenson: so libmirserver-dev is installed? In that case, I'd clean all & try again (qmake not good at seeing dependencies changing): bzr clean-tree && bzr clean-tree --ignore [16:44] then try the qmake command again [16:55] greyback__ I had libmirserver-dev installed, but cleaning the tree seems to have fixed whatever was wrong... [16:56] josharenson: glad to hear it. Usually that happens to me if I run qmake before having all the dependencies installed. Running qmake again after often does not pick up the new header/lib directories [16:56] greyback__ good to know, thanks [16:56] full clean and retry is my usual fix [16:56] np === alan_g is now known as alan_g|EOD === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader === dandrader is now known as dandrader|afk === DalekSec_ is now known as DalekSec [22:54] AlbertA2: hey, while I agree that it's good to hold a lock during a voicecall, this same issue will happen next time we have an app using the proximity sensor [22:54] checking https://code.launchpad.net/~albaguirre/powerd/avoid-suspend-during-call/+merge/232290 [23:01] camako, is there an actual reason why we need the bump of qtmir dep to > 0.7.0? if not, we should just go for a no-change rebuild? [23:02] camako, although the -gles dep should definitely be bumped to actually be in sync with the non-gles one, but in general unless there's API breaks we don't bump the -dev dependency, it will still build with the latest available [23:02] Saviq we broke the ABI.. So I just updated the dependency [23:03] camako, right, so we only need a no-change rebuild against the new -dev, debian/control deps should generally reflect the truth (as in what's the minimal mir version with which qtmir will build) [23:03] Saviq, do you mean, for instance, if there is no client API break then leave the client dependency alone? [23:03] camako, basically if there's no code changes required [23:03] camako, there's no need to touch the deps [23:04] Saviq, but we definitely broke the ABI on the server side... client side perhaps I should leave it alone [23:05] camako, sure, but that will sort itself out through a no-change rebuild [23:05] camako, the build system will select the highest -dev version during the build anyway, and then encode the real versioned dependency in the runtime dependencies [23:06] camako, basically, qtmir can build with 0.6.0 still, can it not? [23:07] Saviq, Oh I see what you mean... I dunno if it can actually.. [23:07] camako, if you only bumped the debian/control version and it still builds, sounds like it can :) [23:08] Saviq, you're right... If mir hasn't changed actual code qtmir (or others) depend on then no need to do what I did [23:08] camako, dependencies (especially versioned ones) in debian/control should be "the least required" [23:09] * Saviq comments on the MP then [23:09] Saviq, I see... so how do we do a no change rebuild in the silo? [23:09] camako, you just need an empty (--unchanged) commit in the MP [23:10] camako, for the -gles counterpart you still need to sync the version number (and silo) [23:11] Saviq, yes Iit's just prelim at this moment [23:11] camako, yup [23:12] Saviq, thanks man... every time I talked to you I'm illuminated :-) [23:12] camako, always [23:14] camako, for fast development like we do now it isn't as important, but once Mir becomes part of the default distro, LTS and such, it pays off to not bump deps unnecessarily so that you're not forced to unnecessarily build a chain of dependencies [23:14] when someone wants to build on an older distro, that is [23:14] Saviq, makes total sense... === dandrader|afk is now known as dandrader [23:20] camako, one thing to note (when not bumping -dev deps), when you build things in silo, you need to make sure of the order things build in the right order [23:20] camako, because if you just build mir and qtmir in parallel, qtmir will actually build against the distro version [23:20] Saviq, ok.. and putting mir MP in the front doesn't help with that right? [23:20] camako, no, they all get pushed to the PPA at the same time [23:21] camako, you just need to run the build job with just "mir" first [23:21] or whatever comes first for real [23:21] Saviq, ok got it [23:22] camako, I think there's a circular dependency between papi and mir still, which means you might actually need to build papi, mir, then papi again or something similar [23:22] kdub_: Yo! [23:22] Saviq, yeah I've encountered that before and I look out for it [23:23] camako, ideally this should be resolved if at all possible [23:27] camako, that would require splitting papi, though [23:27] Saviq, yeah it's a nuisance [23:27] camako, the circular dep might lead into a difficult situation if in a single landing you'll have papi depending on new mir api and mir on new papi, neither will build without the other [23:28] oh and there's even one with mesa too bug #1239037 [23:28] bug 1239037 in mir (Ubuntu) "circular build-dependency between mesa, mir" [Medium,Triaged] https://launchpad.net/bugs/1239037 [23:29] I thought I filed one for papi, mir, will do so now [23:29] Saviq, yeah I guess we change only one first, build the other then we change the other and build the first.. if that makes sense) [23:30] camako, sure, that makes sense, but might mean you have to do the changes in stages unnecessarily [23:30] Saviq, right can't do in one silo... would have to split the landing [23:31] which would be an even bigger PITA than it already is [23:35] Saviq, so it'll look like this right? https://code.launchpad.net/~mir-team/qtmir/devel-mir-next/+merge/232329 [23:41] camako, yes, approved [23:42] camako, btw, looks like the papi/mir loop was resolved in the end, it's only the mesa one that remains [23:42] camako, i.e. mir does not build-depend on anything produced out of platform-api AFAICT [23:42] Saviq, I see [23:44] camako, btw, in theory it should be possible to pull lp:qtmir into lp:qtmir/devel-mir-next instead of merging (is what we do with the gles-sync branch) [23:45] well, not any more, because there were merges and such, but maybe we could make it happen again by overwriting devel-mir-next with trunk at the next landing [23:45] this way trunk and devel-mir-next actually maintain the same history [23:45] Saviq, I see what you mean [23:45] Saviq, good idea [23:45] * Saviq tries to verify [23:46] Saviq, this actually occured to me but I was a chicken to try [23:47] camako, so yeah, as long as there's no commit *more* on devel than on trunk, you can pull [23:48] * camako will do it next time [23:50] Saviq another nice thing with the empty commit option is now the MPs will be approved by Jenkins (before they always failed because Jenkins didn't know anything abt the new release before it happened) [23:51] camako, sure, assuming they will actually build ;) [23:51] camako, and well. sure, jenkins approved, but with old mir, so not really something you wanted [23:51] Saviq, yea I'm hungry so who knows what I did [23:52] camako, so yeah, even now, if you uncommit your last commit to devel-mir-next, you can just pull from trunk, so yeah, just after the next qtmir landing, pull into devel-mir-next and you'll have history nice & clean [23:52] Saviq, yeah but at least we won't see a failure [23:52] Saviq, yeah.. same thing with USC and papi [23:52] camako, yup [23:53] camako, so yeah, until devel-mir-next has actual source changes, jenkins should be happy - if it stops being happy is when you should actually bump the build dependencies appropriately [23:54] Saviq, yea.. it's sort of one extra checkpoint [23:54] camako, yup [23:56] o/ [23:57] * Saviq sleepy time [23:57] Hey Saviq, really thanks man.. good night [23:57] camako, right back at you