[PATCH weston v4 0/9] weston-debug API and tool

[Pekka Paalanen]

On Thu, 12 Oct 2017 13:13:40 +0200
Emre Ucan <eucan at de.adit-jv.com> 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/034227.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.


Thanks,
pq

> Pekka Paalanen (9):
>   protocol: add weston-debug.xml
>   libweston: add weston_debug API and implementation
>   compositor: add option to enable weston_debug
>   clients: add weston-debug
>   compositor: offer logs via weston-debug
>   xwm: dump_property() to use FILE internally
>   xwm: move FILE to the callers of dump_property()
>   xwm: convert WM_DEBUG into a weston-debug scope
>   compositor: protocol logger
> 
>  Makefile.am                         |  28 +-
>  clients/weston-debug.c              | 378 ++++++++++++++++++
>  compositor/main.c                   | 155 ++++++++
>  configure.ac                        |   1 +
>  libweston/compositor.c              |   5 +
>  libweston/compositor.h              |   8 +
>  libweston/libweston-protocols.pc.in |   7 +
>  libweston/weston-debug.c            | 755 ++++++++++++++++++++++++++++++++++++
>  libweston/weston-debug.h            | 107 +++++
>  man/weston-debug.man                |  46 +++
>  man/weston.man                      |  15 +-
>  protocol/weston-debug.xml           | 128 ++++++
>  xwayland/launcher.c                 |   7 +
>  xwayland/selection.c                |  39 +-
>  xwayland/window-manager.c           | 233 +++++------
>  xwayland/xwayland.h                 |   6 +-
>  16 files changed, 1798 insertions(+), 120 deletions(-)
>  create mode 100644 clients/weston-debug.c
>  create mode 100644 libweston/libweston-protocols.pc.in
>  create mode 100644 libweston/weston-debug.c
>  create mode 100644 libweston/weston-debug.h
>  create mode 100644 man/weston-debug.man
>  create mode 100644 protocol/weston-debug.xml
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20171023/3c41d479/attachment.sig>