Re: [PATCH weston] compositor: Respect WESTON_BUILD_DIR env var in weston_load_module

2015-06-07 Thread Pekka Paalanen
On Fri, 5 Jun 2015 10:07:16 -0500 Derek Foreman wrote: > We were loading modules out of the system dirs unconditionally, so > tests that loaded modules would use the system ones, or fail if > they weren't installed. > > Signed-off-by: Derek Foreman > --- > > src/compositor.c | 10 +++---

Re: [PATCH weston] compositor: Respect WESTON_BUILD_DIR env var in weston_load_module

2015-06-07 Thread Pekka Paalanen
On Fri, 05 Jun 2015 12:15:12 -0700 "Jon A. Cruz" wrote: > On 06/05/2015 08:07 AM, Derek Foreman wrote: > > WL_EXPORT void * > > weston_load_module(const char *name, const char *entrypoint) > > { > > + const char *builddir = getenv("WESTON_BUILD_DIR"); > > > There's something 'off' here ab

Re: [PATCH weston] compositor: Respect WESTON_BUILD_DIR env var in weston_load_module

2015-06-05 Thread Jon A. Cruz
On 06/05/2015 08:07 AM, Derek Foreman wrote: > We were loading modules out of the system dirs unconditionally, so > tests that loaded modules would use the system ones, or fail if > they weren't installed. > > Signed-off-by: Derek Foreman > --- > > src/compositor.c | 10 +++--- > 1 file cha

Re: [PATCH weston] compositor: Respect WESTON_BUILD_DIR env var in weston_load_module

2015-06-05 Thread Bryce Harrington
On Fri, Jun 05, 2015 at 10:07:16AM -0500, Derek Foreman wrote: > We were loading modules out of the system dirs unconditionally, so > tests that loaded modules would use the system ones, or fail if > they weren't installed. > > Signed-off-by: Derek Foreman Reviewed-by: Bryce Harrington > --- >

[PATCH weston] compositor: Respect WESTON_BUILD_DIR env var in weston_load_module

2015-06-05 Thread Derek Foreman
We were loading modules out of the system dirs unconditionally, so tests that loaded modules would use the system ones, or fail if they weren't installed. Signed-off-by: Derek Foreman --- src/compositor.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/composi