From: Paolo Bonzini <pbonzini@redhat.com>
To: Drew DeVault <sir@cmpwn.com>, workflows@vger.kernel.org
Subject: Re: New list for people to share maintainer workflows
Date: Tue, 24 Sep 2019 11:26:03 +0200 [thread overview]
Message-ID: <044aea5a-91a9-06a5-8503-38bcf9085be1@redhat.com> (raw)
In-Reply-To: <BWZVL56NCQCT.37ZPAQOP2BDSN@homura>
On 14/09/19 18:45, Drew DeVault wrote:
> # Identifying different versions of the same patchset.
>
> Like you mentioned, no one likes Change Ids, including me. So far the
> best option I've considered is checking the date - it gets perserved
> even through rebases and is /basically/ unique. Enough rebasing and it
> *will* get lost, though, and then manual intervention is required.
>
> I've considered a few ways to address this. One route might be to make
> git send-email smarter, and utilize heuristics on the reflog to try and
> identify that a patch is a -v2. Then we can add an email header like
> `Superceeds: <commit or message id or something new>`.
The date however is lost across git-send-email. However, you can match
series, rather than patches; a Superseeds header would be nice for that,
but often times it is enough to match on the subject to get a decent result.
It's then possible to run a minimal matching algorithm to identify a
patch that "moves" across revisions of the series. This is how "git
range-diff" works.
By the way, let me introduce Patchew! [1] Patchew was started when
patchwork seemed to be mostly dead, so it is somewhat similar to
Patchwork 2.0. But it has some nice functionality such as "git
range-diff"-like version comparison, and especially the patch importer
pushes each submitted series to git, complete with Reviewed-by tags and
the like. The result is then used to run test scripts. It also has a
REST API and a simple plugin API, so it should also be quite easy to
write new plugins to automatically parse syzbot or 0day emails and turn
them into test failures.
Paolo
[1] I have mentioned Patchew before on the kernel.org users mailing
list. The largest project using Patchew is QEMU, whose "patch list"
page on patchew.org is at https://patchew.org/QEMU/.
> Less desirable options would affect people's workflows, which is
> something I'd like to avoid. One example is adding new commands which
> explicitly track the evolution of a feature in your local git
> repository.
prev parent reply other threads:[~2019-09-24 9:26 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-13 7:38 Theodore Y. Ts'o
2019-09-13 15:04 ` Randy Dunlap
2019-09-13 15:09 ` [Ksummit-discuss] " Greg KH
2019-09-14 16:45 ` Drew DeVault
2019-09-24 9:26 ` Paolo Bonzini [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=044aea5a-91a9-06a5-8503-38bcf9085be1@redhat.com \
--to=pbonzini@redhat.com \
--cc=sir@cmpwn.com \
--cc=workflows@vger.kernel.org \
/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