Jeff King <peff@xxxxxxxx> writes: > + - scanf's matching of "%s" is greedy. So the "refs/remotes/%s/HEAD" > + rule would never pull "origin" out of "refs/remotes/origin/HEAD". > + Instead it always produced "origin/HEAD", which is redundant with > + the "refs/remotes/%s" rule. Wow, that thing is new. Fixing it may break expectations by existing script by end-users, but it may probably be OK. Everything looks good.