Re: [PATCH v4 wayland-protocols] test: add make check

2016-01-12 Thread Peter Hutterer
On Tue, Jan 12, 2016 at 07:31:05PM +, Daniel Stone wrote: > Hi, > > On 12 January 2016 at 19:22, Derek Foreman wrote: > > --- /dev/null > > +++ b/tests/scan.sh > > @@ -0,0 +1,10 @@ > > +#!/bin/sh -e > > + > > +if [ "x$SCANNER" = "x" ] ; then > > + echo "No scanner present, test will fai

Re: [PATCH v4 wayland-protocols] test: add make check

2016-01-12 Thread Daniel Stone
Hi, On 12 January 2016 at 19:22, Derek Foreman wrote: > --- /dev/null > +++ b/tests/scan.sh > @@ -0,0 +1,10 @@ > +#!/bin/sh -e > + > +if [ "x$SCANNER" = "x" ] ; then > + echo "No scanner present, test will fail." 1>&2 > + exit 1 > +fi Bikeshed: exit 77 will skip rather than fail. Ch

[PATCH v4 wayland-protocols] test: add make check

2016-01-12 Thread Derek Foreman
We can now test all the protocol files by running make check (or distcheck) which will pass them through the scanner. Signed-off-by: Derek Foreman --- I haven't kept any RBs as this has changed significantly since anyone reviewed it. Changes: v2: Use #/bin/sh -e and drop the && from the script