On 07/08/18 17:46, Michael Orlitzky wrote: > Changes in v3: > > * Undo the setguid exception from v2, and add a comment explaining why. > * Add line breaks for readability in two comments. > * Try to put back the leading "/" in the output list. > * Remove a superfluous comment mentioning the "prefix." > > Michael Orlitzky (2): > bin/install-qa-check.d: add new 90bad-bin-owner QA check. > bin/install-qa-check.d: add new 90bad-bin-group-write QA check. > > bin/install-qa-check.d/90bad-bin-group-write | 55 > ++++++++++++++++++++++++++++ > bin/install-qa-check.d/90bad-bin-owner | 48 ++++++++++++++++++++++++ > 2 files changed, 103 insertions(+) > create mode 100644 bin/install-qa-check.d/90bad-bin-group-write > create mode 100644 bin/install-qa-check.d/90bad-bin-owner > May I just briefly complement you on your patch mail format ..
It's much easier to see, when you make updates to a patchset, that you have added a summary in your 0/x mail for what changed from the previous iteration. I don't think git has an easy way to both describe the patch diffs from the original *and* the changes from last iteration to current - that really would be the cherry on the icing on the cake! (something perhaps the avid perl users may be able to graft together!). Keep up the good work :) Best regards, Michael.
signature.asc
Description: OpenPGP digital signature