Re: [PATCH] Fix mutex hang in colord on output removal

2015-01-20 Thread Bryce Harrington
On Mon, Jan 19, 2015 at 10:39:28AM +, Richard Hughes wrote: > On 15 January 2015 at 14:40, Olivier Fourdan wrote: > > "ocms" is taken from the container now (that was the casting error), so no > > need for the lookup from cms as we already have it. > > Ohh of course, makes sense. > > Signed-

Re: [PATCH weston] text: Fix typo

2015-01-20 Thread Bryce Harrington
On Tue, Jan 20, 2015 at 05:26:16PM -0800, Bryce Harrington wrote: > On Fri, Jan 16, 2015 at 05:47:10PM -0600, Derek Foreman wrote: > > Signed-off-by: Derek Foreman > > --- > > protocol/input-method.xml | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/protocol/input-

Re: [PATCH weston] build: install a login manager session entry

2015-01-20 Thread Bryce Harrington
On Sun, Dec 28, 2014 at 02:20:49PM +0100, Lubomir Rintel wrote: > This makes it possible to run Weston session from GDM (and possibly > other login managers). > --- > Makefile.am| 5 + > src/weston.desktop | 5 + > 2 files changed, 10 insertions(+) > create mode 100644 src/weston.

Re: [weston, v2, 20/20] compositor-drm: use weston_view_to_output_matrix() to test plane viability

