[PATCH v2] compositor: Fix buggy snprintf that sets module path

2012-05-23 Thread Chad Versace
If the MODULEDIR string contains '%', then snprintf(path, sizeof(path), MODULEDIR "/%s", name); does not do what you want. Signed-off-by: Chad Versace --- src/compositor.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compositor.c b/src/compositor.c index 3c1e058

Re: [PATCH] compositor: Fix buggy snprintf that sets module path

2012-05-23 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Oh, that's very simple. Why didn't I think of that? v2 coming soon. On 05/23/2012 11:36 PM, Dima Ryazanov wrote: > Wouldn't this be simpler? > > snprintf(path, sizeof path, "%s/%s", MODULEDIR, name); > > On Wed, May 23, 2012 at 11:32 PM, Chad Versa

Re: [PATCH] compositor: Fix buggy snprintf that sets module path

2012-05-23 Thread Dima Ryazanov
Wouldn't this be simpler? snprintf(path, sizeof path, "%s/%s", MODULEDIR, name); On Wed, May 23, 2012 at 11:32 PM, Chad Versace wrote: > If the MODULEDIR string contains '%', then >snprintf(path, sizeof(path), MODULEDIR "/%s", name); > does not do what you want. > > Fix this by replacing sn

[PATCH] compositor: Fix buggy snprintf that sets module path

2012-05-23 Thread Chad Versace
If the MODULEDIR string contains '%', then snprintf(path, sizeof(path), MODULEDIR "/%s", name); does not do what you want. Fix this by replacing snprintf with stncpy followed by strncat. Signed-off-by: Chad Versace --- src/compositor.c |8 +--- 1 file changed, 5 insertions(+), 3 del

[PATCH radeon] Fix "Unable to retrieve master" with with X backend.

2012-05-23 Thread Darxus
Fix case where, if weston was run with --xserver while using the X backend without root priveledges, it would give an "Unable to retrieve master" error and fail when it tried to launch X. --- src/radeon_kms.c | 18 +++--- 1 files changed, 11 insertions(+), 7 deletions(-) diff --git

[PATCH wlshm] Adapt to new initialisation sequence

