workflows.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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.


      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