Re: [RFC] [tests] Running the wayland tests against compositor-headless

2013-04-22 Thread Sam Spilsbury
On Mon, Apr 22, 2013 at 11:02 PM, Eoff, Ullysses A wrote: >> -Original Message- >> From: Sam Spilsbury [mailto:smspil...@gmail.com] >>> >>> >>> > > Yes, it would be trivial to enable the input-based tests to run on the > headless backend... however, it's unclear to me whether the headl

Re: [PATCH 1/3] Extract the EDID blob when adding a DRM output

2013-04-22 Thread David Herrmann
Hi On Apr 23, 2013 6:38 AM, "Graeme Gill" wrote: > > David Herrmann wrote: > > > If you want to keep the EDID blob, could you describe a specific > > use-case? > > Taking a hash of it to identify a particular display. > > Implementing specific workarounds for displays where > the built in parser

Re: [PATCH 1/3] Extract the EDID blob when adding a DRM output

2013-04-22 Thread Graeme Gill
David Herrmann wrote: > If you want to keep the EDID blob, could you describe a specific > use-case? Taking a hash of it to identify a particular display. Implementing specific workarounds for displays where the built in parser gives up. Dealing with extensions to the EDID standard. > Patch 3/

Pointer lock and warping

2013-04-22 Thread Bill Spitzak
Thinking about Todd Showalter's requests for pointer warping, I think in fact a correctly-behaving Wayland app will always want pointer warping and incremental update when the mouse is held down. This would require a major change in the mouse api, though I think the differences in back-compatib

Re: [RFC] [tests] Running the wayland tests against compositor-headless

2013-04-22 Thread Sam Spilsbury
> So - shall I create and submit some patches to get the tests working > on headless? I suppose this is a desirable thing right? > Ah, to clarify this point - I meant submitting patches which would allow the headless backend to be used in testing, as opposed to just x11 and drm. > Best, > > Sam >

Re: [RFC] [tests] Running the wayland tests against compositor-headless

