Re: [PATCH v2] hooks: add sendemail-validate-series

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Phillip Wood, Apr 11, 2023 at 11:58:
> A hook that wants to check some property of the whole series needs to 
> know which patch is the final one. We could pass that via the 
> environment as we do for external diff commands with 
> GIT_DIFF_PATH_COUNTER and GIT_DIFF_PATH_TOTAL.

That may be an appropriate solution and it would avoid adding another
hook. And it would solve the issue of "\n" in filenames.

The only downside is that you would need to store state in an external
file (maybe in GIT_DIR) so that successive calls of the hook script can
pick up where the previous invocation ended.

It all comes down to ergonomics at this point. I don't mind either
solutions as long as validating whole series is possible before sending
emails.




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux