From: Junio C Hamano <junkio@cox.net>
To: "Steve Frécinaux" <nudrema@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [RFC] git-svn: make git-svn commit-diff able to work without explicit arguments
Date: Tue, 02 Jan 2007 12:30:16 -0800 [thread overview]
Message-ID: <7vr6udtbmv.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <459AA31E.5070705@gmail.com> (Steve Frécinaux's message of "Tue, 02 Jan 2007 19:23:26 +0100")
Steve Frécinaux <nudrema@gmail.com> writes:
> When using git-svn to access a SVN repo, the commit policy may
> vary. While git makes you commit small patches often, svn users tend
> to prefer bigger patches that implement a functionnality at once.
>
> So at the end you have a SVN commit which corresponds to several git ones.
I personally think this is solving a wrong problem. Commit
granularity is a property of the project, the way in which
people involved in the project prefer working. It is not about
"svn users" vs "git users", and it shouldn't be, especially if
the end result is still a single project.
Is git "making you commit small patches often"? I honestly hope
we are not forcing you to do so, although we took pains to make
it easier because it tends to be easier to look at the history
later when commit boundaries match the logical steps of
evolution.
So my suggestion would be to educate people who tend to make too
large commits better separate their commits, and at the same
time coallesce the commits you create on the git side into a
presentable size, if you acquired a bad habit of making too
small commits, so that everybody follows the same commit
granularity guideline set by the project.
next prev parent reply other threads:[~2007-01-02 20:30 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-02 18:23 [RFC] git-svn: make git-svn commit-diff able to work without explicit arguments Steve Frécinaux
2007-01-02 18:40 ` [RFC] " Pierre Habouzit
2007-01-02 19:18 ` [RFC] " Eric Wong
2007-01-02 20:30 ` Junio C Hamano [this message]
2007-01-02 21:13 ` [RFC] " Pierre Habouzit
2007-01-02 21:26 ` Brian Gernhardt
2007-01-02 21:58 ` Jakub Narebski
2007-01-02 22:27 ` Pierre Habouzit
2007-01-02 22:01 ` Junio C Hamano
2007-01-02 23:09 ` Steve Frécinaux
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=7vr6udtbmv.fsf@assigned-by-dhcp.cox.net \
--to=junkio@cox.net \
--cc=git@vger.kernel.org \
--cc=nudrema@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