On Sat, Jul 26, 2014 at 5:57 PM, Janne Grunau <[email protected]> wrote:
> Adding 'branch=release/10' to the fate config file will check the
> release/10 branch instead of master. If no branch is specified it will
> use 'master' so that existing config are still valid.
>
> The server side changes are already deployed, see
> https://fate.libav.org/v10/ for an example. The server supports only the
> release/* branches.
>
> The server enforces that a single slot tests always the same branch.
> Please append "-v$RELEASE" to the slot of release branch configs or make
> the slot otherwise unique.
>
> A different fate samples dir is needed for each release branch. make
> fate-rsync has the correct URL in each branch.
So IOW, this is opt-in for clients. I guess that's fine. Please participate.
> ---
> tests/fate.sh | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/tests/fate.sh b/tests/fate.sh
> index 6e0c0c6..af0f6c0 100755
> --- a/tests/fate.sh
> +++ b/tests/fate.sh
> @@ -19,6 +19,8 @@ test -n "$slot" || die "slot not specified"
> test -n "$repo" || die "repo not specified"
> test -d "$samples" || die "samples location not specified"
>
> +: ${branch:=master}
> +
> lock(){
> lock=$1/fate.lock
> (set -C; exec >$lock) 2>/dev/null || return
> @@ -28,14 +30,14 @@ lock(){
> checkout(){
> case "$repo" in
> file:*|/*) src="${repo#file:}" ;;
> - git:*) git clone --quiet "$repo" "$src" ;;
> + git:*) git clone --quiet --branch "$branch" "$repo" "$src" ;;
> esac
> }
>
> update()(
> cd ${src} || return
> case "$repo" in
> - git:*) git fetch --force; git reset --hard origin/master ;;
> + git:*) git fetch --force; git reset --hard "origin/$branch" ;;
> esac
> )
>
> @@ -79,7 +81,7 @@ clean(){
>
> report(){
> date=$(date -u +%Y%m%d%H%M%S)
> - echo "fate:0:${date}:${slot}:${version}:$1:$2:${comment}" >report
> + echo "fate:1:${date}:${slot}:${version}:$1:$2:${branch}:${comment}"
> >report
> cat ${build}/config.fate ${build}/tests/data/fate/*.rep >>report
> test -n "$fate_recv" && $tar report *.log | gzip | $fate_recv
> }
> --
> 2.0.1
>
Assuming that this patch is what is used on
https://fate.libav.org/armv8-linux-qemu-clang-v10, patch LGTM!
Thanka a lot for making this work!
--
regards,
Reinhard
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
