Re: [PATCH weston] sdk: make C++11 plugins build again

2013-12-09 Thread Thiago Macieira
On segunda-feira, 9 de dezembro de 2013 15:57:16, Kristian Høgsberg wrote: > Ugh, ok, sorry. Maybe we need a private header file where we can hide > all our C macros - ARRAY_LENGTH, static_assert, container_of etc. This "poor man's static_assert" should work in C99 mode too: #include #if !defin

Re: [PATCH weston] sdk: make C++11 plugins build again

2013-12-09 Thread Kristian Høgsberg
On Mon, Dec 09, 2013 at 10:47:58PM +0100, Giulio Camuffo wrote: > compositor.h must not define a 'static_assert' macro, since that > conflicts with the new 'static_assert' in the standard and breaks > the build. Ugh, ok, sorry. Maybe we need a private header file where we can hide all our C macro

[PATCH weston] sdk: make C++11 plugins build again

2013-12-09 Thread Giulio Camuffo
compositor.h must not define a 'static_assert' macro, since that conflicts with the new 'static_assert' in the standard and breaks the build. --- desktop-shell/shell.c | 4 src/compositor.h | 4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/desktop-shell/shell.c b/d

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-08 Thread Quentin Glidic
On 04/07/2013 22:20, Kristian Høgsberg wrote: On Thu, Jul 4, 2013 at 12:35 PM, wrote: And they must link directly! That is the proper way to do that. Providing API using these libraries does not mean forcing anyone to use that. And using a library does require linking to it. Hiding this depend

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-06 Thread Pekka Paalanen
> In any case, the actual behaviour is that Requires.private is used in dynamic > linking. It adds -I flags, but no -L or -l flags. Yeah. This might be nitpicking, but I don't consider -I flags to be about linking. They are for the pre-processor. The linking mode does not affect how

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-05 Thread Daniel Stone
Hi, On 5 July 2013 20:35, Thiago Macieira wrote: > First, it mentions that Requires.private is *not* used in dynamic linking. > Then it changes its mind with that last sentence, which is cryptic. > > In any case, the actual behaviour is that Requires.private is used in dynamic > linking. It adds

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-05 Thread Thiago Macieira
On sexta-feira, 5 de julho de 2013 09.26.23, Pekka Paalanen wrote: > The color management modules are an example of a weston module, that > does not need pixman at all. If you grep through src/cms*.[ch], there > is not a single mention of "pixman". Why do these modules, if they were > external, nee

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-04 Thread Pekka Paalanen
On Thu, 04 Jul 2013 13:18:27 -0700 Thiago Macieira wrote: > On quinta-feira, 4 de julho de 2013 18.35.16, > sardemff7+wayl...@sardemff7.net wrote: > > > Suppose that our headers #include headers from a project whose > > > headers aren't usually in /usr/include (that is, they require > > > -I). In

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-04 Thread Kristian Høgsberg
On Thu, Jul 4, 2013 at 12:35 PM, wrote: > On 04/07/2013 18:25, Thiago Macieira wrote: >> >> On quinta-feira, 4 de julho de 2013 10.12.05, Pekka Paalanen wrote: >>> >>> On Wed, 3 Jul 2013 18:26:06 -0400 No, it has to be plain Requires. By having pixman and libxkbcommon headers in o

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-04 Thread Thiago Macieira
On quinta-feira, 4 de julho de 2013 18.35.16, sardemff7+wayl...@sardemff7.net wrote: > > Suppose that our headers #include headers from a project whose headers > > aren't usually in /usr/include (that is, they require -I). In order for > > our users' applications to build, they need to add those -

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-04 Thread sardemff7+wayland
On 04/07/2013 18:25, Thiago Macieira wrote: On quinta-feira, 4 de julho de 2013 10.12.05, Pekka Paalanen wrote: On Wed, 3 Jul 2013 18:26:06 -0400 No, it has to be plain Requires. By having pixman and libxkbcommon headers in our SDK headers, we expose those dependencies to applications. Hi,

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-04 Thread Thiago Macieira
On quinta-feira, 4 de julho de 2013 10.12.05, Pekka Paalanen wrote: > On Wed, 3 Jul 2013 18:26:06 -0400 > > No, it has to be plain Requires. By having pixman and libxkbcommon > > headers in our SDK headers, we expose those dependencies to > > applications. > > Hi, > > could you explain that logi

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-04 Thread Pekka Paalanen
On Wed, 3 Jul 2013 18:26:06 -0400 Kristian Høgsberg wrote: > On Mon, Jul 01, 2013 at 05:03:01PM +0200, Quentin Glidic wrote: > > From: Quentin Glidic > > > > Weston headers include pixman and libxkbcommon headers > > Using Requires.private means that CFLAGS from pixman-1 and > > xkbcommon are a

