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 <ard.biesheu...@linaro.org> 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 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
