On Sat, Dec 13, 2008 at 10:14:22AM +0800, Wu Fengguang wrote: > Introduce option guilt.diffstat so that we don't have to type > "guilt refresh --diffstat" in its full form every time. Good idea. > Signed-off-by: Wu Fengguang <fengguang.wu@xxxxxxxxx> > --- > guilt | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > --- guilt.orig 2008-12-13 09:53:32.000000000 +0800 > +++ guilt 2008-12-13 10:01:03.000000000 +0800 > @@ -538,7 +538,7 @@ __refresh_patch() > > [ ! -z "$4" ] && diffopts="-C -M --find-copies-harder" > > - if [ ! -z "$5" ]; then > + if [ -n "$5" -o "x$diffstat" = "x1" ]; then Why the 'x' thing? I've seen it is some scripts before, but I can't think of a reason to use it if the variable is surrounded in quotation marks. > ( > echo "---" > git diff --stat $diffopts "$2" > @@ -627,6 +627,9 @@ guilt_push_diff_context=1 > # default autotag value > AUTOTAG_DEFAULT=1 > > +# default diffstat value > +DIFFSTAT_DEFAULT=0 > + > # > # Parse any part of .git/config that belongs to us > # > @@ -635,6 +638,10 @@ AUTOTAG_DEFAULT=1 > autotag=`git config guilt.autotag` > [ -z "$autotag" ] && autotag=$AUTOTAG_DEFAULT > > +# generate diffstat? > +diffstat=`git config guilt.diffstat` > +[ -z "$diffstat" ] && diffstat=$DIFFSTAT_DEFAULT > + > # > # The following gets run every time this file is source'd > # Could you throw a quick note into the manpages? Thanks, Josef 'Jeff' Sipek. -- My public GPG key can be found at http://www.josefsipek.net/gpg/public-0xC7958FFE.txt -- 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
