At Tue, 23 Jan 2007 00:42:31 +0100,
Richard Knutsson wrote:
>
> Michal Piotrowski wrote:
> > How about this script?
> >
> > "d) Ensure that your patch does not add new trailing whitespace. The
> > below
> > script will fix up your patch by stripping off such whitespace.
> >
> > #!/bin/sh
> >
> > strip1()
> > {
> > TMP=$(mktemp /tmp/XXXXXX)
> > cp "$1" "$TMP"
> > sed -e '/^+/s/[ ]*$//' <"$TMP" >"$1"
> > rm "$TMP"
> > }
> >
> > for i in "$@"
> > do
> > strip1 "$i"
> > done
> > "
> > http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt
> I believe:
>
> for i in "$@"; do \
> sed --in-place -e "s/[ ]+$//" "$i"
> done
>
> will do as well...
Hi Richard,
IIRC, `+' is extended regex, so -r option is needed:
sed -r --in-place -e "s/[ ]+$//" "$i"
Satoru Takeuchi
>
> Richard Knutsson
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [EMAIL PROTECTED]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
