Quoting Eric Engestrom (2018-11-08 09:37:22) > On Thursday, 2018-11-08 17:34:33 +0000, Dylan Baker wrote: > > Quoting Eric Engestrom (2018-11-08 09:23:51) > > > The non-ascii char broke the python2 CI: > > > https://travis-ci.org/mesa3d/piglit/jobs/452170869 > > > > > > Fixes: 221d924689e3bb2796ce "framework: Add a vulkan tests profile" > > > Cc: Neil Roberts <nrobe...@igalia.com> > > > Cc: Samuel Iglesias Gonsálvez <sigles...@igalia.com> > > > Cc: Dylan Baker <dy...@pnwbakers.com> > > > Signed-off-by: Eric Engestrom <e...@engestrom.ch> > > > --- > > > framework/test/piglit_test.py | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/framework/test/piglit_test.py b/framework/test/piglit_test.py > > > index c80e355d184fb46ef0d8..83398bf6a719ade2e32c 100644 > > > --- a/framework/test/piglit_test.py > > > +++ b/framework/test/piglit_test.py > > > @@ -248,7 +248,7 @@ def __init__(self, filename): > > > > > > @PiglitBaseTest.command.getter > > > def command(self): > > > - # This is overriden because we don\u2019t want PiglitBaseTest to > > > + # This is overriden because we don't want PiglitBaseTest to > > > > That should just be a plain ', not a unicode, so we should fix that > > That is precisely what this patch is doing :)
Doh! I misread the - and +, my bad :) Reviewed-by: Dylan Baker <dy...@pnwbakers.com>
signature.asc
Description: signature
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit
