git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Duy Nguyen <pclouds@gmail.com>
To: XinLingchao <douglarek@outlook.com>
Cc: "git@vger.kernel.org" <git@vger.kernel.org>,
	"junchunx.guan@gmail.com" <junchunx.guan@gmail.com>
Subject: Re: About *git clone --depth=n* puzzle
Date: Thu, 15 Aug 2013 09:08:38 +0700	[thread overview]
Message-ID: <CACsJy8DDf64M1qruUNH7M=eB=1cJpYuYXLEkpPUPfrREeSLEew@mail.gmail.com> (raw)
In-Reply-To: <BAY169-W472B90AB796C71E8D0D038B4450@phx.gbl>

On Wed, Aug 14, 2013 at 3:20 PM, XinLingchao <douglarek@outlook.com> wrote:
> Hi Guys,
>
> I think I have got some trouble when I use `git clone --depth=n` command. Take a real repo for example:
>
>
> `git clone https://github.com/douglarek/vimrc.git --depth=1`
>
>
> then I use `git log`:
>
> ```
>     commit d04ca09ecc723739123fae11ad56784eb0c9b36a
>     Author: Lingchao Xin <douglarek@outlook.com>
>     Date:   Fri Jun 7 09:39:22 2013 +0800
>
>         Update vnudle to master branch, add wm shortcut, pylint optimization
>
>     commit c66d827dc212f6ae8aab6b5c44631564bcbe2acd
>     Author: icocoa <lingchax@outlook.com>
>     Date:   Tue Mar 12 01:36:34 2013 -0700
>
>         Merge pull request #1 from douglarek/master
>
>         Add markdown, flake8, pylint plugin
>
> ```
>     but when I clone it with `file://`:
>
>     git clone https://github.com/douglarek/vimrc.git
>     git clone file://vimrc.git --depth=1 vimrc1
>
>     I got this log:
> ```
>     commit d04ca09ecc723739123fae11ad56784eb0c9b36a
> Author: Lingchao Xin <douglarek@outlook.com>
> Date:   Fri Jun 7 09:39:22 2013 +0800
>
>     Update vnudle to master branch, add wm shortcut, pylint optimization
> ```
>
> so the two results are not same, is it a bug? or it should be so?
>
> My os is openSUSE 12.3 and git version 1.8.3.4; and myabe it occured in git 1.8.1.4 +.

The fix is at the server side. >=1.8.2 corrects --depth, which is what
you use for file://. github is still on 1.8.1.6, which does not have
that fix. Oh and the commit is 682c7d2 (upload-pack: fix off-by-one
depth calculation in shallow clone - 2013-01-11)
-- 
Duy

      parent reply	other threads:[~2013-08-15  2:09 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-14  8:20 About *git clone --depth=n* puzzle XinLingchao
2013-08-14  8:51 ` Stefan Beller
2013-08-14  9:19   ` XinLingchao
2013-08-14 11:59     ` Jeff King
2013-08-14 12:45       ` XinLingchao
2013-08-15  2:08 ` Duy Nguyen [this message]

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='CACsJy8DDf64M1qruUNH7M=eB=1cJpYuYXLEkpPUPfrREeSLEew@mail.gmail.com' \
    --to=pclouds@gmail.com \
    --cc=douglarek@outlook.com \
    --cc=git@vger.kernel.org \
    --cc=junchunx.guan@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;
as well as URLs for NNTP newsgroup(s).