On Mon, Feb 09, 2015 at 04:13:07PM +0000, Daniel Stone wrote:
> Hi,
> 
> On 9 February 2015 at 15:57, Derek Foreman <[email protected]> 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 <[email protected]>

Pushed both these patches and mine:

   82d19aa..023b265  master -> master

Was waiting in case there was any further commentary on my patch, but
there's still a couple days before the release if anyone has an issue
with it.

Bryce
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to