git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Peter Osterlund <petero2@telia.com>
To: Catalin Marinas <catalin.marinas@gmail.com>
Cc: GIT <git@vger.kernel.org>
Subject: Re: Stacked GIT 0.3 (now more Quilt-like)
Date: 04 Jul 2005 14:32:36 +0200	[thread overview]
Message-ID: <m3y88m21ln.fsf@telia.com> (raw)
In-Reply-To: <1120425269.6845.28.camel@localhost.localdomain>

Catalin Marinas <catalin.marinas@gmail.com> writes:

> On Sun, 2005-07-03 at 14:38 +0200, Peter Osterlund wrote:
> > Catalin Marinas <catalin.marinas@gmail.com> writes:
> > > I know that using -A gives a more detailed output in case of a conflict.
> > > The problem is that you will get a conflict even if the changes are
> > > identical, making it impossible to detect when a patch was merged
> > > upstream.
> > 
> > OK, I see. How about using wiggle instead?
> > 
> >         http://cgi.cse.unsw.edu.au/~neilb/source/wiggle/
> > 
> > That's what patch-utils uses if you run "pushpatch -m". wiggle is also
> > a lot smarter than diff3, so there will be fewer cases that result in
> > a conflict. Maybe a parameter to "stg push" could enable wiggle mode.
> 
> I haven't used wiggle before but I will give it a try (though I prefer
> such a tool not to be too smart since it might make mistakes). Anyway, I
> will make this configurable, i.e. you could put something like below in
> the .stgitrc file:
> 
> merger = 'diff3 -m -E %(branch1)s %(ancestor)s %(branch2)s'
> 
> or
> 
> merger = 'wiggle -m %(branch1)s %(ancestor)s %(branch2)s'
> 
> > Is there a way in StGIT to undo a push that results in a large mess of
> > conflicts?
> 
> Good point. No, there isn't yet. I will think about an undo command. At
> the moment, the old top and bottom ids of a patch are saved so that the
> patch before the merge can be retrieved but there isn't any command to
> make use of them.

I agree with the other comments, it's probably not wise to rely on
wiggle, and wiggle sometimes makes a mess. However, it often does the
right thing, and with a configurable merge program and an undo
function, this should not be a problem. Just undo and try again if you
don't like the result.

-- 
Peter Osterlund - petero2@telia.com
http://web.telia.com/~u89404340

  parent reply	other threads:[~2005-07-04 12:35 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-28 21:26 Stacked GIT 0.3 (now more Quilt-like) Catalin Marinas
2005-07-03  8:38 ` Peter Osterlund
2005-07-03 10:08   ` Catalin Marinas
2005-07-03 12:38     ` Peter Osterlund
2005-07-03 21:14       ` Catalin Marinas
2005-07-04  1:10         ` Horst von Brand
2005-07-04  6:27         ` Martin Langhoff
2005-07-04 12:32         ` Peter Osterlund [this message]
2005-07-04 17:09           ` randy_dunlap
2005-07-04 20:42           ` Catalin Marinas
2005-07-06 20:54           ` Catalin Marinas
2005-07-07 19:17             ` Peter Osterlund
2005-07-07 21:22               ` Catalin Marinas
2005-07-08  1:10 ` Peter Osterlund
2005-07-08  1:24   ` Junio C Hamano
2005-07-08  8:14     ` Peter Osterlund
2005-07-08  9:32       ` Catalin Marinas

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=m3y88m21ln.fsf@telia.com \
    --to=petero2@telia.com \
    --cc=catalin.marinas@gmail.com \
    --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).