Re: [PATCH weston 1/2] libweston: introduce LIBWESTON_API_VERSION macro

On 15 August 2016 at 17:16, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 15/08/2016 17:31, Emil Velikov wrote: >> >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Provides some ease wrt constructing the required version. >> >&

Re: [PATCH weston] Makefile.am: Fix libtool race

On 16 August 2016 at 09:49, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 16/08/2016 10:26, Pekka Paalanen wrote: >> >> On Mon, 15 Aug 2016 16:51:32 +0100 >> Emil Velikov <emil.l.veli...@gmail.com> wrote: >> >>> On 15 August 2

Re: [PATCH weston] Makefile.am: Fix libtool race

stall atm (iirc). On the patch in question - checked with the Makefile.in and things look great. Reviewed-by: Emil Velikov <emil.veli...@collabora.com> -Emil [1] http://git.midipix.org/cgit.cgi/slibtool/tree/README ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel

[PATCH weston 1/2] libweston: introduce LIBWESTON_API_VERSION macro

From: Emil Velikov <emil.veli...@collabora.com> Provides some ease wrt constructing the required version. Cc: Quentin Glidic <sardemff7+...@sardemff7.net> Suggested-by: Quentin Glidic <sardemff7+...@sardemff7.net> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>

[PATCH weston 2/2] README: fix s/libwayland/libweston/ typo

From: Emil Velikov <emil.veli...@collabora.com> Since one is (about to be) using libweston, they should check for it as opposed to libwayland. Silly copy/paste mistake that would have caused a lot of confusion. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Seems

Re: [PATCH weston 2/7] weston-egl-ext.h: remove EGL_EGLEXT_PROTOTYPES sections

On 12 August 2016 at 08:47, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Mon, 4 Jul 2016 15:27:14 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Those the function declarations

Re: [PATCH weston v2 0/4] Libweston version numbering for releases (BLOCKER)

leave the libweston pre-release smarts to the user. Either way is fine with me, so the series is Reviewed-by: Emil Velikov <emil.veli...@collabora.com> Thanks Emil ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Re: [PATCH weston] automake: list static libraries before the shared ones

On 2 August 2016 at 12:03, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Fri, 22 Jul 2016 14:52:42 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> From: Emil Velikov <emil.veli...@collabora.com> >> >> The linker processes those in the or

[PATCH weston] automake: list static libraries before the shared ones

From: Emil Velikov <emil.veli...@collabora.com> The linker processes those in the order that they are given. Thus as-it we can get unresolved symbols. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Some places in weston already follow this approach, and although

Re: [PATCH weston 0/7] EGL/GL definitions cleanup

On 4 July 2016 at 15:27, Emil Velikov <emil.l.veli...@gmail.com> wrote: > It seems that a handful of extensions were missing fallback definitions > and/or were handled sub optimally. > > So this series folds most (the > remainder coming in another series) of the typedef/de

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

On 13 July 2016 at 15:53, Jan Engelhardt wrote: > > On Wednesday 2016-07-13 13:54, Pekka Paalanen wrote: >> >>I think Quentin raised a good point, though. In source-based >>distros, well, in Gentoo at least which I use almost exclusively, >>there are no separate -devel packages.

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

On 11 July 2016 at 17:58, Jan Engelhardt <jeng...@inai.de> wrote: > > On Monday 2016-07-11 16:44, Emil Velikov wrote: >>>>http://ometer.com/parallel.html. I would strongly recommend giving it >>>>a look. >>> >>> I read it now, and I do not b

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

On 9 July 2016 at 17:26, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Thu, 7 Jul 2016 17:45:24 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> On 7 July 2016 at 10:46, Pekka Paalanen <ppaala...@gmail.com> wrote: >> > On Mon, 4 Jul 2016 16:

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

On 9 July 2016 at 16:32, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Thu, 7 Jul 2016 20:08:40 +0200 > Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > >> On 07/07/2016 18:11, Emil Velikov wrote: >> > On 7 July 2016 at 10:05, Pekka

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