2015-01-20 Thread Bryce Harrington
[Just adding a note for patchwork tracking purposes...] This patchset has been in the patch tracker for some time; in chatting with Derek and Pekka, landing of this was deferred as it requires test cases using the headless renderer (bug #83989). But that bug is blocked on bug #83987 (screenshoote

[PATCH libinput] tablet: Add a left handed mode and tests

2015-01-20 Thread Stephen Chandler Paul
On the majority of Wacom tablets, the buttons are on the left side, opposite of the side where the palm is meant to rest. Because of this, it's impossible to use the tablet with your left hand (comfortably, anyway) unless you flip it over, in which case the coordinates need to be inverted for it to

Re: [PATCH weston] zoom: Check the value of level before using it.

2015-01-20 Thread Bryce Harrington
On Tue, Jan 20, 2015 at 02:00:04PM -0600, Derek Foreman wrote: > Nice catch. Zooming with mod+scrollwheel will divide by zero a few times > at the start of zoom without this patch. > > > Reviewed-by: Derek Foreman Agreed, nice clean fix. Pushed to trunk. 553895e..1a873aa master -> master

Re: [PATCH weston] text: Fix typo

2015-01-20 Thread Bryce Harrington
On Fri, Jan 16, 2015 at 05:47:10PM -0600, Derek Foreman wrote: > Signed-off-by: Derek Foreman > --- > protocol/input-method.xml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/protocol/input-method.xml b/protocol/input-method.xml > index 6961c5a..2f596f0 100644 > --- a/p

Re: [PATCH weston 3/6] compositor: set presentation.presented flags

2015-01-20 Thread Bryce Harrington
On Tue, Jan 20, 2015 at 10:11:56AM +0200, Pekka Paalanen wrote: > On Fri, 16 Jan 2015 08:48:21 +0100 > Mario Kleiner wrote: > > > Hi Pekka, > > > > as you know from our off-list conversation i've spent quite a bit of > > time the > > last week reviewing and testing this patch series (patches 1-

[PATCH 4/4] compositor: Add --config-overrides command-line option

2015-01-20 Thread Bryce Harrington
From: "Bryce Harrington" This provides a way to run weston with a modified configuration, such as for testing purposes. Signed-off-by: Bryce Harrington (http://osg.samsung.com) Signed-off-by: Bryce Harrington --- man/weston.man | 9 + src/compositor.c | 24 +++-

[PATCH 1/4] config-parser: Document config_section_get_entry()

2015-01-20 Thread Bryce Harrington
From: "Bryce Harrington" Signed-off-by: Bryce Harrington (http://osg.samsung.com) Signed-off-by: Bryce Harrington --- shared/config-parser.c | 21 + 1 file changed, 21 insertions(+) diff --git a/shared/config-parser.c b/shared/config-parser.c index 4542ca6..b72cb77 100644

[PATCH 3/4] config-parser: Enable updating an already-loaded config

2015-01-20 Thread Bryce Harrington
From: "Bryce Harrington" Adds weston_config_update(), which parses a condensed string of comma-delimited key/value pairs and adds or changes the corresponding parameters in the weston_config structure. Signed-off-by: Bryce Harrington (http://osg.samsung.com) Signed-off-by: Bryce Harrington ---

[PATCH 2/4] config-parser: Allow server-side modification of loaded configs

2015-01-20 Thread Bryce Harrington
From: "Bryce Harrington" Adds a setter, weston_config_section_set(), for changing key values in the loaded copy of the weston.ini configuration. This functionality is not exposed to clients. Changes are not saved to disk. Signed-off-by: Bryce Harrington (http://osg.samsung.com) Signed-off-by:

[PATCH 0/4] Enable configuration tweaking

2015-01-20 Thread Bryce Harrington
For testing, currently we can run tests only against the configuration defaults, but we need a way to launch weston with non-default config parameters. An example would be to turn off the fade-in animation for rendering tests. Or, we may want to test the configuration system itself. This patchse

Re: [PATCH weston v2 1/2] releasing: Typo

2015-01-20 Thread Derek Foreman
I think we can call this obviously correct. Reviewed-by: Derek Foreman On 09/01/15 08:09 PM, Bryce Harrington wrote: > Signed-off-by: Bryce Harrington > --- > releasing.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/releasing.txt b/releasing.txt > index 2db84d5..

Looking for Speaker in the Raleigh, NC, USA area

2015-01-20 Thread Jack Hill
Hi all, I'm on the steering committee for my local free software users group, TriLUG . A talk about Wayland would be of interest to the group. Is there anyone local to Raleigh, North Carolina or who would be willing to travel who would interested in presenting? The format is

Re: [PATCH weston] zoom: Check the value of level before using it.

2015-01-20 Thread Derek Foreman
Nice catch. Zooming with mod+scrollwheel will divide by zero a few times at the start of zoom without this patch. Reviewed-by: Derek Foreman On 17/01/15 01:31 PM, Carlos Olmedo Escobar wrote: > Check the value of level before dividing 1 by it. > > Signed-off-by: Carlos Olmedo Escobar > --- >

Re: [PATCH weston 6/6] zoom: Change how we select a zoom location

2015-01-20 Thread Bill Spitzak
On 01/20/2015 07:44 AM, Derek Foreman wrote: This would have to be changed to only zoom the output containing the center of the current keyboard focus in my proposal, but I kind of feel that would be better anyway. Why is that better? I don't really have much of an opinion one way or the ot

[PATCH weston] tests: Skip buffer count test if wayland EGL extension isn't present

2015-01-20 Thread Derek Foreman
This also skips the test when running on the headless backend. Signed-off-by: Derek Foreman --- tests/buffer-count-test.c | 5 + 1 file changed, 5 insertions(+) diff --git a/tests/buffer-count-test.c b/tests/buffer-count-test.c index 5985a9e..3dd6e1a 100644 --- a/tests/buffer-count-test.c +

Re: [PATCH weston 6/6] zoom: Change how we select a zoom location

2015-01-20 Thread Derek Foreman
On 19/01/15 07:24 PM, Bill Spitzak wrote: > This seems excessively complicated compared to a version that just > figures out an x/y pair to use instead of seat->pointer->x/y and leaves > the rest of the code alone. It's also more complicated than a patch that just removes the keybind for zoom in/z

Re: [PATCH 1/7] protocol: add linux_dmabuf extension RFCv1

2015-01-20 Thread Pekka Paalanen
On Tue, 20 Jan 2015 07:41:52 +0100 Daniel Vetter wrote: > On Thu, Jan 08, 2015 at 02:26:00PM +0200, Pekka Paalanen wrote: > > On Mon, 5 Jan 2015 11:44:49 +0100 > > Daniel Vetter wrote: > > > > > But as long as you don't do crazy things the drm dma-buf import will > > > notice that it's the sam

Re: [PATCH weston 3/6] compositor: set presentation.presented flags

2015-01-20 Thread Pekka Paalanen
On Fri, 16 Jan 2015 08:48:21 +0100 Mario Kleiner wrote: > Hi Pekka, > > as you know from our off-list conversation i've spent quite a bit of > time the > last week reviewing and testing this patch series (patches 1-6). I > implemented > a first basic usable Waylang backend into my own software