All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Kevin Wolf <kwolf@redhat.com>
Cc: Laurent Desnogues <laurent.desnogues@gmail.com>,
	Christoph Egger <Christoph.Egger@amd.com>,
	"C.W. Betts" <computers57@hotmail.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: updating git tree
Date: Tue, 28 Apr 2009 13:14:14 +0200	[thread overview]
Message-ID: <49F6E506.5030506@siemens.com> (raw)
In-Reply-To: <49F6E151.8030507@redhat.com>

Kevin Wolf wrote:
> Jan Kiszka schrieb:
>> Kevin Wolf wrote:
>>> Jan Kiszka schrieb:
>>>> Kevin Wolf wrote:
>>>>> Jan Kiszka schrieb:
>>>>>> If you have non-trivial changes pending, probably in multiple commits, I
>>>>>> can only recommend using stgit (or guilt) to compensate the missing
>>>>>> patch queue feature of git. It allows you to easily navigate back and
>>>>>> forth in your patch queues before finally posting them.
>>>>> I haven't used these yet. Is there a real benefit compared to using a
>>>>> normal git branch and rebase -i? Maybe I should try them if so.
>>>> I'm not only talking about rebasing, also about working within your
>>>> patch queue, editing patches in their middle, splitting it up,
>>>> reordering it etc. There are surely ways to do this with native git
>>>> (stgit is just a front-end and uses normal git), but that's not done
>>>> with two or three git commands.
>>> This is why I said rebase -i and not only rebase. In case you don't know
>>> this yet: It presents you a list of all commits you did since the point
>>> you're rebasing on. You can then drop, merge, edit (which includes
>>> splitting, see the man page of git-rebase) and change the order of them.
>> It still lacks the flexibility and consistency of stgit-managed series
>> as it is designed around the original "rebase" step. With stgit, you are
>> permanently in "rebase -i" mode, you can go back and forth _while_
>> editing. You can switch branches without leaving the rebase mode. You
>> can also hide patches temporarily (how do you do this with rebase -i?).
> 
> Thanks, this is more or less what I wanted to know.
> 
> (And I don't think I had to hide a patch yet, but I would either create
> another branch with the patch dropped or revert the patch and drop the
> revert commit later.)
> 
>> However, in the end it is a question how you set up your personal
>> workflow. There are n ways to skin a cat.
> 
> Sure. But as long as I only know m < n ways, there is always a chance to
> miss the better workflow. ;-)
> 

Yeah, true.

For that and some other reasons it would be great if git itself could
gain fully flexible queue management. stgit is only a workaround that
has some downsides, too (same for guilt). Once I'll find its core
features natively, I'm going to switch over to have only a single UI.

Jan

-- 
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux

  reply	other threads:[~2009-04-28 11:14 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-28  6:50 [Qemu-devel] updating git tree C.W. Betts
2009-04-28  6:50 ` C.W. Betts
2009-04-28  6:54   ` Laurent Desnogues
2009-04-28  7:21     ` Bartlomiej Celary
2009-04-28  8:49     ` Christoph Egger
2009-04-28  9:32       ` Kevin Wolf
2009-04-28  9:38       ` [Qemu-devel] " Jan Kiszka
2009-04-28  9:42         ` Laurent Desnogues
2009-04-28 10:04         ` Kevin Wolf
2009-04-28 10:20           ` Jan Kiszka
2009-04-28 10:28             ` Kevin Wolf
2009-04-28 10:48               ` Jan Kiszka
2009-04-28 10:58                 ` Kevin Wolf
2009-04-28 11:14                   ` Jan Kiszka [this message]
2009-04-28 11:15                 ` François Revol
2009-04-28 10:25           ` Gerd Hoffmann
2009-04-28 11:11       ` [Qemu-devel] " François Revol
2009-04-28 12:03         ` Laurent Desnogues
2009-04-28 12:57           ` François Revol
2009-04-28 13:10             ` Laurent Vivier
2009-04-28 13:49               ` François Revol
2009-04-28 13:58                 ` Kevin Wolf
2009-04-28 15:10                   ` Laurent Vivier
2009-04-28 15:22                     ` Kevin Wolf
2009-04-28 15:40                       ` Laurent Vivier
2009-04-28 16:51               ` Andreas Färber
2009-04-28 19:25                 ` François Revol
2009-04-28 19:49                   ` Andreas Färber
2009-04-28 20:17                     ` François Revol
2009-04-28 21:01                       ` François Revol
2009-04-29  7:54                         ` Markus Armbruster
2009-04-29  8:51                           ` François Revol
2009-04-29  9:22                             ` Markus Armbruster
2009-04-29  9:57                               ` François Revol
2009-04-29  7:44                       ` Markus Armbruster

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=49F6E506.5030506@siemens.com \
    --to=jan.kiszka@siemens.com \
    --cc=Christoph.Egger@amd.com \
    --cc=computers57@hotmail.com \
    --cc=kwolf@redhat.com \
    --cc=laurent.desnogues@gmail.com \
    --cc=qemu-devel@nongnu.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 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.