From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=66.187.233.73; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 65011224C0F43 for ; Mon, 12 Mar 2018 04:59:45 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 8D03A401C9AE; Mon, 12 Mar 2018 12:06:05 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-97.rdu2.redhat.com [10.10.120.97]) by smtp.corp.redhat.com (Postfix) with ESMTP id B3CEC202699A; Mon, 12 Mar 2018 12:06:03 +0000 (UTC) To: Ard Biesheuvel Cc: edk2-devel-01 , Anthony Perard , Brijesh Singh , Jordan Justen , Julien Grall , Phil Dennis-Jordan References: <20180311014926.3049-1-lersek@redhat.com> <20157fd6-a776-fe10-6492-55e85ec03b3f@redhat.com> From: Laszlo Ersek Message-ID: Date: Mon, 12 Mar 2018 13:06:02 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.78 on 10.11.54.4 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.5]); Mon, 12 Mar 2018 12:06:05 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.5]); Mon, 12 Mar 2018 12:06:05 +0000 (UTC) for IP:'10.11.54.4' DOMAIN:'int-mx04.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'lersek@redhat.com' RCPT:'' Subject: Re: [PATCH 00/45] ArmVirtPkg, OvmfPkg: list module-internal headers in INF files X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 12 Mar 2018 11:59:46 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 03/11/18 12:54, Ard Biesheuvel wrote: > I am merely saying that it is not always necessary to share your > personal journey resulting in the patches at this level of detail, > simply because it doesn't scale. Doesn't scale for me, or doesn't scale for reviewers? If the latter, do you suggest that I keep such detailed notes out of the v1 posting as well? (Because, I imagine, if I edit them down for v2 only, then I may have wasted reviewer time already.) The recurrent bottleneck for me is trying to figure out what this or that part of the patch was meant to solve, and why that way. I've also encouraged contributors to capture their exact scenario / use case in commit messages; the more specific the better. (IIRC, one example is commit f5404a3eba1d, "OvmfPkg: Increase the maximum size for Authenticated variables", 2016-03-25.) IOW, I tend to find the focus too wide, and the information lacking. However, if I end up wasting your time instead of saving it, then I'm doing it wrong. I wrote up the commit messages the way I did because I thought it would save time for me, if I had to review the patches (I tend to verify patches maybe a bit too pedantically too, and I appreciate when the commit messages give me crutches for that). If it has the opposite effect on you, then I'm doing it wrong. > In any case, I am happy with this to go in as is, if you prefer. > > Reviewed-by: Ard Biesheuvel Thank you -- peeking ahead at Jordan's review as well, I think I'll save you guys another round of this. I'm honestly confused now about how I should word my future commit messages. Therefore I can't simply promise "I'll keep them short"; I might not know *how* (i.e. what to leave out). I'll need to actively work on that. Thanks Laszlo