{"id":807326,"url":"http://patchwork.ozlabs.org/api/1.2/patches/807326/?format=json","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/1.2/projects/18/?format=json","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/1.2/people/6125/?format=json","name":"Chris Packham","email":"judge.packham@gmail.com"},"delegate":{"id":3184,"url":"http://patchwork.ozlabs.org/api/1.2/users/3184/?format=json","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/1.2/series/488/?format=json","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":"","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=)","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 ;\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 ; Tue, 29 Aug 2017 21:49:03 +0000 (UTC)","by mail-pf0-f194.google.com with SMTP id g13so3102484pfm.2\n\tfor ; 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 ","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 ","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 ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","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\" "},"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 \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 \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"]}