Jeff King <peff@xxxxxxxx> writes: > Really, both here and remove_redundant_pack() should probably be using > unlink_or_warn(). Though if you did want to notice the race on deleting > the actual .pack file, it would need to be hand-coded. Yeah, unlink_or_warn() would probably be the right way to go. Other than that, the patch looks good to you folks? Thanks.