On Thu, Jan 14, 2021 at 04:50:13PM +0100, Christian Couder wrote: > This is a small patch series to refactor the code that actually writes > a promisor file (<pack-name>.promisor) into a write_promisor_file() > function, and then to improve it a bit. > > Compared to version 1 of this patch series, the only difference is > that patch 3/3 has been added to die() in case of error, instead of > ignoring it, when actually writing the content of the file or closing > it. Thanks to Peff and Taylor for their suggestions. These look great to me. Thanks for adding the extra error handling. -Peff