All of lore.kernel.org
 help / color / mirror / Atom feed
From: A Large Angry SCM <gitzilla@gmail.com>
To: gitzilla@gmail.com
Cc: "Shawn O. Pearce" <spearce@spearce.org>,
	Nicolas Pitre <nico@fluxnic.net>,
	Scott Chacon <schacon@gmail.com>,
	"Mike.lifeguard" <mike.lifeguard@gmail.com>,
	Avery Pennarun <apenwarr@gmail.com>,
	Junio C Hamano <gitster@pobox.com>,
	Jonathan Nieder <jrnieder@gmail.com>, git <git@vger.kernel.org>
Subject: Re: Tree with leading '0' modes in 1.7.0.3
Date: Sat, 27 Mar 2010 15:44:29 -0400	[thread overview]
Message-ID: <4BAE601D.6010205@gmail.com> (raw)
In-Reply-To: <4BAE5EEC.5090804@gmail.com>

A Large Angry SCM wrote:
> Shawn O. Pearce wrote:
>> A Large Angry SCM <gitzilla@gmail.com> wrote:
>>> Shawn O. Pearce wrote:
>>>> IMHO, this leading '0' thing is a similar breakage.  We shouldn't
>>>> relax CGit or JGit to accept it just because the Ruby implementation
>>>> of Git got the tree encoding wrong.  If anything, we should teach
>>>> these implementations to catch these sorts of problems earlier.
>>> Just add an additional data point, it looks like up to 16 of these 
>>> trees  with zero-padded file modes are reachable from Linus' kernel 
>>> master ref.
>>
>> Frell.
>>
>> We can't ask Linus to rewrite his history to repair this breakage.
>> The fact that its made it into the kernel history means we have
>> to accept this.  The kernel project is simply too large and move
>> too fast for us to ask them to fix their repository history.
>> Smaller projects of 1-2 people, we could have gotten away with
>> asking them to fix their history.
>>
>> I guess that answers the questions then.  CGit permits this with
>> a warning, and must always continue to do that.  And JGit needs to
>> fix itself to do the same.
>>
> 
> Wait a minute, something strange is going on here.
> 
> My combined kernel repository has 16 of these things according to 
> git-fsck. And when I do 'git-fsck torvalds/linux-2.6/master' I get the 
> same 16 BUT when I 'git-rev-list --objects torvalds/linux-2.6/master' 
> they do not appear in the output.
> 

Ignore my noise until some more checking is done!

My combined repository also includes Scott's progit book and examples 
repositories. I'm guessing that git-fsck did not limit itself to just 
the object reachable from the torvalds/linux-2.6/master ref.

  reply	other threads:[~2010-03-27 19:44 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-26 21:56 Tree with leading '0' modes in 1.7.0.3 Shawn O. Pearce
2010-03-26 22:26 ` Jonathan Nieder
2010-03-26 22:29   ` Shawn O. Pearce
2010-03-26 22:40     ` Jonathan Nieder
2010-03-26 23:09       ` Junio C Hamano
2010-03-26 22:59     ` Mike.lifeguard
2010-03-26 23:05       ` Shawn O. Pearce
2010-03-26 23:22         ` Mike.lifeguard
2010-03-26 23:49           ` Jonathan Nieder
2010-03-26 23:50         ` Junio C Hamano
2010-03-26 23:56           ` Avery Pennarun
2010-03-27  0:00             ` Mike.lifeguard
2010-03-27  1:22               ` Shawn O. Pearce
2010-03-27  1:30                 ` Nicolas Pitre
2010-03-27  1:34                   ` Shawn O. Pearce
2010-03-27  1:56                     ` Nicolas Pitre
2010-03-27  2:33                       ` Avery Pennarun
2010-03-27 12:44                       ` Scott Chacon
2010-03-27 14:21                         ` Nicolas Pitre
2010-03-27 19:14                           ` Shawn O. Pearce
2010-03-27 19:30                             ` A Large Angry SCM
2010-03-27 19:32                               ` Shawn O. Pearce
2010-03-27 19:39                                 ` A Large Angry SCM
2010-03-27 19:44                                   ` A Large Angry SCM [this message]
2010-03-27 19:57                                     ` A Large Angry SCM
2010-03-28 17:38                                     ` Sitaram Chamarty
2010-03-28 23:28                                       ` A Large Angry SCM
2010-03-27 20:13                             ` A Large Angry SCM
2010-03-27 20:16                             ` Junio C Hamano
2010-03-27 22:16                             ` Avery Pennarun
2010-03-27  5:16                     ` Junio C Hamano
2010-03-27 19:20                       ` Shawn O. Pearce
2010-03-27 20:04                         ` 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=4BAE601D.6010205@gmail.com \
    --to=gitzilla@gmail.com \
    --cc=apenwarr@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=jrnieder@gmail.com \
    --cc=mike.lifeguard@gmail.com \
    --cc=nico@fluxnic.net \
    --cc=schacon@gmail.com \
    --cc=spearce@spearce.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 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.