2012-05-23 Thread Darxus
--- src/wlshm.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/src/wlshm.c b/src/wlshm.c index 8f5cf8a..91c8099 100644 --- a/src/wlshm.c +++ b/src/wlshm.c @@ -601,16 +601,17 @@ wlshm_pre_init(ScrnInfoPtr pScrn, int flags) xf86ProcessOptions(pScrn->scrnInd

[PATCH radeon] Adapt to new initialisation sequence

2012-05-23 Thread Darxus
--- src/radeon_kms.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/radeon_kms.c b/src/radeon_kms.c index 5e36d91..db3de74 100644 --- a/src/radeon_kms.c +++ b/src/radeon_kms.c @@ -797,13 +797,17 @@ Bool RADEONPreInit_KMS(ScrnInfoPtr pScrn, int flags) #ifdef

[PATCH] xwayland: change library name

2012-05-23 Thread Tiago Vignatti
Signed-off-by: Tiago Vignatti --- configure.ac | 12 ++-- src/Makefile.am |2 +- src/compositor.c |2 +- src/xwayland/Makefile.am | 14 +++--- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/configure.ac b/configure.ac inde

[PATCH 3/3] clients: desktop-shell: show tooltip for panel items

2012-05-23 Thread Tiago Vignatti
Signed-off-by: Tiago Vignatti --- clients/desktop-shell.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/clients/desktop-shell.c b/clients/desktop-shell.c index 2473f36..d497807 100644 --- a/clients/desktop-shell.c +++ b/clients/desktop-shell.c @@ -31,6 +31,7 @@ #include

[PATCH 2/3] window: add simple text tooltip handlers

2012-05-23 Thread Tiago Vignatti
Using set_transient. Signed-off-by: Tiago Vignatti --- clients/window.c | 188 ++ clients/window.h |7 ++ 2 files changed, 195 insertions(+) diff --git a/clients/window.c b/clients/window.c index 489c35a..9e9b209 100644 --- a/clients/wind

[PATCH 1/3] shell: use transient flags for activate or not new surfaces

2012-05-23 Thread Tiago Vignatti
Inactive surfaces doesn't set keyboard focus, so it can be used for tooltips, toolbars and some other type of windows. This requires protocol side changes. Signed-off-by: Tiago Vignatti --- src/shell.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/shell.c b/src/

Re: [PATCH] Weston logging functionality.

2012-05-23 Thread Pekka Paalanen
On Wed, 23 May 2012 17:21:27 +0200 Martin Minarik wrote: > In compositor, create a fprintf stderr handler for messages > from libwayland. > --- > src/compositor.c |9 + > src/compositor.h |2 ++ > 2 files changed, 11 insertions(+), 0 deletions(-) > > diff --git a/src/compositor.

[PATCH] Weston logging functionality.

2012-05-23 Thread Martin Minarik
In compositor, create a fprintf stderr handler for messages from libwayland. --- src/compositor.c |9 + src/compositor.h |2 ++ 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index 3c1e058..a8966d3 100644 --- a/src/compositor.c +

[PATCH 2/2] Wayland logging, replace printf by WL_PRINTF

2012-05-23 Thread Martin Minarik
WL_PRINTF(..) is macro to wl_log(..), takes the same parameters as regular printf() --- src/connection.c | 58 - src/event-loop.c |8 +++--- src/wayland-client.c | 20 +++- src/wayland-server.c | 26 +

[PATCH 1/2] Wayland logging functionality

2012-05-23 Thread Martin Minarik
The core libwayland libraries should not handle logging, only passing the error messages to subscribed functions. An application linked to libwayland-server or libwayland-client will be able to set own functions (one per library) to handle error messages. --- src/wayland-client.c |6 ++ s

Re: [RFC] [PATCH 0/3] libwayland-cursor

2012-05-23 Thread Kristian Høgsberg
On Wed, May 23, 2012 at 04:19:34PM +0300, Ander Conselvan de Oliveira wrote: > On 05/22/2012 06:25 PM, Kristian Høgsberg wrote: > >On Tue, May 22, 2012 at 03:39:39PM +0300, Ander Conselvan de Oliveira wrote: > >>Hi, > >> > >>In order to get rid of the X11 dependency that libXcursor brings, > >>Kris

Re: [PATCH wayland] shm: use mremap on pool resize

2012-05-23 Thread Kristian Høgsberg
On Wed, May 23, 2012 at 04:09:55PM +0300, Ander Conselvan de Oliveira wrote: Thanks, Kristian > --- > src/wayland-shm.c | 12 +--- > 1 files changed, 5 insertions(+), 7 deletions(-) > > diff --git a/src/wayland-shm.c b/src/wayland-shm.c > index e071392..62afc3b 100644 > --- a/src/wayl

Re: [PATCH 1/2] xwayland: Automatically load module if wayland support requested

2012-05-23 Thread Kristian Høgsberg
On Wed, May 23, 2012 at 10:05:53PM +1000, Christopher James Halse Rogers wrote: > This makes life a bit easier for the drivers These look good, all committed. Except for the nouveau patch, of course. Kristian > Signed-off-by: Christopher James Halse Rogers > > --- > hw/xfree86/common/xf86Con

Re: [PATCH] Wayland: Add missing extern "C"

2012-05-23 Thread Kristian Høgsberg
On Wed, May 23, 2012 at 02:14:30PM +0800, yan.w...@linux.intel.com wrote: > From: Yan Wang > > Add extern "C" macro definition for C++. That's better, thanks. Kristian > --- > cursor/wayland-cursor.h |8 > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/cursor/w

RE: [E-devel] [PATCH] Wayland: Fix not add input.

2012-05-23 Thread Chris Michael
Fixed in EFL svn now, Thanks :) Dh > -Original Message- > From: zhiwen...@linux.intel.com [mailto:zhiwen...@linux.intel.com] > Sent: 23 May 2012 08:32 > To: enlightenment-de...@lists.sourceforge.net > Cc: wayland-devel@lists.freedesktop.org > Subject: [E-devel] [PATCH] Wayland: Fix not a

Re: [RFC] [PATCH 0/3] libwayland-cursor

