On 25/02/12 08:50AM, Junio C Hamano wrote: > Justin Tobler <jltobler@xxxxxxxxx> writes: > > > +NOTES > > +---- > > + > > +`diff-pairs` should handle any input generated by `diff-tree --raw -z`. > > +It may choke or otherwise misbehave on output from `diff-files`, etc. > > + > > +Here's an incomplete list of things that `diff-pairs` could do, but > > +doesn't (mostly in the name of simplicity): > > + > > + - Only `-z` input is accepted, not normal `--raw` input. > > + > > + - Abbreviated sha1s are rejected in the input from `diff-tree`; if you > > + want to abbreviate the output, you can pass `--abbrev` to > > + `diff-pairs`. > > + > > + - Pathspecs are not handled by `diff-pairs`; you can limit the diff via > > + the initial `diff-tree` invocation. > > Which of the above limitations are fundamental, and which are merely > due to incomplete implementation that could be improved in the > future iterations? Thinking about this some more, I'm a bit unsure whether git-diff-pairs(1) should support "normal" `--raw` input. Furthermore, if we do want to support it, maybe it should be the default?