Re: [Piglit] [PATCH 1/4] README.md: Note that python 3.3.x+ is supported

2018-06-11 Thread Dylan Baker
Quoting Rhys Kidd (2018-06-09 12:37:48)
> On 8 June 2018 at 12:15, Dylan Baker  wrote:
> 
> ---
>  README.md | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/README.md b/README.md
> index 30d7a2b5f..2d7d95674 100644
> --- a/README.md
> +++ b/README.md
> @@ -30,7 +30,7 @@ The original tests have been taken from
> 
>  First of all, you need to make sure that the following are installed:
> 
> -  - Python 2.7.x
> +  - Python 2.7.x or 3.3.x+
> 
> 
> Clearer to say?:
> 
> -  - Python 2.7.x
> +  - Python 2.7.x or >=3.4
> 
> As you know Python 3.3.x has reached end-of-life.
> 
> Whilst piglit's code may still works with 3.3, there are a number of
> dependencies which no longer do - as seen in piglit's continuous integration
> testing. I'll separately send a patch for that.

Yeah, it has. On the other hand python 2.7 is positively ancient, and I don't
see us dropping support for it any time soon. I guess before we drop 3.3 support
lets make sure there's no one relying on 3.3.

Dylan

>    - Python Mako module
>    - numpy (http://www.numpy.org)
>    - six (https://pypi.python.org/pypi/six)
> --
> 2.17.1
> 
> ___
> Piglit mailing list
> Piglit@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit
> 
> 


signature.asc
Description: signature
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH 3/4] README.md: Remove PIGLIT_FORCE_GLSLPASER_DESKTOP env variable

2018-06-11 Thread Dylan Baker
Quoting Eric Anholt (2018-06-08 16:33:29)
> Dylan Baker  writes:
> 
> > This is no longer honored, instead use the glslparser_arb_compat
> > profile.
> 
> I still see it in the tree here.

It's used at compile time to generate the glslparser_arb_compat.xml.gz, but when
you load the XML that variable is ignored. Maybe it would be better to rename
the variable in tree to something like _PIGLIT_BUILD_TIME_ARB_COMPAT?

Dylan


signature.asc
Description: signature
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH 1/2] tox: Remove testing of end-of-life Python 3.3

2018-06-11 Thread Eric Engestrom
On Monday, 2018-06-11 06:39:46 -0400, Rhys Kidd wrote:
> On 11 June 2018 at 05:40, Eric Engestrom  wrote:
> 
> > On Saturday, 2018-06-09 15:45:36 -0400, Rhys Kidd wrote:
> > > Python 3.3 reached end-of-life in September 2017 [0].
> > >
> > > Continuous integration testing has identified a number of dependencies
> > are
> > > now failing on python 3.3, such as:
> > >
> > >   Collecting wheel
> > >   wheel requires Python '>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*' but
> > the running Python is 3.3.6
> > >   ...
> > >   OSError: Command /home/travis/build/E...generator/bin/python -
> > setuptools pip wheel failed with error code 1
> > >
> > > [0] https://www.python.org/dev/peps/pep-0398/#x-end-of-life
> > >
> > > Signed-off-by: Rhys Kidd 
> >
> > I just set up the travis build for the github mirror of piglit [1] an
> > hour ago, and noticed this issue.
> >
> > Reviewed-by: Eric Engestrom 
> >
> > Have you also noticed the other issues [2] (asserts firing up)?
> >
> 
> Yes, have seen those. Those asserts() were the next thing I was going to
> dig into.

Brilliant, thanks :)

> 
> 
> >
> > Is there another canonical travis instance of piglit, and I should
> > remove the one I just set up?
> >
> 
> That's as canonical as any other; I believe the Travis CI functionality
> was added originally for individual developers to have the option of testing
> their own patches. Having it run against the mesa3d GitHub read only
> mirror is good though.
> 
> The question of porting this testing functionality over to GitLab's CI is a
> separate, but worthwhile, discussion to have.

Agreed, and until then I'll just leave the Travis instance of the Github
mirror running as is.

