[Mesa-dev] [PATCH v2 18/23] glcpp/tests/glcpp-test: set srcdir/abs_builddir variables

Emil Velikov emil.l.velikov at gmail.com
Fri Mar 17 13:19:44 UTC 2017


From: Emil Velikov <emil.velikov at collabora.com>

Current definitions work fine for the manual invokation of the script,
although the whole script does not consider that one can run it OOT.

The latter will be handled with latter patches, although it will be
extensively using the two variables.

Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
---
 src/compiler/glsl/glcpp/tests/glcpp-test.sh | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/compiler/glsl/glcpp/tests/glcpp-test.sh b/src/compiler/glsl/glcpp/tests/glcpp-test.sh
index 3945ee4f6ce..3546a3e0168 100755
--- a/src/compiler/glsl/glcpp/tests/glcpp-test.sh
+++ b/src/compiler/glsl/glcpp/tests/glcpp-test.sh
@@ -1,15 +1,22 @@
 #!/bin/sh
 
-if [ ! -z "$srcdir" ]; then
-   testdir=$srcdir/glsl/glcpp/tests
-   outdir=`pwd`/glsl/glcpp/tests
-   glcpp=`pwd`/glsl/glcpp/glcpp
-else
-   testdir=.
-   outdir=.
-   glcpp=../glcpp
+if [ -z "$srcdir" -o -z "$abs_builddir" ]; then
+    echo ""
+    echo "Warning: you're invoking the script manually and things may fail."
+    echo "Attempting to determine/set srcdir and abs_builddir variables."
+    echo ""
+
+    # Should point to `dirname Makefile.glsl.am`
+    srcdir=./../../../
+    cd `dirname "$0"`
+    # Should point to `dirname Makefile` equivalent to the above.
+    abs_builddir=`pwd`/../../../
 fi
 
+testdir=$srcdir/glsl/glcpp/tests
+outdir=$abs_builddir/glsl/glcpp/tests
+glcpp=$abs_builddir/glsl/glcpp/glcpp
+
 trap 'rm $test.valgrind-errors; exit 1' INT QUIT
 
 usage ()
-- 
2.11.1



More information about the mesa-dev mailing list