From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dog.elm.relay.mailchannels.net (dog.elm.relay.mailchannels.net [23.83.212.48]) by sourceware.org (Postfix) with ESMTPS id 3D7F1386101E for ; Mon, 7 Dec 2020 09:30:51 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 3D7F1386101E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=gotplt.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=siddhesh@gotplt.org X-Sender-Id: dreamhost|x-authsender|siddhesh@gotplt.org Received: from relay.mailchannels.net (localhost [127.0.0.1]) by relay.mailchannels.net (Postfix) with ESMTP id CD02E209E2; Mon, 7 Dec 2020 09:30:49 +0000 (UTC) Received: from pdx1-sub0-mail-a90.g.dreamhost.com (100-96-9-178.trex.outbound.svc.cluster.local [100.96.9.178]) (Authenticated sender: dreamhost) by relay.mailchannels.net (Postfix) with ESMTPA id 66DC920982; Mon, 7 Dec 2020 09:30:49 +0000 (UTC) X-Sender-Id: dreamhost|x-authsender|siddhesh@gotplt.org Received: from pdx1-sub0-mail-a90.g.dreamhost.com (pop.dreamhost.com [64.90.62.162]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384) by 0.0.0.0:2500 (trex/5.18.11); Mon, 07 Dec 2020 09:30:49 +0000 X-MC-Relay: Neutral X-MailChannels-SenderId: dreamhost|x-authsender|siddhesh@gotplt.org X-MailChannels-Auth-Id: dreamhost X-Exultant-Macabre: 0fbc0fa77ea6cf55_1607333449651_3057202838 X-MC-Loop-Signature: 1607333449651:816431978 X-MC-Ingress-Time: 1607333449651 Received: from pdx1-sub0-mail-a90.g.dreamhost.com (localhost [127.0.0.1]) by pdx1-sub0-mail-a90.g.dreamhost.com (Postfix) with ESMTP id 2233B85E4B; Mon, 7 Dec 2020 01:30:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gotplt.org; h=subject:to :cc:references:from:message-id:date:mime-version:in-reply-to :content-type:content-transfer-encoding; s=gotplt.org; bh=dCqqoo gKOBt4GplAGqcNX2Xy89U=; b=fle4m1qMytCQz83XJKnNKQZ9QOm5oBaNYaKL1I mCUJKK4h19oThzsXOuWuaMJDPz8vu7IbFDxGzNiqDpujcPbPhbj9EjW7Vz2DPXBl 8NjnsRJkvdCZCBl/7XmtFzEMJiKLd4/PFiVvhbboTdenKerkpjwtn2UXR6kr9vhP E6FpY= Received: from [192.168.1.111] (unknown [1.186.101.110]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: siddhesh@gotplt.org) by pdx1-sub0-mail-a90.g.dreamhost.com (Postfix) with ESMTPSA id 884297E600; Mon, 7 Dec 2020 01:30:46 -0800 (PST) Subject: Re: [RFC] Updating patchwork patches on commit To: Florian Weimer Cc: libc-alpha@sourceware.org References: <878saam2bh.fsf@oldenburg2.str.redhat.com> X-DH-BACKEND: pdx1-sub0-mail-a90 From: Siddhesh Poyarekar Message-ID: <7f6bcbfd-1749-b9ea-bb6c-ca9e22f8f303@gotplt.org> Date: Mon, 7 Dec 2020 15:00:41 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.5.0 MIME-Version: 1.0 In-Reply-To: <878saam2bh.fsf@oldenburg2.str.redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, NICE_REPLY_A, RCVD_IN_BARRACUDACENTRAL, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Dec 2020 09:30:53 -0000 On 12/7/20 2:15 PM, Florian Weimer wrote: > * Siddhesh Poyarekar: > >> We could achieve this without additional busy work by having the git >> hook send out [pushed] emails to the list in addition to glibc-cvs >> (libc-alpha should be spared the private branch traffic of course) >> whenever it sees a commit that it can't find on patchwork. A nightly >> script can then trivially mark all [pushed] patches as committed. > > I'm not sure if this useful if we can't find the thread to which the > updated commit belongs. That's a broader problem not limited to these [pushed] patches; we currently don't have a way to associate different versions of the same patch. My thinking was that adding these commits won't make things worse and could at least give us confidence that the patches that remain definitely did not make it into the repo and take stroner actions. It could let us to do things like walking backwards in time from committed patches to find patches with identical subject lines and close them off as superseded. It won't catch all superseded patches, but at least we'll get a majority of them. Once the process is bootstrapped, the likelihood of false positives (i.e. marking unrelated patches with the same subject lines) ought to be negligible. Siddhesh