From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp12.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms12.migadu.com with LMTPS id EJMWI09KhGR1EgEATFOONw (envelope-from ) for ; Sat, 10 Jun 2023 12:02:55 +0200 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp12.migadu.com with LMTPS id 6Ca6Ik9KhGSUlwAAauVa8A (envelope-from ) for ; Sat, 10 Jun 2023 12:02:55 +0200 Received: from mail1.fsfe.org (mail1.fsfe.org [IPv6:2001:aa8:ffed:f5f3::151]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 5D472FDD4 for ; Sat, 10 Jun 2023 12:02:55 +0200 (CEST) From: Jelle Licht DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fsfe.org; s=2021100501; t=1686391373; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=vgunJM/LDa9uZecGzb2n3+7sb0wPqebNueYYmQlBNGQ=; b=H9rzQXja1JQwXAXYdH3QRBv0+TfK0XKg0sxDz4r3ooMs2eCp8Qw5uUAHuUtUEGc3Rt2iSx 5Rt2XI0IgeTDK9S4MwqY4XxTNuJrk224nU2eRR2iF0gcqSJEYtXg/XEwcr8SPScnXAMOef 8/bD5bceEdx5vc2wNqO35n0dX+CGiOI= To: Kyle Meyer Cc: piem@inbox.kyleam.com Subject: Re: [PATCH v2 1/5] piem: Add :gnu-package keyword to piem-inboxes In-Reply-To: <87v8fvnclg.fsf@kyleam.com> References: <87v8fvnclg.fsf@kyleam.com> Date: Sat, 10 Jun 2023 12:02:52 +0200 Message-ID: <87mt17my7n.fsf@fsfe.org> MIME-Version: 1.0 Content-Type: text/plain X-Migadu-Country: DE X-Migadu-Flow: FLOW_IN X-Migadu-Scanner: scn1.migadu.com X-Migadu-Spam-Score: -4.00 Authentication-Results: aspmx1.migadu.com; none X-Migadu-Queue-Id: 5D472FDD4 X-Spam-Score: -4.00 X-TUID: XU2zb5d8fLHz Kyle Meyer writes: > jlicht@fsfe.org writes: > >> With the `:gnu-package` keyword, users can configure a regex to match >> against the X-GNU-PR-PACKAGE email header for the GNU Bug Tracker. > > Is a regex needed, or would plain string comparison (on downcased > string) be sufficient? It is not needed, so I made it a plain-old-string that matches via `regexp-quote' in V3. We can revisit this decision at any point if someone has a need for more extensive matching. > If it is needed, I think the documentation should be adjusted to mention > it's a regex. That's a bit different than the other headers, where the > user input value itself isn't treated as a regex, even if internally the > match is done with a regex, mostly to handle surrounding text that comes > along with the header. (As a complete side remark: looking at the > :address handling, it should probably be more careful to not match just > a substring of an address.) > >> --- >> diff --git a/Documentation/piem.texi b/Documentation/piem.texi >> index 26962a4..99185d3 100644 >> --- a/Documentation/piem.texi >> +++ b/Documentation/piem.texi >> @@ -168,6 +168,12 @@ repository (e.g., if the inbox receives patches for multiple projects, >> or if you use a few dedicated Git worktrees for different types of >> patches), set @code{:coderepo} to a list of locations. >> >> +@cindex issue tracking >> +@cindex Debbugs, issue tracking system >> +When using the Debbugs instance at @uref{https://bugs.gnu.org} to >> +retrieve messages, an inbox can mapped to each message by setting >> +@code{:gnu-package} for an inbox. > > Nice, thanks for updating the texi docs. > > Reading this made me think about other instances (Debian, perhaps > others), but I'll leave worrying about that until someone interested > comes along. AFAIK, the GNU Bug Tracker is a bespoke deployment of Debbugs, to put it nicely. I'd be open to extending support for other instances of Debbugs, but preferably after getting something functional merged first, if it's all the same to you too. >> @@ -393,7 +395,7 @@ files." >> (rx string-start "publicinbox." >> (group (one-or-more not-newline)) "." >> (group >> - (or "address" "coderepo" "listid" "maildir" "url")) >> + (or "address" "coderepo" "listid" "maildir" "url" "gnu-package")) > > Please move "gnu-package" onto the next line to avoid going too far over > ~80 characters. Addressed in V3.