RFE: commit git-patch-id lookup links

Konstantin Ryabitsev konstantin at linuxfoundation.org
Fri Oct 7 17:25:28 UTC 2022


Hi, all:

One of the recent additions to public-inbox was auto-indexing of patches by
their patch-id (see git-patch-id.1). This allows to see a history of many
commits through public-inbox discussions. Using a random commit as an example:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e4dc45b1848bc6bcac31eb1b4ccdd7f6718b3c86

We can convert it to a patch-id and look up its history:

$ echo "https://lore.kernel.org/all/?q=patchid:$(git show e4dc45b1848 | git patch-id --stable | awk '{print $1}')"
https://lore.kernel.org/all/?q=patchid:d91c951338cfe9f06e8b8aa74856f6dc08ac8212

It would be nice to have this feature in cgit:

1. add "patch-id-query-url" parameter, e.g.:

   patch-id-query-url=https://lore.kernel.org/all/?q=patchid:%s

2. when defined, commit views would add another entry in the header table:

   <tr>
     <th>code-review</th>
     <td colspan='2' class='sha1'><a href='https://lore.kernel.org/all/?q=patchid:${patchid}'>patchid:${patchid}</a><br/></td>
   </tr>

This would only make sense for non-merge commits (i.e. those with fewer than 2
parents).

Thoughts?

-K


More information about the CGit mailing list