From: Tejun Heo <tj@kernel.org>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org, kernel-team@fb.com
Subject: Re: [PATCH v2] name_rev: add support for --cherry-picks
Date: Thu, 26 Jul 2018 08:37:14 -0700 [thread overview]
Message-ID: <20180726153714.GX1934745@devbig577.frc2.facebook.com> (raw)
In-Reply-To: <xmqqa7qerpxu.fsf@gitster-ct.c.googlers.com>
Hello, Junio.
On Thu, Jul 26, 2018 at 08:12:45AM -0700, Junio C Hamano wrote:
> Tejun Heo <tj@kernel.org> writes:
>
> > From a6a88c3da252d69547ac8b463098fc4f4c03f322 Mon Sep 17 00:00:00 2001
> > From: Tejun Heo <tj@kernel.org>
> > Date: Thu, 26 Jul 2018 04:14:52 -0700
> > Subject: [PATCH] name_rev: add support for --cherry-picks
>
> The above belongs to the mail header, not the body.
Ah, right, sorry about that.
> > $ git name-rev --cherry-picks 10f7ce0a0e524279f022
> > 10f7ce0a0e524279f022 master~1
> > d433e3b4d5a19b3d29e2c8349fe88ceade5f6190 branch1
> > 82cddd79f962de0bb1e7cdd95d48b48633335816 branch2
> > 58a8d36b2532feb0a14b4fc2a50d587e64f38324 branch3
> > fa8b79edc5dfff21753c2ccfc1a1828336c4c070 branch4~1
>
> "git name-rev X" asks "I want to know about X". And the first line
> of the above tells us that 10f7ce is the first parent of the master
> branch. What does the second line tell us? 10f7ce was created by
> cherry picking d433e3b4 which sits at the tip of branch1?
>
> It appears that you are showing the reverse (d433e3, 58a8d3 and
> fa8b79 sit next to each other, but it cannot be that 10f7ce was
> created by cherry-picking these three). I do not mean to say that
So, it means that d433e, 58a8d and fa8b7 are created by cherry picking
10f7c and 72cdd is created by cherry picking d433e.
> the reverse information is not useful thing to learn about the
> commit (i.e. "X got cherry-picked to all these places") but I am
> having a hard time convincing myself that the feature sits well in
> "describe" and "name-rev".
I should have explained the use case better. Let's say I'm forking
off and maintaining prod releases. We branch it off of a major
upstream version and keeps developing / backporting on that branch
reguarly cutting releases. A lot of commits get cherry-picked back
from master tracking upstream but some are also cherry picked to
sub-release branches for quick-fix releases. e.g.
v4.16
o----o----o----A----o----o..........................................o master
\.----o----o....A'----o----o.................o v4.16-prod
\ \.----o----o v4.16-prod-r2
\ .----o----A'' v4.16-prod-r1
Given a commit, it's useful to find out through which version that got
released, which is where "git-describe --contains" helps. However,
when commits are backported through cherry-picks to prod branches as
in above, that commit is released through multiple versions and it's a
bit painful to hunt them down. This is what --cherry-picks helps
with, so if now I do "git describe --contains --cherry-picks A", it'll
tell me that...
Upstream, it's v4.17-rc1
Backported and released through v4.16-prod-r2
Also backported to v4.16-prod-r1 (cuz the fix happened while r1 was going through rc's)
So, it extends the description of how the original commit is released
(or given name) so that releases through cherry-picks are also
included.
Thanks.
--
tejun
next prev parent reply other threads:[~2018-07-26 15:37 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-26 12:13 [PATCH] name_rev: add support for --cherry-picks Tejun Heo
2018-07-26 14:39 ` [PATCH v2] " Tejun Heo
2018-07-26 15:12 ` Junio C Hamano
2018-07-26 15:37 ` Tejun Heo [this message]
2018-07-26 19:01 ` Junio C Hamano
2018-07-27 8:40 ` Jeff King
2018-07-27 14:47 ` Tejun Heo
2018-07-27 15:55 ` 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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180726153714.GX1934745@devbig577.frc2.facebook.com \
--to=tj@kernel.org \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=kernel-team@fb.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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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 read-only IMAP folder(s) and NNTP newsgroup(s).