git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Shawn O. Pearce" <spearce@spearce.org>
To: A Large Angry SCM <gitzilla@gmail.com>
Cc: 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 12:32:22 -0700	[thread overview]
Message-ID: <20100327193222.GI10910@spearce.org> (raw)
In-Reply-To: <4BAE5CB9.6020905@gmail.com>

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.

-- 
Shawn.

  reply	other threads:[~2010-03-27 19:32 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 [this message]
2010-03-27 19:39                                 ` A Large Angry SCM
2010-03-27 19:44                                   ` A Large Angry SCM
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=20100327193222.GI10910@spearce.org \
    --to=spearce@spearce.org \
    --cc=apenwarr@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=gitzilla@gmail.com \
    --cc=jrnieder@gmail.com \
    --cc=mike.lifeguard@gmail.com \
    --cc=nico@fluxnic.net \
    --cc=schacon@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).