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

Bryce Harrington bryce at osg.samsung.com
Wed Feb 11 18:26:32 PST 2015


On Mon, Feb 09, 2015 at 04:13:07PM +0000, Daniel Stone wrote:
> 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>

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


More information about the wayland-devel mailing list