Git development
 help / color / mirror / Atom feed
From: Junio C Hamano <junkio@cox.net>
To: Jakub Narebski <jnareb@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] add --remote option to git-clone.
Date: Sat, 03 Feb 2007 01:27:04 -0800	[thread overview]
Message-ID: <7v64aj7frr.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <eq1j8i$rtv$1@sea.gmane.org> (Jakub Narebski's message of "Sat, 03 Feb 2007 10:06:44 +0100")

Jakub Narebski <jnareb@gmail.com> writes:

>> I did not find anything MS'esque in what MST did in his patch,
>> though.  I think it is a reasonable thing to ask for from a
>> clone.  For example, if you are coming from CVS or have used
>> Cogito, cloning a single branch is not an unusual operation at
>> all.
>
> But when we clone whole repository we could have download whole
> object database of cloned repo as-is (perhaps packing loose objects
> in smart/git-aware transports).

I think your point is if you have branches A and B, cloning A+B
as a whole is a lot less expensive than cloning A and then B
separately.  While that is true, I do not think that reasoning
would apply to somebody, especially behind a slow link, who is
only interested in A.  She does not want to pay the cost of A+B,
when she only wants A, even if A+B is much less expensive than
twice the cost of getting A.

Also, if she later gets interested in B, git fetch will make the
"what's common" optimization.  It won't be like cloning A and
then fetching B into the same repository later would be twice as
expensive compared to the case where you start from a clone as a
whole.  The overall transfer cost would probably be comparable.

> By the way, there was discussed idea about marking pu-like branches
> as being rewound (non-fast forwarding) in the config file, and somehow
> transferring this information for git-clone for it to have '+' for
> some refspecs. What happened to that idea? Was it abandoned...

Well, I would not call some idle talk without a concrete design
(preferably with a proof-of-concept code, but that is not an
absolute requirement) "idea".  Something that did not even start
cannot get abandoned ;-).

I thought that it would not hurt to have something like that,
but because in 1.5.0 "git clone" creates fetch configuration of
'forcing' kind by default, I do not think it makes that much of
a difference.

  reply	other threads:[~2007-02-03  9:27 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-01 22:01 [PATCH] add --remote option to git-clone Michael S. Tsirkin
2007-02-01 22:06 ` J. Bruce Fields
2007-02-01 22:14   ` Shawn O. Pearce
2007-02-01 23:01   ` Michael S. Tsirkin
2007-02-01 23:42   ` Yann Dirson
2007-02-01 23:29 ` Johannes Schindelin
2007-02-01 23:31   ` Shawn O. Pearce
2007-02-01 23:47   ` Michael S. Tsirkin
2007-02-01 23:53     ` Johannes Schindelin
2007-02-02  1:25       ` Junio C Hamano
2007-02-02 10:48         ` Johannes Schindelin
2007-02-03  3:00           ` Junio C Hamano
2007-02-03  9:06             ` Jakub Narebski
2007-02-03  9:27               ` Junio C Hamano [this message]
2007-02-05 11:51             ` Johannes Schindelin
2007-02-05 17:58               ` Junio C Hamano
2007-02-05 18:43                 ` Linus Torvalds
2007-02-05 18:55                   ` Michael S. Tsirkin
2007-02-05 19:34                   ` Shawn O. Pearce
2007-02-05 18:48                 ` Michael S. Tsirkin

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=7v64aj7frr.fsf@assigned-by-dhcp.cox.net \
    --to=junkio@cox.net \
    --cc=git@vger.kernel.org \
    --cc=jnareb@gmail.com \
    /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