Re: [PATCH weston sdk v2 8/8] shared: Export configuration functions

2013-07-03 Thread Kristian Høgsberg
On Mon, Jul 01, 2013 at 05:03:08PM +0200, Quentin Glidic wrote: > From: Quentin Glidic Yes, thanks, that was part of the plan with the new config parser. Applied. Kristian > Signed-off-by: Quentin Glidic > --- > shared/config-parser.c | 5 + > 1 file changed, 5 insertions(+) > > diff --g

Re: [PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-03 Thread Kristian Høgsberg
On Mon, Jul 01, 2013 at 05:03:01PM +0200, Quentin Glidic wrote: > From: Quentin Glidic > > Weston headers include pixman and libxkbcommon headers > Using Requires.private means that CFLAGS from pixman-1 and xkbcommon are > added to weston CFLAGS, while LIBS are added in case of static linking > o

[PATCH weston sdk v2 5/8] weston.pc: Provide moduledir

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic This allows modules to use the correct directory for the targeted Weston to install themselves. Signed-off-by: Quentin Glidic --- src/weston.pc.in | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/weston.pc.in b/src/weston.pc.in index 3c61c98..7bdf8d8 100644 --- a

[PATCH weston sdk v2 8/8] shared: Export configuration functions

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- shared/config-parser.c | 5 + 1 file changed, 5 insertions(+) diff --git a/shared/config-parser.c b/shared/config-parser.c index 0ff96ee..4e6cf7f 100644 --- a/shared/config-parser.c +++ b/shared/config-parser.c @@ -260,6 +260,7 @@ conf

[PATCH weston sdk v2 6/8] weston.ini: Allow a relative path for input-method

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- man/Makefile.am| 1 + man/weston.ini.man | 3 ++- src/text-backend.c | 7 +++ weston.ini | 2 +- 4 files changed, 11 insertions(+), 2 deletions(-) diff --git a/man/Makefile.am b/man/Makefile.am index 37174e2..b3a3881 100644

[PATCH weston sdk v2 7/8] weston.ini: Allow a relative path for screensaver

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- man/weston.ini.man | 3 ++- src/shell.c| 9 - weston.ini | 2 +- 3 files changed, 11 insertions(+), 3 deletions(-) diff --git a/man/weston.ini.man b/man/weston.ini.man index f457d34..4141e77 100644 --- a/man/weston.

[PATCH weston sdk v2 2/8] autotools: Define paths in configure.ac

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- configure.ac | 26 ++ man/Makefile.am | 2 +- src/Makefile.am | 4 ++-- src/weston.pc.in | 4 ++-- src/xwayland/Makefile.am | 2 +- 5 files changed, 32 insertions(+), 6 delet

[PATCH weston sdk v2 4/8] clients: Install to pkglibexecdir

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- clients/Makefile.am | 2 +- src/Makefile.am | 2 +- src/screenshooter.c | 2 +- src/shell.c | 2 +- src/tablet-shell.c | 2 +- src/text-backend.c | 2 +- src/xwayland/Makefile.am | 3 --- 7 files

[PATCH weston sdk v2 3/8] autotools: Use pkg*dir

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- data/Makefile.am | 4 +--- src/Makefile.am | 3 +-- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/data/Makefile.am b/data/Makefile.am index a7cc944..f6f05f9 100644 --- a/data/Makefile.am +++ b/data/Makefile.am @@ -1,6 +1,4

[PATCH weston sdk v2 1/8] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-07-01 Thread Quentin Glidic
From: Quentin Glidic Weston headers include pixman and libxkbcommon headers Using Requires.private means that CFLAGS from pixman-1 and xkbcommon are added to weston CFLAGS, while LIBS are added in case of static linking only. This way, plugins does not have to use them, but will need to do so exp

RE: [PATCH] Weston: SDK: export evdev.h

2013-05-31 Thread Eoff, Ullysses A
gt; To: wayland-devel@lists.freedesktop.org > Subject: [PATCH] Weston: SDK: export evdev.h > > This allows plugins to emulate a weston evdev device > > To run wl-fits without superuser, apply this patch. > --- > src/Makefile.am |9 ++--- > src/evdev.c |4 ++-- > 2 files ch

[PATCH] Weston: SDK: export evdev.h

2013-05-31 Thread Martin Minarik
This allows plugins to emulate a weston evdev device To run wl-fits without superuser, apply this patch. --- src/Makefile.am |9 ++--- src/evdev.c |4 ++-- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 106ccba..6146aff 10064

[PATCH weston] tests: add simple installcheck test for the weston sdk

2013-05-13 Thread U. Artie Eoff
From: "U. Artie Eoff" This test should be executed with 'make installcheck' after running 'make install'. It is a simple, basic test that attempts to compile a trivial (and inherently useless) weston sdk module to ensure the exported SDK can be used by consumers.

Re: [PATCH weston] sdk: be C++ friendly

2013-03-28 Thread Kristian Høgsberg
On Thu, Mar 28, 2013 at 06:02:42PM +0100, Giulio Camuffo wrote: > This renames the weston_surface's private member to configure_private > and externs "C" the headers of the SDK. > --- > shared/config-parser.h | 8 > shared/matrix.h| 8 > src/compositor.c | 6 +++-

[PATCH weston] sdk: be C++ friendly

2013-03-28 Thread Giulio Camuffo
This renames the weston_surface's private member to configure_private and externs "C" the headers of the SDK. --- shared/config-parser.h | 8 shared/matrix.h| 8 src/compositor.c | 6 +++--- src/compositor.h | 10 +- src/shell.c| 28

RE: Weston SDK

2013-02-15 Thread Eoff, Ullysses A
@lists.freedesktop.org] On Behalf Of Daniel Stone Sent: Friday, February 15, 2013 11:42 AM To: Fred Ollinger Cc: Pekka Paalanen; Kristian Høgsberg; wayland-devel@lists.freedesktop.org Subject: Re: Weston SDK Hi, On 15 February 2013 16:02, Fred Ollinger mailto:folli...@gmail.com>> wrote: Surely th

