On 05/27/2014 06:23 PM, Connor Abbott wrote: > This way, when someone modifies create_test_cases.py and forgets to > commit their changes again, people will notice. > > v2: make sure we parse the right directories and check for existance the > right way. > > Signed-off-by: Connor Abbott <cwabbo...@gmail.com> > --- > src/glsl/tests/optimization-test | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/src/glsl/tests/optimization-test > b/src/glsl/tests/optimization-test > index 8ca7776..bf15153 100755 > --- a/src/glsl/tests/optimization-test > +++ b/src/glsl/tests/optimization-test > @@ -9,6 +9,14 @@ fi > total=0 > pass=0 > > +echo "====== Generating tests ======" > +for dir in tests/*/; do > + if [ -e "${dir}create_test_cases.py" ]; then > + cd $dir; python create_test_cases.py; cd ..
You want to use $PYTHON2 rather than "python" directly. Otherwise, this will break on distros like Arch which (against the Python Foundation's recommendation), install Python 3 as /usr/bin/python. With that fixed, this series is: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> It sounds like Matt may have some comments as well, so I'll hold off on pushing these right away... > + fi > + echo "$dir" > +done > + > echo "====== Testing optimization passes ======" > for test in `find . -iname '*.opt_test'`; do > echo -n "Testing $test..." >
signature.asc
Description: OpenPGP digital signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev