46/5ba11c29031105b1766325a3da624f1208abb8

   1 Return-Path: <amdragon@mit.edu>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 4461C429E3B\r
   6         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 09:38:53 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id p+oOQDImEQt0 for <notmuch@notmuchmail.org>;\r
  16         Tue, 17 Jan 2012 09:38:52 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-4.mit.edu (DMZ-MAILSEC-SCANNER-4.MIT.EDU\r
  18         [18.9.25.15])\r
  19         by olra.theworths.org (Postfix) with ESMTP id AEC7A429E2E\r
  20         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 09:38:52 -0800 (PST)\r
  21 X-AuditID: 1209190f-b7f8a6d000000914-4c-4f15b22c9b3e\r
  22 Received: from mailhub-auth-4.mit.edu ( [18.7.62.39])\r
  23         by dmz-mailsec-scanner-4.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id D7.98.02324.C22B51F4; Tue, 17 Jan 2012 12:38:52 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-4.mit.edu (8.13.8/8.9.2) with ESMTP id q0HHcpF4004162; \r
  27         Tue, 17 Jan 2012 12:38:51 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q0HHcnZk018701\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Tue, 17 Jan 2012 12:38:50 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1RnCzW-0007va-DL; Tue, 17 Jan 2012 12:38:38 -0500\r
  37 Date: Tue, 17 Jan 2012 12:38:38 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Tomi Ollila <tomi.ollila@iki.fi>\r
  40 Subject: Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for\r
  41         emacs/*.el files\r
  42 Message-ID: <20120117173838.GN16740@mit.edu>\r
  43 References: <cun7h0v7ete.fsf@hotblack-desiato.hh.sledj.net>\r
  44         <1326704656-26102-1-git-send-email-tomi.ollila@iki.fi>\r
  45         <1326704656-26102-2-git-send-email-tomi.ollila@iki.fi>\r
  46         <20120117043200.GJ16740@mit.edu> <yf6obu261z4.fsf@taco2.nixu.fi>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=us-ascii\r
  49 Content-Disposition: inline\r
  50 In-Reply-To: <yf6obu261z4.fsf@taco2.nixu.fi>\r
  51 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  52 X-Brightmail-Tracker:\r
  53  H4sIAAAAAAAAA+NgFupnleLIzCtJLcpLzFFi42IRYrdT19XZJOpvsOU5q8X1mzOZLd6snMfq\r
  54         wORx+OtCFo9nq24xBzBFcdmkpOZklqUW6dslcGU8aF3NWnCRr+L3k5NsDYz3ubsYOTkkBEwk\r
  55         1t/fzQ5hi0lcuLeeDcQWEtjHKLF4M18XIxeQvYFR4svzXiYI5ySTxJRz91ggnCWMEtdmTgdr\r
  56         ZxFQlWj+eR3MZhPQkNi2fzkjiC0ioCLxoG09K4jNLCAt8e13MxOILSwQLdHV/4AZxOYV0JF4\r
  57         8fkyM8TQT4wSG3bsY4NICEqcnPmEBaJZS+LGv5dAzRxgg5b/4wAxOYF6dzxzBakQBVo15eQ2\r
  58         tgmMQrOQNM9C0jwLoXkBI/MqRtmU3Crd3MTMnOLUZN3i5MS8vNQiXRO93MwSvdSU0k2MoLDm\r
  59         lOTfwfjtoNIhRgEORiUe3pveIv5CrIllxZW5hxglOZiURHm514r6C/El5adUZiQWZ8QXleak\r
  60         Fh9ilOBgVhLhzU0DyvGmJFZWpRblw6SkOViUxHnVtN75CQmkJ5akZqemFqQWwWRlODiUJHhb\r
  61         NgI1ChalpqdWpGXmlCCkmTg4QYbzAA2vBKnhLS5IzC3OTIfIn2LU5Ti59so5RiGWvPy8VClx\r
  62         3iyQIgGQoozSPLg5sHT0ilEc6C1hXkeQKh5gKoOb9ApoCRPQkpxWIZAlJYkIKakGRkWGOVO1\r
  63         q06fjRJ6Wn9a7IJeSELf0lUxYtuOuuwqVv5YNHNm+aljsrdc9HU8jWadF080M7kQ6Pyg6+YU\r
  64         9runcrym+6w75pz6SmyqsmjgiQUpaj5/uu+yBPkwRrdKbotSEf5Ty/eJy+xNcP/+93+fHi25\r
  65         2M3Anq2WEZG6eAmbt8/qa3fehAdlK7EUZyQaajEXFScCAHcutMsiAwAA\r
  66 Cc: notmuch@notmuchmail.org\r
  67 X-BeenThere: notmuch@notmuchmail.org\r
  68 X-Mailman-Version: 2.1.13\r
  69 Precedence: list\r
  70 List-Id: "Use and development of the notmuch mail system."\r
  71         <notmuch.notmuchmail.org>\r
  72 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  73         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  74 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  75 List-Post: <mailto:notmuch@notmuchmail.org>\r
  76 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  77 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  78         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  79 X-List-Received-Date: Tue, 17 Jan 2012 17:38:53 -0000\r
  80 \r
  81 Quoth Tomi Ollila on Jan 17 at 12:46 pm:\r
  82 > On Mon, 16 Jan 2012 23:32:00 -0500, Austin Clements <amdragon@mit.edu> wrote:\r
  83 > > Cleanup is the type of pain that should only be suffered once, so I'd\r
  84 > > be much happier with this if there was an accompanying git hook that\r
  85 > > prevented more mis-formatted code from slipping in.\r
  86 > \r
  87 > We'd need a script to be called from .git/hooks/pre-commit to do\r
  88 > extra checking; developer needs first activate this pre-commit\r
  89 > and then add call to our checking routine. Imagine the amount\r
  90 > of false positives this hook starts to generate...\r
  91 \r
  92 It's unfortunate (but sensible) that git doesn't provide an automatic\r
  93 way to set up verify scripts like this, but that doesn't mean we can't\r
  94 make it easy.  Put a script in devel (or whatever it winds up being\r
  95 called) that can be sourced from the pre-commit hook and then mention\r
  96 this in HACKING.\r
  97 \r
  98 Why it would generate false positives, assuming the cleanup goes in at\r
  99 the same time the hook goes in (or the script is somehow clever enough\r
 100 to only check changed lines)?\r
 101 \r
 102 > ... but. developer can run 'git commit --no-verify' ...aargh no;\r
 103 > I guess if pre-commit hook fails, commit-msg hook is not run\r
 104 > and this is bypassed; maybe NO_FORMATCHECK_HOOK=1 git commit ...\r
 105 > is the answer.\r
 106 \r
 107 Maybe the script could generate a warning/prompt, rather than\r
 108 preventing the commit altogether?\r
 109 \r
 110 > But what we at least need is Guidelines document that states \r
 111 > these formatting issues clearly and precicely. Surely \r
 112 > self-respecting programmers understands to follow there (and soon \r
 113 > adjusts their workflow -- i.e. activate/run these checkers\r
 114 > after list response).\r
 115 \r
 116 I think the guidelines for elisp are pretty well known.  I agree that\r
 117 we need such a document for the C/C++ code.  I started writing one a\r
 118 while ago, but other things took over before I got very far.\r