On Thu, Feb 21, 2019 at 06:52:05PM +0700, Duy Nguyen wrote: > > the postfix internally, but why can't it be okay with "name.lock.lock") > > Uh oh I miss this case. I only delete ".lock" once, "name.lock" would > still be rejected. Thanks for noticing. Another tricky case is "refs/heads/foo.lock/bar.lock", which would need both ".lock"s removed. I think your v2 handles this correctly, though (because it disallows "." entirely). -Peff