From: "Shawn O. Pearce" <spearce@spearce.org>
To: Simon Hausmann <simon@lst.de>
Cc: git@vger.kernel.org
Subject: Re: merging git-p4 into git/fastimport.git (second try)
Date: Mon, 18 Jun 2007 19:00:18 -0400 [thread overview]
Message-ID: <20070618230018.GA6143@spearce.org> (raw)
In-Reply-To: <200706190036.01950.simon@lst.de>
Simon Hausmann <simon@lst.de> wrote:
> after reworking most of the really short commit logs and adding the missing
> SOB lines I'd like to ask for consideration of merging git-p4 as part of
> contrib/fastimport/ from
>
> git://repo.or.cz/fast-export.git git-p4
Most of the commits look pretty good. The effort you have put into
cleaning them up shows.
However, many of the oneline descriptions from Han-Wen Nienhuys
are still really short and not very descriptive. They won't look
very good in Junio's "What's in git.git". A few of yours are
also pretty long winded for a typical oneline in git.git, but I'd
rather have a descriptive-but-long oneline anytime over a short
and undescriptive one.
So I'm seriously considering merging this at this point. I'm heading
out to dinner and will look at it more again when I get back, but
I think we should just bite the bullet and bring it into core Git.
--
Shawn.
next prev parent reply other threads:[~2007-06-18 23:00 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-18 22:35 merging git-p4 into git/fastimport.git (second try) Simon Hausmann
2007-06-18 23:00 ` Shawn O. Pearce [this message]
2007-06-19 6:42 ` Simon Hausmann
2007-06-22 23:20 ` Shawn O. Pearce
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=20070618230018.GA6143@spearce.org \
--to=spearce@spearce.org \
--cc=git@vger.kernel.org \
--cc=simon@lst.de \
/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.