[PATCH 0/6] http: push and test fixes
- From
Tay Ray Chuan <rctay89@gmail.com>
- Date
- Oct 25, 2009, 15:16 UTC
- Message-ID
- <20091025231651.18c75559.rctay89@gmail.com>
- In-Reply-To
- <1255577814-14745-1-git-send-email-spearce@spearce.org>
This series applies on top of pu, and should apply cleanly on top of Shawn's v3-smart-http branch.
Patch 1: fixes a bug in http-push.c when fetching packs. This issue is minor and is unlikely to be triggered by normal users (for a possible trigger, see the "http-push fetches packed objects" test in t5540-http-push).
Patch 2: here, I collect my comments to Shawn's "Move WebDAV HTTP push under remote-curl" on the disabling of messages to stderr when --helper-status is used with http-push. I hope this will make it easier for Shawn to squash into his series, in the event he decides to.
Patch 3: also in response to Shawn's "Move WebDAV HTTP push under remote-curl", I collect here my mini-patch to add more reports for transport-helper. Again, I hope this will make it easier for Shawn to squash into his series.
Patch 4-7: update tests, as Shawn's patches changed the behaviour of the http push mechanism (dumb).
Tay Ray Chuan (7):
http-push: fix check condition on http.c::finish_http_pack_request()
http-push: allow stderr messages to appear alongside helper_status
ones
http-push: add more 'error <dst> <why>' status reports
t5540-http-push: expect success when pushing without arguments
t5540-http-push: check existence of fetched files
t5540-http-push: when deleting remote refs, don't need to branch -d
-r
t5540-http-push: remove redundant fetcheshttp-push.c | 30 ++++++++++++++++-------------- t/t5540-http-push.sh | 38 +++++++++++++++++++------------------- 2 files changed, 35 insertions(+), 33 deletions(-)