Re: [RFC PATCH v4 11/26] Move WebDAV HTTP push under remote-curl

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Thu, Oct 29, 2009 at 8:00 AM, Shawn O. Pearce <spearce@xxxxxxxxxxx> wrote:
> diff --git a/t/t5540-http-push.sh b/t/t5540-http-push.sh
>[snip]
> @@ -57,11 +58,15 @@ test_expect_failure 'push to remote repository with packed refs' '
>         test $HEAD = $(git rev-parse --verify HEAD))
>  '
>
> -test_expect_success ' push to remote repository with unpacked refs' '
> +test_expect_failure 'push already up-to-date' '
> +       git push
> +'
> +
> +test_expect_success 'push to remote repository with unpacked refs' '
>        (cd "$HTTPD_DOCUMENT_ROOT_PATH"/test_repo.git &&
>         rm packed-refs &&
> -        git update-ref refs/heads/master \
> -               0c973ae9bd51902a28466f3850b543fa66a6aaf4) &&
> +        git update-ref refs/heads/master $ORIG_HEAD &&
> +        git --bare update-server-info) &&
>        git push &&
>        (cd "$HTTPD_DOCUMENT_ROOT_PATH"/test_repo.git &&
>         test $HEAD = $(git rev-parse --verify HEAD))

Clemens, the following addresses your non-desire to remove the
"unpacked refs" test in your earlier email
<20091025161630.GB8532@localhost>.

Now that the first push in "push to remote repository with packed
refs" succeeds, the "unpacked refs" test is redundant. Since http-push
in that first test already updated /refs/heads/master and /info/refs,
'git update-ref' incorrectly reverts the earlier push, and 'git
update-server-info' is redundant.

-- 
Cheers,
Ray Chuan
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]