/srv/irclogs.ubuntu.com/2014/03/06/#ubuntu-mir.txt

kgunnkdub: are you tip ? or at the rev of the 0.1.6 tag ?00:01
kdubthe tag00:02
kdubUSC should have which branches? i see a few likely candidates00:04
kdubhere's what I see http://pastebin.ubuntu.com/7041505/00:15
racarr_https://code.launchpad.net/~robertcarr/unity-mir/update-to-not-use-configure-output/+merge/20956500:19
racarr_more build fixing00:19
racarr_https://code.launchpad.net/~robertcarr/unity-mir/usleep-include/+merge/20956400:19
racarr_not sure why this only appears on my system...00:20
racarr_which branch fixes the00:22
racarr_frontend detail link errors00:22
racarr_or is this new00:22
kdubracarr_, that one is new to me00:24
kdubracarr_, with usleep, AlbertA has https://code.launchpad.net/~albaguirre/unity-mir/remove-usleep-call00:25
racarr_kdub: Ah. thanks00:48
RAOFracarr_: Have you made any progress on multihead inprocess eglL00:51
RAOFL00:51
RAOF?00:51
racarr_RAOF: Nooo01:04
racarr_you need a nested qualifier too :p01:04
racarr_but still no01:04
RAOFracarr_: True.01:04
RAOFOk. I'll do more diving today then.01:04
racarr_RAOF: Thanks01:10
=== duflu_ is now known as duflu
dufluracarr_: That was a purdy video of Chromium you made... and with the fix landed this week it no longer needs to be maximized ;)\02:07
racarr_duflu: Credit to olli :D02:19
racarr_I made a really ugly video02:19
racarr_referred to as the blair witch project of web browser demos actually02:19
racarr_it kind of caught on in the film festival scene but02:20
racarr_:p02:20
dufluHah. Yeah it's probably a job requirement that olli can do videos with flair02:20
RAOFWoot! That's why everything's getting wedged!05:03
RAOFracarr_: So you know how src/server/frontend/surface.cpp::swap_buffers_blocking assumes that the callback to swap_buffers() will always get called? There's absolutely no guarantee of that.05:05
RAOFracarr_: Although it might be a little bit to late for you, I guess ;)05:06
kgunnRAOF: wouldn't the only time that happens is if the gpu crashes ?05:20
kgunnor locks up05:21
RAOFApparently not.05:21
RAOFSpecifically: with the non-blocking client_acquire thingy, if there isn't a free buffer then client_acquire returns without calling the callback.05:22
RAOFNow, compositor_release *should* call the callback once its been called, but does not appear to be doing so.05:22
kgunnalso...blocking on swapbuffer swaps doesn't sound good in general (e.g. just wasting time for a buffer to be free...not good use of time)05:24
RAOFThe other option is to have a buffer-available signal?05:25
RAOFIf you set swap interval 0 then we don't block(ish), and always return a buffer.05:26
RAOFBlocking SwapBuffers until you've got a back buffer seems like a reasonable (and reasonably standard) thing to do?05:27
duflukgunn: Blocking in general is bad. But for swapbuffers, that's what throttles client rendering to the same rate as the compositor consumes frames05:49
dufluSo in places, very important to block05:50
kgunnduflu: are you saying you actually block clients renders (_even_ when they have a free buffer) ...if the compositor is "busy" and doesn't have a free buffer (e.g. double buffered system its easier to do)05:52
kgunnthat would seem to preclude clients from making a decision to just render if they want to...05:53
kgunnseems cheeky05:53
duflukgunn: Clients never have such a thing as a "free" buffer (that's bug 1253868). But they do always have a buffer to render to when not blocking inside SwapBuffers. SwapBuffers for the client will only block for one frame (16ms). Unless the screen is asleep, in which case we block everyone to avoid wasting power.05:55
ubot5bug 1253868 in Mir "[enhancement] Mir protocol design prevents clients from ever being more than double-buffered" [High,Triaged] https://launchpad.net/bugs/125386805:55
dufluNo, that's not right. Clients' SwapBuffers will return ASAP, on a maximum period of the maximum monitor refresh rate05:56
dufluUnless they have explicitly disabled blocking with eglSwapInterval(0), but that's a bad idea usually05:57
kgunnright...ok...video mode panels would likely be 16ms...but command modes don't have a vsynch...its realy an in determinant frequency05:58
RAOFYay! Hypothesis!05:58
duflukgunn: Yeah, indeterminant. But holistically your commands need to complete within that frame time, one way or another, to keep up 60Hz (or whatever)06:00
kgunnusually faster if possible...06:01
dufluWell the good news is that since Mir 0.1.5 we have maximized the rendering time available06:01
dufluOn desktop anyway. I'm not as familiar with Android06:01
RAOFNested creates 1 surface per output, even if the underlying system is cloned (which is pointless, but not super wrong). These surfaces get placed one above the other, meaning that one is completely obscured, meaning that its swaps never complete, and then everything falls in a heap.06:02
RAOFBut! Lunch time.06:03
* duflu assumes other people understand whatever bug that is06:03
RAOFduflu: https://bugs.launchpad.net/mir/+bug/128728206:04
ubot5Ubuntu bug 1287282 in Mir "Nested server may hang with multimonitor and internal clients (Mesa)." [Undecided,New]06:04
dufluThat sounds unpleasant06:04
RAOFYeah. You should really replace the “may” with ‘will definitely’. And the “mesa” with “but we don't have dual-head support on android, so can't expose this bug there” :)06:05
dufluRAOF: Please do06:06
dufluRAOF: On another note... I almost forgot. We can now implement xrandr rotation against XMir06:07
dufluFor fun. Not because anyone *needs* it06:07
RAOFWoot!06:07
RAOFAnyway, really lunch time.06:07
dufluHmm, apparently I had been running Mir on multiple monitors for weeks without knowing (half are off). And didn't notice any slowdown06:11
dufluIs it a good idea to automatically use monitors that are off from the beginning?06:11
RAOFduflu: We can't tell that they're off.06:19
dufluOh. That would do it06:19
RAOFExcept with DisplayPort and *some* HDMI.06:20
racarr_RAOF: Oh...good catch :D I forgot06:47
racarr_we had implemented the obscuring logic...06:47
racarr_I should have figured out thats what it was when06:47
racarr_moving the top nested window06:47
racarr_unblocked them lol06:47
racarr_I also should have remembered to mention that to you06:47
RAOFI should have tried it sooner :)06:47
racarr_:D06:47
racarr_ITERATEEEEE06:47
RAOFSo, shall I fix that FIXME in nested_output.cpp? :)06:48
dufluWhee, bisection without the ability to build old revisions06:48
RAOFOh, why not?06:49
dufluRAOF: The glm package update from trusty, remember?06:49
RAOFOh, right. Buh!06:49
dufluFortunately the offending use case is pretty uncommon right now06:51
racarr_RAOF: Would be excellent or if you are busy I can do it in the morning06:51
racarr_I am now, not busy! (I am sure that statement will be corrected when kgunn wakes up :p)06:52
kgunnracarr_: i have so much planned for you06:52
kgunnmmwwwhahhahahah06:52
racarr_go to sleep or you arent going to grow up to be tall.06:52
kgunntoo late for that06:53
racarr_:p06:53
kgunnprobably causses gray hair too06:53
kgunni so heart this landing process06:53
dufluhttp://memegenerator.net/instance/4650226106:54
racarr_What happened? It seems like everything is in a jumble06:54
racarr_and im missing the root cause of the chaos06:54
racarr_duflu: Lol I see So you do window management06:54
racarr_is the new team meme06:54
dufluThis is really annoying. SwitchingBundle reports multi-monitor frame sync working perfectly. But my eyes disagree06:56
dufluMaybe the parallelized page flipped has somehow found a way to go a double speed?06:57
RAOFBah! Any ideas what the hell is going wrong on the CI hardware? https://code.launchpad.net/~raof/mir/udevify-eventhub/+merge/206659 passes the (memcheck) tests on my desktop *and* my Nexus 4, but fails without any useful info on the build boxes.07:00
racarr_ok sorry australia friends but the more you say the less sleepy I feel :p and thats the wrong direction07:01
racarr_goood night!07:01
dufluracarr_: Night07:02
dufluRAOF: I was wondering... do we need to give CI new instructions as to how to successfully run the tests on armhf?07:02
dufluCos I assume the old method will fail now (since that branch)07:02
RAOFduflu: No, the old method works fine.07:03
dufluThe failures look udev/syscall specific. So possibly actually related to the code changes07:04
RAOFWhat failures are you looking at?07:04
dufluRAOF: A couple of them https://code.launchpad.net/~raof/mir/udevify-eventhub/+merge/20665907:05
RAOFWhich log result?07:05
dufluThis is a fun one:07:06
duflu[----------] 2 tests from UdevWrapperDeathTest07:06
duflu[ RUN      ] UdevWrapperDeathTest.DereferencingEndReturnsInvalidObject07:06
duflu==10840==07:06
duflu==10840== Process terminating with default action of signal 11 (SIGSEGV)07:06
duflu==10840==  Bad permissions for mapped region at address 0x1884DB807:06
duflu==10840==    at 0x1884DB8: ???07:06
duflu==10840==07:06
RAOFThat's entirely expected.07:07
RAOFThe test is checking that SIGSEGV is raised, so valgrind picking that up is pretty normal :)07:07
dufluWell it can't pass with it either, can it?07:07
dufluOh, those are passing!?07:07
racarr_I was trying to write an acceptance test today07:08
racarr_that, a client will block on swap buffers.07:08
RAOFduflu: Yup, those are passing.07:08
racarr_and swap buffers will not turn the display back on07:08
dufluRAOF: A memcheck failure doesn't always mean the test failed. It can just mean non-zero errors reported by valgrind (which there are)07:08
racarr_if its another client thats requested turning the display off07:08
racarr_but its hard to write these tests, i.e. check that like07:09
racarr_swap buffers blocks. when do you time out?07:09
RAOFA second, maybe 2.07:09
racarr_But, if 1, we add some symbolic kind of names to threads, i.e. a thread registers itself I am a compositor thread, etc.07:10
dufluRAOF: Sanity check with some trivial proposals and see if they can land?07:10
racarr_and then use a custom mutex implementaion with a simple...graph traversal kind of thing07:10
racarr_you can write tests like...07:10
racarr_EXPECT_THAT(RPCThreadFor(session, BlocksOn(CompositorThread))07:11
racarr_and then just quit the test07:11
racarr_</idea>07:11
dufluracarr_: If this is you sleeping, may I suggest finding new ways to wind-down evenings? :)07:11
racarr_lol07:11
racarr_i just wanted to put it somewhere because I knew I would forget07:12
dufluracarr_: And archived for you: http://irclogs.ubuntu.com/2014/03/06/%23ubuntu-mir.html07:14
RAOFracarr_: Enjoy https://code.launchpad.net/~raof/mir/nested-one-surface-per-crtc/+merge/20961509:18
duflualan_g: Any chance you could look into https://bugs.launchpad.net/bugs/1288570 some time?09:23
ubot5Ubuntu bug 1288570 in Mir "[regression] Multi-monitor frame sync no longer works (not synchronized)" [Medium,Triaged]09:23
alan_gduflu: sure - although you mentioned something about just using a different approach in comments?09:24
duflualan_g: That was really only relevant when I thought I caused it. I'm not sure how much that rewrite idea would help now09:24
alan_gok09:24
duflualan_g: In fact, it's not just rendering faster, but looks like it's alternating forwards/backwards in time so juddering09:31
dufluNot sure09:31
anpok_http://paste.ubuntu.com/7043201/ I will make an MP out of that - but I still get a fancy off by one error for a rectangle (2x6) placed at (10,10) and rotated by 90 degrees. but only in x direction09:36
anpok_i expected the rectangle to go from 8 to 13, but it starts at 9..09:37
alan_gduflu: it is a big shame we don't have an acceptance test covering it.09:38
* alan_g wishes for a software backend once again09:38
duflualan_g: Yeah I went back and looked at the tests I wrote for that. They seemed OK09:38
dufluOf course, you can't write tests for *everything* you haven't thought of yet09:41
dufluHowever we broke it, I didn't foresee as possible09:42
alan_gduflu: don't let it spoil your evening09:43
duflualan_g: It won't. I was happy to just complete the bisection09:44
kgunnalf_: so greyback is ahead of me and tested silo a bit...any reason mir on android might be looking for mesa & trying to open vts ? (...thinking of the recent backend loading changes)12:20
alf_kgunn: Is this during the build in the silo, or when using the built packages on a phone? Do we have any logs?12:21
greybackalf_: using build packages from the silo. I just flashed my tablet & installed the silo's ppa and updated12:22
greybackah not again, think latest adbd broken on manta12:23
greybackSetting up libmirclientplatform-mesa:armhf (0.1.6+14.04.20140306-0ubuntu1) ...12:26
greybackupdate-alternatives: using /usr/lib/arm-linux-gnueabihf/mir/clientplatform/mesa/ld.so.conf to provide /etc/ld.so.conf.d/arm-linux-gnueabihf_mirclientplatform.conf (arm-linux-gnueabihf_mirclientplatform_conf) in auto mode12:26
ogragreyback, working on that (well, trying to, my manta misbehaves)12:26
greybackogra: Great, thanks. It started working again after reboot. But I was struggling with it this morning, had to reflash12:27
ograoh ? i wonder why it started working ... theoretically it shouldnt :P12:27
greybacklol12:27
kgunni'm trying on a N412:28
greybackogra: any logs useful for you, while it is magically working?12:28
kgunnalf_: what should i look for as this thing attempts to boot...log ? or output ?12:28
ograno, i'm just surprised it started to work12:29
alf_greyback: kgunn: hmm, we shouldn't be installing the mesa packages on phone/tablet. Probably something in our package upgrade dependencies/replaces. In any case, just remove the -mesa packages (and install the -android packages if not already there) and everything should work.12:31
alf_greyback: kgunn: I don't think this is a problem with images, since the proper packages should be seeded for them.12:32
greybackalf_: confirmed that works12:32
kgunni just got the vt err on unity8 manual attempt12:32
greybackkgunn: sudo apt-get install libmirclientplatform-android libmirserverplatform-android && sudo apt-get purge libmirclientplatform-mesa:armhf libmirplatformgraphics-mesa:armhf12:33
greybackthat should help12:33
alf_greyback: kgunn: I think the problem is that both libmirclientplatform-mesa libmirclientplatform-android replace libmirclientplatform, but I am not sure if there is a way around that12:34
alf_greyback: kgunn: anyway, this should only be a problem when upgrading in a system that already has libmirclientplatform. Future images won't have it, so problem solved :)12:36
kgunnlol12:36
kgunnnice try12:37
greybackalf_: how is mesa/android decided by Mir? At runtime?12:37
alf_greyback: dpkg-alternatives set a system-wide default, but you can also override the exact library to use with environment variables12:38
davmor2greyback: magic12:38
greybackalf_: I just don't want someone to accidentally install -android packages on their desktop and break it.12:39
greybackogra: sanity restored: rebooted manta again, no adb12:40
ograheh12:40
ogragreyback, feel free to roll back manually  http://launchpadlibrarian.net/168424245/android-tools_4.2.2%2Bgit20130218-3ubuntu19_4.2.2%2Bgit20130218-3ubuntu20.diff.gz12:41
alf_greyback: nor do I, so I made sure that can't happen :) The -mesa and -android packages have the same alternatives priority, so any existing packages (e.g. mesa) are not overriden by newly installed ones. You need to change the alterntives by hand12:41
ogra(but you will get the annoying disconnects)12:41
greybackogra: you're fixing the disconnects, wonderful! You'll single handedly lower my blood pressure and extend my life span with that fix12:44
kgunnone nit for anyone reading scrollback its "sudo apt-get install libmirclientplatform-android libmirplatformgraphics-android && sudo apt-get purge libmirclientplatform-mesa:armhf libmirplatformgraphics-mesa:armhf"12:44
ograwell it doesnt seemt o work on manta ...12:44
ogratrying to find a proper fix ... but since i use my manta only once a month the battery went completely dead12:44
greybackogra: well if you need tester, let me know12:45
ograwill do12:45
greybackkgunn: with that change, things seem to work. Any idea how the crash is reproduced?12:46
kgunngreyback: mterry & kdub were saying it was on startup12:46
kgunnwhat's strange is kdub could launch unity8 manually...12:47
kgunnand it would work12:47
kgunngreyback: one note...they were building tip mir dev...and not sure about the other components....12:47
kgunnmeaning...its not precisely whats in silo312:48
greybackkgunn: which means I don't care...yet12:48
greybacklet's see if silo3 has that issue12:48
kgunngreyback: right if silo 3 works...12:48
kgunnalf_: greyback ...however, i just removed the mir mesa packages & installed the android ones....and rebooted, stuck at google splash screen12:49
kgunnmanual attempt resulted in: Failed to load platform plugin "ubuntumirserver"12:50
greybackkgunn: is qtubuntu-android:armhf installed?12:50
kgunnjust trying that now12:51
kgunnooo....broken packages12:51
didrockskgunn: broken, like broken? (or broken as you didn't upgrade everything needed?)12:52
kgunndidrocks: just to make sure...i added the ppa(slio3) did a apt-get update, followed by a dist-upgrade12:53
didrocksyeah, that should be right12:53
kgunnhttps://pastebin.canonical.com/106009/12:54
didrocksdid you change the packaging more than usual?12:54
didrockshum12:54
kgunnnope...only thing i did was the removal of "mesa" and install of "android" discussed for the 2 pkgs above12:54
didrockskgunn: ok, I can extract the packaging diff as a first look (cheating and sshing on citrain machine)12:55
didrocksjust to see if there is any crazyness in packaging changes first ;)12:55
kgunndidrocks: if its important to know...i started with a channel devel-proposed image like 30 min ago12:55
didrocksyeah, sounds perfect12:55
kgunndidrocks: hey thanks...12:55
didrockslet me reflash cleanly my phone at the same time12:55
kgunndidrocks: yeah...weird greyback says its ok with manta (i'm on mako)..but greyback did you have an "older image" ?12:55
didrocksyeah, the device shouldn't come into play for deps issues12:56
didrocksapart from a "non cleaned image"12:56
greybackkgunn: ooh wow I did, from Feb20: http://cdimage.ubuntu.com/ubuntu-touch/daily-preinstalled/current/12:56
kgunnok guys...i gotta step away for monring duties...bbian12:56
kgunnbbiab12:57
kgunngreyback: you lazy bastard :)12:57
kgunnjk...lol12:57
greybackI've excuses, but anyway...12:57
didrocks-               android-headers (>=4.2.2) [armhf],12:58
didrocks+               android-headers (>=4.2.2) [i386 amd64 armhf],12:58
didrocksthat's a little bit weird, but not the issue per say12:58
didrockshum12:59
didrocks+         libmirplatformgraphics-mesa (= ${binary:Version}) |12:59
didrocks+           libmirplatformgraphics-android (= ${binary:Version}),12:59
didrocksgreyback: do you know about that? ^12:59
didrockskgunn: description of the binary packages aren't good btw (shouldn't be NEWed that way)13:00
kgunnalf_: might know about that didrocks13:01
greybackdidrocks: I do now13:01
greybackbut alf_ is the author13:02
alf_didrocks: ?13:03
=== alan_g is now known as alan_g|lunch
didrocksalf_: the short description needs to be different between packages13:06
didrocksalf_: both reference the same13:06
didrocks(also, not really sure the per-arch will please everyone ;))13:06
didrocksso, from what I understand, on dist-upgrade, we need to install the -android mir and client packages, right?13:06
alf_didrocks: either the -mesa or -android packages, depending on which platform we are on (and x86 no longer means mesa, nor armhf means android necessarily)13:10
didrocksalf_: right, the thing is that we don't build even -mesa on all platform13:10
alf_didrocks: e.g. on the x86 phone emulator we would need android13:10
alf_didrocks: which platforms are these?13:11
didrocksalf_: powerpc and ppcel6413:12
alf_didrocks: we aren't building any mir package on these platforms AFAICT13:13
didrocksalf_: yeah, I mean, it's something that will need to be looked at some point (especially as being in main intend more or less that)13:13
* didrocks wonders why he can't download from the ppa on the phone :/13:13
didrocksoh I know13:13
didrocksalf_: however, we still need to have different short descriptions between binary packages (which was the start of the discussion)13:14
alf_didrocks: ok, I will take care of that13:15
didrocksI'll be happy to not block on this for NEWing if there are new packages, if there is a MP for that :)13:15
didrocksthanks13:15
didrocksok, so, let me try to apt-get install + dist-upgrade13:15
didrocksalf_: so, with that change, I guess we need to seed libmir*android on Touch?13:16
alf_didrocks: Right. If you just upgrade in an older image you will get the mesa packages, which is wrong (greyback had that problem), but I am not aware of a way around this transition glitch.13:18
didrocksyep13:18
didrocksthat should be noted in the silo line (kgunn)13:18
didrocksok, so  libmirplatformgraphics-android : Depends: libmirplatform (= 0.1.6+14.04.20140306-0ubuntu1) but 0.1.5+14.04.20140212-0ubuntu1 is to be installed13:19
didrocksinteresting, let's see why apt-get doesn't want to upgrade it13:19
didrocksI guess we just need them to be listed explicitely as libmirserver15 will be removed13:19
didrocksThe following packages will be REMOVED:13:20
didrocks  libmirserver1513:20
didrocksThe following NEW packages will be installed:13:21
didrocks  libmirclientplatform-android libmirplatformgraphics-android libmirserver1613:21
didrocksThe following packages will be upgraded:13:21
didrocks  libmirclient7 libmirplatform libmirprotobuf013:21
didrocks  libubuntu-application-api-mirserver1 libunity-mir1 mir-test-tools13:21
didrocks  unity-system-compositor13:21
didrocksalf_: I'm missing no package here, right? ^13:21
alf_didrocks: looks good13:21
* didrocks reboot the phone13:21
didrockskgunn: ok, all good here. As the transition is more complex than usual (new packages that you need to set explictely) and some packages need to be removed, you need to help apt. "sudo apt-get install libmirplatformgraphics-android libmirclientplatform-android libmirplatform" should do it13:24
didrockskgunn: can you please add something along the line "need to seed libmirplatformgraphics-android and libmirclientplatform-android" in the comment of your landing request please? (and set that to red)13:25
alf_kgunn: didrocks: https://code.launchpad.net/~afrantzis/mir/packaging-fixes/+merge/209676 , do you want an MP for urgent inclusion into lp:mir too ?13:28
didrockskgunn: alf_: if you don't have any other reason to rebuild Mir for other fixes, don't worry about it as long as it's merged in next release13:29
didrocksalf_: but you will get bonus point if you can fix tabs vs spaces :p13:30
didrockswhich are in debian/control, rules, the new postinst scripts as well IIRC13:30
alf_didrocks: I can only see spaces (except the make rules that need tab indentation)13:32
didrocksalf_: let me grab you some of the stuff I'm seeing (space and misalignement)13:33
didrocksalf_: oh, maybe you did that on purpose?13:34
didrocks+         libmirplatformgraphics-mesa (= ${binary:Version}) |13:35
didrocks+           libmirplatformgraphics-android (= ${binary:Version}),13:35
didrocks(same on client)13:35
didrocksand yeah, in rules:13:35
didrocks+         -DMIR_RUN_ACCEPTANCE_TESTS=OFF -DMIR_RUN_INTEGRATION_TESTS=OFF \13:35
didrocks+          -DMIR_PLATFORM=android\;mesa13:35
alf_didrocks: @control, yeah that's on purpose, @rules, we don't have that text in debian/rules, at least not in mir/devel13:38
didrocksalf_: it's in the one which was generated on trunk, starting to get anxious if you have such diff there13:39
didrocksalf_: do you mind checking once that's landed? maybe you do have trunk/devel diff with changes only in trunk13:39
alf_didrocks: it's going to be taken care of by https://code.launchpad.net/~mir-team/mir/trunk-0.1.6/+merge/208717 , I think that kgunn is testing this branch in the silo before merging into trunk, so we can fix the big issues before landing13:42
didrocksalf_: yeah, the diff I showed you is the result of that branch13:43
alf_didrocks: ok, thanks for pointing it out13:45
=== dandrader is now known as dandrader|afk
=== alan_g|lunch is now known as alan_g
kgunnalf_: didrocks ...the rules file needs to suppress the android integ tests b/c the silos != our "ci" builders....meaning, anything requiring access to arm gl drivers fails on silo but passes on dev branch "ci"14:06
kgunni say "ci" in quotes since i was corrected that its all CI...but you know what i mean :)14:07
kgunnalf_: and thanks...14:07
kgunnalf_: i'll just make those updates manually14:13
kgunnto the branch for lp:~mir-team/mir/trunk-0.1.614:13
alf_kgunn: ok, in that case perhaps also fix the indentation noted by didrocks in debian/rules (-DMIR_RUN_ACCEPTANCE_TESTS=OFF -DMIR_RUN_INTEGRATION_TESTS=OFF \)14:17
alf_kgunn: actually the next line (-DMIR_PLATFORM=android\;mesa)14:19
kgunnalf_: what's wrong with it ?14:20
rsalvetididrocks: you also need to change the seeds with latest mir14:20
rsalvetiguess you're just landing that together with the mir silo14:20
alf_kgunn: nothing functionally, just one space more indented than it should be14:20
didrocksrsalveti: yeah, I pinged kgunn to add a note for that14:21
rsalveticool14:21
didrocksrsalveti: not sure we can land that together, the packages are NEW to the archive14:21
rsalvetioh, that's true14:22
didrocksrsalveti: something I've speced for the airline and we'll need to change our tooling, but for now, let's do it in a 2 steps process14:22
didrocks(and not kick an image in between :p)14:22
kgunndidrocks: i should rebuild ? or ?.....i just updated the branch for mir0.1.6 that would be merged post landing...14:23
rsalvetididrocks: sure :-)14:23
didrockskgunn: what branches? only the typo ones?14:23
alf_kgunn: actually it's a matter of tabs vs spaces (but not a big deal, we can fix it later)14:23
=== dandrader|afk is now known as dandrader
didrocksyeah, don't rebuild just for that14:24
didrocksget that in the next batch14:24
kgunndidrocks: yeah, the description: in control & spacing for rules14:24
kgunnok14:24
kgunnoff to test then....14:24
didrockskgunn: don't bother to put into the current one :)14:24
didrocksas long as it's not lost, it's fine :)14:24
kgunndidrocks: curious...what was the deal with broken packages ? was that me ?14:30
kgunnits working now14:30
didrockskgunn: did you see my apt-get install line?14:30
kgunn...well qtubuntu-android installed...14:30
kgunndidrocks: you had no trouble ?14:31
didrocks14:24:16 didrocks | kgunn: ok, all good here. As the transition is more complex than usual (new packages that you need to set explictely) and some packages need to be removed, you need to help14:31
didrocks                  | apt. "sudo apt-get install libmirplatformgraphics-android libmirclientplatform-android libmirplatform" should do it14:31
didrockskgunn:  ^14:31
=== alan_g is now known as alan_g|tea
=== alan_g|tea is now known as alan_g
=== dandrader is now known as dandrader|lunch
kgunnkdub: mterry...so if either of you try silo 3 just be aware don't sudo dist-upgrade the ppa...you have to specifically sudo apt-get install libmirplatformgraphics-android libmirclientplatform-android libmirplatform16:27
mterrykgunn, yeah I meant to ask16:27
kgunnin order to avoid loading platform obj's for mesa....16:27
mterrykgunn, I can't have the mesa variants I suppose?16:27
mterrykgunn, that's what was causing my crashes we're guessing?16:28
kgunnotherwise mir will think its mesa and try to open vt's and stuff16:28
kgunnmterry: it would depend on how you loaded it16:28
mterrykgunn, normal everyday booting into unity816:28
kgunn...if you by chance installed mesa platform packages for mir then yes16:28
mterrykgunn, I bet I did dpkg -i ../libmir*.deb16:29
mterrywithout realizing we got new mesa things16:29
kgunnyeah...that'd do it16:29
mterrykgunn, stop making it a pain to dpkg a new mir!16:29
mterry:)16:29
* alan_g goes looking for yet another monitor to add to his laptop16:34
kdubi must have missed some email somewhere, but when we say silo... where is that?16:38
ograwhere the train parks :)16:40
greybackkdub: https://launchpad.net/~ci-train-ppa-service/+archive/landing-003/ is the one I was referring to16:41
kdubthanks greyback16:42
ogragreyback, fyi, adbd should behave a lot better with my recent upload ... (i'm not completely done yet on the mtp side though)16:43
greybackogra: cool, thanks16:43
=== dandrader|lunch is now known as dandrader
=== alan_g is now known as alan_g|EOW
mterrykgunn, so theoretically removing those two -mesa packages would fix the unity8 problems?  didn't seem to work for me19:13
kgunnmterry: well...did you purge ?19:14
kgunnbtw...in the end, i had to reflash & reinstall....the simple uninstall/install only android didn't work for me19:14
kgunnonly greyback claimed success19:15
kgunnandhe was on an ancient image19:15
kgunnancient == 2 week old19:15
mterrykgunn, ok, will do the cycle19:16
kgunnmterry: sorry...sucks i know19:16
mterrykgunn, if this is indeed necessary (instead of some weird other problem), seems like bad packaging...?19:17
kgunnalf_ changed packaging names....and according to didrocks if you dist-upgrade you get the mesa, but if you do the specific install like so19:18
kgunnsudo apt-get install libmirplatformgraphics-android libmirclientplatform-android libmirplatform19:18
kgunnthen it all just works19:18
kgunnand according to didrocks...that'll be fixed when it lands for real19:19
rsalvetikgunn: yes, that should indeed work20:43
rsalvetikgunn: are you having issues with it?20:43
rsalvetiI can give it a try20:43
kgunni just reflashed and it worked20:44
rsalvetiawesome20:44
rsalvetikgunn: any other issue blocking that silo?20:50
rsalvetijust to know if we'll be able to land that today still20:50
kgunnrsalveti: i found a gnarly manual test that revealed a problem with OSK....note AP test for OSK passed 100%20:50
rsalvetioh, ok20:51
racarr_mterry: Want to test a branch for your screen turning on issue?21:40
mterryracarr_, sure...  Can I apply it to 0.1.5 easily enough?21:42
racarr_mterry: Yes unity-mir only21:42
racarr_mterry: lp:~robertcarr/unity-mir/greeter-configuration-for-mterry21:43
racarr_then in your unity8 branch, where main.cpp calls21:43
racarr_runQMirServerWithClient21:43
racarr_call runQMirServerWithGreeterClient21:43
racarr_if you are um...21:43
racarr_the greeter21:43
mterryracarr_, huh, k21:46
mterryracarr_, your branch makes me feel so special  :)21:49
racarr_mterry: Oh no, thats an abbreviation for Mir Thread Enhancing RAII Refactoring (from) Yesterday.21:53
mterryracarr_, lol'21:54
rsalvetialf_: how to run glmark2 with mir? also, can we update the package available in ubuntu to also bring support for mir by default?21:55
rsalvetiit seems the package is really old21:55
rsalvetialf_: I can sponsor the upload if you need any help21:55
mterryracarr_, I must have screwed something up, unity8 is crashing22:22
racarr_mterry: Well. You mayt have not screwed anything up ;)22:22
racarr_can you get a basic backtrace + stdoutstderr etc if there is anything interesting22:23
racarr_and if its not obvious I will try and reproduce it locally22:23
mterryracarr_, I'm more inclined to think it's me right now, will try again.  But I didn't see anything interesting in stdout/stderr22:23
racarr_ok22:24
racarr_API/ABI issues are all over right now so22:24
racarr_one thing to check lol22:24
mterryracarr_, yar22:24
mterryracarr_, if I fail again, note that you can have split greeters too with lp:~mterry/unity8/split and lp:~mterry/unity-system-compositor/greeter-depth22:25
racarr_mterry: Im just trying to avoid setting it all up by hoping it will all work lol22:26
racarr_if you dont get anywhere pretty quick lemme know and ill build it all22:26
mterryracarr_, that's fair.  Don't do anything until I reconfirm my crash22:26
* mterry is rebuilding unity-mir, I might have accidentally built your branch with bad ABI22:26
racarr_storrry of our lives22:29
racarr_I had this idea the other night that everything could be quote continuously integrated22:29
racarr_but I was probably just too drunk thats crazy talk22:30
racarr_:p22:30
mterryracarr_, ok.  Fixed crash by recompiling22:39
mterryracarr_, but now I don't see a change in behavior22:39
racarr_mterry: That is a crying shame23:00
racarr_mterry: Oh wait is this mir 0.1.523:00
mterryracarr_, yes23:00
racarr_mterry: Ok sorry I got23:01
racarr_that whole part about not neeidng mir changes23:01
racarr_I made that up23:01
racarr_you need some revision from alan that does async swapping on the server side annnd23:01
racarr_mterry: and https://code.launchpad.net/~robertcarr/mir/remove-ensure-display-powered/+merge/20973423:01
racarr_so unless this is blocking anything I think the answer is try again next thursday because right now its23:02
racarr_not possible to build those combination of things against unity mir :p23:02
mterryracarr_, I just want lp:mir/devel and all extant branches to land in trusty right now23:04
racarr_ok well hopefully things will land then we will revisit htis next week23:05
mterryhmm23:08
mterryMaybe I need to try to rebuild Mir world again23:09
racarr_unity-mir wont build against23:09
racarr_mir never than 0.1523:09
racarr_newer23:09
mterryracarr_, there aren't branches for it?23:09
racarr_uh there seem to be branches for23:09
racarr_0.1.623:09
racarr_but not for devel23:09
racarr_0.1.6 + my branch23:09
racarr_might get you what you need23:09
kduband... lp:mir/devel has broken api with 0.1.623:10
racarr_right23:10
racarr_devel + unity mir is no go23:10
* mterry gets grumpy23:11
racarr_mterry: XD me too23:11
racarr_which is why I forgot that you needed mir changes too23:11
racarr_because I couldnt get unity-mir to build to make the changes then was like23:11
racarr_oh I can just use 0.1.523:11
racarr_then forgot that we actually need to run them too23:12
racarr_21:27 < kgunn> racarr: for unity-mir against dev branch tip it'd be23:12
racarr_21:28 < kgunn> basically every mp "ready to land" here https://code.launchpad.net/unity-mir/+activereviews23:12
racarr_21:28 < kgunn> in specific order its23:12
racarr_21:29 < kgunn> https://code.launchpad.net/~albaguirre/unity-mir/cross-compile-link-fix/+merge/20569023:12
racarr_21:29 < kgunn> https://code.launchpad.net/~albaguirre/unity-mir/add-cross-compile-scripts/+merge/20605823:12
racarr_21:29 < kgunn> https://code.launchpad.net/~andreas-pokorny/unity-mir/fix-1240400/+merge/20730223:12
racarr_21:29 < kgunn> https://code.launchpad.net/~andreas-pokorny/unity-mir/fix-1281075/+merge/20737523:12
racarr_21:29 < kgunn> https://code.launchpad.net/~mterry/unity-mir/drop-dbusscreen/+merge/20223623:12
racarr_21:29 < kgunn> https://code.launchpad.net/~mzanetti/unity-mir/expose-appimage-sourceSize/+merge/20762623:12
racarr_21:29 < kgunn> https://code.launchpad.net/~gerboland/unity-mir/namespace-to-prevent-collision/+merge/20864523:12
racarr_21:29 < kgunn> https://code.launchpad.net/~kgunn72/unity-mir/um-mir-0.1.6/+merge/20881923:12
racarr_this wont work against dev branch tip lol23:12
* kdub gets grumpy too23:12
racarr_but should against 0.1.623:13
racarr_...lol yeah23:13
racarr_Why did unity-mir ABI updates start landing23:13
racarr_out of sync with23:13
racarr_mir changes23:13
kgunnracarr_: what do you mean ?23:13
kgunnbecause we have a dev-branch for mir23:14
kgunnbut things w/o a dev branch cannot merge stuff that doesn't get to archive23:14
kgunnmake sense ?23:14
racarr_maybe we should have a dev branch23:14
racarr_for unity-mir too23:14
kgunnagreed23:14
racarr_that is always in sync with mir/devel23:14
racarr_I was thinking about this last night23:14
kgunncause afaik you're missing one MP23:15
racarr_and then again with glmark 223:15
racarr_today23:15
* kdub likes that idea23:15
racarr_and we should have a list of projects that depend on mirclient/mirserver ABI23:15
racarr_and they should all always have a branch that builds against devel23:15
racarr_and if you submit a mir branch or example23:15
racarr_that breaks unity-mir build23:15
racarr_you have to point the mir CI at a branch that23:15
racarr_fixes unity-mir build23:15
kgunnracarr_: i don't disagree...but its the mir teams responsibility to make that happen23:15
kgunnif we break abi we should either work with someone or do the job ourself23:16
racarr_right what I am saying is it needs23:16
racarr_to be part of continuous integration reviews23:16
racarr_because we cant keep track of it23:16
racarr_and we need the dev branch so23:17
racarr_we can always make the ABI change/update right after one another23:17
racarr_rather than end up with a list of buffered ABI changes like now23:17
racarr_because now there are unbuildable combinations of branches23:17
racarr_The thing is23:19
racarr_we can never land the changes exactly concurrently obviously because23:19
racarr_unity-mir CI will depend on the new version of mir being published23:19
racarr_if we had no dev branch, and didnt have the problem of CI getting choked up around difficult...times23:20
racarr_then its no problem, mir ABI breaks land and are published, unity-mir immediately follows, everything moves out of proposed23:21
racarr_Err...I kind of lost track of my conclusion there lol23:21
racarr_trying to head back to: We need a dev branch for unity-mir too23:21
kgunnracarr_: if you want...there are 3 branches here23:21
kgunncalled stage23:21
kgunnd23:21
kgunnin the names23:21
kgunnfor each platform-api, usc, unity-mir23:21
kgunnthey should all build together for tip....23:21
kgunnas i already merged all the various MP's in23:22
kgunnand i got all but unity-mir to build for tip23:22
racarr_kgunn: Excellent :D23:22
racarr_so what Ia m saying, is 1. Those branches should always exist23:22
racarr_2. if a mir commit causes those branches to fail to build23:22
racarr_the commit description or something has to include a link to branches that fix it23:22
kdubor, better yet, debian/changelog23:23
kgunnsure...and i don't think its that hard to figure out when abi is broken for server23:23
kgunnits not magic23:23
kgunni mean to me, before even landing a break...every individual on the team should have a matching MP23:23
racarr_Well, rather than trying23:23
kgunnon the relevant projects23:23
racarr_to detect ABI breaks23:24
racarr_lets just literally23:24
racarr_build the dependent projects23:24
kgunnsure23:24
racarr_with the fixing branch merged in you cant23:24
racarr_merge the fixing branch at first but we can have some sort of23:24
racarr_special syntax jenkins bot understands like23:24
kgunnracarr_: there are some nuancess....23:24
kgunnlike, stupid ppa builders...do not today permit running of our android integration tests23:24
racarr_<fixes unity-mir-build> lp:~santaclaus/unity-mir/update-abi23:25
kgunnthey always fail23:25
racarr_Even without testing23:25
kgunnso literally the mir staged branch is mir-dev w/ android integraion tests disabeld23:25
racarr_it still buys us a lot23:25
racarr_we can make it impossible to land a mir commit without23:25
racarr_having an open proposal to fix23:25
racarr_all dependent projects which are broken23:26
racarr_at least in terms of23:26
racarr_building23:26
kgunnso we could simply add that to the jenkins job today could we not ?23:26
kdubkgunn, maybe i can work on getting those tests disabled if no hardware is detected23:26
racarr_kgunn: I think almost.23:26
kgunnkdub: that would mean we could use lp:mir/devel so yeah that'd be awesome23:26
racarr_We will have to play with the jenkins bot to get it to23:26
racarr_1. Understand some list we place somewhere of23:27
racarr_dependent projects23:27
kdubkgunn, I'll see what can be done23:27
racarr_I mean I guess its a new job23:27
racarr_that 1. understands the list of dependent projects. 2. understands this special syntax for specifying branches that fix ABI breaks23:27
racarr_3. Builds everything23:27
racarr_so, probably not today23:28
racarr_but in the scope of a few days espescially if someone knows what they are doing with jenkins23:28
kgunnracarr_: acutally what you are describing is a dedicated build silo from the "ci train" process that didrocks/asac own23:28
racarr_more than me lol23:28
kgunnand i've been asking for this...23:28
racarr_You lost me23:28
racarr_build silo = ?? ci train = ??23:29
kgunnracarr_: basically ci train silo is what i take all the mp's dump them into ...and then it builds matching packages across all the projects23:29
racarr_ok yes something like that23:29
kgunnracarr_: output from a build silo https://launchpad.net/~ci-train-ppa-service/+archive/landing-00323:29
racarr_except integrated in to Mir devel23:29
racarr_continuous integration23:29
kgunnbut its more flexible than recipes23:29
racarr_which requires the mir dependent branches to have devel branches too23:29
kgunnbut not as good as our ci on mir-devel23:30
racarr_and this little syntax to tell jenkins which branches to use23:30
racarr_but produces the same result23:30
racarr_I just want to find a way to get it part of our23:30
racarr_mir devel CI23:30
racarr_once we have a job like this as part of Mir CI too23:31
racarr_we can start adding integration tests23:31
racarr_like you cant land on mir devel if it breaks a Qt window from opening23:31
racarr_or we can run autopilot tests on the produced set, etc23:32
racarr_tbh this whole CI bit I am talking about is23:33
racarr_really step 2.23:33
racarr_just step 1 maintaining a staged/devel branch for unity-mir, platform API, etc.23:33
racarr_I think could help us a lot23:33
racarr_because right now when someone from the shell team needs a mir change you often cant make it against mir devel (so you lose other recent changes)23:34
racarr_or you have to wait a long time on test turn around23:35
racarr_kgunn: Sorry haha hope it doesnt seem I am ranting at you23:35
racarr_I am just brainstorming23:35
racarr_kgunn: If you want I can take some time to try and23:41
racarr_rig something up23:41
racarr_I am kind of waiting on Alan to finish studyibg things to move forward on QML compositor stuff23:41
racarr_so mustly just picking up odd bits and ends atm23:41
kgunnyes...please23:46
kgunnit would help immensely23:46
kgunnfeel free to harnes the staging ppa with the branches...if you want23:46

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