Hi,

Srivardhan Hebbar <sri.heb...@samsung.com> writes:

> +AC_PATH_PROG([wayland_scanner], [wayland-scanner],,
> +             [${WAYLAND_SCANNER_PATH}])

I don't think it makes much sense to search the path for wayland-scanner
when pkg-config already gives you the complete filename. Instead of
trying to derive a directory to search we could just use the filename
directly and check whether it is executable. Then instead of AC_PATH_PROG it
could do something like this:

AC_MSG_CHECKING([for wayland-scanner])
AS_IF([AS_EXECUTABLE_P([${wayland_scanner}])],
      [AC_MSG_RESULT([${wayland_scanner}])
       AC_SUBST(wayland_scanner)],
      [AC_MSG_RESULT([not found])
       AC_MSG_ERROR([wayland-scanner was not found])])

The --wayland-scanner-path option could also be replaced with something
that would just take the whole filename too.

Regards,
- Neil
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to