2012-05-23 Thread Ander Conselvan de Oliveira
On 05/22/2012 06:25 PM, Kristian Høgsberg wrote: On Tue, May 22, 2012 at 03:39:39PM +0300, Ander Conselvan de Oliveira wrote: Hi, In order to get rid of the X11 dependency that libXcursor brings, Kristian suggested that a libwayland-cursor be created. This library handles the X cursor theme for

[PATCH wayland] shm: use mremap on pool resize

2012-05-23 Thread Ander Conselvan de Oliveira
--- src/wayland-shm.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/src/wayland-shm.c b/src/wayland-shm.c index e071392..62afc3b 100644 --- a/src/wayland-shm.c +++ b/src/wayland-shm.c @@ -25,6 +25,8 @@ * */ +#define _GNU_SOURCE + #include #include #

Re: [PATCH] fix typo

2012-05-23 Thread Kristian Høgsberg
On Wed, May 23, 2012 at 02:08:08AM -0400, Diego Viola wrote: Thanks! Kristian > --- > README |2 +- > TODO |4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/README b/README > index b8ad9b4..64904ed 100644 > --- a/README > +++ b/README > @@ -12,7 +12,7 @@ buf

[PATCH] xwayland: Adapt to new initialisation sequence

2012-05-23 Thread Christopher James Halse Rogers
--- src/intel_driver.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/intel_driver.c b/src/intel_driver.c index e414406..7bf8b76 100644 --- a/src/intel_driver.c +++ b/src/intel_driver.c @@ -631,15 +631,18 @@ static Bool I830PreInit(ScrnInfoPtr scrn, int flags

[PATCH (nouveau)] Add xwayland support

2012-05-23 Thread Christopher James Halse Rogers
Signed-off-by: Christopher James Halse Rogers --- This is mostly just for testing, although it should be safe to apply to nouveau trunk. src/nouveau_dri2.c | 34 +++ src/nv_driver.c| 163 +++- src/nv_type.h |7 +++ 3 files

[PATCH 2/2] xwayland: Decouple drm init from screen init.

2012-05-23 Thread Christopher James Halse Rogers
The nouveau driver wants access to its drm fd well before xwl_screen_pre_init can be called. Allow this by decoupling xwl_screen creation, xwl drm initialisation, and xwl_screen_pre_init. Signed-off-by: Christopher James Halse Rogers --- hw/xfree86/xwayland/xwayland-drm.c |8 +++ hw

[PATCH 1/2] xwayland: Automatically load module if wayland support requested

2012-05-23 Thread Christopher James Halse Rogers
This makes life a bit easier for the drivers Signed-off-by: Christopher James Halse Rogers --- hw/xfree86/common/xf86Config.c | 12 1 file changed, 12 insertions(+) diff --git a/hw/xfree86/common/xf86Config.c b/hw/xfree86/common/xf86Config.c index 7364a34..b916422 100644 --- a/h

Re: [PATCH] Wayland: Add missing extern "C"

2012-05-23 Thread Juan Zhao
On 05/23/2012 02:14 PM, yan.w...@linux.intel.com wrote: From: Yan Wang Add extern "C" macro definition for C++. It would be helpful, when some C++ applications include this header file, like chromium. Thanks, Juan --- cursor/wayland-cursor.h |8 1 files changed, 8 inserti

[PATCH] Wayland: Fix not add input.

2012-05-23 Thread zhiwen . wu
From: Alex Wu wl_seat adopation patch didn't change the interface name from "wl_input_device" to "wl_seat" at global object listener handler, so that no input will be added. --- src/lib/ecore_wayland/ecore_wl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/ecore

Re: synaptics rework - multiple frontends

2012-05-23 Thread Peter Hutterer
On 23/05/12 17:16 , Juan Zhao wrote: On 05/23/2012 01:31 PM, Peter Hutterer wrote: Jonas posted some touchpad support patches earlier this month and there were some calls for a input handling module. I've hacked around for a few days to get the synaptics driver ready for wayland. This is not a

Re: synaptics rework - multiple frontends

2012-05-23 Thread Juan Zhao
On 05/23/2012 01:31 PM, Peter Hutterer wrote: Jonas posted some touchpad support patches earlier this month and there were some calls for a input handling module. I've hacked around for a few days to get the synaptics driver ready for wayland. This is not a wayland touchpad driver! It's the X11