Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/807326/?format=api
{ "id": 807326, "url": "http://patchwork.ozlabs.org/api/patches/807326/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/patch/20170829214852.22661-1-judge.packham@gmail.com/", "project": { "id": 18, "url": "http://patchwork.ozlabs.org/api/projects/18/?format=api", "name": "U-Boot", "link_name": "uboot", "list_id": "u-boot.lists.denx.de", "list_email": "u-boot@lists.denx.de", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170829214852.22661-1-judge.packham@gmail.com>", "list_archive_url": null, "date": "2017-08-29T21:48:52", "name": "[U-Boot,v2] patman: add support for omitting bouncing addresses", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "00c80e393f7b8f614d6aa53ce376149db3e14382", "submitter": { "id": 6125, "url": "http://patchwork.ozlabs.org/api/people/6125/?format=api", "name": "Chris Packham", "email": "judge.packham@gmail.com" }, "delegate": { "id": 3184, "url": "http://patchwork.ozlabs.org/api/users/3184/?format=api", "username": "sjg", "first_name": "Simon", "last_name": "Glass", "email": "sjg@chromium.org" }, "mbox": "http://patchwork.ozlabs.org/project/uboot/patch/20170829214852.22661-1-judge.packham@gmail.com/mbox/", "series": [ { "id": 488, "url": "http://patchwork.ozlabs.org/api/series/488/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/list/?series=488", "date": "2017-08-29T21:48:52", "name": "[U-Boot,v2] patman: add support for omitting bouncing addresses", "version": 2, "mbox": "http://patchwork.ozlabs.org/series/488/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/807326/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/807326/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<u-boot-bounces@lists.denx.de>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@bilbo.ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)", "ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"VsIBRTOm\"; dkim-atps=neutral" ], "Received": [ "from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xhj2w4TP7z9s7F\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 07:49:27 +1000 (AEST)", "by lists.denx.de (Postfix, from userid 105)\n\tid DF2D3C2269A; Tue, 29 Aug 2017 21:49:10 +0000 (UTC)", "from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 85AE1C2201C;\n\tTue, 29 Aug 2017 21:49:07 +0000 (UTC)", "by lists.denx.de (Postfix, from userid 105)\n\tid 87CEBC2201C; Tue, 29 Aug 2017 21:49:05 +0000 (UTC)", "from mail-pf0-f194.google.com (mail-pf0-f194.google.com\n\t[209.85.192.194])\n\tby lists.denx.de (Postfix) with ESMTPS id 30F51C2201C\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 21:49:03 +0000 (UTC)", "by mail-pf0-f194.google.com with SMTP id g13so3102484pfm.2\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 14:49:03 -0700 (PDT)", "from chrisp-dl.ws.atlnz.lc ([2001:df5:b000:22:3a2c:4aff:fe70:2b02])\n\tby smtp.gmail.com with ESMTPSA id\n\td69sm5579344pgc.82.2017.08.29.14.48.57\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128);\n\tTue, 29 Aug 2017 14:49:00 -0700 (PDT)" ], "X-Spam-Checker-Version": "SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de", "X-Spam-Level": "", "X-Spam-Status": "No, score=0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id;\n\tbh=nZdI9RXSRcI7k5y6imbqt41qpe5BDo9AiZvZJMy9YEE=;\n\tb=VsIBRTOmOZTojBegEv8caDyRlM29NSDFL9uPFfCDSZodk7ksjjULXWqrrQuvu9OCui\n\tG3RNZ0RwGLtfKNat2eF/kCylUBNHS2a8LW3L5/MeNLCnFuO8lLNzk5RnUcjzLoYZGnvI\n\tTRzkec019RcSbptHxKW+WYzlUqps/aY+PGKIPajttO28/bTBw9VOAIymUguaO9gp3jvs\n\tdx9oW459zfD7mt4/rFsQ84nYDA+cuWouamRMjOayc7NKTUcnBghqFdVTXSqo2Ha+HzwA\n\tP55gpyAFFW+Vxjuo9Q68cKz6ckUT/O16d7RHjXgVB3jtJq8SZujTarRsHLOUWjl/Ax6N\n\tlWdA==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=nZdI9RXSRcI7k5y6imbqt41qpe5BDo9AiZvZJMy9YEE=;\n\tb=A69z2wn2mqkNxv67lHT03Pwh7xiO0Z91+kqJmRph0hNkNyeoMoAyCDxcij4SGZ0NS3\n\twsHvs52L1mrwpDpnuwxc7s5fRILlMVkMBSz319R90UVX3aumrjlRRr90pOuUyVhO6s0o\n\tNm0lY/hSOt/mNco81PRzszPMIxjCS13/KBcgcux/Ol3JVV1Zg0/OBWH95nBpvnO03Rji\n\t1bO/NAIl0E1/9sGmPVGyFzJXiLK9v064Yd1UfZNEhCTL44ayoE1EeL/EAVbHNOHK/4or\n\tqdNREBcqh9PTRzxc97XJjlUr06e/o/298WZCEmpveshAF1a/ERHvoHAy6XP0a505VnH2\n\tctkg==", "X-Gm-Message-State": "AHYfb5hkOFWpr1A8Nc2F9W/vHBiTSdLgdWnD3wuUMRXLlNHHib4LRTQU\n\tWWxsjbE9wMQCwoj8QQQ=", "X-Received": "by 10.98.84.194 with SMTP id i185mr1756442pfb.103.1504043341131; \n\tTue, 29 Aug 2017 14:49:01 -0700 (PDT)", "From": "Chris Packham <judge.packham@gmail.com>", "To": "u-boot@lists.denx.de", "Date": "Wed, 30 Aug 2017 09:48:52 +1200", "Message-Id": "<20170829214852.22661-1-judge.packham@gmail.com>", "X-Mailer": "git-send-email 2.14.1", "Cc": "Chris Packham <judge.packham@gmail.com>", "Subject": "[U-Boot] [PATCH v2] patman: add support for omitting bouncing\n\taddresses", "X-BeenThere": "u-boot@lists.denx.de", "X-Mailman-Version": "2.1.18", "Precedence": "list", "List-Id": "U-Boot discussion <u-boot.lists.denx.de>", "List-Unsubscribe": "<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>", "List-Archive": "<http://lists.denx.de/pipermail/u-boot/>", "List-Post": "<mailto:u-boot@lists.denx.de>", "List-Help": "<mailto:u-boot-request@lists.denx.de?subject=help>", "List-Subscribe": "<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "base64", "Errors-To": "u-boot-bounces@lists.denx.de", "Sender": "\"U-Boot\" <u-boot-bounces@lists.denx.de>" }, "content": "Add support for reading a list of bouncing addresses from a in-tree file\n(doc/bounces) and from the ~/.patman config file. These addresses are\nstripped from the Cc list.\n\nSigned-off-by: Chris Packham <judge.packham@gmail.com>\n---\nThis version supports an in-tree doc/bounces file as well as a section\nin the .patman config file. A slight annoyance is the config file parser\nexpects key: value pairs whereas the bounces file is just a flat list. I\nstarted looking a modeling this after the aliases but then I actually\nthought that if we do want to tag someone as a bouncing address we\nprobably want to be explicit about it.\n\nChanges in v2:\n- better integration with existing configuration\n- documentation\n\n doc/bounces | 3 +++\n tools/patman/README | 12 ++++++++++++\n tools/patman/series.py | 2 ++\n tools/patman/settings.py | 18 ++++++++++++++++++\n 4 files changed, 35 insertions(+)\n create mode 100644 doc/bounces", "diff": "diff --git a/doc/bounces b/doc/bounces\nnew file mode 100644\nindex 000000000000..d1c5f0d246eb\n--- /dev/null\n+++ b/doc/bounces\n@@ -0,0 +1,3 @@\n+# List of addresses picked up by patman/get_maintainer.pl that are known to\n+# bounce. Addresses are listed one per line and need to match the author\n+# information recorded in git.\ndiff --git a/tools/patman/README b/tools/patman/README\nindex e36857dedea1..8582ed6ba12c 100644\n--- a/tools/patman/README\n+++ b/tools/patman/README\n@@ -84,6 +84,18 @@ Aliases are recursive.\n The checkpatch.pl in the U-Boot tools/ subdirectory will be located and\n used. Failing that you can put it into your path or ~/bin/checkpatch.pl\n \n+If you want to avoid sending patches to email addresses that are picked up\n+by patman but are known to bounce you can add a [bounces] section to your\n+.patman file. Unlike the [alias] section these are simple key: value pairs\n+that are not recursive.\n+\n+>>>\n+\n+[bounces]\n+gonefishing: Fred Bloggs <f.bloggs@napier.net>\n+\n+<<<\n+\n \n If you want to change the defaults for patman's command-line arguments,\n you can add a [settings] section to your .patman file. This can be used\ndiff --git a/tools/patman/series.py b/tools/patman/series.py\nindex d3947a7c2ac5..ddc0993d9eac 100644\n--- a/tools/patman/series.py\n+++ b/tools/patman/series.py\n@@ -10,6 +10,7 @@ import os\n \n import get_maintainer\n import gitutil\n+import settings\n import terminal\n \n # Series-xxx tags that we understand\n@@ -233,6 +234,7 @@ class Series(dict):\n cc += add_maintainers\n elif add_maintainers:\n cc += get_maintainer.GetMaintainer(commit.patch)\n+ cc = set(cc) - set(settings.bounces)\n cc = [m.encode('utf-8') if type(m) != str else m for m in cc]\n all_ccs += cc\n print(commit.patch, ', '.join(set(cc)), file=fd)\ndiff --git a/tools/patman/settings.py b/tools/patman/settings.py\nindex 5f207f5ef1c4..d735ff9ba3c6 100644\n--- a/tools/patman/settings.py\n+++ b/tools/patman/settings.py\n@@ -269,6 +269,19 @@ def _ReadAliasFile(fname):\n if bad_line:\n print(bad_line)\n \n+def _ReadBouncesFile(fname):\n+ \"\"\"Read in the bounces file if it exists\n+\n+ Args:\n+ fname: Filename to read.\n+ \"\"\"\n+ if os.path.exists(fname):\n+ with open(fname) as fd:\n+ for line in fd:\n+ if line.startswith('#'):\n+ continue\n+ bounces.add(line.strip())\n+\n def Setup(parser, project_name, config_fname=''):\n \"\"\"Set up the settings module by reading config files.\n \n@@ -293,10 +306,15 @@ def Setup(parser, project_name, config_fname=''):\n for name, value in config.items('alias'):\n alias[name] = value.split(',')\n \n+ _ReadBouncesFile('doc/bounces')\n+ for name, value in config.items('bounces'):\n+ bounces.add(value)\n+\n _UpdateDefaults(parser, config)\n \n # These are the aliases we understand, indexed by alias. Each member is a list.\n alias = {}\n+bounces = set()\n \n if __name__ == \"__main__\":\n import doctest\n", "prefixes": [ "U-Boot", "v2" ] }