[PATCH libinput] Remove LIBINPUT_EVENT_TABLET_PAD_MODE event

2016-07-07 Thread Peter Hutterer
Unimplemented and it wasn't supposed to be in the series. https://lists.freedesktop.org/archives/wayland-devel/2016-June/029376.html Signed-off-by: Peter Hutterer --- src/libinput.c | 2 -- src/libinput.h | 13 - test/litest.c | 3 ---

Re: [PATCH v4 wayland-protocols 4/4] tablet: Add pad support to the tablet protocol

2016-07-07 Thread Jason Gerecke
On 06/30/2016 05:16 AM, Jonas Ådahl wrote: > On Thu, Jun 30, 2016 at 01:10:33PM +0200, Carlos Garnacho wrote: >> Hi!, >> >> On Thu, Jun 30, 2016 at 6:27 AM, Jonas Ådahl wrote: >>> On Tue, Jun 28, 2016 at 11:21:50PM +0200, Carlos Garnacho wrote: The pad's interface is

Re: [PATCH weston] config-parser: Improve error checks for strtol/strtoul calls

2016-07-07 Thread Bryce Harrington
On Thu, Jul 07, 2016 at 02:08:28PM -0700, Bryce Harrington wrote: > Check errno, which is set of over/underflow, out of range, etc. Also > check for empty strings (the usages covered in this patch already also > cover the case where there are non-digits present). Set errno to 0 > before making

[PATCH weston] config-parser: Improve error checks for strtol/strtoul calls

