/srv/irclogs.ubuntu.com/2016/11/15/#ubuntu-mir.txt

=== chihchun_afk is now known as chihchun
fritschbschaefer: perfect05:46
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
om26erbug 1641901 says Hi!10:12
ubot5`bug 1641901 in MirAL "Need surface geometry for autopilot" [Undecided,New] https://launchpad.net/bugs/164190110:12
alan_gom26er: mir_debug_surface_coords_to_screen() waves back10:26
om26eralan_g: hm, that looks like a bool[1], I presume it requires some parameters that we are actually trying to determine. [1] http://bazaar.launchpad.net/~mir-team/mir/development-branch/view/head:/src/client/mir_debug_api.cpp#L3410:31
alan_ghttps://bazaar.launchpad.net/~mir-team/mir/development-branch/view/head:/include/client/mir_toolkit/debug/surface.h#L4810:33
alan_gom26er: the bool tells you whether this extension has been enabled by the server.10:38
=== JanC is now known as Guest44518
=== JanC_ is now known as JanC
alan_ggreyback: any likelyhood of miral reviews today?11:34
greybackalan_g: sure11:34
om26eralan_g: re your comment on the bug report. "Mir-0.25 provides a server-side API that allows this debug extension to be enabled dynamically. E.g. by Unity8"11:57
om26erDo you think it would make more sense to keep this feature one layer above unity8 ? think of the case of a Kiosk-like environment where unity8 is not the server. That way we can enable autopilot to work on more environments.11:57
om26erthat brings me to question, will starting `unity8 --debug` enable the debug extensions ?12:00
alan_gom26er: *any* Mir server will respect the "debug" option which can be supplied at startup via commandline, environment variable or config file. The server-side API is available in Mir-0.25 & MirAL-0.5, but needs a mechanism for the server to call it12:01
alan_gIn short, yes "unity8 --debug" will work12:02
alan_gI understand from greyback that there's a desire to do this dynamically (via dbus) for some test scenarios12:03
alan_gA kiosk implementation probably doesn't need dbus.12:03
alan_ggreyback: thanks for the reviews. (ALL approved!?)12:12
greybackalan_g: you did good :)12:12
greybackalan_g: I only questioned if "debug extensions" should be plural12:13
greybackwhich is a nitpick of the highest order12:13
alan_gbut a perfectly good one12:13
alan_gTo me it sounds better (and future proof) with the "s", but multiple APIs could be "an extension" so I see your point12:16
greybacknot something worth blocking on12:21
=== chrisccoulson_ is now known as chrisccoulson
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|afk
=== chihchun is now known as chihchun_afk
=== dandrader|afk is now known as dandrader
=== JanC_ is now known as JanC
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader

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