On Wed, May 28, 2014 at 1:09 PM, Kenneth Graunke <kenn...@whitecape.org> wrote:
> 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...
Nah, they're find with me. Nothing I can't change later if needed. The series is
Reviewed-by: Matt Turner <matts...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
