From: "Catalin Marinas" <catalin.marinas@gmail.com>
To: "Karl Hasselström" <kha@treskal.com>
Cc: "Jason Baron" <jbaron@redhat.com>,
"David Kågedal" <davidk@lysator.liu.se>,
"Yann Dirson" <ydirson@altern.org>, git <git@vger.kernel.org>
Subject: Re: new stacked git feature
Date: Fri, 22 Feb 2008 13:54:12 +0000 [thread overview]
Message-ID: <b0943d9e0802220554x4c0a6c98q39e8b253bb108f1e@mail.gmail.com> (raw)
In-Reply-To: <20080221074543.GB8250@diana.vm.bytemark.co.uk>
On 21/02/2008, Karl Hasselström <kha@treskal.com> wrote:
> On 2008-02-20 23:06:07 +0000, Catalin Marinas wrote:
> > There is also a situation when patches on the remote stack were
> > reordered and with some conflicts fixed. In this case, merging gets
> > even more complicated (I think darcs solves this by making all the
> > patches immutable but the base feature of StGIT is that patches are
> > easily modifiable).
>
>
> The scheme I described should be able to handle this. When a patch is
> pushed, its base is set, and we can just make a normal 3-way merge.
>
> Consider a stack with two patches, a:A->B and b:B->C (A, B, and C are
> commits). In one branch, this is modified to a:A->B1 and b:B1->C1; and
> in the other, the patch order is changed so we get b:A->X and a:X->C.
>
> When we merge these, the base is unchanged (so still A), and the patch
> order is b, a (since it was changed in one branch but not the other).
"stg sync" does pretty much the same now but in a more manual way. I
don't really like the way the conflicts are presented - i.e. you don't
know which patch was modified afterwards because the patches lose this
information (they are not topic branches).
> 1. First we push b. The A->X variant applies trivially, but the
> B1->C1 variant needs the standard 3-way merge.
3-way merge with X and B1 as base? This leads to the current "sync"
issue, you can't tell which patch was original and which was modified.
Just a thought (not that I'd like this feature in StGIT). Someone
tried a project some time ago, similar to StGIT, but using topic
branches rather than individual commits per patch. The GIT history
looked very ugly, especially after re-ordering, but the advantage was
that you can avoid rebasing patches and simply merging changes from
bottom patches into top ones. This would make synchronisation of
patches between branches much easier.
--
Catalin
next prev parent reply other threads:[~2008-02-22 13:54 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20080111194946.GA7504@redhat.com>
2008-02-12 16:42 ` new stacked git feature Catalin Marinas
2008-02-13 0:08 ` Karl Hasselström
2008-02-13 22:29 ` Catalin Marinas
2008-02-13 23:52 ` Karl Hasselström
2008-02-20 23:06 ` Catalin Marinas
2008-02-21 7:45 ` Karl Hasselström
2008-02-22 13:54 ` Catalin Marinas [this message]
2008-02-22 15:19 ` Karl Hasselström
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=b0943d9e0802220554x4c0a6c98q39e8b253bb108f1e@mail.gmail.com \
--to=catalin.marinas@gmail.com \
--cc=davidk@lysator.liu.se \
--cc=git@vger.kernel.org \
--cc=jbaron@redhat.com \
--cc=kha@treskal.com \
--cc=ydirson@altern.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).