[PATCH weston] tests: Skip Xwayland test if binary isn't available

Daniel Stone daniel at fooishbar.org
Wed Nov 30 10:29:16 UTC 2016


Hey,

On 30 November 2016 at 06:05, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> On Tue, Nov 29, 2016 at 11:06:15AM +0000, Daniel Stone wrote:
>> -xwayland_test_weston_CFLAGS = $(AM_CFLAGS) $(XWAYLAND_TEST_CFLAGS)
>> +xwayland_test_weston_CFLAGS = \
>> +     $(AM_CFLAGS) $(XWAYLAND_TEST_CFLAGS) -DXSERVER_PATH='"@XSERVER_PATH@"'
>
> IMO this is a superfluous, there are plenty of lines going over 80
> chars and this doesn't really make it any more readable. either way:

Fair enough, the style is pretty inconsistent. But it's also
inconsistent in the Meson build, so. ;)

Both pushed, thanks for review!

To ssh://git.freedesktop.org/git/wayland/weston
   698f9bf..2295a62  upstream -> master

Cheers,
Daniel


More information about the wayland-devel mailing list