On Saturday, 2018-08-18 12:25:42 +0100, Eric Engestrom wrote:
> On Friday, 2018-08-17 10:55:28 -0700, Dylan Baker wrote:
> > I don't care one way or the other on this, but it needs to wait for the
> > series I
> > have to make two of the test cases py 3 ready, and for a patch from Mathieu
> > to
> > make another py 3 ready.
> >
> > Just for reference, it is convention (though I don't care to keep it) to use
> > such a shebang to mark whether a script is python2, python3, or both (using
> > python is supposed to mean python 2 and 3 compatible.)
>
> Yeah, for that reason I'd rather keep them...
> Not a nack either, just personal preference, and since they're probably
> not correct with that regard anyway... meh
To be clear, I'm definitely in favour of dropping the execute bit though, and
thinking about it, I've changed my mind: let's drop the shebang as well.
Reviewed-by: Eric Engestrom <eric.engest...@intel.com>
>
> >
> > You have my r-b, but please wait to push thisfor the other patches to land
> > or
> > you'll break the meson build and make Mark and Clayton grumpy:
> > Reviewed-by: Dylan Baker <dy...@pnwbakers.com>
> >
> > Quoting Emil Velikov (2018-08-17 04:11:33)
> > > From: Emil Velikov <emil.veli...@collabora.com>
> > >
> > > Just like the rest of the tree - these should be run either as part of
> > > the build system check target, or at the very least with an explicitly
> > > versioned python executable.
> > >
> > > Cc: Dylan Baker <dy...@pnwbakers.com>
> > > Fixes: db8cd8e3677 ("glcpp/tests: Convert shell scripts to a python
> > > script")
> > > Fixes: 97c28cb0823 ("glsl/tests: Convert optimization-test.sh to pure
> > > python")
> > > Fixes: 3b52d292273 ("glsl/tests: reimplement warnings-test in python")
> > > Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
> > > ---
> > > src/compiler/glsl/glcpp/tests/glcpp_test.py | 1 -
> > > src/compiler/glsl/tests/optimization_test.py | 1 -
> > > src/compiler/glsl/tests/warnings_test.py | 1 -
> > > 3 files changed, 3 deletions(-)
> > > mode change 100755 => 100644 src/compiler/glsl/glcpp/tests/glcpp_test.py
> > > mode change 100755 => 100644 src/compiler/glsl/tests/optimization_test.py
> > > mode change 100755 => 100644 src/compiler/glsl/tests/warnings_test.py
> > >
> > > diff --git a/src/compiler/glsl/glcpp/tests/glcpp_test.py
> > > b/src/compiler/glsl/glcpp/tests/glcpp_test.py
> > > old mode 100755
> > > new mode 100644
> > > index 8ac5d7cb0a1..8c7552124a6
> > > --- a/src/compiler/glsl/glcpp/tests/glcpp_test.py
> > > +++ b/src/compiler/glsl/glcpp/tests/glcpp_test.py
> > > @@ -1,4 +1,3 @@
> > > -#!/usr/bin/env python2
> > > # encoding=utf-8
> > > # Copyright © 2018 Intel Corporation
> > >
> > > diff --git a/src/compiler/glsl/tests/optimization_test.py
> > > b/src/compiler/glsl/tests/optimization_test.py
> > > old mode 100755
> > > new mode 100644
> > > index 577d2dfc20f..f8518a168e0
> > > --- a/src/compiler/glsl/tests/optimization_test.py
> > > +++ b/src/compiler/glsl/tests/optimization_test.py
> > > @@ -1,4 +1,3 @@
> > > -#!/usr/bin/env python2
> > > # encoding=utf-8
> > > # Copyright © 2018 Intel Corporation
> > >
> > > diff --git a/src/compiler/glsl/tests/warnings_test.py
> > > b/src/compiler/glsl/tests/warnings_test.py
> > > old mode 100755
> > > new mode 100644
> > > index 2e0f23180f3..2c4fa5a0d5a
> > > --- a/src/compiler/glsl/tests/warnings_test.py
> > > +++ b/src/compiler/glsl/tests/warnings_test.py
> > > @@ -1,4 +1,3 @@
> > > -#!/usr/bin/env python
> > > # encoding=utf-8
> > > # Copyright © 2017 Intel Corporation
> > >
> > > --
> > > 2.18.0
> > >
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
