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

2015-02-17 Thread Emil Velikov
On 16 February 2015 at 13:04, Andrew Oakley 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-scanner we s

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

2015-02-17 Thread Andrew Oakley
On 16/02/15 22:07, Daniel Stone wrote: > Hi, > > On 16 February 2015 at 20:46, Bill Spitzak wrote: >> I'm pretty certain there was an old version that looked in the PATH, but >> this has the really unfortunate effect that the build does not work, you >> have to manually create the wayland_scanner

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

2015-02-16 Thread Daniel Stone
Hi, On 16 February 2015 at 20:46, Bill Spitzak wrote: > I'm pretty certain there was an old version that looked in the PATH, but > this has the really unfortunate effect that the build does not work, you > have to manually create the wayland_scanner and put it in the path, then run > the build. >

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

2015-02-16 Thread Bill Spitzak
I'm pretty certain there was an old version that looked in the PATH, but this has the really unfortunate effect that the build does not work, you have to manually create the wayland_scanner and put it in the path, then run the build. Would it be possible to search the path, and if not found, s

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

2015-02-16 Thread Andrew Oakley
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-scanner we silently > ignore it. Rework things to check for the relev

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

2015-02-14 Thread Emil Velikov
On 13 February 2015 at 18:58, Bill Spitzak 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 wayland-s

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

2015-02-14 Thread Emil Velikov
On 13 February 2015 at 15:24, Daniel Stone wrote: > Hi, > > On 10 February 2015 at 14:42, Emil Velikov wrote: >> -AC_PATH_PROG([wayland_scanner], [wayland-scanner]) >> +PKG_CHECK_MODULES(WAYLAND_SCANNER, [wayland-scanner], >> + wayland-scanner=`$PKG_CONFIG --variable=wayland-scanner >> way

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

2015-02-13 Thread Bill Spitzak
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 wayland-scanner we silently ignore it. Rework things to check for the releva

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

2015-02-13 Thread Daniel Stone
Hi, On 10 February 2015 at 14:42, Emil Velikov wrote: > -AC_PATH_PROG([wayland_scanner], [wayland-scanner]) > +PKG_CHECK_MODULES(WAYLAND_SCANNER, [wayland-scanner], > + wayland-scanner=`$PKG_CONFIG --variable=wayland-scanner > wayland-scanner, Missing ` (backtick) at the end. The nice th

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

2015-02-13 Thread Emil Velikov
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-scanner we silently ignore it. Rework things to check for the relevant variable in the *.pc file first, and then fal