2013-04-22 Thread Sam Spilsbury
On Mon, Apr 22, 2013 at 3:48 PM, Eoff, Ullysses A wrote: >> -Original Message- >> From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org >> [mailto:wayland-devel- >> bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Eoff, >> Ullysses A >> Sent: Mond

RE: [RFC] [tests] Running the wayland tests against compositor-headless

2013-04-22 Thread Eoff, Ullysses A
> -Original Message- > From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org > [mailto:wayland-devel- > bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Eoff, > Ullysses A > Sent: Monday, April 22, 2013 1:24 PM > To: Sam Spilsbury; wayland > Subjec

RE: [RFC] [tests] Running the wayland tests against compositor-headless

2013-04-22 Thread Eoff, Ullysses A
Changing the tests to run headless is probably not necessary... You can create your own custom "check" script if weston-test-env does not suite your needs. To use it with "make check", just set/export the LOG_COMPILER environment variable to refer to that script before running "make check". "mak

Re: Input and games.

2013-04-22 Thread Todd Showalter
On Mon, Apr 22, 2013 at 1:40 PM, Pekka Paalanen wrote: >> Gamepads, by contrast, are all mostly the same these days, much >> like mice. You can find oddball ones like that PC gamepad that was up >> on Kickstarter recently which had a trackball in place of the right >> thumb stick, but the co

[RFC] [tests] Running the wayland tests against compositor-headless

2013-04-22 Thread Sam Spilsbury
Hi, I'm not sure if this is a work-item that's already in-progress by somebody. I noticed that weston has a "headless" compositor backend and a "no-op" renderer, which effectively do not depend on having OpenGL or *a* windowing system available at the time that weston is run. It would be really g

Re: Input and games.

2013-04-22 Thread Pekka Paalanen
On Mon, 22 Apr 2013 10:42:35 -0400 Todd Showalter wrote: > On Mon, Apr 22, 2013 at 3:41 AM, Pekka Paalanen wrote: > > > The protocol level is not convenient for an application developer in > > some cases, and it's not even meant to be. We explicitly leave lots of > > processing for a so-called

Re: [DRM] Patch for Compositor Drm

2013-04-22 Thread Emilio Pozuelo Monfort
Hi, This is obviously correct. Thanks for the fix! Emilio On 04/22/2013 06:12 PM, Christopher Michael wrote: > From acb79e4a5921525b35e07e48f7f903e42a08fb7c Mon Sep 17 00:00:00 2001 > From: Chris Michael > Date: Mon, 22 Apr 2013 15:22:48 +0100 > Subject: [PATCH] Fix not checking return value of

[DRM] Patch for Compositor Drm

2013-04-22 Thread Christopher Michael
From acb79e4a5921525b35e07e48f7f903e42a08fb7c Mon Sep 17 00:00:00 2001 From: Chris Michael Date: Mon, 22 Apr 2013 15:22:48 +0100 Subject: [PATCH] Fix not checking return value of drmIoctl function call to map dumb buffer. in drm_fb_create_dumb, the return value of the drmIoctl function call to

Re: Input and games.

2013-04-22 Thread Todd Showalter
On Mon, Apr 22, 2013 at 3:41 AM, Pekka Paalanen wrote: > The protocol level is not convenient for an application developer in > some cases, and it's not even meant to be. We explicitly leave lots of > processing for a so-called toolkit library. At several points below I > say that something is ou

[PATCH 3/3] Parse the EDID when outputs are added

2013-04-22 Thread Richard Hughes
At the moment we're only extracting interesting strings. We have to be quite careful parsing the EDID data, as vendors like to do insane things. The original EDID parsing code was written by me for gnome-color-manager. --- src/compositor-drm.c | 166 +++

[PATCH 2/3] Add a 'serial' property on weston_output

2013-04-22 Thread Richard Hughes
--- src/compositor-drm.c | 1 + src/compositor.h | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index 61ef97e..a454676 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -1522,6 +1522,7 @@ create_output_for_connect

[PATCH 1/3] Extract the EDID blob when adding a DRM output

2013-04-22 Thread Richard Hughes
--- src/compositor-drm.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index da1ba79..61ef97e 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -139,6 +139,7 @@ struct drm_output { int pipe;

[PATCH] Extract and parse the EDID when outputs are added

2013-04-22 Thread Richard Hughes
At the moment we're only extracting interesting strings. We have to be quite careful parsing the EDID data, as vendors like to do insane things. The original EDID parsing code was written by me for gnome-color-manager. --- src/compositor-drm.c | 139 +++

[PATCH v2] Extract and parse the EDID when outputs are added

2013-04-22 Thread Richard Hughes
At the moment we're only extracting interesting strings. We have to be quite careful parsing the EDID data, as vendors like to do insane things. The original EDID parsing code was written by me for gnome-color-manager. Richard Hughes (1): Extract and parse the EDID when outputs are added src/

Re: [patches] Add a color management framework to weston

2013-04-22 Thread Pekka Paalanen
On Fri, 5 Apr 2013 14:23:50 -0500 Thomas Daede wrote: > I am not sure that doing the color conversion in the compositor is the > correct place. Some of it has to be there to support vcgt, but for > more general conversions, it gets complicated quickly. > > Color correction is most important for

Re: [PATCH] Fixed simple-egl tear-down order to prevent a crash on exit time.

2013-04-22 Thread Pekka Paalanen
On Fri, 19 Apr 2013 17:49:12 + "Yeh, Sinclair" wrote: > wl_egl_window_destory() distroys the window handle that > dri2_destroy_surface() later uses when eglTerminate() is called. > > Reordering the tear down order prevents such case from occuring. > --- > clients/simple-egl.c | 11 ++-

[PATCH] weston.ini: document background-type

2013-04-22 Thread Emilio Pozuelo Monfort
--- man/weston.ini.man |4 weston.ini |1 + 2 files changed, 5 insertions(+) diff --git a/man/weston.ini.man b/man/weston.ini.man index 8dde82c..2287730 100644 --- a/man/weston.ini.man +++ b/man/weston.ini.man @@ -109,6 +109,10 @@ The entries that can appear in this section a

Re: [PATCH weston] weston-launch: Run weston in the user login shell

2013-04-22 Thread Quentin Glidic
On 22/04/2013 10:37, Emilio Pozuelo Monfort wrote: Hi, What's the use case for this patch? Regards, Emilio Hi, Since 272d243e00c742f9cffc8bc46d09b7d0f19dc43e, weston-launch clears the env before running weston. This breaks several use cases, like a user with a custom PATH or a non-XDG_RUNT

Re: [PATCH weston] weston-launch: Run weston in the user login shell

2013-04-22 Thread Emilio Pozuelo Monfort
Hi, What's the use case for this patch? Regards, Emilio On 04/17/2013 03:04 PM, Quentin Glidic wrote: > From: Quentin Glidic > > Signed-off-by: Quentin Glidic > --- > src/weston-launch.c | 19 --- > 1 file changed, 16 insertions(+), 3 deletions(-) > > diff --git a/src/weston

Re: [PATCH weston] weston-launch: Run weston in the user login shell

2013-04-22 Thread Emilio Pozuelo Monfort
Hi, What's the use case for this patch? Regards, Emilio On 04/17/2013 03:04 PM, Quentin Glidic wrote: > From: Quentin Glidic > > Signed-off-by: Quentin Glidic > --- > src/weston-launch.c | 19 --- > 1 file changed, 16 insertions(+), 3 deletions(-) > > diff --git a/src/weston

[PATCH weston] weston-launch: Run weston in the user login shell

2013-04-22 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- src/weston-launch.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/src/weston-launch.c b/src/weston-launch.c index 64d4a8a..89c3c5a 100644 --- a/src/weston-launch.c +++ b/src/weston-launch.c @@ -60,

Re: Input and games.

2013-04-22 Thread Pekka Paalanen
Hi Todd, Jonas Kulla already replied on several items, but it's easier for me to comment on everything I have something to say, so pardon if I repeat some things. On Fri, 19 Apr 2013 12:31:19 -0400 Todd Showalter wrote: > On Fri, Apr 19, 2013 at 5:18 AM, Pekka Paalanen wrote: > > > I am goin

[PATCH weston 1/2] autotools: Add versions to wayland-* dependencies

2013-04-22 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index d5fea9d..bec8ace 100644 --- a/configure.ac +++ b/configure.ac @@ -43,7 +43,7 @@ AC_CHECK_HEADERS([execinfo.h]) AC_CHEC

[PATCH weston 2/2] weston-launch: Run weston in the user login shell

2013-04-22 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- src/weston-launch.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/src/weston-launch.c b/src/weston-launch.c index 64d4a8a..89c3c5a 100644 --- a/src/weston-launch.c +++ b/src/weston-launch.c @@ -60,