From: Junio C Hamano <gitster@pobox.com>
To: Sahil Dua <sahildua2305@gmail.com>
Cc: Git Mailing List <git@vger.kernel.org>
Subject: Re: What's cooking in git.git (Sep 2017, #03; Fri, 15)
Date: Fri, 22 Sep 2017 09:59:31 +0900 [thread overview]
Message-ID: <xmqqpoajmtu4.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <CALiud+m1PAD8r0ygoUj3g4LzQjdoPu0CmjiejyG8pHOhr3Fk=Q@mail.gmail.com> (Sahil Dua's message of "Thu, 21 Sep 2017 23:14:54 +0200")
Sahil Dua <sahildua2305@gmail.com> writes:
>>
>> * sd/branch-copy (2017-06-18) 3 commits
>> - branch: add a --copy (-c) option to go with --move (-m)
>> - branch: add test for -m renaming multiple config sections
>> - config: create a function to format section headers
>>
>> "git branch" learned "-c/-C" to create and switch to a new branch
>> by copying an existing one.
>>
>> I personally do not think "branch --copy master backup" while on
>> "master" that switches to "backup" is a good UI, and I *will* say
>> "I told you so" when users complain after we merge this down to
>> 'master'.
>
> Junio, what's up with this one? It's been a long time for this being
> cooked in next. Do we have a plan/idea going forward or is it just
> waiting for more feedback?
Thanks for pinging.
I was (in a strange way) hoping that it was just me who felt that a
"copy" operation of the current branch that moves me to a new branch
is a design mistake, and I was planning to start merging this down
to 'next' based on that assumption, but IIRC it turned out that it
wasn't just me. During the renewed discussion I somehow got an
impression that the concensus was for "copy" to just copy without
ever changing what HEAD says, and if an operation that switches to a
new branch based on the current branch is needed, the right place to
do so is to enhance/extend "checkout -b".
My understanding of the next step was for those who are interested
in moving this topic forward to update these patches in that
direction.
next prev parent reply other threads:[~2017-09-22 1:00 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-15 5:58 What's cooking in git.git (Sep 2017, #03; Fri, 15) Junio C Hamano
2017-09-15 7:50 ` Lars Schneider
2017-09-19 1:18 ` Junio C Hamano
2017-09-15 19:14 ` Johannes Schindelin
2017-09-15 20:33 ` Junio C Hamano
2017-09-15 19:23 ` Johannes Schindelin
2017-09-15 20:49 ` Junio C Hamano
2017-09-29 11:39 ` Johannes Schindelin
2017-09-21 21:14 ` Sahil Dua
2017-09-22 0:59 ` Junio C Hamano [this message]
2017-09-22 3:39 ` Junio C Hamano
2017-09-24 16:16 ` Sahil Dua
2017-09-25 0:40 ` Junio C Hamano
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=xmqqpoajmtu4.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=sahildua2305@gmail.com \
/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.