Hello Pekka/Daniel, As the current review comments are not related with the design of weston-debug protocol, can we merge this protocol as having the support only for human-readable log streams. Later the dual-mode framework to support both human-readable and machine-readable output can be supported.
Best regards, Devadoss Maniraj RBEI/ECF3 Tel. +91 80 6136-4382 -----Original Message----- From: wayland-devel [mailto:[email protected]] On Behalf Of Pekka Paalanen Sent: Monday, October 23, 2017 5:32 PM To: [email protected] Cc: Ucan Ahmet Emre (ADITG/ESB) <[email protected]> Subject: Re: [PATCH weston v4 0/9] weston-debug API and tool On Mon, 23 Oct 2017 10:18:33 +0300 Pekka Paalanen <[email protected]> wrote: > On Thu, 12 Oct 2017 13:13:40 +0200 > Emre Ucan <[email protected]> wrote: > > > This series of patches are from weston debug protocol RFC posted by > > Pekka Paalanen <pq at iki.fi> along with the modifications done by Maniraj > > Devadoss <Maniraj.Devadoss at in.bosch.com>. > > > > https://lists.freedesktop.org/archives/wayland-devel/2017-June/03422 > > 7.html > > v3: > > 1. protocol is renamed to weston-debug.xml 2. > > libweston-3-protocols-pc is installed to sysroot > > > > Hi, > > I am happy with the series and patches 2-9 have my R-b. > > I had some questions about the autotools stuff in patch 1, but I don't > think that alone causes a need to re-send, I can just do the > adjustments when merging. > > I would like to aim to land this series tomorrow. > > In the mean time, it would be awesome to get one more acked-by, but I > don't think I'll wait for it unless someone disagrees. Hi, given the review comments, I will not be pushing this series. If people want to continue the work, the latest cleaned up series is at: https://gitlab.collabora.com/pq/weston/commits/weston-debug-v4.3 Thanks, pq _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
