From: Tay Ray Chuan <rctay89@gmail.com>
To: git@vger.kernel.org
Subject: [PATCH 0/3] http-push: fix fetching
Date: Sat, 25 Apr 2009 00:35:16 +0800 [thread overview]
Message-ID: <49F1EA44.6010209@gmail.com> (raw)
http-push.c contains a lot of fetching code, but it's not known if it
really works, given that most people fetch first before pushing (after
all, the git manual recommends this).
The first two patches add tests[1] that demonstrate that http-push.c
fails to fetch from the remote repo, and remedy this by ensuring fetch
requests are sent out. Along the way, code that sends out requests on
the requests queue is refactored (run_request_queue()), and a mechanism
against accidental pushing to the remote side is introduced.
Footnotes:
[1] the tests coerce http-push into fetching objects from the repo by
removing local objects before executing http-push, and represent
just one possible workflow that lead to fetching in http-push.
Tay Ray Chuan (3):
t5540-http-push: test fetching of loose objects
t5540-http-push: test fetching of packed objects
http-push: send out fetch requests on queue
http-push.c | 37 ++++++++++++++++++++++++++-----------
t/t5540-http-push.sh | 36 ++++++++++++++++++++++++++++++++++++
2 files changed, 62 insertions(+), 11 deletions(-)
--
Cheers,
Ray Chuan
reply other threads:[~2009-04-24 16:37 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49F1EA44.6010209@gmail.com \
--to=rctay89@gmail.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).