[PATCH weston 2/5] tests: use variable for test name in weston-tests-env

[Kristian Høgsberg]

On Wed, Sep 11, 2013 at 03:58:08PM +1000, Peter Hutterer wrote:
> Slightly more readable and makes it easier to switch to use $2 for something
> in the future (if that's ever needed).

Yup, that looks better.

Kristian

> ---
>  tests/weston-tests-env | 17 ++++++++++++-----
>  1 file changed, 12 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/weston-tests-env b/tests/weston-tests-env
> index 2e5fa95..b732250 100755
> --- a/tests/weston-tests-env
> +++ b/tests/weston-tests-env
> @@ -1,5 +1,12 @@
>  #!/bin/bash
>  
> +TESTNAME=$1
> +
> +if test -z "$TESTNAME"; then
> +	echo "usage: $(basename $0) <test name>"
> +	exit 1;
> +fi
> +
>  WESTON=$abs_builddir/../src/weston
>  LOGDIR=$abs_builddir/logs
>  
> @@ -18,17 +25,17 @@ else
>  	BACKEND=$abs_builddir/../src/.libs/wayland-backend.so
>  fi
>  
> -case $1 in
> +case $TESTNAME in
>  	*.la|*.so)
>  		$WESTON --backend=$BACKEND \
> -			--socket=test-$(basename $1) \
> -			--modules=$abs_builddir/.libs/${1/.la/.so},xwayland.so \
> +			--socket=test-$(basename $TESTNAME) \
> +			--modules=$abs_builddir/.libs/${TESTNAME/.la/.so},xwayland.so \
>  			--log="$SERVERLOG" \
>  			&> "$OUTLOG"
>  		;;
>  	*)
> -		WESTON_TEST_CLIENT_PATH=$abs_builddir/$1 $WESTON \
> -			--socket=test-$(basename $1) \
> +		WESTON_TEST_CLIENT_PATH=$abs_builddir/$TESTNAME $WESTON \
> +			--socket=test-$(basename $TESTNAME) \
>  			--backend=$BACKEND \
>  			--log="$SERVERLOG" \
>  			--modules=$abs_builddir/.libs/weston-test.so,xwayland.so \
> -- 
> 1.8.3.1
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel