Re: [PATCH weston] config-parser: Drop debug text

2016-08-26 Thread Yong Bakos
On Aug 26, 2016, at 6:55 PM, Bryce Harrington wrote: > > Signed-off-by: Bryce Harrington Reviewed-by: Yong Bakos > --- > shared/config-parser.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git

[PATCH weston] config-parser: Drop debug text

2016-08-26 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- shared/config-parser.c | 1 - 1 file changed, 1 deletion(-) diff --git a/shared/config-parser.c b/shared/config-parser.c index d773cc9..e2b5fa2 100644 --- a/shared/config-parser.c +++ b/shared/config-parser.c @@ -237,7 +237,6 @@

Re: [PATCH wayland-protocols] buildsystem: add missing uninstalled pkg-config infra

2016-08-26 Thread Bryce Harrington
On Fri, Aug 26, 2016 at 06:06:43PM -0700, Bryce Harrington wrote: > It is confusing terminology to call this 'uninstalled'; sounds like this > might remove something from your system. 'non-installed' would be more > accurate but isn't that great of a name either. Oh I see, this is pre-existing

Re: [PATCH wayland-protocols] buildsystem: add missing uninstalled pkg-config infra

2016-08-26 Thread Bryce Harrington
It is confusing terminology to call this 'uninstalled'; sounds like this might remove something from your system. 'non-installed' would be more accurate but isn't that great of a name either. You should also provide an explanation about what the purpose of this is in the changelog. I.e. what

[PATCH wayland 1/2] buildsystem: adjust uninstalled -client/server pkg-config infra

2016-08-26 Thread Derek Foreman
From: "Reynaldo H. Verdejo Pinochet" Add missing protocol dir to uninstalled include path. Signed-off-by: Reynaldo H. Verdejo Pinochet Reviewed-by: Derek Foreman Signed-off-by: Derek Foreman

[PATCH wayland 2/2] build: Fix scanner path in uninstalled pkg-config file

2016-08-26 Thread Derek Foreman
this was generating a pkg-config file that said wayland-scanner was wayland/src/wayland-scanner when it's actually wayland/wayland-scanner Signed-off-by: Derek Foreman --- src/wayland-scanner-uninstalled.pc.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH weston] buildsystem: add missing pkg-config stub for libweston uninstalled

2016-08-26 Thread Derek Foreman
From: "Reynaldo H. Verdejo Pinochet" Signed-off-by: Reynaldo H. Verdejo Pinochet Reviewed-by: Derek Foreman Signed-off-by: Derek Foreman --- configure.ac | 1 +

[PATCH wayland-protocols] buildsystem: add missing uninstalled pkg-config infra

2016-08-26 Thread Derek Foreman
From: "Reynaldo H. Verdejo Pinochet" Signed-off-by: Reynaldo H. Verdejo Pinochet Reviewed-by: Derek Foreman Signed-off-by: Derek Foreman --- configure.ac| 1 +

[PATCH libinput] buildsystem: add missing uninstalled pkg-config infra

2016-08-26 Thread Derek Foreman
From: "Reynaldo H. Verdejo Pinochet" Signed-off-by: Reynaldo H. Verdejo Pinochet Reviewed-by: Derek Foreman Signed-off-by: Derek Foreman --- configure.ac | 1 +

Re: Weston does not do transparent backgrounds

2016-08-26 Thread Pekka Paalanen
On Fri, 26 Aug 2016 16:02:19 +0530 "arunkrish20 ." wrote: > Hi All, > > http://wayland-devel.freedesktop.narkive.com/dXZUCogH/desktop-shell-how-to-enable-really-alpha-blending-of-weston-background > > Our question is related to above mentioned link question. > >

Re: [PATCH wayland 1/2] util: Document wl_list methods

2016-08-26 Thread Yong Bakos
On Aug 26, 2016, at 2:12 AM, Giulio Camuffo wrote: > > Hi, > > >> >> +/** >> + * Removes an element from the list. >> + * >> + * \param elm element to remove from the list >> + */ > I think this should mention that elm is left in an invalid state, and > the user

Weston does not do transparent backgrounds

2016-08-26 Thread arunkrish20 .
Hi All, http://wayland-devel.freedesktop.narkive.com/dXZUCogH/desktop-shell-how-to-enable-really-alpha-blending-of-weston-background Our question is related to above mentioned link question. Environment OS : Linux kernel 3.14 Version : Weston 1.9.0 Weston fbdev-backend. In compositor-fbdev.c

Re: [PATCH wayland 1/2] util: Document wl_list methods

2016-08-26 Thread Giulio Camuffo
Hi, > > +/** > + * Removes an element from the list. > + * > + * \param elm element to remove from the list > + */ I think this should mention that elm is left in an invalid state, and the user should call wl_list_init on it before using it again. Cheers, Giulio > void >

Re: [PATCH wayland 2/2] tests: Add test for wl_list_length

2016-08-26 Thread Eric Engestrom
On Thu, Aug 25, 2016 at 04:12:34PM -0700, Yong Bakos wrote: > From: Yong Bakos > > list-test.c did not cover wl_list_length, so add one test that specifically > tests this method. > > Signed-off-by: Yong Bakos > --- > tests/list-test.c | 13