2016-07-07 Thread Bryce Harrington
Check errno, which is set of over/underflow, out of range, etc. Also check for empty strings (the usages covered in this patch already also cover the case where there are non-digits present). Set errno to 0 before making the strto*l call in case of pre-existing errors (i.e. ENOTTY when running

Re: [PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

2016-07-07 Thread Quentin Glidic
On 07/07/2016 18:28, Emil Velikov wrote: On 7 July 2016 at 10:20, Pekka Paalanen wrote: [snip] Therefore a NAK from me too. As you guys wish. In that case can we drop the pkgincludedir variable ? Most packages don't bother with it (on my local setup only 7 out of 740 do)

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

2016-07-07 Thread Quentin Glidic
On 07/07/2016 18:11, Emil Velikov wrote: On 7 July 2016 at 10:05, Pekka Paalanen wrote: >> [snip] Now that you mentioned the semantics could be of upper or lower limit, the name should imply the meaning. I only thought of using it as both lower limit (as in pkg-config

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

2016-07-07 Thread Thiago Macieira
On segunda-feira, 4 de julho de 2016 15:23:51 PDT Emil Velikov wrote: > +Similar approach is used by ATK, QT and KDE programs/libraries, Qt, with a lowercase t. -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

[PATCH wayland-protocols] Fix grammar for 'an X*'

2016-07-07 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- unstable/input-method/input-method-unstable-v1.xml | 2 +- unstable/text-input/text-input-unstable-v1.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/unstable/input-method/input-method-unstable-v1.xml

Re: [PATCH weston] xwayland: Grammar fixes

2016-07-07 Thread Bryce Harrington
On Thu, Jul 07, 2016 at 01:16:38PM +0100, Eric Engestrom wrote: > On Wed, Jul 06, 2016 at 03:25:19PM -0700, Bryce Harrington wrote: > > Signed-off-by: Bryce Harrington > > --- > > xwayland/xwayland-api.h | 16 > > 1 file changed, 8 insertions(+), 8

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

2016-07-07 Thread Emil Velikov
On 7 July 2016 at 10:46, Pekka Paalanen wrote: > On Mon, 4 Jul 2016 16:25:54 +0100 > Emil Velikov wrote: > >> On 4 July 2016 at 15:35, Quentin Glidic >> wrote: >> > On 04/07/2016 16:23, Emil Velikov wrote: >> >> >>

Re: [PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

2016-07-07 Thread Emil Velikov
On 7 July 2016 at 10:20, Pekka Paalanen wrote: > On Mon, 4 Jul 2016 16:02:23 +0100 > Emil Velikov wrote: > >> On 4 July 2016 at 15:32, Quentin Glidic >> wrote: >> > On 04/07/2016 16:23, Emil Velikov wrote: >> >> >>

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

2016-07-07 Thread Emil Velikov
On 7 July 2016 at 10:05, Pekka Paalanen wrote: > On Wed, 6 Jul 2016 13:57:48 +0100 > Emil Velikov wrote: > >> On 5 July 2016 at 15:46, Pekka Paalanen wrote: >> > On Mon, 4 Jul 2016 15:23:51 +0100 >> > Emil Velikov

[PATCH weston] weston-editor: Close the data source after sending

2016-07-07 Thread Derek Foreman
We're leaking the fd when sending cut'n'paste. Failure to close can also makes the other end unhappy because it doesn't know the paste is finished. Signed-off-by: Derek Foreman --- clients/editor.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/clients/editor.c

RE: [PATCH weston] libweston: Implement wl_output version 3 (release request)

2016-07-07 Thread Ucan, Emre (ADITG/SW1)
Hi, It looks good. Reviewed-by: Emre Ucan Best regards Emre Ucan Software Group I (ADITG/SW1) Tel. +49 5121 49 6937 > -Original Message- > From: wayland-devel [mailto:wayland-devel- > boun...@lists.freedesktop.org] On Behalf Of Quentin Glidic > Sent:

Re: [PATCH weston] xwayland: Include missing config.h

2016-07-07 Thread Eric Engestrom
On Wed, Jul 06, 2016 at 03:14:20PM -0700, Bryce Harrington wrote: > Signed-off-by: Bryce Harrington Indeed, this is needed for XSERVER_PATH Reviewed-by: Eric Engestrom > --- > compositor/xwayland.c | 2 ++ > 1 file changed, 2 insertions(+) >

Re: [PATCH weston] xwayland: Cleanup error message on spawn failure Signed-off-by: Bryce Harrington <br...@osg.samsung.com>

2016-07-07 Thread Eric Engestrom
On Wed, Jul 06, 2016 at 03:18:46PM -0700, Bryce Harrington wrote: > Signed-off-by: Bryce Harrington Reviewed-by: Eric Engestrom > --- > compositor/xwayland.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH weston] xwayland: Grammar fixes

2016-07-07 Thread Eric Engestrom
On Wed, Jul 06, 2016 at 03:25:19PM -0700, Bryce Harrington wrote: > Signed-off-by: Bryce Harrington > --- > xwayland/xwayland-api.h | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/xwayland/xwayland-api.h b/xwayland/xwayland-api.h

Re: [PATCH weston] dmabuf: Fix grammar in a comment

2016-07-07 Thread Eric Engestrom
On Wed, Jul 06, 2016 at 03:30:54PM -0700, Bryce Harrington wrote: > Signed-off-by: Bryce Harrington Reviewed-by: Eric Engestrom > --- > libweston/linux-dmabuf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[PATCH libinput 2/2] test: remove NOTPARALLEL

2016-07-07 Thread Peter Hutterer
We can now run the tests in parallel with make check -j , even though results are mixed: on an i7 here make -j 4 is reliable, on a under-resourced VM running on an i5 make -j2 is the best I can do, anything higher fails with timeouts possibly because of the udev lock contention (see

[PATCH libinput 1/2] test: use the .test suffix instead of a test- prefix

2016-07-07 Thread Peter Hutterer
Having a suffix means we can use some automake features that rely on the suffix Signed-off-by: Peter Hutterer --- .gitignore | 2 +- test/Makefile.am | 202 +++ 2 files changed, 102 insertions(+), 102

Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

2016-07-07 Thread Pekka Paalanen
On Mon, 4 Jul 2016 16:25:54 +0100 Emil Velikov wrote: > On 4 July 2016 at 15:35, Quentin Glidic > wrote: > > On 04/07/2016 16:23, Emil Velikov wrote: > >> > >> From: Emil Velikov > >> > >> Signed-off-by:

Re: [PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

2016-07-07 Thread Pekka Paalanen
On Mon, 4 Jul 2016 16:02:23 +0100 Emil Velikov wrote: > On 4 July 2016 at 15:32, Quentin Glidic > wrote: > > On 04/07/2016 16:23, Emil Velikov wrote: > >> > >> From: Emil Velikov > >> > >> When managing

Patch revision logs in commit messages?

2016-07-07 Thread Pekka Paalanen
On Mon, 4 Jul 2016 17:54:39 +0100 Emil Velikov wrote: > On 4 July 2016 at 15:45, Quentin Glidic > wrote: > > On 04/07/2016 16:23, Emil Velikov wrote: > >> > >> Use the documented libweston-$major.so.0.$minor.$patch scheme. > >> > >>

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

2016-07-07 Thread Pekka Paalanen
On Wed, 6 Jul 2016 13:57:48 +0100 Emil Velikov wrote: > On 5 July 2016 at 15:46, Pekka Paalanen wrote: > > On Mon, 4 Jul 2016 15:23:51 +0100 > > Emil Velikov wrote: > > > >> From: Emil Velikov

Re: [PATCH weston] libweston: Position layers in an absolute way

2016-07-07 Thread Pekka Paalanen
On Tue, 5 Jul 2016 11:26:52 +0200 Quentin Glidic wrote: > On 05/07/2016 11:09, Giulio Camuffo wrote: > > 2016-07-05 10:15 GMT+02:00 Quentin Glidic > > : > >> From: Quentin Glidic > >> > >>