git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tong Sun <suntong@cpan.org>
To: git@vger.kernel.org
Subject: Recommended work flow with git to send in patches
Date: Tue, 27 Jul 2010 11:31:24 -0400	[thread overview]
Message-ID: <AANLkTiksAOpFG3vGVGcbeZ0NcpQ5FbDjnZ7yDxUsAY_r@mail.gmail.com> (raw)

Hi,

Compressing my "life long story" into a single question -- what's the
recommended work flow to work with git and send in patches, when
upstream might be slow in respond, and require squashing relevant
patches into one?

You can use my following message as a start point, and please answer
my last question, which I've been asking twice (in different ways)
with no answer.

Please CC me when replying.

Thanks

---------- Forwarded message ----------
From: Tong Sun <suntong@cpan.org>
Date: Sun, Jun 6, 2010 at 8:56 PM
Subject: Working with git and sending in patches
To: grml-devel@ml.grml.org


Hi,

Just trying to put all jigsaw puzzle together here. Please correct me
if I'm wrong.

First of all, philosophy for version control with git:

. While developing, small/independent commits are good thing, so that
it's easy to decouple different changes.

. But when integrating something in a main branch, commits should contain all
logical/related changes.

Steps (using grml-debootstrap as an example):

- do initial git pull into grml-debootstrap

  git pull git://git.grml.org/grml-debootstrap master

- Go into grml-debootstrap and start a new branch

  git checkout -b t/my-working-branch

- work on the code, commit, hack, commit, hack, commit -- commit often
& commit small

- when AOK and need to integrate patches into main branch, squash all
patches into one

  git rebase -i origin/master

- send in patches via email (to grml-devel@ml.grml.org)

  git format-patch origin
  git send-email --to grml-devel@ml.grml.org ...

Please correct me if anything above is wrong.

Now, question, having done above, if I start to work some logically
unrelated patches, what steps should I take? (I don't want 'git
rebase' to pick up patches that I've already sent in).

             reply	other threads:[~2010-07-27 15:31 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-27 15:31 Tong Sun [this message]
2010-07-27 15:35 ` Recommended work flow with git to send in patches Ævar Arnfjörð Bjarmason
2010-07-27 15:43   ` Tong Sun
2010-07-27 15:39 ` Ramkumar Ramachandra
2010-07-27 15:47   ` Tong Sun
2010-07-27 16:45     ` Ramkumar Ramachandra
2010-07-27 17:11       ` Jakub Narebski
2010-07-27 17:28         ` Ramkumar Ramachandra
2010-07-27 17:35 ` Jakub Narebski
2010-07-27 19:48   ` Tong Sun
2010-07-28 16:49     ` Jakub Narebski
2010-07-28 22:40   ` Tong Sun
2010-07-28 23:20     ` Jakub Narebski
2010-07-28 23:30       ` Tong Sun

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=AANLkTiksAOpFG3vGVGcbeZ0NcpQ5FbDjnZ7yDxUsAY_r@mail.gmail.com \
    --to=suntong@cpan.org \
    --cc=git@vger.kernel.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 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).