From: "Shawn O. Pearce" <spearce@spearce.org>
To: Julian Phillips <julian@quantumfyre.co.uk>
Cc: git@vger.kernel.org
Subject: Re: Is my repository broken?
Date: Tue, 8 Apr 2008 18:55:22 -0400 [thread overview]
Message-ID: <20080408225522.GD10274@spearce.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0804082246160.8753@kaos.quantumfyre.co.uk>
Julian Phillips <julian@quantumfyre.co.uk> wrote:
> [...] In particular I had a number of commits with an empty ident [...]
...
> Also, shouldn't fast-import be imposing the same restrictions on what you
> are allowed to commit that the main git tools do? If not, are such
> restrictions documented so that I can apply them in my conversion script?
Hmm, no. fast-import allows what the generalized data model permits
in the object store, its really plumbing. If you are feeding it
an input stream that creates data that isn't compliant with what
the higher level VCS porcelain wants, well, all I can say is "don't
do that".
The fast-import manual specifically warns in the "merge" command
documentation that you may not want to use more than 15 merge
commands, as it can create a commit that other tools based around
git won't like. But we still let you do it.
We also still let you create a commit with duplicate parents.
Some tools (gitk) have had issues with that in the past, but many of
them have been fixed after a fast-import result was used with them.
From a VCS point of view its silly to list the same ancestor twice.
But from an object model point of view, it may make sense if you
were building something else on top of the core plumbing.
The same holds true for the empty ident.
--
Shawn.
next prev parent reply other threads:[~2008-04-08 22:56 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-08 22:02 Is my repository broken? Julian Phillips
2008-04-08 22:13 ` Junio C Hamano
2008-04-08 22:55 ` Shawn O. Pearce [this message]
2008-04-08 23:21 ` Julian Phillips
2008-04-09 6:50 ` Junio C Hamano
2008-04-09 10:01 ` Julian Phillips
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=20080408225522.GD10274@spearce.org \
--to=spearce@spearce.org \
--cc=git@vger.kernel.org \
--cc=julian@quantumfyre.co.uk \
/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).