On Wed, Jun 5, 2013 at 4:12 AM, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: >> +Caveats: >> + >> +1. Many of the placeholders in "PRETTY FORMATS" are designed to work >> + specifically on commit objects: when non-commit objects are >> + supplied, those placeholders won't work. > > Should "won't work" be expanded upon? It's not clear if this means > that git will outright crash, or if it will abort with an appropriate > error message, or if the directive will be displayed as-is or removed > from the output. It will be displayed as-is but that's a bit inconsistent: %(unknown) prints error and aborts while %unknown simply produces %unknown. The latter is how "git log --format" does it. But I think we could make for-each-ref --pretty to do the former for %unknown. It'll be consistent with %(unknown) and we do not need to elaborate much (it's pretty obvious when it does not work). -- Duy -- 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