public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Markus Klotzbücher" <mk@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] usage of git to send patches to u-boot mailinglist
Date: Thu, 10 Jul 2008 10:34:39 +0200	[thread overview]
Message-ID: <87d4lm2l40.fsf@denx.de> (raw)
In-Reply-To: <20080710002148.GD4657@prithivi.gnumonks.org> (Harald Welte's message of "Thu\, 10 Jul 2008 08\:21\:49 +0800")

Hi Harald,

Harald Welte <laforge@gnumonks.org> writes:

> can do, even though I believe it is by far not the best tool to do so.
> The problem is that I would have to use one local branch per feature
> (i.e. lots of local branches that need to be kept in sync), and even
> then any incremental changes/fixes to one particular feature are visible
> in the commitlog (and thus result in changelog pollution).
>
> My best experience so far really is quilt for maintaining patchsets.
> You can keep a large number of patches, easily switch between them and
> keep your modifications organized per-feature, rather than in the
> chronological commit order of a revision control system.
>
> So what I can probably do is to continue to use quilt up to the point
> where I'd want to send something to a mailinglist, and then put into a
> local git branch, export the patch from there and send it to the list.
>
> However, any further change to that patch based on feedback from the
> list would again go into the quilt tree, I'd have to start with a clean
> 'origin' u-boot git tree and commit the modified change into the git
> tree.   Otherwise we start having all the commit messages (like 'changed
> coding style according to mailinglist feedback') in the code, even
> _before_ that code was ever merged into the respective mainline git
> tree.
>
> So is this really the preferred workflow? How are others dealing with
> this?  How to avoid commitlog pollution?

I never used quilt, but I believe stacked git (stgit) implements more or
less the same behavior on top of git.

Best regards

Markus Klotzbuecher

--
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de

  parent reply	other threads:[~2008-07-10  8:34 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-06 17:06 [U-Boot-Users] [PATCH] Remove code duplication for setting the default environment Harald Welte
2008-07-06 18:57 ` Wolfgang Denk
2008-07-07  7:40   ` Harald Welte
2008-07-09 22:30     ` Wolfgang Denk
2008-07-10  0:21       ` [U-Boot-Users] usage of git to send patches to u-boot mailinglist Harald Welte
2008-07-10  4:36         ` Paul Gortmaker
2008-07-10  8:34         ` Markus Klotzbücher [this message]
2008-07-10  8:49           ` Stefan Roese
2008-07-10  9:00         ` Wolfgang Denk

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=87d4lm2l40.fsf@denx.de \
    --to=mk@denx.de \
    --cc=u-boot@lists.denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox