Re: [PATCH][weston] configure.ac: Add --with-wayland-scanner-path

2017-07-21 Thread Pekka Paalanen
On Thu, 20 Jul 2017 12:21:27 +0200 Quentin Glidic wrote: > On 7/18/17 3:58 PM, Pekka Paalanen wrote: > > On Wed, 24 May 2017 11:32:21 +0300 > > Jussi Kukkonen wrote: > > > >> On 23 May 2017 at 14:51, Quentin Glidic

Re: [PATCH][weston] configure.ac: Add --with-wayland-scanner-path

2017-07-20 Thread Quentin Glidic
On 7/18/17 3:58 PM, Pekka Paalanen wrote: On Wed, 24 May 2017 11:32:21 +0300 Jussi Kukkonen wrote: On 23 May 2017 at 14:51, Quentin Glidic wrote: So you want to break the simple (cross-compiling) cases to fix something that

Re: [PATCH][weston] configure.ac: Add --with-wayland-scanner-path

2017-05-24 Thread Jussi Kukkonen
On 23 May 2017 at 14:51, Quentin Glidic wrote: >> >> >> Also add a AC_SUBST-call (it seems previously the pkg-config value was >> >> never substituted into Makefiles). >> > >> > It was, AC_PATH_PROG() does call AC_SUBST(). >> >> Sure, AC_PATH_PROG() works. As

Re: [PATCH][weston] configure.ac: Add --with-wayland-scanner-path

2017-05-23 Thread Quentin Glidic
On 5/23/17 1:21 PM, Jussi Kukkonen wrote: Thanks for feedback, Anything I didn't comment on I think I agree with -- the wayland-scanner.m4 proposal I'll have to consider but I'd like to be _really_ sure of what most wayland-scanner users want before going that way. If we cannot find a way

Re: [PATCH][weston] configure.ac: Add --with-wayland-scanner-path

2017-05-23 Thread Jussi Kukkonen
Thanks for feedback, Anything I didn't comment on I think I agree with -- the wayland-scanner.m4 proposal I'll have to consider but I'd like to be _really_ sure of what most wayland-scanner users want before going that way. Jussi On 23 May 2017 at 11:56, Quentin Glidic

Re: [PATCH][weston] configure.ac: Add --with-wayland-scanner-path

2017-05-23 Thread Quentin Glidic
On 5/23/17 10:05 AM, Jussi Kukkonen wrote: Modify wayland-scanner lookup: Use the path given by pkg-config but offer an option to override the path with "--with-wayland-scanner-path=PATH". The latter is useful for cross-compile situations. I would rather use --with-wayland-scanner.