On Thu, Oct 01, 2009 at 10:16:48PM +0200, Clemens Buchacher wrote: > Instead of > > $ git commit -a -ammend > [work ce38944] mend > 1 files changed, 2 insertions(+), 0 deletions(-) > > we now get > > $ git commit -a -ammend > error: switch `m' must not be mangled with other options > usage: git commit [options] [--] <filepattern>... > [...] > > Signed-off-by: Clemens Buchacher <drizzd@xxxxxx> > --- > On Fri, Sep 25, 2009 at 04:32:26PM -0700, Shawn O. Pearce wrote: > > I wonder, should the -m flag on commit not allow cuddling its > > value against the switch when its combined in short form with > > other switches? > > Here we go. I thought the proposal was to disallow just cuddling of the value when the switch was combined with others. So you would disallow "git commit -ammend" but it would still be legal to do "git commit -am foo". Your patch disallows the latter. I would prefer to allow the uncuddled form, as it is something I do occasionally (and I don't think "git commit -am foo" looks very much like a typo). To be honest, I am not sure that even the more restricted proposal is that good an idea. You are introducing a heuristic to guess at what is a typo or error from the user; when your guess is wrong, the user will be annoyed (doubly so if it is buried deep in a script, which this change will also impact). So you are guessing that people don't use the cuddled form in this way. I suspect most don't. But I also wonder how many typos you are really helping to save. This was brought up to make "-ammend" DWYM. Is that really that common a double-typo? On the other hand, the cuddled value already has some DWYM magic (it recognizes -amend), so it is already a little bit unsafe to use (interestingly, though, the gitcli(7) documentation actually recommends using the "sticked" form over the separated one. However, it also recommends splitting your short options). So I don't feel _too_ strongly. I am just concerned that we are introducing some DWYM magic that is not really going to help many people out, and is just going to make understanding the rules for option parsing more complex, and introduce the possibility (albeit slim) of breaking people's scripts and trained fingers. -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