From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-3.5 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 18042 invoked from network); 7 Oct 2022 17:25:40 -0000 Received: from lists.zx2c4.com (165.227.139.114) by inbox.vuxu.org with ESMTPUTF8; 7 Oct 2022 17:25:40 -0000 Received: by lists.zx2c4.com (OpenSMTPD) with ESMTP id 2f8e82f2; Fri, 7 Oct 2022 17:25:32 +0000 (UTC) Return-Path: Received: from mail-qv1-xf2d.google.com (mail-qv1-xf2d.google.com [2607:f8b0:4864:20::f2d]) by lists.zx2c4.com (OpenSMTPD) with ESMTPS id 04bb09d5 (TLSv1.3:AEAD-AES256-GCM-SHA384:256:NO) for ; Fri, 7 Oct 2022 17:25:31 +0000 (UTC) Received: by mail-qv1-xf2d.google.com with SMTP id df9so3477108qvb.9 for ; Fri, 07 Oct 2022 10:25:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; h=content-disposition:mime-version:message-id:subject:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=TRuTKUarD3SM2KHiQGoQWBLpeYVxUtk8osohi9jyuBM=; b=F7oE2D+LLqULVWGDbEUXnF3fflunchkUpei8hjSf0pXIazfuYOuZkEDO65o2w2c9Kh BXm2jYeVOhExg1MZ6biE332+HKcoZHlpQVNLAqc+13RIHCrzU5W4XXV7eYyUrlZaatNU rlRVcHRTO6TVd19F1jPucfN3zmx+sKxZ9xV8Y= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-disposition:mime-version:message-id:subject:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=TRuTKUarD3SM2KHiQGoQWBLpeYVxUtk8osohi9jyuBM=; b=KJusk37GvdAzRiNrwEZBgSzJCAJC3Uvpih/vHfshsqouKdVU4eEmUiSh39E/vhiuHn bASAUWO55JSAhgsiWXaEvXIoWNrxiWRHM9gdjURHsF87b6/Mf2p5AzpZzERM0ipSvzE4 Xk9p4m2tjqIzN2yUDMsdkf8WJkngxaWxUPX+v38hSV75mEuos7XuFi0dPaDV+ZV5NQxP feGE6eHQTHQVZFG8WXxjybnjra+ap8UchkD7rvgcIiEsXlOX4oXmva77x7vHzxNXJNVY XbpgsMa2p1IUF2FvgE5EpeuNERx43FiDe3DgWnBJeoHRKOCBcCQ48gSQTzZQd/gHAnsv /Aag== X-Gm-Message-State: ACrzQf3uPYAZG6IND8Ebv/fa4EHypq9S3aBpcHeKOrFRHhhHq9uYdwJ2 zTsFwKv7wU0LmoUVsSavyrZrQ1HAX8FNwQ== X-Google-Smtp-Source: AMsMyM7gYb6tmtILpkBz1rs+bStsKWDwdNurn//13f1aSZteItfOX6q50K7iQ6QSM32ik4vj1CsddA== X-Received: by 2002:ad4:574b:0:b0:4b1:7959:d98f with SMTP id q11-20020ad4574b000000b004b17959d98fmr5286637qvx.38.1665163530034; Fri, 07 Oct 2022 10:25:30 -0700 (PDT) Return-Path: Received: from meerkat.local (bras-base-mtrlpq5031w-grc-33-142-113-79-147.dsl.bell.ca. [142.113.79.147]) by smtp.gmail.com with ESMTPSA id x5-20020a05620a0ec500b006b953a7929csm2354235qkm.73.2022.10.07.10.25.29 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Oct 2022 10:25:29 -0700 (PDT) Date: Fri, 7 Oct 2022 13:25:28 -0400 From: Konstantin Ryabitsev To: cgit@lists.zx2c4.com Subject: RFE: commit git-patch-id lookup links Message-ID: <20221007172528.4ryaoc26wwtsetdw@meerkat.local> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline X-BeenThere: cgit@lists.zx2c4.com X-Mailman-Version: 2.1.30rc1 Precedence: list List-Id: List for cgit developers and users List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: cgit-bounces@lists.zx2c4.com Sender: "CGit" 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: code-review patchid:${patchid}
This would only make sense for non-merge commits (i.e. those with fewer than 2 parents). Thoughts? -K