Michael Haggerty <mhagger@xxxxxxxxxxxx> writes: > On 10/19/2011 08:19 AM, Junio C Hamano wrote: > >> A possible alternative might be to leave check_refname_format() and its >> callers as they are, introduce check_full_refname() function that knows >> the new restriction on top of check-ref-format-fixup, and use that in >> lock_ref_sha1(), lock_any_ref_for_update() and is_refname_available() >> [*2*]. That way, we can keep the potentially useful "ill-formed contents >> in the ref" warning and avoid possible confusion caused by random files >> that are directly under $GIT_DIR, which would be far more preferable in >> the longer term. >> >> Anybody wants to give it a try? > > I think that the refs/-or-ALL_CAPS test fits most naturally in > check_refname_format(), controlled by an option flag. That is fine too, if a separate function check_full_refname() would end up being nothing more than a thin wrapper that passes the "we are checking a full refname" option. > I'm starting by building parts of the solution, namely something like: > ... > > Hopefully I'll have patches before the end of the (Berlin) day. It is unclear how these two functions could be related to the issue, but hopefully we will soon find out when we see your patch. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html