{"id":808588,"url":"http://patchwork.ozlabs.org/api/1.2/patches/808588/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20170901085753.7877-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":"<20170901085753.7877-1-judge.packham@gmail.com>","list_archive_url":null,"date":"2017-09-01T08:57:53","name":"[U-Boot,v3] patman: add support for omitting bouncing addresses","commit_ref":"e11aa602abd3e8007dfd3ed23ebb829101abcfec","pull_url":null,"state":"accepted","archived":false,"hash":"75e8312ceef5f694377784103f8aa6deb9e14fe4","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/20170901085753.7877-1-judge.packham@gmail.com/mbox/","series":[{"id":976,"url":"http://patchwork.ozlabs.org/api/1.2/series/976/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=976","date":"2017-09-01T08:57:53","name":"[U-Boot,v3] patman: add support for omitting bouncing addresses","version":3,"mbox":"http://patchwork.ozlabs.org/series/976/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/808588/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/808588/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=\"DQf3woFe\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xkCnp3Ryfz9t2r\n\tfor ;\n\tFri, 1 Sep 2017 18:58:21 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid B9FA3C21ED6; Fri, 1 Sep 2017 08:58:14 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 7BA3EC21DAA;\n\tFri, 1 Sep 2017 08:58:11 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 76C96C21DAA; Fri, 1 Sep 2017 08:58:10 +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 1AA45C21C51\n\tfor ; Fri, 1 Sep 2017 08:58:09 +0000 (UTC)","by mail-pf0-f194.google.com with SMTP id a2so1292306pfj.4\n\tfor ; Fri, 01 Sep 2017 01:58:09 -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\td196sm2488308pfd.95.2017.09.01.01.58.02\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128);\n\tFri, 01 Sep 2017 01:58:05 -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=veCxV5qxg99zaUo3u3hWueIi5O60RUymiX64MJc146w=;\n\tb=DQf3woFena8ISvoru9RshNCI3DYXi5N69wJI5rPdtKBUfDel7PMVEW/cwXt2OtZ8WT\n\ta5UgGa5PArqEWLHda2qOzlhRGzwxRNAm5Vhn75wsmvjeUv8rT+t+Dob0iRZv2Pun8ZwV\n\tqFU8L/9ZVCx0wvD2d/lSWUzV35ZM4bT33Go+tDbYfJiVY83a/WSRV5XgE4rl+DJ7RRJ9\n\tLmTfhXurO0c2jQq5y7cvzJsTKu5C/iZWGEYvfmdXuW3XF6YHzmLjOvbUxIxIAvJTqN/O\n\t3OK0v53L7XVnnMH0vU7TCDcB5VZUKtOHW8vD9pznw+HWEXFmkCPb1k0J8wPCg13pJ0k7\n\tkakg==","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=veCxV5qxg99zaUo3u3hWueIi5O60RUymiX64MJc146w=;\n\tb=Ns7FWtsfc0e6ge9Nu1N9IfrILOZFm9EExT+j35qOdV7ixoqQQ2318aUJzL+gbjNCS2\n\tAIH5VQ04SFwCdw0D8zXtNy9MaPkWXb408c/AibpxeRUgi3MknbnQR6DmelbV+mnOerhy\n\tRKlSnqAJZ8VBgDEve4FEbE9tnVYadapv2o2cwW99W2zlxbqDH8kzf2g2UauRbhDavJYj\n\tEMzQz6ulTQDA2xkWdNFOI/gYi/JoygkEK3+/91498jJGHQLxiCMjxR8bi7wIS1Ex7Hdv\n\tqN8hJI+ccyqeGyZw/qvIYKNZ9OMSQKaTJPFTE9isGOc1li9axIDPwCXEjNDFF7ncw3az\n\t24+Q==","X-Gm-Message-State":"AHPjjUjfGBU9eriemCHWu2bimUtRzYqaJJ4fXAiYJoqphZ10GEMl9GnA\n\tCAQFsDq3sWAeyt/CNak=","X-Google-Smtp-Source":"ADKCNb5C51H/hMQbPpHK/p9r03XjHC1jWv/EcXx77jm/U5cRHkPw7ObbUB0wPgNXO4tAIV/O0xADfg==","X-Received":"by 10.99.96.69 with SMTP id u66mr1525900pgb.265.1504256287139;\n\tFri, 01 Sep 2017 01:58:07 -0700 (PDT)","From":"Chris Packham ","To":"u-boot@lists.denx.de","Date":"Fri, 1 Sep 2017 20:57:53 +1200","Message-Id":"<20170901085753.7877-1-judge.packham@gmail.com>","X-Mailer":"git-send-email 2.14.1","Cc":"Chris Packham ","Subject":"[U-Boot] [PATCH v3] 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 \nReviewed-by: Simon Glass \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 v3:\n- print addresses that are skipped\n- add review from Simon\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 | 5 +++++\n tools/patman/settings.py | 18 ++++++++++++++++++\n 4 files changed, 38 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..73ee39448614 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@@ -218,6 +219,7 @@ class Series(dict):\n Return:\n Filename of temp file created\n \"\"\"\n+ col = terminal.Color()\n # Look for commit tags (of the form 'xxx:' at the start of the subject)\n fname = '/tmp/patman.%d' % os.getpid()\n fd = open(fname, 'w')\n@@ -233,6 +235,9 @@ class Series(dict):\n cc += add_maintainers\n elif add_maintainers:\n cc += get_maintainer.GetMaintainer(commit.patch)\n+ for x in set(cc) & set(settings.bounces):\n+ print(col.Color(col.YELLOW, 'Skipping \"%s\"' % x))\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","v3"]}