lkml.org 
[lkml]   [2007]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Subjecttrailing whitespace killing (Re: [PATCH -mm] Blackfin: blackfin i2c driver)
    Date
    From
    > From: Andrew Morton
    > Newsgroups: gmane.linux.kernel
    > Subject: Re: [PATCH -mm] Blackfin: blackfin i2c driver
    > Date: Tue, 6 Mar 2007 23:45:29 -0800
    []
    > On Wed, 07 Mar 2007 15:39:27 +0800 "Wu, Bryan" <bryan.wu@analog.com> wrote:
    >
    >> Thanks a lot, could you please give me a script just to kill this
    >> whitespace? So I can do it before sending you patches.
    >
    >
    > Is pretty simple:
    >
    > #!/bin/sh
    > #
    > # Strip any trailing whitespace which a unified diff adds.
    > #
    >
    > strip1()
    > {
    > TMP=$(mktemp /tmp/XXXXXX)
    > cp $1 $TMP
    > sed -e '/^+/s/[ ]*$//' < $TMP > $1
    > rm $TMP
    > }
    >
    > for i in $*
    > do
    > strip1 $i
    > done
    >
    >
    > that'll be in
    > http://www.zip.com.au/~akpm/linux/patches/patch-scripts-0.20/patch-scripts-0.20.tar.gz
    > too

    It doesn't work for me. Maybe i can't understand what you are trying to
    do, anyway.

    General suggestion is can be:

    sed -e 's_[ \t]*$__'

    (i.e any line on stdin with space/tab mixed tails is stripped on stdout)

    You can use it as wrapper for diff, sending patch bombs, etc.
    (very nice with pipes):

    shell$ diff -Npu2 old new | sed -e 's_[ \t]*$__' > patch.diff
    shell$ < patch-set.mbox sed -e 's_[ \t]*$__' | formail -s /usr/sbin/sendmail -bm -t

    similar in scripts; quilt (patch sets manager) notices about them.
    ____
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-03-08 04:51    [W:0.019 / U:64.256 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site