From: Shawn Pearce <spearce@spearce.org>
To: "Michael S. Tsirkin" <mst@mellanox.co.il>
Cc: Junio C Hamano <junkio@cox.net>, git@vger.kernel.org
Subject: Re: [PATCH] Only warn about missing branch.<n>.merge in pull.
Date: Tue, 19 Dec 2006 01:54:07 -0500 [thread overview]
Message-ID: <20061219065407.GB2511@spearce.org> (raw)
In-Reply-To: <20061218202803.GB28925@mellanox.co.il>
"Michael S. Tsirkin" <mst@mellanox.co.il> wrote:
> > Quoting r. Junio C Hamano <junkio@cox.net>:
> > I can see a few possibilities:
> >
> > (1) people do not interact with multiple remote repositories
> > regularly, so this is not a problem in practice.
> >
> > (2) people do, but "the first branch listed" rule is good
> > enough in practice. Because they would always say "git
> > pull second which-branch" instead if they want something
> > different, this is a non-issue.
> >
> > (3) branch.$current.merge was a mistake. It should have been
> > branch.$current.merge.$remote. In other words, the
> > configuration should have been about the current branch and
> > the remote repository pair.
> >
> > (4) the current configuration mechanism is fine, but the code
> > is not. We should forbid "the first branch listed" rule
> > from being applied for "git pull second", and require the
> > users to explicitly say which branch(es) to merge.
> >
> > I am inclined to say that (1) is possible, (2) is implausible
> > (otherwise we would not have done 62b339a5 for the same reason),
> > (3) is confused, and probably (4) is what we need.
>
> As a person who tracks multiple remotes in one repository,
> I would say (4) best matches what I do.
>
> So I currently always do git fetch <remote> to download changes,
> and always use git pull . <branch> to merge a specific branch.
Agreed; #4 best matches what I (and those I work with on that ugly
repository of mine) do.
My git.git repository currently just fetches all of Junio's branches
right into my local refs/heads. I fetch every day or so, but never
merge from next into any branch. Though I create branches off next,
master, or some select commit based on whatever topic I'm hacking.
In my other repositories I tend to fetch before merging, for a
number of reasons:
a) Its distributed backup; if I fetch the branch that's one more copy.
b) I can easily view a branch's recent changes, but not affect my
own topics until I'm ready to take them in. People often ask
me to look at their changes, or wonder why something is suddenly
acting odd - looking at the commits in gitk usually tells me
quite quickly who did what. :-)
c) I can easily start a new topic off any recent change.
d) I can easily merge topics once they are local and reviewed.
a,b,d are not quite as necessary in an email-patch based group such
as git itself, as the mailing list offers most of the reasons I
fetch first. b and c are just convience there.
--
prev parent reply other threads:[~2006-12-19 6:54 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-18 9:12 [PATCH] Only warn about missing branch.<n>.merge in pull Shawn O. Pearce
2006-12-18 11:06 ` Santi Béjar
[not found] ` <7virg9xcvw.fsf@assigned-by-dhcp.cox.net>
[not found] ` <Pine.LNX.4.63.0612182135360.19693@wbgn013.biozentrum.uni-wuerzburg.de>
2006-12-19 0:59 ` Josef Weidendorfer
2006-12-19 1:14 ` Junio C Hamano
2006-12-19 10:28 ` Johannes Schindelin
2006-12-19 10:37 ` Junio C Hamano
2006-12-19 10:30 ` Johannes Schindelin
[not found] ` <20061218202803.GB28925@mellanox.co.il>
2006-12-19 6:54 ` Shawn Pearce [this message]
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=20061219065407.GB2511@spearce.org \
--to=spearce@spearce.org \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
--cc=mst@mellanox.co.il \
/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).