On 24 February 2017 at 16:13, Eric Engestrom <eric.engest...@imgtec.com> wrote:
> On Thursday, 2017-02-23 16:43:10 +0000, Emil Velikov wrote:
>> From: Emil Velikov <emil.veli...@collabora.com>
>>
>> At the moment things are completely bonkers (as can be seen from last
>> commit). Regardless if we run the test as part of "make check" or
>> standalone the most of the paths are wrong.
>>
>> Untangle things by issuing "cd `dirname "$0`" and working from there.
>> Otherwise it's nearly impossible to consider all the permutations one
>> can use while running outside of "make check"
>>
>> Clearly document the variables and the intended behaviour.
>>
>> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
>> ---
>> We really want to cut down all the crazy design here. Dylan ?
>>
>> sh script which executes a python one, with the latter creating a bunch
>> of sh ones and hardcoding binary location. Then call the generated sh
>> scripts, to create the tested OP files. Now compare the expected vs the
>> tested via another python script which decodes [in to a temporary file]
>> and calls "diff" to compare them.
>> ---
>> src/compiler/glsl/tests/optimization-test | 22 +++++++++++++++++-----
>> 1 file changed, 17 insertions(+), 5 deletions(-)
>>
>> diff --git a/src/compiler/glsl/tests/optimization-test
>> b/src/compiler/glsl/tests/optimization-test
>> index d84b83cfaa..e851e13bfb 100755
>> --- a/src/compiler/glsl/tests/optimization-test
>> +++ b/src/compiler/glsl/tests/optimization-test
>> @@ -1,10 +1,18 @@
>> #!/usr/bin/env bash
>>
>> -if [ ! -z "$srcdir" ]; then
>> - compare_ir=`pwd`/tests/compare_ir
>> -else
>> - compare_ir=./compare_ir
>> +# The srcdir variable must point to the location where the Makefile.am or
>> +# Makefile.glsl.am in particular is. Even if we execute this test manually.
>> +
>> +if [ -z "$srcdir" ]; then
>> + srcdir=./../../
>> +
>> + # In the manual invokation case, we must CD otherwise we cannot manage
>> all
>> + # the permutations of our scripts which calls a script to create a
>> script
>> + # then execute the script design.
>> + cd `dirname "$0"`
>> +
>> fi
>> +compare_ir=$srcdir/glsl/tests/compare_ir
>
> Like I said in the other email, I think if someone wants to run the
> checks manually, they can be expected to set $srcdir correctly.
> Bailing out when it's unset would be good enough IMHO.
>
Based on git log, the lack of srcdir is explicitly required/wanted so
I opted to keep it.
>>
>> if [ -z "$PYTHON2" ]; then
>> PYTHON2=python2
>> @@ -20,10 +28,13 @@ total=0
>> pass=0
>> has_tests=0
>>
>> +# Store our location before we start diving into subdirectories.
>> +ORIGDIR=`pwd`
>
> How about pushd/popd instead?
>
They're missing (yes I know) from sh. Rather keep it compliant,
considering how trivial the WA is
>> echo "====== Generating tests ======"
>> -for dir in tests/*/; do
>> +for dir in $srcdir/glsl/tests/*/; do
>> if [ -e "${dir}create_test_cases.py" ]; then
>> cd $dir;
>> + echo found
>
> Guessing this line can go? :)
>
Oops, yes it should.
Thanks
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
