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 <[email protected]>
> > > Cc: Samuel Iglesias Gonsálvez <[email protected]>
> > > Cc: Dylan Baker <[email protected]>
> > > Signed-off-by: Eric Engestrom <[email protected]>
> > > ---
> > >  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 <[email protected]>

Attachment: signature.asc
Description: signature

_______________________________________________
Piglit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to