Cyrill Gorcunov <gorcunov@xxxxxxxxx> writes: > Here is rfc in a sake of having ability to produce > diff body in small series of changes in request-pull. > we could use ">>" here and append diff by hands but > adding "-p" option seems more comfortable. > > Dunno, just a thought. I think this is a useful thing to add, as I think I've seen some request-pull actually made with the patch text, most likely by hand. > git shortlog ^$baserev $headrev > git diff -M --stat --summary $merge_base $headrev > + > +if [ "$usediff" = "yes" ]; then > + echo "---" > + git diff -M -p $merge_base $headrev > +fi You can just use a single "git diff" invocation here, git diff -M --stat --summary $usediff $merge_base $headrev and set up "$usediff" to be either empty or -p in the option parsing part. -- 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