All of lore.kernel.org
 help / color / mirror / Atom feed
From: Finn Arne Gangstad <finnag@pvv.org>
To: Daniel Barkalow <barkalow@iabervon.org>
Cc: skimo@liacs.nl, git@vger.kernel.org
Subject: Re: Partial checkouts / submodules
Date: Tue, 20 Nov 2007 20:22:32 +0100	[thread overview]
Message-ID: <20071120192231.GA23240@pvv.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0711201336530.32410@iabervon.org>

On Tue, Nov 20, 2007 at 01:59:41PM -0500, Daniel Barkalow wrote:
> On Tue, 20 Nov 2007, Finn Arne Gangstad wrote:
> 
> > I'll try to boil this down to the simplest case possible. If
> > submodules can do this I'll be really happy :)
> > 
> > Developer A makes a change in submodule1 and in submodule2
> > Developer B makes a change in submodule2 and in submodule3
> > 
> > A and B don't know about eachother. They send their modifications
> > somewhere (push to a shared repository with a well chosen branch name,
> > for example), or send a mail "please pull from my repo" to the patch
> > queue manager.
> > 
> > It is absolutely crucial that for each developer, either both their
> > modifications go in, or none of them. Git should make picking only
> > one of their modifications hard.
> 
> This is the case; if developer A changes 2 from 2-O to 2-A, and developer 
> B changes 2 from 2-O to 2-B, merging both supermodule commits gets a 
> conflict, which requires a merge in submodule 2 before the supermodule 
> merge can be committed.

And this is partly why I wanted to branch all the involved modules: In
~99% of the cases, 2-A and 2-B modify different files, or at least
wildly different parts of the same file, so the merge should be
trivial/automatic. Therefore, the supermodule merge should also be a
trivial/automatic merge - but it isn't, is it?

- Finn Arne

  reply	other threads:[~2007-11-20 19:22 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-20 15:59 Partial checkouts / submodules Finn Arne Gangstad
2007-11-20 17:33 ` Sven Verdoolaege
2007-11-20 18:19   ` Finn Arne Gangstad
2007-11-20 18:42     ` Sven Verdoolaege
2007-11-20 18:59     ` Daniel Barkalow
2007-11-20 19:22       ` Finn Arne Gangstad [this message]
2007-11-20 20:18         ` Daniel Barkalow
2007-11-21  0:04     ` Jakub Narebski
2007-11-20 18:03 ` Daniel Barkalow
2007-11-20 18:26   ` Steven Grimm
2007-11-20 18:36     ` Daniel Barkalow
2007-11-20 18:55     ` Sergei Organov
2007-11-20 19:15       ` Daniel Barkalow

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=20071120192231.GA23240@pvv.org \
    --to=finnag@pvv.org \
    --cc=barkalow@iabervon.org \
    --cc=git@vger.kernel.org \
    --cc=skimo@liacs.nl \
    /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.