From: Jeff King <peff@peff.net>
To: git@vger.kernel.org
Cc: Emeric Fermas <emeric.fermas@gmail.com>
Subject: [PATCH 0/3] clone --local fixes
Date: Fri, 25 May 2012 23:42:26 -0400 [thread overview]
Message-ID: <20120526034226.GA14287@sigill.intra.peff.net> (raw)
Emeric mentioned that he was confused by the behavior of
git clone --local file:///path/to/repo
in that the --local is silently ignored. Looking over the documentation,
it is quite unclear whether it is supposed to do anything or not. I
don't think it's an especially pressing use case, so one solution could
be to simply clarify the current behavior in the documentation.
However, I figured this should just be a two-line fix to give it more
sensible behavior, so why not just make it to do the sensible thing? And
while my initial version was two lines, it ended up growing to handle
all of the corner cases properly. So I'm not super-excited about patch
2, but given that it is written, I think it's a reasonable thing to do
(and it is not _too_ complex).
And then while I was on the subject, I did patch 3, which is something
that has bugged me for a long time.
[1/3]: t5701: modernize style
[2/3]: clone: make --local handle URLs
[3/3]: clone: allow --no-local to turn off local optimizations
-Peff
next reply other threads:[~2012-05-26 3:42 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-26 3:42 Jeff King [this message]
2012-05-26 3:42 ` [PATCH 1/3] t5701: modernize style Jeff King
2012-05-26 3:45 ` [PATCH 2/3] clone: make --local handle URLs Jeff King
2012-05-28 18:31 ` Johannes Sixt
2012-05-28 19:10 ` Jeff King
2012-05-26 3:45 ` [PATCH 3/3] clone: allow --no-local to turn off local optimizations Jeff King
2012-05-26 4:11 ` [PATCH 4/3] clone: send diagnostic messages to stderr Jeff King
2012-05-27 6:32 ` [PATCH 0/3] clone --local fixes Junio C Hamano
2012-05-28 5:36 ` Jeff King
2012-05-29 17:43 ` Junio C Hamano
2012-05-30 11:03 ` Jeff King
2012-05-30 11:08 ` Jeff King
2012-05-30 11:09 ` [PATCH 1/2] docs/clone: mention that --local may be ignored Jeff King
2012-05-30 11:10 ` [PATCH 2/2] clone: allow --no-local to turn off local optimizations Jeff King
2012-05-30 17:20 ` Junio C Hamano
2012-05-30 21:59 ` Jeff King
2012-05-30 22:10 ` Junio C Hamano
2012-05-30 23:21 ` Jeff King
2012-05-30 23:33 ` Junio C Hamano
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=20120526034226.GA14287@sigill.intra.peff.net \
--to=peff@peff.net \
--cc=emeric.fermas@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).