From: Junio C Hamano <gitster@pobox.com>
To: Ben Boeckel <mathstuf@gmail.com>
Cc: git@vger.kernel.org, "Martin Ågren" <martin.agren@gmail.com>,
"Jeff King" <peff@peff.net>,
"Jeff Hostetler" <jeffhost@microsoft.com>,
"Nguyễn Thái Ngọc Duy" <pclouds@gmail.com>,
"Taylor Blau" <me@ttaylorr.com>
Subject: Re: [PATCH 1/1] config: support setting up a remote tracking branch upon creation
Date: Wed, 28 Jul 2021 10:48:56 -0700 [thread overview]
Message-ID: <xmqqim0u9vnb.fsf@gitster.g> (raw)
In-Reply-To: <20210728135041.501850-2-mathstuf@gmail.com> (Ben Boeckel's message of "Wed, 28 Jul 2021 09:50:41 -0400")
Ben Boeckel <mathstuf@gmail.com> writes:
> The `branch.autoSetupMerge` works well for setting up tracking a local
> branch, but there is no mechanism to automatically set up a remote
> tracking situation.
This description is probably insufficient to explain what's missing,
probably because "set up a remote tracking situation" is a bit
fuzzy.
Without this patch, I can do this already:
$ git checkout -t -b topic origin/topic
And after the above, we have
[branch "topic"]
remote = origin
merge = refs/heads/topic
Of course, you can instead use a short-hand DWIM, e.g.
$ git checkout topic ;# assuming origin/topic exists
gives us the same thing. In either case, topic knows to integrate
with the 'topic' branch from remote 'origin' and push back there.
So instead of saying "there is no mechanism to ...", be a bit more
specific what you can and cannot do in this first paragraph.
Then we can describe the solution we'd propose in the second and
subsequent paragraphs.
Thanks.
> +branch.defaultRemote::
> + When a new branch is created with 'git branch', 'git switch' or 'git
> + checkout', this value will be used to initialize the
> + "branch.<name>.remote" setting.
You mean without "-t"? I offhand do not think of a reason why this
is a good idea. How would one create local topic branches that you
plan to merge locally into your own larger topic branch to be shared
with others? Shouldn't there be an easy way to countermand the
setting by this configuration?
> +branch.defaultPushRemote::
> + When a new branch is created with 'git branch', 'git switch' or 'git
> + checkout', this value will be used to initialize the
> + "branch.<name>.pushRemote" setting.
Ditto.
> +branch.defaultMerge::
> + When a new branch is created with 'git branch', 'git switch' or 'git
> + checkout', this value will be used to initialize the
> + "branch.<name>.merge" setting.
So the expected use case is to fork multiple local branches, to
integrate with the same branch from the remote? I think we can
already do
$ git checkout -t -b xyzzy origin/master
$ git checkout -t -b frotz origin/master
and you can lose -t with branch.autosetupmerge setting. As the "git
branch" command needs to get the name of your branch (e.g. 'xyzzy'
or 'frotz') and which remote tracking branch you start from
(e.g. 'origin/master'), even with branch.{defaultRemote,defaultMerge}
configuration, you wouldn't lose that many keystrokes, I suspect.
Or do you plan to make
$ git branch xyzzy
a short-hand for
$ git branch -t xyzzy origin/master
when defaultRemote and defaultMerge are set to 'origin' and
'refs/heads/master'? It would be way too confusing to start the
new branch from origin/master in such a case, as everybody learned
that "git branch xyzzy" that does not say where the branch initially
points at uses HEAD.
next prev parent reply other threads:[~2021-07-28 17:49 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-28 13:50 [PATCH 0/1] Improve automatic setup of tracking for new branches Ben Boeckel
2021-07-28 13:50 ` [PATCH 1/1] config: support setting up a remote tracking branch upon creation Ben Boeckel
2021-07-28 17:48 ` Junio C Hamano [this message]
2021-07-28 18:26 ` Ben Boeckel
2021-07-28 18:39 ` Junio C Hamano
2021-07-29 2:01 ` [PATCH 0/1] Improve automatic setup of tracking for new branches Ben Boeckel
2021-07-29 2:01 ` [PATCH 1/1] config: support a default remote tracking setup upon branch creation Ben Boeckel
2021-07-30 13:35 ` Philippe Blain
2021-07-30 14:07 ` Ben Boeckel
2021-07-30 17:32 ` Junio C Hamano
2021-08-02 13:02 ` Ævar Arnfjörð Bjarmason
2021-08-02 13:16 ` Ben Boeckel
2021-08-02 15:20 ` Ævar Arnfjörð Bjarmason
2021-07-30 1:04 ` [PATCH 0/1] Improve automatic setup of tracking for new branches Junio C Hamano
2021-07-30 1:33 ` Ben Boeckel
2021-07-30 13:35 ` Philippe Blain
2021-07-30 13:57 ` Ben Boeckel
2021-07-30 16:01 ` Philippe Blain
2021-07-30 17:45 ` Ben Boeckel
2021-08-02 21:17 ` Johannes Schindelin
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=xmqqim0u9vnb.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=jeffhost@microsoft.com \
--cc=martin.agren@gmail.com \
--cc=mathstuf@gmail.com \
--cc=me@ttaylorr.com \
--cc=pclouds@gmail.com \
--cc=peff@peff.net \
/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).