> 
> 
> >
> > [1] https://travis-ci.org/mesa3d/piglit
> > [2] https://travis-ci.org/mesa3d/piglit/builds/390672919
> >
> > > ---
> > >  .travis.yml | 2 --
> > >  tox.ini | 6 +++---
> > >  2 files changed, 3 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/.travis.yml b/.travis.yml
> > > index b4cbcca49..b6e8136e8 100644
> > > --- a/.travis.yml
> > > +++ b/.travis.yml
> > > @@ -15,8 +15,6 @@ matrix:
> > >include:
> > >  - python: 2.7
> > >env: TOX_ENV="py27-{generator,noaccel,accel-nix,streams}"
> > > -- python: 3.3
> > > -  env: TOX_ENV="py33-{generator,noaccel,accel-nix,streams}"
> > >  - python: 3.4
> > >env: TOX_ENV="py34-{generator,noaccel,accel-nix,streams}"
> > >  - python: 3.5
> > > diff --git a/tox.ini b/tox.ini
> > > index 12e8abd9f..411972873 100644
> > > --- a/tox.ini
> > > +++ b/tox.ini
> > > @@ -1,5 +1,5 @@
> > >  [tox]
> > > -envlist = py{27,33,34,35,36}-{generator,noaccel},
> > py{27,33,34,35,36}-accel-{win,nix}, py{27,33,34,35,36}-streams
> > > +envlist = py{27,34,35,36}-{generator,noaccel},
> > py{27,34,35,36}-accel-{win,nix}, py{27,34,35,36}-streams
> > >  skipsdist = True
> > >
> > >  [pytest]
> > > @@ -16,7 +16,7 @@ deps =
> > >  accel: simplejson
> > >  generator: numpy==1.7.0
> > >  mock==1.0.1
> > > -py27-accel-nix,py{33,34,35,36}-{accel,noaccel}: psutil
> > > +py27-accel-nix,py{34,35,36}-{accel,noaccel}: psutil
> > >  py27-accel-nix: backports.lzma
> > >  py27-accel-nix: subprocess32
> > >  py{35,36}: mako==1.0.2
> > > @@ -26,7 +26,7 @@ deps =
> > >  pytest-raises
> > >  pytest-warnings
> > >  pytest-timeout
> > > -py{27,33,34}: mako==0.8.0
> > > +py{27,34}: mako==0.8.0
> > >  six==1.5.2
> > >  {accel,noaccel,streams}: jsonschema
> > >  streams: jsonstreams>=0.4.1
> > > --
> > > 2.17.1
> > >
> > > ___
> > > Piglit mailing list
> > > Piglit@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/piglit
> >
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH 1/2] tox: Remove testing of end-of-life Python 3.3

2018-06-11 Thread Rhys Kidd
On 11 June 2018 at 05:40, Eric Engestrom  wrote:

> On Saturday, 2018-06-09 15:45:36 -0400, Rhys Kidd wrote:
> > Python 3.3 reached end-of-life in September 2017 [0].
> >
> > Continuous integration testing has identified a number of dependencies
> are
> > now failing on python 3.3, such as:
> >
> >   Collecting wheel
> >   wheel requires Python '>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*' but
> the running Python is 3.3.6
> >   ...
> >   OSError: Command /home/travis/build/E...generator/bin/python -
> setuptools pip wheel failed with error code 1
> >
> > [0] https://www.python.org/dev/peps/pep-0398/#x-end-of-life
> >
> > Signed-off-by: Rhys Kidd 
>
> I just set up the travis build for the github mirror of piglit [1] an
> hour ago, and noticed this issue.
>
> Reviewed-by: Eric Engestrom 
>
> Have you also noticed the other issues [2] (asserts firing up)?
>

Yes, have seen those. Those asserts() were the next thing I was going to
dig into.


>
> Is there another canonical travis instance of piglit, and I should
> remove the one I just set up?
>

That's as canonical as any other; I believe the Travis CI functionality
was added originally for individual developers to have the option of testing
their own patches. Having it run against the mesa3d GitHub read only
mirror is good though.

The question of porting this testing functionality over to GitLab's CI is a
separate, but worthwhile, discussion to have.


