[PATCH weston 0/2] Make distcheck run xwayland-test and pass

Daniel Stone daniel at fooishbar.org
Mon Feb 9 08:13:07 PST 2015


Hi,

On 9 February 2015 at 15:57, Derek Foreman <derekf at osg.samsung.com> wrote:
> This should get make distcheck to successfully run the new Xwayland test.
>
> I changed the default location of Xwayland to /usr/bin/Xwayland.  The
> default location of $bindir/Xwayland doesn't make sense since Xwayland
> isn't part of weston.
>
> I wonder if we need a better test for its location and presence, as
> currently this requires the releaser to have Xwayland installed.

Looks fine to me. You could use AC_PATH_PROG to seed the initial
value, but this is definitely an improvement, so for both:
Reviewed-by: Daniel Stone <daniels at collabora.com>

Cheers,
Daniel


More information about the wayland-devel mailing list