On 10 July 2016 at 13:23, Jan Engelhardt <jeng...@inai.de> wrote: > > On Sunday 2016-07-10 12:46, Emil Velikov wrote: >>> PKG_CHECK_EXISTS([gtk-3.0], [PKG_CHECK_MODULES([gtk], [gtk-3.0])], [ >>> ...repeat the fun... >>> ])] >>> >>Ye

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

On 10 July 2016 at 11:11, Jan Engelhardt wrote: > > On Saturday 2016-07-09 18:24, Pekka Paalanen wrote: >>> >>> First, what kind of parallel installability is sought? >>> >>> * just runtime >>> * parallel development environment (like what e.g. libabw, >>> librevenge.. do) >>

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

On 8 July 2016 at 12:00, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 08/07/2016 12:52, Emil Velikov wrote: >> >> On 7 July 2016 at 19:08, Quentin Glidic <sardemff7+wayl...@sardemff7.net> >> wrote: >>> >>> On 07/07/2016 18:11, Emil

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

On 7 July 2016 at 19:08, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 07/07/2016 18:11, Emil Velikov wrote: >> >> On 7 July 2016 at 10:05, Pekka Paalanen <ppaala...@gmail.com> wrote: >>> >>> >>> [snip] > >>> &

Re: [PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

On 7 July 2016 at 19:18, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 07/07/2016 18:28, Emil Velikov wrote: >> >> On 7 July 2016 at 10:20, Pekka Paalanen <ppaala...@gmail.com> wrote: >>> >>> [snip] >>> Therefore a NAK from me

Re: Create cross wayland-scanner for toolchain or cross-development environment

Hi all, Jumping the gun a bit, hope you'll forgive me :-) On 8 July 2016 at 09:17, Quentin Glidic wrote: > On 18/05/2016 14:55, Andrew Kosteltsev wrote: >> Then the user can make use this cross-wayland-scanner in his SDK, for >> example, like follow: >> >> $

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

On 7 July 2016 at 10:46, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Mon, 4 Jul 2016 16:25:54 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> On 4 July 2016 at 15:35, Quentin Glidic <sardemff7+wayl...@sardemff7.net> >> wrote: >&

Re: [PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

On 7 July 2016 at 10:20, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Mon, 4 Jul 2016 16:02:23 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> On 4 July 2016 at 15:32, Quentin Glidic <sardemff7+wayl...@sardemff7.net> >> wrote: >&

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

On 7 July 2016 at 10:05, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Wed, 6 Jul 2016 13:57:48 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> On 5 July 2016 at 15:46, Pekka Paalanen <ppaala...@gmail.com> wrote: >> > On Mon, 4 Jul 2016 15:

Re: [PATCH weston 3/6] README: Document versioning scheme, forward compatibility

On 5 July 2016 at 15:46, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Mon, 4 Jul 2016 15:23:51 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Signed-off-by: Emil Velikov

Re: [PATCH weston 2/2] automake: add missing git-version.h dependency

On 5 July 2016 at 13:18, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 04/07/2016 15:57, Emil Velikov wrote: >> >> From: Emil Velikov <emil.veli...@collabora.com> >> >> compositor/main.c depends on the header, while the dependency isn't &g

Re: [PATCH weston 3/9] gl-renderer: move check_extension() to shared/

On 4 July 2016 at 15:34, Emil Velikov <emil.l.veli...@gmail.com> wrote: > diff --git a/shared/config-parser.c b/shared/config-parser.c > index 2256469..d316958 100644 > --- a/shared/config-parser.c > +++ b/shared/config-parser.c > @@ -39,6 +39,7 @@ > > #include &

Re: [PATCH weston 4/6] libweston: use new versioning scheme

On 4 July 2016 at 15:45, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 04/07/2016 16:23, Emil Velikov wrote: >> >> Use the documented libweston-$major.so.0.$minor.$patch scheme. >> >> An (almost) identical one is used by GLIB, GTK{2,3}, QT5,

Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc

On 4 July 2016 at 15:35, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 04/07/2016 16:23, Emil Velikov wrote: >> >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> >> --

Re: [PATCH weston 0/6] Rework libweston versioning, take 2(?)

On 4 July 2016 at 15:55, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Mon, 4 Jul 2016 15:23:48 +0100 > Emil Velikov <emil.l.veli...@gmail.com> wrote: > >> Hi all, >> >> Here is a respin of the earlier series which changes how libweston is >> nam

Re: [PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

On 4 July 2016 at 15:32, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > On 04/07/2016 16:23, Emil Velikov wrote: >> >> From: Emil Velikov <emil.veli...@collabora.com> >> >> When managing headers there's normally two ways to handle them >>

[PATCH weston 0/9] Finalise EGL/GL define(s) rework

Hi all, This series depends on "EGL/GL definitions cleanup" and does a couple of things: - makes sure that we consistently use check_extension over strstr for extension checking, as the latter might trigger false positives. - The remainder of the fall-back definitions has been moved to

[PATCH weston 2/9] libweston: don't install shared/platform.h

From: Emil Velikov <emil.veli...@collabora.com> The header contains a set of extension check (helpers), which do not provide any compositor related functionality and are not required for using libweston. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Feel free to disagr

[PATCH weston 7/9] weston-egl-ext.h: add EGL platform definitions

From: Emil Velikov <emil.veli...@collabora.com> Will allow us to consolidate the multiple definitions through the tree. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- libweston/weston-egl-ext.h | 19 +++ 1 file changed, 19 insertions(+) diff --git

[PATCH weston 4/9] shared/platform.h: use weston_check_extension over strstr

From: Emil Velikov <emil.veli...@collabora.com> The later can give false positives. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- shared/platform.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/shared/platform.h b/shared/platform.h i

[PATCH weston 6/9] clients/simple-egl: use weston_check_extension over strstr

From: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- clients/simple-egl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/clients/simple-egl.c b/clients/simple-egl.c index d5a2660..0214b2d 100644 --- a/cl

[PATCH weston 1/9] shared/platform.h: add missing stdbool.h include

From: Emil Velikov <emil.veli...@collabora.com> Required by the bool type, used through the header. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- shared/platform.h | 1 + 1 file changed, 1 insertion(+) diff --git a/shared/platform.h b/shared/platform.h index dd55

[PATCH weston 8/9] gl-renderer: remove local EGL platform (re)definitions

From: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- libweston/gl-renderer.h | 17 - 1 file changed, 17 deletions(-) diff --git a/libweston/gl-renderer.h b/libweston/gl-renderer.h index 71f6b46..ad

[PATCH weston 5/9] clients/nested: use weston_check_extension over strstr

From: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- clients/nested.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/clients/nested.c b/clients/nested.c index 6c21aa4..9f74326 100644 --- a/clients/ne

[PATCH weston 3/9] gl-renderer: move check_extension() to shared/

From: Emil Velikov <emil.veli...@collabora.com> ... prefixing it with a "weston_". This way we can reuse it across the board, instead of the current strstr. The latter of which can give us false positives, thus it will be resolved with next commit(s). Signed-off-by: Emil Ve

[PATCH weston 2/7] weston-egl-ext.h: remove EGL_EGLEXT_PROTOTYPES sections

From: Emil Velikov <emil.veli...@collabora.com> Those the function declarations alongside the EGL_EGLEXT_PROTOTYPES guards. Although those are copy/paste sections from the header(s), having and/or using them is a bad idea, as per the EGL spec. Signed-off-by: Emil Velikov <

[PATCH weston 0/7] EGL/GL definitions cleanup

It seems that a handful of extensions were missing fallback definitions and/or were handled sub optimally. So this series folds most (the remainder coming in another series) of the typedef/defines in weston-egl-ext.h and cleanups all the ifdef spaghetti through the codebase. -Emil

[PATCH weston 7/7] simple-egl: remove incomplete EGL_EXT_buffer_age definition

From: Emil Velikov <emil.veli...@collabora.com> A more complete alternative is already provided by the weston-egl-ext.h header. The latter of which we already include. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- clients/simple-egl.c | 5 - 1 file changed,

[PATCH weston 1/7] client/nested: reuse weston-egl-ext.h declarations

From: Emil Velikov <emil.veli...@collabora.com> Rather than introducing a local copy of the EGL_WL_create_wayland_buffer_from_image (re)definition, just use the local header. This also gives us access to EGL_WL_bind_wayland_display which is also used in the client, yet the C file is m

[PATCH weston 6/7] gl-renderer: remove EGL_EXT_image_dma_buf_import guards

From: Emil Velikov <emil.veli...@collabora.com> We provide a (workaround) definition in weston-egl-ext.h, thus we don't need any guards. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- libweston/gl-renderer.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/

[PATCH weston 3/7] weston-egl-ext.h: add EGL_EXT_swap_buffers_with_damage definitions

From: Emil Velikov <emil.veli...@collabora.com> ... and use it from simple-egl and gl-renderer. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- clients/simple-egl.c | 6 +- libweston/gl-renderer.c| 12 libweston/weston-egl-ext.h | 5 ++

[PATCH weston 4/7] weston-egl-ext.h: add EGL_MESA_configless_context definitions

From: Emil Velikov <emil.veli...@collabora.com> ... and use it in gl-renderer. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- libweston/gl-renderer.c| 4 libweston/weston-egl-ext.h | 5 + 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/

[PATCH weston 5/7] weston-egl-ext.h: add GL_EXT_unpack_subimage definitions

From: Emil Velikov <emil.veli...@collabora.com> ... and use it in gl-renderer. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- libweston/gl-renderer.c| 9 - libweston/weston-egl-ext.h | 7 +++ 2 files changed, 7 insertions(+), 9 deletions(-) diff --git

[PATCH weston 6/6] libweston: do not use weston version in libweston.pc

From: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- configure.ac | 1 + libweston/libweston.pc.in | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index be40f10..46

[PATCH weston 1/6] README: clarify libweston purpose/goals.

From: Emil Velikov <emil.veli...@collabora.com> v2: Rewrap, add a couple of missing words (Pekka). v3: Use alternative wording (Yong). Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- README | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-)

[PATCH weston 4/6] libweston: use new versioning scheme

. Signed-off-by: Emil Velikov <emil.l.veli...@collabora.com> --- XXX: - Should we rename libweston{,-0}.pc.in ? - Drop the s/LIBWESTON_ABI_VERSION/LIBWESTON_MAJOR/ hunk ? - Keep the configure libweston_*_version variables shorter ? - Yes, the LT_VERSION_INFO hunk looks a bit nasty, yet this i

[PATCH weston 2/6] README: minor the libweston ABI/API documentation

From: Emil Velikov <emil.veli...@collabora.com> v2: Elaborate what is meant with "major ABI versions" (Pekka). Signed-off-by: Emil Velikov <emil.veli...@collabora.com> Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk> (v1) --- README | 25 +

[PATCH weston 5/6] libweston: do not add libweston-$version to the Cflags

From: Emil Velikov <emil.veli...@collabora.com> When managing headers there's normally two ways to handle them - with or without the subfolder. Opting for the latter case here, since it will provide direct feedback, whether one is using libweston-0 or any other version. Which in turn

[PATCH weston 0/6] Rework libweston versioning, take 2(?)

Hi all, Here is a respin of the earlier series which changes how libweston is named, and thus shipped. I believe all the logic/reasoning is explicitly stated, although if something feels amiss, please let me know. NOTE: WARNING: The series exposes a fatal bug in weston thus the tests fail to

[PATCH weston 3/6] README: Document versioning scheme, forward compatibility

From: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Pekka, There's a couple of things to 'break' - forward and backward compatibility. Latter implies changing (removing) certain existing API, while the former is used

[PATCH weston 2/2] automake: add missing git-version.h dependency

From: Emil Velikov <emil.veli...@collabora.com> compositor/main.c depends on the header, while the dependency isn't specified. Thus depending on the order of how things are build we could get a build failure. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Strictly

[PATCH weston 1/2] automake: list the the builddir include before the srcdir

From: Emil Velikov <emil.veli...@collabora.com> Otherwise we'll pick up the stale (in-tree) generated source(s) over the fresh (out-of-tree) ones. Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- It's quite a corner case, although it could save you a lot of ti

Re: [PATCH weston 1/4] README: clarify libweston purpose/goals.

Hi Yong, On 4 June 2016 at 01:58, Yong Bakos <j...@humanoriented.com> wrote: > Hi Emil, > > On Jun 3, 2016, at 8:27 AM, Emil Velikov <emil.l.veli...@gmail.com> wrote: >> >> From: Emil Velikov <emil.veli...@collabora.com> >> >> v2: Rewrap, add

[PATCH weston 2/4] README: minor the libweston ABI/API documentation

From: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- README | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/README b/README index 76be7b9..b8aa2e0 100644 --- a/README +++ b/READ

[RFC weston 4/4] WIP: libweston: rework versioning scheme

From: Emil Velikov <emil.veli...@collabora.com> Use libweston-$major.so.0.$minor.$patch over the current scheme. It allows for separation (distinction) of the backwards incompatible changes from forward compatible feature/bugfix ones. TODO: - Check if we need the -@LIBWESTON_VERSION

[PATCH weston 1/4] README: clarify libweston purpose/goals.

From: Emil Velikov <emil.veli...@collabora.com> v2: Rewrap, add a couple of missing words (Pekka). Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- README | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/README b/README i

[RFC weston 3/4] XXX: README: Mention bits that we really want for libweston

From: Emil Velikov <emil.veli...@collabora.com> --- README | 12 1 file changed, 12 insertions(+) diff --git a/README b/README index b8aa2e0..e411343 100644 --- a/README +++ b/README @@ -161,3 +161,15 @@ major ABI-versions, except those explicitly mentioned. Weston's bui

[PATCH 0/4] libweston inspired fixes (?)

Hi all, Seeing that libweston have landed, I've noticed a few things that seemed odd. - libweston.so is installed alongside the libweston-0.so one. - Current versioning scheme does not attribute for fwd compat feature (minor) and bugfix (patch) version bumps. So here it is. A short series

Re: [PATCH weston] build: Define wayland prereq version

On 20 May 2016 at 21:35, Bryce Harrington <br...@osg.samsung.com> wrote: > On Thu, May 12, 2016 at 09:48:17AM +0100, Emil Velikov wrote: >> On 12 May 2016 at 09:13, Pekka Paalanen <ppaala...@gmail.com> wrote: >> > On Thu, 12 May 2016 11:12:28 +1000 >> > P

Re: [PATCH weston] build: Define wayland prereq version

On 12 May 2016 at 09:13, Pekka Paalanen wrote: > On Thu, 12 May 2016 11:12:28 +1000 > Peter Hutterer wrote: > >> On Wed, May 11, 2016 at 01:18:59PM -0700, Bryce Harrington wrote: >> > Establishes a single variable for defining the libwayland version

Re: [PATCH wayland 2/3] server: move include directives before extern "C" wrapper

On 19 April 2016 at 09:51, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Mon, 18 Apr 2016 08:16:37 +0200 > Hardening <rdp.eff...@gmail.com> wrote: > >> Le 17/04/2016 13:29, Emil Velikov a écrit : >> > Analogous to previous commit but for the server(-core) h

[PATCH wayland 1/3] scanner: move include directives before extern "C" wrapper

ingcandcpluspluscode-305840.html Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> --- src/scanner.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/scanner.c b/src/scanner.c index 65df3ae..52c07a6 100644 --- a/src/scanner.c +++ b/src/scanner.c @@ -1423,

[PATCH wayland 2/3] server: move include directives before extern "C" wrapper

Analogous to previous commit but for the server(-core) header. Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> --- src/wayland-server-core.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/wayland-server-core.h b/src/wayland-server-core.h index e

[PATCH wayland 3/3] utils: move include directives before extern "C" wrapper

Analogous to last two commits. Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> --- src/wayland-util.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/wayland-util.h b/src/wayland-util.h index 35b50ea..cc1999d 100644 --- a/src/wayland-util.h +++

Re: [PATCH v2 2/3] xwayland: Remove a useless out-of-memory check

Hello gents, On 15 July 2015 at 09:51, Marek Chalupa mchqwe...@gmail.com wrote: Reviewed-by: Marek Chalupa mchqwe...@gmail.com (http://lists.freedesktop.org/archives/wayland-devel/2015-May/021952.html) On 05/16/2015 07:38 AM, Dima Ryazanov wrote: snprintf does not allocate memory, so we

Re: [PATCH v2 2/3] xwayland: Remove a useless out-of-memory check

On 15 July 2015 at 19:11, Bryce Harrington br...@osg.samsung.com wrote: On Wed, Jul 15, 2015 at 01:09:03PM +0100, Emil Velikov wrote: Hello gents, On 15 July 2015 at 09:51, Marek Chalupa mchqwe...@gmail.com wrote: Reviewed-by: Marek Chalupa mchqwe...@gmail.com (http

Re: [PATCH weston] configure.ac: make use of wayland-scanner.pc

On 23 February 2015 at 15:31, Emil Velikov emil.l.veli...@gmail.com wrote: On 23 February 2015 at 14:10, sardemff7+wayl...@sardemff7.net wrote: On 2015-02-17 16:13, Emil Velikov wrote: Currently we use the wayland-scanner executable as found with AC_PATH_PROG, and then check the presence

Re: [PATCH weston] configure.ac: make use of wayland-scanner.pc

On 23 February 2015 at 14:10, sardemff7+wayl...@sardemff7.net wrote: On 2015-02-17 16:13, Emil Velikov wrote: Currently we use the wayland-scanner executable as found with AC_PATH_PROG, and then check the presence of wayland-scanner.pc Currently the latter is unused even if AC_PATH_PROG

Re: [PATCH weston] configure.ac: make use of wayland-scanner.pc

On 23 February 2015 at 19:20, Bill Spitzak spit...@gmail.com wrote: On 02/22/2015 11:52 PM, Pekka Paalanen wrote: I'll give Bill and Daniels a chance to comment, but meanwhile: Reviewed-by: Bryce Harrington br...@osg.samsung.com My only concern is that trying to build Wayland from scratch

Re: [PATCH weston] configure.ac: make use of wayland-scanner.pc

Hi Bryce On 20 February 2015 at 23:26, Bryce Harrington br...@osg.samsung.com wrote: On Tue, Feb 17, 2015 at 03:13:32PM +, Emil Velikov wrote: Currently we use the wayland-scanner executable as found with AC_PATH_PROG, and then check the presence of wayland-scanner.pc Currently

[PATCH weston] configure.ac: make use of wayland-scanner.pc

...@espial.com Cc: Bill Spitzak spit...@gmail.com Cc: Daniel Stone dan...@fooishbar.org Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 1db9f79..b44675f 100644 --- a/configure.ac

Re: [PATCH weston] configure.ac: honour the wayland-scanner.pc

On 16 February 2015 at 13:04, Andrew Oakley aoak...@espial.com wrote: On 10/02/15 14:42, Emil Velikov wrote: Currently we use the wayland-scanner executable as found with AC_PATH_PROG, and after that check the presence of wayland-scanner.pc Even if the latter is pointing to another wayland

Re: [PATCH weston] configure.ac: honour the wayland-scanner.pc

On 13 February 2015 at 15:24, Daniel Stone dan...@fooishbar.org wrote: Hi, On 10 February 2015 at 14:42, Emil Velikov emil.l.veli...@gmail.com wrote: -AC_PATH_PROG([wayland_scanner], [wayland-scanner]) +PKG_CHECK_MODULES(WAYLAND_SCANNER, [wayland-scanner], + wayland-scanner

Re: [PATCH weston] configure.ac: honour the wayland-scanner.pc

On 13 February 2015 at 18:58, Bill Spitzak spit...@gmail.com wrote: On 02/10/2015 06:42 AM, Emil Velikov wrote: Currently we use the wayland-scanner executable as found with AC_PATH_PROG, and after that check the presence of wayland-scanner.pc Even if the latter is pointing to another

[PATCH weston] configure.ac: honour the wayland-scanner.pc

, and then fall back to checking via AC_PATH_PROG. XXX: Should we just drop the AC_PATH_PROG check altogether ? Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 1db9f79

<    1   2   3   4