On Thu, Sep 02, 2010 at 10:41:56AM +0200, Michael J Gruber wrote: > > i.e. I think that is the same issue after all and that > > builtin/help.c ought to learn another heuristic than is_git_command(). > > One heuristic, which I would have left for a later patch because of its > radicality (and I think we're in some phase of some rc something), is to > simply not do any checks when calling the viewers. This requires that > everything is prepended with "git-", which I see you have done in > builtin/help.c. Still, none-command help pages will not show up with > "git help -a". So it's not a complete solution. That was my first thought upon reading your patch, too. And something like "s/gitworkflows/git-workflows/" is reasonable. But you can do "git help attributes" and "git help ignore" now, and those should almost certainly remain "gitattributes" and "gitignore" (at the very least, we should still allow "git help gitattributes", which works now, but which your proposal would break). So maybe we could get by with "prepend git- unless it starts with git". I dunno. The current behavior seems hack-ish, but I am loathe to change names if people really prefer the existing ones. FWIW, I would actually say "git help git-sh-setup", which in fact works (leading me to get puzzled by your patch at first). So who knows what people are typing in practice. -Peff -- 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