Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-07-14 Thread Pekka Paalanen
On Mon, 13 Jul 2015 14:19:17 -0700 Jon A. Cruz j...@osg.samsung.com wrote: On 07/10/2015 04:32 AM, Pekka Paalanen wrote: As I see it, the main reason the backends are .so files instead of built into libweston or the main executable is that we can restrict the dependencies. I can

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-07-13 Thread Jon A. Cruz
On 07/10/2015 04:32 AM, Pekka Paalanen wrote: As I see it, the main reason the backends are .so files instead of built into libweston or the main executable is that we can restrict the dependencies. I can well imagine a packager splitting Weston into several packages, for example: core,

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-07-10 Thread Pekka Paalanen
On Sat, 27 Jun 2015 12:43:09 +0300 Giulio Camuffo giuliocamu...@gmail.com wrote: 2015-06-24 4:46 GMT+03:00 Jon A. Cruz j...@osg.samsung.com: On 06/22/2015 10:51 PM, Giulio Camuffo wrote: 2015-06-23 3:52 GMT+03:00 Bill Spitzak spit...@gmail.com: It seems like prev On Mon, Jun 22, 2015

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-06-27 Thread Giulio Camuffo
2015-06-24 4:05 GMT+03:00 Jon A. Cruz j...@osg.samsung.com: Just a few minor low-level items in this feedback... [snip] diff --git a/src/compositor.c b/src/compositor.c index 6e111f1..b063265 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -4544,6 +4544,43 @@ fail: return

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-06-23 Thread Jon A. Cruz
On 06/22/2015 10:51 PM, Giulio Camuffo wrote: 2015-06-23 3:52 GMT+03:00 Bill Spitzak spit...@gmail.com: It seems like prev On Mon, Jun 22, 2015 at 1:02 PM, Giulio Camuffo giuliocamu...@gmail.com wrote: The backends used to have lots of code dealing with weston specific configs. To allow

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-06-23 Thread Jon A. Cruz
Just a few minor low-level items in this feedback... On 06/22/2015 01:02 PM, Giulio Camuffo wrote: The backends used to have lots of code dealing with weston specific configs. To allow them to be used by other compositors with their own configurations remove all the usage of weston_config from

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-06-22 Thread Bill Spitzak
It seems like prev On Mon, Jun 22, 2015 at 1:02 PM, Giulio Camuffo giuliocamu...@gmail.com wrote: The backends used to have lots of code dealing with weston specific configs. To allow them to be used by other compositors with their own configurations remove all the usage of weston_config

[PATCH weston 04/11] make the backends compositor-neutral

2015-06-22 Thread Giulio Camuffo
The backends used to have lots of code dealing with weston specific configs. To allow them to be used by other compositors with their own configurations remove all the usage of weston_config from the backends code and move it in weston.c. The backends are now initialized with the new

Re: [PATCH weston 04/11] make the backends compositor-neutral

2015-06-22 Thread Giulio Camuffo
2015-06-23 3:52 GMT+03:00 Bill Spitzak spit...@gmail.com: It seems like prev On Mon, Jun 22, 2015 at 1:02 PM, Giulio Camuffo giuliocamu...@gmail.com wrote: The backends used to have lots of code dealing with weston specific configs. To allow them to be used by other compositors with their