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

2015-02-24 Thread Bill Spitzak
On 02/24/2015 02:43 AM, Pekka Paalanen wrote: On Mon, 23 Feb 2015 11:20:36 -0800 Bill Spitzak 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 My only concern is that trying to build Waylan

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

2015-02-24 Thread Bill Spitzak
This sounds good to me. Thanks for running the test, I got swamped with other stuff and could not get to it. I think the reason it does not fail is that the output of wayland-scanner is in the git repository and therefore it does not need to be run the first time. I suppose it would fail if so

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

2015-02-24 Thread Pekka Paalanen
On Mon, 23 Feb 2015 11:20:36 -0800 Bill Spitzak 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 > > My only concern is that trying to build Wayland from scratch on a > machine

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

2015-02-23 Thread Bryce Harrington
On Mon, Feb 23, 2015 at 03:10:35PM +0100, 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

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

2015-02-23 Thread Emil Velikov
On 23 February 2015 at 19:20, Bill Spitzak 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 > > > My only concern is that trying to build Wayland from scratch on a machine > that does

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

2015-02-23 Thread Emil Velikov
On 23 February 2015 at 15:31, Emil Velikov wrote: > On 23 February 2015 at 14:10, 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 lat

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

2015-02-23 Thread Emil Velikov
On 23 February 2015 at 14:10, 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 fails to find the >> bin

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

2015-02-23 Thread sardemff7+wayland
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 fails to find the binary. Rework things to use the pkg-config variable as a f

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

2015-02-22 Thread Pekka Paalanen
On Sat, 21 Feb 2015 12:10:43 -0800 Bryce Harrington wrote: > On Sat, Feb 21, 2015 at 11:04:13AM +, Emil Velikov wrote: > > Hi Bryce > > > > On 20 February 2015 at 23:26, Bryce Harrington > > wrote: > > > On Tue, Feb 17, 2015 at 03:13:32PM +, Emil Velikov wrote: > > >> Currently we use

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

2015-02-21 Thread Bryce Harrington
On Sat, Feb 21, 2015 at 11:04:13AM +, Emil Velikov wrote: > Hi Bryce > > On 20 February 2015 at 23:26, Bryce Harrington 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

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

2015-02-21 Thread Emil Velikov
Hi Bryce On 20 February 2015 at 23:26, Bryce Harrington 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 the latter is unused e

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

2015-02-20 Thread Bryce Harrington
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 the latter is unused even if AC_PATH_PROG fails to find the > binary. Rework things to use t

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

2015-02-18 Thread Andrew Oakley
On 17/02/15 15: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 fails to find the > binary. Rework things to use the pkg-config variable

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

2015-02-17 Thread Emil Velikov
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 fails to find the binary. Rework things to use the pkg-config variable as a fall-back. Cc: Andrew Oakley Cc: Bill Spit