Re: Weston SDK

2013-02-15 Thread Daniel Stone
Hi, On 15 February 2013 16:02, Fred Ollinger wrote: > Surely there is functionality that plugin authors will want which will > not change? > Yes, but exactly what that is is a far more difficult question. Help welcome. Cheers, Daniel ___ wayland-dev

Re: Weston SDK

2013-02-15 Thread Fred Ollinger
Why not have an external api which does not change? Surely there is functionality that plugin authors will want which will not change? I know that one objection is that there's the question of who is going to maintain the glue b/w the external and internal. Perhaps we can leave that to the plugin

Re: Weston SDK

2013-02-15 Thread Pekka Paalanen
On Thu, 14 Feb 2013 11:27:22 -0500 Kristian Høgsberg wrote: > Hi, > > I made a little experiment last night: > > http://cgit.freedesktop.org/~krh/overlay-plugin > > It's an out-of-tree weston plugin. It's just a silly little overlay > that you can pop up with mod-space, but the interesting

RE: Weston SDK

2013-02-14 Thread Eoff, Ullysses A
y Dahlin >Cc: wayland >Subject: Re: Weston SDK > >On Thu, Feb 14, 2013 at 11:51 AM, Casey Dahlin wrote: >> On Thu, Feb 14, 2013 at 11:27:22AM -0500, Kristian Høgsberg wrote: >>> Hi, >>> >>> I made a little experiment last night: >>> >>>

Re: Weston SDK

2013-02-14 Thread Kristian Høgsberg
On Thu, Feb 14, 2013 at 11:51 AM, Casey Dahlin wrote: > On Thu, Feb 14, 2013 at 11:27:22AM -0500, Kristian Høgsberg wrote: >> Hi, >> >> I made a little experiment last night: >> >> http://cgit.freedesktop.org/~krh/overlay-plugin >> >> It's an out-of-tree weston plugin. It's just a silly little

Re: Weston SDK

2013-02-14 Thread Casey Dahlin
On Thu, Feb 14, 2013 at 11:27:22AM -0500, Kristian Høgsberg wrote: > Hi, > > I made a little experiment last night: > > http://cgit.freedesktop.org/~krh/overlay-plugin > > It's an out-of-tree weston plugin. It's just a silly little overlay > that you can pop up with mod-space, but the interes

Weston SDK

2013-02-14 Thread Kristian Høgsberg
Hi, I made a little experiment last night: http://cgit.freedesktop.org/~krh/overlay-plugin It's an out-of-tree weston plugin. It's just a silly little overlay that you can pop up with mod-space, but the interesting part here is that it's building outside weston [1]. Current, that works by co