Re: Rework a patch serie

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



moreau francis wrote:
Hello

Several times I have a topic branch that contains several commits like
this:

            "master"
        o---o
             \                    "topic"
              o---o---o---o---o---o


When I think everything is in a good shape I generate several patches to
submit them. For that, I do:

        $ git checkout topic
        $ git-format-patch -s -k -n master
        001-aaa.patch
        002-bbb.patch
        [...]
        006-fff.patch

Then I submit these patches for reviewing. Unfortunately, they're not perfect,
so after some feedbacks from the communauty it appears that I need to make
small modifications in the second commit in the topic branch for example...

What is the best and fastest way to do that ? Should I create a new topic branch
and cherry pick from the old one ?


It really depends on what the problem is. If it's a thinko in the design I usually find it worth keeping to avoid falling in the same trap again. I do the same for bugfixes, although some don't like that (I'm lazy, they're anal). In this case, patches go on top of topic. Typos and stuff can usually be changed inside the patch-file, in which case you can just apply them to a new topic-branch.

It all boils down to how lazy you are, how much of your f***ups you want to preserve (they *are* useful sometimes) and what you prefer, really.

--
Andreas Ericsson                   andreas.ericsson@xxxxxx
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231
-
: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]