>
> [1] https://travis-ci.org/mesa3d/piglit
> [2] https://travis-ci.org/mesa3d/piglit/builds/390672919
>
> > ---
> >  .travis.yml | 2 --
> >  tox.ini | 6 +++---
> >  2 files changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/.travis.yml b/.travis.yml
> > index b4cbcca49..b6e8136e8 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -15,8 +15,6 @@ matrix:
> >include:
> >  - python: 2.7
> >env: TOX_ENV="py27-{generator,noaccel,accel-nix,streams}"
> > -- python: 3.3
> > -  env: TOX_ENV="py33-{generator,noaccel,accel-nix,streams}"
> >  - python: 3.4
> >env: TOX_ENV="py34-{generator,noaccel,accel-nix,streams}"
> >  - python: 3.5
> > diff --git a/tox.ini b/tox.ini
> > index 12e8abd9f..411972873 100644
> > --- a/tox.ini
> > +++ b/tox.ini
> > @@ -1,5 +1,5 @@
> >  [tox]
> > -envlist = py{27,33,34,35,36}-{generator,noaccel},
> py{27,33,34,35,36}-accel-{win,nix}, py{27,33,34,35,36}-streams
> > +envlist = py{27,34,35,36}-{generator,noaccel},
> py{27,34,35,36}-accel-{win,nix}, py{27,34,35,36}-streams
> >  skipsdist = True
> >
> >  [pytest]
> > @@ -16,7 +16,7 @@ deps =
> >  accel: simplejson
> >  generator: numpy==1.7.0
> >  mock==1.0.1
> > -py27-accel-nix,py{33,34,35,36}-{accel,noaccel}: psutil
> > +py27-accel-nix,py{34,35,36}-{accel,noaccel}: psutil
> >  py27-accel-nix: backports.lzma
> >  py27-accel-nix: subprocess32
> >  py{35,36}: mako==1.0.2
> > @@ -26,7 +26,7 @@ deps =
> >  pytest-raises
> >  pytest-warnings
> >  pytest-timeout
> > -py{27,33,34}: mako==0.8.0
> > +py{27,34}: mako==0.8.0
> >  six==1.5.2
> >  {accel,noaccel,streams}: jsonschema
> >  streams: jsonstreams>=0.4.1
> > --
> > 2.17.1
> >
> > ___
> > Piglit mailing list
> > Piglit@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/piglit
>
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH 1/2] tox: Remove testing of end-of-life Python 3.3

2018-06-11 Thread Eric Engestrom
On Saturday, 2018-06-09 15:45:36 -0400, Rhys Kidd wrote:
> Python 3.3 reached end-of-life in September 2017 [0].
> 
> Continuous integration testing has identified a number of dependencies are
> now failing on python 3.3, such as:
> 
>   Collecting wheel
>   wheel requires Python '>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*' but the 
> running Python is 3.3.6
>   ...
>   OSError: Command /home/travis/build/E...generator/bin/python - setuptools 
> pip wheel failed with error code 1
> 
> [0] https://www.python.org/dev/peps/pep-0398/#x-end-of-life
> 
> Signed-off-by: Rhys Kidd 

I just set up the travis build for the github mirror of piglit [1] an
hour ago, and noticed this issue.

Reviewed-by: Eric Engestrom 

Have you also noticed the other issues [2] (asserts firing up)?

Is there another canonical travis instance of piglit, and I should
remove the one I just set up?

[1] https://travis-ci.org/mesa3d/piglit
[2] https://travis-ci.org/mesa3d/piglit/builds/390672919

> ---
>  .travis.yml | 2 --
>  tox.ini | 6 +++---
>  2 files changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index b4cbcca49..b6e8136e8 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -15,8 +15,6 @@ matrix:
>include:
>  - python: 2.7
>env: TOX_ENV="py27-{generator,noaccel,accel-nix,streams}"
> -- python: 3.3
> -  env: TOX_ENV="py33-{generator,noaccel,accel-nix,streams}"
>  - python: 3.4
>env: TOX_ENV="py34-{generator,noaccel,accel-nix,streams}"
>  - python: 3.5
> diff --git a/tox.ini b/tox.ini
> index 12e8abd9f..411972873 100644
> --- a/tox.ini
> +++ b/tox.ini
> @@ -1,5 +1,5 @@
>  [tox]
> -envlist = py{27,33,34,35,36}-{generator,noaccel}, 
> py{27,33,34,35,36}-accel-{win,nix}, py{27,33,34,35,36}-streams
> +envlist = py{27,34,35,36}-{generator,noaccel}, 
> py{27,34,35,36}-accel-{win,nix}, py{27,34,35,36}-streams
>  skipsdist = True
>  
>  [pytest]
> @@ -16,7 +16,7 @@ deps =
>  accel: simplejson
>  generator: numpy==1.7.0
>  mock==1.0.1
> -py27-accel-nix,py{33,34,35,36}-{accel,noaccel}: psutil
> +py27-accel-nix,py{34,35,36}-{accel,noaccel}: psutil
>  py27-accel-nix: backports.lzma
>  py27-accel-nix: subprocess32
>  py{35,36}: mako==1.0.2
> @@ -26,7 +26,7 @@ deps =
>  pytest-raises
>  pytest-warnings
>  pytest-timeout
> -py{27,33,34}: mako==0.8.0
> +py{27,34}: mako==0.8.0
>  six==1.5.2
>  {accel,noaccel,streams}: jsonschema
>  streams: jsonstreams>=0.4.1
> -- 
> 2.17.1
> 
> ___
> Piglit mailing list
> Piglit@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit