[rancid] Re: [patch] Using diffstat in the output mail

[Christian Hammers]

Hello

On Tue, May 23, 2006 at 12:13:30PM -0700, john heasley wrote:
> :0:maildir.lock
> * Subject: .* diffs
> * !^X-MAILLOOP: diffstat
> | (cd $MAILDIR; tee copy | sed -e '1,/^$/d' > body; sed -e '1,/^$/p' -e '/^$/q' copy > headers; cat headers; diffstat body; echo ; cat body; rm -f copy headers body; ) | formail -A "X-MAILLOOP: diffstat" -s procmail

*shudder* Ok, ok, I acknowledge that it really *is* possible to do with
procmail, but you won't propose me to fix it this way, or? :)

As I still think that the additional diffstat output would be beneficial
to/liked by the majority of the rancid users, I would still prefer to
see my one-line patch included... maybe with a "if $WANT_DIFFSTAT then"
surrounded to make it configurable. I would even send you patches for
this.

> > > > --- old/bin/control_rancid.in   2006-03-15 10:05:33.000000000 +0100
> > > > +++ new/bin/control_rancid.in   2006-03-16 09:33:44.964493641 +0100
> > > > @@ -392,6 +392,8 @@
> > > >  Subject: $subject
> > > >  Precedence: bulk
> > > >  
> > > > +`diffstat $TMP.diff`
> > > > +
> > > >  `cat $TMP.diff`
> > > >  EMAIL
> > > >  fi

bye,

-christian-

-- 
Christian Hammers             WESTEND GmbH  |  Internet-Business-Provider
Technik                       CISCO Systems Partner - Authorized Reseller
                              Lütticher Straße 10      Tel 0241/701333-11
ch at westend.com                D-52064 Aachen              Fax 0241/911879