On Fri, Jul 31, 2009 at 7:17 PM, Junio C Hamano<gitster@xxxxxxxxx> wrote: > Giuseppe Bilotta <giuseppe.bilotta@xxxxxxxxx> writes: > >> Before I go on with the next revision of the patch, I would like to >> have some kind of agreed convention to implement. >> >> My suggestion would be the following: >> (1) implement options --ignore-space-change, --ignore-all-space >> mirroring their 'git diff' meaning. >> (2) add --ignore-whitespace as a synonym to --ignore-space-change, for >> consistency with 'patch' >> (3) apply.ignore-whitespace accepts values >> * false,no,none,0 to mean no whitespace ignoring >> * true,yes,change,1 to mean ignore whitespace change >> * all,2 to mean ignore all whitespace > > I'd suggest taking one small bite at a time. Sure. > A later follow-up patch would complete your (1) and (2) by supporting > ignore-all-space. And perhaps you would give it --ignore-all-whitespace > synonym perhaps? You may want to ask "GNU patch" people if they are > interested in ignoring all whitespaces, and if so what their plan is to > name that option, so that you can use the same name. I was thinking about making --ignore-whitespace (I'd rather keep the -- because that's what patch goes for) accept an optional argument, allowing e.g. --ignore-whitespace=no to overrule the config option. In this case we would have --ignore-whitespace=all as synonym to -ignore-all-space. It would also make the --ignore-whitespace option more in line with the ignorewhitespace config -- Giuseppe "Oblomov" Bilotta -- 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