{"id":18726,"url":"https://patchwork.libcamera.org/api/1.1/covers/18726/?format=json","web_url":"https://patchwork.libcamera.org/cover/18726/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20230612224751.4437-1-laurent.pinchart@ideasonboard.com>","date":"2023-06-12T22:47:47","name":"[libcamera-devel,v1,0/4] utils: checkstyle: Add a commit message trailers checker","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/1.1/people/2/?format=json","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/18726/mbox/","series":[{"id":3920,"url":"https://patchwork.libcamera.org/api/1.1/series/3920/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=3920","date":"2023-06-12T22:47:47","name":"utils: checkstyle: Add a commit message trailers checker","version":1,"mbox":"https://patchwork.libcamera.org/series/3920/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/18726/comments/","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 1375CC31E9\n\tfor ;\n\tMon, 12 Jun 2023 22:47:53 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 70A1761E51;\n\tTue, 13 Jun 2023 00:47:52 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id BB01C6020C\n\tfor ;\n\tTue, 13 Jun 2023 00:47:51 +0200 (CEST)","from pendragon.ideasonboard.com (213-243-189-158.bb.dnainternet.fi\n\t[213.243.189.158])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id A9498C7F\n\tfor ;\n\tTue, 13 Jun 2023 00:47:21 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1686610072;\n\tbh=EKKN1kSquDXDKGzr48/iKveCivDb6CTbX3zXSfmPrEo=;\n\th=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:\n\tList-Help:List-Subscribe:From:Reply-To:From;\n\tb=j+IctNHMBOa6V5AefZFAsafbg1jRbakkyReFn722HO0EM8qNlHOsWCYKCcPlBYd8e\n\tf2SEnPIV99n5AJZvrvZrZgN47AvjK9ntse5HnD24vG1bQfl3BchsKsLes2m01lMLDT\n\tGQBBeV3IlY75pa44syoOOYF3L9UZH1XLprKDAC+UQ6fVNmgiHvGVaa6WYS5qCbMxYl\n\t7wjrGb1VfG7WLlUkRhDRmXgmHbFsdQbnMM0SiL4SwK7YcMdYjf7IZdcojYJdMOSEHZ\n\tKOYVl18pcFYEcXCkJ6uvZDc3AdRmVZ2w6VuIhjCD/5D/aHUEjqvucRcBlK+4WdjDOu\n\tA4mrGIzeKXRnw==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1686610042;\n\tbh=EKKN1kSquDXDKGzr48/iKveCivDb6CTbX3zXSfmPrEo=;\n\th=From:To:Subject:Date:From;\n\tb=nRWachTaJje8Ei/IOB5e0egAXkrEhh0KqHuHyaLiiMMBWtyJQ8I55/Umw9S6Qy2Gw\n\tcHyIkFPCxYxQaMZ0w3DPaSet8eBoSHKv6XulC2nGVMBla9ZAmD6HkipXEM/LTFn+xk\n\tSimQsxIaFe4G9+WDoTn/LR/vb+JBpXG2SiURjVCA="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"nRWachTa\"; dkim-atps=neutral","To":"libcamera-devel@lists.libcamera.org","Date":"Tue, 13 Jun 2023 01:47:47 +0300","Message-Id":"<20230612224751.4437-1-laurent.pinchart@ideasonboard.com>","X-Mailer":"git-send-email 2.39.3","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v1 0/4] utils: checkstyle: Add a commit\n\tmessage trailers checker","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","From":"Laurent Pinchart via libcamera-devel\n\t","Reply-To":"Laurent Pinchart ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"Hello,\n\nThis patch series originates from a discussion in the review of \"[PATCH\nv1 3/3] ipa: rpi: agc: Gracefully handle missing agc modes\" related to\nusage of git commit message trailers. It led me to realize that quite a\nfew issues went unnoticed through review (about 1% of all commits in the\nrepository). This shows, in my opinion, that humans are not great at\ncatching those issues. The option of fixing humans being deemed too\ndifficult (not to mention ethical problems), so I went for automation\ninstead.\n\nPatches 1/4 to 3/4 are small improvements and cleanups to checkstyle.py,\nwith the main change in patch 4/4. Please see individual patches for\ndetails. There is, in particular, information in 4/4 that may require\nsome discussion.\n\nLaurent Pinchart (4):\n utils: checkstyle: Add __repr__ method to CommitFile class\n utils: checkstyle: Support running checkers selectively\n utils: checkstyle: Don't include commit ID in commit title\n utils: checkstyle: Add trailers checker\n\n utils/checkstyle.py | 127 +++++++++++++++++++++++++++++++++++++-------\n 1 file changed, 109 insertions(+), 18 deletions(-)\n\n\nbase-commit: 0ee9339331c648232e87d2de2ccd5a92cc61cab2"}