From: Junio C Hamano <gitster@pobox.com>
To: Jeff King <peff@peff.net>
Cc: Michael Haggerty <mhagger@alum.mit.edu>,
Matt Whiteley <mattwhiteley@gmail.com>,
git@vger.kernel.org
Subject: Re: [PATCH 1/2] clone: add disassociate alias to dissociate option
Date: Sun, 22 Feb 2015 10:32:22 -0800 [thread overview]
Message-ID: <xmqqvbitx0eh.fsf@gitster.dls.corp.google.com> (raw)
In-Reply-To: <20150221073509.GA32548@peff.net> (Jeff King's message of "Sat, 21 Feb 2015 02:35:09 -0500")
Jeff King <peff@peff.net> writes:
> I wonder if there is some better word that could become a synonym for
> "--reference --dissociate". Maybe "--borrow", but that does not
> necessarily carry the implication that the relationship ends as soon as
> the clone is done.
You are retracing the exact line of the thinking that led me to a
cop-out that is a separate "--dissociate".
The original idea was to give "--borrow /local/repository/path", but
that would have made it unclear what the differences between that
new option and existing "--reference" were. Both borrow the objects
in order to reduce the network cost, and the difference is that one
keeps borrowing while the other one limits the borrowing to strictly
the initial phase. The two words, "borrow" and "reference", would
not convey that key distinction. "Do the reference thing (which is
well understood from old days, even before v1.6.0) and then severe
the ties" was suboptimal but was easy to explain, and that is why I
call it a cop-out.
> What is really happening is that we are reusing
> objects in order to save bandwidth. Maybe "--reuse-from" would work?
>
> I dunno. I am not extremely happy with any of the suggestions I made,...
I dunno, either.
We are all on the same page. We know the cop-out is suboptimal, we
understand why the cop-out is better than "--borrow", and we cannot
come up with a better name that contrasts with the existing
"--reference" to make it clear how the new thing is different.
next prev parent reply other threads:[~2015-02-22 18:32 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-20 19:10 [PATCH] --disassociate alias for --dissociate clone option Matt Whiteley
2015-02-20 19:10 ` [PATCH 1/2] clone: add disassociate alias to dissociate option Matt Whiteley
2015-02-21 6:27 ` Jeff King
2015-02-21 7:13 ` Junio C Hamano
2015-02-21 7:35 ` Jeff King
2015-02-22 18:32 ` Junio C Hamano [this message]
2015-02-24 9:06 ` Michael Haggerty
2015-02-24 20:06 ` Junio C Hamano
2015-02-24 22:00 ` Michael Haggerty
2015-02-20 19:10 ` [PATCH 2/2] clone: Realign lines near disassociate option Matt Whiteley
2015-02-20 22:01 ` [PATCH] --disassociate alias for --dissociate clone option Eric Sunshine
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=xmqqvbitx0eh.fsf@gitster.dls.corp.google.com \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=mattwhiteley@gmail.com \
--cc=mhagger@alum.mit.edu \
--cc=peff@peff.net \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.