Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> writes: > Read HEAD from disk instead of relying on local variable > our_head_points_at, so that if earlier code fails to make HEAD > properly, it'll be detected. The end result might be more or less the same with your patch from the end-user's point of view, but "if earlier code fails", shouldn't you detect and diagnose it right there? If you observe lack of "HEAD" in checkout(), you cannot tell if that was because the remote did not have anything usable in the first place, or because we knew where it should point at (and may have even attempted to create it) but somehow failed to make it point at it. -- 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