On 26 June 2018 at 16:31, Dylan Baker <dy...@pnwbakers.com> wrote:
> Quoting Emil Velikov (2018-06-26 06:19:08)
>> On 21 June 2018 at 16:15, Dylan Baker <dy...@pnwbakers.com> wrote:
>> > Quoting Emil Velikov (2018-04-24 10:49:21)
>> >> From: Emil Velikov <emil.veli...@collabora.com>
>> >>
>> >> Bring back the "detection" of the said variables, to allow
>> >> standalone execution.
>> >>
>> >> Fixes: db8cd8e36771 ("glcpp/tests: Convert shell scripts to a python
>> >> script")
>> >> Cc: Dylan Baker <dy...@pnwbakers.com>
>> >> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
>> >> ---
>> >> src/compiler/glsl/glcpp/tests/glcpp-test.sh | 13 +++++++++++++
>> >> 1 file changed, 13 insertions(+)
>> >>
>> >> diff --git a/src/compiler/glsl/glcpp/tests/glcpp-test.sh
>> >> b/src/compiler/glsl/glcpp/tests/glcpp-test.sh
>> >> index 7ca8aa26a8..b8397ec890 100755
>> >> --- a/src/compiler/glsl/glcpp/tests/glcpp-test.sh
>> >> +++ b/src/compiler/glsl/glcpp/tests/glcpp-test.sh
>> >> @@ -1,3 +1,16 @@
>> >> #!/bin/sh
>> >>
>> >> +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
>> >> +
>> >> $PYTHON2 $srcdir/glsl/glcpp/tests/glcpp_test.py
>> >> $abs_builddir/glsl/glcpp/glcpp $srcdir/glsl/glcpp/tests --unix --windows
>> >> --oldmac --bizarro
>> >> --
>> >> 2.16.0
>> >>
>> >
>> > Hi Emil,
>> >
>> > This doesn't apply cleanly to 18.1 because it needs the previous patch.
>> > Would
>> > you like me to pull that as well, or drop this?
>> >
>> As long as the final patch in the series (mentioned below) is in, I
>> don't have strong opinion on the rest.
>>
>> commit d589eddc8be5240632d42ae1931b0b6a82ff524c
>> Author: Emil Velikov <emil.veli...@collabora.com>
>> Date: Tue Apr 24 18:49:22 2018 +0100
>>
>> glsl/tests/glcpp: reinstate "error out if no tests found"
>>
>>
>> Thanks
>> Emil
>
> That commit is already staged. I don't have strong opinions about these two
> either, but since this and the previous patch are mainly aimed at developers
> and
> most developers don't work in the stable branches, I'm inclined to not pull
> them. Does that sound reasonable?
>
Perfectly aligns with my train of thought ;-)
-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
