On Wed, Dec 05, 2007 at 11:45:23AM -0800, Junio C Hamano wrote: > I actually regret to have suggested "git less". Not only because you > can always say "git show" instead, but because the error message you > would get with usage string will _not_ say "git-less", but some other > command's name if you say "git less nonsense". > > I on the other hand find the "view" alias moderately less problematic. > As long as the future direction for the "view" alias is to allow it to > notice user preference and launch something other than the default > "gitk", iow, it is crystal clear that "git view" is just a short-hand > for launching a history browser and the users are free to choose > whichever viewer available, it won't feel inconsistent if underlying > "gitk" barfed on malformed input using its own name. The pattern I see here is that we get into trouble when we _pretend_ that builtin aliases are real commands, and not just handy shortcuts for the real commands. IOW, if a user is told that "git less" is the command to look at objects, then they will: 1. get confused when "git less" claims to be "git cat-file" or "git show" in error messages 2. get confused when there is no "git less" manpage 3. get confused when their coworker's "git less" behaves completely differently OTOH, if a user is told that "git less" is an alias for the user's preferred method for viewing objects, that the default is "git show", and that they can customize it themselves using alias.less, then I don't think any of the above will be surprising. So I think it is a bad idea to use such aliases to satisfy user requests for simple commands, even when they can obviously be implemented as such an alias. That being said... > By extension to this reasoning, I am not too keen on adding "update", > "up", "checkin", "ci", nor "co". I do not think of any alternative I think "checkin", "ci", and "co" are well-understood as aliases (and will be doubly so if they are presented in the documentation as such). After all, they come from CVS, which treats them this way: $ cvs co cvs checkout: No CVSROOT specified! Please use the `-d' option ^^^^^^^^ -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